Skip to content

feat(commonly): forward-port runtime collaboration tools - #10

Open
lilyshen0722 wants to merge 1 commit into
mainfrom
feat/commonly-runtime-tools-forward-port
Open

feat(commonly): forward-port runtime collaboration tools#10
lilyshen0722 wants to merge 1 commit into
mainfrom
feat/commonly-runtime-tools-forward-port

Conversation

@lilyshen0722

Copy link
Copy Markdown

Summary

  • forward-port commonly_read_my_memory, commonly_save_my_memory, commonly_log_cycle, commonly_open_dm, and commonly_read_attachment from the frozen rebase branch
  • preserve main-only commonly_react_to_message
  • keep cycles append-only and return truncation/eviction observability flags from newer backends
  • install the PDF/fallback extraction dependencies the attachment tool advertises

Dependency / rollout

commonly_read_attachment is intentionally dependent on Team-Commonly/commonly#830: runtime tokens must be accepted by the backend attachment read ACL. Do not update the Commonly submodule pin or deploy this extension until openclaw#830 is merged and live.

Verification

  • pnpm exec vitest run --config vitest.extensions.config.ts extensions/commonly/src/client.test.ts extensions/commonly/src/channel.ensemble.test.ts extensions/commonly/src/websocket.test.ts extensions/commonly/src/tools.runtimeForwardPort.test.ts --maxWorkers=1 (28 passed)
  • pnpm lint (0 errors)
  • oxfmt --check and git diff --check
  • mutation: remove commonly_read_attachment registration -> 2 forward-port tests fail
  • validated poppler-utils + markitdown install/import in a Node 22 Bookworm container

Known baseline

pnpm tsgo still reports four existing errors in untouched extensions/commonly/src/channel.ts and extensions/commonly/src/websocket.test.ts; no changed file is in its output.

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.

1 participant