Skip to content

chore(cleanup): tree hygiene + verification-token typo rename#36

Open
abdout wants to merge 1 commit into
mainfrom
chore/tree-hygiene
Open

chore(cleanup): tree hygiene + verification-token typo rename#36
abdout wants to merge 1 commit into
mainfrom
chore/tree-hygiene

Conversation

@abdout
Copy link
Copy Markdown
Contributor

@abdout abdout commented Apr 25, 2026

Summary

  • Ignore `.idea/` and `.playwright-mcp/`; `git rm --cached` 5 tracked IDE-config files.
  • Delete `fix-dev-server.bat` and `start-dev.ps1` — Windows-only crutches that mask a Turbopack watcher issue instead of fixing it.
  • Rename typo `verificiation-token.ts` → `verification-token.ts` (atomic with caller updates).

Skipped

The audit suggested deduping two `auth.ts` files, but investigation showed:

  • `src/auth.ts` — NextAuth instance (`handlers`, `auth`, `signIn`, `signOut`)
  • `src/components/auth/auth.ts` — slim wrapper exposing `currentUser` and `currentRole` helpers built on the NextAuth instance

Different roles, not a duplicate. Both retained.

Changes

  • `.gitignore` — `.idea/` and `.playwright-mcp/`
  • Removed: `.idea/*` (untracked), `fix-dev-server.bat`, `start-dev.ps1`
  • Renamed: `src/components/auth/verification/verificiation-token.ts` → `verification-token.ts`
  • Updated: `src/components/auth/tokens.ts`, `src/components/auth/verification/action.ts`

Test plan

  • `grep -r verificiation src/` returns nothing
  • `pnpm tsc --noEmit` clean (re-tested after deps sync)
  • Auth verification flow (new-verification page) still resolves the token correctly

Closes #35

🤖 Generated with Claude Code

- .gitignore: ignore .idea/ and .playwright-mcp/
- git rm --cached .idea/* (5 IDE config files)
- delete fix-dev-server.bat and start-dev.ps1 (Windows-only crutches
  for a Turbopack/watcher issue; should fix root cause not workaround)
- rename verificiation-token.ts → verification-token.ts and update
  two importers (auth/tokens.ts, auth/verification/action.ts)

Skipped the audit's "two auth.ts files" dedup — investigation showed
they serve different roles (root NextAuth instance vs slim helper
wrapper) so kept both.

Closes #35

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 25, 2026

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

Project Deployment Actions Updated (UTC)
marketing Ready Ready Preview, Comment Apr 25, 2026 2:04pm

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.

chore(cleanup): tree hygiene + verification-token typo rename

1 participant