ReviewOS

also looking at this

craft-native/craft

fix(build): select the core Zig project explicitly

#383
Merged chris wants to merge codex/zig-core-build-root into main
4 files +44 -2

Review threads live on the whole diff, not on one commit, so none are shown here - a thread's line means something in the branch's final form, and painting it into an intermediate step would put it on code it is not about.

scripts/verify-zig-core-project.tsmodified+2-1
Changes to scripts/verify-zig-core-project.ts
@@ -1,10 +1,11 @@
1import { mkdtempSync, mkdirSync, readFileSync, rmSync, writeFileSync } from 'node:fs'
1import { existsSync, mkdtempSync, mkdirSync, readFileSync, rmSync, writeFileSync } from 'node:fs'
22import { tmpdir } from 'node:os'
33import { join, resolve } from 'node:path'
44import { resolveZig } from './zig'
55
66const source = resolve(import.meta.dir, '..')
77const zig = resolveZig(source)
8console.log('Tracked core project:', JSON.stringify({ file: join(source, 'packages/zig/build.zig'), exists: existsSync(join(source, 'packages/zig/build.zig')) }))
89const root = mkdtempSync(join(tmpdir(), 'craft-core-project-'))
910const core = join(root, 'packages', 'zig')
1011try {