Repository navigation
chore: release labelle-assembler 0.126.0 for lazy root build deps and inspector outlining - #826
Conversation
… inspector outlining Version-only release of #824 (manifest-v2 optional `.lazy` on root build deps; lazy-safe emsdk preflight, fixes #823) and #822 (debug entity inspector outlining, labelle-engine#922), part of labelle-toolkit/labelle-cli#528. Claude-Session: https://claude.ai/code/session_01Hpt1haxcboZPznpG1KqDRS
|
No actionable comments were generated in the recent review.
|
| Layer / File(s) | Summary |
|---|---|
Update package version build.zig.zon |
The manifest package version changes from 0.125.0 to 0.126.0. |
Estimated code review effort: 1 (Trivial) | ~2 minutes
Change: Other · Severity of issue fixed: Medium
Possibly related issues
- labelle-toolkit/labelle-assembler#823 — The release includes manifest-v2 lazy dependency support related to the reported eager emsdk download, although the provided summary does not confirm that it fully resolves the issue.
Possibly related PRs
- labelle-toolkit/labelle-assembler#824 — This release includes the lazy dependency and emsdk preflight changes from that PR.
- labelle-toolkit/labelle-assembler#822 — This release includes the entity inspector code-size changes from that PR.
Merge Risk: 
Minimal · up to 10682
-
No actionable merge-blocking risk was identified in this version-only release change.
Pre-merge checks: 
5
-
Full details: Docstring Coverage
-
Explanation
No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)
-
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
A rabbit checks the version line,
From zero-one-two-five to six,
The manifest records the change,
No extra edits join the mix.
Then hops away beneath the moon.
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
Version-only release of merged #824 and #822 (
build.zig.zon0.125.0 -> 0.126.0):.lazy = trueon a root build dep (manifest-v2RootBuildDep.lazy, default false); the generatedbuild.zig.zonentry is then lazy, so Zig fetches it only when a build script callsb.lazyDependency. The emittedensureEmsdkActivatedpreflight resolves emsdk withb.lazyDependency(...) orelse return, so it works for lazy and eager entries. Manifests without the field emit byte-identical zon; older assemblers ignore the field.Curated framework pins and version floors remain unchanged (the new manifest field is optional and backward compatible, so no floor applies).
Part of labelle-toolkit/labelle-cli#528.
https://claude.ai/code/session_01Hpt1haxcboZPznpG1KqDRS
Summary by CodeRabbit