refactor: add SDK transcript identity target API by jalehman · Pull Request #95030 · openclaw/openclaw

jalehman · 2026-06-19T16:43:40Z

What

Adds one combined public SDK transcript runtime API bundle for Path 3. This supersedes #89261 and #89262 by exposing the coherent contract in a single review unit: stable transcript identity plus scoped transcript target/read/write/publish/lock helpers.

Why

#89261 and #89262 were split across a stale stack, but the public API is one contract. Plugins need identity helpers and scoped target writers together so reads, appends, update publishes, and write locks bind to the same session transcript target without treating sessionFile as durable identity. The compatibility behavior remains additive and limited to callers that already receive active transcript artifacts.

Changes

Add transcript runtime SDK subpath
Add transcript memory keys
Add scoped target/write helpers
Keep identity reads read-only
Re-export memory keys from lightweight hit subpath
Update SDK docs and metadata
Refresh plugin SDK API baseline

Changes Walkthrough

File	Change
`src/plugin-sdk/session-transcript-runtime.ts`	Adds public identity, target, read, append, publish, and lock helpers
`src/config/sessions/transcript-append.ts`	Adds scoped append transaction helper over existing write lock and FIFO
`src/config/sessions/session-accessor.ts`	Preserves malformed-line tolerance for parsed transcript reads
`src/plugin-sdk/session-transcript-hit.ts`	Re-exports transcript memory hit key helpers from a lightweight contract
`src/plugin-sdk/session-transcript-memory-hit.ts`	Hosts shared memory key helpers without transcript writer imports
`docs/plugins/sdk-runtime.md`	Documents the combined transcript runtime contract
`docs/plugins/sdk-subpaths.md`	Lists the new SDK subpath
`package.json` / `scripts/lib/plugin-sdk-entrypoints.json`	Publish the SDK subpath
`docs/.generated/plugin-sdk-api-baseline.sha256`	Refreshes the public API baseline hash

Verification

pnpm docs:list - passed after workspace deps installed
node scripts/run-vitest.mjs src/plugin-sdk/session-transcript-runtime.test.ts src/plugin-sdk/session-transcript-hit.test.ts src/config/sessions/session-accessor.test.ts - passed
node scripts/sync-plugin-sdk-exports.mjs --check - passed
node scripts/check-plugin-sdk-subpath-exports.mjs - passed
node --max-old-space-size=8192 --import tsx scripts/generate-plugin-sdk-api-baseline.ts --check - passed
node scripts/check-session-accessor-boundary.mjs - passed
./node_modules/.bin/oxfmt --check --threads=1 ... - passed
node scripts/format-docs.mjs --check docs/plugins/sdk-runtime.md docs/plugins/sdk-subpaths.md - passed
node scripts/run-oxlint.mjs --tsconfig config/tsconfig/oxlint.core.json ... - passed
node scripts/run-tsgo.mjs -p tsconfig.core.json --incremental --tsBuildInfoFile .artifacts/tsgo-cache/sdk-transcript-api-core.tsbuildinfo - passed
node scripts/run-tsgo.mjs -p test/tsconfig/tsconfig.core.test.json --incremental --tsBuildInfoFile .artifacts/tsgo-cache/sdk-transcript-api-core-test.tsbuildinfo - passed
git diff --check - passed
.agents/skills/autoreview/scripts/autoreview --mode local --base upstream/main ... - first run found a read-side mutation bug; fixed and retested
.agents/skills/autoreview/scripts/autoreview --mode local --base upstream/main ... - clean

Refs #88838.
Supersedes #89261.
Supersedes #89262.

Real behavior proof

Behavior addressed: Public SDK transcript runtime contract for Path 3: stable transcript identity plus scoped target/read/write/lock helpers, including deprecated sessionFile compatibility for callers that already receive an active transcript artifact.

Real environment tested: Local PR checkout on macOS, PR head 1125743c702f0c66c609ff343090a0a0f7556385, Node v24.15.0. The proof imported the built package subpaths after corepack pnpm build.

Exact steps or command run after this patch:

node scripts/run-vitest.mjs src/plugin-sdk/session-transcript-runtime.test.ts src/plugin-sdk/session-transcript-hit.test.ts test/scripts/plugin-sdk-surface-report.test.ts
node scripts/check-plugin-sdk-subpath-exports.mjs
node --max-old-space-size=8192 scripts/plugin-sdk-surface-report.mjs --check
node scripts/sync-plugin-sdk-exports.mjs --check
./node_modules/.bin/oxfmt --check --threads=1 src/plugin-sdk/session-transcript-memory-hit.ts src/plugin-sdk/session-transcript-hit.ts src/plugin-sdk/session-transcript-hit.test.ts src/plugin-sdk/session-transcript-runtime.ts src/plugin-sdk/session-transcript-runtime.test.ts
node scripts/run-oxlint.mjs --tsconfig config/tsconfig/oxlint.core.json src/plugin-sdk/session-transcript-memory-hit.ts src/plugin-sdk/session-transcript-hit.ts src/plugin-sdk/session-transcript-hit.test.ts src/plugin-sdk/session-transcript-runtime.ts src/plugin-sdk/session-transcript-runtime.test.ts
node scripts/run-tsgo.mjs -p tsconfig.core.json --incremental --tsBuildInfoFile .artifacts/tsgo-cache/sdk-transcript-api-core.tsbuildinfo
node scripts/run-tsgo.mjs -p test/tsconfig/tsconfig.core.test.json --incremental --tsBuildInfoFile .artifacts/tsgo-cache/sdk-transcript-api-core-test.tsbuildinfo
git diff --check
corepack pnpm build
node --input-type=module <<'NODE'
import fs from 'node:fs';
import os from 'node:os';
import path from 'node:path';
import {
  appendSessionTranscriptMessageByIdentity,
  formatSessionTranscriptMemoryHitKey,
  parseSessionTranscriptMemoryHitKey,
  publishSessionTranscriptUpdateByIdentity,
  readSessionTranscriptEvents,
  resolveSessionTranscriptIdentity,
  resolveSessionTranscriptMemoryHitKeyToSessionKeys,
  resolveSessionTranscriptTarget,
  withSessionTranscriptWriteLock,
} from 'openclaw/plugin-sdk/session-transcript-runtime';
import {
  formatSessionTranscriptMemoryHitKey as formatHitSubpathKey,
  parseSessionTranscriptMemoryHitKey as parseHitSubpathKey,
} from 'openclaw/plugin-sdk/session-transcript-hit';
import { getSessionEntry, upsertSessionEntry } from 'openclaw/plugin-sdk/session-store-runtime';
// Creates a temp sessions.json + active transcript JSONL, resolves identity,
// reads malformed-tolerant events, appends by identity, appends under lock,
// publishes with stale caller-supplied identity metadata, verifies persisted
// bytes and public shapes, then removes the temp root.
NODE
.agents/skills/autoreview/scripts/autoreview --mode local --base upstream/main --prompt 'Review only the local fix for PR #95030: memory hit key helpers were moved out of session-transcript-runtime so session-transcript-hit no longer imports the writer/read-lock runtime. Check public SDK export compatibility, module-load topology, and tests. Treat broader PR design questions outside this local patch as already tracked unless this patch regresses them.'

Evidence after fix: Focused Vitest passed 2 shards with 42 tests. SDK subpath/export/surface checks passed with public callable exports budgeted at 5183. Oxfmt, oxlint, both tsgo wrappers, git diff --check, and corepack pnpm build passed. The standalone package-import proof exited 0 and printed:

{
  "importSurface": "openclaw/plugin-sdk/session-transcript-runtime",
  "hitSubpathImportSurface": "openclaw/plugin-sdk/session-transcript-hit",
  "memoryKey": "transcript:main:proof-session%3Acurrent-head",
  "hitSubpathSameKey": true,
  "parsedAgentId": "main",
  "hitParsedAgentId": "main",
  "resolvedKeys": ["agent:main:proof"],
  "identityReturnedSessionFile": false,
  "targetReturnedSessionFile": false,
  "targetKind": "active-session-file",
  "lockedTargetReturnedSessionFile": false,
  "beforeCount": 2,
  "malformedLineSkipped": true,
  "appendReturnedMessage": true,
  "afterCount": 4,
  "persistedContainsAppend": true,
  "persistedContainsLockAppend": true,
  "publishWithStaleInputCompleted": true,
  "finalStoreSessionFileUnchanged": true
}
{"cleanupRemoved":true}

Autoreview result: .agents/skills/autoreview/scripts/autoreview --mode local --base upstream/main ... exited 0: autoreview clean: no accepted/actionable findings reported; overall patch correct, confidence 0.84.

Observed result after fix: The built public package export resolved from openclaw/plugin-sdk/session-transcript-runtime, and the existing hit subpath resolved the same memory key helpers without importing the transcript runtime writer/read-lock path. Identity, target, and locked target helpers returned public shapes without sessionFile. Reads skipped malformed JSONL rows, append and write-lock helpers persisted to the active transcript target, publish helpers completed while overriding stale caller-supplied publish identity internally, and cleanup removed the temporary proof state.

What was not tested: No third-party plugin package was installed, no future SQLite runtime flip was exercised, and no broad Crabbox/Testbox matrix was run for this narrow import-boundary fix. The proof used the current built package and focused SDK/runtime checks rather than restarting the local live gateway. A full local pnpm test run was also attempted on this machine and failed in unrelated provider, tooling, infra, media, iMessage, and security shards; the transcript SDK tests and SDK surface checks pass after the budget correction.

clawsweeper · 2026-06-19T16:45:05Z

Codex review: needs maintainer review before merge. Reviewed June 20, 2026, 1:27 PM ET / 17:27 UTC.

Summary
The PR adds a public openclaw/plugin-sdk/session-transcript-runtime subpath with transcript identity, target/read/write/publish/lock helpers, memory-hit exports, docs, export metadata, API baseline updates, and focused tests.

PR surface: Source +406, Tests +441, Docs +3, Config +4, Generated 0, Other +5. Total +859 across 14 files.

Reproducibility: not applicable. This is a public SDK/API refactor PR rather than a bug report; the review target is API behavior, proof quality, and merge risk.

Review metrics: 1 noteworthy metric.

SDK Surface Budget: +1 public entrypoint, +30 public exports, +10 public function exports, +1 deprecated-export entrypoint budget. The budget deltas quantify that this is a real public SDK expansion maintainers should consciously accept before merge.

Stored data model
Persistent data-model change detected: serialized state: src/config/sessions/session-accessor.ts, serialized state: src/config/sessions/transcript-append.ts, serialized state: src/plugin-sdk/session-transcript-hit.test.ts, serialized state: src/plugin-sdk/session-transcript-hit.ts, serialized state: src/plugin-sdk/session-transcript-memory-hit.ts, serialized state: src/plugin-sdk/session-transcript-runtime.test.ts, and 6 more. Confirm migration or upgrade compatibility proof before merge.

Root-cause cluster
Relationship: fixed_by_candidate
Canonical: #88838
Summary: This PR is a candidate implementation slice for the broader Path 3 session/transcript SQLite accessor-seam migration; the earlier split PRs are superseded by this combined review unit.

Members:

canonical: Track core session/transcript SQLite migration via accessor seam #88838 - The issue is the open Path 3 session/transcript SQLite migration tracker and this PR explicitly references it as the broader plan.
superseded: refactor: add public transcript identity API #89261 - The earlier public transcript identity draft is closed unmerged and this PR combines its identity surface into one review unit.
superseded: refactor: add scoped transcript target writers #89262 - The earlier scoped transcript target writer draft is closed unmerged and this PR combines its writer surface with the identity API.
partial_overlap: Feature Request: Expose appendAssistantMessageToSessionTranscript in Plugin Runtime API #73274 - That issue asks for durable public transcript append capability for plugins, overlapping the writer-helper portion with a narrower security/product focus.
partial_overlap: feat(plugin-sdk): export transcript.runtime subpath #80208 - That open PR proposes a narrower raw transcript runtime export, but it does not safely supersede this broader structured identity/target contract.
1 more in the report.

Proposal only: this assessment does not dispatch repair, suppress jobs, mutate sibling items, close, or merge anything.

Merge readiness
Overall: 🐚 platinum hermit
Proof: 🦞 diamond lobster
Patch quality: 🐚 platinum hermit
Result: ready for maintainer review.

Overall follows the weaker of proof and patch quality, so missing proof can cap an otherwise strong patch.

Rank-up moves:

[P2] Record maintainer acceptance of the public transcript SDK boundary before merge.

Risk before merge

[P1] Once released, the new SDK subpath, helper names, behavior, and deprecated sessionFile compatibility parameter become third-party plugin compatibility constraints.
[P1] The helpers intentionally expose trusted-plugin read/write/publish/lock access to persistent transcript state, so maintainers need to accept the plugin data boundary and target-binding semantics before merge.
[P1] The proof is strong for the current file-backed built package, but it does not prove a separately installed third-party plugin package or the future SQLite-backed implementation.

Maintainer options:

Accept The SDK Contract
A maintainer can approve the new subpath as the intended plugin contract after confirming the compatibility window and transcript access boundary are acceptable.
Narrow The Public Surface
If write, publish, or lock helpers are too broad for the first public release, remove or defer those exports while keeping only the identity/read portion maintainers are ready to support.
Pause Behind The Migration Tracker
If the API should wait for more Path 3 migration slices, keep this PR open or pause it under the canonical tracker rather than freezing the SDK now.

Next step before merge

[P2] The next action is human maintainer judgment on the public SDK transcript contract; no narrow automated code repair is indicated.

Security
Cleared: No concrete supply-chain, secrets, workflow, dependency, or code-execution regression was found; trusted-plugin transcript access remains a maintainer merge-risk decision.

Review details

Best possible solution:

Merge only after maintainers explicitly accept this transcript SDK contract as the intended Path 3 public boundary and are comfortable with the upgrade, security-boundary, and session-state implications.

Do we have a high-confidence way to reproduce the issue?

Not applicable. This is a public SDK/API refactor PR rather than a bug report; the review target is API behavior, proof quality, and merge risk.

Is this the best way to solve the issue?

Yes, contingent on maintainer API acceptance. The combined identity/target/read/write/lock subpath is a coherent shape over the existing accessor seam and avoids the narrower raw-export approach, but it is still a public contract decision.

AGENTS.md: found and applied where relevant.

Codex review notes: model internal, reasoning high; reviewed against ad5d2cbc1bcb.

Label changes

Label justifications:

P2: The PR expands a public SDK/session-state surface with bounded but real compatibility and data-boundary impact.
merge-risk: 🚨 compatibility: The diff adds a documented public plugin SDK subpath and transitional sessionFile parameter that third-party plugins may depend on after release.
merge-risk: 🚨 session-state: The new helpers can read, append, publish, and lock persistent transcript state by scoped session identity, so target-binding mistakes could affect session history.
merge-risk: 🚨 security-boundary: The new public helpers expand trusted-plugin transcript read/write/publish operations and require maintainer acceptance of that data boundary.
rating: 🐚 platinum hermit: Overall readiness is 🐚 platinum hermit; proof is 🦞 diamond lobster and patch quality is 🐚 platinum hermit.
status: 👀 ready for maintainer look: ClawSweeper has no concrete contributor-facing blocker left for this PR. Sufficient (terminal): The PR body includes current-head terminal proof from a built package covering import resolution, memory keys, read, append, lock, publish, and cleanup behavior.
proof: sufficient: Contributor real behavior proof is sufficient. The PR body includes current-head terminal proof from a built package covering import resolution, memory keys, read, append, lock, publish, and cleanup behavior.

Evidence reviewed

PR surface:

Source +406, Tests +441, Docs +3, Config +4, Generated 0, Other +5. Total +859 across 14 files.

View PR surface stats

Area	Files	Added	Removed	Net
Source	5	408	2	+406
Tests	2	441	0	+441
Docs	2	4	1	+3
Config	1	4	0	+4
Generated	1	2	2	0
Other	3	8	3	+5
Total	14	867	8	+859

What I checked:

Repository policy applied: Root policy was read fully and treats plugin APIs, auth/session state, persisted preferences, config, migrations, and fallback behavior as compatibility-sensitive review surfaces. (AGENTS.md:1, ad5d2cbc1bcb)
Scoped SDK policy applied: The scoped SDK guide defines src/plugin-sdk as the public contract between plugins and core and requires narrow, documented, aligned public subpaths with cheap module load. (src/plugin-sdk/AGENTS.md:1, ad5d2cbc1bcb)
Live PR state requires human handling: Live GitHub state shows the PR open at head e93458f39e663d31857b9b6fd2e84559e28531c7 with maintainer, proof: sufficient, and compatibility/session-state/security-boundary merge-risk labels. (e93458f39e66)
Current main does not already implement the new subpath: Current main has plugin-sdk/session-transcript-hit and internal transcript helpers, but no ./plugin-sdk/session-transcript-runtime package export, so the PR is not obsolete on main. (package.json:982, ad5d2cbc1bcb)
Public SDK export is added: The PR adds ./plugin-sdk/session-transcript-runtime to package exports and session-transcript-runtime to the SDK entrypoint list. (package.json:978, e93458f39e66)
Runtime contract is coherent over accessor seam: The new SDK module exposes identity, target, read, append, publish, and write-lock helpers while keeping sessionFile marked as deprecated active-artifact compatibility. (src/plugin-sdk/session-transcript-runtime.ts:41, e93458f39e66)

Likely related people:

jalehman: Recent merged history includes Path 3/session accessor and SDK session compatibility slices that this public transcript SDK PR builds on; this goes beyond simply opening the current PR. (role: migration stack contributor; confidence: high; commits: d216f7c876dd, f1cab04966fa, 49e6f5a524bc; files: src/config/sessions/session-accessor.ts, src/plugin-sdk, docs/plugins/sdk-runtime.md)
vincentkoc: Recent history touches session accessor, transcript append, and session helper cleanup around the current-main code paths used by the PR. (role: recent area contributor; confidence: medium; commits: 6f5fdb1e6bea, cc451f98cbae, e2292d18e215; files: src/config/sessions/session-accessor.ts, src/config/sessions/transcript-append.ts, src/config/sessions/transcript-write-context.ts)
steipete: Recent history includes plugin SDK runtime documentation, session transcript helper docs, and broader plugin SDK boundary/import-surface refactors relevant to a new public subpath. (role: adjacent SDK and docs contributor; confidence: medium; commits: c1b49bb1d0e6, feffb6d02ff8, c74fd6f01540; files: docs/plugins/sdk-runtime.md, docs/plugins/sdk-subpaths.md, src/plugin-sdk)

What the crustacean ranks mean

🦀 challenger crab: rare, exceptional readiness with strong proof, clean implementation, and convincing validation.
🦞 diamond lobster: very strong readiness with only minor maintainer review expected.
🐚 platinum hermit: good normal PR, likely mergeable with ordinary maintainer review.
🦐 gold shrimp: useful signal, but proof or patch confidence is still limited.
🦪 silver shellfish: thin signal; proof, validation, or implementation needs work.
🧂 unranked krab: not merge-ready because proof is missing/unusable or there are serious correctness or safety concerns.
🌊 off-meta tidepool: rating does not apply to this item.

Shiny media proof means a screenshot, video, or linked artifact directly shows the changed behavior. Runtime, network, CSP, and security claims still need visible diagnostics.

How this review workflow works

ClawSweeper keeps one durable marker-backed review comment per issue or PR.
Re-runs edit this comment so the latest verdict, findings, and automation markers stay together instead of adding duplicate bot comments.
A fresh review can be triggered by eligible @clawsweeper re-review comments, exact-item GitHub events, scheduled/background review runs, or manual workflow dispatch.
PR/issue authors and users with repository write access can comment @clawsweeper re-review or @clawsweeper re-run on an open PR or issue to request a fresh review only.
Maintainers can also comment @clawsweeper review to request a fresh review only.
Fresh-review commands do not start repair, autofix, rebase, CI repair, or automerge.
Maintainer-only repair and merge flows require explicit commands such as @clawsweeper autofix, @clawsweeper automerge, @clawsweeper fix ci, or @clawsweeper address review.
Maintainers can comment @clawsweeper explain to ask for more context, or @clawsweeper stop to stop active automation.

jalehman · 2026-06-19T17:07:13Z

@clawsweeper re-review

clawsweeper · 2026-06-19T17:07:16Z

🦞🧹
ClawSweeper re-review requested.

I asked ClawSweeper to review this item again.
Action: item re-review queued (workflow sweep.yml, event repository_dispatch).
Result: the existing ClawSweeper review comment will be edited in place when the review finishes.

jalehman · 2026-06-19T17:24:57Z


    Prefer `getSessionEntry(...)`, `listSessionEntries(...)`, `patchSessionEntry(...)`, or `upsertSessionEntry(...)` for session workflows. These helpers address sessions by agent/session identity so plugins do not depend on the legacy `sessions.json` storage shape. Use `preserveActivity: true` for metadata-only patches that should not refresh session activity, and `replaceEntry: true` only when the callback returns a complete entry and deleted fields must stay deleted.

+    For transcript reads and writes, import `openclaw/plugin-sdk/session-transcript-runtime` and use `resolveSessionTranscriptIdentity(...)`, `resolveSessionTranscriptTarget(...)`, `readSessionTranscriptEvents(...)`, `appendSessionTranscriptMessageByIdentity(...)`, or `withSessionTranscriptWriteLock(...)` with `{ agentId, sessionKey, sessionId }`. These APIs return stable transcript identity and storage-neutral memory hit keys without exposing `sessionFile` as the identity. `sessionFile` is deprecated transitional compatibility for plugins that already own an active transcript file artifact during the pre-SQLite window and need scoped helpers to bind to that artifact; new code should use structured transcript identity before the SQLite storage flip.


This reads like implementation notes moreso than actual public documentation.

Addressed in f3d4c70: rewrote the SDK runtime docs to describe the caller-visible transcript operations and when to pass an active transcript artifact, without implementation/storage migration wording.

jalehman · 2026-06-19T17:25:20Z

    | `plugin-sdk/reply-reference` | `createReplyReferencePlanner` |
    | `plugin-sdk/reply-chunking` | Narrow text/markdown chunking helpers |
    | `plugin-sdk/session-store-runtime` | Session workflow helpers (`getSessionEntry`, `listSessionEntries`, `patchSessionEntry`, `upsertSessionEntry`), legacy session store path/session-key helpers, updated-at reads, and transition-only whole-store/file-path compatibility helpers |
+    | `plugin-sdk/session-transcript-runtime` | Transcript identity, scoped target/read/write helpers, and storage-neutral memory hit key helpers that do not expose `sessionFile` as identity |


See previous comment. No need to mention storage-neutral or not exposing sessionFile

Addressed in f3d4c70: simplified the subpath row to list the public capabilities only.

jalehman · 2026-06-19T17:25:58Z

  return await activeLockRunner(() => appendSessionTranscriptMessageLocked(params));
 }

+export async function runSessionTranscriptAppendTransaction<T>(


If this is public interface it should have a docstring

Addressed in f3d4c70: added a docstring for runSessionTranscriptAppendTransaction.

jalehman · 2026-06-19T17:26:57Z

+}
+
+/**
+ * Builds the storage-neutral memory hit key for one session transcript.


Stop talking about storage-neutral. That's an implementation detail.

Addressed in f3d4c70: removed storage implementation language from the public SDK docstrings and kept them focused on behavior.

jalehman · 2026-06-19T17:36:42Z

@clawsweeper re-review

clawsweeper · 2026-06-19T17:36:45Z

🦞🧹
ClawSweeper re-review requested.

I asked ClawSweeper to review this item again.
Action: item re-review queued (workflow sweep.yml, event repository_dispatch).
Result: the existing ClawSweeper review comment will be edited in place when the review finishes.

jalehman · 2026-06-19T17:37:06Z

@clawsweeper re-review

…identity-target-api

jalehman · 2026-06-20T16:41:40Z

@clawsweeper re-review

clawsweeper · 2026-06-20T16:41:43Z

🦞🧹
ClawSweeper re-review requested.

I asked ClawSweeper to review this item again.
Action: item re-review queued (workflow sweep.yml, event repository_dispatch).
Result: the existing ClawSweeper review comment will be edited in place when the review finishes.

jalehman · 2026-06-20T17:42:25Z

@clawsweeper re-review

clawsweeper · 2026-06-20T17:42:28Z

🦞🧹
ClawSweeper re-review requested.

I asked ClawSweeper to review this item again.
Action: item re-review queued (workflow sweep.yml, event repository_dispatch).
Result: the existing ClawSweeper review comment will be edited in place when the review finishes.

openclaw-barnacle Bot added docs Improvements or additions to documentation scripts Repository scripts plugin: file-transfer size: L maintainer Maintainer-authored PR labels Jun 19, 2026

This was referenced Jun 19, 2026

refactor: add scoped transcript target writers #89262

Closed

refactor: add public transcript identity API #89261

Closed

Track core session/transcript SQLite migration via accessor seam #88838

Closed

jalehman marked this pull request as ready for review June 19, 2026 17:00

clawsweeper Bot added the proof: sufficient ClawSweeper judged the real behavior proof convincing. label Jun 19, 2026

jalehman commented Jun 19, 2026

View reviewed changes

jalehman force-pushed the clawdbot-d02.1.9.1.20-24/31b-sdk-transcript-identity-target-api branch 2 times, most recently from b562ce5 to cb9f329 Compare June 19, 2026 17:56

clawsweeper Bot added merge-risk: 🚨 session-state 🚨 May lose, corrupt, stale, or mis-associate session, agent, or context state. merge-risk: 🚨 security-boundary 🚨 May affect sandboxing, authorization, credentials, or sensitive data. labels Jun 19, 2026

This was referenced Jun 19, 2026

feat(plugin-sdk): export transcript.runtime subpath #80208

Closed

Expose transcript update emitter to plugins #80596

Closed

Merge branch 'main' into clawdbot-d02.1.9.1.20-24/31b-sdk-transcript-…

5300011

…identity-target-api

clawsweeper Bot added rating: 🦪 silver shellfish Thin PR readiness signal; proof, validation, or implementation needs work. and removed rating: 🦐 gold shrimp Decent PR readiness signal, but merge confidence is limited. labels Jun 19, 2026

This was referenced Jun 20, 2026

refactor: migrate bundled transcript target lookups #89911

Merged

refactor: migrate plugin transcript mirrors #89518

Merged

fix: keep transcript hit helpers lightweight

e93458f

test: align transcript sdk surface budget

1125743

jalehman merged commit 7a0d36f into main Jun 20, 2026
174 of 176 checks passed

jalehman deleted the clawdbot-d02.1.9.1.20-24/31b-sdk-transcript-identity-target-api branch June 20, 2026 21:01

github-actions Bot mentioned this pull request Jun 20, 2026

📡 Upstream Digest — 2026-06-20 21:06 UTC curtismercier/openclaw-mods#1117

Open

github-actions Bot pushed a commit to Desicool/openclaw that referenced this pull request Jun 21, 2026

refactor: add SDK transcript identity target API (openclaw#95030)

a1baed6

clawsweeper Bot mentioned this pull request Jun 21, 2026

Fix live orphan session transcript visibility #92261

Merged

jalehman mentioned this pull request Jun 23, 2026

refactor: add embedded run session target seam #90439

Merged

Azz1 mentioned this pull request Jun 24, 2026

Bug: Ollama tool_calls.function.arguments sent as object instead of string (regression in 2026.6.10) #96472

Closed

clawsweeper Bot mentioned this pull request Jun 26, 2026

Feature Request: Expose appendAssistantMessageToSessionTranscript in Plugin Runtime API #73274

Closed

github-actions Bot mentioned this pull request Jun 29, 2026

chore(openclaw): bump pinned version v2026.5.28 → v2026.6.10 Evans-Software-Solutions-Limited/axel-saas#120

Open

4 tasks


		Prefer `getSessionEntry(...)`, `listSessionEntries(...)`, `patchSessionEntry(...)`, or `upsertSessionEntry(...)` for session workflows. These helpers address sessions by agent/session identity so plugins do not depend on the legacy `sessions.json` storage shape. Use `preserveActivity: true` for metadata-only patches that should not refresh session activity, and `replaceEntry: true` only when the callback returns a complete entry and deleted fields must stay deleted.

		For transcript reads and writes, import `openclaw/plugin-sdk/session-transcript-runtime` and use `resolveSessionTranscriptIdentity(...)`, `resolveSessionTranscriptTarget(...)`, `readSessionTranscriptEvents(...)`, `appendSessionTranscriptMessageByIdentity(...)`, or `withSessionTranscriptWriteLock(...)` with `{ agentId, sessionKey, sessionId }`. These APIs return stable transcript identity and storage-neutral memory hit keys without exposing `sessionFile` as the identity. `sessionFile` is deprecated transitional compatibility for plugins that already own an active transcript file artifact during the pre-SQLite window and need scoped helpers to bind to that artifact; new code should use structured transcript identity before the SQLite storage flip.

Uh oh!

Uh oh!

Conversation

jalehman commented Jun 19, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

What

Why

Changes

Changes Walkthrough

Verification

Real behavior proof

Uh oh!

clawsweeper Bot commented Jun 19, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

jalehman commented Jun 19, 2026

Uh oh!

clawsweeper Bot commented Jun 19, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

jalehman commented Jun 19, 2026

Uh oh!

clawsweeper Bot commented Jun 19, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

jalehman commented Jun 19, 2026

Uh oh!

jalehman commented Jun 20, 2026

Uh oh!

clawsweeper Bot commented Jun 20, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

jalehman commented Jun 20, 2026

Uh oh!

clawsweeper Bot commented Jun 20, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

jalehman commented Jun 19, 2026 •

edited

Loading

clawsweeper Bot commented Jun 19, 2026 •

edited

Loading

clawsweeper Bot commented Jun 19, 2026 •

edited

Loading

clawsweeper Bot commented Jun 19, 2026 •

edited

Loading

clawsweeper Bot commented Jun 20, 2026 •

edited

Loading

clawsweeper Bot commented Jun 20, 2026 •

edited

Loading