style(agent-designer): strip redundant commentary from extracted package

Author: TheGreatAxios

agents/designer/README.md

@@ -2,11 +2,8 @@
 
 ## What this is
 
-A ready-to-go Corbits agent for the **designer** role (`@corbits/code-agent-designer`)
-plus its named component parts so a consumer can recreate or remix it. Designer is
-the UI/UX design engineering specialist: it owns DESIGN.md creation and updates,
-impeccable.style design laws, design tokens, typography, spatial layout, and
-micro-interaction polish.
+Ready `@corbits/code-agent-designer` agent + independently importable components:
+`agent`, `defineAgent`, `director`, `tools`, `systemPrompt`, `config`.
 
 ## Consume (runLocal)
 
@@ -17,32 +14,22 @@ const run = runLocal(defining(agent), env);
 
 ## Consume (deploy runWorkflow / hub / sidecar)
 
-Hand the same `AgentDefinition` (`agent`) to a workflow, hub, or sidecar. The
-package never exports a workflow itself — a workflow _consumes_ `agent`.
+Hand the same `AgentDefinition` to a workflow — the package never exports a
+workflow itself.
 
 ## Remix components
 
 ```ts
-import {
-  director,
-  tools,
-  systemPrompt,
-  config,
-  defineAgent,
-} from "@corbits/code-agent-designer";
-
-// Swap components at the consumer side.
-defineAgent({ director, tools, systemPrompt, config }); // or spread overrides
+import { director, tools, systemPrompt, config } from "@corbits/code-agent-designer";
+defineAgent({ director, ... }); // or swap tools / systemPrompt / config
 ```
 
-Each of `agent`, `defineAgent`, `director`, `tools`, `systemPrompt`, `config` is a
-named, independently importable part.
+Each of `director`, `tools`, `systemPrompt`, and `config` is a named,
+independently importable part.
 
-## Byte-stability & single authority
+## Single authority
 
-- `systemPrompt.build()` is pure and byte-stable — identical string every call.
-- `tools` is content-equal to the app's `REVIEW_TOOLS` surface constant
-  (`src/agent/directors/tool-sets.ts`); the in-tree drift-guard
-  (`src/agent/directors/designer/package.test.ts`) enforces they match.
-- This package is the single prompt authority for designer; the in-tree
-  `src/agent/directors/designer/package.ts` is deleted.
+`prompt.ts` `systemPrompt.build()` is byte-stable (the raw card, no formatting).
+`toolset.ts` `tools` literal equals the `REVIEW_TOOLS` surface constant in
+`src/agent/directors/tool-sets.ts`; drift is enforced by the in-tree drift-guard
+`src/agent/directors/designer/package.test.ts`.

agents/designer/src/agent.ts

@@ -2,7 +2,6 @@ import {
   defineAgent as coreDefineAgent,
   type AgentDefinition,
 } from "@intx/agent";
-
 import { systemPrompt } from "./prompt.js";
 
 export const agent: AgentDefinition = defineAgent();
@@ -11,7 +10,7 @@ export function defineAgent(): AgentDefinition {
   return coreDefineAgent({
     id: "designer",
     systemPrompt: systemPrompt.build(),
-    tools: [],
+    tools: [], // tool mounting is consumer/app-side; surface enforced by drift-guard
     capabilities: [],
     inference: { sources: [] },
     tags: { fleet: "0.3.36", lane: "designer" },

agents/designer/src/config.ts

@@ -1,13 +1,9 @@
-// Designer is a leaf worker: maySpawn false, no allowlist. modelRole is
-// "implement" (matches the in-tree card).
 export const config = {
-  spawn: {
-    maySpawn: false,
-  },
+  spawn: { maySpawn: false },
   tier: "leaf",
   modelRole: "implement",
 } as const satisfies {
-  spawn: { maySpawn: boolean; allowlist?: readonly string[] };
+  spawn: { maySpawn: boolean };
   tier: string;
   modelRole: string;
 };

agents/designer/src/director.ts

@@ -15,8 +15,8 @@ export const director = {
   description:
     "UI/UX designer — owns DESIGN.md, impeccable style design laws, tokens, and interface polish",
   systemPrompt: systemPrompt.build(),
-  tools: { allow: tools },
+  tools: { allow: [...tools] },
   spawn: config.spawn,
-  modelRole: config.modelRole,
   tier: config.tier,
+  modelRole: config.modelRole,
 } as const;

agents/designer/src/prompt.ts

@@ -1,4 +1,4 @@
-// Designer card — the single prompt authority. `build()` is byte-stable.
+// Designer card. Byte-stable build, single prompt authority.
 const CARD = `You are DesignerDirector (Designer), a specialist in Corbits Code.
 
 PRIMARY INTENT: own interface design, design tokens, styling, and DESIGN.md. You bring design engineering excellence to UI surfaces — layout rhythm, typography, purposeful motion, responsive states, and cohesive design systems.
@@ -25,7 +25,6 @@ OUT OF LANE: backend business logic or database migrations, general backend defe
 
 export const theme = "designer";
 
-// Byte-stable (see prompt authority test).
 export const systemPrompt = {
   theme,
   build: (): string => CARD,

agents/designer/src/toolset.ts

@@ -1,5 +1,5 @@
-// BUILD_TOOLS literal (= REVIEW_TOOLS); drift-guarded against REVIEW_TOOLS in
-// src/agent/directors/tool-sets.ts.
+// BUILD_TOOLS literal (= REVIEW_TOOLS); drift-guarded against REVIEW_TOOLS
+// by src/agent/directors/designer/package.test.ts.
 const BUILD_TOOLS = [
   "read_file",
   "grep",

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

@@ -15,8 +15,6 @@ describe("designerPackage", () => {
     const asDirector: DirectorPackage = director;
     expect(asDirector.id).toBe("designer");
     expect(DIRECTOR_REGISTRY.designer).toBe(director);
-    // In-tree card used BUILD_TOOLS; BUILD_TOOLS === REVIEW_TOOLS content, so
-    // the drift-guard asserts against REVIEW_TOOLS (the brief's surface rule).
     expect([...director.tools.allow]).toEqual([...REVIEW_TOOLS]);
     expect(new Set(director.tools.allow).size).toBe(
       director.tools.allow.length,
@@ -29,7 +27,6 @@ describe("designerPackage", () => {
     expect(director.modelRole).toBe(config.modelRole);
     expect(director.spawn.maySpawn).toBe(false);
     expect(director.tier).toBe("leaf");
-    // modelRole preserved from the in-tree card (implement, not §5 docs).
     expect(director.modelRole).toBe("implement");
   });