Skip to content

test(zigux): restack lane16 list_sort bench replay runner#14450

Merged
adybag14-cyber merged 2 commits into
masterfrom
zigux-builder/lane16-list-sort-bench-replay-runner-current-master-20260529t0036z
Jun 1, 2026
Merged

test(zigux): restack lane16 list_sort bench replay runner#14450
adybag14-cyber merged 2 commits into
masterfrom
zigux-builder/lane16-list-sort-bench-replay-runner-current-master-20260529t0036z

Conversation

@adybag14-cyber
Copy link
Copy Markdown
Owner

Summary

  • restack the Lane 16 list_sort bench replay onto current master
  • add zigux/tests/phase1_list_sort_bench_replay_build.zig so the replay has a dedicated direct zig build route
  • expose both a named phase1-list-sort-bench-replay step and a plain test alias without reopening zigux/tests/phase1_bench.zig, shared build wiring, or tools/lib/list_sort.zig

Why This Lane

  • Lane 16 owns Phase 1 hardening gates around bench-sensitive helper packets
  • older draft PR test(zigux): add lane16 list_sort bench replay #13027 carried the replay alone but is now substantially behind current master
  • this current-master restack keeps the write set additive and Zig-only while making the list_sort packet directly runnable like the newer Lane 16 replay runners

Validation

  • exact current-master compare before branch cut used base SHA bdc09769b16f1dc4815d86a3cce99330955cb177
  • exact current-master reread confirmed zigux/tests/phase1_list_sort_bench_replay.zig returned 404 Not Found before this restack
  • exact branch reread of old PR test(zigux): add lane16 list_sort bench replay #13027 confirmed the replay body and confirmed zigux/tests/phase1_list_sort_bench_replay_build.zig was absent there before the runner follow-up
  • focused scratch validation used the attached Zigux-master.zip scaffold plus attached Zig 0.17.0-dev.87+9b177a7d2 toolchain because direct live checkout is not reliable in this scheduled runtime
  • zig fmt --check zigux/tests/phase1_list_sort_bench_replay.zig zigux/tests/phase1_list_sort_bench_replay_build.zig tools/lib/list_sort.zig
  • cd zigux/tests && zig build --build-file phase1_list_sort_bench_replay_build.zig --list-steps
    • result: step list includes phase1-list-sort-bench-replay and test
  • cd zigux/tests && zig build phase1-list-sort-bench-replay --build-file phase1_list_sort_bench_replay_build.zig --summary all
    • result: Build Summary: 3/3 steps succeeded; 2/2 tests passed
    • result: phase1-list-sort-bench-replay success
  • authenticated compare after publication:
    • changed files: 2
    • result: ahead_by: 2
    • result: behind_by: 0
    • result: status: ahead

Scope

  • lane: Lane 16 — Phase 1 hardening gates
  • session issue: Zigux session 2026-05-29 00:33 UTC Lane 16 list_sort replay runner #14446
  • branch: zigux-builder/lane16-list-sort-bench-replay-runner-current-master-20260529t0036z
  • base commit: bdc09769b16f1dc4815d86a3cce99330955cb177
  • files:
    • zigux/tests/phase1_list_sort_bench_replay.zig
    • zigux/tests/phase1_list_sort_bench_replay_build.zig

Coordination

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 29, 2026

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: ad0c9051-70ad-4d66-967d-98ceb72dca30

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch zigux-builder/lane16-list-sort-bench-replay-runner-current-master-20260529t0036z

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant