claude-dev-env 1.52.1 → 1.54.0

package/skills/autoconverge/workflow/converge.run-input.test.mjs

@@ -0,0 +1,81 @@
+import { test } from 'node:test';
+import { strict as assert } from 'node:assert';
+import { readFileSync } from 'node:fs';
+import { fileURLToPath } from 'node:url';
+import { dirname, join } from 'node:path';
+
+const workflowDirectory = dirname(fileURLToPath(import.meta.url));
+const convergeSource = readFileSync(join(workflowDirectory, 'converge.mjs'), 'utf8');
+
+function sourceSliceBetween(startNeedle, endNeedle) {
+  const sliceStart = convergeSource.indexOf(startNeedle);
+  assert.notEqual(sliceStart, -1, `expected ${startNeedle} to exist`);
+  const sliceEnd = convergeSource.indexOf(endNeedle, sliceStart + startNeedle.length);
+  assert.notEqual(sliceEnd, -1, `expected ${endNeedle} to exist after ${startNeedle}`);
+  return convergeSource.slice(sliceStart, sliceEnd);
+}
+
+const productionModule = new Function(
+  `${sourceSliceBetween('function normalizeRunInput(', '\nconst runInput =')}\n` +
+    'return { normalizeRunInput, classifyRunInput };',
+)();
+
+const { normalizeRunInput, classifyRunInput } = productionModule;
+
+const VALID_COORDINATES = { owner: 'jl-cmd', repo: 'claude-code-config', prNumber: 543 };
+
+test('an object payload passes through unchanged', () => {
+  assert.deepEqual(normalizeRunInput(VALID_COORDINATES), VALID_COORDINATES);
+});
+
+test('a JSON-encoded string payload is parsed into coordinates', () => {
+  assert.deepEqual(normalizeRunInput(JSON.stringify(VALID_COORDINATES)), VALID_COORDINATES);
+});
+
+test('a non-JSON string returns null rather than throwing', () => {
+  assert.equal(normalizeRunInput('not json at all'), null);
+});
+
+test('an empty string returns null rather than throwing', () => {
+  assert.equal(normalizeRunInput(''), null);
+});
+
+test('classifyRunInput accepts coordinates carrying owner, repo, and prNumber', () => {
+  const classified = classifyRunInput(VALID_COORDINATES);
+  assert.deepEqual(classified.input, VALID_COORDINATES);
+  assert.equal(classified.blocker, null);
+});
+
+test('classifyRunInput blocks a failed parse with a structured blocker and no input', () => {
+  const classified = classifyRunInput('not json at all');
+  assert.equal(classified.input, null);
+  assert.match(classified.blocker, /coordinates/i);
+});
+
+test('classifyRunInput blocks a null payload with a structured blocker', () => {
+  const classified = classifyRunInput(null);
+  assert.equal(classified.input, null);
+  assert.match(classified.blocker, /coordinates/i);
+});
+
+test('classifyRunInput blocks a payload missing owner', () => {
+  const classified = classifyRunInput({ repo: 'claude-code-config', prNumber: 543 });
+  assert.equal(classified.input, null);
+  assert.match(classified.blocker, /coordinates/i);
+});
+
+test('classifyRunInput blocks a payload missing prNumber', () => {
+  const classified = classifyRunInput({ owner: 'jl-cmd', repo: 'claude-code-config' });
+  assert.equal(classified.input, null);
+  assert.match(classified.blocker, /coordinates/i);
+});
+
+test('the top-level run guards an unusable input into a structured blocker before reading input.owner', () => {
+  const guardBlock = convergeSource.slice(
+    convergeSource.indexOf('const runInput = classifyRunInput('),
+    convergeSource.indexOf('const prCoordinates ='),
+  );
+  assert.match(guardBlock, /runInput\.blocker/);
+  assert.match(guardBlock, /converged: false/);
+  assert.match(guardBlock, /return/);
+});

package/skills/bugteam/SKILL.md

@@ -2,7 +2,7 @@
 name: bugteam
 description: >-
   Open pull request audit–fix until convergence: CODE_RULES gate, clean-room
-  audit (`code-quality-agent`, opus) and fix (`clean-coder`, opus), per-loop
+  audit (`code-quality-agent`, opus) and fix (`clean-coder`, fable), per-loop
   GitHub reviews, 20-audit cap; grant then revoke `.claude/**`. Spawns
   background subagents (`Agent(..., run_in_background=true)`). Triggers: '/bugteam', 'run
   the bug team', 'auto-fix the PR until clean', 'loop audit and fix'.

package/skills/bugteam/reference/audit-and-teammates.md

@@ -144,7 +144,7 @@ Spawn:
 Agent(
   subagent_type="clean-coder",
   name="bugfix-pr<N>-loop<L>",
-  model="opus",
+  model="fable",
   mode="bypassPermissions",
   run_in_background=true,
   description="Bugfix PR <N> loop <L>",

package/skills/bugteam/reference/team-setup.md

@@ -110,7 +110,7 @@ truth.
 
 - **Subagent roles (spawned per loop, not at invocation start):**
   - `bugfind` — `code-quality-agent`, model opus (Opus 4.7 at default xhigh effort)
-  - `bugfix` — `clean-coder`, model opus (Opus 4.7 at default xhigh effort)
+  - `bugfix` — `clean-coder`, model fable
 
 ### Loop state block
 

package/skills/qbug/SKILL.md

@@ -107,7 +107,7 @@ Then call `Agent` twice in the same message — the primary clean-coder and the
 ```
 Agent(
   subagent_type="clean-coder",
-  model="opus",
+  model="fable",
   description="qbug primary audit/fix cycle for PR <number>",
   prompt="<filled cycle XML; see § Subagent cycle prompt>",
   run_in_background=False

package/skills/qbug/test_qbug_skill_audit_schema.py

@@ -113,10 +113,10 @@ def test_contract_should_require_files_opened_in_proof_of_absence() -> None:
     )
 
 
-def test_step2_spawn_should_include_model_opus_parameter() -> None:
+def test_step2_spawn_should_include_model_fable_parameter() -> None:
     skill_text = _load_skill_text()
-    assert 'model="opus"' in skill_text, (
-        "Step 2 Agent() spawn template must include model=\"opus\" for the primary subagent"
+    assert 'model="fable"' in skill_text, (
+        "Step 2 Agent() spawn template must include model=\"fable\" for the primary subagent"
     )