Skip to content

ci(website): reinstall platform dependencies from clean tree#307

Closed
hongjr03 wants to merge 1 commit into
masterfrom
fix/docs-rolldown-native-binding
Closed

ci(website): reinstall platform dependencies from clean tree#307
hongjr03 wants to merge 1 commit into
masterfrom
fix/docs-rolldown-native-binding

Conversation

@hongjr03

Copy link
Copy Markdown
Member

Summary

  • Reinstall website dependencies from a clean node_modules tree in docs preview and deployment workflows.
  • Keep the existing lockfile validation while ensuring the runner resolves platform-specific native optional dependencies.

Root cause

The docs job failed because npm did not install @rolldown/binding-linux-x64-gnu. npm can prune platform-specific optional dependencies when node_modules already exists, so the existing npm ci followed by npm install --no-save did not repair the dependency tree.

Validation

  • actionlint .github/workflows/docs-preview.yml .github/workflows/deploy-docs.yml (no YAML errors; existing ShellCheck warnings remain in deploy-docs.yml).
  • npm ci --ignore-scripts in website.
  • npm --prefix playground run build:embed passed.
  • Full npm run build reached the expected WASM asset check but could not continue locally because CI-generated website/playground/public/wasm/vide-lsp.js was not present.

@hongjr03 hongjr03 closed this Jul 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant