Skip to content

[skills-daily] guard app auth route smoke alignment#110

Merged
david30907d merged 1 commit into
mainfrom
docs/agent-skills-ongoing-20260714-route-smoke
Jul 14, 2026
Merged

[skills-daily] guard app auth route smoke alignment#110
david30907d merged 1 commit into
mainfrom
docs/agent-skills-ongoing-20260714-route-smoke

Conversation

@taiiiiiiiiiiii

Copy link
Copy Markdown
Contributor

Summary

  • Update app-playwright-ci-debugging to cover auth navigation drift between BottomTabBar and tests/e2e/smoke.spec.ts.
  • Add a small locked-tab behavior checklist so agents verify whether guests should stay on the current route with connect flow or navigate to the sign-in gate.
  • Add a merge guardrail: final PR-head GitHub Actions must pass; “new CI will validate” is not enough.

Why

Main CI run 29308567967 failed in lint-test at Run check-variant tasks, while security audit was skipped. The merged #108 implementation changed inaccessible tabs to call account.connect() and return, but the current smoke spec still asserts locked tabs navigate to /strategy, /activity, and /account with the sign-in gate. This is a project-specific app auth / route-smoke mismatch, not generic Playwright advice.

Scope

Docs only:

  • .agents/skills/app-playwright-ci-debugging/SKILL.md

No implementation, test, dependency, lockfile, or CI gate changes.

Validation

@vercel

vercel Bot commented Jul 14, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
zap-engine-frontend Ready Ready Preview, Comment Jul 14, 2026 6:05am
zap-engine-landing-page Ready Ready Preview, Comment Jul 14, 2026 6:05am

@david30907d
david30907d merged commit ee23e5b into main Jul 14, 2026
8 of 9 checks passed
@david30907d
david30907d deleted the docs/agent-skills-ongoing-20260714-route-smoke branch July 14, 2026 06:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants