Skip to content
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

feat: react 19 ftw #7329

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

feat: react 19 ftw #7329

wants to merge 3 commits into from

Conversation

harish-sethuraman
Copy link
Collaborator

@harish-sethuraman harish-sethuraman commented Dec 6, 2024

  • upgraded react to 19 for site
  • upgraded next 15
  • upgraded mdx to 3
  • updated remark and rehype plugins

update: works in local properly

build fails due to

./node_modules/next/dist/server/lib/router-utils/filesystem.d.ts:65:72
Type error: '>' expected.

  63 |     dynamicRoutes: FilesystemDynamicRoute[];
  64 |     nextDataRoutes: Set<string>;
> 65 |     exportPathMapRoutes: undefined | ReturnType<typeof buildCustomRoute<Rewrite>>[];
     |                                                                        ^
  66 |     devVirtualFsItems: Set<string>;
  67 |     prerenderManifest: PrerenderManifest;
  68 |     middlewareMatcher: MiddlewareRouteMatch | undefined;
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Copy link

vercel bot commented Dec 6, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
19-react-dev ❌ Failed (Inspect) Dec 6, 2024 9:26am
react-dev ❌ Failed (Inspect) Dec 6, 2024 9:26am

@@ -60,13 +60,15 @@ export default async function compileMDX(
.join('\n');

// Turn the MDX we just read into some JS we can execute.
const {remarkPlugins} = require('../../plugins/markdownToHtml');
// const {remarkPlugins} = require('../../plugins/markdownToHtml');
Copy link
Collaborator Author

@harish-sethuraman harish-sethuraman Dec 6, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

check why we aren't able to use remark / rehype plugins after upgrading it.

getting

Error: Expected usable value but received an empty preset, which is probably a mistake: presets typically come with `plugins` and sometimes with `settings`, but this has neither

Comment on lines +39 to +45
// const cached = await store.get(hash);
// if (cached) {
// console.log(
// 'Reading compiled MDX for /' + path + ' from ./node_modules/.cache/'
// );
// return cached;
// }
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

for now removed caching re enable before we push it to main

type: type,
key: key,
ref: null,
props: props,
_owner: null,
_store: null,
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

https://github.com/facebook/react/blob/c56c6234328a29930487295afe61597db48f058c/packages/react/src/ReactChildren.js#L245

Im unsure what this is used for and also should I update react to soft check for _store?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants