Skip to content

fix(cursor-runner): correct stale delegate invocation in agent doc#11

Merged
freema merged 2 commits into
freema:mainfrom
TiberiusNemesis:20260610-FixCursorRunnerInvocation
Jun 10, 2026
Merged

fix(cursor-runner): correct stale delegate invocation in agent doc#11
freema merged 2 commits into
freema:mainfrom
TiberiusNemesis:20260610-FixCursorRunnerInvocation

Conversation

@TiberiusNemesis

@TiberiusNemesis TiberiusNemesis commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

Summary

The cursor-runner subagent's step 6 instructed it to invoke delegation via node_modules/.bin/tsx …/plugins/cursor/scripts/delegate.ts — stale from before the zero-deps .mjs rewrite: tsx is not a dependency, there is no .ts file, and the path double-counted plugins/cursor. So any delegation routed through the subagent (as opposed to the /cursor:delegate slash command, which is correct) failed if followed literally.

Replaced it with the invocation the slash command already uses — node "${CLAUDE_PLUGIN_ROOT}/scripts/delegate.mjs" -- … — and fixed the matching delegate.ts mention in the Output format section. Docs-only. Closes #10..

Test plan

  • npm test — all specs green
  • npm run lint — prettier + eslint clean (agent doc is under plugins/cursor)
  • No code paths changed; verified the corrected command matches commands/delegate.md

Checklist

  • No new runtime dependencies
  • No build step, no new dist/ or .ts files
  • User-visible behaviour change noted under ## Unreleased in CHANGELOG.md

Step 6 of the cursor-runner agent told the subagent to run
`node_modules/.bin/tsx …/plugins/cursor/scripts/delegate.ts`, left over
from before the zero-deps `.mjs` rewrite: `tsx` is not a dependency,
there is no `.ts` file, and the path double-counted `plugins/cursor`
(CLAUDE_PLUGIN_ROOT already points at the plugin dir).

Replace it with the invocation the `/cursor:delegate` slash command
already uses: `node "${CLAUDE_PLUGIN_ROOT}/scripts/delegate.mjs" -- …`,
and fix the matching `delegate.ts` mention in the output-format section.
@TiberiusNemesis

Copy link
Copy Markdown
Contributor Author

forgot to link the issue, my bad.
this fixes ➔ #10

@freema freema left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Confirmed: agents/cursor-runner.md step 6 still pointed at node_modules/.bin/tsx …/plugins/cursor/scripts/delegate.ts — stale since the zero-deps .mjs rewrite (#3). The corrected invocation matches commands/delegate.md exactly, and the delegate.ts mention in Output format is fixed too.

Ran the branch locally: 83/83 tests green, prettier + eslint clean.

Note: #9 will merge first and both PRs add an ## Unreleased section to CHANGELOG.md, so this will need a trivial rebase afterwards — the cursor-runner.md hunks themselves merge cleanly.

@freema freema merged commit 4f95b06 into freema:main Jun 10, 2026
6 checks passed
@TiberiusNemesis TiberiusNemesis deleted the 20260610-FixCursorRunnerInvocation branch June 10, 2026 22:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

cursor-runner agent's delegate invocation points at non-existent tsx/delegate.ts

2 participants