Skip to content

chore: pin labelle-assembler 0.104.0 - #377

Merged
apotema merged 1 commit into
mainfrom
chore/pin-assembler-0.104.0
Sep 6, 2026
Merged

apotema merged 1 commit into
mainfrom
chore/pin-assembler-0.104.0

Conversation

@apotema

@apotema apotema commented Sep 6, 2026 •

Copy link
Copy Markdown
Contributor

Routine pin bump, same shape as #374.

0.104.0 completes the #685 family: the discovered local framework slot is now keyed by checkout (assembler#696). Previously packages/local/<pkg> was one shared name regardless of which checkout populated it — activation was provenance-gated but population wasn't, so two clones (or a monorepo with wt-* worktrees) repointed the slot at each other on every build, and two concurrent builds could have one resolve the slot while the other repointed it mid-generate.

This bump is what gets that fix to projects with no assembler_version pin and no LABELLE_ASSEMBLER override.

Verified before pinning: all five 0.104.0 release assets return 200 ({linux,macos}-{x86_64,aarch64} and windows-x86_64), so no platform 404s on the cold-download path.

Local: 6/6 steps succeeded; 786/811 tests passed (25 skipped), zero compile errors, zero test failures — identical to main. The released-path init smoke in ci.yml is what exercises this constant end to end.

https://claude.ai/code/session_01AM5hJ7sWCZNrKWo1xMJDEA


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

0.104.0 completes the #685 family: the discovered local framework slot is
keyed by checkout (assembler#696), so two clones — or a monorepo with `wt-*`
worktrees — no longer repoint one shared `packages/local/<pkg>` at each other
on every build. Two concurrent builds could previously have one resolve the
slot while the other repointed it mid-generate.

Nothing in the CLI changes shape; this is the usual dedicated pin bump so
projects with no `assembler_version` and no LABELLE_ASSEMBLER override get the
fix. All five 0.104.0 release assets were checked for 200 before pinning.

Claude-Session: https://claude.ai/code/session_01AM5hJ7sWCZNrKWo1xMJDEA
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 6, 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-06T11:30:07.542773Z 7b97234 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 6, 2026 •

Copy link
Copy Markdown

Review Change Stack

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: 8db8ab21-71cc-4940-ac65-674e6ac82965

📥 Commits

Reviewing files that changed from the base of the PR and between 660f407 and 7b97234.

📒 Files selected for processing (1)
  • src/cli/assembler.zig

Included review availability: 1 review is currently available. Your included PR review attempts over the past 7 days set your current allowance at 2 reviews per hour.


📝 Walkthrough

Walkthrough

The CLI default assembler version changes from 0.103.0 to 0.104.0. Other assembler resolution, download, caching, local-build, worktree, and test behavior remains unchanged.

Changes

Assembler version

Layer / File(s) Summary
Update default assembler version
src/cli/assembler.zig
DEFAULT_ASSEMBLER_VERSION now uses 0.104.0 for fallback resolution and downloads.

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

Merge Risk: ⚪ Minimal · up to 7b972

The default assembler release advances to 0.104.0 for unpinned installations, with validation completing successfully and no remaining merge-readiness risk.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely identifies the main change: pinning labelle-assembler to version 0.104.0.
Description check ✅ Passed The description directly explains the version bump, the included assembler fix, validation results, and affected projects.
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)
  • Create PR with unit tests
  • Commit unit tests in branch chore/pin-assembler-0.104.0

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

@apotema
apotema merged commit a8d9e35 into main Sep 6, 2026
8 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