Skip to content

Build Zwanzig as a host lint dependency - #348

Merged
forketyfork merged 2 commits into
mainfrom
agent/update-zwanzig-usage
Jul 22, 2026
Merged

Build Zwanzig as a host lint dependency#348
forketyfork merged 2 commits into
mainfrom
agent/update-zwanzig-usage

Conversation

@forketyfork

@forketyfork forketyfork commented Jul 22, 2026

Copy link
Copy Markdown
Owner

Summary

Architect pinned Zwanzig v0.13.1 and built the linter for the application's target. That works for native builds, but a cross-compiled build can produce a Zwanzig executable that the host cannot run.

This updates the integration to the documented pattern:

  • bump Zwanzig to v0.14.0 with its fetched Zig package hash
  • build the linter for b.graph.host in ReleaseFast
  • forward the application's target architecture and operating system to Zwanzig through --target
  • expose the dependency through the direct zig build lint setup
  • document Zwanzig as a build-time dependency

Validation

  • zig build
  • zig build test
  • nix develop --command just lint
  • nix develop --command zig build lint -Dtarget=x86_64-linux-gnu

@forketyfork
forketyfork marked this pull request as ready for review July 22, 2026 11:27

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 530421700c

ℹ️ 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".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread build.zig Outdated
@forketyfork
forketyfork merged commit 1776c8f into main Jul 22, 2026
4 checks passed
@forketyfork
forketyfork deleted the agent/update-zwanzig-usage branch July 22, 2026 12:32
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