Skip to content

release: v0.1.1236 - #3702

Closed
kojiwakayama wants to merge 1 commit into
mainfrom
release/v0.1.1236
Closed

release: v0.1.1236#3702
kojiwakayama wants to merge 1 commit into
mainfrom
release/v0.1.1236

Conversation

@kojiwakayama

@kojiwakayama kojiwakayama commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

Release v0.1.1236

Ships the two fixes for the staging preview outage.

#3698 — bind the esbuild binary path outside project scope. esbuild resolves its binary once, when its module evaluates. The adapter imported it lazily, so that read landed inside a project env scope on the first transform, saw undefined, and no esbuild service ever started — every TSX transform failed for the life of the process. Importing at startup binds the path while the host environment is still on process.env, with project env isolation untouched.

#3697 — keep the failure that caused the ownership error. The ownership latch was created without a cause and ??= then discarded the real error, so every genuine esbuild failure surfaced as a lifecycle message. That masking is why the above took a day to find.

Release hygiene

  • Exactly the 3 required files, all three agreeing on 0.1.1236.
  • deno.json diff is the version line only — the release script's unrelated reformatting of two dependencies arrays was reverted.
  • deno task generate run and amended in; the script alone produced only 2 files.
  • No local tag — CI creates it.

After merge

This does not complete the fix. Staging needs the release published, a server artifact rebuilt on it, and a deploy, before preview recovers. I'll verify against the deployed pods and logs rather than assuming the chain ran, then dispatch a staging Remote E2E Health run before merging veryfront-e2e#139.

Summary by CodeRabbit

  • Chores
    • Updated the application version to 0.1.1236.

@github-actions

Copy link
Copy Markdown

📦 Client bundle boundary

Entrypoint Modules Source size Server leaks
src/index.client.ts 454 3062 KiB ⚠️ 39 known

A server module in a client graph aborts hydration in the browser. New leaks fail CI; known leaks are tracked in scripts/lint/client-bundle-baseline.json to burn down.

@coderabbitai

coderabbitai Bot commented Aug 14, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 0fa656b1-d73a-4fec-83fd-ae10d18aaf5f

📥 Commits

Reviewing files that changed from the base of the PR and between c161dc6 and de269af.

⛔ Files ignored due to path filters (1)
  • src/html/hydration-script-builder/hydration-runtime.generated.ts is excluded by !**/*.generated.*
📒 Files selected for processing (2)
  • deno.json
  • src/utils/version-constant.ts

📝 Walkthrough

Walkthrough

The release version changes from 0.1.1235 to 0.1.1236 in package metadata and the exported VERSION constant.

Changes

Version synchronization

Layer / File(s) Summary
Update release version
deno.json, src/utils/version-constant.ts
The package version and exported VERSION constant now use 0.1.1236.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to de269

This release updates the version consistently across the required generated and configuration files; no actionable merge-blocking risk remains after normal checks and review.

Possibly related PRs

Suggested reviewers: kwakayama

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly identifies the version release reflected by the package and exported VERSION updates.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch release/v0.1.1236

Comment @coderabbitai help to get the list of available commands.

@kojiwakayama

Copy link
Copy Markdown
Contributor Author

Closing as a duplicate of #3701, which was cut first (10:15Z vs 10:26Z).

I checked before deferring rather than assuming: #3701 is correctly formed — exactly the 3 required files, all three agreeing on 0.1.1236, and its deno.json diff is the version line only.

Worth stating clearly, because the branch histories differ but the released content does not: both #3697 and #3698 are already on main, so whichever release PR merges, the published 0.1.1236 contains both fixes. #3701 not having c161dc6 as a branch ancestor changes nothing about what ships.

No reason to prefer mine. Verification of the staging recovery continues against #3701.

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