Skip to content

chore(agent-memory): ADR-003 Phase 2b — bump openclaw + gateway tag#192

Merged
lilyshen0722 merged 1 commit intomainfrom
feat/adr-003-phase2b-tools-deploy
Apr 15, 2026
Merged

chore(agent-memory): ADR-003 Phase 2b — bump openclaw + gateway tag#192
lilyshen0722 merged 1 commit intomainfrom
feat/adr-003-phase2b-tools-deploy

Conversation

@lilyshen0722
Copy link
Copy Markdown
Contributor

Summary

Phase 2b deployment artifacts — submodule pointer + values bump for the new OpenClaw tool shims (`commonly_read_my_memory`, `commonly_save_my_memory`) that wrap the Phase 2a `POST /memory/sync` kernel endpoint.

  • `_external/clawdbot` → `826d46475` (Team-Commonly/openclaw `rebase-2026.3.29`)
  • `agents.clawdbot.image.tag` → `20260414192025` (gateway image with the new tools)
  • `backend.image.tag` → `20260414125715` (Phase 2a, already deployed; committing in-tree value to match live)

Tool work in the submodule

Already pushed to `Team-Commonly/openclaw rebase-2026.3.29` at `826d46475`. Highlights:

  • New typed envelope types in `extensions/commonly/src/client.ts` + new `syncAgentMemory(sections, { mode, sourceRuntime })` method
  • New `commonly_read_my_memory({ section? })` tool — returns full envelope or one validated section
  • New `commonly_save_my_memory({ section, content?, visibility?, entries? })` tool — patch-mode sync; siblings preserved; daily merges by date; relationships by otherInstanceId
  • Visibility defaults to `private` at the tool layer (ADR-003 invariant fixing github workflow for unit testing #2 enforced on the wire, not just server-side)
  • Section names validated against `MemorySectionName` — typos rejected with structured error
  • `entries` + `content` conflict rejected up-front (no silent drops)
  • Kernel 4xx caught and returned as `{ ok:false, error:'kernel rejected: ...' }`
  • Old `commonly_read_agent_memory` / `commonly_write_agent_memory` untouched (per ADR §Phase 2 "Old tools remain as thin wrappers")
  • 5 new client tests; 20/20 ext tests pass locally

Reviewer findings (pre-commit on submodule)

Verdict was Approve with suggestions; all 5 Important + relevant Nits applied before commit. No Critical.

Test plan

  • 20/20 commonly extension vitest tests pass locally
  • Gateway image `clawdbot-gateway:20260414192025` built + pushed
  • CI green
  • Post-merge: helm upgrade to revision 58, verify the new tools appear in agent session jsonl, live-test `commonly_save_my_memory` end-to-end via one agent

Deferred to Phase 2c

  • `commonly_ask_agent` — cross-agent mediated DM, needs a new backend primitive
  • `commonly_read_shared_memory` — needs visibility-filtering read endpoint (Phase 4)

🤖 Generated with Claude Code

Submodule pointer bump for Team-Commonly/openclaw rebase-2026.3.29 →
826d46475 (commonly_read_my_memory + commonly_save_my_memory tools that
wrap POST /memory/sync). Helm chart bumps:

- agents.clawdbot.image.tag → 20260414192025 (gateway image with the new
  v2 memory tools)
- backend.image.tag → 20260414125715 (Phase 2a commit, already deployed
  to dev — committing the in-tree value to match)

No code changes in this PR — submodule + values only. Live verification
of /memory/sync via the new tools happens after rollout.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@lilyshen0722 lilyshen0722 merged commit 6507dd7 into main Apr 15, 2026
7 checks passed
@lilyshen0722 lilyshen0722 deleted the feat/adr-003-phase2b-tools-deploy branch April 15, 2026 02:35
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