Add Mojo language wiring#939
Conversation
b8e3577 to
5d785b0
Compare
Signed-off-by: Anand Aiyer <aaiyer@gmail.com>
5d785b0 to
222e97a
Compare
|
Thanks for picking up Mojo wiring. Triage: draft language-support PR, backlog priority. Before review can proceed, please remove or fully justify the unrelated vendored/nomic/code_vectors_blob.S change. Mojo language wiring should not include an unrelated vendored/generated blob. Once that is gone, review can focus on the expected pieces: extension mapping, language spec wiring, user-config alias, and grammar regression coverage. |
099b31f to
222e97a
Compare
|
Thanks for the triage. Addressed by force-pushing the PR branch back to the Mojo-only commit
|
Summary
Adds the missing Mojo language wiring now that a Mojo tree-sitter grammar is vendored in the C backend.
Root Cause
The repository includes
tree_sitter_mojo, but the language was not reachable through the normal discovery/indexing path:CBM_LANG_MOJOenum entry.mojoextension mappingCBMLangSpecrow for Mojo parser/query node typesAs a result,
.mojofiles did not produce Mojo symbols during indexing.Changes
CBM_LANG_MOJO..mojofiles to Mojo.mojoto user-config language name lookup.tree_sitter_mojoparser intolang_specswith function, class/trait, call, import, branch, assignment, and module node types.Validation
Result:
218 passed.I also verified the patched backend against a small
.mojosample locally: it indexedfn,struct, and method symbols, and outbound trace found a method calling a top-level function.