Skip to content

Add OpenClaw dkg_share tool#387

Open
Jurij89 wants to merge 13 commits intomainfrom
codex/openclaw-dkg-share-tool
Open

Add OpenClaw dkg_share tool#387
Jurij89 wants to merge 13 commits intomainfrom
codex/openclaw-dkg-share-tool

Conversation

@Jurij89
Copy link
Copy Markdown
Contributor

@Jurij89 Jurij89 commented May 4, 2026

Summary

  • Synced codex/openclaw-dkg-share-tool with latest main (200d987b) and resolved with main logic prevailing.
  • Latest main already contains the OpenClaw dkg_share implementation, shared DKG skill docs, and plugin regression suite from the newer mainline implementation.
  • This PR now retains one focused additional regression: DkgDaemonClient.share() forwards an explicit subGraphName through to POST /api/shared-memory/write alongside explicit team-visible localOnly: false.

Related

Hermes parity notes

  • Main now exposes OpenClaw dkg_share as the direct Shared Working Memory convenience helper with the portable schema: required content and context_graph_id, optional sub_graph_name.
  • OpenClaw writes through POST /api/shared-memory/write; the lower-level client supports subGraphName forwarding, which this PR pins explicitly.
  • The canonical staged workflow remains dkg_assertion_create -> dkg_assertion_write -> dkg_assertion_promote; dkg_share remains a direct SWM convenience helper, not a replacement for assertions.

Files changed

File What
packages/adapter-openclaw/test/dkg-client.test.ts Adds focused coverage that share('research-x', quads, { localOnly: false, subGraphName: 'protocols' }) posts { contextGraphId, quads, localOnly: false, subGraphName: 'protocols' } to /api/shared-memory/write.

Tool schema and route/body mapping

  • Tool name: dkg_share (implemented on current main)
  • Required input: content, context_graph_id
  • Optional input: sub_graph_name
  • Route: POST /api/shared-memory/write
  • Client body pinned by this PR: { contextGraphId, quads, localOnly, subGraphName? }

Test plan

  • pnpm --filter @origintrail-official/dkg-core run build - passed after the latest main sync.
  • pnpm --filter @origintrail-official/dkg-adapter-openclaw test -- test/plugin.test.ts test/dkg-client.test.ts - passed, 240 tests.
  • pnpm --filter @origintrail-official/dkg-adapter-openclaw run build - passed.
  • git diff --check - passed.
  • PR hygiene: git diff upstream/main...HEAD shows only packages/adapter-openclaw/test/dkg-client.test.ts; no ignored/local-only files are staged.

Known risks or follow-ups

  • The PR diff is intentionally narrow after latest main absorbed the broader dkg_share implementation. Reviewers should evaluate this as a supplemental regression test rather than the original full implementation diff.
  • Fresh CI/check results are pending from the latest sync push (b9d64ce9). Immediate post-push state had CodeQL, Codex Review, Knip, Detect changes, and Build packages queued/in progress.

Comment thread packages/adapter-openclaw/src/DkgNodePlugin.ts Outdated
Comment thread packages/adapter-openclaw/src/DkgNodePlugin.ts Outdated
Comment thread packages/adapter-openclaw/test/plugin.test.ts Outdated
Comment thread packages/adapter-openclaw/src/DkgNodePlugin.ts Outdated
Comment thread packages/adapter-openclaw/test/plugin.test.ts Outdated
Comment thread packages/cli/skills/dkg-node/SKILL.md Outdated
Comment thread packages/adapter-openclaw/test/plugin.test.ts Outdated
Comment thread packages/adapter-openclaw/src/DkgNodePlugin.ts Outdated
Comment thread packages/cli/skills/dkg-node/SKILL.md Outdated
Comment thread packages/adapter-openclaw/src/DkgNodePlugin.ts Outdated
Comment thread packages/adapter-openclaw/test/plugin.test.ts Outdated
Comment thread packages/adapter-openclaw/src/DkgNodePlugin.ts Outdated
Comment thread packages/adapter-openclaw/src/DkgNodePlugin.ts Outdated
Comment thread packages/cli/skills/dkg-node/SKILL.md Outdated
Comment thread packages/adapter-openclaw/src/DkgNodePlugin.ts Outdated
Comment thread packages/adapter-openclaw/src/DkgNodePlugin.ts Outdated
Comment thread packages/adapter-openclaw/src/DkgNodePlugin.ts Outdated
Comment thread packages/cli/skills/dkg-node/SKILL.md Outdated
Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Codex review completed — no issues found.

…share-tool

# Conflicts:
#	packages/adapter-openclaw/src/dkg-client.ts
#	packages/adapter-openclaw/test/plugin.test.ts
#	packages/cli/skills/dkg-node/SKILL.md
Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Codex review completed — no issues found.

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