claude-dev-env 1.54.0 → 1.55.1

package/agents/clean-coder.md

@@ -2,7 +2,6 @@
 name: clean-coder
 description: "Use PROACTIVELY for ALL code generation — feature development, bug fixes, refactoring, hook creation, automation scripts, and any task that produces code. Internalizes CODE_RULES.md and the 8-dimension readability standard so thoroughly that /check finds zero issues. The definitive code-writing agent."
 tools: Read, Write, Edit, Bash, Grep, Glob, Task, Skill
-model: fable
 color: green
 ---
 

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "claude-dev-env",
-    "version": "1.54.0",
+    "version": "1.55.1",
     "description": "Claude Code development standards — rules, hooks, agents, commands, and skills",
     "type": "module",
     "bin": {

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`, fable), per-loop
+  audit (`code-quality-agent`, opus) and fix (`clean-coder`), 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,6 @@ Spawn:
 Agent(
   subagent_type="clean-coder",
   name="bugfix-pr<N>-loop<L>",
-  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 fable
+  - `bugfix` — `clean-coder`, no model pin (the lead picks the model per task)
 
 ### Loop state block
 

package/skills/qbug/SKILL.md

@@ -107,7 +107,6 @@ Then call `Agent` twice in the same message — the primary clean-coder and the
 ```
 Agent(
   subagent_type="clean-coder",
-  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,16 @@ def test_contract_should_require_files_opened_in_proof_of_absence() -> None:
     )
 
 
-def test_step2_spawn_should_include_model_fable_parameter() -> None:
+def test_step2_primary_spawn_should_not_pin_a_model() -> None:
     skill_text = _load_skill_text()
-    assert 'model="fable"' in skill_text, (
-        "Step 2 Agent() spawn template must include model=\"fable\" for the primary subagent"
+    primary_spawn_start = skill_text.find('subagent_type="clean-coder"')
+    assert primary_spawn_start != -1, (
+        "Step 2 Agent() spawn template must spawn the clean-coder primary subagent"
+    )
+    primary_spawn_end = skill_text.find(")", primary_spawn_start)
+    primary_spawn_block = skill_text[primary_spawn_start:primary_spawn_end]
+    assert "model=" not in primary_spawn_block, (
+        "Primary clean-coder spawn must omit model= so the lead picks the spawn effort"
     )
 
 

package/skills/update/SKILL.md

@@ -111,6 +111,16 @@ git -C "<path>" log --oneline -1 main
 
 Report the move as `main <old> → <new>`, or "already up to date" when nothing changed.
 
+Then report what the checkout runs — a move of the `main` ref says nothing about the files on disk:
+
+```
+git -C "<path>" branch --show-current
+git -C "<path>" status --short
+```
+
+- **Checked-out branch.** When a branch other than `main` is checked out, say so plainly: the fast-forward moves a ref only and no file on disk changes. Code that runs from this checkout comes from the checked-out branch, not from `main`.
+- **Dirty tracked files.** List every modified tracked file from `status --short`. Uncommitted edits sit on top of the checked-out branch and are what runs — flag them, because a ref update can neither see nor repair them. When the tree is clean, report "working tree clean".
+
 ## Constraints (non-negotiable)
 
 - **Fast-forward only.** If the remote's `main` is not a descendant of local `main`, stop. Never `--force`, never `branch -f`, never a merge commit. Divergence is a job for `/rebase`.
@@ -125,6 +135,7 @@ Report the move as `main <old> → <new>`, or "already up to date" when nothing
 - `<remote>/main` only moves after an explicit `git fetch`. Fetch inside every run; never compare against a remote-tracking ref left over from an earlier fetch.
 - `origin` is not always the source of truth. When a fork is `origin` and the canonical repo is another remote (often `upstream`), the confirmed remote should be the canonical one, not whichever is named `origin`.
 - Quote the path on every command — `git -C "<path>"` — so paths with spaces or a NAS drive letter survive.
+- "Up to date" describes the `main` ref, not the running code. A checkout deployed on a feature branch, or carrying uncommitted edits, runs that branch plus those edits regardless of where `main` points. Phase 4's checkout-state report exists so the operator sees that gap on every run.
 
 ## What this skill does NOT do