From ccd5ca43c2891119dfc22c2539c6faf65bc6c0d4 Mon Sep 17 00:00:00 2001 From: Agents Agent Date: Wed, 1 Jul 2026 12:50:23 +0000 Subject: [PATCH] fix(node-ci): exclude vendored tree via config, not deletion Replace the rm of .github-workflows with reliance on default excludes in consuming projects (shared eslint/ktlint ignores, scaffolded .gitignore honoured by antfu-eslint, and .prettierignore). Keeps the CI checkout immutable and lets local runs and templates benefit from the same exclusion, instead of a workflow-only side effect. --- .github/workflows/node-ci.yml | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/.github/workflows/node-ci.yml b/.github/workflows/node-ci.yml index 8b0a40c..c6500f0 100644 --- a/.github/workflows/node-ci.yml +++ b/.github/workflows/node-ci.yml @@ -105,11 +105,10 @@ jobs: install-command: ${{ inputs.install-command }} github-packages-token: ${{ secrets.packages-token || github.token }} - # The vendored actions repo is only needed for setup-node above. Remove it before any - # quality command runs so workspace-wide linters/formatters (e.g. `eslint .`, `prettier - # --check .`) only ever see the consuming project's own files, never this CI machinery. - - name: Remove vendored CI actions before quality checks - run: rm -rf .github-workflows + # The vendored actions tree lives at .github-workflows/. Consuming projects exclude it from + # linting/formatting by default (it is in the shared eslint/ktlint config ignores, the + # scaffolded .gitignore — which antfu-eslint honours — and .prettierignore), so quality + # commands see only the project's own files without this workflow mutating the checkout. - name: Package check if: ${{ inputs.package-check-command != '' }}