Skip to content

Stabilize Session protocol full-suite tests - #168

Merged
yyjeqhc merged 2 commits into
mainfrom
fix/session-protocol-full-suite-stability
Aug 26, 2026
Merged

Stabilize Session protocol full-suite tests#168
yyjeqhc merged 2 commits into
mainfrom
fix/session-protocol-full-suite-stability

Conversation

@yyjeqhc

@yyjeqhc yyjeqhc commented Aug 26, 2026

Copy link
Copy Markdown
Owner

Summary

  • run the large request-scoped ACK HTTP integration fixture on an explicit 8 MiB test thread stack without changing production request handling
  • align Session collaboration OAuth/shared-key test fixtures with the new session:collaborate authority split
  • keep startup workflow output schema and compact model guidance synchronized with piggyback Session message resolution
  • make the Windows LSP initialize-cleanup budget test assert the supervisor-owned restart outcome rather than a scheduler-sensitive child start marker; disable its unrelated background reaper

Validation

  • cargo fmt -- --check
  • focused startup brief tests: 14 passed
  • focused collaboration tests: 18 passed
  • focused request-scoped ACK HTTP test: 1 passed
  • focused Session OAuth scope test: 1 passed
  • full cargo test on the implementation commit: 3020 passed, 0 failed
  • follow-up LSP initialize-cleanup regression on the current test-only head: 20/20 focused runs passed
  • git diff --check

@yyjeqhc yyjeqhc added the run-ci Run the heavy pull-request CI jobs for an owner-authored PR label Aug 26, 2026
@yyjeqhc
yyjeqhc merged commit f5ab6c6 into main Aug 26, 2026
11 checks passed
@yyjeqhc
yyjeqhc deleted the fix/session-protocol-full-suite-stability branch August 26, 2026 02:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

run-ci Run the heavy pull-request CI jobs for an owner-authored PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant