From ea3bb9282ca18bbbba37017771a60456837bf909 Mon Sep 17 00:00:00 2001 From: Vincent Koc Date: Mon, 25 May 2026 03:33:32 +0200 Subject: [PATCH] fix(scripts): remove stale deadcode allowlist entries --- CHANGELOG.md | 1 + scripts/deadcode-unused-files.allowlist.mjs | 33 +-------------------- 2 files changed, 2 insertions(+), 32 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 11e22eda154..290d9cfd431 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -8,6 +8,7 @@ Docs: https://docs.openclaw.ai ### Fixes +- Scripts: remove stale Knip unused-file allowlist entries so the dead-code gate fails only on current findings. - Config: keep benign legacy metadata write anomalies out of default doctor and config command output while preserving explicit anomaly logging for diagnostics. - Codex: log when implicit app-server `never` approvals are promoted for OpenClaw tool policy, including whether the trigger was a `before_tool_call` hook or trusted tool policy. - Google Vertex: support production ADC modes such as Workload Identity Federation, service-account credentials, and metadata-server ADC for the native Vertex transport. (#83971) Thanks @damianFelixPago. diff --git a/scripts/deadcode-unused-files.allowlist.mjs b/scripts/deadcode-unused-files.allowlist.mjs index c56a6f64a5a..e725c4be569 100644 --- a/scripts/deadcode-unused-files.allowlist.mjs +++ b/scripts/deadcode-unused-files.allowlist.mjs @@ -1,38 +1,7 @@ // Intentional Knip unused-file findings. These are dynamic entrypoints, // generated/build inputs, manifest-discovered plugin surfaces, live-test // helpers, or package bridge files that static production scanning cannot see. -export const KNIP_UNUSED_FILE_ALLOWLIST = [ - "extensions/acpx/src/runtime-internals/error-format.mjs", - "extensions/acpx/src/runtime-internals/mcp-command-line.mjs", - "extensions/acpx/src/runtime-internals/mcp-proxy.mjs", - "extensions/canvas/src/host/a2ui-app/bootstrap.js", - "extensions/canvas/src/host/a2ui-app/rolldown.config.mjs", - "extensions/diffs/src/viewer-client.ts", - "extensions/diffs/src/viewer-payload.ts", - "extensions/matrix/src/plugin-entry.runtime.js", - "extensions/memory-core/src/memory-tool-manager-mock.ts", - "src/agents/subagent-registry.runtime.ts", - "src/auto-reply/inbound.group-require-mention-test-plugins.ts", - "src/auto-reply/reply/get-reply.test-loader.ts", - "src/cli/daemon-cli-compat.ts", - "src/commands/doctor/shared/deprecation-compat.ts", - "src/config/doc-baseline.runtime.ts", - "src/config/doc-baseline.ts", - "src/gateway/gateway-cli-backend.live-helpers.ts", - "src/gateway/gateway-cli-backend.live-probe-helpers.ts", - "src/gateway/gateway-codex-harness.live-helpers.ts", - "src/infra/changelog-unreleased.ts", - "src/mcp/openclaw-tools-serve.ts", - "src/mcp/plugin-tools-handlers.ts", - "src/mcp/plugin-tools-serve.ts", - "src/mcp/tools-stdio-server.ts", - "src/plugins/build-smoke-entry.ts", - "src/plugins/contracts/host-hook-fixture.ts", - "src/plugins/contracts/rootdir-boundary-canary.ts", - "src/plugins/contracts/tts-contract-suites.ts", - "src/plugins/runtime-sidecar-paths-baseline.ts", - "src/tasks/task-registry-control.runtime.ts", -]; +export const KNIP_UNUSED_FILE_ALLOWLIST = []; // Knip can disagree across supported local/CI platforms for files that are // only reachable through test-only import graphs. Ignore these when reported,