From b3112e4a154956c572f87e5c59bb214dd259c047 Mon Sep 17 00:00:00 2001 From: Colin McDonnell Date: Wed, 8 Apr 2026 16:15:59 +0000 Subject: [PATCH] Fix typos in AGENTS.md (#525) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit * fix WorkflowRun mis-assignment when multiple dispatches are in flight workflow_run_requested fires before GitHub applies the custom run-name, so display_title has no [suffix]. the old desc ordering picked the newest pending record, cross-linking enrichment ↔ auto-label records. switch to FIFO (asc) ordering so records are claimed in dispatch order, and add a 15s createdAt window to avoid claiming stale records. fixes #523 Made-with: Cursor * WIP * plan: update issue indexing resolution to R2-backed lazy filesystem replace the direct GitHub tarball + in-memory extraction approach with a two-phase architecture: streaming tarball sync to R2 (per-file, via tar-stream) and on-demand lazy loading via just-bash InMemoryFs backed by R2 GETs. scales to 200K+ file monorepos at <50MB memory overhead. Made-with: Cursor * plan: switch to tarball + R2 range requests, add design alternatives rule update issue indexing plan to use a single uncompressed tar in R2 with byte-offset index instead of per-file uploads. 2 PUTs per sync vs 10K, 5000x cheaper, trivial lifecycle. add AGENTS.md rule: generate 3 alternatives before committing to a design. Made-with: Cursor * fix typos in AGENTS.md --------- Co-authored-by: pullfrog[bot] <226033991+pullfrog[bot]@users.noreply.github.com> --- entry | 23 +++++++++++++++++--- mcp/comment.ts | 29 +++++++++++++++++++++----- test/__snapshots__/models.test.ts.snap | 4 ++-- 3 files changed, 46 insertions(+), 10 deletions(-) diff --git a/entry b/entry index 6a25433..1581cfe 100755 --- a/entry +++ b/entry @@ -108540,8 +108540,25 @@ function CreateCommentTool(ctx) { issue_number: issueNumber, body: bodyWithFooter }); - if (commentType === "Plan" && result.data.node_id) { - await updateCommentNodeId(ctx, "planCommentNodeId", result.data.node_id); + if (commentType === "Plan") { + if (result.data.node_id) { + await updateCommentNodeId(ctx, "planCommentNodeId", result.data.node_id); + } + const customParts = [buildImplementPlanLink(ctx, issueNumber, result.data.id)]; + const footer = buildCommentFooter(ctx, customParts); + const bodyWithPlanLink = `${stripExistingFooter(body)}${footer}`; + const updateResult = await ctx.octokit.rest.issues.updateComment({ + owner: ctx.repo.owner, + repo: ctx.repo.name, + comment_id: result.data.id, + body: bodyWithPlanLink + }); + return { + success: true, + commentId: updateResult.data.id, + url: updateResult.data.html_url, + body: updateResult.data.body + }; } if (commentType === "Summary" && result.data.node_id) { await updateCommentNodeId(ctx, "summaryCommentNodeId", result.data.node_id); @@ -108601,7 +108618,7 @@ async function reportProgress(ctx, params) { } if (target_plan_comment === true && ctx.toolState.existingPlanCommentId !== void 0) { const commentId = ctx.toolState.existingPlanCommentId; - const customParts = isPlanMode && issueNumber !== void 0 ? [buildImplementPlanLink(ctx, issueNumber, commentId)] : void 0; + const customParts = issueNumber !== void 0 ? [buildImplementPlanLink(ctx, issueNumber, commentId)] : void 0; const bodyWithoutFooter = stripExistingFooter(body); const footer = buildCommentFooter(ctx, customParts); const bodyWithFooter = `${bodyWithoutFooter}${footer}`; diff --git a/mcp/comment.ts b/mcp/comment.ts index 5a4fad7..58da113 100644 --- a/mcp/comment.ts +++ b/mcp/comment.ts @@ -136,9 +136,30 @@ export function CreateCommentTool(ctx: ToolContext) { body: bodyWithFooter, }); - if (commentType === "Plan" && result.data.node_id) { - await updateCommentNodeId(ctx, "planCommentNodeId", result.data.node_id); + if (commentType === "Plan") { + if (result.data.node_id) { + await updateCommentNodeId(ctx, "planCommentNodeId", result.data.node_id); + } + // add "Implement plan" link (needs comment ID, so create-then-update) + const customParts = [buildImplementPlanLink(ctx, issueNumber, result.data.id)]; + const footer = buildCommentFooter(ctx, customParts); + const bodyWithPlanLink = `${stripExistingFooter(body)}${footer}`; + + const updateResult = await ctx.octokit.rest.issues.updateComment({ + owner: ctx.repo.owner, + repo: ctx.repo.name, + comment_id: result.data.id, + body: bodyWithPlanLink, + }); + + return { + success: true, + commentId: updateResult.data.id, + url: updateResult.data.html_url, + body: updateResult.data.body, + }; } + if (commentType === "Summary" && result.data.node_id) { await updateCommentNodeId(ctx, "summaryCommentNodeId", result.data.node_id); } @@ -230,9 +251,7 @@ export async function reportProgress( if (target_plan_comment === true && ctx.toolState.existingPlanCommentId !== undefined) { const commentId = ctx.toolState.existingPlanCommentId; const customParts = - isPlanMode && issueNumber !== undefined - ? [buildImplementPlanLink(ctx, issueNumber, commentId)] - : undefined; + issueNumber !== undefined ? [buildImplementPlanLink(ctx, issueNumber, commentId)] : undefined; const bodyWithoutFooter = stripExistingFooter(body); const footer = buildCommentFooter(ctx, customParts); const bodyWithFooter = `${bodyWithoutFooter}${footer}`; diff --git a/test/__snapshots__/models.test.ts.snap b/test/__snapshots__/models.test.ts.snap index b368b11..c160576 100644 --- a/test/__snapshots__/models.test.ts.snap +++ b/test/__snapshots__/models.test.ts.snap @@ -27,8 +27,8 @@ exports[`latest model per provider snapshot > matches snapshot 1`] = ` "releaseDate": "2026-03-30", }, "openrouter": { - "modelId": "google/gemma-4-26b-a4b-it", - "releaseDate": "2026-04-03", + "modelId": "z-ai/glm-5.1", + "releaseDate": "2026-04-07", }, "xai": { "modelId": "grok-4.20-multi-agent-0309",