Skip to content

feat(code-review): add semgrep clean-code runner#70

Merged
djm81 merged 1 commit intodevfrom
feature/code-review-05-semgrep-clean-code-rules
Mar 13, 2026
Merged

feat(code-review): add semgrep clean-code runner#70
djm81 merged 1 commit intodevfrom
feature/code-review-05-semgrep-clean-code-rules

Conversation

@djm81
Copy link
Copy Markdown
Contributor

@djm81 djm81 commented Mar 13, 2026

Summary

  • add a Semgrep runner and bundle-local clean-code rules for project-specific code-review findings
  • cover the new rules with bad/good fixtures, unit tests, and docs updates for the code-review bundle
  • bump and re-sign the specfact-code-review bundle manifest after the payload changes

Test plan

  • hatch run format
  • hatch run type-check
  • hatch run lint
  • hatch run yaml-lint
  • SPECFACT_CLI_REPO="/home/dom/git/nold-ai/specfact-cli" hatch run contract-test
  • SPECFACT_CLI_REPO="/home/dom/git/nold-ai/specfact-cli" hatch run smart-test
  • SPECFACT_CLI_REPO="/home/dom/git/nold-ai/specfact-cli" hatch run test
  • hatch run scan-all
  • hatch run verify-modules-signature --require-signature --enforce-version-bump
  • python scripts/publish-module.py --bundle specfact-code-review

Made with Cursor

Add Semgrep-based clean-code and architecture checks to the code-review bundle so review runs can flag project-specific anti-patterns with governed findings. Cover the new rules with bundle fixtures, unit tests, docs updates, and a versioned signed bundle manifest.

Made-with: Cursor
@chatgpt-codex-connector
Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

@djm81 djm81 self-assigned this Mar 13, 2026
@djm81 djm81 added enhancement New feature or request module Specfact Module related topic codebase Specfact codebase related topic labels Mar 13, 2026
@djm81 djm81 merged commit 6da71b6 into dev Mar 13, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

codebase Specfact codebase related topic enhancement New feature or request module Specfact Module related topic

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Change] code-review-05 - Project-Specific Semgrep Rules for Clean Code Patterns

1 participant