Skip to content

Split security package, add Groq happy E2E - #21

Merged
SemTiOne merged 2 commits into
mainfrom
feat/m1-coverage
Sep 4, 2026
Merged

Split security package, add Groq happy E2E#21
SemTiOne merged 2 commits into
mainfrom
feat/m1-coverage

Conversation

@SemTiOne

@SemTiOne SemTiOne commented Sep 3, 2026

Copy link
Copy Markdown
Owner

Split security.py into a package with no logic changes, plus one Groq happy-path E2E as a regression test.

@github-actions

github-actions Bot commented Sep 3, 2026

Copy link
Copy Markdown

♟️ chess-review-bot — PR Game Review

Accuracy: 43.0/100 · 8 files · +601/-428 · 2 Blunder??

File Category Why
standup/security.py Good no notable risk or quality signals
standup/security/__init__.py Mistake? change appears to be keyboard smash or low-entropy content
standup/security/_keyring.py Mistake? contains debug artifact (print, pdb, console.log, etc.)
standup/security/_permissions.py Blunder?? 3 possible credential(s) detected in diff
standup/security/_redact.py Blunder?? 4 possible credential(s) detected in diff
tests/test_ci_quality.py Excellent moderate change, no risk signals
tests/test_main_e2e.py Mistake? contains debug artifact (print, pdb, console.log, etc.)
tests/test_security.py Excellent moderate change, no risk signals

chess-review-bot scores diffs, not people. Run chessreview --format text locally for the full report.

@codecov

codecov Bot commented Sep 3, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 98.73418% with 2 lines in your changes missing coverage. Please review.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
standup/security/_permissions.py 96.72% 2 Missing ⚠️

📢 Thoughts on this report? Let us know!

@SemTiOne
SemTiOne merged commit 7b7cc7f into main Sep 4, 2026
18 checks passed
@SemTiOne
SemTiOne deleted the feat/m1-coverage branch September 4, 2026 01:33
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