refactor(channels): keep status contract net-negative

RomneyDa · RomneyDa · commit dd5acf11fab3 · 2026-07-09T19:18:35.000-07:00
diff --git a/docs/.generated/plugin-sdk-api-baseline.sha256 b/docs/.generated/plugin-sdk-api-baseline.sha256
@@ -1,2 +1,2 @@
-14ff327d9f8d7d823d2391b857441a7b2e7092db4ae07b4c3b93f5972969ee53  plugin-sdk-api-baseline.json
-533532f5fc00832ca36e708031923ad0a9ec246e178e0027ab5eb235960ff2a8  plugin-sdk-api-baseline.jsonl
+71725394b9addf613ecf74faf08808ae6a68a3b7c7bf82222f9ba24e0ddc6d98  plugin-sdk-api-baseline.json
+3eaab17c6854d67dda2235d501687200ac38e106890c33d7e67e0974f3807b01  plugin-sdk-api-baseline.jsonl
diff --git a/docs/plugins/sdk-channel-plugins.md b/docs/plugins/sdk-channel-plugins.md
@@ -377,19 +377,8 @@ status adapter, and channel secret target metadata needed for those
 summaries. Do not start clients, listeners, or transport runtimes from the
 setup entry.
 
-### Status contracts
-
-Use `ChannelAccountStatus` for credential-free snapshots returned by status
-commands and Gateway clients. Plugin status hooks, `defaultRuntime`, and
-`getStatus`/`setStatus` use `ChannelAccountSnapshotInput` because those values
-are contributions that core projects through the public status schema before
-delivery.
-
-`ChannelAccountSnapshot` remains as a deprecated alias for contribution input
-during the migration window. New plugins should import the explicit type they
-need from `openclaw/plugin-sdk/channel-contract` or
-`openclaw/plugin-sdk/status-helpers`; do not put raw credential values in
-`ChannelAccountStatus`.
+Status output uses `ChannelAccountStatus`; contributions use
+`ChannelAccountSnapshotInput` (`ChannelAccountSnapshot` is its deprecated alias).
 
 Keep the main channel entry import path narrow too. Discovery can evaluate
 the entry and the channel plugin module to register capabilities without
diff --git a/docs/plugins/sdk-migration.md b/docs/plugins/sdk-migration.md
@@ -88,6 +88,9 @@ If a manifest field is still accepted, keep using it until docs and
 diagnostics say otherwise. New code should prefer the documented replacement;
 existing plugins should not break during ordinary minor releases.
 
+For channel status, use `ChannelAccountStatus` for output and
+`ChannelAccountSnapshotInput` for input; its old alias is deprecated.
+
 Audit the current migration queue with `pnpm plugins:boundary-report`:
 
 | Flag                                                    | Effect                                                                         |
@@ -370,21 +373,6 @@ SDK.
   </Step>
 </Steps>
 
-## Channel status type migration
-
-Channel status output and plugin/runtime contributions now use separate types:
-
-- Use `ChannelAccountStatus` for credential-free snapshots returned by status
-  commands and Gateway clients.
-- Use `ChannelAccountSnapshotInput` for `buildAccountSnapshot`,
-  `defaultRuntime`, and channel runtime `getStatus`/`setStatus` values.
-- `ChannelAccountSnapshot` is a deprecated compatibility alias for
-  `ChannelAccountSnapshotInput`. Migrate to an explicit type before the next
-  major-version cleanup.
-
-Both modern types are available from `plugin-sdk/channel-contract`; status
-helper consumers can also import them from `plugin-sdk/status-helpers`.
-
 ## Import path reference
 
 <Accordion title="Common import path table">
diff --git a/packages/gateway-protocol/src/channels.schema.test.ts b/packages/gateway-protocol/src/channels.schema.test.ts
@@ -38,58 +38,38 @@ describe("ChannelsStatusResultSchema", () => {
   const validate = Compile(ChannelsStatusResultSchema);
 
   it("accepts gateway event-loop diagnostics emitted by channels.status", () => {
-    expect(
-      validate.Check({
-        ts: Date.now(),
-        channelOrder: ["discord"],
-        channelLabels: { discord: "Discord" },
-        channels: { discord: { configured: true } },
-        channelAccounts: {
-          discord: [
-            {
-              accountId: "default",
-              enabled: true,
-              configured: true,
-              running: true,
-              connected: false,
-              healthState: "stale-socket",
-            },
-          ],
-        },
-        channelDefaultAccountId: { discord: "default" },
-        partial: true,
-        warnings: ["discord:default probe timed out after 1000ms"],
-        eventLoop: {
-          degraded: true,
-          reasons: ["event_loop_delay", "cpu"],
-          intervalMs: 62_000,
-          delayP99Ms: 1_250.5,
-          delayMaxMs: 62_000,
-          utilization: 0.98,
-          cpuCoreRatio: 1.2,
-        },
-      }),
-    ).toBe(true);
-  });
-
-  it("rejects credential-shaped and undeclared account status fields", () => {
     const result = {
       ts: Date.now(),
-      channelOrder: ["line"],
-      channelLabels: { line: "LINE" },
-      channels: { line: { configured: true } },
+      channelOrder: ["discord"],
+      channelLabels: { discord: "Discord" },
+      channels: { discord: { configured: true } },
       channelAccounts: {
-        line: [
+        discord: [
           {
             accountId: "default",
+            enabled: true,
             configured: true,
-            channelAccessToken: "secret-token",
+            running: true,
+            connected: false,
+            healthState: "stale-socket",
           },
         ],
       },
-      channelDefaultAccountId: { line: "default" },
+      channelDefaultAccountId: { discord: "default" },
+      partial: true,
+      warnings: ["discord:default probe timed out after 1000ms"],
+      eventLoop: {
+        degraded: true,
+        reasons: ["event_loop_delay", "cpu"],
+        intervalMs: 62_000,
+        delayP99Ms: 1_250.5,
+        delayMaxMs: 62_000,
+        utilization: 0.98,
+        cpuCoreRatio: 1.2,
+      },
     };
-
+    expect(validate.Check(result)).toBe(true);
+    Object.assign(result.channelAccounts.discord[0], { channelAccessToken: "secret-token" });
     expect(validate.Check(result)).toBe(false);
   });
 });
diff --git a/scripts/plugin-sdk-surface-report.mjs b/scripts/plugin-sdk-surface-report.mjs
@@ -198,7 +198,7 @@ export function readPluginSdkSurfaceBudgets(env = process.env) {
     ),
     publicExports: readPluginSdkSurfaceBudgetEnv(
       "OPENCLAW_PLUGIN_SDK_MAX_PUBLIC_EXPORTS",
-      10471,
+      10481,
       env,
     ),
     publicFunctionExports: readPluginSdkSurfaceBudgetEnv(
@@ -208,7 +208,7 @@ export function readPluginSdkSurfaceBudgets(env = process.env) {
     ),
     publicDeprecatedExports: readPluginSdkSurfaceBudgetEnv(
       "OPENCLAW_PLUGIN_SDK_MAX_PUBLIC_DEPRECATED_EXPORTS",
-      3265,
+      3269,
       env,
     ),
     publicWildcardReexports: readPluginSdkSurfaceBudgetEnv(
diff --git a/src/channels/plugins/status.test.ts b/src/channels/plugins/status.test.ts
diff --git a/src/plugins/contracts/plugin-sdk-subpaths.test.ts b/src/plugins/contracts/plugin-sdk-subpaths.test.ts
@@ -6,9 +6,7 @@ import type {
   BaseProbeResult as ContractBaseProbeResult,
   BaseTokenResolution as ContractBaseTokenResolution,
   ChannelAgentTool as ContractChannelAgentTool,
-  ChannelAccountStatus as ContractChannelAccountStatus,
   ChannelAccountSnapshot as ContractChannelAccountSnapshot,
-  ChannelAccountSnapshotInput as ContractChannelAccountSnapshotInput,
   ChannelGroupContext as ContractChannelGroupContext,
   ChannelMessageActionAdapter as ContractChannelMessageActionAdapter,
   ChannelMessageActionContext as ContractChannelMessageActionContext,
@@ -34,9 +32,7 @@ import type {
   BaseProbeResult,
   BaseTokenResolution,
   ChannelAgentTool,
-  ChannelAccountStatus,
   ChannelAccountSnapshot,
-  ChannelAccountSnapshotInput,
   ChannelGroupContext,
   ChannelMessageActionAdapter,
   ChannelMessageActionName,
@@ -1329,9 +1325,7 @@ describe("plugin-sdk subpath exports", () => {
     expectTypeOf<ContractBaseProbeResult>().toMatchTypeOf<BaseProbeResult>();
     expectTypeOf<ContractBaseTokenResolution>().toMatchTypeOf<BaseTokenResolution>();
     expectTypeOf<ContractChannelAgentTool>().toMatchTypeOf<ChannelAgentTool>();
-    expectTypeOf<ContractChannelAccountStatus>().toMatchTypeOf<ChannelAccountStatus>();
     expectTypeOf<ContractChannelAccountSnapshot>().toMatchTypeOf<ChannelAccountSnapshot>();
-    expectTypeOf<ContractChannelAccountSnapshotInput>().toMatchTypeOf<ChannelAccountSnapshotInput>();
     expectTypeOf<ContractChannelGroupContext>().toMatchTypeOf<ChannelGroupContext>();
     expectTypeOf<ContractChannelMessageActionAdapter>().toMatchTypeOf<ChannelMessageActionAdapter>();
     expectTypeOf<ContractChannelMessageActionContext>().toMatchTypeOf<ChannelMessageActionContext>();
