PR #74769: feat(gateway): add artifact RPCs

• Repository: openclaw/openclaw
• Author: volcano303
• State: open | merged: False
• Link: https://github.com/openclaw/openclaw/pull/74769

Description (problem / solution / changelog)

Summary

Describe the problem and fix in 2–5 bullets:

• Problem: Gateway clients have no supported artifact RPC surface, so SDK/app clients must scrape chat history and infer downloadable media themselves.
• Why it matters: External app clients need stable artifact discovery and download semantics for generated or attached media without depending on transcript internals.
• What changed: Added artifacts.list, artifacts.get, and artifacts.download gateway RPCs over session transcript artifact blocks, with protocol schemas, method discovery, read-scope authorization, and typed unsupported/not-found errors.
• What did NOT change (scope boundary): This does not add a new artifact store or expose arbitrary local file paths; unsupported artifact sources return explicit unsupported errors.

Change Type (select all)

• Bug fix
• Feature
• Refactor required for the fix
• Docs
• Security hardening
• Chore/infra

Scope (select all touched areas)

• Gateway / orchestration
• Skills / tool execution
• Auth / tokens
• Memory / storage
• Integrations
• API / contracts
• UI / DX
• CI/CD / infra

Linked Issue/PR

• Closes #74706
• Related #74704
• This PR fixes a bug or regression

Root Cause (if applicable)

• Root cause: Artifacts existed only as transcript content blocks, with no supported gateway RPC projection for SDK/app clients.
• Missing detection / guardrail: There was no focused gateway test asserting artifact list/get/download behavior or typed missing/unsupported errors.
• Contributing context (if known): The SDK happy path needs artifact access without requiring clients to parse chat history payload details.

Regression Test Plan (if applicable)

• Coverage level that should have caught this:
  • Unit test
  • Seam / integration test
  • End-to-end test
  • Existing coverage already sufficient
• Target test or file: src/gateway/server-methods/artifacts.test.ts
• Scenario the test should lock in: Session transcript media blocks are listed as stable artifact summaries, can be fetched by ID, can be downloaded as base64 bytes, and missing/unsupported query paths return typed errors.
• Why this is the smallest reliable guardrail: The new behavior is a gateway RPC projection over existing transcript state, so handler-level tests cover the API contract directly.
• Existing test that already covers this (if any): N/A
• If no new test is added, why not: N/A

User-visible / Behavior Changes

SDK and app clients can call artifacts.list, artifacts.get, and artifacts.download with sessionKey, runId, or taskId query context. Inline transcript artifacts download as base64 bytes; URL-backed artifacts return a URL; unsupported sources return a typed error.

Diagram (if applicable)

Before:
SDK client -> chat.history -> parse transcript media internals

After:
SDK client -> artifacts.list/get/download -> normalized artifact contract

Security Impact (required)

• New permissions/capabilities? (No)
• Secrets/tokens handling changed? (No)
• New/changed network calls? (No)
• Command/tool execution surface changed? (No)
• Data access scope changed? (Yes)
• If any Yes, explain risk + mitigation: Artifact RPCs expose only artifacts already present in authorized chat/session transcript data and are gated by existing operator.read scope. They do not read arbitrary local file paths.

Repro + Verification

Environment

• OS: Linux
• Runtime/container: Node 22.14.0, pnpm
• Model/provider: N/A
• Integration/channel (if any): Gateway RPC
• Relevant config (redacted): Default test config

Steps

1. Seed a session transcript message with an image/file content block.
2. Call artifacts.list with the session key.
3. Call artifacts.get and artifacts.download with the returned artifact ID.
4. Call with missing/unknown artifact context.

Expected

• Artifact summaries are stable and omit raw bytes.
• artifacts.download returns base64 bytes or URL depending on source.
• Missing/unsupported cases return typed errors.

Actual

• Matches expected after this PR.

Evidence

Attach at least one:

• Failing test/log before + passing after
• Trace/log snippets
• Screenshot/recording
• Perf numbers (if relevant)

Human Verification (required)

What you personally verified (not just CI), and how:

• Verified scenarios: artifacts.list, artifacts.get, artifacts.download, runId-to-session lookup, missing query scope, missing artifact ID, protocol schema compilation, method-scope discovery.
• Edge cases checked: Inline base64 artifact bytes are omitted from summaries and returned only from download.
• What you did not verify: A live SDK/OpenMeow client against a running gateway.

Review Conversations

• I replied to or resolved every bot review conversation I addressed in this PR.
• I left unresolved only the conversations that still need reviewer or maintainer judgment.

If a bot review conversation is addressed by this PR, resolve that conversation yourself. Do not leave bot review conversation cleanup for maintainers.

Compatibility / Migration

• Backward compatible? (Yes)
• Config/env changes? (No)
• Migration needed? (No)
• If yes, exact upgrade steps: N/A

Risks and Mitigations

• Risk: The first artifact source is intentionally conservative and may return artifact_download_unsupported for transcript blocks that reference local-only paths.
  • Mitigation: This avoids exposing arbitrary local filesystem paths while establishing the stable RPC contract for inline and URL-backed artifacts.

Changed files

• src/gateway/method-scopes.ts (modified, +3/-0)
• src/gateway/protocol/index.ts (modified, +24/-0)
• src/gateway/protocol/schema.ts (modified, +1/-0)
• src/gateway/protocol/schema/artifacts.ts (added, +73/-0)
• src/gateway/protocol/schema/protocol-schemas.ts (modified, +16/-0)
• src/gateway/protocol/schema/types.ts (modified, +7/-0)
• src/gateway/server-methods-list.ts (modified, +3/-0)
• src/gateway/server-methods.ts (modified, +2/-0)
• src/gateway/server-methods/artifacts.test.ts (added, +180/-0)
• src/gateway/server-methods/artifacts.ts (added, +375/-0)

PR #74847: feat(gateway): add SDK task ledger RPCs

• Repository: openclaw/openclaw
• Author: tmimmanuel
• State: open | merged: False
• Link: https://github.com/openclaw/openclaw/pull/74847

Description (problem / solution / changelog)

Summary

• Problem: @openclaw/sdk exposed oc.tasks.*, but list, get, and cancel still threw unsupported errors because Gateway had no SDK-facing task ledger RPCs.
• Why it matters: OpenMeow and other app clients need stable task visibility for detached/background work without reading internal logs or CLI-only surfaces.
• What changed: Added tasks.list, tasks.get, and tasks.cancel Gateway RPCs, SDK methods/types, protocol schemas, scopes, docs, and regression coverage.
• What did NOT change (scope boundary): This does not add tool invocation, artifact APIs, environment provisioning, or new task runtime semantics.

Change Type (select all)

• Bug fix
• Feature
• Refactor required for the fix
• Docs
• Security hardening
• Chore/infra

Scope (select all touched areas)

• Gateway / orchestration
• Skills / tool execution
• Auth / tokens
• Memory / storage
• Integrations
• API / contracts
• UI / DX
• CI/CD / infra

Linked Issue/PR

• Closes #74707
• Related #74704
• Related #74705
• Related #74706
• Related #74708
• Related #74709
• Related https://github.com/OpenCoven/open-meow-sdk
• This PR fixes a bug or regression

Root Cause (if applicable)

• Root cause: Gateway already had a durable task registry, but no public RPC contract or SDK bridge exposed it to external app clients.
• Missing detection / guardrail: SDK coverage only locked in unsupported task errors instead of the app-client task ledger contract.
• Contributing context (if known): The OpenMeow gap map calls out tasks.list, tasks.get, and tasks.cancel as missing basics.

Regression Test Plan (if applicable)

• Coverage level that should have caught this: Unit test and seam / integration test.
• Target test or file: src/gateway/server-methods/tasks.test.ts, src/gateway/protocol/index.test.ts, packages/sdk/src/index.test.ts, packages/sdk/src/index.e2e.test.ts.
• Scenario the test should lock in: SDK task methods route to Gateway RPCs, task statuses are app-facing and stable, filters work, scopes are classified, and cancel returns a clear result contract.
• Why this is the smallest reliable guardrail: It exercises the protocol and SDK seams without requiring a live detached task runtime.
• Existing test that already covers this (if any): Existing task registry tests cover the underlying task lifecycle.
• If no new test is added, why not: N/A.

User-visible / Behavior Changes

oc.tasks.list(), oc.tasks.get(taskId), and oc.tasks.cancel(taskId) now call Gateway task ledger RPCs instead of throwing unsupported errors.

Diagram (if applicable)

Before:
[external app] -> [@openclaw/sdk oc.tasks.*] -> [unsupported error]

After:
[external app] -> [@openclaw/sdk oc.tasks.*] -> [Gateway tasks.* RPC] -> [durable task registry]

Security Impact (required)

• New permissions/capabilities? (Yes/No) Yes
• Secrets/tokens handling changed? (Yes/No) No
• New/changed network calls? (Yes/No) No
• Command/tool execution surface changed? (Yes/No) No
• Data access scope changed? (Yes/No) Yes
• If any Yes, explain risk + mitigation: tasks.list and tasks.get require operator.read; tasks.cancel requires operator.write. The RPCs expose task metadata from the existing durable task registry and do not expose secrets.

Repro + Verification

Environment

• OS: Ubuntu/Linux local workspace
• Runtime/container: Node 22.22.0 via nvm, pnpm 10.33.2
• Model/provider: N/A
• Integration/channel (if any): Gateway WebSocket RPC / @openclaw/sdk
• Relevant config (redacted): N/A

Steps

1. On current main, call oc.tasks.list(), oc.tasks.get("task-id"), or oc.tasks.cancel("task-id").
2. Observe the SDK unsupported error.
3. Apply this PR.
4. Call the same SDK methods against a Gateway with task records.

Expected

• SDK task methods route to Gateway task ledger RPCs and return stable task summaries or cancel results.

Actual

• Before this PR, SDK task methods throw unsupported errors.

Evidence

• Failing test/log before + passing after
• Trace/log snippets
• Screenshot/recording
• Perf numbers (if relevant)

Lint evidence:

• git diff --check HEAD~1 HEAD
• targeted oxlint on changed TS files
• markdownlint-cli2 docs/concepts/openclaw-sdk.md

Review Conversations

• I replied to or resolved every bot review conversation I addressed in this PR.
• I left unresolved only the conversations that still need reviewer or maintainer judgment.

Compatibility / Migration

• Backward compatible? (Yes/No) Yes
• Config/env changes? (Yes/No) No
• Migration needed? (Yes/No) No
• If yes, exact upgrade steps: N/A

Risks and Mitigations

• Risk: SDK-facing task statuses intentionally differ from internal registry statuses.
• Mitigation: Tests lock in the public mapping, including succeeded to completed and lost to failed.

Changed files

• apps/macos/Sources/OpenClawProtocol/GatewayModels.swift (modified, +214/-0)
• apps/shared/OpenClawKit/Sources/OpenClawProtocol/GatewayModels.swift (modified, +214/-0)
• docs/concepts/openclaw-sdk.md (modified, +12/-5)
• packages/sdk/src/client.ts (modified, +13/-9)
• packages/sdk/src/index.e2e.test.ts (modified, +54/-0)
• packages/sdk/src/index.test.ts (modified, +64/-9)
• packages/sdk/src/index.ts (modified, +6/-0)
• packages/sdk/src/types.ts (modified, +50/-0)
• src/gateway/method-scopes.test.ts (modified, +3/-0)
• src/gateway/method-scopes.ts (modified, +3/-0)
• src/gateway/protocol/index.test.ts (modified, +21/-0)
• src/gateway/protocol/index.ts (modified, +31/-0)
• src/gateway/protocol/schema.ts (modified, +1/-0)
• src/gateway/protocol/schema/protocol-schemas.ts (modified, +16/-0)
• src/gateway/protocol/schema/tasks.ts (added, +91/-0)
• src/gateway/protocol/schema/types.ts (modified, +7/-0)
• src/gateway/server-methods-list.ts (modified, +3/-0)
• src/gateway/server-methods.ts (modified, +2/-0)
• src/gateway/server-methods/tasks.test.ts (added, +167/-0)
• src/gateway/server-methods/tasks.ts (added, +205/-0)
• src/tasks/detached-task-runtime-contract.ts (modified, +1/-0)
• src/tasks/task-registry.ts (modified, +5/-2)