Repository navigation
linux: work-655 - #985
linux: work-655#985
Conversation
Agent-Team: linux Worker: codex
PM note: how this PR will be handledWhat it is for: linux: work-655 Rules that apply: touches the SDK or plugin ABI: the shape fingerprint must be recorded; touches build scripts. Things to watch (checked by rule):
What happens next:
A living status comment below is kept current. Nothing here is a merge decision. |
Where this PR standsStage: Done: approved What the PM needs before approving (ticks as it is met; the full CI runs only at the end, after the reviews are clean):
Rules that apply: touches the SDK or plugin ABI: the shape fingerprint must be recorded; touches build scripts. Reviews at the current commit (
Updated 2026-10-28 19:44 UTC by the PM. This comment is edited in place. |
Part of fizzyedit/work#655
Goal
Part of fizzyedit/work#655: mechanical std deprecation sweep for plans#50 (L3).
What changed and why
Replace
ArrayListUnmanagedwithArrayListand the three in-scopestd.fmt.bufPrintcalls withstd.mem.print. The installed Zig 0.17 std declares the former list name as an alias and forwards the former print API directly to its replacement, preserving behavior. Across 33 files, 60 lines replaced, net zero; no formatting churn or generated/vendor changes.How it was verified
Formatting, lint (3/3 steps), native build (85/85), web compilation (45/45), SDK fingerprint (35/35), and test reachability pass. An exact substitution check against HEAD proves every source diff contains only the requested replacements; the exact-pattern scan leaves only the skipped files. Existing tests provide behavioral coverage; no new regression test applies to alias removal. Unit tests: 184/189 steps, 892/919 tests passed, 3 skipped, 24 sandbox socket failures. Integration: 97/100 steps, 904/923 tests passed, 19 sandbox socket failures. GATES.md names each failed test and records every gate result.
Self-review: Q1 implementation complete; full green gates require socket access. Q2 no redundant code or Zig-standard issues found. Q3 keep the direct std replacements. Q4 aligned with the mechanical Zig-standard sweep; history storage behavior and formats unchanged.
Not done / follow-ups
Intentionally skipped
src/editor/Editor.zig(PR #951),plugins/dashboard/src/View.zig(PR #967), andtests/integration.zig(PR #969); sweep them after those PRs merge. The Windows spike'sbufPrintZis a different API and outside this task. The harness must rerun socket-dependent tests outside the sandbox and handle closing work#48 and work#50 on merge. No commits, pushes, or PR creation performed.Changes (33 files, +60/-60)
Gates: 6 of 6 passed
scripts/check-fmt.sh(exit 0)scripts/zb build --summary all(exit 0)scripts/zb build test --summary all(exit 0)scripts/zb build test-integration --summary all(exit 0)scripts/zb build check-web --summary all(exit 0)scripts/zb build test-sdk-version --summary all(exit 0)