Skip to content

fix: recognize query syntax in supported Semgrep rules - #1043

Draft
yashrajp22 wants to merge 21 commits into
mainfrom
yashraj/semgrep-rule-ownership-followup
Draft

yashrajp22 wants to merge 21 commits into
mainfrom
yashraj/semgrep-rule-ownership-followup

Conversation

@yashrajp22

@yashrajp22 yashrajp22 commented Nov 5, 2026 •

Copy link
Copy Markdown
Collaborator

Valid Semgrep queries and explanatory messages can be interpreted as shell commands, leaving static inspection incomplete.

Recognize their roles after a complete bounded YAML parse and a supported rule/formula structure. Query fields receive matching-language ownership, and explanatory messages use Markdown boundaries. Dynamic execution and resource limits in messages retain their diagnostics. Raw and decoded security findings remain active. Unsupported schemas, runtime instructions in other fields or comments, and missing finding provenance retain their existing handling.

Depends on #1036.

Signed-off-by: yashrajbasav <yashrajbasav@nvidia.com>
Signed-off-by: yashrajbasav <yashrajbasav@nvidia.com>
Signed-off-by: yashrajbasav <yashrajbasav@nvidia.com>
Signed-off-by: yashrajbasav <yashrajbasav@nvidia.com>
Signed-off-by: yashrajbasav <yashrajbasav@nvidia.com>
Signed-off-by: yashrajbasav <yashrajbasav@nvidia.com>
Signed-off-by: yashrajbasav <yashrajbasav@nvidia.com>
Signed-off-by: yashrajbasav <yashrajbasav@nvidia.com>
Signed-off-by: yashrajbasav <yashrajbasav@nvidia.com>
Signed-off-by: yashrajbasav <yashrajbasav@nvidia.com>
Signed-off-by: yashrajbasav <yashrajbasav@nvidia.com>
Signed-off-by: yashrajbasav <yashrajbasav@nvidia.com>
Signed-off-by: yashrajbasav <yashrajbasav@nvidia.com>
Signed-off-by: yashrajbasav <yashrajbasav@nvidia.com>
Signed-off-by: yashrajbasav <yashrajbasav@nvidia.com>
Signed-off-by: yashrajbasav <yashrajbasav@nvidia.com>
Signed-off-by: yashrajbasav <yashrajbasav@nvidia.com>
Signed-off-by: yashrajbasav <yashrajbasav@nvidia.com>
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.

1 participant