mini-coder 0.5.13 → 0.5.14

package/src/prompt.ts

@@ -251,7 +251,13 @@ function buildCorePrompt(opts: BuildSystemPromptOpts): string {
     "- Avoid creating extra files, systems or documentation outside of what was asked.",
     "- Check requirements, and plan your changes before editing code.",
     "- Implement the necessary changes, following good practices and proper error handling.",
+    "- Prefer the smallest path that leaves the requested end state already true; do not stop at helper scripts, instructions, or half-finished setup when the user asked for the live result itself.",
     "- Always verify your changes using compilation, testing, and manual verification when possible.",
+    "- Before you finish, re-check the explicit deliverables and current state. If the user named files, paths, ports, services, commands, or output values, make sure they already exist and work now.",
+    "- If the request includes structural constraints on files or outputs (for example allowed commands, required lines, exact formats, or counts), treat those as acceptance criteria too and verify them directly against what you produced, not just through downstream behavior.",
+    "- Treat concrete command sequences and expected outputs in the user's request as acceptance criteria for the end state. If you verify that flow during the task, do not roll the environment back afterward unless the user explicitly asked for a reset.",
+    "- If a check or tool result contradicts your expectation, trust the evidence and resolve the mismatch before you answer.",
+    "- When multiple outputs or end states seem plausible, do not guess or swap in a cleaner alternative after verification. Run the smallest check that distinguishes them, and if you change the state later, verify again.",
     "- When verifying with build or test commands, avoid leaving generated binaries or scratch artifacts in the requested output location; use temporary paths or remove them before finishing.",
     "- Do not leave helpers, tests, or any other form of temporary files; clean up after yourself and leave no trace.",
     "- Ensure you match the requested output exactly. This applies to file names, directory structure, number of files, output formats, and all other details.",
@@ -266,7 +272,9 @@ function buildCorePrompt(opts: BuildSystemPromptOpts): string {
     "- A todo item is only complete if the requested work is actually finished and verified to the degree the task requires.",
     "- Use `cancelled` to remove tasks that are no longer relevant.",
     "- Skip todo tools for single trivial tasks and purely conversational/informational requests.",
-    '- You have the option to delegate tasks to copies of yourself with `mc -p "subtask prompt"` in the shell.',
+    "- Use the `delegate` tool for bounded subtasks when another focused agent pass would help.",
+    "- Prefer `delegate` over shelling out to `mc -p` unless you specifically need to exercise the CLI itself.",
+    "- Do not re-delegate the whole task, spin on repeated self-review prompts, or ask a delegated child to delegate again.",
     "- Delegate when you are orchestrating a large to-do/plan execution.",
     "",
   );

package/src/session-message.ts

@@ -126,23 +126,61 @@ function isToolCallContentBlock(
   );
 }
 
-function isAssistantUsage(value: unknown): value is AssistantMessage["usage"] {
+/**
+ * Parse and validate an assistant usage payload from unknown input.
+ *
+ * @param value - Unknown value to validate.
+ * @returns The parsed assistant usage, or `null` when invalid.
+ */
+export function readAssistantUsage(
+  value: unknown,
+): AssistantMessage["usage"] | null {
   const usageRecord = toRecord(value);
   const costRecord = toRecord(usageRecord?.cost);
-  return (
-    usageRecord !== null &&
-    costRecord !== null &&
-    readFiniteNumber(usageRecord, "input") !== null &&
-    readFiniteNumber(usageRecord, "output") !== null &&
-    readFiniteNumber(usageRecord, "cacheRead") !== null &&
-    readFiniteNumber(usageRecord, "cacheWrite") !== null &&
-    readFiniteNumber(usageRecord, "totalTokens") !== null &&
-    readFiniteNumber(costRecord, "input") !== null &&
-    readFiniteNumber(costRecord, "output") !== null &&
-    readFiniteNumber(costRecord, "cacheRead") !== null &&
-    readFiniteNumber(costRecord, "cacheWrite") !== null &&
-    readFiniteNumber(costRecord, "total") !== null
-  );
+  if (!usageRecord || !costRecord) {
+    return null;
+  }
+
+  const input = readFiniteNumber(usageRecord, "input");
+  const output = readFiniteNumber(usageRecord, "output");
+  const cacheRead = readFiniteNumber(usageRecord, "cacheRead");
+  const cacheWrite = readFiniteNumber(usageRecord, "cacheWrite");
+  const totalTokens = readFiniteNumber(usageRecord, "totalTokens");
+  const costInput = readFiniteNumber(costRecord, "input");
+  const costOutput = readFiniteNumber(costRecord, "output");
+  const costCacheRead = readFiniteNumber(costRecord, "cacheRead");
+  const costCacheWrite = readFiniteNumber(costRecord, "cacheWrite");
+  const costTotal = readFiniteNumber(costRecord, "total");
+
+  if (
+    input === null ||
+    output === null ||
+    cacheRead === null ||
+    cacheWrite === null ||
+    totalTokens === null ||
+    costInput === null ||
+    costOutput === null ||
+    costCacheRead === null ||
+    costCacheWrite === null ||
+    costTotal === null
+  ) {
+    return null;
+  }
+
+  return {
+    input,
+    output,
+    cacheRead,
+    cacheWrite,
+    totalTokens,
+    cost: {
+      input: costInput,
+      output: costOutput,
+      cacheRead: costCacheRead,
+      cacheWrite: costCacheWrite,
+      total: costTotal,
+    },
+  };
 }
 
 function isStopReason(value: unknown): value is AssistantMessage["stopReason"] {
@@ -206,9 +244,9 @@ function parseAssistantMessageRecord(value: unknown): AssistantMessage | null {
     api,
     provider,
     model,
-    usage: isAssistantUsage(record.usage)
-      ? record.usage
-      : structuredClone(EMPTY_ASSISTANT_USAGE),
+    usage:
+      readAssistantUsage(record.usage) ??
+      structuredClone(EMPTY_ASSISTANT_USAGE),
     stopReason: record.stopReason,
     ...(errorMessage !== null ? { errorMessage } : {}),
     timestamp,
@@ -343,51 +381,5 @@ export function getAssistantUsage(
     return null;
   }
 
-  const messageRecord = toRecord(message);
-  const usageRecord = toRecord(messageRecord?.usage);
-  const costRecord = toRecord(usageRecord?.cost);
-  if (!usageRecord || !costRecord) {
-    return null;
-  }
-
-  const input = readFiniteNumber(usageRecord, "input");
-  const output = readFiniteNumber(usageRecord, "output");
-  const cacheRead = readFiniteNumber(usageRecord, "cacheRead");
-  const cacheWrite = readFiniteNumber(usageRecord, "cacheWrite");
-  const totalTokens = readFiniteNumber(usageRecord, "totalTokens");
-  const costInput = readFiniteNumber(costRecord, "input");
-  const costOutput = readFiniteNumber(costRecord, "output");
-  const costCacheRead = readFiniteNumber(costRecord, "cacheRead");
-  const costCacheWrite = readFiniteNumber(costRecord, "cacheWrite");
-  const costTotal = readFiniteNumber(costRecord, "total");
-
-  if (
-    input === null ||
-    output === null ||
-    cacheRead === null ||
-    cacheWrite === null ||
-    totalTokens === null ||
-    costInput === null ||
-    costOutput === null ||
-    costCacheRead === null ||
-    costCacheWrite === null ||
-    costTotal === null
-  ) {
-    return null;
-  }
-
-  return {
-    input,
-    output,
-    cacheRead,
-    cacheWrite,
-    totalTokens,
-    cost: {
-      input: costInput,
-      output: costOutput,
-      cacheRead: costCacheRead,
-      cacheWrite: costCacheWrite,
-      total: costTotal,
-    },
-  };
+  return readAssistantUsage(toRecord(message)?.usage);
 }