Skip to content

test(gateway): cover trace id tieouts - #85

Open
XnLemon wants to merge 107 commits into
mainfrom
codex/phase2-trace-tieout
Open

test(gateway): cover trace id tieouts#85
XnLemon wants to merge 107 commits into
mainfrom
codex/phase2-trace-tieout

Conversation

@XnLemon

@XnLemon XnLemon commented Jul 11, 2026

Copy link
Copy Markdown
Owner

Objective

Close the Phase2 trace tie-out coverage gap for gateway inbound handling.

Completed changes

  • Extends gateway usage-record test coverage so the same request/trace ID is asserted across result, outbound reply, usage record, audit record, message events, and queued outbox message.
  • Switches the test to the outbox-backed outbound store so the persisted reply path is part of the tie-out.

Validation

  • gofmt -w platform/gateway/service_test.go
  • go test ./platform/gateway
  • go vet ./platform/gateway
  • git diff --check
  • open-code-review subagent review: 0 High / 0 Medium / 0 Low. OCR CLI was installed, but LLM test was not configured, so strict manual review was used.

Risks / limitations

  • Test-only change; no production behavior change expected.

Follow-up

  • Continue Phase2 closure with the next independently reviewable acceptance gap.

XnLemon added 30 commits July 8, 2026 11:47
@XnLemon
XnLemon force-pushed the codex/phase2-tool-deny-audit branch from ffe5fe3 to 75002b5 Compare July 11, 2026 13:38
@XnLemon
XnLemon force-pushed the codex/phase2-trace-tieout branch from a47d369 to 83e8ed4 Compare July 11, 2026 13:38
@XnLemon
XnLemon changed the base branch from codex/phase2-tool-deny-audit to main July 13, 2026 02:23
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.

2 participants