[Copilot]

The failing Actions job was in the Playwright signup flow. CI was exercising E2E against next dev, which introduced cold-start/HMR instability and broke the expected signup → verify path.

• CI runtime

  • Switch the workflow’s E2E phase to run against the built app via npm run start instead of next dev.
  • Keep the existing build step as the source of truth for the server E2E runs against.

• Signup verification behavior in CI

  • Extract localVerificationEnabled() for the local-mail verification path.
  • Allow local verification tokens during CI production-mode runs only when:
    • ECHO_LOCAL_MAIL=true
    • storage is non-AWS
    • CI=true
  • Reuse that gate both for the email-delivery guard and the signup response payload.

• Regression coverage

  • Add isolated integration coverage for the signup API path under CI production-mode settings.
  • Move the child-process route check into a dedicated fixture for readability and maintenance.

• Test DB bootstrap

  • Replace duplicated ad hoc schema setup with a shared helper that boots auth test databases through the repo’s existing SQLite prep + Prisma migration path.

export function localVerificationEnabled() {
  return (
    process.env.ECHO_LOCAL_MAIL === 'true' &&
    process.env.ECHO_STORAGE !== 'aws' &&
    (process.env.NODE_ENV !== 'production' || process.env.CI === 'true')
  );
}

[vercel]

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

Project	Deployment	Actions	Updated
echo-knowledge-decay	Ready	Preview	Sep 20, 2026 6:36pm UTC

[coderabbitai]

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: bd39fa95-1e8f-4eb6-a129-623713562a56

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[Copilot]

Copilot review overview

🔵 Needs a closer look

The route-type reference can break clean CI typechecking, and signup coverage needs boundary cases and deterministic NEXTAUTH_URL.

Review effort: Lite
Findings: None

What changed in this PR

Stabilizes CI E2E by running the built production server and adds controlled CI signup verification coverage.

Changes:

• Runs E2E against npm run start.
• Centralizes local verification gating.
• Adds signup regression coverage and shared Prisma migration setup.

File	Summary
echo/​tests/​integration/​fixtures/​apply-prisma-migrations.ts	Shared SQLite migration setup.
echo/​tests/​integration/​fixtures/​account-signup-ci-check.ts	Isolated signup route fixture.
echo/​tests/​integration/​accounts.test.ts	Uses shared database initialization.
echo/​tests/​integration/​account-signup-ci.test.ts	Tests CI production-mode signup.
echo/​next-env.d.ts	References production route types.
echo/​lib/​auth/​accounts.ts	Adds local verification gating.
echo/​echo.db	SQLite database artifact.
echo/​app/​api/​account/​[action]/​route.ts	Applies verification gating to signup.
.github/​workflows/​ci.yml	Runs E2E against the production server.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.