(feat):added up the .robot and .resource file extension in CRG

[Bakul2006]

Pull Request

Linked issue

Closes #963

What & why

This PR adds support for .robot Robot Framework test files in the code-review graph.

Previously, Robot Framework files were not recognized by the file extension handling, which meant .robot test files were skipped during processing.

This change adds the .robot extension to the supported test file extensions so that Robot Framework test files can be included in the existing parsing and code-review graph workflow.

How it was tested

The test suite was run successfully:

uv run pytest tests/ --tb=short -q

[github-actions]

code-review-graph review

Overall risk: 0.40 (MEDIUM) — 8 changed function(s)/class(es), 0 affected flow(s), 5 test gap(s)

Risk-scored changes

Risk	Level	Symbol	Location	Tested
0.40	medium	code_review_graph/parser.py::CodeParser.qualified	code_review_graph/parser.py:4503	no
0.40	medium	code_review_graph/parser.py::CodeParser.finish	code_review_graph/parser.py:4506	no
0.15	low	code_review_graph/parser.py::CodeParser	code_review_graph/parser.py:2937	yes
0.15	low	code_review_graph/search.py::hybrid_search	code_review_graph/search.py:869	yes
0.10	low	code_review_graph/parser.py::CodeParser._extract_bytes	code_review_graph/parser.py:3186	no
0.10	low	code_review_graph/parser.py::CodeParser._parse_robot	code_review_graph/parser.py:4479	no
0.05	low	tests/test_parser.py::TestCodeParser	tests/test_parser.py:13	no
0.05	low	tests/test_parser.py::TestCodeParser.test_parse_robot_and_resource_files	tests/test_parser.py:26	(test)

Test gaps

• code_review_graph/parser.py::CodeParser._extract_bytes (code_review_graph/parser.py:3186)
• code_review_graph/parser.py::CodeParser._parse_robot (code_review_graph/parser.py:4479)
• code_review_graph/parser.py::CodeParser.qualified (code_review_graph/parser.py:4503)
• code_review_graph/parser.py::CodeParser.finish (code_review_graph/parser.py:4506)
• tests/test_parser.py::TestCodeParser (tests/test_parser.py:13)

Token savings: this graph-backed report used ~318,738 fewer tokens (~99%) than reading every changed file in full (estimated, chars/4 approximation).

---

Powered by code-review-graph — local-first analysis; no code leaves the CI runner.

[tirth8205]

Outside this sweep's mission of demonstrated coding-session token savings: this adds a new Robot Framework parser. Bare resource/keyword targets and incomplete setup, teardown and control-flow handling do not yet establish reliable compact cross-file context.

[tirth8205]

This no longer merges into staging. Conflicts in README.md.

git fetch origin && git merge origin/staging
# resolve, then
uv run pytest tests/ -q
uv run ruff check code_review_graph/
uv run mypy code_review_graph/ --ignore-missing-imports --no-strict-optional
git push

When resolving:

• Rebase PR onto current origin/staging and resolve README.md conflicts (two hunks: the 'Language coverage' paragraph around line 142, and the feature table 'Language and notebook support' row around line 279); staging rewrote both sections to point at a single Language coverage anchor instead of list...
• After rebase, re-run the full gate set (pytest on 3.13, ruff, mypy) and the edge-case coverage review for the Robot Framework parser change in code_review_graph/parser.py and tests/test_parser.py.

A large integration branch landed on staging today, which is why this drifted.

PRs now target staging, not main. Yours was retargeted already, so nothing to do there.

[Bakul2006]

@tirth8205 solved up the readme.md conflicts

[Copilot]

🟡 Changes recommended

Parser correctness gaps and duplicate or stale README documentation remain unresolved.

Get a fresh assessment by requesting another Copilot review.

Pull request overview

Adds structural Robot Framework support for .robot and .resource files in the parser and code-review graph.

Changes:

• Maps both extensions to the Robot fallback parser.
• Extracts Robot sections, tests, keywords, calls, and imports.
• Adds parser tests and documentation updates.

File summaries

File	Summary	Review findings
tests/test_parser.py	Adds Robot parsing coverage.	None.
README.md	Documents Robot Framework support.	Nit (3 votes): consolidate duplicate parser-support text and feature tables.
CONTRIBUTING.md	Documents Robot parser maintenance.	None.
code_review_graph/parser.py	Adds Robot extension handling and structural parsing.	Moderate: support pipe-delimited rows (3 votes), resolve relative resources (2 votes), emit TESTED_BY mirrors (3 votes), and normalize keyword matching (1 vote).

Review details

Suppressed comments (1)

code_review_graph/parser.py:4057

• Robot keyword matching is case-insensitive and treats spaces and underscores equivalently, but this stores the call target verbatim and the later resolver matches node names exactly. A call such as prepare_environment or prepare environment will remain unresolved instead of linking to Prepare environment, degrading callers/callees and test coverage; preserve a canonical Robot keyword key or add a Robot-specific resolver.

            keyword = cells[0]
            if keyword == "...":
                continue
            edges.append(EdgeInfo(

• Files reviewed: 4/4 changed files
• Comments generated: 5
• Review effort level: Lite

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[tirth8205]

The merge is clean now and the gates pass: your head 4a7167e merged into staging 9c13f4c gives 3336 passed, 9 skipped, 2 xpassed, ruff clean, mypy clean, and all 16 CI checks green.

The README is not resolved though. Both sides were kept instead of one.

On the merged tree:

• The feature table has 55 rows where staging has 31, and 20 of them appear twice: Incremental updates, Framework-aware PHP parsing, Blast-radius analysis, Auto-update hooks, Semantic search, Interactive visualisation, Surprise scoring, Knowledge gap analysis, Suggested questions, Edge confidence, Graph traversal, Export formats, Estimated context savings, Memory loop, Community auto-split, Execution flows, Community detection, Architecture overview, Risk-scored reviews, Custom languages, Token benchmarking. Reproduce with grep -oE '^\| \*\*[^*]+\*\*' README.md | sort | uniq -d.
• README lines 142 and 143 are two paragraphs saying the same thing. 142 is your pre-merge wording, 143 is staging's.

What to do: drop your side in both places and re-apply only the Robot part on top of staging's text. Add Robot Framework files (`.robot` and `.resource`) to the sentence now at line 143, leave the Language and notebook support | See Language coverage above row alone, and delete the Broad language + notebook support row you re-added at line 277. Staging replaced that row with the anchor on purpose.

Second, the parser edge cases, which were the other half of the last round. Two probes against your merged branch.

Robot keyword matching is case-insensitive and treats spaces and underscores as equivalent, but the CALLS target is stored verbatim:

*** Test Cases ***
T1
    prepare_environment
    Prepare Environment

*** Keywords ***
Prepare environment
    Log    x

CALLS targets come out as ['prepare_environment', 'Prepare Environment', 'Log'] against a keyword node named Prepare environment. Two of the three spellings of the same keyword never resolve, so callers, callees and test coverage are wrong for them. Normalise the key on both the keyword node and the edge target, or add a Robot-specific resolver.

Pipe-delimited files, which are valid Robot syntax, produce junk nodes:

*** Test Cases ***
| T2 |
|    | Log | hello |

yields a Test named | T2 | and a second Test named |. Either strip the pipe framing or skip pipe-delimited files rather than indexing them wrongly.

The one test you have covers the happy path only. Add a case for each of the two above.

Removing needs-rebase since it merges cleanly, adding changes-requested.

[Bakul2006]

Dependency setup is now resolved using uv sync --extra dev. The full test suite completed successfully through collection and execution.

Results: 3,957 passed, 732 skipped, 46 deselected, 2 xfailed, 2 xpassed, 2 failed.

The remaining failures are:

test_hybrid_search_ranks_impact_radius_nodes_first
test_windows_server_still_prewarms_before_mcp_run

These are genuine behavioral/environment-specific failures rather than dependency/collection errors. No source code or tests were modified.
should i commit these or solve up for 2 failling tests

[tirth8205]

The type-check job is red and this branch turned it red. code_review_graph/main.py lines 71 and 73 fail mypy:

code_review_graph/main.py:71: error: Module has no attribute "WindowsSelectorEventLoopPolicy"  [attr-defined]
code_review_graph/main.py:73: error: Module has no attribute "WindowsSelectorEventLoopPolicy"  [attr-defined]

Staging reports Success: no issues found in 77 source files. Past the type error, that block assigns onto the stdlib asyncio module at import time. On macOS and Linux, hasattr(asyncio, "WindowsSelectorEventLoopPolicy") is False before importing code_review_graph.main and True afterwards, holding _UnixDefaultEventLoopPolicy. Any library in the same process that uses that attribute as a Windows check will take the Windows branch.

The search.py edit regresses ranking. identifier_words(symbol or row["name"]) becomes identifier_words(row["name"]), which drops the class half of a qualified symbol from coverage scoring. On a repo holding UserService.authenticate, authenticate_request and service_authenticate_helper, the query user service authenticate ranks UserService.authenticate first on staging at 0.0317 and third here at 0.0212, behind the other two. UserService authenticate falls from 0.1311 to 0.0410. That partially reverts the search-quality work in #1009.

Both edits chase failures that are not real. On unmodified origin/staging in a clean environment:

uv run pytest -k "hybrid_search_ranks_impact_radius_nodes_first or windows_server_still_prewarms_before_mcp_run" -q
# 2 passed

They pass on your branch too. So neither option. Revert both files to staging.

The two parser probes from last round reproduce verbatim against a real code-review-graph build, not just against the parser in isolation. The prepare_environment / Prepare Environment file still produces three CALLS rows with target_resolution=unresolved sitting next to the node tests_rf/login.robot::Prepare environment. The pipe-delimited file still writes Test nodes named | T2 | and | into graph.db. tests/test_parser.py is byte-identical to the version I reviewed last time, so the test per probe is missing as well.

README.md is unchanged. grep -oE '^\| \*\*[^*]+\*\*' README.md | sort | uniq -d still returns the same 20 rows, and the table is 56 rows against staging's 32. The only Robot mention left is line 282, inside the Broad language + notebook support row I asked you to delete, so deleting that row leaves Robot Framework documented nowhere. The last commit also pasted the same 24 English rows into README.hi-IN.md, README.ja-JP.md, README.ko-KR.md and README.zh-CN.md, taking each from 32 to 56 rows, so four translated files now carry an untranslated copy of their own feature table.

The feature itself works. .robot and .resource both map to robot, staging parses neither, an exactly-spelled keyword call resolves end to end with target_resolution=direct, and 5,000 keywords parse in 0.04s. The branch merges into staging with no conflicts.

What would make it mergeable:

• Revert code_review_graph/main.py and code_review_graph/search.py to staging.
• Normalise the Robot keyword key on both the node name and the CALLS target, case-insensitive and treating spaces and underscores as equal.
• Strip the pipe framing or skip pipe-delimited files.
• One test for each of those two.
• Fix README.md as described last round, and revert the four translated READMEs.

Two more parser bugs. A test case and a keyword sharing a name in one file collide on <file>::<name> and the Test node is silently overwritten, so the test case disappears from the graph. FOR, IF, ELSE and END become CALLS edges, now mirrored as TESTED_BY, so the graph claims END is tested by the test.