castle-web-cli 0.4.77 → 0.4.79

package/dist/agent-prompts.d.ts

@@ -12,6 +12,7 @@ export interface PromptTask {
     files?: string[];
     error?: string;
     blockedBy?: string[];
+    suspectNoChanges?: boolean;
 }
 export interface PromptSibling {
     title: string;
@@ -22,6 +23,7 @@ export declare function buildRouterPrompt(opts: {
     deckLabel: string;
     quickReference?: string;
     deckTree?: string;
+    deckContents?: string;
     messages: PromptMessage[];
     tasks: PromptTask[];
     instruction: string;
@@ -39,8 +41,9 @@ export declare function buildTaskPrompt(opts: {
     progressPath: string;
     notesPath: string;
     depsSummary?: string;
-    backend?: "cursor" | "claude";
+    backend?: "cursor" | "claude" | "smith";
     deckTree?: string;
+    deckContents?: string;
     quickReference?: string;
     siblings?: PromptSibling[];
 }): string;

package/dist/agent-prompts.js

@@ -93,17 +93,29 @@ function renderTasks(tasks) {
         const blockedBy = t.blockedBy && t.blockedBy.length > 0
             ? ` -- blocked by: ${t.blockedBy.join(", ")}`
             : "";
-        return `- [${t.status} ${t.progress}%] ${t.title} (${t.id})${notes}${files}${error}${blockedBy}`;
+        const suspect = t.suspectNoChanges
+            ? " -- caution: done but touched no tracked files (bash side effects aren't tracked); verify the work actually landed"
+            : "";
+        return `- [${t.status} ${t.progress}%] ${t.title} (${t.id})${notes}${files}${error}${blockedBy}${suspect}`;
     })
         .join("\n");
 }
 export function buildRouterPrompt(opts) {
+    // Making the ABSENCE explicit, not just omitting the section, matters: a
+    // router on a bare/greenfield deck otherwise has no way to tell "no docs
+    // exist" from "the docs section just wasn't included in this prompt", and
+    // burns its first tool-call turns hunting for CLAUDE.md / **/*.md that
+    // isn't there (real incident: 2 of a router turn's 9 iterations, on an
+    // 8-file deck with zero markdown files).
     const quickReference = opts.quickReference?.trim()
         ? `\n\n== how this deck works ==\n${opts.quickReference.trim()}`
-        : "";
+        : "\n\n== how this deck works ==\nThis deck has no CLAUDE.md / AGENTS.md quick reference. There is no documentation to read -- do not search for one (CLAUDE.md, README, **/*.md, etc.).";
     const deckFiles = opts.deckTree?.trim()
         ? `\n\n== deck files ==\n${opts.deckTree.trim()}`
         : "";
+    const deckSource = opts.deckContents?.trim()
+        ? `\n\n== deck source (current contents -- no need to read_file these to see what's already there) ==\n${opts.deckContents.trim()}`
+        : "";
     // Section order is stable-to-volatile for prompt caching: rules + deck
     // identity never change within a serve, the transcript is append-only, and
     // the churning state (file tree, task board, instruction) rides at the tail
@@ -114,7 +126,7 @@ export function buildRouterPrompt(opts) {
 ${opts.deckLabel}${quickReference}
 
 == conversation so far ==
-${renderTranscript(opts.messages)}${deckFiles}
+${renderTranscript(opts.messages)}${deckFiles}${deckSource}
 
 == background tasks ==
 ${renderTasks(opts.tasks)}
@@ -139,7 +151,10 @@ export function userTurnInstruction(opts) {
             .join("\n\n")}`);
     }
     if (opts.attachments && opts.attachments.length > 0) {
-        parts.push(`The user attached image file(s), saved in the deck at: ${opts.attachments.join(", ")}. Open them with your read tool and take them into account; pass the paths along to task agents that need them.`);
+        // Wording covers both delivery modes: smith gets the images inline as
+        // content blocks in this same message (its read tool rejects binaries),
+        // while the CLI backends open the saved files themselves.
+        parts.push(`The user attached image file(s), saved in the deck at: ${opts.attachments.join(", ")}. If the images are not already visible in this message, open them with your read tool. Take them into account; pass the paths along to task agents that need them.`);
     }
     return parts.join("\n\n");
 }
@@ -153,12 +168,18 @@ export function buildTaskPrompt(opts) {
     const siblings = siblingRows
         ? `\n\nOther tasks on this deck's board (snapshot at your start):\n${siblingRows}\nRunning and waiting rows are sibling agents working in this same directory. If a sibling plainly owns something your prompt only references (art, a scene, a behavior), leave it to them -- create only what YOUR prompt says to create. Finished rows' file lists show where recent work landed.\n`
         : "";
+    // Same "say there are no docs" reasoning as buildRouterPrompt's
+    // quickReference above -- a task agent wastes turns the same way a router
+    // does if it has to discover the absence itself via a failed read/grep.
     const quickReference = opts.quickReference?.trim()
         ? `\n\nHow this deck works (from its docs -- read the full CLAUDE.md / AGENTS.md only when you need detail beyond this):\n\n${opts.quickReference.trim()}\n`
-        : "";
+        : "\n\nThis deck has no CLAUDE.md / AGENTS.md quick reference. There is no documentation to read -- do not search for one (CLAUDE.md, README, **/*.md, etc.).\n";
     const layout = opts.deckTree?.trim()
         ? `\n\nDeck files (snapshot at task start; parallel sibling tasks may create files named in your prompt that are absent here):\n\n${opts.deckTree.trim()}\n`
         : "";
+    const deckSource = opts.deckContents?.trim()
+        ? `\n\nDeck source (complete current contents of the deck's text files, snapshot at task start -- you do NOT need to read these files before editing them, edit directly; a file marked "not inlined" is either binary/generated or didn't fit the budget, so read it yourself first if you need it):\n\n${opts.deckContents.trim()}\n`
+        : "";
     // Claude-only: collapsing the wrap-up (progress 90 + restart + notes) into
     // one shell call reliably saves 1-2 serial ~7s turns there. Cursor's
     // composer sometimes reacts to the same rule with MORE calls, so it stays
@@ -166,10 +187,10 @@ export function buildTaskPrompt(opts) {
     // (We deliberately do NOT tell agents to overwrite files via `cat > … <<EOF`
     // -- blind whole-file rewrites made parallel agents clobber each other's
     // edits. Read-then-edit is slower but safe; that is the right tradeoff.)
-    const wrapUp = opts.backend === "claude"
+    const wrapUp = opts.backend === "claude" || opts.backend === "smith"
         ? `\n- Wrap up in ONE tool call, not several: once your last file edit is done, combine the 90-progress write, the final \`npm run restart\`, and writing the notes file into a single shell command (\`;\`-separated so the notes land even if the restart hiccups). Then stop -- no extra turns after it.`
         : "";
-    return `You are a background build agent for the Castle deck "${opts.deckLabel}" (current directory). A separate conversation agent dispatched you with one task. Follow the deck's CLAUDE.md / AGENTS.md conventions, and reload the served deck after changes (\`npm run restart\`).${quickReference}${layout}
+    return `You are a background build agent for the Castle deck "${opts.deckLabel}" (current directory). A separate conversation agent dispatched you with one task. Follow the deck's CLAUDE.md / AGENTS.md conventions, and reload the served deck after changes (\`npm run restart\`).${quickReference}${layout}${deckSource}
 
 Your task (id ${opts.taskId}): ${opts.title}
 

package/dist/agent.d.ts

@@ -2,8 +2,9 @@ import * as http from "http";
 import { Duplex } from "stream";
 export declare const AGENT_WS_PATH = "/__castle/agent";
 export declare const AGENT_ATTACHMENT_PREFIX = "/__castle/agent/attachments/";
-export type AgentBackend = "cursor" | "claude";
-export type ClaudeModel = "sonnet" | "opus" | "fable";
+export declare const AGENT_PLAYTEST_PREFIX = "/__castle/agent/playtest/";
+export type AgentBackend = "cursor" | "claude" | "smith";
+export type ClaudeModel = "sonnet" | "opus" | "fable" | "openrouter";
 type TaskStatus = "waiting" | "running" | "blocked" | "done" | "failed" | "interrupted";
 interface TaskRecord {
     id: string;
@@ -20,12 +21,14 @@ interface TaskRecord {
     pid?: number;
     originMessageId?: string;
     files?: string[];
+    playtestFrames?: string[];
     resultSummary?: string;
     avatar?: string;
     phase?: string;
     acknowledged?: boolean;
     rejected?: boolean;
     blockedBy?: string[];
+    suspectNoChanges?: boolean;
 }
 export interface DepsState {
     kind: "ready" | "waiting" | "blocked";
@@ -42,5 +45,7 @@ export interface AgentServer {
 export declare function createAgentServer(opts: {
     deckDir: string;
     deckLabel: string;
+    port?: number;
+    restart?: () => void;
 }): AgentServer;
 export {};