Delete orphaned Pages Router file src/pages/security/TransactionSecurity.tsx
Labels / Complexity: maintenance — Trivial
Summary
The app routes through the App Router (src/app/), but one Pages Router file survives: src/pages/security/TransactionSecurity.tsx is the only file under src/pages/ and nothing imports or links to it. The live feature lives in the App Router world: src/components/security/TransactionSecuritySettings.tsx is used by src/app/dashboard/page.tsx (line 177), and the store it uses (useTransactionSecurityStore) is consumed from src/components/TransactionConfirmation.tsx and the settings component. Keeping a dead route in the tree invites confusion (two files with nearly the same name, one of which does nothing) and can confuse static-analysis and coverage tooling.
Proposal
- Delete
src/pages/security/TransactionSecurity.tsx and the now-empty src/pages/ directory.
- Confirm no imports reference it (
grep -rn "TransactionSecurity" src/ | grep -v "TransactionSecuritySettings" should show only the live components).
- Verify the build and test suites.
Acceptance criteria
src/pages/ no longer exists.
npm run typecheck and npm test pass.
git grep "pages/security" returns nothing.
Getting started
Files in scope: src/pages/security/TransactionSecurity.tsx. Verify with npm run typecheck and npm test (scripts in package.json). Pattern-match: the live implementation is src/components/security/TransactionSecuritySettings.tsx.
Delete orphaned Pages Router file src/pages/security/TransactionSecurity.tsx
Labels / Complexity: maintenance — Trivial
Summary
The app routes through the App Router (
src/app/), but one Pages Router file survives:src/pages/security/TransactionSecurity.tsxis the only file undersrc/pages/and nothing imports or links to it. The live feature lives in the App Router world:src/components/security/TransactionSecuritySettings.tsxis used bysrc/app/dashboard/page.tsx(line 177), and the store it uses (useTransactionSecurityStore) is consumed fromsrc/components/TransactionConfirmation.tsxand the settings component. Keeping a dead route in the tree invites confusion (two files with nearly the same name, one of which does nothing) and can confuse static-analysis and coverage tooling.Proposal
src/pages/security/TransactionSecurity.tsxand the now-emptysrc/pages/directory.grep -rn "TransactionSecurity" src/ | grep -v "TransactionSecuritySettings"should show only the live components).Acceptance criteria
src/pages/no longer exists.npm run typecheckandnpm testpass.git grep "pages/security"returns nothing.Getting started
Files in scope:
src/pages/security/TransactionSecurity.tsx. Verify withnpm run typecheckandnpm test(scripts inpackage.json). Pattern-match: the live implementation issrc/components/security/TransactionSecuritySettings.tsx.