Skip to content

feat: add ReScript language support (#309 with conflict resolution)#323

Merged
tirth8205 merged 4 commits intomainfrom
feat/rescript-support-pr309
Apr 18, 2026
Merged

feat: add ReScript language support (#309 with conflict resolution)#323
tirth8205 merged 4 commits intomainfrom
feat/rescript-support-pr309

Conversation

@tirth8205
Copy link
Copy Markdown
Owner

Summary

This PR incorporates the ReScript language support from PR #309 (by @jagguji / juspay) with merge conflicts resolved against the current main branch.

Original PR: #309 (Feat/rescript support) by @jagguji
Changes from original PR:

  • code_review_graph/parser.py: ReScript AST extraction (functions, types, modules, let bindings)
  • code_review_graph/rescript_resolver.py: cross-file symbol resolution for .res/.resi pairs
  • code_review_graph/incremental.py: register ReScript in supported-language set
  • tests/test_multilang.py: parser + resolver tests (4 new test classes)
  • tests/fixtures/sample.res, sample.resi: fixture files

Conflict resolution:

  • parser.py: Added .gd/.ksh entries from main alongside new .res/.resi entries
  • tests/test_multilang.py: Preserved TestGDScriptParsing, TestJuliaParsing, and TestJavaImportResolution from main alongside new TestRescriptParser, TestRescriptInterfaceParser, TestRescriptEdgeCases, TestRescriptCrossModuleResolver

Closes #309

Co-Authored-By: Claude Opus 4.6 (1M context) noreply@anthropic.com

jagguji and others added 4 commits April 17, 2026 13:26
Adds ReScript (.res/.resi) parsing, cross-file resolution, and tests.

- parser.py: ReScript AST extraction (functions, types, modules, let bindings)
- rescript_resolver.py: cross-file symbol resolution for .res/.resi pairs
- incremental.py: register ReScript in supported-language set
- tests/test_multilang.py: parser + resolver tests
- tests/fixtures/sample.res, sample.resi: fixture files

Related: #308

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@tirth8205 tirth8205 merged commit 20251ed into main Apr 18, 2026
4 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants