Skip to content

test(coordination): align claim contention recovery - #5444

Open
Duang777 wants to merge 1 commit into
loopx-project:mainfrom
Duang777:codex/fix-claim-contention-test-baseline
Open

Duang777 wants to merge 1 commit into
loopx-project:mainfrom
Duang777:codex/fix-claim-contention-test-baseline

Conversation

@Duang777

@Duang777 Duang777 commented Oct 2, 2026

Copy link
Copy Markdown
Collaborator

What changed

  • Update the receipt-only claim test to expect semantic owner rejection after a provider-revision conflict is retried.
  • Keep the existing no-receipt and winning-owner checks for the conflict path.
  • Keep the lost-response recovery assertions, including changed=false and durable receipt recovery.

Why

#5371 made canonical claims reread and revalidate authority after conclusive CAS contention. The older fixture still expected the first provider conflict to escape unchanged. It now fails deterministically because the retry observes agent-b as the current owner and returns claim_owner_mismatch. This surfaced in #5442 typescript-core (1/3).

This PR changes only the stale test contract. It does not modify the claim implementation or #5442.

Validation

  • Node 22.22.3 targeted test: 5/5 repeated runs passed.
  • Node 22.22.3 local_authority_runtime.test.ts: 35/35 passed.
  • npm run typecheck:control-plane: passed.
  • git diff --check: passed.

Signed-off commit: ef2d1ce64.

Signed-off-by: duanjialing.777 <duanjialing.777@bytedance.com>
@Duang777

Duang777 commented Oct 2, 2026

Copy link
Copy Markdown
Collaborator Author

Exact-head CI attribution for ef2d1ce649a33b1ac6c92acad890c119aee1a185:

Keep the fixes isolated: land/sync #5442 rather than adding its cleanup change to this PR. Remaining matrix jobs are still running. No merge action was taken.

This branch has not been deployed

No deployments
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