release: prepare v8.0.4#94
Merged
Merged
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (5)
✅ Files skipped from review due to trivial changes (5)
📝 WalkthroughWalkthroughRemoved the PermissionRequest Codex hook probe from the plugin to address security concerns, updated SpeakSwiftly to 9.0.2 and TextForSpeech to 0.22.1, incremented plugin version to 8.0.4, updated hook validation tooling, and added security audit documentation with roadmap follow-up tracking. ChangesPermissionRequest Hook Removal and v8.0.4 Release
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 2
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
docs/maintainers/skills-surface-audit.md (1)
3-3:⚠️ Potential issue | 🟡 Minor | ⚡ Quick winUpdate the
Last auditeddate to this audit pass.
Last audited: 2026-05-04looks stale given this PR’s security-audit follow-through updates on 2026-05-12. Please bump the date so maintainer metadata stays accurate.🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@docs/maintainers/skills-surface-audit.md` at line 3, Update the maintainer metadata by changing the "Last audited: 2026-05-04" entry in docs/maintainers/skills-surface-audit.md to "Last audited: 2026-05-12" so the file reflects this PR’s audit date; locate the literal "Last audited" line and replace the date accordingly.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@API.md`:
- Line 229: The Supported Versions matrix in API.md lists TextForSpeech as
0.22.0 but the PR updates it to 0.22.1; update the version string for
TextForSpeech in the document (replace "TextForSpeech from 0.22.0" with
"TextForSpeech from 0.22.1") so the Supported Versions section matches the
resolved dependency in this PR and keep the rest of the version entries
unchanged.
In `@docs/maintainers/plugin-install-testing.md`:
- Around line 44-45: The sentence describing hook doctor output is duplicated
and ambiguous; update the line that currently reads "The hook doctor reports one
installed-cache dispatcher command for `Stop` and one for `Stop`." to explicitly
state there are two such commands — e.g. "The hook doctor reports two
installed-cache dispatcher commands for `Stop`." — so readers clearly understand
the expectation from the hook doctor output.
---
Outside diff comments:
In `@docs/maintainers/skills-surface-audit.md`:
- Line 3: Update the maintainer metadata by changing the "Last audited:
2026-05-04" entry in docs/maintainers/skills-surface-audit.md to "Last audited:
2026-05-12" so the file reflects this PR’s audit date; locate the literal "Last
audited" line and replace the date accordingly.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro Plus
Run ID: 28912ab4-3882-4b9f-8446-77e4da632b49
📒 Files selected for processing (19)
.codex-plugin/plugin.json.codex/hooks.jsonAGENTS.mdAPI.mdPackage.resolvedPackage.swiftROADMAP.mddocs/codex-hooks-tts.mddocs/maintainers/plugin-install-testing.mddocs/maintainers/skills-surface-audit.mddocs/maintainers/source-layout.mddocs/releases/v8.0.4-release-notes.mddocs/security-audits/2026-05-12-repo-wide-audit.mddocs/security-audits/README.mdhooks/hooks.jsonhooks/permission-request-log.mjsscripts/codex-hooks-doctor.mjsscripts/codex-hooks-doctor.test.mjsskills/speak-swiftly-codex-hooks/SKILL.md
💤 Files with no reviewable changes (3)
- .codex/hooks.json
- hooks/permission-request-log.mjs
- scripts/codex-hooks-doctor.mjs
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Release
security/repo-wide-auditmainupdates behind pull request review and CIv8.0.4will be created after CI and the review-comment gate pass, so failed or still-discussed release candidates do not get taggedReview Loop
Before merge and tagging,
scripts/repo-maintenance/release.shwatches CI and stops on review comments unless the maintainer has already addressed or resolved them and reruns with--review-comments-addressed.Summary by CodeRabbit
Chores
Documentation