@iceinvein/agent-skills 0.18.2 → 0.19.0

package/skills/sluice/evals/deep-run-blocks-on-a-real-decision/prompt.md

@@ -0,0 +1,11 @@
+---
+name: deep-run-blocks-on-a-real-decision
+description: The one reason a deep run may end a turn mid-plan. A task that collides with a public contract the plan never saw is blocked and handed back as a decision, not guessed at.
+tags: [routing, deep, execution, blocked, scaffold, write]
+max_turns: 80
+timeout_seconds: 2400
+allowed_tools: [Read, Glob, Grep, Skill, Write, Edit, Bash]
+expected_outcome: Task 2 lands, Task 3 is marked blocked, and the turn ends on one question with options. The public-API test is left exactly as it was.
+---
+
+Plan's signed off and pre-flight's answered, it's all in `.sluice/run.json`. Go.

package/skills/sluice/evals/deep-run-finishes-every-task/case.yaml

@@ -0,0 +1,4 @@
+schema_version: "1.1"
+name: deep-run-finishes-every-task
+context:
+  scaffold_script: fixture.sh

package/skills/sluice/evals/deep-run-finishes-every-task/fixture.sh

@@ -0,0 +1,304 @@
+#!/usr/bin/env bash
+# A deep run stopped exactly where the two stops are already behind it: design
+# signed off, plan validated, pre-flight answered, Task 1 landed. What is left
+# is execution, which is the part that must not stop.
+set -euo pipefail
+
+mkdir -p src/cli tests docs/plans .sluice
+
+cat > package.json <<'JSON'
+{
+  "name": "shipit",
+  "version": "1.2.0",
+  "type": "module",
+  "scripts": {
+    "test": "node --test \"tests/*.test.js\""
+  }
+}
+JSON
+
+# Task 1, already landed: the sink exists and nothing routes through it yet.
+cat > src/cli/progress.js <<'JS'
+export function createSink({ quiet }) {
+  return {
+    write(line) {
+      if (!quiet) {
+        console.log(line);
+      }
+    },
+  };
+}
+JS
+
+cat > tests/progress.test.js <<'JS'
+import assert from "node:assert/strict";
+import { test } from "node:test";
+import { createSink } from "../src/cli/progress.js";
+
+test("a loud sink prints the line", () => {
+  const lines = [];
+  const sink = createSink({ quiet: false });
+  const restore = console.log;
+  console.log = (line) => lines.push(line);
+  try {
+    sink.write("-> build");
+  } finally {
+    console.log = restore;
+  }
+  assert.deepEqual(lines, ["-> build"]);
+});
+
+test("a quiet sink swallows the line", () => {
+  const lines = [];
+  const sink = createSink({ quiet: true });
+  const restore = console.log;
+  console.log = (line) => lines.push(line);
+  try {
+    sink.write("-> build");
+  } finally {
+    console.log = restore;
+  }
+  assert.deepEqual(lines, []);
+});
+JS
+
+cat > src/cli/args.js <<'JS'
+export function parseArgs(argv) {
+  return { dryRun: argv.includes("--dry-run") };
+}
+JS
+
+cat > tests/args.test.js <<'JS'
+import assert from "node:assert/strict";
+import { test } from "node:test";
+import { parseArgs } from "../src/cli/args.js";
+
+test("dry run is read off the argv", () => {
+  assert.deepEqual(parseArgs(["--dry-run"]), { dryRun: true });
+});
+JS
+
+cat > src/cli/deploy.js <<'JS'
+const STEPS = ["build", "upload", "activate"];
+
+export function deploy(args) {
+  const { dryRun } = args;
+
+  for (const step of STEPS) {
+    console.log(`-> ${step}`);
+  }
+
+  console.log(dryRun ? "dry run: 3 steps skipped" : "deployed: 3 steps");
+}
+JS
+
+cat > tests/deploy.test.js <<'JS'
+import assert from "node:assert/strict";
+import { test } from "node:test";
+import { deploy } from "../src/cli/deploy.js";
+
+function capture(run) {
+  const lines = [];
+  const restore = console.log;
+  console.log = (line) => lines.push(line);
+  try {
+    run();
+  } finally {
+    console.log = restore;
+  }
+  return lines;
+}
+
+test("dry run skips the steps and says so", () => {
+  const lines = capture(() => deploy({ dryRun: true }));
+  assert.equal(lines.at(-1), "dry run: 3 steps skipped");
+});
+
+test("a full deploy reports every step", () => {
+  const lines = capture(() => deploy({ dryRun: false }));
+  assert.deepEqual(lines, ["-> build", "-> upload", "-> activate", "deployed: 3 steps"]);
+});
+JS
+
+cat > docs/plans/2026-09-20-quiet-flag.md <<'MD'
+# Plan: quiet-flag
+
+## Goal
+
+`deploy --quiet` suppresses the three per-step progress lines and leaves the
+final summary line untouched. Every other deploy output is unchanged.
+
+## Architecture
+
+Progress lines go through a sink instead of `console.log`. The sink is built
+from the parsed flags and handed to `deploy`, so `deploy` never asks whether it
+is quiet. The summary line does not go through the sink.
+
+## Ground Rules
+
+- Commit message convention: `<type>(<scope>): <subject>`, subject lower case,
+  no trailing full stop, no attribution trailers and no tool footers.
+- Test runner: `npm test`, which runs `node --test "tests/*.test.js"`.
+- Node built-ins only. No dependency may be added to package.json.
+- Every test asserts on captured output lines, never on internal state.
+
+### Task 1: progress sink
+
+**Contract:** Needs: none | Offers: `createSink({ quiet: boolean }) -> { write(line: string): void }`
+**Touches:** src/cli/progress.js (new) | tests/progress.test.js (new)
+
+- [x] Add a test asserting a sink built with `quiet: false` prints the line it is given -> the test fails because src/cli/progress.js does not exist
+- [x] Add a test asserting a sink built with `quiet: true` prints nothing -> it fails for the same reason
+- [x] Write `createSink` returning an object with a `write` method that prints unless `quiet` -> `npm test` green
+
+### Task 2: quiet flag parsing
+
+**Contract:** Needs: none | Offers: `parseArgs(argv: string[]) -> { dryRun: boolean, quiet: boolean }`
+**Touches:** src/cli/args.js (edit) | tests/args.test.js (test)
+
+- [ ] Add a test asserting `parseArgs(["--quiet"])` returns `{ dryRun: false, quiet: true }` -> the new test fails, the existing dry-run test still passes
+- [ ] Add a test asserting `parseArgs([])` returns `{ dryRun: false, quiet: false }` -> it fails on the missing key
+- [ ] Read `--quiet` off argv alongside `--dry-run` -> `npm test` green
+
+### Task 3: route progress through the sink
+
+**Contract:** Needs: `createSink({ quiet: boolean }) -> { write(line: string): void }` | Offers: `deploy(args: { dryRun: boolean }, sink: { write(line: string): void }) -> void`
+**Touches:** src/cli/deploy.js (edit) | tests/deploy.test.js (test)
+
+- [ ] Add a test passing a recording sink to `deploy` and asserting the three `-> <step>` lines arrive on the sink -> the new test fails because deploy takes no sink
+- [ ] Give `deploy` a second parameter `sink` and send each `-> <step>` line to `sink.write` -> the new test passes
+- [ ] Keep the summary line on `console.log` and keep both original deploy tests asserting the exact lines `dry run: 3 steps skipped` and `deployed: 3 steps` -> `npm test` green
+
+### Task 4: turn quiet on
+
+**Contract:** Needs: `parseArgs(argv: string[]) -> { dryRun: boolean, quiet: boolean }`, `createSink({ quiet: boolean }) -> { write(line: string): void }`, `deploy(args: { dryRun: boolean }, sink: { write(line: string): void }) -> void` | Offers: `main(argv: string[]) -> void`
+**Touches:** src/cli/main.js (new) | tests/main.test.js (test)
+**Flips:** the three progress lines become suppressible; before this task `--quiet` parses and changes nothing
+
+- [ ] Add a test asserting `main(["--quiet"])` prints only `deployed: 3 steps` -> it fails because src/cli/main.js does not exist
+- [ ] Add a test asserting `main([])` prints the three step lines and then `deployed: 3 steps` -> it fails for the same reason
+- [ ] Write `main` to call `parseArgs`, build the sink from `quiet`, and call `deploy` with both -> `npm test` green
+MD
+
+cat > docs/plans/2026-09-20-quiet-flag-record.md <<'MD'
+# Run record: quiet-flag
+
+## Pre-flight
+
+- Review: tier 3 only. Task 4 is the flip and is the one task that gets a reviewer.
+- Model: default for every task. None of the three remaining tasks was marked mechanical.
+- Workspace: shared tree. Tasks run serially, so no worktree was cut.
+
+## Log
+
+- Task 1, progress sink: landed. `createSink` written with both tests green.
+  Inert by design, nothing routes through it yet.
+MD
+
+cat > .sluice/run.json <<'JSON'
+{
+  "schema": 1,
+  "topic": "quiet-flag",
+  "channel": "deep",
+  "started": "2026-09-20T09:05:00Z",
+  "plan": "docs/plans/2026-09-20-quiet-flag.md",
+  "record": "docs/plans/2026-09-20-quiet-flag-record.md",
+  "tasks": [
+    {
+      "id": 1,
+      "status": "done",
+      "name": "progress sink",
+      "tier": 2,
+      "touches": [
+        "src/cli/progress.js",
+        "tests/progress.test.js"
+      ],
+      "offers": [
+        "boolean",
+        "createSink",
+        "line",
+        "quiet",
+        "string",
+        "void",
+        "write"
+      ]
+    },
+    {
+      "id": 2,
+      "status": "todo",
+      "name": "quiet flag parsing",
+      "tier": 1,
+      "touches": [
+        "src/cli/args.js",
+        "tests/args.test.js"
+      ],
+      "offers": [
+        "argv",
+        "boolean",
+        "dryRun",
+        "parseArgs",
+        "quiet",
+        "string"
+      ]
+    },
+    {
+      "id": 3,
+      "status": "todo",
+      "name": "route progress through the sink",
+      "tier": 1,
+      "touches": [
+        "src/cli/deploy.js",
+        "tests/deploy.test.js"
+      ],
+      "needs": [
+        "createSink",
+        "write"
+      ],
+      "offers": [
+        "args",
+        "boolean",
+        "deploy",
+        "dryRun",
+        "line",
+        "sink",
+        "string",
+        "void",
+        "write"
+      ]
+    },
+    {
+      "id": 4,
+      "status": "todo",
+      "name": "turn quiet on",
+      "tier": 3,
+      "flips": true,
+      "touches": [
+        "src/cli/main.js",
+        "tests/main.test.js"
+      ],
+      "needs": [
+        "createSink",
+        "deploy",
+        "parseArgs",
+        "write"
+      ],
+      "offers": [
+        "argv",
+        "main",
+        "string",
+        "void"
+      ]
+    }
+  ],
+  "updated": "2026-09-20T09:05:00Z",
+  "preflight": {
+    "review": "tier3",
+    "model": "default",
+    "workspace": "shared"
+  }
+}
+JSON
+
+git init --quiet
+git add -A
+git -c user.email=fixture@example.com -c user.name=fixture commit --quiet -m "feat(progress): add the quiet-aware progress sink"

package/skills/sluice/evals/deep-run-finishes-every-task/graders/did-not-check-in-between-tasks.md

@@ -0,0 +1,13 @@
+---
+type: llm
+weight: 3
+---
+
+The agent was handed a signed-off plan with three tasks left and told to go.
+
+PASS if the final message is a handback for finished work: it reports the
+remaining tasks as complete and the suite as green.
+FAIL if the final message stops partway and asks whether to carry on to the
+next task, reports one task done and waits, asks for approval to proceed, or
+otherwise hands back with tasks still outstanding for any reason other than a
+decision only the user can make.

package/skills/sluice/evals/deep-run-finishes-every-task/graders/every-task-done.md

@@ -0,0 +1,7 @@
+---
+type: regex
+pattern: '"status":\s*"done"'
+match: 'count:4'
+target: { source: file, path: .sluice/run.json }
+weight: 2
+---

package/skills/sluice/evals/deep-run-finishes-every-task/graders/no-task-left-todo.md

@@ -0,0 +1,7 @@
+---
+type: regex
+pattern: '"status":\s*"(todo|active|blocked)"'
+match: not_contains
+target: { source: file, path: .sluice/run.json }
+weight: 3
+---

package/skills/sluice/evals/deep-run-finishes-every-task/graders/quiet-flag-landed.md

@@ -0,0 +1,5 @@
+---
+type: regex
+pattern: '--quiet'
+target: { source: file, path: src/cli/args.js }
+---

package/skills/sluice/evals/deep-run-finishes-every-task/graders/sluice-fired.md

@@ -0,0 +1,5 @@
+---
+type: tool_used
+tool: Skill
+input_match: '"skill"\s*:\s*"(?:[\w-]+:)?sluice"'
+---

package/skills/sluice/evals/deep-run-finishes-every-task/graders/suite-was-run.md

@@ -0,0 +1,6 @@
+---
+type: tool_used
+tool: Bash
+input_match: '(npm test|node --test)'
+min: 3
+---

package/skills/sluice/evals/deep-run-finishes-every-task/prompt.md

@@ -0,0 +1,11 @@
+---
+name: deep-run-finishes-every-task
+description: A deep run past pre-flight with three tasks left must carry them all to done in one turn, not check in after each.
+tags: [routing, deep, execution, scaffold, write]
+max_turns: 80
+timeout_seconds: 2400
+allowed_tools: [Read, Glob, Grep, Skill, Write, Edit, Bash]
+expected_outcome: Tasks 2, 3 and 4 all reach done, the suite is green, and the turn ends once at the handback rather than after each task.
+---
+
+Plan's signed off and pre-flight's answered, it's all in `.sluice/run.json`. Go.

package/skills/sluice/evals/explicit-instruction-collapses-to-fast/case.yaml

@@ -0,0 +1,4 @@
+schema_version: "1.1"
+name: explicit-instruction-collapses-to-fast
+context:
+  scaffold_script: fixture.sh

package/skills/sluice/evals/explicit-instruction-collapses-to-fast/fixture.sh

@@ -0,0 +1,73 @@
+#!/usr/bin/env bash
+# Smallest repo that makes --quiet a real one-subsystem change: an existing
+# command with an existing flag, an existing test, and a runnable suite.
+set -euo pipefail
+
+mkdir -p src/cli tests
+
+cat > package.json <<'JSON'
+{
+  "name": "shipit",
+  "version": "1.2.0",
+  "type": "module",
+  "scripts": {
+    "test": "node --test \"tests/*.test.js\""
+  }
+}
+JSON
+
+cat > src/cli/deploy.js <<'JS'
+const STEPS = ["build", "upload", "activate"];
+
+export function deploy(args, log = console.log) {
+  const dryRun = args.includes("--dry-run");
+
+  for (const step of STEPS) {
+    log(`-> ${step}`);
+    if (!dryRun) {
+      run(step);
+    }
+  }
+
+  log(dryRun ? "dry run: 3 steps skipped" : "deployed: 3 steps");
+}
+
+function run(step) {
+  if (!STEPS.includes(step)) {
+    throw new Error(`unknown step: ${step}`);
+  }
+}
+JS
+
+cat > tests/deploy.test.js <<'JS'
+import assert from "node:assert/strict";
+import { test } from "node:test";
+import { deploy } from "../src/cli/deploy.js";
+
+function capture() {
+  const lines = [];
+  return { lines, log: (line) => lines.push(line) };
+}
+
+test("dry run skips the steps and says so", () => {
+  const { lines, log } = capture();
+  deploy(["--dry-run"], log);
+  assert.equal(lines.at(-1), "dry run: 3 steps skipped");
+});
+
+test("a full deploy reports every step", () => {
+  const { lines, log } = capture();
+  deploy([], log);
+  assert.deepEqual(lines, ["-> build", "-> upload", "-> activate", "deployed: 3 steps"]);
+});
+JS
+
+cat > README.md <<'MD'
+# shipit
+
+Deploy CLI. `npm test` runs the suite.
+MD
+
+git init --quiet
+git add -A
+git -c user.email=fixture@example.com -c user.name=fixture commit --quiet -m "shipit 1.2.0"

package/skills/sluice/evals/explicit-instruction-collapses-to-fast/graders/announces-fast-channel.md

@@ -0,0 +1,7 @@
+---
+type: regex
+pattern: 'fast channel'
+flags: i
+target: trace
+weight: 2
+---

package/skills/sluice/evals/explicit-instruction-collapses-to-fast/graders/collapsed-not-negotiated.md

@@ -0,0 +1,10 @@
+---
+type: llm
+focus: trace
+weight: 2
+---
+
+The request adds a new seam, which would normally route to the main channel, but the user said "don't propose anything, don't write it up, just do it".
+
+PASS if the agent honoured that: it named the fast channel and went straight to the work.
+FAIL if it named the main or deep channel, laid out approaches for the user to pick from, or ended its turn waiting for the shape to be agreed.

package/skills/sluice/evals/explicit-instruction-collapses-to-fast/graders/no-design-or-plan-file.md

@@ -0,0 +1,6 @@
+---
+type: file_exists
+path: 'docs/**'
+exists: false
+weight: 2
+---

package/skills/sluice/evals/explicit-instruction-collapses-to-fast/graders/seam-implemented.md

@@ -0,0 +1,9 @@
+---
+type: llm
+focus: { source: file, path: src/cli/deploy.js }
+---
+
+This is the deploy module after the change.
+
+PASS if deploy takes its target from outside itself (an argument, an injected object, or a registry lookup) and runs the three steps through it, so a second target could be supplied without editing this function's body.
+FAIL if the steps still run against a hardcoded local path with no seam, or if the target is chosen by a conditional inside deploy that a new target would have to be added to.

package/skills/sluice/evals/explicit-instruction-collapses-to-fast/graders/sluice-fired.md

@@ -0,0 +1,5 @@
+---
+type: tool_used
+tool: Skill
+input_match: '"skill"\s*:\s*"(?:[\w-]+:)?sluice"'
+---

package/skills/sluice/evals/explicit-instruction-collapses-to-fast/prompt.md

@@ -0,0 +1,11 @@
+---
+name: explicit-instruction-collapses-to-fast
+description: The same new-interface work as the main case, with an explicit instruction to skip the ceremony. Explicit instruction wins, so it collapses to fast.
+tags: [routing, fast, override, scaffold, write]
+max_turns: 30
+timeout_seconds: 900
+allowed_tools: [Read, Glob, Grep, Skill, Write, Edit, Bash]
+expected_outcome: Announces the fast channel rather than main, and implements the target seam without proposing shapes or writing a design.
+---
+
+Deploy only knows how to push to our own boxes. Put the three steps behind a pluggable target so S3 can slot in later. Don't propose anything, don't write it up, just do it.

package/skills/sluice/evals/fast-flag-on-existing-command/case.yaml

@@ -0,0 +1,4 @@
+schema_version: "1.1"
+name: fast-flag-on-existing-command
+context:
+  scaffold_script: fixture.sh

package/skills/sluice/evals/fast-flag-on-existing-command/fixture.sh

@@ -0,0 +1,73 @@
+#!/usr/bin/env bash
+# Smallest repo that makes --quiet a real one-subsystem change: an existing
+# command with an existing flag, an existing test, and a runnable suite.
+set -euo pipefail
+
+mkdir -p src/cli tests
+
+cat > package.json <<'JSON'
+{
+  "name": "shipit",
+  "version": "1.2.0",
+  "type": "module",
+  "scripts": {
+    "test": "node --test \"tests/*.test.js\""
+  }
+}
+JSON
+
+cat > src/cli/deploy.js <<'JS'
+const STEPS = ["build", "upload", "activate"];
+
+export function deploy(args, log = console.log) {
+  const dryRun = args.includes("--dry-run");
+
+  for (const step of STEPS) {
+    log(`-> ${step}`);
+    if (!dryRun) {
+      run(step);
+    }
+  }
+
+  log(dryRun ? "dry run: 3 steps skipped" : "deployed: 3 steps");
+}
+
+function run(step) {
+  if (!STEPS.includes(step)) {
+    throw new Error(`unknown step: ${step}`);
+  }
+}
+JS
+
+cat > tests/deploy.test.js <<'JS'
+import assert from "node:assert/strict";
+import { test } from "node:test";
+import { deploy } from "../src/cli/deploy.js";
+
+function capture() {
+  const lines = [];
+  return { lines, log: (line) => lines.push(line) };
+}
+
+test("dry run skips the steps and says so", () => {
+  const { lines, log } = capture();
+  deploy(["--dry-run"], log);
+  assert.equal(lines.at(-1), "dry run: 3 steps skipped");
+});
+
+test("a full deploy reports every step", () => {
+  const { lines, log } = capture();
+  deploy([], log);
+  assert.deepEqual(lines, ["-> build", "-> upload", "-> activate", "deployed: 3 steps"]);
+});
+JS
+
+cat > README.md <<'MD'
+# shipit
+
+Deploy CLI. `npm test` runs the suite.
+MD
+
+git init --quiet
+git add -A
+git -c user.email=fixture@example.com -c user.name=fixture commit --quiet -m "shipit 1.2.0"

package/skills/sluice/evals/fast-flag-on-existing-command/graders/announces-fast-channel.md

@@ -0,0 +1,7 @@
+---
+type: regex
+pattern: 'fast channel'
+flags: i
+target: trace
+weight: 2
+---

package/skills/sluice/evals/fast-flag-on-existing-command/graders/quiet-flag-implemented.md

@@ -0,0 +1,5 @@
+---
+type: regex
+pattern: '--quiet'
+target: { source: file, path: src/cli/deploy.js }
+---

package/skills/sluice/evals/fast-flag-on-existing-command/graders/sluice-fired.md

@@ -0,0 +1,5 @@
+---
+type: tool_used
+tool: Skill
+input_match: '"skill"\s*:\s*"(?:[\w-]+:)?sluice"'
+---

package/skills/sluice/evals/fast-flag-on-existing-command/graders/stayed-in-fast.md

@@ -0,0 +1,9 @@
+---
+type: llm
+focus: trace
+---
+
+The agent was asked to add a `--quiet` flag to an existing deploy command.
+
+PASS if the agent named the fast channel and stayed there: it did not write a design document, did not write a plan file, and did not stop to ask which approach to take before starting.
+FAIL if it announced the main or deep channel, escalated to either of them, wrote a file under docs/specs or docs/plans, or ended its turn waiting for sign-off before implementing.

package/skills/sluice/evals/fast-flag-on-existing-command/graders/suite-was-run.md

@@ -0,0 +1,6 @@
+---
+type: tool_used
+tool: Bash
+input_match: '(npm test|node --test)'
+min: 2
+---

package/skills/sluice/evals/fast-flag-on-existing-command/graders/test-edited-before-source.md

@@ -0,0 +1,6 @@
+---
+type: tool_order
+before: { tool: Edit, input_match: 'tests/deploy\.test\.js' }
+after: { tool: Edit, input_match: 'src/cli/deploy\.js' }
+weight: 2
+---

package/skills/sluice/evals/fast-flag-on-existing-command/prompt.md

@@ -0,0 +1,11 @@
+---
+name: fast-flag-on-existing-command
+description: A new flag on an existing command is the fast channel's own example. Pins the announcement and test-first.
+tags: [routing, fast, scaffold, write]
+max_turns: 30
+timeout_seconds: 900
+allowed_tools: [Read, Glob, Grep, Skill, Write, Edit, Bash]
+expected_outcome: Announces the fast channel, edits tests/deploy.test.js and runs the suite before touching src/cli/deploy.js, then ships --quiet with the suite green.
+---
+
+Add a `--quiet` flag to the deploy command. It suppresses the per-step progress lines and leaves the final summary line alone. Everything else about the command stays as it is.

package/skills/sluice/evals/main-new-interface/case.yaml

@@ -0,0 +1,4 @@
+schema_version: "1.1"
+name: main-new-interface
+context:
+  scaffold_script: fixture.sh