Skip to content

Add server-initiated-burst-cadence scenario (#58)#66

Closed
balloob wants to merge 1 commit into
claude/conformance-drift-injectionfrom
claude/conformance-burst-cadence
Closed

Add server-initiated-burst-cadence scenario (#58)#66
balloob wants to merge 1 commit into
claude/conformance-drift-injectionfrom
claude/conformance-burst-cadence

Conversation

@balloob
Copy link
Copy Markdown
Collaborator

@balloob balloob commented May 27, 2026

Summary

Notes

  • Per the audit, only sendspin-cpp, sendspin-jvm, and sendspin-js implement burst emission inside the SDK; the rest leave cadence to the embedder. The capability stays off on every client until the per-SDK adapter is wired to honor it.
  • Stacked on top of PR Add server-initiated-drift-injection scenario (#57) #65 (drift-injection / capability mechanism).

Test plan

  • python -m compileall src scripts
  • python -m conformance.cli run --results-dir results --from aiosendspin --to aiosendspin --timeout-seconds 25 shows the new scenario row with the expected fail-fast reason

Adds the burst-cadence scenario from #58. The
scenario observes `client/time` cadence over a window and asserts
bursts of 8 messages ~10 s apart (per the audit's recommended cadence).

`client-time-burst-cadence` is declared on the aiosendspin server only.
No SDK in the matrix declares the capability yet, so every client case
fails fast. The audit identifies sendspin-cpp, sendspin-jvm, and
sendspin-js as candidates to flip the flag once their SDK-side burst
emission is wired into the conformance adapter — the rest leave cadence
to the embedder and would still need a code change.
@balloob
Copy link
Copy Markdown
Collaborator Author

balloob commented May 27, 2026

Closing per discussion in #68 — dropping scaffolding-only scenarios in favor of real end-to-end tests as SDK surface lands.

@balloob balloob closed this May 27, 2026
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