Skip to content

test(zigux): run phase2 closure gap contract by default#23149

Closed
adybag14-cyber wants to merge 1 commit into
masterfrom
zigux-builder/lane22-gap-default-route-20260609t1249z
Closed

test(zigux): run phase2 closure gap contract by default#23149
adybag14-cyber wants to merge 1 commit into
masterfrom
zigux-builder/lane22-gap-default-route-20260609t1249z

Conversation

@adybag14-cyber

Copy link
Copy Markdown
Owner

Summary

  • harden the merged Lane 22 zigux/tests/phase2_closure_gap_packet_contract_build.zig wrapper so the plain default build route runs the Phase 2 closure gap-packet contract
  • keep the existing standalone phase2-closure-gap-packet-contract route and test alias unchanged
  • keep scope to exactly one Zig build-wrapper file

Why This Lane

  • Lane 22 owns the Phase 2 closure tranche
  • the merged gap-packet contract already guards the kconfig bridge repo-reality boundary, but its build wrapper predated the newer closure-tranche convention where the plain default route runs the same contract as the named route and test alias
  • fresh open PR searches found no active same-wrapper/default-route owner before publication
  • this avoids edits to documentation, Python validators/checkers, manifests, workflow YAML, Makefile routes, helper implementations, fixtures, archive payloads, or shared build roots

Validation

Focused validation used attached Zig 0.17.0-dev.87+9b177a7d2 in a compact current file root grounded by authenticated current-master reads of:

  • zigux/tests/phase2_closure_gap_packet_contract.zig
  • zigux/tests/phase2_closure_gap_packet_contract_build.zig

Passed:

  • zig fmt --check zigux/tests/phase2_closure_gap_packet_contract.zig zigux/tests/phase2_closure_gap_packet_contract_build.zig
  • direct zig test zigux/tests/phase2_closure_gap_packet_contract.zig: All 3 tests passed.
  • build-step listing exposed phase2-closure-gap-packet-contract, test, and the default install route
  • named route phase2-closure-gap-packet-contract: Build Summary: 3/3 steps succeeded; 3/3 tests passed
  • test route: Build Summary: 3/3 steps succeeded; 3/3 tests passed
  • plain default route: Build Summary: 3/3 steps succeeded; 3/3 tests passed
  • ASCII scan found no non-ASCII bytes in the contract or wrapper files

Publish Readback

  • session issue: Zigux session 2026-06-09 12:49 UTC - Lane 22 gap default route #23147
  • branch: zigux-builder/lane22-gap-default-route-20260609t1249z
  • base / merge base: e940571de07b64f5c7709c92309f906fdd5fc689
  • head before PR creation: 24cbe0169c3e5a9bd1810dd29b6b736ec1e103d0
  • authenticated compare against master:
    • changed files: exactly one modified Zig file
    • additions: 2
    • deletions: 0
    • ahead_by: 1
    • behind_by: 0
    • status: ahead
  • authenticated branch readback confirmed b.default_step.dependOn(&run_tests.step);
  • immediate combined-status readback returned no statuses yet

Boundary

CI/review is the first exact full live-tree execution against complete current repository content. This intentionally does not mutate the contract body, Markdown docs, Python validators/checkers, manifests, workflow YAML, Makefile routes, helper implementations, fixtures, archive payloads, or shared roots.

@coderabbitai

coderabbitai Bot commented Jun 9, 2026

Copy link
Copy Markdown

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: 4b2f2028-781b-4ad2-8b54-8c6394296e2b

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/lane22-gap-default-route-20260609t1249z

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