Skip to content

ci: add sem semantic diff to pre-commit and PR review#9

Merged
matthiasdebernardini merged 1 commit intomainfrom
ci/sem-review
Mar 8, 2026
Merged

ci: add sem semantic diff to pre-commit and PR review#9
matthiasdebernardini merged 1 commit intomainfrom
ci/sem-review

Conversation

@matthiasdebernardini
Copy link
Copy Markdown
Owner

Summary

  • Add sem (entity-level Rust diffs) as an advisory pre-commit hook showing semantic changes on staged .rs files
  • Add sem-review.yml CI workflow that posts a sticky PR comment with the semantic diff summary, highlighting deleted entities
  • Add .coderabbit.yaml with project Rust style rules (functional/iterator style, no mutable vars)
  • Add justfile recipes: setup-tools (install sem), sem-diff, sem-diff-json

Test plan

  • pre-commit run sem-diff --all-files passes locally
  • zizmor --no-online-audits . passes with no findings
  • Verify sem-review.yml workflow runs on this PR and posts a comment
  • Verify CodeRabbit activates with the .coderabbit.yaml config

🤖 Generated with Claude Code

Integrate sem (entity-level Rust diffs) in two places:
- Pre-commit hook: advisory sem diff on staged .rs files
- CI workflow: posts semantic diff as sticky PR comment

Also adds CodeRabbit config with project Rust style rules
and justfile recipes for setup-tools, sem-diff.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Mar 8, 2026

Semantic Diff

Entity-level changes detected by sem:

�[2mNo changes detected.�[0m

@matthiasdebernardini matthiasdebernardini merged commit 588fe96 into main Mar 8, 2026
7 checks passed
@matthiasdebernardini matthiasdebernardini deleted the ci/sem-review branch March 8, 2026 04:17
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