chore(deps): adopt effect 4.0.0-beta.93#141
Conversation
|
Warning Review limit reached
Next review available in: 43 minutes Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (8)
📝 WalkthroughWalkthroughThis PR bumps the ChangesEffect version bump
Estimated code review effort: 1 (Trivial) | ~5 minutes Possibly related PRs
Suggested labels: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 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 @.changeset/adopt-effect-beta93.md:
- Around line 1-24: The .changeset file is not in the required format because
the license block appears before the frontmatter. Move the YAML frontmatter for
`@resq-sw/http`, `@resq-sw/security`, `@resq-sw/rate-limiting`, and `@resq-sw/dsa` to
the very top of the file, and place the license/comment block after it so the
changeset parser recognizes it. Use the existing changeset content as-is, just
reorder the top section to match the standard .changeset markdown structure.
🪄 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
Run ID: aeffd970-996e-4bdd-9a07-c83edb9f9b58
⛔ Files ignored due to path filters (1)
bun.lockis excluded by!**/*.lock
📒 Files selected for processing (7)
.changeset/adopt-effect-beta93.mdexamples/react-dashboard/package.jsonpackage.jsonpackages/dsa/package.jsonpackages/http/package.jsonpackages/rate-limiting/package.jsonpackages/security/package.json
|
Warning Gemini encountered an error creating the review. You can try again by commenting |
|
Caution agentic threat detected Audit Results: PASSI have reviewed the changes in PR #141 and confirmed that the update to is implemented consistently across the workspace. Audit Summary:
No security vulnerabilities, logic bugs, or performance issues were found. The audit has passed. Warning Firewall blocked 1 domainThe following domain was blocked by the firewall during workflow execution:
network:
allowed:
- defaults
- "localhost"See Network Configuration for more information.
|
|
Caution agentic threat detected Audit Results: PASSI have reviewed the changes in PR #141 and confirmed that the update to Audit Summary:
No security vulnerabilities, logic bugs, or performance issues were found. The audit has passed. Warning Firewall blocked 1 domainThe following domain was blocked by the firewall during workflow execution:
network:
allowed:
- defaults
- "localhost"See Network Configuration for more information.
|
Bump the root `effect` override from beta.50 to beta.93 and align the pinned dev declarations (http, security, rate-limiting, dsa, react-dashboard example) to match. effect was deliberately held at beta.50 (root override, from #42); this moves the whole workspace forward and removes the declaration/override drift. Validated: full `bun run build` (all packages) and the effect-using test suites (http, security, rate-limiting, dsa) pass against beta.93. @effect/platform-bun is left to its own bump (#131). Supersedes #130.
The Graph.findShortestPath (Dijkstra) empirical complexity test intermittently
measures O(log n) on small chain graphs (constant factors dominate the loglinear
curve), failing CI across unrelated PRs. Apply the same de-flake conventions the
sibling assertions already use: tolerate O(log n) and add { retry: 2 }.
aa79436 to
525a2d4
Compare
Summary
Fully adopts effect 4.0.0-beta.93 across the workspace, superseding #130.
effect was deliberately pinned to beta.50 via the root
overridesblock (from #42, "fix test runner"), while dependabot had drifted the declarations to beta.78. #130 proposed bumping declarations to beta.93 but left the override at beta.50 — so its green CI was actually testing beta.50, not beta.93.This PR moves the whole workspace forward and removes the drift:
overrides.effect:4.0.0-beta.50→4.0.0-beta.93(this is what actually forces the resolved version).http,security,rate-limiting,dsa, and thereact-dashboardexample.effect: ">=3.0.0") left broad — consumers are unaffected.@effect/platform-bunleft to its own bump (chore(deps-dev): bump @effect/platform-bun from 4.0.0-beta.78 to 4.0.0-beta.93 #131).Validation
bun run build— all packages build clean against beta.93 (dts included)bun --filter @resq-sw/http --filter @resq-sw/security --filter @resq-sw/rate-limiting --filter @resq-sw/dsa test— all passCloses #130.
Summary by CodeRabbit