Fix prettier formatting for Skywalker prompt overhaul

Author: TheGreatAxios

docs/ARCHITECTURE.md

@@ -265,21 +265,21 @@ Every shipped specialist is a **director package** — a prompt-first `DirectorP
 
 **Intent → director** (`task(intent=…)` when `agent` is omitted)
 
-| Intent    | Default director                  |
-| --------- | --------------------------------- |
-| implement | builder                           |
-| explore   | explorer                          |
-| plan      | counsel                           |
-| review    | critic (override with `agent=…`)  |
-| general   | **none** — reclassify only        |
+| Intent    | Default director                 |
+| --------- | -------------------------------- |
+| implement | builder                          |
+| explore   | explorer                         |
+| plan      | counsel                          |
+| review    | critic (override with `agent=…`) |
+| general   | **none** — reclassify only       |
 
 **Spawn matrix**
 
-| Who                         | Spawn rights                     |
-| --------------------------- | -------------------------------- |
-| skywalker (primary session) | Full closed fleet                |
-| greybeard                   | intern, explorer, critic only    |
-| All other directors         | no `task`                        |
+| Who                         | Spawn rights                  |
+| --------------------------- | ----------------------------- |
+| skywalker (primary session) | Full closed fleet             |
+| greybeard                   | intern, explorer, critic only |
+| All other directors         | no `task`                     |
 
 **Tool envelopes** prefer small `tools.allow` mounts over deny-everything. Shipped docs/design directors (shakespeare, rand, bruckheimer) mount write tools with no path-level lock. Lane routing is spawn policy (shakespeare = P/A/I docs, rand = DESIGN.md, bruckheimer = product discovery), not a file lock. There is no static per-package write-path declaration (CL-6952 removed it — no shipped director ever set one); instead the task tool records, without blocking, when two concurrently running dispatches land on the same cwd (see `intervention-log.ts`'s `conflict` class).
 
@@ -422,12 +422,12 @@ Each `<base>/<skill-name>/SKILL.md` is one skill. Discovery dedupes by directory
 
 A skill file begins with a YAML frontmatter block, followed by the body that holds the instructions. Discovery parses `description` and `disable-model-invocation`; `loadSkillCommands` also reads `user-invocable`. The skill's identifier (what `use_skill` and `/<skill-name>` take) is its directory name. A skill with no `SKILL.md` or an empty body is skipped.
 
-| Field                       | Required     | Description                                                                                                                                                                                                 |
-| --------------------------- | ------------ | ----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- |
-| `description`               | yes          | One-line summary shown in the prompt's lazy skills listing and the slash picker                                                                                                                             |
-| `name`                      | conventional | Conventionally matches the directory name; the directory name is what is actually used as the identifier                                                                                                    |
-| `user-invocable`            | no           | When `false`, `loadSkillCommands` skips slash synthesis; the skill remains `use_skill` only. Untagged skills still become slashes (marketplace BC)                                                          |
-| `disable-model-invocation`  | no           | When `true`, `discoverSkills` omits the skill from the lazy listing (but still claims the name for first-wins). Explicit `resolveSkillBody` / `use_skill("name")` still loads the body. Does not affect slash emission. |
+| Field                      | Required     | Description                                                                                                                                                                                                             |
+| -------------------------- | ------------ | ----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- |
+| `description`              | yes          | One-line summary shown in the prompt's lazy skills listing and the slash picker                                                                                                                                         |
+| `name`                     | conventional | Conventionally matches the directory name; the directory name is what is actually used as the identifier                                                                                                                |
+| `user-invocable`           | no           | When `false`, `loadSkillCommands` skips slash synthesis; the skill remains `use_skill` only. Untagged skills still become slashes (marketplace BC)                                                                      |
+| `disable-model-invocation` | no           | When `true`, `discoverSkills` omits the skill from the lazy listing (but still claims the name for first-wins). Explicit `resolveSkillBody` / `use_skill("name")` still loads the body. Does not affect slash emission. |
 
 There is no skill `type` field required for model invocation — a skill body is plain instruction text. Background libraries (e.g. `git-worktrees`) set both `user-invocable: false` and `disable-model-invocation: true` so they are absent from slash and listing, yet recipes can still `use_skill("git-worktrees")`. `argument-hint` on frontmatter is preserved for the slash picker (greyed arg guidance). Multi-step orchestration is a separate mechanism (see Workflows above), not a skill `type`.
 

docs/PRODUCT.md

@@ -144,12 +144,12 @@ Capabilities beyond the core toolset are opt-in plugins, enabled per workspace t
 
 The primary session is always **orchestrator** (single-agent mode is gone). Its identity is **Skywalker** (product name remains Corbits Code; when asked its name, answer Skywalker): classify work, DIY tiny/single-file/one-route product edits, dispatch a **closed fleet of 16 directors** for substantial work, track the fleet, and synthesize. Product mutation tools (`write_file` / `edit_file` / `delete_file`) are mounted on the primary (CORE / `SKYWALKER_TOOLS`) — path tools are the DIY surface; spawn remains the default for substantial, multi-file, parallel, or specialist work. Shell file-writes stay denied. MCP tools are not re-filtered by a product-write deny list (that list is gone). There is no static per-package write-path declaration (CL-6952 removed it — no shipped director ever set one). A concurrent dispatch landing on the same working directory as another still-running lane is recorded as a `conflict` intervention, not blocked. Operator slash recipes (`/implement`, `/plan`, `/refactor`, `/review`, `/pull-request-review`, `/create-issue`, `/scribe`, `/interview`, `/ast-grep`) tell Skywalker which directors to spawn for substantial work; tiny/bounded edits may run on the primary.
 
-| Lane      | Directors                                                                            |
-| --------- | ------------------------------------------------------------------------------------ |
-| Primary   | skywalker                                                                            |
+| Lane      | Directors                                                                              |
+| --------- | -------------------------------------------------------------------------------------- |
+| Primary   | skywalker                                                                              |
 | Eng       | builder, explorer, counsel, intern, critic, greybeard, neckbeard, bruckheimer, gaasbot |
-| Design    | draper, emil, rand                                                                   |
-| Docs / QA | shakespeare, testsmith, tester                                                       |
+| Design    | draper, emil, rand                                                                     |
+| Docs / QA | shakespeare, testsmith, tester                                                         |
 
 There is **no catch-all worker**. `task` requires `agent=…` or a non-general `intent` (implement/explore/plan/review→critic); bare dispatch and `intent=general` are refused. Named `task(agent=…)` selects a director package without requiring a plugin profile, except `skywalker` which is the primary session identity and is refused as a spawned worker. Nested spawn is runtime-enforced: only skywalker (full fleet allowlist) and greybeard (intern/explorer/critic) may spawn; other workers have no `task`. Primary omits an allowlist so plugin profiles remain reachable from the main session.
 

plugins/corbits-skills/skills/dispatch/SKILL.md

@@ -31,11 +31,11 @@ If the spec is vague, incomplete, or contradictory: stop and report Blockers. Do
 
 | Work                                                                                             | Director                  |
 | ------------------------------------------------------------------------------------------------ | ------------------------- |
-| Map the codebase, gather facts                                                                   | `task(agent="explorer")`   |
-| Eng plan from a spec (no ship)                                                                   | `task(agent="counsel")`      |
-| Write `dispatch.yaml` / `plan.md` / status artifacts (mechanical brief; no product feature work) | `task(agent="builder")`     |
-| Ship product code + tests                                                                        | `task(agent="builder")`     |
-| Review a landed task (defects, evidence, no fix)                                                 | `task(agent="critic")`  |
+| Map the codebase, gather facts                                                                   | `task(agent="explorer")`  |
+| Eng plan from a spec (no ship)                                                                   | `task(agent="counsel")`   |
+| Write `dispatch.yaml` / `plan.md` / status artifacts (mechanical brief; no product feature work) | `task(agent="builder")`   |
+| Ship product code + tests                                                                        | `task(agent="builder")`   |
+| Review a landed task (defects, evidence, no fix)                                                 | `task(agent="critic")`    |
 | Architecture judgment before a large DAG                                                         | `task(agent="greybeard")` |
 | Independent suite / repro evidence                                                               | `task(agent="tester")`    |
 

src/agent/directors/critic/package.test.ts

@@ -44,9 +44,7 @@ describe("criticPackage", () => {
       /returning Promise when callers expect a plain value/i,
     );
     expect(criticPackage.systemPrompt).toMatch(/blocking correctness defect/i);
-    expect(criticPackage.systemPrompt).toMatch(
-      /parameter order\/optionality\/return-type drift/i,
-    );
+    expect(criticPackage.systemPrompt).toMatch(/parameter order\/optionality\/return-type drift/i);
     expect(criticPackage.systemPrompt).toMatch(/Rank these as blocking, not style nits/i);
   });
 
@@ -73,9 +71,7 @@ describe("criticPackage", () => {
   });
 
   test("primaryIntent and outOfLane match critique lane", () => {
-    expect(criticPackage.primaryIntent).toBe(
-      "Evidence-based code review; never fix product code",
-    );
+    expect(criticPackage.primaryIntent).toBe("Evidence-based code review; never fix product code");
     expect(criticPackage.outOfLane).toContain("implementing fixes");
     expect(criticPackage.outOfLane).toContain("architecture portfolio without code evidence");
     expect(criticPackage.outOfLane).toContain("visual brand");

src/agent/directors/tool-sets.ts

@@ -59,12 +59,7 @@ export const DOCS_TOOLS = [
 export const REVIEW_TOOLS = [...READ_TOOLS, ...PRODUCT_WRITE_TOOLS] as const;
 
 /** Mechanical intern: shell-first + path writes when the brief requires them. */
-export const INTERN_TOOLS = [
-  "run_shell",
-  "read_file",
-  "list_dir",
-  ...PRODUCT_WRITE_TOOLS,
-] as const;
+export const INTERN_TOOLS = ["run_shell", "read_file", "list_dir", ...PRODUCT_WRITE_TOOLS] as const;
 
 /** Nested orchestrator surface (greybeard / package filter): dispatch + path writes. */
 export const ORCHESTRATOR_TOOLS = [

src/agent/tools.ts

@@ -39,7 +39,11 @@ import {
   type SubAgentProvider,
   type SubAgentSessionStore,
 } from "../subagent/index.js";
-import { createFleetRecords, createSpawnAgentTool, createWaitAgentsTool } from "../subagent/agent-fleet.js";
+import {
+  createFleetRecords,
+  createSpawnAgentTool,
+  createWaitAgentsTool,
+} from "../subagent/agent-fleet.js";
 import {
   createCloseAgentTool,
   createResumeAgentTool,

src/subagent/retain-salvage.test.ts

@@ -4,7 +4,12 @@ import { createSubAgentSessionStore } from "./session-store.js";
 describe("retained session lifecycle", () => {
   test("a salvaged (deadline/cancel) run lands resumable even though run.ts disposed its agent", () => {
     const store = createSubAgentSessionStore({ maxCompleted: 5 });
-    const s = store.start({ description: "worker", agentId: "builder", brief: "b", retained: true });
+    const s = store.start({
+      description: "worker",
+      agentId: "builder",
+      brief: "b",
+      retained: true,
+    });
     store.markRunning(s.id);
     // run.ts salvage path RETURNS a report (does not throw) with stopReason
     // "deadline", but leaves turnSucceeded=false so finally disposes the
@@ -21,7 +26,12 @@ describe("retained session lifecycle", () => {
 
   test("cancelAll does not close retained completed sessions", () => {
     const store = createSubAgentSessionStore({ maxCompleted: 5 });
-    const s = store.start({ description: "worker", agentId: "builder", brief: "b", retained: true });
+    const s = store.start({
+      description: "worker",
+      agentId: "builder",
+      brief: "b",
+      retained: true,
+    });
     let closed = false;
     store.registerClose(s.id, async () => {
       closed = true;
@@ -44,7 +54,12 @@ describe("retained session lifecycle", () => {
   test("retained completed sessions are bounded by maxRetained, not the display cap", () => {
     const store = createSubAgentSessionStore({ maxCompleted: 3, maxRetained: 3 });
     for (let i = 0; i < 50; i++) {
-      const s = store.start({ description: `w${i}`, agentId: "builder", brief: "b", retained: true });
+      const s = store.start({
+        description: `w${i}`,
+        agentId: "builder",
+        brief: "b",
+        retained: true,
+      });
       store.registerClose(s.id, async () => {});
       store.complete(s.id, "done");
     }
@@ -54,7 +69,12 @@ describe("retained session lifecycle", () => {
 
   test("a genuinely retained clean completion IS resumable, and cancelAll releases it", () => {
     const store = createSubAgentSessionStore({ maxCompleted: 5 });
-    const s = store.start({ description: "worker", agentId: "builder", brief: "b", retained: true });
+    const s = store.start({
+      description: "worker",
+      agentId: "builder",
+      brief: "b",
+      retained: true,
+    });
     store.markRunning(s.id);
     let closed = false;
     store.registerClose(s.id, async () => {
@@ -71,7 +91,12 @@ describe("retained session lifecycle", () => {
 
   test("clear() releases every retained session's close handle instead of dropping it silently", () => {
     const store = createSubAgentSessionStore({ maxCompleted: 5 });
-    const s = store.start({ description: "worker", agentId: "builder", brief: "b", retained: true });
+    const s = store.start({
+      description: "worker",
+      agentId: "builder",
+      brief: "b",
+      retained: true,
+    });
     store.markRunning(s.id);
     let closed = false;
     store.registerClose(s.id, async () => {
@@ -84,7 +109,12 @@ describe("retained session lifecycle", () => {
 
   test("close_agent during the setup window waits for the handle instead of falsely reporting shutdown", async () => {
     const store = createSubAgentSessionStore({ maxCompleted: 5 });
-    const s = store.start({ description: "worker", agentId: "builder", brief: "b", retained: true });
+    const s = store.start({
+      description: "worker",
+      agentId: "builder",
+      brief: "b",
+      retained: true,
+    });
     // No registerClose yet — closeOne races the agent-setup window.
     const closePromise = store.closeOne(s.id, 200);
     let registeredClose = false;
@@ -100,7 +130,12 @@ describe("retained session lifecycle", () => {
 
   test("close_agent gives up honestly (not a false shutdown) if the handle never arrives in time", async () => {
     const store = createSubAgentSessionStore({ maxCompleted: 5 });
-    const s = store.start({ description: "worker", agentId: "builder", brief: "b", retained: true });
+    const s = store.start({
+      description: "worker",
+      agentId: "builder",
+      brief: "b",
+      retained: true,
+    });
     store.markRunning(s.id);
     const status = await store.closeOne(s.id, 30);
     expect(status).not.toBe("shutdown");

tests/unit/corbits-skills-catalog.test.ts

@@ -148,9 +148,7 @@ test("only background libs carry disable-model-invocation", async () => {
 });
 
 test("linear-issue-workflow references use_skill(git-worktrees)", async () => {
-  const skill = await Bun.file(
-    join(pluginRoot, "skills/linear-issue-workflow/SKILL.md"),
-  ).text();
+  const skill = await Bun.file(join(pluginRoot, "skills/linear-issue-workflow/SKILL.md")).text();
   expect(skill).toContain('use_skill("git-worktrees")');
   expect(skill).not.toContain("git worktree add");
 });