From 875669e38ec87ca9ecc2d10e1ae32f88cf7da828 Mon Sep 17 00:00:00 2001 From: Vincent Koc Date: Tue, 16 Jun 2026 20:04:22 +0200 Subject: [PATCH] fix(ci): keep approval type contracts off runtime imports --- src/infra/approval-handler-runtime-types.ts | 2 +- .../heartbeat-runner.returns-default-unset.test.ts | 10 ++++++---- src/scripts/test-projects.test.ts | 3 ++- 3 files changed, 9 insertions(+), 6 deletions(-) diff --git a/src/infra/approval-handler-runtime-types.ts b/src/infra/approval-handler-runtime-types.ts index 92b4e9a20a4d..3dff871efd5b 100644 --- a/src/infra/approval-handler-runtime-types.ts +++ b/src/infra/approval-handler-runtime-types.ts @@ -1,7 +1,7 @@ // Defines channel-native approval handler runtime types. import type { OpenClawConfig } from "../config/types.openclaw.js"; import type { ChannelApprovalNativePlannedTarget } from "./approval-native-delivery.js"; -import type { PreparedChannelNativeApprovalTarget } from "./approval-native-runtime.js"; +import type { PreparedChannelNativeApprovalTarget } from "./approval-native-runtime-types.js"; import type { ChannelApprovalKind } from "./approval-types.js"; import type { ExpiredApprovalView, diff --git a/src/infra/heartbeat-runner.returns-default-unset.test.ts b/src/infra/heartbeat-runner.returns-default-unset.test.ts index 9341fe5569b8..b5937d3b1fd8 100644 --- a/src/infra/heartbeat-runner.returns-default-unset.test.ts +++ b/src/infra/heartbeat-runner.returns-default-unset.test.ts @@ -14,7 +14,11 @@ import { } from "../config/sessions.js"; import { getActivePluginRegistry, setActivePluginRegistry } from "../plugins/runtime.js"; import { buildAgentPeerSessionKey } from "../routing/session-key.js"; -import { createOutboundTestPlugin, createTestRegistry } from "../test-utils/channel-plugins.js"; +import { + createDirectOutboundTestAdapter, + createOutboundTestPlugin, + createTestRegistry, +} from "../test-utils/channel-plugins.js"; import { typedCases } from "../test-utils/typed-cases.js"; import { type HeartbeatDeps, @@ -292,9 +296,7 @@ beforeAll(async () => { const discordPlugin = createOutboundTestPlugin({ id: "discord", - outbound: { - deliveryMode: "direct", - }, + outbound: createDirectOutboundTestAdapter({ channel: "discord" }), }); testRegistry = createTestRegistry([ diff --git a/src/scripts/test-projects.test.ts b/src/scripts/test-projects.test.ts index 7792edb622d2..2508cd272d71 100644 --- a/src/scripts/test-projects.test.ts +++ b/src/scripts/test-projects.test.ts @@ -858,7 +858,7 @@ describe("test-projects args", () => { { config: "test/vitest/vitest.unit-fast.config.ts", forwardedArgs: [], - includePatterns: ["src/install-sh-version.test.ts"], + includePatterns: ["src/install-sh-version.test.ts", "test/scripts/android-version.test.ts"], watchMode: false, }, { @@ -874,6 +874,7 @@ describe("test-projects args", () => { "src/scripts/docs-link-audit.test.ts", "src/scripts/sync-plugin-versions.test.ts", "test/helpers/temp-dir.test.ts", + "test/scripts/android-pin-version.test.ts", "test/scripts/ios-configure-signing.test.ts", "test/scripts/ios-pin-version.test.ts", "test/scripts/ios-team-id.test.ts",