claude-dev-env 8.49.2 → 8.49.4

package/bin/install-pstack-plugin.mjs

@@ -27,7 +27,7 @@ export const USAGE_WRAPUP_PLUGIN_SPEC = Object.freeze({
     label: 'Usage-wrapup',
     marketplaceRepository: 'jl-cmd/claude-dev-env',
     marketplaceName: 'claude-dev-env',
-    marketplaceAddArguments: Object.freeze(['--sparse', '.claude-plugin']),
+    marketplaceAddArguments: Object.freeze(['--sparse', '.claude-plugin', 'packages/usage-wrapup']),
     pluginIdentifier: 'usage-wrapup@claude-dev-env',
     hosts: Object.freeze(['claude']),
     optOutFlag: '--no-usage-wrapup',

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

@@ -213,14 +213,12 @@ test('a non-zero exit that is not command-not-found still reads as a failure', (
     assert.match(outcome.hosts[0].warning, /rejected the catalog/);
 });
 
-const USAGE_WRAPUP_PINNED_COMMIT = 'd78c314e5787786e866bf5c4967107365854dc78';
-
 test('the usage-wrapup plan adds this repository marketplace and installs the plugin on Claude', () => {
     const plan = marketplacePluginPlan(USAGE_WRAPUP_PLUGIN_SPEC, 'claude');
     assert.equal(plan.executable, 'claude');
     assert.equal(plan.homeVariable, 'CLAUDE_CONFIG_DIR');
     assert.deepEqual(plan.commands, [
-        ['plugin', 'marketplace', 'add', 'jl-cmd/claude-dev-env', '--sparse', '.claude-plugin'],
+        ['plugin', 'marketplace', 'add', 'jl-cmd/claude-dev-env', '--sparse', '.claude-plugin', 'packages/usage-wrapup'],
         ['plugin', 'install', 'usage-wrapup@claude-dev-env'],
     ]);
 });
@@ -238,7 +236,7 @@ test('installing usage-wrapup runs only the Claude commands, inside the managed
     assert.deepEqual(runner.calls.map(call => [call.executable, ...call.commandArguments]), [
         ['claude', 'plugin', 'uninstall', 'usage-wrapup@claude-dev-env'],
         ['claude', 'plugin', 'marketplace', 'remove', 'claude-dev-env'],
-        ['claude', 'plugin', 'marketplace', 'add', 'jl-cmd/claude-dev-env', '--sparse', '.claude-plugin'],
+        ['claude', 'plugin', 'marketplace', 'add', 'jl-cmd/claude-dev-env', '--sparse', '.claude-plugin', 'packages/usage-wrapup'],
         ['claude', 'plugin', 'install', 'usage-wrapup@claude-dev-env'],
     ]);
     for (const call of runner.calls) {
@@ -270,7 +268,7 @@ test('the usage-wrapup opt-out flag and variable turn off only usage-wrapup', ()
     );
 });
 
-test('the repository marketplace lists usage-wrapup from the fork, pinned to the reviewed commit', () => {
+test('the repository marketplace lists usage-wrapup from its package folder in this repository', () => {
     const marketplace = JSON.parse(readFileSync(
         new URL('../../../.claude-plugin/marketplace.json', import.meta.url),
         'utf8',
@@ -279,12 +277,7 @@ test('the repository marketplace lists usage-wrapup from the fork, pinned to the
     assert.equal(marketplace.name, marketplaceName);
     const entry = marketplace.plugins.find(plugin => plugin.name === pluginName);
     assert.ok(entry, `the marketplace lists ${pluginName}`);
-    assert.deepEqual(entry.source, {
-        source: 'github',
-        repo: 'jl-cmd/usage-wrapup',
-        ref: 'main',
-        sha: USAGE_WRAPUP_PINNED_COMMIT,
-    });
+    assert.equal(entry.source, './packages/usage-wrapup');
 });
 
 test('a clean install deletes the orphaned pstack versions and keeps the installed one', () => {

package/bin/install.test.mjs

@@ -3317,7 +3317,7 @@ test('a pstack install after a --no-pstack install seeds the Codex model sheet',
 const USAGE_WRAPUP_CLAUDE_COMMANDS = Object.freeze([
     'claude plugin uninstall usage-wrapup@claude-dev-env',
     'claude plugin marketplace remove claude-dev-env',
-    'claude plugin marketplace add jl-cmd/claude-dev-env --sparse .claude-plugin',
+    'claude plugin marketplace add jl-cmd/claude-dev-env --sparse .claude-plugin packages/usage-wrapup',
     'claude plugin install usage-wrapup@claude-dev-env',
 ]);
 

package/package.json

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

package/scripts/account_broker.py

@@ -211,6 +211,7 @@ class _RunContext:
     all_attempts: list[tuple[str, str]]
     all_spent_accounts: set[Account]
     all_spent_resets: dict[Account, datetime]
+    start_failure_text: str = ""
 
 
 def _prepare_run(
@@ -234,7 +235,7 @@ def _wait_outcome(context: _RunContext, decision: Decision) -> JobOutcome:
     context.report.final_decision = decision
     all_attempt_statuses = {each_status for _, each_status in context.all_attempts}
     if all_attempt_statuses == {"start_failed"}:
-        return JobOutcome(COMMAND_MISSING_EXIT_CODE, "", "", None, tuple(context.all_attempts), "start_failed", None, None)
+        return JobOutcome(COMMAND_MISSING_EXIT_CODE, "", context.start_failure_text, None, tuple(context.all_attempts), "start_failed", None, None)
     status = "exhausted" if context.all_attempts else "wait"
     return JobOutcome(WAIT_EXIT_CODE, "", "", None, tuple(context.all_attempts), status, None, decision.resets_at, decision.reason)
 
@@ -287,6 +288,7 @@ def _attempt_once(context: _RunContext, decision: Decision) -> JobOutcome | None
     except (OSError, subprocess.TimeoutExpired) as error:
         status = "timeout" if isinstance(error, subprocess.TimeoutExpired) else "start_failed"
         _record_spent_attempt(context, account, status, COMMAND_MISSING_EXIT_CODE)
+        context.start_failure_text = str(error)
         if isinstance(error, subprocess.TimeoutExpired) or not context.all_readings:
             context.report.final_decision = decision
             final_status = "advisor_blocked" if context.product is Product.CLAUDE else status

package/scripts/test_account_broker.py

@@ -908,6 +908,20 @@ def test_should_record_start_failure_without_launching_a_command_missing_from_pa
     assert outcome.returncode == 127
 
 
+def test_should_keep_the_start_failure_text_when_no_account_can_start(monkeypatch: pytest.MonkeyPatch) -> None:
+    monkeypatch.setitem(account_broker.all_product_adapters, Product.CLAUDE, _adapter(
+        (_account("first", Product.CLAUDE), _account("second", Product.CLAUDE)),
+        {"first": _meters(80, 80), "second": _meters(70, 70)},
+    ))
+    monkeypatch.setattr(shutil, "which", lambda name: None)
+    monkeypatch.setattr(account_broker.support.subprocess, "run", lambda *arguments, **options: pytest.fail("a missing command ran"))
+
+    outcome, _ = account_broker._execute(Product.CLAUDE, ("claude", "-p"), now=NOW)
+
+    assert outcome.status == "start_failed"
+    assert "claude" in outcome.stderr
+
+
 @pytest.mark.parametrize("marked_name", ["retired", "first"])
 def test_should_run_past_a_stored_spent_mark_beyond_the_platform_range(
     monkeypatch: pytest.MonkeyPatch, capsys: pytest.CaptureFixture[str], marked_name: str