claude-dev-env 2.33.0 → 2.33.1

package/audit-rubrics/prompts/category-e-dead-code.md

@@ -20,7 +20,7 @@ Inline the artifact under this section using the section types defined in the ch
 **E1. New imports without references**
 - Walk every `import` line introduced or modified by the artifact. For each, locate at least one body reference in the same file (function call, attribute access, type annotation, decorator, default-argument expression).
 - Confirm `__all__` re-exports: if the file declares `__all__`, an import that appears only in `__all__` still counts as referenced; if no `__all__` is declared, the exemption is inert and must be stated as such.
-- Confirm `# noqa` markers: every `# noqa` on an import line must be justified by a specific lint code (e.g., `E402` for module-level imports following a `sys.path` mutation). State the justification.
+- Confirm changed `# noqa` directives are removed or resolved. Leave directives on untouched lines outside the change scope.
 - Confirm `TYPE_CHECKING` blocks: imports inside `if TYPE_CHECKING:` are referenced only by string-form annotations or runtime `typing.get_type_hints` consumers; verify at least one such consumer or mark the exemption inert.
 - Constants-only or re-export-only files (no imports of their own) — state explicitly that there is nothing to sweep.
 

package/docs/REACT_PATTERNS.md

@@ -104,7 +104,7 @@ components/
 
 - **Complete type annotations** - All props, state, and return types defined
 - **No `any` types** - Use generics, union types, or `unknown` with type guards
-- **No `@ts-ignore` or `@ts-expect-error`** without clear justification
+- **Resolve `@ts-ignore` and `@ts-expect-error`** instead of adding or justifying them
 - **Functional components only** - No class components (except error boundaries)
 - **Custom hooks for logic** - Separate business logic from UI
 - **Props interfaces** - Define explicit Props type for every component

package/package.json

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

package/scripts/tests/test_rules.py

@@ -187,3 +187,20 @@ def test_comment_policy_uses_changed_comment_names_and_type_directives() -> None
     assert "production and test code" in code_quality
     assert "comments tied to touched code are removed" in gate
     assert "comments tied to untouched code remain unchanged" in gate
+
+
+def test_comment_policy_removes_directive_justification_guidance() -> None:
+    react_patterns = (
+        _PACKAGE_ROOT / "docs" / "REACT_PATTERNS.md"
+    ).read_text(encoding="utf-8").lower()
+    category_e_prompt = (
+        _PACKAGE_ROOT
+        / "audit-rubrics"
+        / "prompts"
+        / "category-e-dead-code.md"
+    ).read_text(encoding="utf-8").lower()
+
+    assert "resolve `@ts-ignore` and `@ts-expect-error`" in react_patterns
+    assert "without clear justification" not in react_patterns
+    assert "changed `# noqa` directives are removed or resolved" in category_e_prompt
+    assert "every `# noqa` on an import line must be justified" not in category_e_prompt