Skip to content

fix: restore OpenClaw PDF and browser runtime dependencies#689

Merged
mrcfps merged 6 commits intomainfrom
fix-openclaw-prune
Mar 31, 2026
Merged

fix: restore OpenClaw PDF and browser runtime dependencies#689
mrcfps merged 6 commits intomainfrom
fix-openclaw-prune

Conversation

@nettee
Copy link
Copy Markdown
Contributor

@nettee nettee commented Mar 30, 2026

What

Restore the OpenClaw runtime dependencies needed for PDF parsing and Playwright-backed browser actions, and document the validation results in the pruning regression spec.

Why

Recent runtime pruning removed packages that OpenClaw still needs for user-visible PDF and browser workflows. URL-based PDF parsing and Playwright interactions now work again after restoring those dependencies, while the remaining Feishu direct-upload PDF problem is tracked separately in #688.

Closes #425
Closes #431
Refs #688

How

  • stop pruning pdfjs-dist, @napi-rs, and playwright-core from openclaw-runtime
  • update the prune-size comments with fresh measured runtime sizes after the new pruning rules
  • update the spec with the successful local validation results and clarify that the remaining direct-upload PDF failure is a Feishu file handoff issue, not a missing parser dependency

Affected areas

  • Desktop app (Electron shell)
  • Controller (backend / API)
  • Web dashboard (React UI)
  • OpenClaw runtime
  • Skills
  • Shared schemas / packages
  • Build / CI / Tooling

Checklist

  • pnpm typecheck passes
  • pnpm lint passes
  • pnpm test passes
  • pnpm generate-types run (if API routes/schemas changed)
  • No credentials or tokens in code or logs
  • No any types introduced (use unknown with narrowing)

Notes for reviewers

  • Packaging size impact: restores about 75 MB of uncompressed OpenClaw sidecar dependencies (pdfjs-dist ~40 MB, @napi-rs ~25 MB, playwright-core ~10 MB). The refreshed runtime install baseline also dropped about 13 MB, so the expected net packaged size increase is about 64 MB before DMG/ZIP compression.
  • pnpm test still has an existing unrelated failure in tests/desktop/skill-dir-watcher-workspace.test.ts (watches workspace directories after start and reconciles removals timed out waiting for watcher state).
  • Direct Feishu PDF uploads are still not wired through to the PDF tool path; that follow-up is filed in fix: Feishu PDF uploads are not handed off to the PDF tool #688.

@mrcfps mrcfps merged commit 8fa3c27 into main Mar 31, 2026
11 of 12 checks passed
@mrcfps mrcfps deleted the fix-openclaw-prune branch March 31, 2026 06:03
@lefarcen lefarcen mentioned this pull request Apr 1, 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.

[Bug] Playwright runtime not available [Bug] PDF file recognition failure

2 participants