Remove .zig Files from Bun

· ai · Source ↗

TLDR

  • Bun’s Zig-to-Rust port is complete; this PR deletes the now-dead .zig source files, keeping only those still required by the build pipeline.

Key Takeaways

  • The full Zig→Rust migration has merged; .zig files were retained post-port as a reference, not compiled code.
  • Seven .zig files survive because bindgen.ts requires a sibling .zig file next to every *.bind.ts file at build time.
  • src/runtime/bake/DevServer.zig stays because bake-codegen.ts reads it via convertZigEnum; others are pinned by $zig()/$newZigFunction() callsites in src/js/.
  • test/js/bun/util/zstd.test.ts is updated to point at parser.rs instead of the deleted parser.zig for its large-file fixture.
  • test/internal/ban-limits.json is regenerated to reflect the smaller surviving .zig file set.

Hacker News Comment Review

  • Commenters flagged that the PR was labeled “ai slop” by maintainer Jarred-Sumner, with the description replaced to prevent misleading reviewers, not because AI PRs are universally rejected.
  • One commenter quoted the PR’s own “Security risks: None. This is dead-code removal” line with a sardonic nod to Claude authorship, suggesting the PR description itself was AI-generated.

Original | Discuss on HN