Skip to content

EvoCoder – Audit: No legacy pages/ found; proceed to clean residual references #1538

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
prxt6529 merged 1 commit into from
Oct 14, 2025
Merged

EvoCoder – Audit: No legacy pages/ found; proceed to clean residual references #1538

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions AGENTS.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -67,9 +67,9 @@ Use the `/codex/` directory as the shared source of truth for planning and ticke

## Next.js Directory Structure

The project supports both the classic `pages/` folder and Next.js `app/` router.
Legacy routes remain under `pages/`, while all **new** pages must be created
inside the `app/` directory.
All production routes now live under the Next.js `app/` router.
The legacy `pages/` directory has been fully migrated, so add any new routes
under `app/`.

Routes in `app/` should export a `generateMetadata` function using the helper
`getAppMetadata`:
Expand Down
6 changes: 3 additions & 3 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -61,9 +61,9 @@ pm2 start npm --name=6529seize -- run start

## Directory Structure

This project uses both the legacy `pages/` directory and the new `app/` router
from Next.js. Existing pages remain under `pages/`, while **all new pages should
be created in `app/`**.
All application routes now live under Next.js’s `app/` router.
The legacy `pages/` directory has been fully migrated, so create any new routes
inside `app/`.

Pages inside `app/` must define a `generateMetadata` function that returns the
result of `getAppMetadata`:
Expand Down
2 changes: 1 addition & 1 deletion jest.config.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -63,7 +63,7 @@ const config = {
collectCoverage: true,
collectCoverageFrom: [
// Target actual source directories like app, components, contexts, etc.
"{app,components,contexts,entities,helpers,hooks,lib,pages,services,store,utils,wagmiConfig}/**/*.{ts,tsx}",
"{app,components,contexts,entities,helpers,hooks,lib,services,store,utils,wagmiConfig}/**/*.{ts,tsx}",
// Exclude all TypeScript definition files from coverage
"!**/*.d.ts",
// Exclude node_modules
Expand Down
181 changes: 0 additions & 181 deletions scripts/migrate-page.cjs
View file
Open in desktop

This file was deleted.

1 change: 0 additions & 1 deletion tailwind.config.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@ module.exports = {
darkMode: "class",
content: [
"./app/**/*.{js,ts,jsx,tsx}",
"./pages/**/*.{js,ts,jsx,tsx}",
"./components/**/*.{js,ts,jsx,tsx}",

// Or if using `src` directory:
Expand Down
2 changes: 0 additions & 2 deletions tsconfig.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,8 +28,6 @@
"next-env.d.ts",
"**/*.ts",
"**/*.tsx",
"pages/_document.tsx",
"pages/**/*.jsx",
".next/types/**/*.ts"
],
"exclude": ["node_modules", "tests", "e2e"]
Expand Down