From d0f0fe97a6f03540590c3704a08c4b2ad54d7681 Mon Sep 17 00:00:00 2001 From: Peter Steinberger Date: Sun, 3 May 2026 14:42:47 +0100 Subject: [PATCH] chore: move root tool configs --- .pre-commit-config.yaml | 2 +- knip.config.ts => config/knip.config.ts | 0 .../markdownlint-cli2.jsonc | 0 .shellcheckrc => config/shellcheckrc | 0 package.json | 8 ++++---- scripts/changed-lanes.mjs | 2 +- scripts/check-deadcode-unused-files.mjs | 2 +- scripts/check-duplicates.mjs | 2 +- test/scripts/changed-lanes.test.ts | 3 +++ 9 files changed, 11 insertions(+), 8 deletions(-) rename knip.config.ts => config/knip.config.ts (100%) rename .markdownlint-cli2.jsonc => config/markdownlint-cli2.jsonc (100%) rename .shellcheckrc => config/shellcheckrc (100%) diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml index c917f63d3ae..5f88e66d46c 100644 --- a/.pre-commit-config.yaml +++ b/.pre-commit-config.yaml @@ -25,7 +25,7 @@ repos: rev: v0.11.0 hooks: - id: shellcheck - args: [--severity=error] # Only fail on errors, not warnings/info + args: [--rcfile=config/shellcheckrc, --severity=error] # Only fail on errors, not warnings/info # Exclude vendor and scripts with embedded code or known issues exclude: "^(vendor/|scripts/e2e/)" diff --git a/knip.config.ts b/config/knip.config.ts similarity index 100% rename from knip.config.ts rename to config/knip.config.ts diff --git a/.markdownlint-cli2.jsonc b/config/markdownlint-cli2.jsonc similarity index 100% rename from .markdownlint-cli2.jsonc rename to config/markdownlint-cli2.jsonc diff --git a/.shellcheckrc b/config/shellcheckrc similarity index 100% rename from .shellcheckrc rename to config/shellcheckrc diff --git a/package.json b/package.json index f87b47a9cf7..2c7052d3a7b 100644 --- a/package.json +++ b/package.json @@ -1343,8 +1343,8 @@ "crabbox:stop": "node scripts/crabbox-wrapper.mjs stop", "crabbox:warmup": "node scripts/crabbox-wrapper.mjs warmup", "deadcode:ci": "pnpm deadcode:report:ci:knip && pnpm deadcode:report:ci:ts-unused", - "deadcode:dependencies": "pnpm --config.minimum-release-age=0 dlx knip@6.8.0 --config knip.config.ts --production --no-progress --reporter compact --dependencies --no-config-hints", - "deadcode:knip": "pnpm dlx knip --config knip.config.ts --production --no-progress --reporter compact --files --dependencies", + "deadcode:dependencies": "pnpm --config.minimum-release-age=0 dlx knip@6.8.0 --config config/knip.config.ts --production --no-progress --reporter compact --dependencies --no-config-hints", + "deadcode:knip": "pnpm dlx knip --config config/knip.config.ts --production --no-progress --reporter compact --files --dependencies", "deadcode:report": "pnpm deadcode:knip; pnpm deadcode:ts-prune; pnpm deadcode:ts-unused", "deadcode:report:ci:knip": "mkdir -p .artifacts/deadcode && pnpm deadcode:knip > .artifacts/deadcode/knip.txt 2>&1 || true", "deadcode:report:ci:ts-prune": "mkdir -p .artifacts/deadcode && pnpm deadcode:ts-prune > .artifacts/deadcode/ts-prune.txt 2>&1 || true", @@ -1403,8 +1403,8 @@ "lint:auth:pairing-account-scope": "node scripts/check-pairing-account-scope.mjs", "lint:core": "node scripts/run-oxlint.mjs --tsconfig tsconfig.oxlint.core.json src ui packages", "lint:docker-e2e": "node scripts/check-docker-e2e-boundaries.mjs", - "lint:docs": "pnpm dlx --config.resolution-mode=highest markdownlint-cli2", - "lint:docs:fix": "pnpm dlx --config.resolution-mode=highest markdownlint-cli2 --fix", + "lint:docs": "pnpm dlx --config.resolution-mode=highest markdownlint-cli2 --config config/markdownlint-cli2.jsonc", + "lint:docs:fix": "pnpm dlx --config.resolution-mode=highest markdownlint-cli2 --config config/markdownlint-cli2.jsonc --fix", "lint:extensions": "node scripts/run-oxlint.mjs --tsconfig tsconfig.oxlint.extensions.json extensions", "lint:extensions:bundled": "node scripts/run-bundled-extension-oxlint.mjs", "lint:extensions:channels": "node scripts/run-extension-channel-oxlint.mjs", diff --git a/scripts/changed-lanes.mjs b/scripts/changed-lanes.mjs index b011710f05e..62333a42a62 100644 --- a/scripts/changed-lanes.mjs +++ b/scripts/changed-lanes.mjs @@ -9,7 +9,7 @@ const APP_PATH_RE = /^(?:apps\/|Swabble\/|appcast\.xml$)/u; const EXTENSION_PATH_RE = /^extensions\/[^/]+(?:\/|$)/u; const CORE_PATH_RE = /^(?:src\/|ui\/|packages\/)/u; const TOOLING_PATH_RE = - /^(?:scripts\/|test\/vitest\/|\.github\/|\.vscode\/|deploy\/|git-hooks\/|Dockerfile\.sandbox(?:-(?:browser|common))?$|Makefile$|docker-setup\.sh$|setup-podman\.sh$|openclaw\.podman\.env$|skills\/pyproject\.toml$|vitest(?:\..+)?\.config\.ts$|tsconfig.*\.json$|\.dockerignore$|\.gitignore$|\.jscpd\.json$|\.npmignore$|\.pre-commit-config\.yaml$|\.swiftformat$|\.oxlint.*|\.oxfmt.*)/u; + /^(?:scripts\/|test\/vitest\/|\.github\/|\.vscode\/|config\/|deploy\/|git-hooks\/|Dockerfile\.sandbox(?:-(?:browser|common))?$|Makefile$|docker-setup\.sh$|setup-podman\.sh$|openclaw\.podman\.env$|skills\/pyproject\.toml$|vitest(?:\..+)?\.config\.ts$|tsconfig.*\.json$|\.dockerignore$|\.gitignore$|\.jscpd\.json$|\.npmignore$|\.pre-commit-config\.yaml$|\.swiftformat$|\.oxlint.*|\.oxfmt.*)/u; const ROOT_GLOBAL_PATH_RE = /^(?:package\.json$|pnpm-lock\.yaml$|pnpm-workspace\.yaml$|tsdown\.config\.ts$|vitest\.config\.ts$)/u; const LEGACY_ROOT_ASSET_PATH_RE = /^assets\//u; diff --git a/scripts/check-deadcode-unused-files.mjs b/scripts/check-deadcode-unused-files.mjs index d251acbd0f3..eb2fc11f7ad 100644 --- a/scripts/check-deadcode-unused-files.mjs +++ b/scripts/check-deadcode-unused-files.mjs @@ -9,7 +9,7 @@ import { const KNIP_VERSION = "6.8.0"; const KNIP_ARGS = [ "--config", - "knip.config.ts", + "config/knip.config.ts", "--production", "--no-progress", "--reporter", diff --git a/scripts/check-duplicates.mjs b/scripts/check-duplicates.mjs index cf0c3fc1835..c0c55667ac8 100644 --- a/scripts/check-duplicates.mjs +++ b/scripts/check-duplicates.mjs @@ -18,7 +18,7 @@ const targets = [ "security", "test", "openclaw.mjs", - "knip.config.ts", + "config/knip.config.ts", "tsdown.config.ts", "vitest.config.ts", ]; diff --git a/test/scripts/changed-lanes.test.ts b/test/scripts/changed-lanes.test.ts index b02a677f832..bead8dc0539 100644 --- a/test/scripts/changed-lanes.test.ts +++ b/test/scripts/changed-lanes.test.ts @@ -328,6 +328,9 @@ describe("scripts/changed-lanes", () => { ".pre-commit-config.yaml", ".swiftformat", "Makefile", + "config/knip.config.ts", + "config/markdownlint-cli2.jsonc", + "config/shellcheckrc", "deploy/fly.private.toml", "docker-setup.sh", "openclaw.podman.env",