PR #66293: [codex] Align openclaw with shared handoff contract

• Repository: openclaw/openclaw
• Author: BananaAccurate
• State: open | merged: False
• Link: https://github.com/openclaw/openclaw/pull/66293

Description (problem / solution / changelog)

Intent / Scope

<!-- prh:single-intent -->

• Single intent: align openclaw with the shared Codex handoff contract as a consumer-only compatibility overlay.

<!-- prh:out-of-scope -->

• Out of scope: any runtime enforcement engine, unrelated dependency work in openclaw/openclaw#60168, JobApp or Middleman work, and any repo-local fork of the shared contract.

Files / Subsystems Expected to Change

<!-- prh:expected-paths -->

• Expected paths: AGENTS.md, README.md, package.json, test/qc-alignment.test.mjs, and test/codex-handoff-contract.test.mjs.

<!-- prh:hotspot-scope-note -->

• Hotspot scope note: this PR stays narrow to the repo-local consumer overlay and QC compatibility checks; it does not add prompt rendering or a second enforcement stack.

Ownership

<!-- prh:primary-owner -->

• Primary owner: @BananaAccurate

<!-- prh:review-routing -->

• Review routing: openclaw maintainers reviewing the consumer/QC overlay only. Review the canonical shared contract first in Intrepid-Analytica/Shared-Repo-Resources#8 and AI-Staff runtime enforcement context in Intrepid-Analytica/AI-Staff#42.

Stacked PR

<!-- prh:is-stacked -->

• Is this stacked?: no

<!-- prh:prerequisite-pr -->

• Prerequisite PR: Intrepid-Analytica/Shared-Repo-Resources#8 is review context only; this PR targets main directly.

<!-- prh:stack-order-note -->

• Stack order note: review after #8 so the consumer overlay is evaluated against the settled shared contract.

Docs / Workflow / Policy Files

<!-- prh:docs-included -->

• Intentionally included?: yes

<!-- prh:docs-why -->

• Why: this consumer-only package intentionally updates AGENTS/README guidance, package test wiring, and QC alignment coverage to stay synced with the shared handoff contract.

Supersession / Replacement

<!-- prh:supersedes -->

• Supersedes / replaces: none

<!-- prh:review-context -->

• Review context to preserve: none

Validation

<!-- prh:local-validation -->

• Local validation performed: node test/qc-alignment.test.mjs; node test/codex-handoff-contract.test.mjs

Operational Closeout

<!-- prh:operational-closeout -->

• Requires operational closeout after merge?: no

<!-- prh:operational-evidence -->

• Post-merge evidence / verification path: none

<!-- prh:rollback-plan -->

• Rollback / recovery path: revert this isolated consumer/QC overlay commit if the shared contract changes or the upstream landing needs to be backed out.

Review Topology

• This is the real upstream-capable consumer/QC PR for openclaw/openclaw.
• No runtime enforcement engine is added here.
• Ignore openclaw/openclaw#60168; it is an unrelated Dependabot dependency PR and not part of this rollout.

Changed files

• AGENTS.md (modified, +12/-0)
• README.md (modified, +9/-0)
• package.json (modified, +2/-0)
• test/codex-handoff-contract.test.mjs (added, +48/-0)
• test/qc-alignment.test.mjs (added, +53/-0)

---

PR #70669: [codex] Fix QA-lab abort catalog lifecycle test

• Repository: openclaw/openclaw
• Author: BananaAccurate
• State: open | merged: False
• Link: https://github.com/openclaw/openclaw/pull/70669

Description (problem / solution / changelog)

Summary

Fixes #70659.

This is a separate unblock lane for the current-main QA-lab extension failure that was blocking unrelated PR checks. It is intentionally separate from #66293 and does not alter that five-file handoff-contract overlay.

Root cause

The QA-lab abort-catalog test treated marker file existence as equivalent to complete marker content. Under CI, the test could observe the marker immediately after writeFileSync created/truncated it but before the expected content was visible, producing an empty read. On Windows, SIGTERM cleanup handlers are also not a reliable process-termination oracle.

Fix

• Wait for complete marker content that proves OPENCLAW_CODEX_DISCOVERY_LIVE=0 was propagated.
• Record the spawned catalog process pid and verify that lab.stop() terminates it.
• Preserve the graceful SIGTERM cleanup assertion on non-Windows platforms.

Validation

• node scripts/run-vitest.mjs run --config test/vitest/vitest.extension-qa.config.ts extensions/qa-lab/src/lab-server.test.ts failed on baseline, then passed 3/3 after the fix.
• npm exec --yes [email protected] -- exec oxfmt --check extensions/qa-lab/src/lab-server.test.ts
• node scripts/run-oxlint.mjs --tsconfig tsconfig.oxlint.extensions.json extensions/qa-lab/src/lab-server.test.ts
• npm exec --yes [email protected] -- tsgo:extensions:test

Note: local Windows pnpm test:extensions:batch -- qa-lab still hits pre-existing Windows path/symlink expectations outside this patch; the Linux PR CI extension shard is the relevant aggregate validation for #70659.

Changed files

• extensions/qa-lab/src/lab-server.test.ts (modified, +93/-7)