Support new openclaw plugin request#1191
Merged
Merged
Conversation
sscargal
approved these changes
Mar 5, 2026
sscargal
left a comment
sscargal
left a comment
Contributor
There was a problem hiding this comment.
LGTM.
This has a dependency on PR #1195, which will version control the Typescript client with MemMachine. This plugin requires @memmachine/client version 0.3.0 or newer. Currently, the Typescript is v0.2.5 (hard-coded). The Typescript client PR will create a v0.3.x that satisfies this plugin requirement.
malatewang
force-pushed
the
upgrade_openclaw
branch
from
839ff2f to
0d6d441
Compare
Contributor
There was a problem hiding this comment.
Pull request overview
Updates the OpenClaw integration package to meet newer OpenClaw plugin requirements (notably distributing an .mjs entrypoint) and adjusts build/publish tooling to produce and publish the expected artifacts.
Changes:
- Switch build output to include an explicit
.mjsbundle (viatsup.config.ts) and move source entry toindex.mts. - Add local TS type shims and TS config path mappings for dependencies used by the plugin build.
- Update the OpenClaw plugin publish workflow to support
workflow_dispatchversioned builds and to rewritepackage.jsonversions during publish.
Reviewed changes
Copilot reviewed 5 out of 9 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| integrations/openclaw/types/shims/typebox.d.ts | Adds a local shim for @sinclair/typebox types. |
| integrations/openclaw/types/shims/openclaw-plugin-sdk.d.ts | Adds a local shim for openclaw/plugin-sdk types. |
| integrations/openclaw/types/shims/memmachine-client.d.ts | Adds a local shim for @memmachine/client types. |
| integrations/openclaw/tsup.config.ts | New tsup config to emit .mjs for ESM builds. |
| integrations/openclaw/tsconfig.json | Adjust TS compiler settings and wire in shim path mappings. |
| integrations/openclaw/package.json | Switch to a development placeholder versioning approach; adjust scripts for tsup config. |
| integrations/openclaw/package-lock.json | Removes rimraf; otherwise retains prior resolved versions. |
| integrations/openclaw/index.mts | Updates env API key lookup to avoid direct process reference. |
| .github/workflows/openclaw_plugin_publish.yml | Adds manual version input and rewrites package version/dependency before publishing. |
Files not reviewed (1)
- integrations/openclaw/package-lock.json: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
malatewang
force-pushed
the
upgrade_openclaw
branch
from
0d6d441 to
38b6975
Compare
o-love
approved these changes
Mar 9, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Purpose of the change
[Describe the high-level purpose of this pull request. What problem does it solve? What feature does it add? Link to any related issues or discussions.]
Support new openclaw versions
Description
openclaw upgraded the plugin security. It requires the plugin to be compiled to mjs, instead of js. So the current plugin can not support new openclaw version. This PR updated the openclaw plugin to support the new requirement.
Fixes/Closes
Fixes #(issue number)
Type of change
[Please delete options that are not relevant.]
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration.
[Please delete options that are not relevant.]
manually tested. The package will have additional files undere the dist directory:
index.d.cts index.d.ts index.js index.js.map index.mjs index.mjs.map
Test Results: [Attach logs, screenshots, or relevant output]
Checklist
[Please delete options that are not relevant.]
Maintainer Checklist
Screenshots/Gifs
[If applicable, add screenshots or GIFs that show the changes in action. This is especially helpful for API responses. Otherwise, delete this section or type "N/A".]
Further comments
[Add any other relevant information here, such as potential side effects, future considerations, or any specific questions for the reviewer. Otherwise, type "None".]