chore: move slimclaw runtime ownership and cut startup cost#1137
chore: move slimclaw runtime ownership and cut startup cost#1137PerishCode wants to merge 8 commits intomainfrom
Conversation
Codecov Report❌ Patch coverage is 📢 Thoughts on this report? Let us know! |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 767dbe2d16
ℹ️ 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".
| const weixinResult = await timedStep( | ||
| "prepare-builtin-weixin-plugin", | ||
| async () => prepareBuiltinWeixinPlugin(runtimeDir), | ||
| timings, | ||
| ); |
There was a problem hiding this comment.
This migration installs openclaw-weixin into slimclaw’s builtin runtime, but it does not remove the legacy copy that older versions already wrote to OPENCLAW_STATE_DIR/extensions/openclaw-weixin. Since controller config still loads plugins from the state extensions path (plugins.load.paths), upgraded users can keep running the stale copied plugin instead of the new builtin one, so fixes in the migrated plugin may never take effect after upgrade. Add an explicit cleanup/migration step for the old state extension directory when switching ownership to builtin.
Useful? React with 👍 / 👎.
Deploying nexu-docs with
|
| Latest commit: |
92bf186
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://8ff87bb7.nexu-docs.pages.dev |
| Branch Preview URL: | https://feat-minor-issue-resolver.nexu-docs.pages.dev |
1 participant
What
Move slimclaw's prepared runtime ownership fully into
packages/slimclaw, migrate the bundled Weixin runtime there, and reduce OpenClaw warm-start latency with new timing visibility and Feishu startup-path cleanup.Why
The branch tightens runtime ownership boundaries around slimclaw and removes avoidable startup cost on the OpenClaw critical path. In practice this cuts warm
portReadyMsmaterially versusmain, with the biggest win coming from moving the Weixin runtime into slimclaw and additional wins from pruning obsolete runtime overlays and improving startup diagnostics.How
openclaw-weixinruntime assets and ownership into slimclaw's prepared runtime flowAffected areas
Checklist
pnpm typecheckpassespnpm lintpassespnpm testpassespnpm generate-typesrun (if API routes/schemas changed)anytypes introduced (useunknownwith narrowing)Notes for reviewers
4.2sfaster onportReadyMsthanorigin/mainin the comparison run, with most of the gain coming from the Weixin runtime move.