-
Notifications
You must be signed in to change notification settings - Fork 44
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.
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
Migrate to nextjs app router #695
Changes from 5 commits
ecf4ccb
6efe2ea
2dde882
5328d7d
512e7ed
801cda2
9de1046
077e45b
5131f07
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -5,6 +5,7 @@ coverage | |
|
||
# Editor | ||
.vscode | ||
.idea | ||
|
||
# System | ||
.DS_Store | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -7,7 +7,8 @@ | |
import util from 'node:util'; | ||
import ora from 'ora'; | ||
import shell from 'shelljs'; | ||
import customNextIndex from '../lib/ui/next/customNextIndex.js'; | ||
import customNextPage from '../lib/ui/next/customNextPage.js'; | ||
import customNextLayout from '../lib/ui/next/customNextLayout.js'; | ||
import customNuxtIndex from '../lib/ui/nuxt/customNuxtIndex.js'; | ||
import nuxtGradientBackground from '../lib/ui/nuxt/nuxtGradientBackground.js'; | ||
import customLayoutSvelte from '../lib/ui/svelte/customLayoutSvelte.js'; | ||
|
@@ -314,12 +315,13 @@ | |
// set the project name and default flags | ||
// https://nextjs.org/docs/api-reference/create-next-app#options | ||
let args = [ | ||
'[email protected].3', | ||
'[email protected].12', | ||
'ui', | ||
'--use-npm', | ||
'--src-dir', | ||
'--no-src-dir', | ||
'--ts', | ||
'--import-alias "@/*"', | ||
'--no-app', | ||
'--app', | ||
]; | ||
|
||
spawnSync('npx', args, { | ||
|
@@ -328,19 +330,8 @@ | |
}); | ||
|
||
shell.rm('-rf', path.join('ui', '.git')); // Remove NextJS' .git; we will init .git in our monorepo's root. | ||
// Read in the NextJS config file and add the middleware. | ||
|
||
let useTypescript = true; | ||
try { | ||
// Determine if generated project is a ts project by looking for a tsconfig file | ||
fs.readFileSync(path.join('ui', 'tsconfig.json')); | ||
} catch (err) { | ||
if (err.code !== 'ENOENT') { | ||
console.error(err); | ||
} | ||
useTypescript = false; | ||
} | ||
|
||
// Read in the NextJS config file and add the middleware. | ||
const nextConfig = fs.readFileSync( | ||
path.join('ui', 'next.config.mjs'), | ||
'utf8' | ||
|
@@ -354,8 +345,9 @@ | |
|
||
`; | ||
newNextConfig += nextConfig.replace( | ||
/^};(.*?)$/gm, // Search for the last '};' in the file. | ||
'};', | ||
` | ||
reactStrictMode: false, | ||
webpack(config, { isServer }) { | ||
if (!isServer) { | ||
config.resolve.alias = { | ||
|
@@ -390,36 +382,41 @@ | |
};` | ||
); | ||
|
||
// This prevents usEffect from running twice on initial mount. | ||
newNextConfig = newNextConfig.replace( | ||
'reactStrictMode: true', | ||
'reactStrictMode: false' | ||
); | ||
|
||
fs.writeFileSync(path.join('ui', 'next.config.mjs'), newNextConfig); | ||
|
||
const indexFileName = useTypescript ? 'index.tsx' : 'index.js'; | ||
const pageFileName = 'page.tsx'; | ||
|
||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. It looks like there are some legacy There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Good catch, cleared out the whole directory |
||
fs.writeFileSync( | ||
path.join('ui', 'app', pageFileName), | ||
customNextPage, | ||
'utf8' | ||
); | ||
|
||
const layoutFileName = 'layout.tsx'; | ||
|
||
fs.writeFileSync( | ||
path.join('ui', 'src/pages', indexFileName), | ||
customNextIndex, | ||
path.join('ui', 'app', layoutFileName), | ||
customNextLayout, | ||
'utf8' | ||
); | ||
|
||
// Adds landing page components directory and files to NextJS project. | ||
fs.copySync( | ||
path.join(__dirname, 'ui', 'next', 'components'), | ||
path.join('ui', 'src', 'components') | ||
path.join('ui', 'components') | ||
); | ||
|
||
// Adds landing page style directory and files to NextJS project. | ||
fs.copySync( | ||
path.join(__dirname, 'ui', 'next', 'styles'), | ||
path.join('ui', 'src', 'styles') | ||
path.join('ui', 'styles') | ||
); | ||
|
||
// Removes create-next-app assets | ||
fs.emptyDirSync(path.join('ui', 'public')); | ||
if (fs.existsSync(path.join('ui', 'app', 'favicon.ico'))) { | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Similar comment to above. |
||
shell.rm('-rf', path.join('ui', 'app', 'favicon.ico')); | ||
} | ||
|
||
// Adds landing page assets directory and files to NextJS project. | ||
fs.copySync( | ||
|
@@ -452,23 +449,26 @@ | |
"jsx": "preserve", | ||
"paths": { | ||
"@/*": ["./src/*"] | ||
} | ||
}, | ||
"plugins": [ | ||
{ | ||
"name": "next" | ||
} | ||
] | ||
}, | ||
"include": ["next-env.d.ts", "**/*.ts", "**/*.tsx"], | ||
"include": ["next-env.d.ts", "**/*.ts", "**/*.tsx", ".next/types/**/*.ts"], | ||
"exclude": ["node_modules"] | ||
} | ||
`; | ||
|
||
if (useTypescript) { | ||
fs.writeFileSync(path.join('ui', 'tsconfig.json'), tsconfig); | ||
fs.writeFileSync(path.join('ui', 'tsconfig.json'), tsconfig); | ||
|
||
// Add a script to the package.json | ||
let x = fs.readJsonSync(path.join('ui', 'package.json')); | ||
x.scripts['ts-watch'] = 'tsc --noEmit --incremental --watch'; | ||
x.scripts['build'] = 'next build --no-lint'; | ||
x.type = 'module'; | ||
fs.writeJSONSync(path.join('ui', 'package.json'), x, { spaces: 2 }); | ||
} | ||
// Add a script to the package.json | ||
let x = fs.readJsonSync(path.join('ui', 'package.json')); | ||
x.scripts['ts-watch'] = 'tsc --noEmit --incremental --watch'; | ||
x.scripts['build'] = 'next build --no-lint'; | ||
x.type = 'module'; | ||
fs.writeJSONSync(path.join('ui', 'package.json'), x, { spaces: 2 }); | ||
|
||
if (useGHPages) { | ||
const isWindows = process.platform === 'win32'; | ||
|
@@ -510,13 +510,6 @@ | |
return config;` | ||
); | ||
|
||
// update page extensions | ||
newNextConfig = newNextConfig.replace( | ||
'reactStrictMode: false,', | ||
`reactStrictMode: false, | ||
pageExtensions: ['page.tsx', 'page.ts', 'page.jsx', 'page.js'],` | ||
); | ||
|
||
fs.writeFileSync(path.join('ui', 'next.config.mjs'), newNextConfig); | ||
|
||
// Add some scripts to the package.json | ||
|
@@ -548,42 +541,23 @@ | |
); | ||
shell.cd('..'); | ||
|
||
const appFileName = useTypescript ? '_app.tsx' : '_app.js'; | ||
const appPagesFileName = useTypescript ? '_app.page.tsx' : '_app.page.js'; | ||
const indexFileName = useTypescript ? 'index.tsx' : 'index.js'; | ||
const indexPagesFileName = useTypescript | ||
? 'index.page.tsx' | ||
: 'index.page.js'; | ||
const reactCOIServiceWorkerFileName = useTypescript | ||
? 'reactCOIServiceWorker.tsx' | ||
: 'reactCOIServiceWorker.js'; | ||
shell.mv( | ||
path.join('ui', 'src/pages', appFileName), | ||
path.join('ui', 'src/pages', appPagesFileName) | ||
); | ||
shell.mv( | ||
path.join('ui', 'src/pages', indexFileName), | ||
path.join('ui', 'src/pages', indexPagesFileName) | ||
); | ||
const reactCOIServiceWorkerFileName = 'reactCOIServiceWorker.tsx'; | ||
|
||
let appFile = fs.readFileSync( | ||
path.join('ui', 'src', 'pages', appPagesFileName), | ||
let pageFile = fs.readFileSync( | ||
path.join('ui', 'app', pageFileName), | ||
'utf8' | ||
); | ||
|
||
appFile = appFile.replace( | ||
pageFile = pageFile.replace( | ||
'export default function', | ||
`import './reactCOIServiceWorker'; | ||
|
||
export default function` | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @essential-breads were you able to test if the deploy to GitHub Pages flow works correctly with the appRouter changes? It might be out of scope for this PR, but I will add an issue so we don't forget to verify that everything works. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Yep GH pages is fully tested! |
||
); | ||
fs.writeFileSync( | ||
path.join('ui', 'src', 'pages', appPagesFileName), | ||
appFile | ||
); | ||
fs.writeFileSync(path.join('ui', 'app', pageFileName), pageFile); | ||
|
||
fs.writeFileSync( | ||
path.join('ui', 'src', 'pages', reactCOIServiceWorkerFileName), | ||
path.join('ui', 'app', reactCOIServiceWorkerFileName), | ||
`export {}; | ||
|
||
function loadCOIServiceWorker() { | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -205,12 +205,13 @@ describe('project.js', () => { | |
expect(spawnSync).toHaveBeenCalledWith( | ||
'npx', | ||
[ | ||
'[email protected].3', | ||
'[email protected].12', | ||
'ui', | ||
'--use-npm', | ||
'--src-dir', | ||
'--no-src-dir', | ||
'--ts', | ||
'--import-alias "@/*"', | ||
'--no-app', | ||
'--app', | ||
], | ||
{ | ||
stdio: 'inherit', | ||
|
@@ -621,12 +622,13 @@ describe('project.js', () => { | |
expect(spawnSync).toHaveBeenCalledWith( | ||
'npx', | ||
[ | ||
'[email protected].3', | ||
'[email protected].12', | ||
'ui', | ||
'--use-npm', | ||
'--src-dir', | ||
'--no-src-dir', | ||
'--ts', | ||
'--import-alias "@/*"', | ||
'--no-app', | ||
'--app', | ||
], | ||
{ | ||
stdio: 'inherit', | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,18 @@ | ||
export default `import "../styles/globals.css"; | ||
|
||
export const metadata = { | ||
title: 'Mina zkApp UI', | ||
description: 'built with o1js', | ||
icons: { | ||
icon: '/assets/favicon.ico', | ||
}, | ||
}; | ||
|
||
export default function RootLayout({ children }: { children: React.ReactNode }) { | ||
return ( | ||
<html lang="en"> | ||
<body>{children}</body> | ||
</html> | ||
); | ||
} | ||
`; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we add this change to the changelog?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1