Skip to content

test(adapter-sessions): stabilize cleanup race coverage - #164

Merged
avksp merged 1 commit into
mainfrom
fix/2.9-external-job-cleanup
Aug 30, 2026
Merged

test(adapter-sessions): stabilize cleanup race coverage#164
avksp merged 1 commit into
mainfrom
fix/2.9-external-job-cleanup

Conversation

@avksp

@avksp avksp commented Aug 30, 2026

Copy link
Copy Markdown
Owner

Summary

  • cancel immediately after the persisted RUNNING state and retain strict first-cancel assertions
  • coordinate same-byte artifact replacement strictly between the two real artifact collections
  • remove timing-dependent daemon writes and module-global time mocking

Validation

  • focused changed tests: 50 repeats on Python 3.12 and 50 on Python 3.14
  • full suite: 2039 tests PASS on Python 3.14
  • Python quality evidence and validation: PASS
  • repository hygiene: PASS
  • neutrality tracked-release: zero findings
  • Grok 4.6 xhigh independent review: PASS after one remediation round
  • OpenCode GLM 5.3 strict read-only independent review: PASS

This patch changes tests only. It does not alter cleanup, cancellation, timeout, artifact, security, architecture, or quality policy behavior.

@avksp
avksp requested a review from BJuice1984 August 30, 2026 21:08
@avksp
avksp merged commit 0d84bac into main Aug 30, 2026
22 checks passed
@avksp
avksp deleted the fix/2.9-external-job-cleanup branch August 30, 2026 21:12
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