release: prepare v0.1.1252 RC for staging verification - #4012
Conversation
📦 Client bundle boundary
A server module in a client graph aborts hydration in the browser. New leaks fail CI; known leaks are tracked in |
|
Warning Review limit reached
Next review available in: 24 minutes Limit details: You’ve used the included review currently available. You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. How can I continue?Wait for the limit to reset, then comment An organization admin can change what happens after included review limits in Billing. How do review limits work?CodeRabbit enforces per-developer PR review limits within each organization. For paid Pro and Pro+ reviews, CodeRabbit uses a developer's included PR review attempts over the past 7 days to set the current hourly allowance. At typical activity levels, the full plan allowance applies. Higher sustained activity can lower the allowance until earlier attempts leave the 7-day window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (2)
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 |
|
Reviewed. The bump itself is clean. The three red jobs are not this PR's fault, but they are worth naming because they will now be red on every release PR. The bump is complete and matches the documented procedure
Nothing rides along: the diff is 3 files, 1 line each, all of it the version string. Why the runtime critical flow jobs fail
The gate packs the local CLI at the new version, runs So the gate is structurally unsatisfiable on any version-bump PR. It is green on This is new, not a regression here. The gate arrived with None of the three is a required check ( The part worth fixing separately: left alone, three jobs go red on every RC from now on, and a gate that is always red on a known class of PR stops being read on the ones where it matters. The fix belongs in the flow script, resolving first-party extensions from the packed local tarballs rather than from npm, or skipping the install leg when the version under test is unpublished. Worth its own issue rather than holding this RC. |
|
@codex review |
|
Codex Review: Didn't find any major issues. Already looking forward to the next diff. Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
Summary
0.1.1252-rcprerelease line somainpublishes numbered RCs againWhy this is needed now
deno.jsonreads a clean0.1.1251, and0.1.1251is already published to npm(
npm view veryfront@0.1.1251 gitHead→0f38b23eb7). The release job readsdeno.json, sees noversion change, and correctly no-ops.
src/utils/version.test.tsassertsVERSION === deno.json.versionbut not that it increased,so CI stays fully green the whole time. The symptom is silent: merged, green, and shipped nowhere.
21 commits currently sit above the published version and have reached no environment, including:
3bb52bbc88fix(provider): retry stream header timeouts before output (fix(provider): retry stream header timeouts before output #3993) — the chore: replace --allow-all with explicit permissions in test tasks #710 fix223c2a9faffix(sandbox): encode worker bytes on supported Node (fix(sandbox): encode worker bytes on supported Node #3987) — issue-inbox#7327f23c8f645fix(workflow): fail closed when a child graph stalls (fix(workflow): fail closed when a child graph stalls #3994)45d277d149fix(workflow): charge crash recovery when a node starts, not when it queues (fix(workflow): charge crash recovery when a node starts, not when it queues #3981)9497625e44fix(agent): time out hung service heartbeats (fix(agent): time out hung service heartbeats #3990)c43cc407c0fix(renderer): publish cycle artifacts atomically (fix(renderer): publish cycle artifacts atomically #3996)f8dbecb0c1/7eee200268bothfix(init)fixes9d6cdefb86test: gate the critical app path across Deno, Node, and Bun (test: gate the critical app path across Deno, Node, and Bun #3977)1f470fab50test(node): run portable API loader coverage (test(node): run portable API loader coverage #3989)Verification
deno task generate:manifests:checkdeno task fmt:checkdeno test -A src/config/cicd-stable-release.test.ts src/utils/version.test.ts(
var VERSION = "0.1.1251"→"0.1.1252-rc"); diff is 3 files, 1 line eachAfter this
RC publishes → server / job-runner / sandbox staging deploys → strict staging Remote E2E Health →
stable
0.1.1252→ production promotion. Production currently runs0.1.1248.