Skip to content

Commit 39fabec

Browse files
committed
Say Spawn build in the primary harness facts
1 parent e6ed73c commit 39fabec

3 files changed

Lines changed: 4 additions & 4 deletions

File tree

plugins/corbits-skills/skills/pull-request-review/SKILL.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -101,7 +101,7 @@ Or leave it and tell the operator it remains for further investigation.
101101

102102
## Hard rules
103103

104-
- This recipe reviews; it does not land product patches. If the operator then asks for a tiny/bounded fix, DIY with write_file/edit_file/delete_file; spawn implement for substantial fixes.
104+
- This recipe reviews; it does not land product patches. If the operator then asks for a tiny/bounded fix, DIY with write_file/edit_file/delete_file; spawn build for substantial fixes.
105105
- Skywalker MUST NOT run the worktree git; intern does, via `run_shell`.
106106
- Do not implement fixes as part of the review.
107107
- Do not impersonate GitHub-Claude review comments.

src/agent/prompts.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,7 @@ export function buildHarnessFacts(
6464
"- Change files with write_file/edit_file and remove files with delete_file; shell file-writes and deletions are blocked.",
6565
]
6666
: [
67-
"- Change files with write_file/edit_file and remove files with delete_file for tiny/single-file/one-route bounded edits. Spawn implement for substantial/multi-file/parallel/specialist work (hard cap 4 workers). Docs/design still spawn shakespeare/bruckheimer/brand-reviewer except one-line fixes.",
67+
"- Change files with write_file/edit_file and remove files with delete_file for tiny/single-file/one-route bounded edits. Spawn build for substantial/multi-file/parallel/specialist work (hard cap 4 workers). Docs/design still spawn shakespeare/bruckheimer/brand-reviewer except one-line fixes.",
6868
"- Shell file-writes and deletions are blocked; never use echo/heredoc/sed/rm as a substitute for product tools. Path tools are the DIY surface.",
6969
]),
7070
"- Use the provided tools for file reads/searches instead of shelling out as a substitute.",
@@ -113,7 +113,7 @@ export function buildGuidelines(opts: { subAgent?: boolean; sessionMode?: Sessio
113113
"- read_file for file contents; grep or search_files to locate code; lsp for symbols, types, references, or call flow before opening large files.",
114114
subAgent
115115
? "- edit_file for targeted changes; write_file for new files or full rewrites; delete_file to remove files — never echo, heredoc, sed, or rm in the shell for those jobs."
116-
: "- edit_file for targeted DIY tiny/single-file/one-route edits; write_file for new files or full rewrites; delete_file to remove files — never shell-write (echo/heredoc/sed/rm). Spawn implement (or a docs director) for substantial/multi-file/parallel/specialist work.",
116+
: "- edit_file for targeted DIY tiny/single-file/one-route edits; write_file for new files or full rewrites; delete_file to remove files — never shell-write (echo/heredoc/sed/rm). Spawn build (or a docs director) for substantial/multi-file/parallel/specialist work.",
117117
"- run_shell for builds, tests, git, and one-off commands — not for shell find, head-position rg, or recursive grep -r (OOM risk), cat, or messaging the user.",
118118
...(subAgent
119119
? []

src/prompts.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ test("harness facts state only the non-derivable tool and safety rules", () => {
4949
const facts = buildHarnessFacts();
5050
expect(facts).toContain("write_file/edit_file");
5151
expect(facts).toContain("tiny/single-file/one-route");
52-
expect(facts).toContain("Spawn implement");
52+
expect(facts).toContain("Spawn build");
5353
expect(facts).not.toContain("not mounted on the primary Skywalker session");
5454
expect(facts).toContain("blocked");
5555
expect(facts).toContain("15s timeout");

0 commit comments

Comments
 (0)