Release files generated by ocaml-tree-sitter e36ced3.#2
Merged
malaverdiere merged 1 commit intoJul 6, 2026
Conversation
2 tasks
kopecs
approved these changes
Jul 6, 2026
malaverdiere
added a commit
to semgrep/ocaml-tree-sitter-semgrep
that referenced
this pull request
Jul 6, 2026
…in JSX (#619) ### Checklist - [x] Any new parsing code was already published, integrated, and merged into Semgrep. DO NOT MERGE THIS PR BEFORE THE SEMGREP INTEGRATION WORK WAS COMPLETED. - [x] Change has no security implications (otherwise, ping the security team) ### **Related PRs:** - semgrep/semgrep-javascript#2 - **NOTE:** no integration in semgrep-proprietary at this stage. ### What Closes LANG-584: an agent skill that repairs the semgrep extension grammar after an upstream tree-sitter bump, plus the JavaScript fix produced by running it. - **`.agents/skills/fix-semgrep-grammar/SKILL.md`** — drives an agent through: preconditions, then a bounded fix loop with strict edit boundaries, and machine-readable exit statuses. - **`doc/semgrep-constructs.md`** — inventory mapping user-facing semgrep pattern syntax to the grammar-level rule names used across the wrappers; linked from the doc index and referenced by the skill. - **`doc/fix-semgrep-grammar-skill.md`** — short usage doc (inputs, where it operates, how to run it by hand, exit statuses), per LANG-584. - **`lang/semgrep-grammars/src/semgrep-javascript/`** — We obtain parity with the Menhir parser by making `_jsx_child` to accept a bare `>` among JSX children and add a tree-sitter grammar test. ### Validation - Ran the skill for java script, then `test-lang javascript` exits 0 with no Blank-node warnings. The skill was trained on fixes required for JS as well as Solidity. The Solidity fix will be in a separate PR. This also relates to LANG-579 but does NOT close it.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Relates to LANG-584 but does NOT close it.
Security