Migrate documentation site to Rspress - #621
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 4bd2850885
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| engines: {node: '>=6.9.0'} | ||
| peerDependencies: | ||
| '@babel/core': ^7.0.0 | ||
| --- |
There was a problem hiding this comment.
Regenerate the lockfile as one YAML document
The second --- starts another complete lockfile document, so dependency installation cannot parse this file; running pnpm install --frozen-lockfile reports ERR_PNPM_BROKEN_LOCKFILE: expected a single document in the stream, but found more. This blocks both fresh local setup and the workflow's Install dependencies step, so regenerate the lockfile rather than concatenating the package-manager bootstrap data with the project lockfile.
AGENTS.md reference: AGENTS.md:L18-L18
Useful? React with 👍 / 👎.
|
|
||
| ``` | ||
| $ pnpm start | ||
| pnpm start |
There was a problem hiding this comment.
Replace the nonexistent start command
After this migration, package.json defines dev but no start script, so following the Local Development instructions fails with ERR_PNPM_NO_SCRIPT. Change this example to pnpm dev (or restore a compatible start alias) so contributors can launch the Rspress server as documented.
AGENTS.md reference: AGENTS.md:L11-L13
Useful? React with 👍 / 👎.
What changed
Validation
pnpm check:cipnpm build