claude-dev-env 8.94.0 → 8.95.1

package/.agents/agents/lean-worker.md

@@ -0,0 +1,7 @@
+---
+name: lean-worker
+description: Default worker for a delegated task that only reads, searches, or edits files and runs shell commands. Loads only those tools, so it starts near 26k tokens where a full-tool agent starts near 55k. Pick it first for fixes, lookups, scripted checks, and reviews. Pick a full-tool agent when the task follows a skill, drives the browser or another MCP connector, uses artifacts, watches or stops background tasks, loads deferred tools, or spawns further subagents.
+tools: Read, Edit, Write, Glob, Grep, Bash, PowerShell
+---
+
+Do the task in the brief. Work only in the files and paths it names. Report what changed, the command output that proves it, and anything that blocked you.

package/.agents/skills/correction/playbooks/fix.md

@@ -8,7 +8,7 @@ Use this playbook when this session lands the fix for the correction itself. One
 4. Search for an existing control: a rule, hook, lint, test, or skill that already names this mistake. A control that exists and still let the mistake through makes this a repeat.
 5. Choose the layer with [`correction-lens.md`](../../../../docs/rule-guides/correction-lens.md). When `Repeat` is `yes`, or step 4 found a control, go one layer above the one that failed.
 6. Land the control in the repository whose code, CI, or agents it guards, per the guide's "Where the control lands" section. When that repository is not in this session, add it. When it cannot be added, print the brief, name the missing repository, and stop.
-7. Prove the control. Show it fail on the mistake from the brief's `Evidence` line, then pass with the change, per [`falsify-before-green.md`](../../../../rules/falsify-before-green.md) and [`proof-before-pull-request.md`](../../../../docs/rule-guides/proof-before-pull-request.md). Without that proof, open no pull request.
-8. Open the pull request through `pr-lifecycle`. Put the brief, the layer, and both proof results in the body.
+7. Prove the control. Show it fail on the mistake from the brief's `Evidence` line, then pass with the change, per [`falsify-before-green.md`](../../../../rules/falsify-before-green.md) and [`proof-before-pull-request.md`](../../../../docs/rule-guides/proof-before-pull-request.md). Without that proof, open no pull request. When the control is a hook, read the correction instructions of the repository from step 6 as step 3 does. When they name an incident eval, add the incident as a case to that eval in the same pull request. Run the eval's compare against the base branch and keep the line that shows the case fails on the base and holds on the branch.
+8. Open the pull request through `pr-lifecycle`. Put the brief, the layer, and both proof results in the body. When step 7 added an incident case, also name its path and its compare line.
 
 **Reply:** the layer chosen, why each higher layer cannot hold the lesson, and the pull request link.

package/.agents/skills/correction/scripts/test_correction_task_card.py

@@ -140,3 +140,10 @@ def test_should_return_the_playbook_from_step_two() -> None:
     handed_off_steps = correction_task_card.handed_off_playbook_steps(FIX_PLAYBOOK_TEXT)
     assert handed_off_steps.startswith("2. ")
     assert "pull request link" in handed_off_steps
+
+
+def test_should_hand_off_the_incident_case_step_with_the_proof() -> None:
+    handed_off_steps = correction_task_card.handed_off_playbook_steps(FIX_PLAYBOOK_TEXT)
+    proof_step = handed_off_steps.split("\n7. ", 1)[1].split("\n8. ", 1)[0]
+    assert "incident eval" in proof_step
+    assert "same pull request" in proof_step

package/bin/install-pstack-plugin.mjs

@@ -67,6 +67,9 @@ const HOST_DEFINITIONS = Object.freeze({
         orphanedVersionsDirectory: (homeDirectory, spec) => join(
             homeDirectory, 'plugins', 'cache', spec.marketplaceName, spec.name,
         ),
+        staleCloneDirectory: (homeDirectory, spec) => join(
+            homeDirectory, 'plugins', 'marketplaces', `${spec.marketplaceRepository.replaceAll('/', '-')}..clone`,
+        ),
         installVerb: 'install',
         uninstallVerb: 'uninstall',
     }),
@@ -215,6 +218,24 @@ function singleLineDetail(text) {
     return (text ?? '').split(/\r?\n/).map(eachLine => eachLine.trim()).filter(Boolean).join(' ');
 }
 
+/**
+ * Delete the clone folder an interrupted marketplace add left behind.
+ *
+ * Claude Code clones a marketplace into `<owner>-<repository>..clone` and
+ * renames it when the clone finishes. A clone that stops partway leaves that
+ * folder with its `.git/shallow.lock`, and `marketplace remove` does not
+ * delete it, so every later add fails on the lock. Removing the folder
+ * before the add lets the add start from an empty slot.
+ *
+ * @param {string|undefined} cloneDirectory The leftover clone folder.
+ * @returns {boolean} True when a folder was removed.
+ */
+export function removeStaleMarketplaceClone(cloneDirectory) {
+    if (!cloneDirectory || !existsSync(cloneDirectory)) return false;
+    rmSync(cloneDirectory, { recursive: true, force: true, maxRetries: 3 });
+    return true;
+}
+
 const ORPHANED_VERSION_MARKER = '.orphaned_at';
 
 /**
@@ -269,6 +290,9 @@ function installForHost(allSpecs, host, homeDirectory, environment, runCommand,
         });
         if (namesAnAbsentCommand(outcome)) return absentHostOutcome(allSpecs, host, executable);
     }
+    if (HOST_DEFINITIONS[host].staleCloneDirectory && homeDirectory) {
+        removeStaleMarketplaceClone(HOST_DEFINITIONS[host].staleCloneDirectory(homeDirectory, allSpecs[0]));
+    }
     for (const commandArguments of plan.commands) {
         const outcome = runCommand(executable, [...commandArguments], {
             environment: commandEnvironment,
@@ -300,6 +324,8 @@ function installForHost(allSpecs, host, homeDirectory, environment, runCommand,
  * Each host first removes every plugin in the set and then the marketplace,
  * so an older version never survives beside the new one. A removal that finds
  * nothing to remove exits non-zero on a fresh home, and the install goes on.
+ * Before the Claude marketplace add, the install deletes the half-made clone
+ * folder an interrupted earlier add left, so the add starts in an empty slot.
  *
  * Each host is one member of the batch. A host without its command-line tool
  * is skipped and a host whose command fails is reported, so the rules, hooks,

package/bin/install-pstack-plugin.test.mjs

@@ -15,6 +15,7 @@ import {
     installMarketplacePlugins,
     marketplacePluginsPlan,
     removeOrphanedPluginVersions,
+    removeStaleMarketplaceClone,
     removeRetiredMarketplacePlugins,
     shouldInstallMarketplacePlugin,
 } from './install-pstack-plugin.mjs';
@@ -478,3 +479,24 @@ test('a held folder that was not searched adds no claim about what holds it', ()
     assert.match(claude.warning, /EPERM: operation not permitted$/);
     assert.doesNotMatch(claude.warning, /likely holder|Close /);
 });
+
+test('should delete a half-made marketplace clone before the shared marketplace add', () => {
+    const claudeRoot = mkdtempSync(join(tmpdir(), 'cde-stale-clone-'));
+    const cloneDirectory = join(claudeRoot, 'plugins', 'marketplaces', 'jl-cmd-projects-claude-dev-env..clone');
+    mkdirSync(join(cloneDirectory, '.git'), { recursive: true });
+    writeFileSync(join(cloneDirectory, '.git', 'shallow.lock'), '');
+    const allCloneStatesAtAdd = [];
+    const outcome = installMarketplacePlugins([USAGE_WRAPUP_PLUGIN_SPEC], { claudeRoot, hosts: ['claude'] }, {
+        runCommand: (executable, commandArguments) => {
+            if (commandArguments[2] === 'add') allCloneStatesAtAdd.push(existsSync(cloneDirectory));
+            return { status: 0, stderr: '' };
+        },
+    });
+    assert.equal(outcome.status, 'installed');
+    assert.deepEqual(allCloneStatesAtAdd, [false]);
+    rmSync(claudeRoot, { recursive: true, force: true });
+});
+
+test('should report no removal when no stale marketplace clone exists', () => {
+    assert.equal(removeStaleMarketplaceClone(join(tmpdir(), 'cde-no-such-clone')), false);
+});

package/package.json

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