Skip to content

release: engine 3.5.1 with Windows save-read fix - #913

Merged
apotema merged 1 commit into
mainfrom
codex/972-engine-release
Sep 30, 2026
Merged

apotema merged 1 commit into
mainfrom
codex/972-engine-release

Conversation

@apotema

@apotema apotema commented Sep 30, 2026 •

Copy link
Copy Markdown
Contributor

Prepare engine v3.5.1 so FP #972 can consume the merged Windows storage fix from #912 through a released URL/hash. The diff against main is one package-version line; storage code and dependencies are unchanged.

#912 merged as293c66483822ab32c3d2c4c25ebd6c85ad433e71 after Claude exact2c4b66d approval5916055242, green Linux/Windows CI and clean bot threads. Independent Windows storage17pass2skip; FP save regressions improve from321pass1skip6crashes to327pass1skip0crashes. Claude's macOS full suite1503pass and storage19pass. git diff --check passes for this metadata-only change; current-head CI must pass before merge/tag. No duplicate local build.

Coordinate through labelle-bgfx RFC172. After exact-head peer approval and green CI, merge, tag/release v3.5.1, independently fetch its Zig package hash, then prepare the FP released pin. No release has been published yet.


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 30, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-09-30T17:17:48.845472Z f717c5f PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@coderabbitai

coderabbitai Bot commented Sep 30, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Team

Run ID: a76ce7f3-9d35-4aeb-aa41-e66f61eb3cc0

📥 Commits

Reviewing files that changed from the base of the PR and between 293c664 and f717c5f.

📒 Files selected for processing (1)
  • build.zig.zon

Included review availability: This review used your included allowance. 0 included reviews remain after this review. Your included PR review attempts over the past 7 days set your current allowance at 1 review per hour.


📝 Walkthrough

Walkthrough

The package version in build.zig.zon changes from 3.5.0 to 3.5.1.

Changes

Package version update

Layer / File(s) Summary
Update package version
build.zig.zon
The package version changes from 3.5.0 to 3.5.1.

Priority: ➖ Normal

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

Change: Other

Suggested reviewers: leonardomag31-lab

Merge Risk: ⚪ Minimal · up to f717c

No merge-blocking issue is identified in this version-only change. Complete the stated current-head CI and release checks before merging and tagging.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly identifies the release version and the Windows save-read fix that the release enables.
Description check ✅ Passed The description directly explains the version bump, its purpose, validation results, and the release steps.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
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 unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Autopilot is currently an internal CodeRabbit preview.


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

@apotema

apotema commented Sep 30, 2026

Copy link
Copy Markdown
Contributor Author

VERDICT: APPROVED

Reviewed head: f717c5fe42f6220530b2a33297ae8b2ad98cf4c1. Reviewer: Claude (Anthropic LLM).

@apotema
apotema merged commit 72026a6 into main Sep 30, 2026
3 checks passed
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