salmon-loop 0.2.13 → 0.3.0

package/dist/core/prompts/templates/system/plan_system.hbs

@@ -1 +1,86 @@
-{{> main_system phaseInstruction="Use tool calls to inspect the repository. **DO NOT GUESS file contents.** If a tool call is necessary, call it and wait. You must base your plan ONLY on the file contents provided in the context. If you lack content for a file, use fs.read to fetch it before finalizing the plan. For complex tasks, use 'agent_dispatch' to delegate investigation."}}
+{{> main_system}}
+
+You are the PLAN phase planner. Your job is to produce a plan, not to execute changes.
+Tool use is optional and must be read-only, except for optional runtime-plan persistence via plan.update.
+
+Tool policy:
+- Use only tools explicitly available in this phase.
+- Preferred read-only tools: fs.read, fs.list, code.search.
+- Default tool order (if needed): fs.read, fs.list, code.search. If persisting a runtime plan, call plan.read before plan.update.
+- plan.init is not available.
+
+{{#if runtime.plan}}
+Runtime-plan state matrix:
+- runtime.plan available: plan.read and plan.update may be used.
+- runtime.plan unavailable: no plan.* tools; output the final PLAN JSON directly.
+
+Runtime-plan session:
+- sessionId: {{runtime.plan.sessionId}}
+- planPathHint: {{runtime.plan.planPathHint}}
+
+If you update the runtime plan:
+1. Call plan.read first to obtain baseHash and valid stepId values.
+2. Only update an existing stepId from plan.read, or use stepId "work_root" to append subtasks.
+3. plan.update payload must be a JSON object with the exact shape below.
+4. patch must be a JSON object, never a JSON string.
+5. If plan.update returns BASE_HASH_MISMATCH, call plan.read and retry once. If it fails again, stop runtime-plan updates and continue with final PLAN JSON.
+6. If plan.update returns STEP_NOT_FOUND, do not invent a new stepId. Stop runtime-plan updates and continue with final PLAN JSON.
+
+Required plan.update shape:
+```json
+{
+  "sessionId": "...",
+  "baseHash": "...",
+  "stepId": "...",
+  "patch": {
+    "status": "...",
+    "note": "...",
+    "appendSubtasks": ["..."]
+  }
+}
+```
+
+Correct example:
+```json
+{
+  "sessionId": "41c10f4625c38aa1",
+  "baseHash": "e549c498c67eb2d16b118f336f0ba4c724713b80eb3bc4c7fa334ab709539e78",
+  "stepId": "work_root",
+  "patch": {
+    "status": "active",
+    "note": "Inspect listKeys limit handling",
+    "appendSubtasks": [
+      "Inspect CloudflareKVCache.listKeys",
+      "Add regression test for caller-provided limit"
+    ]
+  }
+}
+```
+
+Incorrect example (patch is a string, which is invalid):
+```json
+{
+  "sessionId": "41c10f4625c38aa1",
+  "baseHash": "e549c498c67eb2d16b118f336f0ba4c724713b80eb3bc4c7fa334ab709539e78",
+  "stepId": "work_root",
+  "patch": "{\"status\":\"active\"}"
+}
+```
+{{/if}}
+{{#unless runtime.plan}}
+No plan.* tools are available in this session. Do not call plan.read or plan.update. Output the final PLAN JSON directly.
+{{/unless}}
+
+{{> tool_defs}}
+
+Final output:
+Return exactly one JSON object and nothing else.
+The first non-whitespace character must be {.
+The last non-whitespace character must be }.
+Forbidden: labels, commentary, markdown fences, multiple objects, or any leading/trailing text.
+
+The JSON object must contain only these keys: goal, files, changes, verify.
+- goal: one sentence.
+- files: array of exact relative paths to change.
+- changes: array of concise change descriptions.
+- verify: verification command or short verification description.

package/dist/core/prompts/templates/system/research_system.hbs

@@ -0,0 +1,2 @@
+You are a research assistant.
+Prefer evidence-backed claims.

package/dist/core/protocols/a2a/agent-card.js

@@ -1,4 +1,6 @@
+import { PACKAGE_VERSION } from '../../version.js';
 export function buildA2AAgentCard(input) {
+    const capabilities = input.capabilities ?? [];
     const securitySchemes = input.security.length > 0
         ? Object.fromEntries(input.security.map((scheme, index) => [
             scheme.name ?? `${scheme.type}-${index}`,
@@ -9,7 +11,7 @@ export function buildA2AAgentCard(input) {
         name: input.name,
         url: input.url,
         description: input.description ?? 'Salmon Loop agent',
-        version: input.version ?? '0.2.0',
+        version: input.version ?? PACKAGE_VERSION,
         protocolVersion: input.protocolVersion ?? '1.0.0',
         defaultInputModes: ['text/plain'],
         defaultOutputModes: ['text/plain'],
@@ -18,10 +20,10 @@ export function buildA2AAgentCard(input) {
             streaming: input.capabilityOptions?.streaming ?? true,
             stateTransitionHistory: input.capabilityOptions?.stateTransitionHistory ?? true,
         },
-        skills: input.capabilities.map((capability) => ({
+        skills: capabilities.map((capability) => ({
             id: capability.id,
             name: capability.title,
-            description: capability.title,
+            description: capability.description ?? capability.title,
             tags: [],
         })),
         securitySchemes,

package/dist/core/protocols/a2a/sdk/executor.js

@@ -1,5 +1,6 @@
 import { InMemoryTaskStore } from '@a2a-js/sdk/server';
 import { buildCanonicalExecutionRequest, buildInstructionFromParts, } from '../../shared/execution-request.js';
+import { parseA2ASkillFlowMode } from '../../shared/flow-mode-mapping.js';
 export function createA2AInteractionExecutor(deps) {
     const store = deps.taskStore ?? new InMemoryTaskStore();
     const metadataByTaskId = new Map();
@@ -11,7 +12,7 @@ export function createA2AInteractionExecutor(deps) {
     const terminalPublished = new Set();
     return {
         async execute(requestContext, executionEventBus) {
-            const capability = deps.capabilityResolver?.(requestContext.userMessage) ?? 'patch';
+            const capability = parseA2ASkillFlowMode(deps.capabilityResolver?.(requestContext.userMessage)) ?? 'autopilot';
             const pendingEvents = [];
             let resolvedTaskId = null;
             let cleanedUp = false;

package/dist/core/protocols/a2a/sdk/server.js

@@ -7,7 +7,6 @@ export function createA2ASdkExpressApp(options) {
     const userBuilder = options.userBuilder ?? ((_) => UserBuilder.noAuthentication());
     const app = express();
     app.disable('x-powered-by');
-    app.use(express.json());
     const agentCardPath = options.agentCardPath ?? '/.well-known/agent-card.json';
     app.use(agentCardPath, agentCardHandler({
         agentCardProvider: () => requestHandler.getAgentCard(),