GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — Declare openclaw as a peer dependency of the OpenClaw plugin by edwinyyyu · Pull Request #1891 · MemMachine/MemMachine

Conversation

@edwinyyyu

@edwinyyyu edwinyyyu commented Nov 2, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Contributor

Purpose of the change

Stop counting OpenClaw's own dependency tree as runtime dependencies of the OpenClaw plugin. integrations/openclaw/package-lock.json has 219 open Dependabot alerts, 205 of them runtime scope. The plugin listed openclaw, the whole host application, as a regular dependency, and the lockfile has not changed since May (#1393).

Description

  • openclaw moves from dependencies to peerDependencies and keeps the plugin's range, ^2026.4.10. OpenClaw's plugin guide declares the host this way. Its dependency resolution page says managed plugin installs run npm install --omit=dev --omit=peer ... and link the host's own package into plugins that declare the host peer.
  • openclaw is also a development dependency, ~2026.7.35, so the type check, build, and smoke test have a host package to run against. 2026.7 is the newest line the plugin builds against (see Further comments). tsup still keeps openclaw/plugin-sdk/core external in dist.
  • The lockfile is refreshed with npm update within the declared ranges, with no range changes. This moves @memmachine/client from 0.3.4 to 0.3.9, axios from 1.16.0 to 1.21.0, and the development tree to the newest versions its ranges allow.

After this change the runtime tree is @memmachine/client, @sinclair/typebox, and their dependencies, 27 packages in all. npm audit --omit=dev reports no advisories for it; it reads the GitHub Advisory Database, the same source Dependabot uses. The remaining advisories are all in the development tree.

Type of change

  • Project Maintenance (updates to build scripts, CI, etc., that do not affect the main project)
  • Security (improves security without changing functionality)

How Has This Been Tested?

  • Unit Test
  • Manual verification

Run at the head in integrations/openclaw/, on Node 22.23.4, macOS arm64:

  • npm ci --ignore-scripts, npm run lint, npm test (3 suites, 37 tests passed), and npm run test:smoke (builds dist and imports dist/index.mjs against the installed OpenClaw) pass.
  • npm audit --omit=dev reports 0 advisories. npm audit reports 49, all in development scope. On main's lockfile, npm audit reports 224.

Test Results: as listed above.

Checklist

  • I have signed the commit(s) within this pull request
  • I have performed a self-review of my own code
  • New and existing unit tests pass locally with my changes

Further comments

  • The plugin does not build against OpenClaw 2026.8.1 or later, and this PR does not change that. OpenClaw 2026.8.1 (2026-08-31) removed three things index.mts uses: the root openclaw/plugin-sdk entry point, api.registerMemoryPromptSection (OpenClaw's migration notes replace it with registerMemoryCapability), and the before_agent_start hook. The type check passes against 2026.7.35 and fails on exactly these three against 2026.8.1, 2026.8.2, 2026.8.33, 2026.8.34, 2026.9.36, and 2026.10.9. The 2026.10.9 runtime defines no registerMemoryPromptSection, and register() calls it before registering anything else. The plugin's ^2026.4.10 range admits all of those hosts. Porting the plugin to the current SDK is separate work; once that lands, the development pin can move to the current release.
  • Dependabot still reports the development-scope advisories. A repository admin can triage them with a custom auto-triage rule scoped to this manifest and to development scope.
  • Build(deps): Bump the npm_and_yarn group across 2 directories with 13 updates #1718 bumps this lockfile and packages/ts-client's. This PR supersedes its integrations/openclaw half.

🤖 Written by Claude Code (Claude Opus 5.5) on behalf of @edwinyyyu.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
edwinyyyu and others added 4 commits November 2, 2026 10:47
@edwinyyyu @claude
Declare openclaw as a peer dependency of the OpenClaw plugin
e98d2bd
The plugin runs inside an OpenClaw host and imports the host's SDK
through openclaw/plugin-sdk/*. OpenClaw's plugin guide declares the
host as a peer dependency: managed plugin installs omit peers and link
the host's own package into plugins that declare it. As a regular
dependency, openclaw put the host's whole dependency tree into the
plugin's dependency graph, and the repository lockfile counted that
tree as runtime dependencies of the plugin.

The peer range keeps the hosts the plugin already accepted
(^2026.4.10). The plugin's type check, build, and smoke test need a
concrete host package, so openclaw is also a development dependency,
pinned to the 2026.7 line (~2026.7.35): it is the newest line the
plugin builds against. OpenClaw 2026.8.1 removed the root
openclaw/plugin-sdk entry point, api.registerMemoryPromptSection, and
the before_agent_start hook, all of which index.mts uses.

Co-Authored-By: Claude Opus 5.5 <[email protected]>
Signed-off-by: Edwin Yu <[email protected]>
@edwinyyyu @claude
Refresh the OpenClaw plugin's lockfile
45fbce2
npm update within the ranges package.json already declares; no range
changes. The runtime tree moves to @memmachine/client 0.3.9 and axios
1.21.0, and the development tree to the newest releases its ranges
allow. Lockfile only.

Co-Authored-By: Claude Opus 5.5 <[email protected]>
Signed-off-by: Edwin Yu <[email protected]>
@edwinyyyu @claude
Build and test the OpenClaw plugin on pull requests
12c1ff4
No workflow checked integrations/openclaw before a release: the
publish workflow was the only one that built it. Changes to the
plugin, including dependency update pull requests for its lockfile,
merged without a type check, test run, or build.

The workflow installs from the lockfile with npm ci, then runs the
plugin's type check, Jest suite, and smoke test, which builds dist and
imports the built module against the installed OpenClaw package. It
runs on Node 22, the version the publish workflow builds with, and
follows ts-client-test.yml's triggers and draft skip.

Co-Authored-By: Claude Opus 5.5 <[email protected]>
Signed-off-by: Edwin Yu <[email protected]>
@edwinyyyu @claude
Drop the OpenClaw plugin test workflow
2636097
No other integration or example has its own CI workflow, and the
dependency change does not need one.

Co-Authored-By: Claude Opus 5.5 <[email protected]>
Signed-off-by: Edwin Yu <[email protected]>
@edwinyyyu edwinyyyu changed the title Declare openclaw as a peer dependency of the OpenClaw plugin and test the plugin on pull requests Declare openclaw as a peer dependency of the OpenClaw plugin Nov 2, 2026
@edwinyyyu
edwinyyyu requested a review from malatewang November 2, 2026 19:38
@edwinyyyu edwinyyyu mentioned this pull request Nov 2, 2026
integrations/openclaw: maintain with CI or remove (uses APIs current OpenClaw removed; no pull-request CI) #1901
Open

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@malatewang malatewang Awaiting requested review from malatewang

At least 2 approving reviews are required to merge this pull request.

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

1 participant

@edwinyyyu
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.
You are visitor 01402520 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.