Merge branch 'main' into unmaintained-repo-comment

Author: ErikSchierboom

.github/workflows/check-no-important-files-changed.yml

@@ -20,39 +20,79 @@ jobs:
     name: No important files changed
     runs-on: ubuntu-22.04
     steps:
-      - name: Checkout code
-        uses: actions/checkout@d632683dd7b4114ad314bca15554477dd762a938
-        with:
-          repository: ${{ inputs.repository }}
-          ref: ${{ inputs.ref }}
-
       - name: Check if important files changed
         id: check
-        run: |
-          set -exo pipefail
+        uses: actions/github-script@60a0d83039c74a4aee543508d2ffcb1c3799cdea
+        with:
+          script: |
+            const { owner, repo } = context.repo;
+            const pull_number = context.issue.number
+
+            return github.rest.pulls
+              .listFiles({ owner, repo, pull_number })
+              .then(async ({ data: files }) => {
+                const filenames = files
+                  .filter((file) => file.status !== "added")
+                  .map((file) => file.filename);
+                console.log(`Files in PR: ${filenames}`);
+
+                // Cache the parsed exercise config file's invalidator files
+                let exerciseInvalidatorFiles = {};
+
+                for (const filename of filenames) {
+                  const match =
+                    /^exercises\/(?<type>practice|concept)\/(?<slug>[^\/]+)\/(?<path>.+)$/i.exec(
+                      filename
+                    );
+                  if (match?.groups === undefined) {
+                    console.log(`${filename}: skipped (can't invalidate test results)`);
+                    return false;
+                  }
+
+                  const { type, slug, path } = match.groups;
+                  const configFile = `exercises/${type}/${slug}/.meta/config.json`;
+
+                  const parseInvalidatorFiles = (path) => {
+                    return github.rest.repos
+                      .getContent({ owner, repo, path })
+                      .then(({ data: { content } }) =>
+                        JSON.parse(Buffer.from(content, "base64").toString())
+                      )
+                      .then((config) => {
+                        const files = config.files;
+                        if (files === undefined) {
+                          return [];
+                        }
 
-          git remote set-branches origin main
-          git fetch --depth 1 origin main
+                        return [].concat(
+                          files["invalidator"] || [],
+                          files["test"] || [],
+                          files["editor"] || []
+                        );
+                      })
+                      .catch((err) => []);
+                  };
 
-          git diff --diff-filter=M --name-only origin/main
+                  exerciseInvalidatorFiles[slug] ||= await parseInvalidatorFiles(
+                    configFile
+                  );
+                  const invalidatesTests = exerciseInvalidatorFiles[slug].includes(path);
 
-          for changed_file in $(git diff --diff-filter=M --name-only origin/main); do
-            slug="$(echo "${changed_file}" | sed --regexp-extended 's#exercises/[^/]+/([^/]+)/.*#\1#' )"
-            path_before_slug="$(echo "${changed_file}" | sed --regexp-extended "s#(.*)/${slug}/.*#\\1#" )"
-            path_after_slug="$( echo "${changed_file}" | sed --regexp-extended "s#.*/${slug}/(.*)#\\1#" )"
-            config_json_file="${path_before_slug}/${slug}/.meta/config.json"
+                  if (invalidatesTests) {
+                    console.log(`${filename}: invalidates test results`);
+                  } else {
+                    console.log(`${filename}: does not invalidate test results`);
+                  }
 
-            if ! [ -f "${config_json_file}" ]; then
-              # cannot determine if important files changed without .meta/config.json
-              continue
-            fi
+                  return invalidatesTests;
+                }
 
-            changed=$(jq --arg path "${path_after_slug}" '[.files.test, .files.invalidator, .files.editor] | flatten | index($path) != null' "${config_json_file}")
-            echo "important_files_changed=${changed}" >> "$GITHUB_OUTPUT"
-          done
+                return false;
+              })
+              .catch((err) => false);
 
       - name: Suggest to add [no important files changed]
-        if: steps.check.outputs.important_files_changed == 'true'
+        if: steps.check.outputs.result == 'true'
         uses: actions/github-script@60a0d83039c74a4aee543508d2ffcb1c3799cdea
         with:
           script: |

.github/workflows/configlet.yml

@@ -22,7 +22,7 @@ jobs:
     runs-on: ubuntu-22.04
     steps:
       - name: Checkout code
-        uses: actions/checkout@d632683dd7b4114ad314bca15554477dd762a938
+        uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683
 
       - name: Fetch configlet
         uses: exercism/github-actions/configlet-ci@main

.github/workflows/docker-build-push-image.yml

@@ -62,7 +62,7 @@ jobs:
 
     steps:
       - name: Checkout code
-        uses: actions/checkout@d632683dd7b4114ad314bca15554477dd762a938
+        uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683
         with:
           # Never deploy from non-main branches
           ref: main
@@ -78,7 +78,7 @@ jobs:
 
       - name: Set up Docker Buildx
         if: steps.dockerfile-exists.outputs.result == 'true'
-        uses: docker/setup-buildx-action@988b5a0280414f521da01fcc63a27aeeb4b104db
+        uses: docker/setup-buildx-action@c47758b77c9736f4b2ef4073d4d51994fabfe349
 
       - name: Login to DockerHub
         uses: docker/login-action@9780b0c442fbb1117ed29e0efdff1e18412f7567

.github/workflows/keep-alive.yml

@@ -27,7 +27,7 @@ jobs:
 
     steps:
       - name: Checkout code
-        uses: actions/checkout@d632683dd7b4114ad314bca15554477dd762a938
+        uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683
         with:
           repository: exercism/${{ inputs.repository || github.event.inputs.repository }}
           token: ${{ secrets.pat_empty_commits }}

.github/workflows/labels.yml

@@ -12,7 +12,7 @@ jobs:
     runs-on: ubuntu-22.04
     steps:
       - name: Checkout code
-        uses: actions/checkout@d632683dd7b4114ad314bca15554477dd762a938
+        uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683
 
       - name: Sync labels
         uses: micnncim/action-label-syncer@3abd5ab72fda571e69fffd97bd4e0033dd5f495c

.github/workflows/shellcheck.yml

@@ -12,7 +12,7 @@ jobs:
     runs-on: ubuntu-22.04
     steps:
       - name: Checkout
-        uses: actions/checkout@d632683dd7b4114ad314bca15554477dd762a938
+        uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683
 
       - name: Run shellcheck
         uses: ludeeus/action-shellcheck@00cae500b08a931fb5698e11e79bfbd38e612a38