Skip to content

test(e2e): fix playwright regressions pre-release#151

Merged
avrabe merged 3 commits intomainfrom
fix/playwright-pre-release
Apr 19, 2026
Merged

test(e2e): fix playwright regressions pre-release#151
avrabe merged 3 commits intomainfrom
fix/playwright-pre-release

Conversation

@avrabe
Copy link
Copy Markdown
Contributor

@avrabe avrabe commented Apr 19, 2026

Summary

  • Fix coverage-view strict-mode violation by scoping to first coverage table
  • Drop title-truthy assertion in API test — control-actions use name/action instead
  • Extend per-test timeouts on 5 tests that hit /graph (currently ~57s on dogfood dataset)
  • Make waitForHtmx timeout configurable via helper

Test plan

  • Full Playwright suite passes locally: 354/354
  • No production code changed — test-only fixes
  • CI green

Follow-up

The /graph perf is a real bug: the page renders all ~1800 artifacts with no node budget despite graph.spec.ts:17 claiming one exists. Worth a dedicated issue after this release.

🤖 Generated with Claude Code

avrabe added 2 commits April 19, 2026 13:04
- coverage-view: handle multi-table strict-mode violation by scoping to the
  first table (coverage rules table)
- api: drop title-truthy assertion since control-actions use name/action
  instead of title
- audit-regression/graph/navigation/print-and-errors/self-contained:
  extend per-test timeouts on routes hitting /graph, which dogfoods on the
  project's ~1800 artifacts and currently takes ~57s (tracked as a perf
  follow-up; not a release blocker)
- helpers: make waitForHtmx's timeout configurable

Full suite: 354/354 passing locally.

Trace: skip
Patches RUSTSEC-2026-0098 and RUSTSEC-2026-0099, which published on
2026-04-14 and started failing the Security Audit job on any PR.

Trace: skip
Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Performance Alert ⚠️

Possible performance regression was detected for benchmark 'Rivet Criterion Benchmarks'.
Benchmark result of this commit is worse than the previous benchmark result exceeding threshold 1.20.

Benchmark suite Current: e91c26f Previous: 912530c Ratio
traceability_matrix/1000 65491 ns/iter (± 202) 46027 ns/iter (± 110) 1.42

This comment was automatically generated by workflow using github-action-benchmark.

Rust 1.95 promoted clippy::collapsible-match to the default lint set,
which failed the Clippy CI job on the stable toolchain update. Merges
the outer guard into the match arm pattern.

Trace: skip
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 19, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@avrabe avrabe merged commit 913ce29 into main Apr 19, 2026
19 of 23 checks passed
@avrabe avrabe deleted the fix/playwright-pre-release branch April 19, 2026 13:01
avrabe added a commit that referenced this pull request Apr 21, 2026
Addresses three gaps found in the post-v0.4.0 dogfooding audit.

**v0.4.0 shipped-work artifacts** — `artifacts/v040-features.yaml` was
last touched 2026-04-12 and describes variant/PLE work (FEAT-106..114),
not the verification pyramid that actually shipped on 2026-04-19. New
file `artifacts/v040-verification.yaml` authors 4 design decisions
(DD-052 four-layer verification pyramid, DD-053 suffix-based
yaml-section matching, DD-054 non-blocking framing for formal CI
jobs, DD-055 cfg-gate platform syscalls), 8 features
(FEAT-115..122 covering Kani 27-harness expansion, differential YAML
tests, operation-sequence proptest, STPA-Sec suite, suffix-based UCA
extraction, nested control-action extraction, Zola export, Windows
support), and 1 requirement (REQ-060 cross-platform binaries).
Counts were verified against the actual codebase — 27 `#[kani::proof]`
attrs in proofs.rs, 6 differential tests, 16 STPA-Sec tests.

**Retroactive trailer map** — extended `AGENTS.md` with three more
legacy orphans (51f2054 #126, f958a7e, 75521b8 #44), a new v0.4.0
PR-level section for #150/#151/#152/#153, and an honest
"genuinely-unmappable" section calling out `ca97dd9f` (#95) whose
`SC-EMBED-*` trailers point to artifacts that were never authored.

**Verus Proofs → hard gate** — rules_verus PR #21 (merged as
5bc96f39) fixes the hub-repo's ambiguous `:all` alias by emitting
proper `toolchain()` wrappers per platform. Updates the git_override
pin from e2c1600a (Feb 2026, broken) to 5bc96f39 and removes
`continue-on-error: true` from the Verus job.

Implements: REQ-030, REQ-060
Refs: DD-052, DD-053, DD-054, DD-055, FEAT-115, FEAT-116, FEAT-117, FEAT-118, FEAT-119, FEAT-120, FEAT-121, FEAT-122
Verifies: REQ-030
avrabe added a commit that referenced this pull request Apr 21, 2026
Addresses three gaps found in the post-v0.4.0 dogfooding audit.

**v0.4.0 shipped-work artifacts** — `artifacts/v040-features.yaml` was
last touched 2026-04-12 and describes variant/PLE work (FEAT-106..114),
not the verification pyramid that actually shipped on 2026-04-19. New
file `artifacts/v040-verification.yaml` authors 4 design decisions
(DD-052 four-layer verification pyramid, DD-053 suffix-based
yaml-section matching, DD-054 non-blocking framing for formal CI
jobs, DD-055 cfg-gate platform syscalls), 8 features
(FEAT-115..122 covering Kani 27-harness expansion, differential YAML
tests, operation-sequence proptest, STPA-Sec suite, suffix-based UCA
extraction, nested control-action extraction, Zola export, Windows
support), and 1 requirement (REQ-060 cross-platform binaries).
Counts were verified against the actual codebase — 27 `#[kani::proof]`
attrs in proofs.rs, 6 differential tests, 16 STPA-Sec tests.

**Retroactive trailer map** — extended `AGENTS.md` with three more
legacy orphans (51f2054 #126, f958a7e, 75521b8 #44), a new v0.4.0
PR-level section for #150/#151/#152/#153, and an honest
"genuinely-unmappable" section calling out `ca97dd9f` (#95) whose
`SC-EMBED-*` trailers point to artifacts that were never authored.

**Verus Proofs → hard gate** — rules_verus PR #21 (merged as
5bc96f39) fixes the hub-repo's ambiguous `:all` alias by emitting
proper `toolchain()` wrappers per platform. Updates the git_override
pin from e2c1600a (Feb 2026, broken) to 5bc96f39 and removes
`continue-on-error: true` from the Verus job.

Implements: REQ-030, REQ-060
Refs: DD-052, DD-053, DD-054, DD-055, FEAT-115, FEAT-116, FEAT-117, FEAT-118, FEAT-119, FEAT-120, FEAT-121, FEAT-122
Verifies: REQ-030
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