Skip to content

fix: publish compiled OpenClaw plugin entry#2013

Draft
RerankerGuo wants to merge 1 commit into
MemTensor:mainfrom
RerankerGuo:fix/openclaw-compiled-package-entry
Draft

fix: publish compiled OpenClaw plugin entry#2013
RerankerGuo wants to merge 1 commit into
MemTensor:mainfrom
RerankerGuo:fix/openclaw-compiled-package-entry

Conversation

@RerankerGuo

Copy link
Copy Markdown

Fixes #1675.

Summary

  • Point the npm package and OpenClaw manifests at the compiled entrypoint ./dist/index.js instead of index.ts.
  • Include dist and tsconfig.json in the published package, and run npm run build during prepublish.
  • Compile from the package root so the root OpenClaw plugin entry becomes dist/index.js.
  • Align the package type with the existing CommonJS TypeScript output and avoid requiring an unavailable openclaw/plugin-sdk type package at build time.
  • Add a package manifest regression test for the compiled entrypoint configuration.

Validation

  • node manifest check for package.json, openclaw.plugin.json, and tsconfig.json passed.
  • npm pack --dry-run --ignore-scripts --cache /tmp/npm-cache passed and showed package metadata can be packed without the read-only global npm cache.
  • Attempted: npm test -- --run tests/package-manifest.test.ts; blocked because vitest is not installed in this worktree.
  • Attempted: npm run build; blocked because tsc is not installed in this worktree.
  • Attempted: make format; blocked because poetry is not installed in this environment.

@Memtensor-AI Memtensor-AI changed the base branch from main to dev-v2.0.22 July 1, 2026 13:14
@Memtensor-AI

Copy link
Copy Markdown
Collaborator

Automated Test Results: QUEUED

Conflict with dev-v2.0.22 was resolved by merging the base branch into this PR head. Manifest assertions passed locally; cloud test-engine rerun has been queued.

  • Run: tr-186b16f3-de9 on cloud test-engine 10011
  • Branch tested: refs/pull/2013/head

I will post the final pass/fail result when the run completes.

@Memtensor-AI

Copy link
Copy Markdown
Collaborator

Automated Test Results: PASSED

Cloud test-engine rerun against dev-v2.0.22 completed successfully.

  • Run: tr-186b16f3-de9 on cloud test-engine 10011
  • memos_local_openclaw/unit: 10 passed, 0 failed, 0 skipped

Manual code review is still required before merge.

@CarltonXiang CarltonXiang deleted the branch MemTensor:main July 3, 2026 07:25
@syzsunshine219 syzsunshine219 reopened this Jul 3, 2026
@syzsunshine219 syzsunshine219 added the needs-audit Requires manual audit before merge label Jul 3, 2026
@syzsunshine219 syzsunshine219 changed the base branch from dev-v2.0.22 to main July 3, 2026 08:20
@RerankerGuo

Copy link
Copy Markdown
Author

Hi @Memtensor-AI / @CarltonXiang — friendly ping on this draft.

Quick observation: this PR still has base branch main, but MemTensor/MemOS's current merge flow is into dev-v2.0.22 (every recent PR #2065 / #2058 / #2047 / #2046 / #2042 / #2034 / #2033 / #2026 lands on dev-v2.0.22). The bot already retargeted the base SHA for testing and cloud test-engine returned PASSED against dev-v2.0.22 on every rerun.

Could a maintainer please change the base branch from maindev-v2.0.22 (Edit → base branch dropdown) so this can enter the normal merge queue? The head commit is a single focused change and does not depend on any main-only commits.

Thanks!

@RerankerGuo

Copy link
Copy Markdown
Author

Hi @syzsunshine219 — looks like you've merged 35 of the last 50 PRs on this repo. Friendly ping on this one.

This draft (and the other 21 OPEN PRs I have here) is small, single-commit, has cloud test-engine PASSED, and is fully mergeable. The only thing blocking it is that the base branch dropdown still says main — every recent merge (e.g. #2038, #2037, #2034, #2032) landed on dev-v2.0.22.

When you have a moment, would you mind one of:

  1. Flipping the base branch on this PR from maindev-v2.0.22 (Edit → base branch dropdown), or
  2. Telling me which base branch you'd prefer and I'll re-push accordingly.

Both fix and feature PRs in this batch are independent — each can be reviewed/merged on its own. Happy to rebase / split / drop any that don't fit the direction.

Thanks!

@RerankerGuo RerankerGuo force-pushed the fix/openclaw-compiled-package-entry branch from 9a9aba2 to 0016bcd Compare July 8, 2026 07:45
@Memtensor-AI Memtensor-AI added the area:plugin Plugin, adapter, bridge, or apps layer | 插件、适配器、桥接层或 apps 目录 label Jul 8, 2026
@Memtensor-AI Memtensor-AI requested a review from bittergreen July 8, 2026 11:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:plugin Plugin, adapter, bridge, or apps layer | 插件、适配器、桥接层或 apps 目录 needs-audit Requires manual audit before merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix: memos-local-openclaw-plugin@1.0.10 打包缺少 dist/ 目錄,無法在 OpenClaw 2026.5.x 使用

5 participants