Skip to content

docs: attribute default-test waits to executed operations - #1926

Merged
schlessera merged 2 commits into
mainfrom
docs/1855-default-wait-evidence-e1849-f4e75e84
Oct 14, 2026
Merged

schlessera merged 2 commits into
mainfrom
docs/1855-default-wait-evidence-e1849-f4e75e84

Conversation

@schlessera

Copy link
Copy Markdown
Owner

Refs #1855

What changed

Adds source-attributed observations for 72 default-test runs across 71 files, including actual awaited counts and elapsed spans, full-file/per-case timings, resource usage, source hashes, namespace profiles and explicit measurement limits. Corrects the track-draft timestamp-separation role without rewriting the historical source inventory.

Why this shape

The observations distinguish native waits, logical clock advances, fixture latency, overlapping helpers and losing watchdogs. They retain all original waits and assertions and do not claim a repository-wide runtime census, a speedup, or a universal minimum observation window. The guide preserves the minimum justified window policy and identifies native boundaries that a replacement must preserve.

Proof

  • Frozen observation source: 04a659593379a0c8da9c049fa649124d6fd54f70; documentation base: f6f5c713b2c52b400a5d34fc1d04120d3ea54107.
  • Main observation set: 70 complete files plus five selected enrichment cases, 1009 tests / 5337 expectations passing. A separate imported-client diagnostic repeats 28 tests / 70 expectations; it is not an additional unique assertion count.
  • Each measurement used the existing test preload/doubles inside a verified loopback-only user/network namespace. Timings exclude namespace setup and benchmark-lock waiting. No browser or build ran. Shared-host load was uncontrolled.
  • Temporary synchronous counters preserved original await/assertion expressions byte-for-byte; source restored after every batch. One attempted nested-IIFE observer was rejected before execution; the successful receipt explicitly leaves that site uninstrumented.
  • bun run check:pr --base origin/main passed on 7f75a79fc5ab913c3c6b6919d26ff2602450eb75: lint/leakage, environment documentation and changeset checks. This documentation-only change adds no behavioral test or mutation requirement. Hosted selected proof remains pending.

Checklist

  • Focused local preflight/behavioral receipts and complete selected hosted proof passed; actual head/base/job checkout evidence is in Proof.
  • Acceptance criteria from the issue are all met, or the gap is named below.
  • The PR and completed/advanced issues have verified release milestones under docs/process/github.md.
  • A changeset is included, or this changes nothing a consumer can see.
  • No personal data anywhere in the diff — the leakage gate covers the whole tree.
  • The integration contract index or relevant component is updated in this same commit, or nothing in the machine surface moved.
  • Docs under docs/ reflect the change, or nothing documented moved.

Anything left open

#1855 remains open: imported callbacks, remaining default/native/experiment families, complete operational minimum-window justifications and final tier ownership are not covered by this sample. Existing native negative windows remain unchanged. #1918 owns imported scrape-retry clock coverage; #1922 owns the recovery deadline. Higher measurement cost alone does not exempt a family from required proof.

@schlessera schlessera added this to the 0.44.0 milestone Oct 14, 2026
@schlessera schlessera added type: test Coverage, harness or proof work with no behaviour change. area: ui-sdk Wire protocol, runtime schemas, the server and client SDK surfaces. area: ui-server Hono app factory: turn coordinator, auth, session catalog, routes, cron. area: ui-react React chat/files/voice components, stores, WebSocket transport. area: ci CI workflows, the gates they run, release scripts, packaging. labels Oct 14, 2026
@schlessera schlessera self-assigned this Oct 14, 2026
@schlessera
schlessera marked this pull request as ready for review October 14, 2026 18:57
@schlessera
schlessera merged commit 43c6b9a into main Oct 14, 2026
19 of 21 checks passed
@schlessera

Copy link
Copy Markdown
Owner Author

Historical wait-evidence PR #1926 merged as 43c6b9a96fe230504ae4adb6c33eb72283778ec7; #1855 remains open for the remaining audit, replacements and native-horizon rationale. PR and issue retain milestone0.44.0(58), read back around merge.

Automatic ready CI 38619609256, attempt1, passed both selected documentation jobs and current contract proof passed. Planner logs identify the three docs files and deliberately exclude typecheck, unit/runtime, browser/layout, captures and packaging; aggregate validates those skips. Both executed checkout logs identify aae7a5a764791e8dbd9b1125507d53944dd290e1, parents main 009e45714701844a632101a181be2d5e8d66afbb and head 7f75a79fc5ab913c3c6b6919d26ff2602450eb75, tree e63bf5de17af13314063d6a311969e34a979d713. Immutable API/local Git agree. The matching-head squash has that same parent/tree and is verified on main. Selected PR job cost:43seconds; automatic main proof 38619808365 is pending separately.

The data covers72runs across71files at frozen source04a, with source hashes, actual calls/spans, case names, missing timing nulls and explicit namespace profiles. Original1009cases/5337expectations and a separately repeated28/70 diagnostic remain distinguishable. Parent review independently checked all source hashes and observation expressions, including imported sources. Native/process/browser boundaries and overlapping spans are not collapsed into a total saving. New independent audit work continues on a separate tree; this merged branch receives no further pushes.

@schlessera

Copy link
Copy Markdown
Owner Author

Automatic main-push CI 38619808365, attempt1, passed both selected documentation jobs for #1926 squash 43c6b9a96fe230504ae4adb6c33eb72283778ec7. Both executed checkout logs identify that squash; the remaining proof domains are intentional planner exclusions for this docs-only change. Selected execution cost47job seconds,55event-to-completion seconds including queue. This verifies the documentation slice, not the remaining #1855 runtime inventory or minimum-window criteria.

The merged clean documentation worktree was confirmed unused and removed; branch and evidence remain. Independent audit work continues on its own frozen-source checkout. PR and advanced issue retain milestone0.44.0(58).

@schlessera

Copy link
Copy Markdown
Owner Author

Additional automatic website publication-proof occurrence after docs PR #1926: run 38619808068, attempt 1, artifact job 116045238202, fails the existing verifyRichDemo HTML-preview heading wait at website/scripts/rich-demo-check.ts:129. It waits for “Departure from Ogygia” in iframe[title="HTML preview"] and exceeds the unchanged 7000 ms locator bound. The later HTML image/PDF export checks were not reached; this is not #1331's no-download assertion.

Actual website checkout is 43c6b9a96fe230504ae4adb6c33eb72283778ec7, parent 009e45714701844a632101a181be2d5e8d66afbb, tree e63bf5de17af13314063d6a311969e34a979d713, confirmed by job log, immutable API and local Git. The rebuilt artifact separately selected product tag @schlessera/brain@0.43.0, source 5dc2595ca07042f2e2024c95bbc361f31637559f; those identities are explicit in the assembly and check logs. The docs squash changes only wait guidance/evidence. Website sources, workflows, lockfile and product files are unchanged from its parent.

Static verification passed 157 HTML pages and 501 artifact digests; share/publication/release-selection guards and all four initial demo smoke scenes passed before the heading failure. Diagnostic artifact 11921217856 is retained. This follow-up inspected the source and complete failed job log, not the artifact's runtime DOM or network trace; it does not independently re-establish the false-404 causal diagnosis already owned here.

The artifact job failed, and deploy was skipped. Execution cost was 368 job seconds; event-to-completion was 510 seconds including queue. No website publication, unchanged retry, timeout increase or source correction was attempted. Preserve this newer 0.43.0 occurrence under #1772 / its blocked PR #1776. The #1926 documentation CI proof passed separately and does not imply the website workflow passed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: ci CI workflows, the gates they run, release scripts, packaging. area: ui-react React chat/files/voice components, stores, WebSocket transport. area: ui-sdk Wire protocol, runtime schemas, the server and client SDK surfaces. area: ui-server Hono app factory: turn coordinator, auth, session catalog, routes, cron. type: test Coverage, harness or proof work with no behaviour change.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant