Skip to content

Integrate atomic Task claim recovery into qa-tools - #37

Merged
YueZh127 merged 18 commits into
qa-toolsfrom
qa-tools-fkst-staging
Sep 8, 2026
Merged

YueZh127 merged 18 commits into
qa-toolsfrom
qa-tools-fkst-staging

Conversation

@YueZh127

@YueZh127 YueZh127 commented Sep 8, 2026

Copy link
Copy Markdown

Purpose

Human-controlled integration of the reviewed Issue #34 implementation from qa-tools-fkst-staging into qa-tools. The PR is Ready only after the staging package, follow-up CI fix, independent reviews, and all four integration checks completed. Auto-merge remains disabled.

Immutable chain

qa-tools base:                 748a8e71b0a8b5626dffdedded7778b67ec52382
reviewed PR #35 head:          e7bb6412af65d980b676dd693130402343ec7382
PR #35 staging merge:          5a29093247b4d29a16499fe0576ec9d84ee9c6ec
readiness fix commit:          3afcd3773fc174c66b8c2fff76e1cfcff2d7ff61
PR #39 staging merge / head:   f104b1664ce1f1048d9aa1dc7e5c18303645b271

Implementation PR: #35
CI-fix PR: #39
Tracking issue: #34

Included behavior

  • Linearizable non-testing Task claim generation and version CAS.
  • Exact, capacity-safe machine and profile claim projections.
  • Bounded, restart-safe claim recovery and index-bounded expiry maintenance.
  • Generation fencing across heartbeat, completion, expiry/requeue, cancellation, interactive close, and action polling/results.
  • Bounded authorized-user CAS retries with stable 409 classification and no pre-CAS pending-input/handoff side effects.
  • FKST-compatible lowercase test check.
  • Deterministic proxy integration readiness synchronized to successful worker claim, without changing production session semantics or retrying expected 409 responses.

No Talos Testing Tool/QARun, provider, deployment, or external authority contract is changed.

Verification and review

  • Local gate: build, typecheck, lint, worker bundle, fixture regeneration/no drift, human and JSON contract demos.
  • Full suite: 53 files / 370 tests passed; 93.75% statement coverage.
  • Standalone Mongo contract: 80 / 80 passed.
  • PR github-devloop implementation for #34 #35 CI run 34220614928 passed all implementation checks at e7bb6412.
  • PR test(sessions): synchronize worker readiness in proxy flow #39 CI run 34243955406 passed all build/test/image/demo steps at 3afcd377.
  • Final PR Integrate atomic Task claim recovery into qa-tools #37 CI run 34244237242 passed all build/test/image/demo steps at f104b166.
  • Carver, Franklin, and Lagrange approved focused implementation packages after fix/re-review cycles.
  • Locke approved implementation range 748a8e71...e7bb6412.
  • Herschel requested and then approved the readiness fix after same-reviewer re-review; final composed range review is recorded before merge.

Merge gate

  • Base must remain qa-tools; head must remain qa-tools-fkst-staging@f104b166.
  • Use a normal merge commit only.
  • Do not squash, rebase, force-push, enable auto-merge, or delete source branches.
  • Confirm all four checks and the final composed-range review before merge.

Follow-up boundary

Issues #29, #27, #23, and #24 remain ordered follow-ups. Pending-input and handoff payload durability after a successful Task CAS is tracked separately in #38. No deployment or real Browser canary is claimed by this PR.

@YueZh127
YueZh127 marked this pull request as ready for review September 8, 2026 14:42
…ss-fix

test(sessions): synchronize worker readiness in proxy flow
@chronoai-fkst

chronoai-fkst Bot commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

github-devloop-integration rollup observe sample

status=clean
reason=clean
head_sha=f104b1664ce1f1048d9aa1dc7e5c18303645b271
first_clean_observed_at_ms=1788880950210
sampled_at_ms=1788881842612

@YueZh127
YueZh127 merged commit f252687 into qa-tools Sep 8, 2026
4 checks passed
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