Skip to content

Windows CI 错误 #2

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
Loongphy merged 4 commits into from
Feb 10, 2026
Merged

Windows CI 错误 #2

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
102b1f2
fix(ci): support windows time and tty
cursoragent Feb 10, 2026
ae6f6b8
fix(windows): prefer localtime_s and tty width
cursoragent Feb 10, 2026
72c19d3
fix(windows): use _localtime64_s fallback
cursoragent Feb 10, 2026
2a9c181
fix(windows): bind _localtime64_s and guard cast
cursoragent Feb 10, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
42 changes: 36 additions & 6 deletions src/cli.zig
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
const std = @import("std");
const builtin = @import("builtin");
const registry = @import("registry.zig");
const io_util = @import("io_util.zig");
const timefmt = @import("timefmt.zig");
Expand All @@ -14,7 +15,7 @@ const ansi = struct {
};

fn colorEnabled() bool {
return std.posix.isatty(std.posix.STDOUT_FILENO);
return std.fs.File.stdout().isTty();
}

pub const OutputFormat = enum { table, json, csv, compact };
Expand Down Expand Up @@ -133,10 +134,16 @@ pub fn runCodexLogin(allocator: std.mem.Allocator) !void {
}

pub fn selectAccount(allocator: std.mem.Allocator, reg: *registry.Registry) !?[]const u8 {
return selectInteractive(allocator, reg) catch selectWithNumbers(reg);
return if (comptime builtin.os.tag == .windows)
selectWithNumbers(reg)
else
selectInteractive(allocator, reg) catch selectWithNumbers(reg);
}

pub fn selectAccountsToRemove(allocator: std.mem.Allocator, reg: *registry.Registry) !?[]usize {
if (comptime builtin.os.tag == .windows) {
return selectRemoveWithNumbers(allocator, reg);
}
return selectRemoveInteractive(allocator, reg) catch selectRemoveWithNumbers(allocator, reg);
}

Expand Down Expand Up @@ -726,19 +733,42 @@ const ResetParts = struct {
}
};

fn localtimeCompat(ts: i64, out_tm: *c.struct_tm) bool {
if (comptime builtin.os.tag == .windows) {
// Bind directly to the exported CRT symbol on Windows.
if (comptime @hasDecl(c, "_localtime64_s") and @hasDecl(c, "__time64_t")) {
var t64 = std.math.cast(c.__time64_t, ts) orelse return false;
return c._localtime64_s(out_tm, &t64) == 0;
}
return false;
}

var t = std.math.cast(c.time_t, ts) orelse return false;
if (comptime @hasDecl(c, "localtime_r")) {
return c.localtime_r(&t, out_tm) != null;
}

if (comptime @hasDecl(c, "localtime")) {
const tm_ptr = c.localtime(&t);
if (tm_ptr == null) return false;
out_tm.* = tm_ptr.*;
return true;
}

return false;
}

fn resetPartsAlloc(allocator: std.mem.Allocator, reset_at: i64, now: i64) !ResetParts {
var t: c.time_t = @intCast(reset_at);
var tm: c.struct_tm = undefined;
if (c.localtime_r(&t, &tm) == null) {
if (!localtimeCompat(reset_at, &tm)) {
return ResetParts{
.time = try std.fmt.allocPrint(allocator, "-", .{}),
.date = try std.fmt.allocPrint(allocator, "-", .{}),
.same_day = true,
};
}
var now_t: c.time_t = @intCast(now);
var now_tm: c.struct_tm = undefined;
if (c.localtime_r(&now_t, &now_tm) == null) {
if (!localtimeCompat(now, &now_tm)) {
return ResetParts{
.time = try std.fmt.allocPrint(allocator, "-", .{}),
.date = try std.fmt.allocPrint(allocator, "-", .{}),
Expand Down
72 changes: 54 additions & 18 deletions src/format.zig
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,12 +1,12 @@
const std = @import("std");
const builtin = @import("builtin");
const registry = @import("registry.zig");
const cli = @import("cli.zig");
const io_util = @import("io_util.zig");
const timefmt = @import("timefmt.zig");
const c = @cImport({
@cInclude("time.h");
});
const linux = std.os.linux;

const ansi = struct {
const reset = "\x1b[0m";
Expand All @@ -15,7 +15,7 @@ const ansi = struct {
};

fn colorEnabled() bool {
return std.posix.isatty(std.posix.STDOUT_FILENO);
return std.fs.File.stdout().isTty();
}

fn planDisplay(rec: *const registry.AccountRecord, missing: []const u8) []const u8 {
Expand Down Expand Up @@ -260,19 +260,42 @@ const ResetParts = struct {
}
};

fn localtimeCompat(ts: i64, out_tm: *c.struct_tm) bool {
if (comptime builtin.os.tag == .windows) {
// Bind directly to the exported CRT symbol on Windows.
if (comptime @hasDecl(c, "_localtime64_s") and @hasDecl(c, "__time64_t")) {
var t64 = std.math.cast(c.__time64_t, ts) orelse return false;
return c._localtime64_s(out_tm, &t64) == 0;
}
return false;
}

var t = std.math.cast(c.time_t, ts) orelse return false;
if (comptime @hasDecl(c, "localtime_r")) {
return c.localtime_r(&t, out_tm) != null;
}

if (comptime @hasDecl(c, "localtime")) {
const tm_ptr = c.localtime(&t);
if (tm_ptr == null) return false;
out_tm.* = tm_ptr.*;
return true;
}

return false;
}

fn resetPartsAlloc(reset_at: i64, now: i64) !ResetParts {
var t: c.time_t = @intCast(reset_at);
var tm: c.struct_tm = undefined;
if (c.localtime_r(&t, &tm) == null) {
if (!localtimeCompat(reset_at, &tm)) {
return ResetParts{
.time = try std.fmt.allocPrint(std.heap.page_allocator, "-", .{}),
.date = try std.fmt.allocPrint(std.heap.page_allocator, "-", .{}),
.same_day = true,
};
}
var now_t: c.time_t = @intCast(now);
var now_tm: c.struct_tm = undefined;
if (c.localtime_r(&now_t, &now_tm) == null) {
if (!localtimeCompat(now, &now_tm)) {
return ResetParts{
.time = try std.fmt.allocPrint(std.heap.page_allocator, "-", .{}),
.date = try std.fmt.allocPrint(std.heap.page_allocator, "-", .{}),
Expand Down Expand Up @@ -370,14 +393,12 @@ fn remainingPercent(used: f64) i64 {
}

fn formatResetTimeAlloc(ts: i64, now: i64) ![]u8 {
var t: c.time_t = @intCast(ts);
var tm: c.struct_tm = undefined;
if (c.localtime_r(&t, &tm) == null) {
if (!localtimeCompat(ts, &tm)) {
return try std.fmt.allocPrint(std.heap.page_allocator, "-", .{});
}
var now_t: c.time_t = @intCast(now);
var now_tm: c.struct_tm = undefined;
if (c.localtime_r(&now_t, &now_tm) == null) {
if (!localtimeCompat(now, &now_tm)) {
return try std.fmt.allocPrint(std.heap.page_allocator, "-", .{});
}

Expand Down Expand Up @@ -596,12 +617,28 @@ fn tableTotalWidth(widths: []const usize) usize {
}

fn terminalWidth() usize {
const fd = std.posix.STDOUT_FILENO;
if (!std.posix.isatty(fd)) return 0;
var wsz: std.posix.winsize = undefined;
const rc = linux.syscall3(.ioctl, @bitCast(@as(isize, fd)), linux.T.IOCGWINSZ, @intFromPtr(&wsz));
if (linux.E.init(rc) != .SUCCESS) return 0;
return @as(usize, wsz.col);
const stdout_file = std.fs.File.stdout();
if (!stdout_file.isTty()) return 0;

if (comptime builtin.os.tag == .windows) {
var info: std.os.windows.CONSOLE_SCREEN_BUFFER_INFO = undefined;
if (std.os.windows.kernel32.GetConsoleScreenBufferInfo(stdout_file.handle, &info) == std.os.windows.FALSE) {
return 0;
}
const width = @as(i32, info.srWindow.Right) - @as(i32, info.srWindow.Left) + 1;
if (width <= 0) return 0;
return @as(usize, @intCast(width));
} else {
var wsz: std.posix.winsize = .{
.row = 0,
.col = 0,
.xpixel = 0,
.ypixel = 0,
};
const rc = std.posix.system.ioctl(stdout_file.handle, std.posix.T.IOCGWINSZ, @intFromPtr(&wsz));
if (std.posix.errno(rc) != .SUCCESS) return 0;
return @as(usize, wsz.col);
}
}

fn truncateAlloc(value: []const u8, max_len: usize) ![]u8 {
Expand All @@ -613,9 +650,8 @@ fn truncateAlloc(value: []const u8, max_len: usize) ![]u8 {

fn formatTimestampAlloc(ts: i64) ![]u8 {
if (ts < 0) return try std.fmt.allocPrint(std.heap.page_allocator, "-", .{});
var t: c.time_t = @intCast(ts);
var tm: c.struct_tm = undefined;
if (c.localtime_r(&t, &tm) == null) {
if (!localtimeCompat(ts, &tm)) {
return try std.fmt.allocPrint(std.heap.page_allocator, "-", .{});
}

Expand Down