Skip to content

test(site): stabilize landing smoke contract - #80

Merged
StringKe merged 1 commit into
mainfrom
agent/fix-ui-smoke-selector
Aug 18, 2026
Merged

test(site): stabilize landing smoke contract#80
StringKe merged 1 commit into
mainfrom
agent/fix-ui-smoke-selector

Conversation

@StringKe

Copy link
Copy Markdown
Owner

Summary

  • add a dedicated data-smoke-architecture contract to the Nimbus landing page
  • make the three-worker smoke check query the stable contract instead of a presentation CSS class
  • prevent future visual refactors from silently invalidating the smoke selector

Root cause

The redesigned landing page replaced .architecture-panel with .architecture-list, while scripts/run-three-worker-browser-smoke.mjs still waited for the removed class. The rendered page was healthy, but the main CI smoke timed out on the stale selector.

Validation

  • PASS node --check scripts/run-three-worker-browser-smoke.mjs
  • PASS pnpm run test:web-routes (19 tests)
  • PASS pnpm run security:secret-scan
  • PASS pnpm --filter @xid-kit/site check
  • PASS pnpm --filter @xid-kit/site test (298 tests)
  • PASS pnpm --filter @xid-kit/site build (360 pages, route audit PASS)
  • PASS git diff --check
  • SKIP local browser run by explicit request; the existing main CI performs the authoritative three-worker browser smoke after merge

Signed-off-by: StringKE <stringke.me@gmail.com>
@StringKe
StringKe marked this pull request as ready for review August 18, 2026 06:02
@StringKe
StringKe merged commit 6fa2df2 into main Aug 18, 2026
11 checks passed
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