claude-dev-env 8.53.2 → 8.54.0

package/docs/rule-guides/request-scope.md

@@ -0,0 +1,20 @@
+Back to the [rule entry](../../rules/request-scope.md).
+
+# Request scope
+
+**When this applies:** The user asks whether part of a prompt, rule, card, or check is still needed, or asks you to change one.
+
+## Rule
+
+Answer for the whole block that part belongs to. When a shipped hook, gate, or check now enforces what a block tells an agent to do, every line of that block that restates the enforced behavior is dead text. Propose removing all of those lines, and name each line that stays with the reason it stays.
+
+Before you report the change done, read the output the user will use and compare it with the user's own words.
+
+## Example
+
+A starter's naming block holds title rules, rename steps, and steps to install a Stop hook. The user asks whether the Stop hook is still needed, and release notes show a shipped Stop gate and a title format gate.
+
+- Narrow answer: remove the Stop hook steps and keep the rest. The title length rule and the rename steps stay, though the gates enforce both.
+- Whole-block answer: remove the Stop hook steps, the rename steps, and every title rule the format gate checks. Keep the lines no gate checks, each with its reason.
+
+**Enforcement:** none, the agent applies it.

package/package.json

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

package/rules/request-scope.md

@@ -0,0 +1,7 @@
+# Request scope
+
+**When:** Asked whether part of a block is still needed, or asked to change one.
+
+A shipped gate makes each block line restating what it enforces dead text. Propose removing all of them, past the part asked about, and give each kept line's reason. Read the output back before reporting done.
+
+**Full text:** [`docs/rule-guides/request-scope.md`](../docs/rule-guides/request-scope.md)

package/scripts/account_broker.py

@@ -54,6 +54,7 @@ from dev_env_scripts_constants.account_broker_constants import (
 from dev_env_scripts_constants.codex_account_constants import (
     CODEX_HOME_ENVIRONMENT_VARIABLE,
     MAIN_CODEX_HOME_DIRECTORY_NAME,
+    NO_ROSTER_ACCOUNT_NAME,
     TIER_NORMAL,
 )
 
@@ -78,7 +79,7 @@ def choose_from_readings(
     """
     if product is Product.CODEX and not all_readings:
         home = Path(os.environ.get(CODEX_HOME_ENVIRONMENT_VARIABLE) or Path.home() / MAIN_CODEX_HOME_DIRECTORY_NAME).resolve()
-        return Decision("run", Account(product, "default", home, True), None, "no roster is configured", TIER_NORMAL)
+        return Decision("run", Account(product, NO_ROSTER_ACCOUNT_NAME, home, True), None, "no roster is configured", TIER_NORMAL)
     all_available = [each_reading for each_reading in all_readings if each_reading.account not in all_spent_accounts]
     selected = (
         _choose_claude(all_available, preferred_command)
@@ -366,7 +367,8 @@ def _parse_spent_mark(all_accounts_by_name: Mapping[str, Reading], mark: str, pr
         raise BrokerConfigurationError(f"invalid reset for account {name}") from error
     reading = all_accounts_by_name.get(name)
     if reading is None:
-        print(f"warning: spent mark names {name}, an account outside the roster; choose waits until its reset", file=sys.stderr)
+        if all_accounts_by_name or name != NO_ROSTER_ACCOUNT_NAME:
+            print(f"warning: spent mark names {name}, an account outside the roster; choose waits until its reset", file=sys.stderr)
         return Reading(Account(product, name, Path(), False), None), reset
     return reading, reset
 

package/scripts/dev_env_scripts_constants/codex_account_constants.py

@@ -66,6 +66,9 @@ CODEX_PROFILES_ROOT_ENVIRONMENT_VARIABLE: str = "CODEX_PROFILES_ROOT"
 MAIN_CODEX_HOME_DIRECTORY_NAME: str = ".codex"
 """Directory under the user home that holds the shared Codex setup."""
 
+NO_ROSTER_ACCOUNT_NAME: str = "default"
+"""Account name the broker gives the shared Codex home when no roster is configured."""
+
 CODEX_HOME_ENVIRONMENT_VARIABLE: str = "CODEX_HOME"
 """Environment variable that points Codex at one account's home."""
 

package/scripts/test_account_broker.py

@@ -1026,6 +1026,38 @@ def test_should_warn_when_a_spent_mark_names_an_account_outside_the_roster(
     assert "outside the roster" in all_warning_lines[0]
 
 
+def test_should_not_warn_when_the_default_account_is_marked_without_a_roster(
+    monkeypatch: pytest.MonkeyPatch, capsys: pytest.CaptureFixture[str]
+) -> None:
+    monkeypatch.setitem(account_broker.all_product_adapters, Product.CODEX, _adapter((), {}))
+    _freeze_clock(monkeypatch, NOW)
+    reset = NOW + timedelta(hours=2)
+
+    account_broker.main(("choose", "--product", "codex", "--spent", f"default:{int(reset.timestamp())}"))
+
+    assert capsys.readouterr().err == ""
+
+
+@pytest.mark.parametrize(
+    ("all_roster_names", "marked_name"),
+    [(("first",), "default"), ((), "visitor")],
+)
+def test_should_warn_when_a_spent_mark_names_an_account_the_roster_lacks(
+    monkeypatch: pytest.MonkeyPatch, capsys: pytest.CaptureFixture[str], all_roster_names: tuple[str, ...], marked_name: str
+) -> None:
+    all_accounts = tuple(_account(each_name) for each_name in all_roster_names)
+    monkeypatch.setitem(account_broker.all_product_adapters, Product.CODEX, _adapter(all_accounts, {each_name: _meters(80, 80) for each_name in all_roster_names}))
+    _freeze_clock(monkeypatch, NOW)
+    reset = NOW + timedelta(hours=2)
+
+    account_broker.main(("choose", "--product", "codex", "--spent", f"{marked_name}:{int(reset.timestamp())}"))
+
+    all_warning_lines = capsys.readouterr().err.splitlines()
+    assert len(all_warning_lines) == 1
+    assert marked_name in all_warning_lines[0]
+    assert "outside the roster" in all_warning_lines[0]
+
+
 @pytest.mark.parametrize("mark", ["", ":1800000000"])
 def test_should_reject_a_spent_mark_without_an_account_name(
     monkeypatch: pytest.MonkeyPatch, capsys: pytest.CaptureFixture[str], mark: str

package/scripts/tests/test_rule_load_scopes.py

@@ -20,6 +20,7 @@ ALWAYS_ON_RULE_NAMES = frozenset(
         "no-contrast-framing.md",
         "question-presentation.md",
         "research-mode.md",
+        "request-scope.md",
         "session-title.md",
         "skill-pointers.md",
         "shell-invocation.md",