claude-dev-env 8.36.2 → 8.36.4

package/scripts/tests/test_codex_compat_materializer.py

@@ -174,8 +174,8 @@ def test_public_legacy_call_forms_validate_collection_entries(tmp_path: Path) ->
 
 
 def test_render_codex_instruction_excerpt_requires_the_excerpt_heading() -> None:
-    with pytest.raises(MaterializerError, match="rules/correction-lens.md requires a Codex excerpt"):
-        materializer.render_codex_instruction_excerpt("# No excerpt\n", "rules/correction-lens.md")
+    with pytest.raises(MaterializerError, match="docs/rule-guides/correction-lens-excerpt.md requires a Codex excerpt"):
+        materializer.render_codex_instruction_excerpt("# No excerpt\n", "docs/rule-guides/correction-lens-excerpt.md")
 
 
 def test_validation_rejects_reparse_point_from_portable_attribute_seam(
@@ -820,11 +820,11 @@ def test_frontmatter_rejects_non_string_name() -> None:
 
 
 def test_failure_blast_radius_projection_uses_the_canonical_excerpt() -> None:
-    canonical_rule_path = Path(__file__).parents[2] / "rules" / "failure-blast-radius.md"
+    canonical_rule_path = Path(__file__).parents[2] / "docs" / "rule-guides" / "failure-blast-radius.md"
     canonical_rule = canonical_rule_path.read_text(encoding="utf-8")
 
     projected_instruction = materializer.render_codex_instruction_excerpt(
-        canonical_rule, "rules/failure-blast-radius.md"
+        canonical_rule, "docs/rule-guides/failure-blast-radius.md"
     )
 
     assert projected_instruction.startswith("Failure handling for this run")
@@ -833,12 +833,12 @@ def test_failure_blast_radius_projection_uses_the_canonical_excerpt() -> None:
 
 
 def test_codex_instruction_projection_carries_every_listed_rule(tmp_path: Path) -> None:
-    rules_root = Path(__file__).parents[2] / "rules"
+    package_root = Path(__file__).parents[2]
     source = tmp_path / "source"
     target = tmp_path / "target"
-    (source / "rules").mkdir(parents=True)
     for each_relative_path in materializer.codex_instruction_rule_relative_paths:
-        canonical_path = rules_root / Path(each_relative_path).name
+        canonical_path = package_root / each_relative_path
+        (source / each_relative_path).parent.mkdir(parents=True, exist_ok=True)
         (source / each_relative_path).write_text(
             canonical_path.read_text(encoding="utf-8"), encoding="utf-8"
         )
@@ -851,17 +851,17 @@ def test_codex_instruction_projection_carries_every_listed_rule(tmp_path: Path)
     assert "Failure handling for this run" in projected_instruction
     assert "Correction handling for this run" in projected_instruction
     manifest_record = _manifest_files(_required_manifest_path(config))["AGENTS.md"]
-    assert manifest_record["source"] == "rules/failure-blast-radius.md, rules/correction-lens.md"
+    assert manifest_record["source"] == "docs/rule-guides/failure-blast-radius.md, docs/rule-guides/correction-lens-excerpt.md"
 
 
 def test_codex_instruction_projection_skips_an_absent_listed_rule(
     tmp_path: Path,
 ) -> None:
-    rules_root = Path(__file__).parents[2] / "rules"
+    guides_root = Path(__file__).parents[2] / "docs" / "rule-guides"
     source = tmp_path / "source"
-    (source / "rules").mkdir(parents=True)
-    (source / "rules" / "correction-lens.md").write_text(
-        (rules_root / "correction-lens.md").read_text(encoding="utf-8"),
+    (source / "docs" / "rule-guides").mkdir(parents=True)
+    (source / "docs" / "rule-guides" / "correction-lens-excerpt.md").write_text(
+        (guides_root / "correction-lens-excerpt.md").read_text(encoding="utf-8"),
         encoding="utf-8",
     )
     config = MaterializerConfig(source, tmp_path / "target", should_apply=False)
@@ -869,7 +869,7 @@ def test_codex_instruction_projection_skips_an_absent_listed_rule(
     projection = materializer._build_codex_instruction_projection(config)
 
     assert projection is not None
-    assert projection.source_identity == "rules/correction-lens.md"
+    assert projection.source_identity == "docs/rule-guides/correction-lens-excerpt.md"
     assert "Correction handling for this run" in projection.content
     assert "Failure handling for this run" not in projection.content
 
@@ -877,10 +877,10 @@ def test_codex_instruction_projection_skips_an_absent_listed_rule(
 def test_build_plan_publishes_owned_agents_projection_and_tracks_drift(
     tmp_path: Path,
 ) -> None:
-    canonical_rule_path = Path(__file__).parents[2] / "rules" / "failure-blast-radius.md"
+    canonical_rule_path = Path(__file__).parents[2] / "docs" / "rule-guides" / "failure-blast-radius.md"
     source = tmp_path / "source"
     target = tmp_path / "target"
-    rule_path = source / "rules" / "failure-blast-radius.md"
+    rule_path = source / "docs" / "rule-guides" / "failure-blast-radius.md"
     rule_path.parent.mkdir(parents=True)
     rule_path.write_text(canonical_rule_path.read_text(encoding="utf-8"), encoding="utf-8")
     config = MaterializerConfig(source, target, should_apply=True)
@@ -892,7 +892,7 @@ def test_build_plan_publishes_owned_agents_projection_and_tracks_drift(
     first_projection = projected_path.read_text(encoding="utf-8")
     manifest_record = _manifest_files(_required_manifest_path(config))["AGENTS.md"]
     assert manifest_record["ownership"] == "codex-compat"
-    assert manifest_record["source"] == "rules/failure-blast-radius.md"
+    assert manifest_record["source"] == "docs/rule-guides/failure-blast-radius.md"
 
     rule_path.write_text(
         rule_path.read_text(encoding="utf-8").replace("Three attempts, then park.", "Three tested attempts, then park."),
@@ -944,12 +944,12 @@ def _prepare_projection_fixture(
 ) -> tuple[Path, Path, dict[str, Any], Path]:
     source = tmp_path / "source"
     target = tmp_path.parent / "codex-prod-target"
-    source_rule = source / "rules" / "failure-blast-radius.md"
+    source_rule = source / "docs" / "rule-guides" / "failure-blast-radius.md"
     source_hooks = source / "hooks" / "hooks.json"
     source_rule.parent.mkdir(parents=True)
     source_hooks.parent.mkdir(parents=True)
     source_rule.write_text(
-        (Path(__file__).parents[2] / "rules" / "failure-blast-radius.md").read_text(
+        (Path(__file__).parents[2] / "docs" / "rule-guides" / "failure-blast-radius.md").read_text(
             encoding="utf-8"
         ),
         encoding="utf-8",