@farmslot/skills 0.1.1 → 0.1.3

package/CHANGELOG.md

@@ -4,6 +4,21 @@ All notable changes to `@farmslot/skills` are tracked here.
 
 ## Unreleased
 
+- Active-development baseline; add user-facing changes here before release or package publication.
+
+## 0.1.3 - 2026-07-08
+
+- Update recipe-quality skill guidance to delegate runtime artifact generation to `@farmslot/agent-runtime`
+- `packet`, a skill for packet-shaped operator communication aligned with ADR-048
+- Convert task lifecycle scripts to compatibility shims over `@farmslot/agent-runtime`
+
+## 0.1.2 - 2026-07-03
+
+- Colocate finish-contract scripts in `packages/skills/scripts/` (`mark-checklist-step.cjs`, `worker-terminal-contract.cjs`, `check-task-artifact-contract.mjs`) for the initial `@farmslot/skills` and consensys-skills delegation path
+- **dev / fix-bug:** terminal `./mark complete` requires `artifacts/learnings.md` + `artifacts/pr-description.md` (not `report.md`)
+- `./mark` infers dev/fix-bug flow from embedded checklist `Skill:` header (standalone recipe skills)
+- Export `./scripts/worker-terminal-contract.cjs` and `./scripts/check-task-artifact-contract.mjs` from the package
+
 ## 0.1.1 - 2026-06-30
 
 - Single-path mark CLI for worker terminal signals.

package/README.md

@@ -42,6 +42,7 @@ Supported layouts:
 | `recipe-doctor`  | Check local readiness for recipe authoring or execution.                                        |
 | `recipe-harness` | Help install or run the local recipe runtime when the project has one.                          |
 | `project-adopt`  | Recommend the smallest useful Farmslot integration level for the project.                       |
+| `packet`         | Communicate plans, blockers, decisions, and evidence reviews as compact operator packets.       |
 
 ## Adoption Ladder
 
@@ -55,13 +56,13 @@ Supported layouts:
 
 - `src/`: TypeScript CLI and install-library entry points.
 - `skills/`: Packaged skill directories copied into consumer repos.
-- `scripts/`: CommonJS recipe-cook runner utilities exposed through package bin scripts.
+- `scripts/`: Compatibility shims for legacy runtime paths. Runtime logic lives in `@farmslot/agent-runtime`.
 - `templates/`: Scenario fixtures and task templates used by the recipe-cook workflow.
 - `test/`: Node TAP-style smoke and behavior tests for installers and runner scripts.
 
 ## Maintenance rules
 
-- `scripts/mark-checklist-step.cjs` mirrors consensys-skills `recipe-harness/scripts/mark-checklist-step.cjs`. Edit the recipe-harness copy first; sync the mirror before shipping Farmslot gateway or npm changes.
+- `scripts/mark-checklist-step.cjs`, `scripts/worker-terminal-contract.cjs`, and `scripts/check-task-artifact-contract.mjs` are compatibility shims that delegate to `@farmslot/agent-runtime`.
 - Keep package payloads self-contained; consumers should not need the Farmslot monorepo.
 - Update `CHANGELOG.md` before packaging or publishing.
 - Run the package readiness guard when files move, bins change, or exports are added.

package/dist/index.d.ts

@@ -1,5 +1,5 @@
 export declare const FARMSLOT_SKILLS_PACKAGE = "@farmslot/skills";
-export declare const FARMSLOT_SKILL_NAMES: readonly ["recipe-cook", "recipe-quality", "recipe-doctor", "recipe-harness", "project-adopt"];
+export declare const FARMSLOT_SKILL_NAMES: readonly ["recipe-cook", "recipe-quality", "recipe-doctor", "recipe-harness", "project-adopt", "packet"];
 export type FarmslotSkillName = (typeof FARMSLOT_SKILL_NAMES)[number];
 export declare const FARMSLOT_SKILL_INSTALL_LAYOUTS: readonly ["agents", "claude", "codex", "cursor"];
 export type FarmslotSkillInstallLayout = (typeof FARMSLOT_SKILL_INSTALL_LAYOUTS)[number];

package/dist/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAAA,eAAO,MAAM,uBAAuB,qBAAqB,CAAC;AAE1D,eAAO,MAAM,oBAAoB,gGAMvB,CAAC;AAEX,MAAM,MAAM,iBAAiB,GAAG,CAAC,OAAO,oBAAoB,CAAC,CAAC,MAAM,CAAC,CAAC;AAEtE,eAAO,MAAM,8BAA8B,kDAAmD,CAAC;AAE/F,MAAM,MAAM,0BAA0B,GAAG,CAAC,OAAO,8BAA8B,CAAC,CAAC,MAAM,CAAC,CAAC"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAAA,eAAO,MAAM,uBAAuB,qBAAqB,CAAC;AAE1D,eAAO,MAAM,oBAAoB,0GAOvB,CAAC;AAEX,MAAM,MAAM,iBAAiB,GAAG,CAAC,OAAO,oBAAoB,CAAC,CAAC,MAAM,CAAC,CAAC;AAEtE,eAAO,MAAM,8BAA8B,kDAAmD,CAAC;AAE/F,MAAM,MAAM,0BAA0B,GAAG,CAAC,OAAO,8BAA8B,CAAC,CAAC,MAAM,CAAC,CAAC"}

package/dist/index.js

@@ -5,6 +5,7 @@ export const FARMSLOT_SKILL_NAMES = [
     'recipe-doctor',
     'recipe-harness',
     'project-adopt',
+    'packet',
 ];
 export const FARMSLOT_SKILL_INSTALL_LAYOUTS = ['agents', 'claude', 'codex', 'cursor'];
 //# sourceMappingURL=index.js.map

package/dist/index.js.map

@@ -1 +1 @@
-{"version":3,"file":"index.js","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAAA,MAAM,CAAC,MAAM,uBAAuB,GAAG,kBAAkB,CAAC;AAE1D,MAAM,CAAC,MAAM,oBAAoB,GAAG;IAClC,aAAa;IACb,gBAAgB;IAChB,eAAe;IACf,gBAAgB;IAChB,eAAe;CACP,CAAC;AAIX,MAAM,CAAC,MAAM,8BAA8B,GAAG,CAAC,QAAQ,EAAE,QAAQ,EAAE,OAAO,EAAE,QAAQ,CAAU,CAAC"}
+{"version":3,"file":"index.js","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAAA,MAAM,CAAC,MAAM,uBAAuB,GAAG,kBAAkB,CAAC;AAE1D,MAAM,CAAC,MAAM,oBAAoB,GAAG;IAClC,aAAa;IACb,gBAAgB;IAChB,eAAe;IACf,gBAAgB;IAChB,eAAe;IACf,QAAQ;CACA,CAAC;AAIX,MAAM,CAAC,MAAM,8BAA8B,GAAG,CAAC,QAAQ,EAAE,QAAQ,EAAE,OAAO,EAAE,QAAQ,CAAU,CAAC"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@farmslot/skills",
-  "version": "0.1.1",
+  "version": "0.1.3",
   "type": "module",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
@@ -16,7 +16,9 @@
     },
     "./skills/*": "./skills/*/SKILL.md",
     "./schemas/*": "./schemas/*",
-    "./scripts/mark-checklist-step.cjs": "./scripts/mark-checklist-step.cjs"
+    "./scripts/mark-checklist-step.cjs": "./scripts/mark-checklist-step.cjs",
+    "./scripts/worker-terminal-contract.cjs": "./scripts/worker-terminal-contract.cjs",
+    "./scripts/check-task-artifact-contract.mjs": "./scripts/check-task-artifact-contract.mjs"
   },
   "scripts": {
     "typecheck": "tsc --noEmit --project tsconfig.json",
@@ -30,6 +32,9 @@
     "prepack": "cd ../.. && node scripts/quality/check-farmslot-package-readiness.mjs --require-changelog --packages @farmslot/skills && yarn workspace @farmslot/skills build",
     "prepublishOnly": "node ../../scripts/quality/check-farmslot-package-readiness.mjs --publish --packages @farmslot/skills"
   },
+  "dependencies": {
+    "@farmslot/agent-runtime": "0.1.1"
+  },
   "devDependencies": {
     "@types/node": "^22.0.0",
     "eslint": "^10.3.0",
@@ -64,6 +69,7 @@
     "dist/**/*.js.map",
     "schemas/**/*.json",
     "scripts/**/*.cjs",
+    "scripts/**/*.mjs",
     "skills/**/*.md",
     "templates/**/*.json"
   ],

package/scripts/check-task-artifact-contract.mjs

@@ -0,0 +1,24 @@
+#!/usr/bin/env node
+import { spawnSync } from 'node:child_process';
+import { createRequire } from 'node:module';
+import path from 'node:path';
+import { fileURLToPath } from 'node:url';
+
+const requireFromHere = createRequire(import.meta.url);
+const packageRoot = path.resolve(path.dirname(fileURLToPath(import.meta.url)), '..');
+
+function resolveRuntimeScript() {
+  try {
+    return requireFromHere.resolve(
+      '@farmslot/agent-runtime/scripts/check-task-artifact-contract.mjs',
+    );
+  } catch (error) {
+    if (error.code !== 'MODULE_NOT_FOUND') throw error;
+    return path.resolve(packageRoot, '../agent-runtime/scripts/check-task-artifact-contract.mjs');
+  }
+}
+
+const result = spawnSync(process.execPath, [resolveRuntimeScript(), ...process.argv.slice(2)], {
+  stdio: 'inherit',
+});
+process.exit(result.status ?? 1);

package/scripts/mark-checklist-step.cjs

@@ -1,277 +1,18 @@
 #!/usr/bin/env node
-const fs = require('node:fs');
-const path = require('node:path');
-
-const START_COMMANDS = new Set(['start']);
-const TERMINAL_COMMANDS = new Set(['complete', 'no-change', 'blocked']);
-const NO_CHANGE_REPORT = path.join('artifacts', 'no-change-report.md');
-
-const usageLine =
-  'usage: mark <step-number|start|complete|no-change|blocked> [--reason text] [--already-fixed] [--mark-last] [--no-self-review]';
-
-function printHelp() {
-  console.log(
-    [
-      usageLine,
-      '',
-      'Bootstrap: ./mark start — worker-owned SIGNAL.json with status running (no checklist box).',
-      'Progress: ./mark 1, ./mark 2, ... — checks the box and appends checklistTiming.',
-      'Terminal:',
-      '  ./mark complete [--mark-last] [--no-self-review]',
-      '  ./mark no-change --reason "..." [--already-fixed] [--mark-last]',
-      '  ./mark blocked --reason "..." [--mark-last]',
-      'Do not write SIGNAL.json by hand or with echo.',
-    ].join('\n'),
-  );
-}
-
-function usage() {
-  console.error(usageLine);
-  console.error('run ./mark --help for details');
-  process.exit(2);
-}
-
-const args = process.argv.slice(2);
-if (args.length === 1 && (args[0] === '--help' || args[0] === '-h')) {
-  printHelp();
-  process.exit(0);
-}
-
-const [taskPath, signalPath, stepRaw, ...rest] = args;
-if (stepRaw === '--help' || stepRaw === '-h') {
-  printHelp();
-  process.exit(0);
-}
-if (!taskPath || !signalPath || !stepRaw) usage();
-
-const opts = {};
-for (let i = 0; i < rest.length; i += 1) {
-  const key = rest[i];
-  if (key === '--mark-last' || key === '--already-fixed' || key === '--no-self-review') {
-    opts[key.slice(2)] = true;
-    continue;
-  }
-  const value = rest[i + 1];
-  if (!key?.startsWith('--') || value === undefined) usage();
-  opts[key.slice(2)] = value;
-  i += 1;
-}
+'use strict';
 
-const stepToken = String(stepRaw).toLowerCase();
-const isStartCommand = START_COMMANDS.has(stepToken);
-const terminalCommand = TERMINAL_COMMANDS.has(stepToken) ? stepToken : null;
-let stepNumber = null;
-if (!isStartCommand && !terminalCommand) {
-  stepNumber = Number(stepRaw);
-  if (!Number.isInteger(stepNumber) || stepNumber < 1) usage();
-}
-
-if (isStartCommand && Object.keys(opts).length > 0) usage();
-if (terminalCommand === 'no-change' || terminalCommand === 'blocked') {
-  if (!opts.reason?.trim()) {
-    console.error(`${terminalCommand} requires --reason`);
-    process.exit(1);
-  }
-}
-
-const SIGNAL_PASSTHROUGH_KEYS = ['role', 'contextId', 'prNumber'];
-
-function pickSignalPassthrough(signal) {
-  const out = {};
-  for (const key of SIGNAL_PASSTHROUGH_KEYS) {
-    if (signal[key] !== undefined) out[key] = signal[key];
-  }
-  return out;
-}
-
-function resolveTerminalPreset(command) {
-  switch (command) {
-    case 'complete':
-      return { status: 'complete', outcome: 'success', disposition: 'fixed' };
-    case 'no-change':
-      return {
-        status: 'complete',
-        outcome: 'success',
-        disposition: opts['already-fixed'] ? 'already_fixed' : 'not_reproducible',
-      };
-    case 'blocked':
-      return { status: 'blocked', outcome: 'partial', disposition: 'blocked' };
-    default:
-      usage();
-      return null;
-  }
-}
-
-function assertNoChangeReport(taskDir) {
-  const reportAbs = path.join(taskDir, NO_CHANGE_REPORT);
-  if (!fs.existsSync(reportAbs) || fs.statSync(reportAbs).size === 0) {
-    console.error(`missing required report: ${NO_CHANGE_REPORT}`);
-    process.exit(1);
-  }
-  return NO_CHANGE_REPORT;
-}
-
-function buildSignalUpdate(signal, terminal, target, timing, events, now, taskPath, taskDir) {
-  const base = {
-    ...pickSignalPassthrough(signal),
-    step: target?.label ?? signal.step ?? 'complete',
-    checklistTiming: {
-      schemaVersion: 1,
-      source: timing.source || path.basename(taskPath),
-      events,
-    },
-    timestamp: now,
-  };
-
-  if (!terminal) {
-    return { ...base, status: 'running' };
-  }
-
-  const preset = resolveTerminalPreset(terminal.command);
-  const next = {
-    ...base,
-    status: preset.status,
-    outcome: preset.outcome,
-    disposition: preset.disposition,
-    ...(opts.reason ? { reason: opts.reason } : {}),
-    ...(terminal.command === 'complete' && opts['no-self-review']
-      ? { needsSelfReview: false }
-      : {}),
-  };
-
-  if (terminal.command === 'no-change') {
-    next.evidence = { reportPath: assertNoChangeReport(taskDir) };
-  }
-
-  return next;
-}
+const path = require('node:path');
+const { createRequire } = require('node:module');
 
-function atomicWrite(file, content, mode) {
-  const dir = path.dirname(file);
-  fs.mkdirSync(dir, { recursive: true });
-  const tmp = path.join(dir, `.${path.basename(file)}.${process.pid}.tmp`);
-  fs.writeFileSync(tmp, content, mode ? { mode } : undefined);
-  fs.renameSync(tmp, file);
-}
+const requireFromHere = createRequire(__filename);
 
-function readJson(file) {
+function resolveRuntimeScript() {
   try {
-    return JSON.parse(fs.readFileSync(file, 'utf8'));
-  } catch (err) {
-    if (err && err.code === 'ENOENT') return {};
-    throw err;
+    return requireFromHere.resolve('@farmslot/agent-runtime/scripts/mark-checklist-step.cjs');
+  } catch (error) {
+    if (error.code !== 'MODULE_NOT_FOUND') throw error;
+    return path.resolve(__dirname, '../../agent-runtime/scripts/mark-checklist-step.cjs');
   }
 }
 
-function stripLabel(raw) {
-  return raw
-    .replace(/^\*\*(.*?)\*\*\s*[—-]?\s*/, '$1 ')
-    .replace(/`([^`]+)`/g, '$1')
-    .replace(/\s+/g, ' ')
-    .trim();
-}
-
-function parseChecklist(markdown) {
-  const lines = markdown.split(/\n/);
-  const items = [];
-  let inFence = false;
-  let seen = 0;
-  for (let i = 0; i < lines.length; i += 1) {
-    if (/^\s*```/.test(lines[i])) {
-      inFence = !inFence;
-      continue;
-    }
-    if (inFence) continue;
-    const match = lines[i].match(/^(\s*[-*]\s+\[)( |x|X)(\]\s+)(.*)$/);
-    if (!match) continue;
-    seen += 1;
-    items.push({
-      lineIndex: i,
-      stepNumber: seen,
-      checked: match[2].toLowerCase() === 'x',
-      prefix: match[1],
-      suffix: match[3],
-      rawLabel: match[4],
-      label: stripLabel(match[4]),
-    });
-  }
-  return { lines, items };
-}
-
-function markStepInLines(lines, item) {
-  if (item.checked) return false;
-  lines[item.lineIndex] = `${item.prefix}x${item.suffix}${item.rawLabel}`;
-  return true;
-}
-
-function resolveTarget(taskPath, stepNumber, markLast) {
-  const original = fs.readFileSync(taskPath, 'utf8');
-  const parsed = parseChecklist(original);
-  let item = null;
-  if (stepNumber != null) {
-    item = parsed.items.find((entry) => entry.stepNumber === stepNumber) ?? null;
-    if (!item) {
-      console.error(`checklist step ${stepNumber} not found in ${taskPath}`);
-      process.exit(1);
-    }
-  } else if (markLast) {
-    const unchecked = parsed.items.filter((entry) => !entry.checked);
-    item = unchecked.length ? unchecked[unchecked.length - 1] : null;
-  }
-  let updated = original;
-  if (item) {
-    const nextLines = [...parsed.lines];
-    markStepInLines(nextLines, item);
-    updated = nextLines.join('\n');
-    if (updated !== original) atomicWrite(taskPath, updated);
-  }
-  return item
-    ? { stepNumber: item.stepNumber, label: item.label }
-    : { stepNumber: null, label: 'complete' };
-}
-
-const taskDir = path.dirname(signalPath);
-const target = isStartCommand
-  ? { stepNumber: null, label: 'started' }
-  : resolveTarget(taskPath, stepNumber, Boolean(terminalCommand && opts['mark-last']));
-
-const now = new Date().toISOString();
-const signal = readJson(signalPath);
-const timing =
-  signal.checklistTiming && typeof signal.checklistTiming === 'object'
-    ? signal.checklistTiming
-    : { schemaVersion: 1, source: path.basename(taskPath), events: [] };
-const events = Array.isArray(timing.events) ? [...timing.events] : [];
-
-if (
-  !isStartCommand &&
-  target.stepNumber != null &&
-  !events.some(
-    (event) =>
-      event && (event.stepNumber === target.stepNumber || event.index === target.stepNumber - 1),
-  )
-) {
-  events.push({ stepNumber: target.stepNumber, label: target.label, checkedAt: now });
-}
-
-const next = buildSignalUpdate(
-  signal,
-  terminalCommand ? { command: terminalCommand } : null,
-  target,
-  timing,
-  events,
-  now,
-  taskPath,
-  taskDir,
-);
-atomicWrite(signalPath, `${JSON.stringify(next, null, 2)}\n`, 0o644);
-
-if (isStartCommand) {
-  console.log('signal started');
-} else if (terminalCommand) {
-  console.log(
-    `signal ${terminalCommand}: status=${next.status} disposition=${next.disposition ?? 'n/a'}`,
-  );
-} else {
-  console.log(`marked ${target.stepNumber}: ${target.label}`);
-}
+require(resolveRuntimeScript());

package/scripts/worker-terminal-contract.cjs

@@ -0,0 +1,17 @@
+'use strict';
+
+const path = require('node:path');
+const { createRequire } = require('node:module');
+
+const requireFromHere = createRequire(__filename);
+
+function resolveRuntimeScript() {
+  try {
+    return requireFromHere.resolve('@farmslot/agent-runtime/scripts/worker-terminal-contract.cjs');
+  } catch (error) {
+    if (error.code !== 'MODULE_NOT_FOUND') throw error;
+    return path.resolve(__dirname, '../../agent-runtime/scripts/worker-terminal-contract.cjs');
+  }
+}
+
+module.exports = require(resolveRuntimeScript());

package/skills/packet/SKILL.md

@@ -0,0 +1,136 @@
+---
+name: packet
+description: Communicate complex agent output as compact, reviewable Farmslot operator packets with artifact anchors and safe action requests
+compatibility: Claude, Codex, Cursor, and Markdown skill runners with access to the working directory
+metadata:
+  package: '@farmslot/skills'
+  adr: ADR-048
+  outputs: artifacts/interactive/*.packet.json and optional sibling markdown when supported
+allowed-tools: Read Write Bash(rg:*) Bash(node:*) Bash(git:*)
+---
+
+# Interactive Operator Packets
+
+Use this skill when a plan, decision, blocker, comparison, or evidence review would be easier for the operator to inspect as a structured review packet than as terminal prose.
+
+This skill teaches the communication pattern. It does not assume Farmslot runtime support is installed. If the current project does not support Interactive Operator Packets yet, produce a concise markdown fallback and do not claim that Command Center or Companion can render it.
+
+## Use When
+
+- You need the operator to review a plan before implementation.
+- You need a decision with clear options and consequences.
+- You found a blocker and need targeted human input.
+- You want feedback anchored to a diff, artifact, screenshot, recipe result, or report section.
+- You are summarizing evidence and want the operator to approve, reject, or request a revision.
+
+## Do Not Use When
+
+- A one-sentence answer is enough.
+- The operator asked for raw terminal output.
+- The action would be unsafe without an existing Farmslot confirmation surface.
+- You would need to execute arbitrary HTML or JavaScript to make the packet useful.
+- The packet would only restate a normal `SIGNAL.json`, recipe manifest, or PR description.
+
+## Packet Contract
+
+When the project supports packets, write JSON to:
+
+```text
+artifacts/interactive/<short-id>.packet.json
+```
+
+Use this shape:
+
+```json
+{
+  "schema": "farmslot.interactive.operator-packet.v1",
+  "id": "review-plan-1",
+  "title": "Review implementation plan",
+  "intent": "plan",
+  "summary": "Confirm the scoped approach before changes start.",
+  "body": {
+    "format": "markdown",
+    "path": "artifacts/interactive/review-plan-1.md"
+  },
+  "anchors": [
+    {
+      "id": "diff-risk",
+      "label": "Risky diff area",
+      "artifactPath": "artifacts/diff.txt",
+      "line": 42
+    }
+  ],
+  "actions": [
+    {
+      "id": "approve",
+      "label": "Approve plan",
+      "kind": "terminal.send",
+      "safety": "operator-confirmed",
+      "payload": {
+        "text": "Approved packet review-plan-1. Proceed with the scoped plan."
+      }
+    }
+  ],
+  "createdAt": "2026-07-03T00:00:00.000Z"
+}
+```
+
+Keep `body.text` or the sibling markdown short. A packet is a review surface, not a report dump.
+
+## Writing Rules
+
+1. Start with the operator's next decision or review task.
+2. Use a short title and one-sentence summary.
+3. Pick one intent: `plan`, `decision`, `review`, `blocker`, `evidence`, or `comparison`.
+4. Anchor feedback to concrete artifacts when possible.
+5. Offer only actions the operator can understand and confirm.
+6. Print a terminal fallback after writing the packet.
+
+Terminal fallback format:
+
+```text
+Interactive packet ready: <title>
+Packet: artifacts/interactive/<short-id>.packet.json
+Fallback: artifacts/interactive/<short-id>.md
+```
+
+If Farmslot deep links are known to be supported, also print the route. Never encode a mutating action in the link.
+
+## Action Safety
+
+Allowed v1 action kinds:
+
+- `terminal.send` — send a concise approved response back to the worker.
+- `decision.resolve` — resolve an existing Farmslot decision.
+- `copy` — copy text for manual use.
+- `open-artifact` — open a referenced artifact.
+
+Rules:
+
+- Mark mutating actions as `operator-confirmed`.
+- Keep generated `terminal.send` text short and specific.
+- Do not hide shell commands inside action payloads.
+- Do not ask the UI to execute arbitrary RPC methods.
+- Do not treat a deep link as approval.
+
+## Fallback Mode
+
+If packet runtime support is absent:
+
+1. Write the same information as markdown in the conversation or artifact directory.
+2. Include anchors as plain file paths and line numbers.
+3. Ask for the operator response in normal chat/terminal.
+4. Do not say that a packet has been opened, rendered, or sent to Companion.
+
+## Response Handling
+
+When the operator responds to a packet, continue with the referenced packet id and anchor ids:
+
+```text
+Received response for packet review-plan-1:
+- action: approve
+- anchor: diff-risk
+- note: Keep the protocol typed; defer HTML.
+```
+
+Then act on the feedback through the normal task flow. The packet is a communication aid; the runner session remains the source of execution truth.

package/skills/recipe-quality/SKILL.md

@@ -43,3 +43,5 @@ Return:
 - weak or missing claims
 - evidence gaps
 - concrete recipe delta
+
+When the task requires `artifacts/recipe-quality.json`, do not hand-author the full schema. Provide the compact fields to `farmslot-agent recipe-quality build` (or `buildRecipeQualityArtifact()` from `@farmslot/agent-runtime`) so the artifact passes the `@farmslot/protocol` validator.