Skip to content

fix: repair plugin bundle dependency closure for qqbot#1001

Merged
Siri-Ray merged 3 commits intomainfrom
fix/qqbot-plugin-bundle-deps
Apr 10, 2026
Merged

fix: repair plugin bundle dependency closure for qqbot#1001
Siri-Ray merged 3 commits intomainfrom
fix/qqbot-plugin-bundle-deps

Conversation

@Siri-Ray
Copy link
Copy Markdown
Contributor

@Siri-Ray Siri-Ray commented Apr 9, 2026

What

Repair the qqbot bundled plugin dependency closure so packaged desktop builds include silk-wasm, and fail packaging early if the qqbot plugin is incomplete.

Why

Upgraded packaged installs could keep an incomplete openclaw-qqbot runtime plugin, leaving QQ stuck in a restarting/starting state with Cannot find module 'silk-wasm' during plugin preload.

How

  • Start qqbot bundled dependency collection from the plugin's own node_modules before falling back to the pnpm virtual-store parent.
  • Add a regression test covering qqbot bundled runtime dependencies during extension materialization.
  • Add controller sidecar packaging checks for openclaw-qqbot/node_modules/silk-wasm/package.json.
  • Add mac packaged-app validation to ensure the final .app includes the qqbot silk-wasm 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

  • I ran the targeted regression test: pnpm --filter @nexu/controller exec vitest run tests/openclaw-runtime-plugin-writer.test.ts.
  • I also built an Apple Silicon unsigned zip and verified the packaged app contains runtime/controller/plugins/openclaw-qqbot/node_modules/silk-wasm/package.json.
  • I did not mark full pnpm test as passing because the current main branch already has unrelated failing tests.

@sentry
Copy link
Copy Markdown

sentry Bot commented Apr 9, 2026

@Siri-Ray Siri-Ray merged commit a655585 into main Apr 10, 2026
15 checks passed
@lefarcen lefarcen mentioned this pull request Apr 10, 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.

2 participants