[EmilVorre]

Pull Request

Linked issue

Closes #

What & why

How it was tested

uv run pytest tests/ --tb=short -q
uv run ruff check code_review_graph/
uv run mypy code_review_graph/ --ignore-missing-imports --no-strict-optional

Checklist

• Tests added for new functionality
• All tests pass: uv run pytest tests/ --tb=short -q
• Linting passes: uv run ruff check code_review_graph/
• Type checking passes: uv run mypy code_review_graph/ --ignore-missing-imports --no-strict-optional
• Lines are at most 100 characters
• Docs updated where behavior changed (README, docs/, docstrings)

[EmilVorre]

I have taken the code from pr: #816

and merged main and fixed conflicts, this is done as the other pr seems to be going stale and i really need this feature.

Verification:

• 4287 passed, 851 skipped, 2 xfailed, 2 xpassed
• ruff passed
• mypy passed
• git diff --check passed

[tirth8205]

Thank you for this PR.

I built, updated and ran detect-changes on test repos with many YAML files.

Before I can merge it, please fix these:

1. A block's line range runs onto the next key's line. _yaml_end_line returns end_mark.line + 1 (code_review_graph/parser.py:2895). Example: build:\n steps: [compile, test]\ndeploy:\n region: eu\n. $.build gets lines 1-3, but line 3 is deploy:. So adding deploy also marks $.build as changed. Failing test: test_appending_a_sibling_key_does_not_flag_the_previous_block.

2. pnpm-lock.yaml is ignored only at the repo root (code_review_graph/incremental.py:259). packages/web/pnpm-lock.yaml is still indexed, with package names as nodes. Adding **/pnpm-lock.yaml fixes it. Failing test: test_pnpm_lockfiles_are_ignored_at_any_depth.

Both tests are in tests/test_generic_yaml_edge_cases.py on the branch review/pr-1092 in this repo. If a test there is marked xfail, please remove that mark with your fix.

After you push the fix, I will test it again.

[Copilot]

🟡 Changes recommended

Repeated expansion of shared YAML subtrees can stall indexing despite existing limits, while source-range, key-handling, metadata, and response-bounding defects remain unresolved.

8 open issues

• Bound traversal of aliases in unsupported complex keys · New
• Apply response limits to YAML change results · New
• End recursive alias ranges at the reference line · New
• Exclude next-sibling lines from YAML node ranges · New
• Canonicalize scalar keys when detecting duplicates · New
• Track scalar-key alias events and occurrence lines · New
• Detect YAML merges by key tag, not text · New
• Include YAML metadata in semantic search results · New

What changed in this PR

Adds value-free generic YAML indexing to the knowledge graph, extending structural queries and change analysis while preserving specialized Spring and Ansible parsing.

Changes:

• Introduces aggregated YamlPath nodes, duplicate-key diagnostics, and alias references.
• Adds YAML metadata, change reporting, search filtering, and visualization support.
• Adds regression coverage, updates documentation, and excludes pnpm-lock.yaml by default.

File	Description
tests/​test_yaml_parser.py	Adds parser, metadata, limits, and graph integration coverage.
tests/​test_visualization.py	Checks YAML node and reference-edge visuals.
tests/​test_spring_config.py	Updates expectations for generic YAML fallback.
tests/​test_incremental.py	Checks the pnpm lockfile exclusion.
tests/​test_embedding_initialization.py	Reorders Windows test setup.
tests/​test_changes.py	Tests YAML occurrence-range change reporting.
tests/​test_ansible_parser.py	Checks fallback for ordinary YAML.
tests/​conftest.py	Clears inherited Git configuration environment variables.
README.md	Advertises YAML indexing and change analysis.
docs/​USAGE.md	Documents YAML support and node types.
docs/​schema.md	Describes YAML metadata and identities.
docs/​COMMANDS.md	Updates filters and change-analysis documentation.
code_review_graph/​visualization.py	Adds YAML styling, filters, and reference-edge visuals.
code_review_graph/​tools/​review.py	Exposes YAML change results and minimal-mode counts.
code_review_graph/​parser.py	Implements generic YAML parsing and fallback routing.
code_review_graph/​main.py	Updates tool docstrings for YAML support.
code_review_graph/​incremental.py	Excludes pnpm-lock.yaml by default.
code_review_graph/​graph.py	Serializes public YAML metadata.
code_review_graph/​docs/​LLM-OPTIMIZED-REFERENCE.md	Updates the embedded language reference.
code_review_graph/​cli.py	Adds the YamlPath search filter.
code_review_graph/​changes.py	Maps YAML ranges and reports changed paths separately.

_{Review effort: Balanced}