Skip to content

fix(toolchain): package Zig requirement - #52

Merged
GenericJam merged 1 commit into
masterfrom
fix/packaged-zig-requirement
Aug 30, 2026
Merged

fix(toolchain): package Zig requirement#52
GenericJam merged 1 commit into
masterfrom
fix/packaged-zig-requirement

Conversation

@GenericJam

@GenericJam GenericJam commented Aug 30, 2026

Copy link
Copy Markdown
Owner

Summary

  • keep the exact Zig requirement in shipped MobDev source instead of compile-time reading a repository-only dotfile
  • enforce exact lockstep with the root toolchain pin in source tests
  • unpack the real Hex artifact, compile it fresh without a packaged .tool-versions file, and probe doctor/native exact, mismatch, and failed-command behavior

Validation

  • full suite: 2162 passed, 10 excluded
  • packed Hex compile and runtime preflight regression
  • format, warnings-as-errors compile, strict Credo, erlfmt, and pre-push checks
  • local-only security scan: zero source findings; strict exit remains from six pre-existing cached-runtime manifest-drift findings

Bead: app-840

Keep the exact Zig requirement in shipped source and verify an unpacked Hex artifact compiles and retains doctor and native preflight behavior.

Bead: clarity-840
@GenericJam

Copy link
Copy Markdown
Owner Author

Reviewed: inlining the pin into packaged source with a lockstep test against root .tool-versions is the right call for a lib package (no priv indirection needed), and the packed-artifact regression (fetch layout, confirm dotfile absent, fresh compile + doctor/native probes) permanently covers the failure mode 0.6.29 shipped with. Merging and cutting 0.6.30.

@GenericJam
GenericJam merged commit c80f195 into master Aug 30, 2026
3 checks passed
GenericJam added a commit that referenced this pull request Sep 1, 2026
Keep the exact Zig requirement in shipped source and verify an unpacked Hex artifact compiles and retains doctor and native preflight behavior.

Bead: app-840
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