@oh-my-pi/pi-coding-agent 14.5.8 → 14.5.10

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
Files changed (58) hide show
  1. package/CHANGELOG.md +56 -0
  2. package/package.json +7 -15
  3. package/scripts/build-binary.ts +1 -1
  4. package/src/cli/update-cli.ts +25 -1
  5. package/src/config/model-registry.ts +21 -19
  6. package/src/config/settings-schema.ts +14 -19
  7. package/src/discovery/claude-plugins.ts +28 -3
  8. package/src/edit/modes/atom.lark +7 -5
  9. package/src/edit/modes/atom.ts +510 -73
  10. package/src/edit/modes/hashline.ts +172 -91
  11. package/src/extensibility/extensions/runner.ts +34 -1
  12. package/src/extensibility/extensions/types.ts +8 -0
  13. package/src/lsp/client.ts +27 -35
  14. package/src/lsp/index.ts +2 -4
  15. package/src/lsp/render.ts +0 -3
  16. package/src/lsp/types.ts +1 -4
  17. package/src/lsp/utils.ts +18 -14
  18. package/src/memories/index.ts +5 -0
  19. package/src/modes/components/settings-defs.ts +1 -1
  20. package/src/modes/controllers/command-controller.ts +17 -0
  21. package/src/modes/controllers/input-controller.ts +7 -1
  22. package/src/modes/controllers/selector-controller.ts +2 -2
  23. package/src/modes/interactive-mode.ts +57 -26
  24. package/src/modes/theme/theme.ts +10 -1
  25. package/src/modes/types.ts +5 -3
  26. package/src/modes/utils/context-usage.ts +294 -0
  27. package/src/modes/utils/ui-helpers.ts +19 -6
  28. package/src/prompts/system/auto-continue.md +1 -0
  29. package/src/prompts/tools/atom.md +99 -44
  30. package/src/prompts/tools/exit-plan-mode.md +5 -39
  31. package/src/prompts/tools/github.md +3 -3
  32. package/src/prompts/tools/lsp.md +2 -3
  33. package/src/prompts/tools/{run-command.md → recipe.md} +1 -1
  34. package/src/prompts/tools/task.md +34 -147
  35. package/src/prompts/tools/todo-write.md +22 -64
  36. package/src/sdk.ts +13 -2
  37. package/src/session/agent-session.ts +175 -79
  38. package/src/session/compaction/compaction.ts +35 -22
  39. package/src/session/session-dump-format.ts +1 -0
  40. package/src/session/session-manager.ts +19 -2
  41. package/src/slash-commands/builtin-registry.ts +12 -5
  42. package/src/tools/bash.ts +9 -4
  43. package/src/tools/debug.ts +57 -70
  44. package/src/tools/gh.ts +267 -119
  45. package/src/tools/index.ts +7 -7
  46. package/src/tools/{run-command → recipe}/index.ts +19 -19
  47. package/src/tools/recipe/render.ts +19 -0
  48. package/src/tools/{run-command → recipe}/runner.ts +28 -7
  49. package/src/tools/{run-command → recipe}/runners/pkg.ts +23 -53
  50. package/src/tools/renderers.ts +2 -2
  51. package/src/utils/git.ts +61 -2
  52. package/src/web/search/providers/searxng.ts +71 -13
  53. package/src/tools/run-command/render.ts +0 -18
  54. /package/src/tools/{run-command → recipe}/runners/cargo.ts +0 -0
  55. /package/src/tools/{run-command → recipe}/runners/index.ts +0 -0
  56. /package/src/tools/{run-command → recipe}/runners/just.ts +0 -0
  57. /package/src/tools/{run-command → recipe}/runners/make.ts +0 -0
  58. /package/src/tools/{run-command → recipe}/runners/task.ts +0 -0
@@ -414,7 +414,7 @@ export class UiHelpers {
414
414
  this.ctx.ui.requestRender();
415
415
  }
416
416
 
417
- renderInitialMessages(): void {
417
+ renderInitialMessages(prebuiltContext?: SessionContext): void {
418
418
  // This path is used to rebuild the visible chat transcript (e.g. after custom/debug UI).
419
419
  // Clear existing rendered chat first to avoid duplicating the full session in the container.
420
420
  this.ctx.chatContainer.clear();
@@ -422,8 +422,8 @@ export class UiHelpers {
422
422
  this.ctx.pendingBashComponents = [];
423
423
  this.ctx.pendingPythonComponents = [];
424
424
 
425
- // Get aligned messages and entries from session context
426
- const context = this.ctx.sessionManager.buildSessionContext();
425
+ // Reuse a pre-built context when available (e.g. from navigateTree) to avoid a second O(N) walk.
426
+ const context = prebuiltContext ?? this.ctx.sessionManager.buildSessionContext();
427
427
  this.ctx.renderSessionContext(context, {
428
428
  updateFooter: true,
429
429
  populateHistory: true,
@@ -610,9 +610,22 @@ export class UiHelpers {
610
610
  await this.ctx.session.prompt(message.text);
611
611
  }
612
612
 
613
- const promptPromise = this.ctx.session.prompt(firstPrompt.text).catch((error: unknown) => {
614
- restoreQueue(error);
615
- });
613
+ // Pass streamingBehavior so that if the session is still streaming when
614
+ // compaction-end fires (race window between isStreaming flipping false and
615
+ // the event landing here), prompt() routes the message into the steer/
616
+ // follow-up queue instead of throwing AgentBusyError. When the session is
617
+ // genuinely idle, streamingBehavior is ignored and a fresh prompt runs as
618
+ // before. This keeps the steer preview honest: if delivery has to be
619
+ // deferred, the message lands in the same queue every other consumer
620
+ // (Alt+Up dequeue, post-stream drain) already drains, instead of being
621
+ // stranded in compactionQueuedMessages with no drainer.
622
+ const promptPromise = this.ctx.session
623
+ .prompt(firstPrompt.text, {
624
+ streamingBehavior: firstPrompt.mode === "followUp" ? "followUp" : "steer",
625
+ })
626
+ .catch((error: unknown) => {
627
+ restoreQueue(error);
628
+ });
616
629
 
617
630
  for (const message of rest) {
618
631
  if (this.ctx.isKnownSlashCommand(message.text)) {
@@ -0,0 +1 @@
1
+ Resume work on the user's most recent intent. Re-read the kept recent messages above the summary to confirm what the user asked for last; if their latest request supersedes earlier plans recorded in the summary, follow the latest request. If there is nothing left to do, say so briefly instead of inventing further work.
@@ -1,26 +1,45 @@
1
- Your patch language is a compact, file-oriented edit format.
1
+ Your patch language is a compact, line-anchored edit format.
2
2
 
3
- When emitting a patch, the first non-blank line **MUST** be `---PATH`.
4
- A Lid is the anchor emitted in read/grep etc. (line number + id, e.g. `5th`).
3
+ A patch contains one or more file sections. The first non-blank line of every section **MUST** be `---PATH`.
4
+ A "Lid" is a per-line anchor emitted by `read`, `grep`, etc. `<lineNumber><2-letter-hash>`, e.g. `5th`, `123ab`. You **MUST** copy a Lid verbatim from the latest output for the file you're editing.
5
+
6
+ This format is purely textual. The tool has NO awareness of language, indentation, brackets, fences, or table widths. You are responsible for emitting valid syntax in your replacements/insertions.
5
7
 
6
8
  <ops>
7
- ---PATH start editing PATH with cursor at EOF
8
- !rm delete PATH
9
- !mv X move file to X
10
- $ move cursor to BOF
11
- ^ move cursor to EOF
12
- @Lid move cursor after Lid
13
- +X insert X at the cursor; `+` alone inserts a blank line
14
- Lid=X replace whole line with X; `Lid=` blanks it out
15
- -Lid delete line (repeat for multi)
9
+ ---PATH start a section editing PATH; cursor begins at EOF
10
+ ^ move cursor to BOF (before line 1)
11
+ $ move cursor to EOF (after the last line)
12
+ @Lid move cursor to AFTER the anchored line (does not modify the file)
13
+ ^Lid move cursor to BEFORE the anchored line (does not modify the file)
14
+ +TEXT insert one line containing TEXT at the cursor
15
+ + insert one blank line at the cursor
16
+ Lid=TEXT replace the anchored line with TEXT
17
+ LidA..LidB=TEXT replace the range with one line; following `\TEXT` lines append literal lines to the replacement
18
+ \TEXT append literal TEXT to the active replacement (after `Lid=…` or `LidA..LidB=…`)
19
+ \ append a blank line to the active replacement
20
+ Lid= blank the anchored line's content but KEEP the line (results in an empty line, NOT a removed line; use `-Lid` to remove)
21
+ -Lid delete the anchored line (repeat for multi-line delete)
22
+ -LidA..LidB delete the contiguous line range LidA..LidB (inclusive)
23
+ !rm delete the section's PATH (**MUST** be the only op in the section)
24
+ !mv DEST rename the section's PATH to DEST (**MUST** be the only op in the section)
16
25
  </ops>
17
26
 
18
27
  <rules>
19
- - You may have multiple `---PATH` sections to edit multiple files at once.
20
- - Ops starting with `$` / `^` / `@Lid` do not alter lines; you must still issue an op like `+` afterwards.
21
- - Consecutive `+X` ops insert consecutive lines.
22
- - `Lid=X` replaces the whole line. X must be the complete new line, not a fragment.
23
- - To rewrite multiple adjacent lines, delete each with `-Lid` then emit the new content as `+TEXT` lines. Do not stack `Lid=X` over a contiguous range it requires the new block to match the old line count and silently corrupts the file when they differ.
28
+ - Cursor-only ops (`^`, `$`, `@Lid`, `^Lid`) reposition without modifying. To insert anything you **MUST** follow them with `+TEXT` (or `+` for a blank).
29
+ - TEXT in `+TEXT`, `Lid=TEXT`, and `\TEXT` is literal line content, INCLUDING leading whitespace. You **MUST NOT** trim or re-indent it.
30
+ - Consecutive `+TEXT` ops produce consecutive lines in the order written. You **MUST NOT** separate them with a stray `+` unless you intend to insert a blank line.
31
+ - `Lid=TEXT` rewrites ONE line. To rewrite K adjacent lines, you **MUST** use `LidA..LidB=FIRST_LINE` followed immediately by `\NEXT_LINE` continuation lines (canonical form for any block replacement). You **MUST** use bare `\` for blank replacement lines.
32
+ - You **MUST** prefix every replacement continuation line with `\`, especially when the replacement line starts with edit syntax characters such as `#`, `+`, `-`, `@`, `$`, `^`, `!`, or a Lid-shaped token.
33
+ - `\TEXT` **MUST** appear only immediately after an active `Lid=…` or `LidA..LidB=…` replacement. It **MUST NOT** be used as a general insert operator.
34
+ - A `\TEXT` line **MUST** be the immediate continuation of a `Lid=…` or `LidA..LidB=…` op on the line above (or another `\` line rooted in one). If the line above is `+TEXT`, a bare Lid, a cursor op, or whitespace, the `\` is invalid and the tool will not interpret it as part of a replacement.
35
+ - The legacy `-LidA..LidB` + `+TEXT…` block-rewrite form also works.
36
+ - To insert ABOVE a line, you **MUST** use `^Lid` then `+TEXT`. To insert above line 1, you **MUST** use `^` (BOF) then `+TEXT`. To insert below a line, you **MUST** use `@Lid` then `+TEXT`.
37
+ - Multiple `---PATH` sections **MAY** appear in one input; each section is applied in order.
38
+ - `!rm` / `!mv DEST` **MUST NOT** be combined with line edits in the same section.
39
+ - Lids contain a content hash. If a line has changed since you read it, the tool rejects the edit and shows the current content; you **MUST** re-read and retry with fresh Lids. Small drift (≤5 lines) where the original hash still matches a nearby line auto-rebases with a warning. Larger shifts may show a hash-only candidate, but two-letter hashes collide; verify surrounding content or re-read before using it.
40
+ - After `+TEXT` (or `+`) the cursor advances past the inserted line, so consecutive `+TEXT` ops stack in order. After `Lid=TEXT` the cursor sits on the modified anchor; after `-Lid` it sits on the slot the deleted line vacated. You **MUST** use a fresh `@Lid` / `^Lid` / `^` / `$` to reposition.
41
+ - The tool is syntax-blind: it will not check brackets, indentation, table column counts, or fence integrity. You **MUST** verify indentation-sensitive or structured files after editing (Python, Markdown tables/fences).
42
+ - A section whose PATH does not yet exist creates the file from your `+TEXT` lines (use `^` or `$` then `+TEXT…`). No separate "create file" op is needed.
24
43
  </rules>
25
44
 
26
45
  <case file="a.ts">
@@ -33,11 +52,11 @@ Lid=X replace whole line with X; `Lid=` blanks it out
33
52
  </case>
34
53
 
35
54
  <examples>
36
- # Replace line
55
+ # Replace one line (preserve the leading tab from the original)
37
56
  ---a.ts
38
57
  {{hrefr 5}}= return clean.trim().toUpperCase();
39
58
 
40
- # Rewrite multiple adjacent lines (delete the old, insert the new)
59
+ # Rewrite multiple adjacent lines (delete each, then insert new content)
41
60
  ---a.ts
42
61
  -{{hrefr 3}}
43
62
  -{{hrefr 4}}
@@ -47,48 +66,84 @@ Lid=X replace whole line with X; `Lid=` blanks it out
47
66
  + return (name || DEF).trim().toUpperCase();
48
67
  +}
49
68
 
50
- # Append after
69
+ # Same rewrite using a range (equivalent to four `-Lid` lines)
51
70
  ---a.ts
52
- @{{hrefr 4}}
53
- + const suffix = "";
71
+ -{{hrefr 3}}..{{hrefr 6}}
72
+ +export function label(name: string): string {
73
+ + return (name || DEF).trim().toUpperCase();
74
+ +}
54
75
 
55
- # Delete a line
76
+ # Replace a contiguous range with one line (range-replace shorthand)
56
77
  ---a.ts
57
- -{{hrefr 2}}
78
+ {{hrefr 3}}..{{hrefr 6}}=export const label = (name: string) => (name || DEF).trim().toUpperCase();
58
79
 
59
- # Prepend and append
80
+ # Replace a contiguous range with multiple lines (continuation form)
60
81
  ---a.ts
61
- $
82
+ {{hrefr 3}}..{{hrefr 6}}=export function label(name: string): string {
83
+ \ return (name || DEF).trim().toUpperCase();
84
+ \}
85
+
86
+ # Replace a block with a longer multi-line block, including blank lines (canonical form for refactors)
87
+ ---a.ts
88
+ {{hrefr 3}}..{{hrefr 6}}=/** Format a display label, falling back to DEF when empty. */
89
+ \export function label(name: string): string {
90
+ \ const clean = (name || DEF).trim();
91
+ \
92
+ \ if (clean.length === 0) return DEF;
93
+ \ return clean.toUpperCase();
94
+ \}
95
+
96
+ # Insert ABOVE a line
97
+ ---a.ts
98
+ ^{{hrefr 5}}
99
+ + const debug = false;
100
+
101
+ # Insert BELOW a line
102
+ ---a.ts
103
+ @{{hrefr 4}}
104
+ + const debug = false;
105
+
106
+ # Insert above the first line (use BOF)
107
+ ---a.ts
108
+ ^
62
109
  +// Copyright (c) 2026
63
110
  +
64
- ^
111
+
112
+ # Append at end of file
113
+ ---a.ts
114
+ $
65
115
  +export { DEF };
66
116
 
67
- # File ops
117
+ # Delete a single line
68
118
  ---a.ts
69
- !rm
70
- ---b.ts
71
- !mv a.ts
119
+ -{{hrefr 2}}
72
120
 
73
- # Wrong: `@Lid=TEXT` is not replacement syntax
121
+ # Delete the file (no other ops in the section)
74
122
  ---a.ts
75
- @{{hrefr 5}}= return clean.trim().toUpperCase();
123
+ !rm
76
124
 
77
- # Wrong: do not split `Lid=TEXT` across lines
125
+ # Rename a file
78
126
  ---a.ts
79
- {{hrefr 5}}=
80
- return clean.trim().toUpperCase();
127
+ !mv b.ts
81
128
 
82
- # Wrong: do not replace by deleting then adding
129
+ # Multi-file edit in one input
83
130
  ---a.ts
84
- -{{hrefr 5}}
85
- +{{hrefr 5}}= return clean.trim().toUpperCase();
131
+ {{hrefr 1}}=const DEF = "user";
132
+ ---other.ts
133
+ $
134
+ +// new footer
86
135
  </examples>
87
136
 
88
137
  <critical>
89
- - Copy Lids **EXACTLY** from prior tool output. Never guess, shorten, or omit the letters.
90
- - Only emit lines that change. Never repeat unchanged context anchors imply it.
91
- - This is **NOT** unified diff. Never send `@@`, `-OLD` / `+NEW` pairs, or unchanged context.
92
- - Never split `Lid=TEXT` across two physical lines.
93
- - Never stack `Lid=X` over a contiguous range. Use `-Lid`+`+TEXT` for block rewrites.
138
+ - You **MUST** copy Lids EXACTLY from the latest read/grep output. You **MUST NOT** guess, shorten, drop letters, or invent line numbers.
139
+ - Current/added preview lines include fresh `LINE+hash|content` anchors. Removed preview lines show deleted content and **MUST NOT** be reused as anchors.
140
+ - You **MUST** emit only lines that change. You **MUST NOT** echo unchanged context; the anchor implies position.
141
+ - You **MUST NOT** write `Lid=<sameTextThatIsAlreadyOnThatLine>`; the tool reports a no-op (no change applied). Emit `Lid=TEXT` only when TEXT differs.
142
+ - A line of the form `Lid|content` (a Lid, then `|`, then text, with NO leading `+`/`-`/`^`/`@`/`\`/`=`/`..`) is **FORBIDDEN**. That shape only appears in `read`/`grep` output as an anchor for *you*; it is never an edit op. If you copy a `Lid|content` line verbatim from a read into a patch, you have made an error — every edit op must start with `+`, `-`, `^`, `@`, `\`, `$`, `!`, or a Lid immediately followed by `=` or `..`.
143
+ - To replace a contiguous block with new content, the canonical form is `LidA..LidB=FIRST_LINE` + `\NEXT_LINE…`. You **MUST NOT** write the old block and then the new block — that is unified-diff thinking and the tool does not understand it. If you find yourself emitting pre-image lines (with or without operators) before your new content, STOP and rewrite the section as a single range-replace.
144
+ - TEXT after `=`, `+`, or `\` includes leading whitespace verbatim. You **MUST NOT** trim or re-indent it.
145
+ - This is NOT unified diff. You **MUST NOT** write `@@` headers, `-OLD`/`+NEW` pairs, context lines, or `+Lid|…` (bad: `+5th|new text`; good: `5th=new text`).
146
+ - You **MUST NOT** split `Lid=TEXT` across two physical lines.
147
+ - For a contiguous range replacement, you **MAY** use either `Lid=FIRST_LINE` + `\NEXT_LINE…` (extends one anchor) or `LidA..LidB=FIRST_LINE` + `\NEXT_LINE…` (collapses an existing range), or fall back to `-LidA..LidB` + `+TEXT…` (delete + insert).
148
+ - The tool is syntax-blind. Indentation, brackets, fences, table widths — you remain responsible.
94
149
  </critical>
@@ -1,40 +1,6 @@
1
- Signals plan completion, requests user approval, and provides the final plan title for handoff.
1
+ Submits a finalized implementation plan for user approval.
2
2
 
3
- <conditions>
4
- Use when:
5
- - Plan written to `local://PLAN.md`
6
- - No unresolved questions about requirements or approach
7
- - Ready for user review and approval
8
- </conditions>
9
-
10
- <instruction>
11
- - You **MUST** write plan to plan file BEFORE calling this tool
12
- - Tool reads plan from file—does not take plan content as parameter
13
- - You **MUST** provide a `title` argument for the final plan artifact (example: `WP_MIGRATION_PLAN`)
14
- - `.md` is optional in `title`; it is appended automatically when omitted
15
- - User sees plan contents when reviewing
16
- </instruction>
17
-
18
- <output>
19
- Presents plan to user for approval. If approved, plan mode exits with full tool access restored and the plan is renamed to `local://<title>.md`.
20
- </output>
21
-
22
- <examples>
23
- # Ready
24
- Plan complete at local://PLAN.md, no open questions.
25
- → Call `exit_plan_mode` with `{ "title": "WP_MIGRATION_PLAN" }`
26
- # Unclear
27
- Unsure about auth method (OAuth vs JWT).
28
- → Use `ask` first to clarify, then call `exit_plan_mode`
29
- </examples>
30
-
31
- <avoid>
32
- - **MUST NOT** call before plan is written to file
33
- - **MUST NOT** omit `title`
34
- - **MUST NOT** use `ask` to request plan approval (this tool does that)
35
- - **MUST NOT** call after pure research tasks (no implementation planned)
36
- </avoid>
37
-
38
- <critical>
39
- You **MUST** only use when planning implementation steps. Research tasks (searching, reading, understanding) do not need this tool.
40
- </critical>
3
+ Write the plan to `local://PLAN.md` first, then call this with `title` (e.g. `WP_MIGRATION_PLAN`); on approval the file is renamed to `local://<title>.md` and full tool access is restored.
4
+ - Use only after planning implementation steps; not for pure research.
5
+ - **MUST NOT** call before the plan file exists.
6
+ - **MUST NOT** use `ask` to request plan approval — this tool does that.
@@ -4,9 +4,9 @@ GitHub CLI tool with a single op-based dispatch. Wraps `gh` for repository, issu
4
4
  Pick the operation via `op`. Each op uses a subset of the parameters:
5
5
  - `repo_view` — Read repository metadata. Optional `repo` (owner/repo) and `branch`. Falls back to the current checkout or default `gh` repo.
6
6
  - `issue_view` — Read an issue. Required `issue` (number or URL). Optional `repo`. Set `comments: false` to skip discussion.
7
- - `pr_view` — Read a pull request, including reviews and inline review comments. Optional `pr` (number, URL, or branch); omitting it targets the current branch's PR. Optional `repo`. Set `comments: false` for a lighter summary.
8
- - `pr_diff` — Read a pull request diff. Optional `pr`, `repo`. Set `nameOnly: true` for changed file names. Use `exclude` to drop generated paths from the diff.
9
- - `pr_checkout` — Check a pull request out into a dedicated git worktree. Optional `pr`, `repo`, `branch` (local), `worktree` (path), `force` (reset existing local branch).
7
+ - `pr_view` — Read one or more pull requests, including reviews and inline review comments. Optional `pr` (number, URL, branch, or array of any — pass an array to fetch multiple PRs in one call); omitting it targets the current branch's PR. Optional `repo`. Set `comments: false` for a lighter summary.
8
+ - `pr_diff` — Read one or more pull request diffs. Optional `pr` (single identifier or array for batch). Optional `repo`. Set `nameOnly: true` for changed file names. Use `exclude` to drop generated paths from the diff.
9
+ - `pr_checkout` — Check one or more pull requests out into dedicated git worktrees. Optional `pr` (number, URL, branch, or array of any of those — pass an array to batch-check-out multiple PRs in one call), `repo`, `force` (reset existing local branch).
10
10
  - `pr_push` — Push a checked-out PR branch back to its source branch. Requires the branch to have been checked out via `op: pr_checkout` (carries push metadata). Optional `branch`; defaults to the current checked-out git branch. Optional `forceWithLease`.
11
11
  - `search_issues` — Search issues using normal GitHub issue search syntax. Required `query`. Optional `repo`, `limit`.
12
12
  - `search_prs` — Search pull requests using normal GitHub PR search syntax. Required `query`. Optional `repo`, `limit`.
@@ -17,8 +17,7 @@ Interacts with Language Server Protocol servers for code intelligence.
17
17
  <parameters>
18
18
  - `file`: File path, glob pattern (e.g. `src/**/*.ts`), or `"*"` for workspace scope. Globs are expanded locally before dispatch. `"*"` routes `diagnostics`/`symbols`/`reload` to their workspace-wide form.
19
19
  - `line`: 1-indexed line number for position-based actions
20
- - `symbol`: Substring on the target line used to resolve column automatically
21
- - `occurrence`: 1-indexed match index when `symbol` appears multiple times on the same line
20
+ - `symbol`: Substring on the target line used to resolve column automatically. Append `#N` to pick the Nth occurrence on that line (1-indexed; default 1) — e.g. `foo#2` selects the second `foo`.
22
21
  - `query`: Symbol search query, code-action kind filter (list mode), or code-action selector (apply mode)
23
22
  - `new_name`: Required for rename
24
23
  - `apply`: Apply edits for rename/code_actions (default true for rename, list mode for code_actions unless explicitly true)
@@ -29,7 +28,7 @@ Interacts with Language Server Protocol servers for code intelligence.
29
28
  - Requires running LSP server for target language
30
29
  - Some operations require file to be saved to disk
31
30
  - Glob expansion samples up to 20 files per request; use `file: "*"` for broader coverage
32
- - When `symbol` is provided for position-based actions, missing symbols or out-of-bounds `occurrence` values return an explicit error instead of silently falling back
31
+ - When `symbol` is provided for position-based actions, missing symbols or out-of-bounds `#N` occurrence selectors return an explicit error instead of silently falling back
33
32
  </caution>
34
33
 
35
34
  <critical>
@@ -10,7 +10,7 @@ Run a recipe / script / target from the project's task runners.
10
10
  {{#each runners}}
11
11
  <runner id="{{id}}" label="{{label}}" command="{{commandPrefix}}">
12
12
  {{#each tasks}}
13
- - `{{name}}{{#if paramSig}} {{paramSig}}{{/if}}`{{#if doc}} — {{doc}}{{/if}}{{#if command}} (`{{command}}`){{/if}}
13
+ - `{{name}}{{#if paramSig}} {{paramSig}}{{/if}}`{{#if doc}} — {{doc}}{{/if}}{{#if command}} (`{{command}}`{{#if cwd}} in `{{cwd}}`{{/if}}){{/if}}
14
14
  {{/each}}
15
15
  </runner>
16
16
  {{/each}}
@@ -1,167 +1,54 @@
1
1
  Launches subagents to parallelize workflows.
2
2
 
3
3
  {{#if asyncEnabled}}
4
- - Use `read jobs://` to inspect state; `read jobs://<job_id>` for detail.
5
- - Use the `job` tool (with `poll`) to wait until completion. You **MUST NOT** poll `read jobs://` in a loop.
4
+ - `read jobs://` for state, `read jobs://<id>` for detail.
5
+ - Use `job` (with `poll`) to wait. **MUST NOT** poll `read jobs://` in a loop.
6
6
  {{/if}}
7
7
 
8
- {{#if defaultMode}}
9
- Current input mode: `default`. Shared `context` and custom task-call `schema` are available.
10
- {{/if}}
11
- {{#if schemaFreeMode}}
12
- Current input mode: `schema-free`. Shared `context` is available; custom task-call `schema` is disabled. For structured output, rely on the agent definition or inherited session schema.
13
- {{/if}}
14
- {{#if independentMode}}
15
- Current input mode: `independent`. Shared `context` and custom `schema` are both disabled. Every assignment must stand on its own.
16
- {{/if}}
17
-
18
- {{#if contextEnabled}}
19
- Subagents lack your conversation history. Every decision, file content, and user requirement they need **MUST** be explicit in `context` or `assignment`.
20
- {{else}}
21
- Subagents lack your conversation history. Every decision, file content, and user requirement they need **MUST** be explicit in each task `assignment`.
22
- {{/if}}
8
+ Subagents have no access to your conversation history. Every fact, file path, and decision they need **MUST** be explicit in {{#if contextEnabled}}`context` or `assignment`{{else}}each `assignment`{{/if}}.
23
9
 
24
10
  <parameters>
25
- - `agent`: Agent type for all tasks.
26
- - `.id`: CamelCase, max 32 chars
27
- - `.description`: UI display only — subagent never sees it
28
- - `.assignment`: Complete self-contained instructions. One-liners PROHIBITED; missing acceptance criteria = too vague.
29
- {{#if contextEnabled}}
30
- - `context`: Shared background prepended to every assignment. Session-specific info only.
31
- {{/if}}
32
- {{#if customSchemaEnabled}}
33
- - `schema`: JSON-encoded JTD schema for expected output. Format lives here — **MUST NOT** be duplicated in assignments.
34
- {{/if}}
35
- - `tasks`: Tasks to execute in parallel.
36
- {{#if isolationEnabled}}
37
- - `isolated`: Run in isolated environment; returns patches. Use when tasks edit overlapping files.
38
- {{/if}}
11
+ - `agent`: agent type for all tasks
12
+ - `tasks`: tasks to execute in parallel
13
+ - `.id`: CamelCase, ≤32 chars
14
+ - `.description`: UI label only subagent never sees it
15
+ - `.assignment`: complete self-contained instructions; one-liners and missing acceptance criteria are PROHIBITED
16
+ {{#if contextEnabled}}- `context`: shared background prepended to every assignment; session-specific only{{/if}}
17
+ {{#if customSchemaEnabled}}- `schema`: JTD schema for expected structured output (do not put format rules in assignments){{/if}}
18
+ {{#if isolationEnabled}}- `isolated`: run in isolated env; use when tasks edit overlapping files{{/if}}
39
19
  </parameters>
40
20
 
41
- <critical>
42
- {{#if contextEnabled}}
43
- - **MUST NOT** duplicate shared constraints across assignments put them in `context` once.
44
- {{else}}
45
- - Every `assignment` must repeat any constraints, reference paths, and acceptance criteria it needs — there is no shared `context` field.
46
- {{/if}}
47
- - **MUST NOT** tell tasks to run project-wide build/test/lint. Parallel agents share the working tree; each task edits, stops. Caller verifies after all complete.
48
- - For large payloads (traces, JSON blobs), write to `local://<path>` and pass the path in {{#if contextEnabled}}`context`{{else}}the relevant `assignment`{{/if}}.
49
- - Prefer `task` agents that investigate **and** edit in one pass. Launch a dedicated read-only discovery step only when affected files are genuinely unknown.
50
- </critical>
51
-
52
- <scope>
53
- Each task: **at most 3–5 files**. Globs, "update all", or package-wide scope = too broad. Enumerate files explicitly and fan out to a cluster of agents.
54
- </scope>
21
+ <rules>
22
+ - **MUST NOT** assign tasks to run project-wide build/test/lint. Caller verifies after the batch.
23
+ - Each task: ≤3–5 explicit files. No globs, no "update all", no package-wide scope. Fan out to a cluster instead.
24
+ - Pass large payloads via `local://<path>` URIs, not inline.
25
+ {{#if contextEnabled}}- Put shared constraints in `context` once; do not duplicate across assignments.{{/if}}
26
+ - Prefer agents that investigate **and** edit in one pass; only spin a read-only discovery step when affected files are genuinely unknown.
27
+ </rules>
55
28
 
56
29
  <parallelization>
57
- **Test:** Can task B produce correct output without seeing A's result? Yes parallel. Nosequential.
58
-
59
- |Sequential first|Then|Reason|
60
- |---|---|---|
61
- |Types/interfaces|Consumers|Need contract|
62
- |API exports|Callers|Need signatures|
63
- |Core module|Dependents|Import dependency|
64
- |Schema/migration|App logic|Schema dependency|
65
-
66
- **Safe to parallelize:** independent modules, isolated file-scoped refactors, tests for existing code.
30
+ Test: can task B run correctly without seeing A's output? If no, sequence AB.
31
+ Sequential when one task produces a contract (types, API, schema, core module) the other consumes.
32
+ Parallel when tasks touch disjoint files or are independent refactors/tests.
67
33
  </parallelization>
68
34
 
69
- <templates>
70
- {{#if contextEnabled}}
71
- **context:**
72
- ```
73
- ## Goal ← one sentence: what the batch accomplishes
74
- ## Non-goals ← what tasks must not touch
75
- ## Constraints ← MUST/MUST NOT rules and session decisions
76
- ## API Contract ← exact types/signatures if tasks share an interface (omit if N/A)
77
- ## Acceptance ← definition of done; build/lint runs AFTER all tasks complete
78
- ```
79
- {{else}}
80
- No shared `context` field exists in this mode. Fold goal, non-goals, constraints, and acceptance criteria into each `assignment`.
81
- {{/if}}
82
- **assignment:**
83
- ```
84
- ## Target ← exact file paths; named symbols; explicit non-goals
85
- ## Change ← step-by-step what to add/remove/rename; patterns/APIs to use
86
- ## Edge Cases ← tricky inputs; existing behavior that must survive
87
- ## Acceptance ← observable result proving the task is done; no project-wide commands
88
- ```
89
- </templates>
90
-
91
- <checklist>
92
- Before invoking:
93
35
  {{#if contextEnabled}}
94
- - `context` contains only session-specific info
95
- {{else}}
96
- - Every `assignment` includes its own goal, constraints, and acceptance criteria (no shared context)
36
+ <context-fmt>
37
+ # Goal ← one sentence: what the batch accomplishes
38
+ # Constraints ← **MUST**/**MUST NOT** rules and session decisions
39
+ # Contract ← exact types/signatures if tasks share an interface
40
+ </context-fmt>
97
41
  {{/if}}
98
- - Every `assignment` follows the template; no one-liners; edge cases covered
99
- - Tasks are truly parallel — you can articulate why none depends on another's output
100
- - File paths are explicit; no globs
101
- {{#if customSchemaEnabled}}
102
- - `schema` is set if you expect structured output
103
- {{else}}
104
- - Do not pass a custom task-call `schema` in this mode
105
- {{/if}}
106
- </checklist>
107
-
108
- {{#if contextEnabled}}
109
- <example label="Rename exported symbol + update all call sites">
110
- Two tasks with non-overlapping file sets — demonstrates scope partitioning.
111
42
 
112
- <context>
113
- ## Goal
114
- Rename `parseConfig` `loadConfig` in `src/config/parser.ts` and all callers.
115
- ## Non-goals
116
- No behavior or signature changes; rename only.
117
- ## Acceptance (global)
118
- Caller runs `bun check:ts` after both tasks complete. Tasks must NOT run it.
119
- </context>
120
- <tasks>
121
- <task name="RenameExport">
122
- <assignment>
123
- ## Target
124
- - `src/config/parser.ts`: function `parseConfig`
125
- - If `src/config/index.ts` re-exports it, update the re-export
126
- - Non-goals: do not touch callers or tests
127
-
128
- ## Change
129
- - Rename `parseConfig` → `loadConfig` (declaration + any JSDoc references)
130
-
131
- ## Edge Cases
132
- - Rename all overload signatures if overloaded
133
- - Internal helpers like `_parseConfigValue` are different symbols — leave untouched
134
- - Do not add a backwards-compat alias
135
-
136
- ## Acceptance
137
- - `parseConfig` no longer appears as a top-level export in `parser.ts`
138
- </assignment>
139
- </task>
140
- <task name="UpdateCallers">
141
- <assignment>
142
- ## Target
143
- - `src/cli/init.ts`, `src/server/bootstrap.ts`, `src/worker/index.ts`
144
- - Non-goals: do not touch `src/config/parser.ts` or `src/config/index.ts`
145
-
146
- ## Change
147
- - Replace `import { parseConfig }` → `import { loadConfig }`
148
- - Replace every call site `parseConfig(` → `loadConfig(`
149
- - For `import * as cfg` users, update `cfg.parseConfig` property access
150
-
151
- ## Edge Cases
152
- - String literals containing "parseConfig" (logs, comments) are documentation — leave them
153
- - If a file re-exports to an external package boundary, keep the old name via `export { loadConfig as parseConfig }` with a `// TODO: remove after next major` comment
154
-
155
- ## Acceptance
156
- - No bare `parseConfig` identifier remains in the three target files
157
- </assignment>
158
- </task>
159
- </tasks>
160
- </example>
161
- {{/if}}
43
+ <assignment-fmt>
44
+ # Target ← exact files and symbols; explicit non-goals
45
+ # Change ← step-by-step add/remove/rename; APIs and patterns
46
+ # Acceptance ← observable result; no project-wide commands
47
+ </assignment-fmt>
162
48
 
49
+ <agents>
163
50
  {{#list agents join="\n"}}
164
- ### Agent: {{name}}
165
- **Tools:** {{default (join tools ", ") "All"}}
51
+ # {{name}}
166
52
  {{description}}
167
53
  {{/list}}
54
+ </agents>