refactor(zig-cookbook): adopt Zig 0.16 I/O system by jiacai2050 · Pull Request #122 · zigcc/zig-cookbook

jiacai2050 · 2026-04-17T09:13:30Z

Refactor main entry points to accept std.process.Init. Migrate file I/O, networking, and crypto operations to use the new std.Io interface. Removes zigcli dependency and bumps minimum Zig version to 0.16.0.

gemini-code-assist

Code Review

This pull request updates the cookbook examples to be compatible with Zig 0.16.0, migrating from legacy std.fs and std.net APIs to the new std.Io system. Key changes include updating main signatures to accept std.process.Init, utilizing std.Io for file and network operations, and replacing std.Thread.Pool with Io.Group. A compilation error was identified in assets/src/08-02.zig where a union tag was incorrectly cased as exited instead of Exited.

Copilot

Pull request overview

This PR migrates the zig-cookbook examples and build harness to Zig 0.16.0, adopting the new std.Io-based I/O model and std.process.Init entrypoint pattern while removing the zigcli dependency.

Changes:

Bump minimum Zig version to 0.16.0 and remove the zigcli dependency.
Refactor many example main functions to accept std.process.Init and route allocations/I/O through init.gpa and init.io.
Update filesystem, networking, process spawning, and crypto examples to use Zig 0.16 std.Io APIs and updated build/link APIs.

Reviewed changes

Copilot reviewed 27 out of 27 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
build.zig.zon	Bumps minimum Zig version to 0.16.0 and removes `zigcli` from dependencies.
build.zig	Updates directory iteration to `std.Io` and adjusts linking/build logic; skips the prior `zigcli`-dependent example.
assets/src/14-03.zig	Switches `main` to `std.process.Init` and uses `init.gpa` for allocations.
assets/src/12-03.zig	Switches `main` to `std.process.Init` and uses `init.gpa`.
assets/src/12-02.zig	Switches `main` to `std.process.Init` and uses `init.gpa`.
assets/src/10-03.zig	Switches `main` to `std.process.Init` and uses `init.gpa`.
assets/src/10-02.zig	Updates ZON parsing to allocation-based APIs and frees via `zon.parse.free`; uses `init.gpa`.
assets/src/10-01.zig	Switches `main` to `std.process.Init` and uses `init.gpa`.
assets/src/08-02.zig	Replaces manual child process handling with `std.process.run(gpa, io, ...)` and frees outputs.
assets/src/07-04.zig	Replaces removed `std.once` with an atomic-based “call once” implementation.
assets/src/07-03.zig	Migrates thread pool example to `std.Io.Group` async/await style.
assets/src/07-02.zig	Migrates mutex usage to `std.Io.Mutex` and threads use `init.io`.
assets/src/06-01.zig	Migrates crypto RNG usage to `std.Random.IoSource` backed by `init.io`.
assets/src/05-03.zig	Migrates HTTP server example to `std.Io.net` sockets/streams and `io`-based readers/writers.
assets/src/05-02.zig	Migrates HTTP client example to `http.Client` configured with `{ .allocator = gpa, .io = io }`.
assets/src/05-01.zig	Migrates HTTP client example to `http.Client` configured with `{ .allocator = gpa, .io = io }`.
assets/src/04-03.zig	Migrates UDP example from `std.posix` socket calls to `std.Io.net` datagram socket APIs.
assets/src/04-02.zig	Migrates TCP client and argv parsing to `init.minimal.args` and `std.Io.net`.
assets/src/04-01.zig	Migrates TCP server to `std.Io.net` and stream reader/writer APIs using `io`.
assets/src/03-01.zig	Migrates timing/sleep to `std.Io.Clock` and `Io.sleep`.
assets/src/02-03.zig	Migrates secure randomness to `std.Io.randomSecure` and threads `io` into Argon2 KDF call.
assets/src/02-01.zig	Migrates file hashing example to `std.Io.File` + `file.reader(io, ...)`.
assets/src/01-05.zig	Migrates directory walking to `std.Io.Dir` + `walker.next(io)` and uses `init.gpa`.
assets/src/01-04.zig	Migrates file existence check to `std.Io.Dir.cwd().access(io, ...)`.
assets/src/01-03.zig	Migrates directory walking/stat to `std.Io` APIs and uses `std.Io.Clock` for current time.
assets/src/01-02.zig	Migrates file operations to `std.Io.Dir`/`std.Io.File` APIs and updates `mmap` flags style.
assets/src/01-01.zig	Migrates file reading example to `std.Io.Dir`/`std.Io.File` reader APIs using `init.io`.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Refactor build process to use translate_c for C headers instead of @cImport. Add zigcli dependency and example integration for 13-01. Improve compatibility with Zig 0.16 by adding library wrappers and modular imports. This change streamlines external library handling and facilitates future upgrades.

Clarify that the main branch now tracks Zig 0.16.x instead of 0.15.x. Reflecting this change in both English and Chinese README avoids confusion about current Zig version support and communicates updated compatibility to users and contributors.

Copilot

Pull request overview

Copilot reviewed 89 out of 90 changed files in this pull request and generated 3 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-04-23T02:36:06Z

+    const io = b.graph.io;
+    const src_dir = try std.Io.Dir.cwd().openDir(io, b.path("assets/src").getPath(b), .{ .iterate = true });

    const target = b.standardTargetOptions(.{});
    var it = src_dir.iterate();


src_dir is opened but never closed. In Zig 0.16 std.Io.Dir holds OS resources; please add a defer src_dir.close(io) (and make it var if needed) to avoid leaking a directory handle during zig build (especially noticeable on Windows).

Copilot · 2026-04-23T02:36:06Z

+                const name = std.mem.trimEnd(u8, entry.name, ".zig");
                const exe = b.addExecutable(.{


std.mem.trimEnd(u8, entry.name, ".zig") trims any of the characters in the cutset, not the ".zig" suffix. This can produce wrong example names (e.g. a filename ending with 'g' before the extension would lose that too). Use a suffix-aware approach (e.g. std.mem.trimSuffix / std.fs.path.stem) to reliably strip the extension.

Copilot · 2026-04-23T02:36:06Z

+    callOnce();
 }

 pub fn main() !void {


This example still defines pub fn main() !void while the rest of the cookbook is moving to pub fn main(init: std.process.Init) !void (e.g. assets/src/01-01.zig:4, assets/src/05-01.zig:5, assets/src/10-03.zig:6). For consistency with the new entrypoint pattern (and to keep I/O/allocators coming from Init), update this main signature too.

Suggested change

pub fn main() !void {

pub fn main(init: std.process.Init) !void {

_ = init;

Ensure src_dir is closed after use to prevent resource leaks. This change improves code safety and reliability during build process by properly managing directory handles.

Copilot

Pull request overview

Copilot reviewed 93 out of 94 changed files in this pull request and generated 5 comments.

Comments suppressed due to low confidence (1)

build.zig:29

All examples and addTranslateC invocations hardcode .optimize = .Debug, which ignores the user's -Doptimize= setting and makes it impossible to build/run the cookbook in release modes. Consider plumbing const optimize = b.standardOptimizeOption(.{}); through and using it for modules/libraries/translate-c steps.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-04-23T03:04:50Z

 const std = @import("std");
 const print = std.debug.print;
-const c = @cImport({
-    @cInclude("libpq-fe.h");
-});
+const c = @import("c");



PR description states that main entry points are refactored to accept std.process.Init, but this example still has pub fn main() !void further down in the file. If the intent is to standardize on the new 0.16 I/O setup (init.gpa/init.io), update this example's main signature too; otherwise the PR description should be adjusted.

Copilot · 2026-04-23T03:04:50Z

 const std = @import("std");
 const print = std.debug.print;
-const c = @cImport({
-    @cInclude("regex.h");
-    // This is our static library.
-    @cInclude("regex_slim.h");
-});
+const c = @import("c");



PR description states that main entry points are refactored to accept std.process.Init, but this example still has pub fn main() !void further down in the file. If the intent is to standardize on the new 0.16 I/O setup (init.gpa/init.io), update this example's main signature too; otherwise the PR description should be adjusted.

Copilot · 2026-04-23T03:04:51Z

+    const io = b.graph.io;
+    var src_dir = try std.Io.Dir.cwd().openDir(io, b.path("assets/src").getPath(b), .{ .iterate = true });


src_dir is opened but never closed. Add a defer src_dir.close(io); after openDir to avoid leaking a directory handle during the build (especially important if more directories get opened in the future).

Copilot · 2026-04-23T03:04:51Z

    .dependencies = .{
        .zigcli = .{
-            .url = "https://github.com/jiacai2050/zigcli/archive/edd29d2fbcf977cae40990e091bfb821712afb48.zip",
-            .hash = "zigcli-0.3.1-ORC7jLbOAgCvnBYGJqGslo1s-ujF9cMQ3ux05i1OpBMp",
+            .url = "git+https://github.com/jiacai2050/zigcli.git?ref=v0.6.0#621920da8f2235c6b9c15addf353e7747e1e899c",
+            .hash = "zigcli-0.6.0-ORC7jHD5BQBipVTQYdQwlRQt-d7uVU_Jlz_HUyDouqpa",
        },


PR description says the zigcli dependency is removed, but build.zig.zon still declares a zigcli dependency (and the examples/build logic still reference it). Either update the PR description, or remove the dependency and adjust the argparse example accordingly.

Copilot · 2026-04-23T03:04:51Z

 const std = @import("std");
-const c = @cImport({
-    @cInclude("sqlite3.h");
-});
+const c = @import("c");
 const print = std.debug.print;


PR description states that main entry points are refactored to accept std.process.Init, but this example still has pub fn main() !void further down in the file. If the intent is to standardize on the new 0.16 I/O setup (init.gpa/init.io), update this example's main signature too; otherwise the PR description should be adjusted.

refactor(zig-cookbook): adopt Zig 0.16 I/O system

1a81153

Refactor main entry points to accept std.process.Init. Migrate file I/O, networking, and crypto operations to use the new std.Io interface. Removes zigcli dependency and bumps minimum Zig version to 0.16.0.

jiacai2050 requested a review from xihale as a code owner April 17, 2026 09:13

Copilot AI review requested due to automatic review settings April 17, 2026 09:13

Copilot started reviewing on behalf of jiacai2050 April 17, 2026 09:13 View session

gemini-code-assist Bot reviewed Apr 17, 2026

View reviewed changes

Comment thread assets/src/08-02.zig

Copilot AI reviewed Apr 17, 2026

View reviewed changes

Comment thread build.zig.zon

Comment thread build.zig Outdated

jiacai2050 added 3 commits April 22, 2026 21:13

update docs up to date

f0bc2a4

jiacai2050 requested a review from Copilot April 23, 2026 02:31

Copilot started reviewing on behalf of jiacai2050 April 23, 2026 02:31 View session

Copilot AI reviewed Apr 23, 2026

View reviewed changes

jiacai2050 added 2 commits April 23, 2026 10:57

more doc fix

174b0f1

add GEMINI.md

64c8c47

Copilot AI review requested due to automatic review settings April 23, 2026 02:59

Copilot started reviewing on behalf of jiacai2050 April 23, 2026 03:00 View session

refactor(build): Close src_dir properly in addExample

eb4f7ee

Ensure src_dir is closed after use to prevent resource leaks. This change improves code safety and reliability during build process by properly managing directory handles.

Copilot AI reviewed Apr 23, 2026

View reviewed changes

jiacai2050 merged commit 3676a3d into main Apr 23, 2026
4 checks passed

jiacai2050 deleted the 0.16 branch April 23, 2026 03:05

		const name = std.mem.trimEnd(u8, entry.name, ".zig");
		const exe = b.addExecutable(.{

	pub fn main() !void {
	pub fn main(init: std.process.Init) !void {
	_ = init;

		const io = b.graph.io;
		var src_dir = try std.Io.Dir.cwd().openDir(io, b.path("assets/src").getPath(b), .{ .iterate = true });

Conversation

jiacai2050 commented Apr 17, 2026

Uh oh!

gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

Code Review

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI Apr 23, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 23, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 23, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI Apr 23, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 23, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 23, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 23, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 23, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants