Skip to content

Commit

Permalink
feat(nextjs): update to Next.js 13.2.4
Browse files Browse the repository at this point in the history
  • Loading branch information
jaysoo committed Apr 5, 2023
1 parent 775f651 commit 41d8187
Show file tree
Hide file tree
Showing 9 changed files with 77 additions and 3 deletions.
7 changes: 7 additions & 0 deletions docs/generated/packages/react/generators/library.json
Original file line number Diff line number Diff line change
Expand Up @@ -185,6 +185,13 @@
"default": false,
"x-priority": "internal"
},
"includeDeepImports": {
"description": "Add a deep-import entry to tsconfig.",
"type": "boolean",
"hidden": true,
"default": false,
"x-priority": "internal"
},
"minimal": {
"description": "Create a React library with a minimal setup, no separate test files.",
"type": "boolean",
Expand Down
9 changes: 9 additions & 0 deletions packages/js/src/utils/typescript/ts-config.ts
Original file line number Diff line number Diff line change
Expand Up @@ -59,6 +59,9 @@ export function updateRootTsConfig(
importPath?: string;
projectRoot: string;
js?: boolean;
// For Next.js RSC users need to use deep-imports as a workaround for mixed RSC + 'use client' libraries.
// See: https://github.com/nrwl/nx/issues/15830
includeDeepImports?: boolean;
}
) {
if (!options.importPath) {
Expand All @@ -85,6 +88,12 @@ export function updateRootTsConfig(
),
];

if (options.includeDeepImports) {
c.paths[`${options.importPath}/*`] = [
joinPathFragments(options.projectRoot, './*'),
];
}

return json;
});
}
17 changes: 17 additions & 0 deletions packages/next/migrations.json
Original file line number Diff line number Diff line change
Expand Up @@ -301,6 +301,23 @@
"alwaysAddToPackageJson": false
}
}
},
"15.9.3": {
"version": "15.9.3-beta.0",
"packages": {
"next": {
"version": "13.2.4",
"alwaysAddToPackageJson": false
},
"eslint-config-next": {
"version": "13.2.4",
"alwaysAddToPackageJson": false
},
"sass": {
"version": "1.60.0",
"alwaysAddToPackageJson": false
}
}
}
}
}
7 changes: 7 additions & 0 deletions packages/next/plugins/with-nx.ts
Original file line number Diff line number Diff line change
Expand Up @@ -224,6 +224,13 @@ export function getNextConfig(
const userWebpack = nextConfig.webpack || ((x) => x);
const { nx, ...validNextConfig } = nextConfig;
return {
// Need to skip type checks for now when app layout is enabled until Next.js releases a fix.
// The PR is merged but unreleased.
// PR: https://github.com/vercel/next.js/pull/47534
typescript: {
ignoreBuildErrors: !!validNextConfig?.experimental?.appDir,
...(validNextConfig.typescript ?? {}),
},
eslint: {
ignoreDuringBuilds: true,
...(validNextConfig.eslint ?? {}),
Expand Down
23 changes: 23 additions & 0 deletions packages/next/src/generators/library/library.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -4,9 +4,11 @@ import { createTreeWithEmptyWorkspace } from '@nrwl/devkit/testing';
import { Linter } from '@nrwl/linter';
import libraryGenerator from './library';
import { Schema } from './schema';

// need to mock cypress otherwise it'll use the nx installed version from package.json
// which is v9 while we are testing for the new v10 version
jest.mock('@nrwl/cypress/src/utils/cypress-version');

describe('next library', () => {
let mockedInstalledCypressVersion: jest.Mock<
ReturnType<typeof installedCypressVersion>
Expand Down Expand Up @@ -133,4 +135,25 @@ describe('next library', () => {
.jsxImportSource
).toEqual('@emotion/react');
});

it('should add deep import paths in tsconfig', async () => {
const appTree = createTreeWithEmptyWorkspace();

await libraryGenerator(appTree, {
name: 'myLib',
linter: Linter.EsLint,
skipFormat: false,
skipTsConfig: false,
unitTestRunner: 'jest',
style: 'css',
component: true,
});

expect(
readJson(appTree, 'tsconfig.base.json').compilerOptions.paths
).toMatchObject({
'@proj/my-lib': ['my-lib/src/index.ts'],
'@proj/my-lib/*': ['my-lib/*'],
});
});
});
3 changes: 3 additions & 0 deletions packages/next/src/generators/library/library.ts
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,9 @@ export async function libraryGenerator(host: Tree, options: Schema) {
...options,
compiler: 'swc',
skipFormat: true,
// Always include deep import paths so users can mix server and client react components in the same library.
// See: https://github.com/nrwl/nx/issues/15830
includeDeepImports: true,
});
tasks.push(libTask);

Expand Down
6 changes: 3 additions & 3 deletions packages/next/src/utils/versions.ts
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
export const nxVersion = require('../../package.json').version;

export const nextVersion = '13.1.1';
export const eslintConfigNextVersion = '13.1.1';
export const sassVersion = '1.55.0';
export const nextVersion = '13.2.4';
export const eslintConfigNextVersion = '13.2.4';
export const sassVersion = '1.60.0';
export const lessLoader = '11.1.0';
export const stylusLoader = '7.1.0';
export const emotionServerVersion = '11.10.0';
Expand Down
1 change: 1 addition & 0 deletions packages/react/src/generators/library/schema.d.ts
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@ export interface Schema {
globalCss?: boolean;
importPath?: string;
inSourceTests?: boolean;
includeDeepImports?: boolean;
js?: boolean;
linter: Linter;
name: string;
Expand Down
7 changes: 7 additions & 0 deletions packages/react/src/generators/library/schema.json
Original file line number Diff line number Diff line change
Expand Up @@ -188,6 +188,13 @@
"default": false,
"x-priority": "internal"
},
"includeDeepImports": {
"description": "Add a deep-import entry to tsconfig.",
"type": "boolean",
"hidden": true,
"default": false,
"x-priority": "internal"
},
"minimal": {
"description": "Create a React library with a minimal setup, no separate test files.",
"type": "boolean",
Expand Down

0 comments on commit 41d8187

Please sign in to comment.