`81934c3`

Ignore nested reference paths

Authored by

espadonne 2 weeks ago

SHA: 81934c30ac494d019427a6abdb8039214de6be74
Parents: 3719f44
Tree: a85dad9

2 changed files

Status	File	+	-
M	`src/loader/runtime/dod.py`	8	2
M	`tests/test_dod.py`	33	0

src/loader/runtime/dod.pymodified

      paths: list[str] = []
      seen: set[str] = set()
      directory_stack: list[tuple[int, str]] = []
 +    read_only_stack: list[int] = []
      for line in lines:
 -        if _line_describes_read_only_file_change(line):
 -            continue
          indent = len(line) - len(line.lstrip(" "))
 +        while read_only_stack and indent <= read_only_stack[-1]:
 +            read_only_stack.pop()
          while directory_stack and indent <= directory_stack[-1][0]:
              directory_stack.pop()
 +        if _line_describes_read_only_file_change(line):
 +            read_only_stack.append(indent)
 +            continue
 +        if read_only_stack:
 +            continue
          backticked = re.findall(r"`([^`]+)`", line)
          if backticked:

tests/test_dod.pymodified

+     ]
 +def test_collect_planned_artifact_targets_ignores_nested_read_only_reference_paths(
 +    tmp_path: Path,
 +) -> None:
 +    implementation_plan = tmp_path / "implementation.md"
 +    implementation_plan.write_text(
 +        "\n".join(
 +            [
 +                "# Implementation Plan",
 +                "",
 +                "## File Changes",
 +                "1. Create directory structure for nginx guide:",
 +                f"   - `{tmp_path / 'Loader' / 'guides' / 'nginx' / 'index.html'}`",
 +                f"   - `{tmp_path / 'Loader' / 'guides' / 'nginx' / 'chapters'}/`",
 +                "2. Analyze existing fortran guide structure to understand the format:",
 +                "   - `~/Loader/guides/fortran/`",
 +                "   - `~/Loader/guides/fortran/chapters/`",
 +                "3. Create nginx guide content following the same structure and cadence as the fortran guide",
 +                "",
 +            ]
 +        )
 +    )
++
 +    dod = create_definition_of_done("Create an nginx guide from a Fortran reference.")
 +    dod.implementation_plan = str(implementation_plan)
++
 +    targets = collect_planned_artifact_targets(dod, project_root=tmp_path)
++
 +    assert targets == [
 +        (tmp_path / "Loader" / "guides" / "nginx" / "index.html", False),
 +        (tmp_path / "Loader" / "guides" / "nginx" / "chapters", True),
 +    ]
++
++
  def test_all_planned_artifacts_exist_requires_file_contents_for_planned_output_directory(
      tmp_path: Path,
  ) -> None: