Skip to content

Semgrep tool test#132

Closed
PhilipD8 wants to merge 66 commits into
mainfrom
semgrep-tool-test
Closed

Semgrep tool test#132
PhilipD8 wants to merge 66 commits into
mainfrom
semgrep-tool-test

Conversation

@PhilipD8

@PhilipD8 PhilipD8 commented Mar 12, 2026

Copy link
Copy Markdown

Pros

  • Lightweight integration

  • Easy to save results with a small addition to package.json:
    "scripts": { "semgrep": "semgrep scan --config auto" }

  • Has flexible outputs (supports JSON as well), making it convenient for future CI/CD use

  • Before running, you have flexible priori customization settings on what to ignore (for example: ".semgrepignore"

Cons

  • Needs to be continuously customized to control what outputs are being given (might be too noisy)
  • Installation is a bit complicated

Q2L2 and others added 30 commits February 8, 2026 19:38
…here I needed to ad everything and rerouted the inital compser file to the one that i added into vendor
Adding HTML and CSS for a make visible to  staff-only Button
Added Mark Endorsed Button to the posts' tools dropdown menu
Anonymous button front-end implemented
Mr-Weather and others added 26 commits February 27, 2026 01:46
Anonymous button fully implemented. 
Coverage error persisting, but besides that all tests are passing.
feat: complete backend question status implementation
Change the button formatting so that on click it becomes red/white so that users know which state it is in
Completed implementing the backend for the endorse post button as a new feature for the topics tools.
The staff only button was implemented such that if clicked other users except admin and poster cannot see the post
Added Endorse Topic Feature user guide to UserGuide.md, explaining how to use the feature as an admin and how to conduct user-testing on the feature.
Added documentation for the Staff Only Post feature,
Document Staff Only Post feature and user testing
Add user configuration to NodeBB service (reci 7)
@PhilipD8

PhilipD8 commented Mar 18, 2026

Copy link
Copy Markdown
Author
  • The tool being used is Semgrep. This tool identifies security vulnerabilities, code smells, and repeated patterns of bugs in code. Link to documentation.
  • Semgrep is a static analysis tool
  • Semgrep catches security vulnerabilities (like risks of code injections, etc.), unsafe hardcoded sensitive data, incorrect use of language features, as well as other general code quality issues.
  • Custom rules can be defined in ".semgrep.yml", specific files can be ignored when compiling in ".semgrepignore", and the output format can be changed as well.
  • Semgrep can be integrated locally using npm scripts either before you start to code, or right before to you merge to check for code issues.
  • Most flagged issues for the pattern-based code detection are technically valid, but they often aren't relevant, so technically, the false positive rate is quite low. Additionally, the pattern-based code detection might miss certain patterns not covered by the rules. The default configurations flag a lot of irrelevant issues, meaning there is a moderate amount of true positives that we don't care about.

@PhilipD8 PhilipD8 closed this Mar 19, 2026
@PhilipD8 PhilipD8 reopened this Mar 19, 2026
@PhilipD8 PhilipD8 closed this Mar 19, 2026
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.

4 participants