Repository navigation
Release v3.88.0 - #1980
Release v3.88.0#1980
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configuration
📒 Files selected for processing (41)
Included review availability: This review used your included allowance. Your plan provides up to 4 included reviews per hour; 3 remain after this review. 📜 Recent review details🧰 Additional context used📚 Code guidelines (2)📓 Path-based instructions (8)For persisted settings, verify the complete schema/storage/runtime/webview round trip, shared default semantics, and focused true plus false/unset tests.⚙️ CodeRabbit configuration file Files:
Enforce repository policy: routine PRs must not add changesets or edit changelogs except during release preparation.⚙️ CodeRabbit configuration file Files:
Require regression coverage at the lowest valid harness with behavior-focused assertions, including relevant negative, error, false/unset, and boundary cases.⚙️ CodeRabbit configuration file Files:
Check strict typing and exhaustive behavior across normal, boundary, error, cancellation, retry, and compatibility paths.⚙️ CodeRabbit configuration file Files:
Check React state and effect dependencies, cleanup, accessibility, i18n, and light/dark theme behavior.⚙️ CodeRabbit configuration file Files:
Verify extension/webview contracts, cancellation and error propagation, VS Code lifecycle correctness, and behavior under retries and partial failure.⚙️ CodeRabbit configuration file Files:
Act as an adversarial second-opinion reviewer.⚙️ CodeRabbit configuration file Files:
Source excerpt: All notable changes to Zoo Code will be documented in this file.📄 CodeRabbit inference engine (src/CHANGELOG.md) Files:
🪛 GitHub Check: mutation-diffsrc/core/webview/ClineProvider.ts[warning] 329-329: Mutation test advisory 🪛 LanguageToollocales/tr/README.md[misspelling] ~50-~50: Söz ve sayı arasında defis yoqtur: "v-3" (NUMBER_BEFORE_DEFIS_MISSING) locales/it/README.md[uncategorized] ~52-~52: L’asterisco si usa per un’omissione nel numero di tre consecutivi: "***". (GR_09_001) [uncategorized] ~53-~53: L’asterisco si usa per un’omissione nel numero di tre consecutivi: "***". (GR_09_001) [uncategorized] ~54-~54: L’asterisco si usa per un’omissione nel numero di tre consecutivi: "***". (GR_09_001) locales/ca/README.md[style] ~53-~53: Si és el nom d’una operació tècnica, val més usar una altra forma. (NOMS_OPERACIONS_DESAT) 🔇 Additional comments (3)
📝 SummarySummary by CodeRabbit
WalkthroughThe release documentation and localized announcements now describe v3.88.0. The package version and announcement identifier are updated, and the announcement rendering test checks the new release text. ChangesZoo Code v3.88.0 release
Priority: ⬇️ Low Estimated code review effort: 1 (Trivial) | ~8 minutes Change: Other Merge Risk: ⚪ Minimal · up to The release-preparation changes present no identified blocker to merge. The described release approval and publication steps remain separate workflow requirements. Pre-merge checks |
  |
| Check name | Status | Explanation | Resolution |
|---|---|---|---|
| Regression Evidence |  |
The PR changes durable visible announcement content in all 18 chat.json locales and changes the announcement version shown by ClineProvider.latestAnnouncementId. Announcement.spec.tsx checks the… |
Add a Playwright component screenshot assertion for the rendered v3.88.0 announcement and commit its baseline snapshot. Load the real translation resources in that coverage, or add focused locale-resource assertions, so the changed announce… |
✅ Passed checks (7 passed)
| Check name | Status | Explanation |
|---|---|---|
| Linked Issues check |  |
Check skipped because no linked issues were found for this pull request. |
| Out of Scope Changes check | |
Check skipped because no linked issues were found for this pull request. |
| Security Boundaries | |
PASS. The pull request changes release documentation, localized announcement strings, the package version, the static announcement ID, and its test. The announcement ID only controls display state and… |
| Persistence Integrity | |
The changed-file diff contains release metadata, changelogs, README files, locale strings, one announcement ID, and announcement test data. It does not change a persistence operation or consumer. Pers… |
| Lifecycle Resource Cleanup | |
The pull request changes release documentation, localized announcement strings, package version metadata, announcement test data, and ClineProvider.latestAnnouncementId. The changed announcement ID … |
| Title check | |
The title, “Release v3.88.0,” clearly and concisely identifies the primary change: preparing the v3.88.0 release. |
| Description check | |
The description is detailed and relevant. It explains the release scope, changed files, validation results, and release workflow. It does not explicitly include an approved issue number or the templat… |
Full details: Regression Evidence
Explanation
The PR changes durable visible announcement content in all 18 chat.json locales and changes the announcement version shown by ClineProvider.latestAnnouncementId. Announcement.spec.tsx checks the new English strings, but it mocks useAppTranslation, so it does not exercise the changed locale resources. The existing Playwright component test Announcement.links.visual.tsx checks link messages only; it does not call toHaveScreenshot, and no announcement snapshot exists under __screenshots__. The changed announcement rendering therefore lacks the required focused Playwright component snapshot.
Resolution
Add a Playwright component screenshot assertion for the rendered v3.88.0 announcement and commit its baseline snapshot. Load the real translation resources in that coverage, or add focused locale-resource assertions, so the changed announcement strings are tested instead of only duplicated in the unit-test mock.
- Fix all pre-merge checks with AI
✨ Finishing Touches
🧪 Generate unit tests (beta)
- Commit to this branch
- Create a new PR
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.
Comment @coderabbitai help to get the list of available commands.
Review statusThanks for contributing. This comment tracks the review sequence and the next action. Current step: The required review sequence passed. Remaining merge requirements apply. Review-state labels are managed by this workflow; do not edit them manually. |
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
github-actions
Bot
added
coderabbit-review-active
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
github-actions
Bot
added
awaiting-maintainer
github-actions
Bot
removed
the
awaiting-maintainer
Release v3.88.0
Release preparation for Zoo Code v3.88.0. Base: main @
b7ab5a83e930d0ed59c608d4a8d77436d5c21bfa.Contents
3.88.0insrc/package.json. No separate version bump PR is needed.CHANGELOG.mdandsrc/CHANGELOG.mdcarry the same## [3.88.0]section.README.md"What's New in v3.88.0". The build bundle copies it tosrc/README.md.chat.jsonlocales, with the unique announcement IDoct-2026-v3.88.0-providers-tasks-speedand the updatedAnnouncement.spec.tsx.Note: PR #1897 ("Release v3.86.0") is prior-release housekeeping. It merged after v3.86.0 publication, so the changelog labels it accordingly.
Validation
turbo lint): 11/11 packages pass. No ESLint suppression count increases.turbo check-types): 11/11 packages pass.zoo-code, publisherZooCodeOrganization, version3.88.0.src/README.mdequals the root README.Release workflow gate
No tag exists yet. Publication has not started.