From d073fb8ea722be8ff7b2e3efea78f2b2bf338d3a Mon Sep 17 00:00:00 2001
From: Ludovic Fernandez <ldez@users.noreply.github.com>
Date: Wed, 1 May 2024 04:24:14 +0200
Subject: [PATCH 01/14] doc: add information about annotation limitations

---
 README.md | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/README.md b/README.md
index d1754e92ec..e05b4df60c 100644
--- a/README.md
+++ b/README.md
@@ -154,7 +154,7 @@ You will also likely need to add the following `.gitattributes` file to ensure t
 *.go text eol=lf
 ```
 
-## Comments and Annotations
+## Annotations
 
 Currently, GitHub parses the action's output and creates [annotations](https://github.blog/2018-12-14-introducing-check-runs-and-annotations/).
 
@@ -163,6 +163,7 @@ The restrictions of annotations are the following:
 1. Currently, they don't support Markdown formatting (see the [feature request](https://github.community/t5/GitHub-API-Development-and/Checks-Ability-to-include-Markdown-in-line-annotations/m-p/56704))
 2. They aren't shown in the list of comments.
    If you would like to have comments - please, up-vote [the issue](https://github.com/golangci/golangci-lint-action/issues/5).
+3. The number of annotations is [limited](https://github.com/actions/toolkit/blob/main/docs/problem-matchers.md#limitations).
 
 To enable annotations, you need to add the `checks' permission to your action.
 

From 3519a25e8931d06f18be2317c3f7f55c992666ed Mon Sep 17 00:00:00 2001
From: Fernandez Ludovic <ldez@users.noreply.github.com>
Date: Sat, 4 May 2024 00:37:13 +0200
Subject: [PATCH 02/14] chore: update tests workflow

---
 .github/workflows/test.yml | 11 ++++-------
 1 file changed, 4 insertions(+), 7 deletions(-)

diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
index 9a16cb3069..46092cbc3b 100644
--- a/.github/workflows/test.yml
+++ b/.github/workflows/test.yml
@@ -52,8 +52,8 @@ jobs:
         version:
           - ""
           - "latest"
-          - "v1.56"
-          - "v1.56.1"
+          - "v1.58"
+          - "v1.58.0"
     runs-on: ${{ matrix.os }}
     permissions:
       contents: read
@@ -63,7 +63,6 @@ jobs:
       - uses: actions/setup-go@v5
         with:
           go-version: oldstable
-          cache: false # setup-go v4 caches by default
       - uses: ./
         with:
           version: ${{ matrix.version }}
@@ -81,8 +80,8 @@ jobs:
         version:
           - ""
           - "latest"
-          - "v1.56.1"
-          - "bf5008a11acf2da5fe76716eb21d808499e079fa"
+          - "v1.58.0"
+          - "4bf574a12bb61234e28e3d6172be6ed95b0e8baf"
     runs-on: ${{ matrix.os }}
     permissions:
       contents: read
@@ -92,7 +91,6 @@ jobs:
       - uses: actions/setup-go@v5
         with:
           go-version: oldstable
-          cache: false # setup-go v4 caches by default
       - uses: ./
         with:
           version: ${{ matrix.version }}
@@ -116,7 +114,6 @@ jobs:
       - uses: actions/setup-go@v5
         with:
           go-version: oldstable
-          cache: false # setup-go v4 caches by default
       - uses: ./
         with:
           working-directory: sample-go-mod

From 57c4c9d18962515bb76a13f1b7204198cd7c2361 Mon Sep 17 00:00:00 2001
From: Fernandez Ludovic <ldez@users.noreply.github.com>
Date: Sat, 4 May 2024 00:37:23 +0200
Subject: [PATCH 03/14] docs: update readme

---
 README.md | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/README.md b/README.md
index e05b4df60c..73cb06fd30 100644
--- a/README.md
+++ b/README.md
@@ -43,14 +43,14 @@ jobs:
       - uses: actions/checkout@v4
       - uses: actions/setup-go@v5
         with:
-          go-version: '1.21'
+          go-version: '1.22'
       - name: golangci-lint
         uses: golangci/golangci-lint-action@v5
         with:
           # Require: The version of golangci-lint to use.
           # When `install-mode` is `binary` (default) the value can be v1.2 or v1.2.3 or `latest` to use the latest version.
           # When `install-mode` is `goinstall` the value can be v1.2.3, `latest`, or the hash of a commit.
-          version: v1.57
+          version: v1.58
 
           # Optional: working directory, useful for monorepos
           # working-directory: somedir
@@ -105,7 +105,7 @@ jobs:
   golangci:
     strategy:
       matrix:
-        go: ['1.21']
+        go: ['1.22']
         os: [ubuntu-latest, macos-latest, windows-latest]
     name: lint
     runs-on: ${{ matrix.os }}
@@ -120,7 +120,7 @@ jobs:
           # Require: The version of golangci-lint to use.
           # When `install-mode` is `binary` (default) the value can be v1.2 or v1.2.3 or `latest` to use the latest version.
           # When `install-mode` is `goinstall` the value can be v1.2.3, `latest`, or the hash of a commit.
-          version: v1.57
+          version: v1.58
 
           # Optional: working directory, useful for monorepos
           # working-directory: somedir
@@ -165,7 +165,7 @@ The restrictions of annotations are the following:
    If you would like to have comments - please, up-vote [the issue](https://github.com/golangci/golangci-lint-action/issues/5).
 3. The number of annotations is [limited](https://github.com/actions/toolkit/blob/main/docs/problem-matchers.md#limitations).
 
-To enable annotations, you need to add the `checks' permission to your action.
+To enable annotations, you need to add the `checks` permission to your action.
 
 ```yaml annotate
 permissions:
@@ -173,7 +173,7 @@ permissions:
   contents: read
   # Optional: allow read access to pull request. Use with `only-new-issues` option.
   pull-requests: read
-  # Optional: Allow write access to checks to allow the action to annotate code in the PR.
+  # Optional: allow write access to checks to allow the action to annotate code in the PR.
   checks: write
 ```
 

From 7a6f31107b9dc1a55e7f317bbfac69d8af339e60 Mon Sep 17 00:00:00 2001
From: Fernandez Ludovic <ldez@users.noreply.github.com>
Date: Sat, 4 May 2024 01:17:12 +0200
Subject: [PATCH 04/14] doc: improve options description

---
 README.md | 136 +++++++++++++++++++++++++++++++-----------------------
 1 file changed, 78 insertions(+), 58 deletions(-)

diff --git a/README.md b/README.md
index 73cb06fd30..1aefc374d9 100644
--- a/README.md
+++ b/README.md
@@ -47,35 +47,7 @@ jobs:
       - name: golangci-lint
         uses: golangci/golangci-lint-action@v5
         with:
-          # Require: The version of golangci-lint to use.
-          # When `install-mode` is `binary` (default) the value can be v1.2 or v1.2.3 or `latest` to use the latest version.
-          # When `install-mode` is `goinstall` the value can be v1.2.3, `latest`, or the hash of a commit.
-          version: v1.58
-
-          # Optional: working directory, useful for monorepos
-          # working-directory: somedir
-
-          # Optional: golangci-lint command line arguments.
-          #
-          # Note: By default, the `.golangci.yml` file should be at the root of the repository.
-          # The location of the configuration file can be changed by using `--config=`
-          # args: --timeout=30m --config=/my/path/.golangci.yml --issues-exit-code=0 
-
-          # Optional: Show only new issues.
-          # If you are using `merge_group` event (merge queue) you should add the option `fetch-depth: 0` to `actions/checkout` step.
-          # The default value is `false`.
-          # only-new-issues: true
-
-          # Optional: if set to true, then all caching functionality will be completely disabled,
-          #           takes precedence over all other caching options.
-          # skip-cache: true
-
-          # Optional: if set to true, caches will not be saved, but they may still be restored,
-          #           subject to other options
-          # skip-save-cache: true
-
-          # Optional: The mode to install golangci-lint. It can be 'binary' or 'goinstall'.
-          # install-mode: "goinstall"
+          version: latest
 ```
 
 We recommend running this action in a job separate from other jobs (`go test`, etc.)
@@ -117,35 +89,7 @@ jobs:
       - name: golangci-lint
         uses: golangci/golangci-lint-action@v5
         with:
-          # Require: The version of golangci-lint to use.
-          # When `install-mode` is `binary` (default) the value can be v1.2 or v1.2.3 or `latest` to use the latest version.
-          # When `install-mode` is `goinstall` the value can be v1.2.3, `latest`, or the hash of a commit.
-          version: v1.58
-
-          # Optional: working directory, useful for monorepos
-          # working-directory: somedir
-
-          # Optional: golangci-lint command line arguments.
-          #
-          # Note: By default, the `.golangci.yml` file should be at the root of the repository.
-          # The location of the configuration file can be changed by using `--config=`
-          # args: --timeout=30m --config=/my/path/.golangci.yml --issues-exit-code=0 
-
-          # Optional: Show only new issues.
-          # If you are using `merge_group` event (merge queue) you should add the option `fetch-depth: 0` to `actions/checkout` step.
-          # The default value is `false`.
-          # only-new-issues: true
-
-          # Optional: if set to true, then all caching functionality will be completely disabled,
-          #           takes precedence over all other caching options.
-          # skip-cache: true
-
-          # Optional: if set to true, caches will not be saved, but they may still be restored,
-          #           subject to other options
-          # skip-save-cache: true
-
-          # Optional: The mode to install golangci-lint. It can be 'binary' or 'goinstall'.
-          # install-mode: "goinstall"
+          version: latest
 ```
 
 You will also likely need to add the following `.gitattributes` file to ensure that line endings for Windows builds are properly formatted:
@@ -154,6 +98,82 @@ You will also likely need to add the following `.gitattributes` file to ensure t
 *.go text eol=lf
 ```
 
+## Options
+
+`version`: (required) The version of golangci-lint to use.
+* When `install-mode` is `binary` (default) the value can be v1.2 or v1.2.3 or `latest` to use the latest version.
+* When `install-mode` is `goinstall` the value can be v1.2.3, `latest`, or the hash of a commit.
+
+```yml
+uses: golangci/golangci-lint-action@v5
+with:
+  version: v1.58
+  # ...
+```
+
+`install-mode`: (optional) The mode to install golangci-lint.
+It can be `binary` or `goinstall`.
+The default value is `binary`.
+
+```yml
+uses: golangci/golangci-lint-action@v5
+with:
+  install-mode: "goinstall"
+  # ...
+```
+
+`only-new-issues`: (optional) Show only new issues.
+If you are using `merge_group` event (merge queue) you should add the option `fetch-depth: 0` to `actions/checkout` step.
+The default value is `false`.
+
+```yml
+uses: golangci/golangci-lint-action@v5
+with:
+  only-new-issues: true
+  # ...
+```
+
+`working-directory`: (optional) working directory, useful for monorepos.
+
+```yml
+uses: golangci/golangci-lint-action@v5
+with:
+  working-directory: somedir
+  # ...
+```
+
+`skip-cache`: (optional) If set to `true`, then all caching functionality will be completely disabled,
+takes precedence over all other caching options.
+The default value is `false`.
+
+```yml
+uses: golangci/golangci-lint-action@v5
+with:
+  skip-cache: true
+  # ...
+```
+
+`skip-save-cache`: (optional) If set to `true`, caches will not be saved, but they may still be restored, required `skip-cache: false`.
+The default value is `false`.
+
+```yml
+uses: golangci/golangci-lint-action@v5
+with:
+  skip-save-cache: true
+  # ...
+```
+
+`args`: (optional) golangci-lint command line arguments.
+Note: By default, the `.golangci.yml` file should be at the root of the repository.
+The location of the configuration file can be changed by using `--config=`
+
+```yml
+uses: golangci/golangci-lint-action@v5
+with:
+  args: --timeout=30m --config=/my/path/.golangci.yml --issues-exit-code=0
+  # ...
+```
+
 ## Annotations
 
 Currently, GitHub parses the action's output and creates [annotations](https://github.blog/2018-12-14-introducing-check-runs-and-annotations/).

From 2bff40627723077fc557d83b223b2fcf2d4c6974 Mon Sep 17 00:00:00 2001
From: Fernandez Ludovic <ldez@users.noreply.github.com>
Date: Sat, 4 May 2024 01:27:11 +0200
Subject: [PATCH 05/14] doc: improve options documentation

---
 README.md  |  2 +-
 action.yml | 12 ++++++------
 2 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/README.md b/README.md
index 1aefc374d9..4f8d941830 100644
--- a/README.md
+++ b/README.md
@@ -107,7 +107,7 @@ You will also likely need to add the following `.gitattributes` file to ensure t
 ```yml
 uses: golangci/golangci-lint-action@v5
 with:
-  version: v1.58
+  version: latest
   # ...
 ```
 
diff --git a/action.yml b/action.yml
index 4f3cb3a4bf..a85cf66f1e 100644
--- a/action.yml
+++ b/action.yml
@@ -8,9 +8,9 @@ inputs:
       When `install-mode` is `binary` (default) the value can be v1.2 or v1.2.3 or `latest` to use the latest version.
       When `install-mode` is `goinstall` the value can be v1.2.3, `latest`, or the hash of a commit.
     required: false
-  args:
-    description: "golangci-lint command line arguments"
-    default: ""
+  install-mode:
+    description: "The mode to install golangci-lint. It can be 'binary' or 'goinstall'."
+    default: "binary"
     required: false
   working-directory:
     description: "golangci-lint working directory, default is project root"
@@ -35,9 +35,9 @@ inputs:
       restore existing caches, subject to other options.
     default: 'false'
     required: false
-  install-mode:
-    description: "The mode to install golangci-lint. It can be 'binary' or 'goinstall'."
-    default: "binary"
+  args:
+    description: "golangci-lint command line arguments"
+    default: ""
     required: false
 runs:
   using: "node20"

From aebff4bd9cd0198ff4f020915c27258a9edc4c01 Mon Sep 17 00:00:00 2001
From: Fernandez Ludovic <ldez@users.noreply.github.com>
Date: Sat, 4 May 2024 01:36:48 +0200
Subject: [PATCH 06/14] feat: add option to enable/disable annotations

---
 README.md              | 12 ++++++++++++
 action.yml             |  4 ++++
 dist/post_run/index.js | 21 ++++++++++++---------
 dist/run/index.js      | 21 ++++++++++++---------
 src/run.ts             | 24 ++++++++++++++----------
 5 files changed, 54 insertions(+), 28 deletions(-)

diff --git a/README.md b/README.md
index 4f8d941830..b33532660f 100644
--- a/README.md
+++ b/README.md
@@ -163,6 +163,18 @@ with:
   # ...
 ```
 
+`annotations`: (optional) To enable/disable GitHub Action annotations.
+If disabled (`false`), the output format(s) will follow the golangci-lint configuration file and use the same default as golangci-lint (i.e. `colored-line-number`).
+https://golangci-lint.run/usage/configuration/#output-configuration
+The default value is `true`.
+
+```yml
+uses: golangci/golangci-lint-action@v5
+with:
+  annotations: false
+  # ...
+```
+
 `args`: (optional) golangci-lint command line arguments.
 Note: By default, the `.golangci.yml` file should be at the root of the repository.
 The location of the configuration file can be changed by using `--config=`
diff --git a/action.yml b/action.yml
index a85cf66f1e..b498ad078f 100644
--- a/action.yml
+++ b/action.yml
@@ -35,6 +35,10 @@ inputs:
       restore existing caches, subject to other options.
     default: 'false'
     required: false
+  annotations:
+    description: "To Enable/disable GitHub Action annotations"
+    default: 'true'
+    required: false
   args:
     description: "golangci-lint command line arguments"
     default: ""
diff --git a/dist/post_run/index.js b/dist/post_run/index.js
index 9dd2226db6..b6746c4c53 100644
--- a/dist/post_run/index.js
+++ b/dist/post_run/index.js
@@ -89287,15 +89287,18 @@ async function runLint(lintPath, patchPath) {
         .map(([key, value]) => [key.toLowerCase(), value ?? ""]);
     const userArgsMap = new Map(userArgsList);
     const userArgNames = new Set(userArgsList.map(([key]) => key));
-    const formats = (userArgsMap.get("out-format") || "")
-        .trim()
-        .split(",")
-        .filter((f) => f.length > 0)
-        .filter((f) => !f.startsWith(`github-actions`))
-        .concat("github-actions")
-        .join(",");
-    addedArgs.push(`--out-format=${formats}`);
-    userArgs = userArgs.replace(/--out-format=\S*/gi, "").trim();
+    const annotations = core.getInput(`annotations`).trim() !== "false";
+    if (annotations) {
+        const formats = (userArgsMap.get("out-format") || "")
+            .trim()
+            .split(",")
+            .filter((f) => f.length > 0)
+            .filter((f) => !f.startsWith(`github-actions`))
+            .concat("github-actions")
+            .join(",");
+        addedArgs.push(`--out-format=${formats}`);
+        userArgs = userArgs.replace(/--out-format=\S*/gi, "").trim();
+    }
     if (isOnlyNewIssues()) {
         if (userArgNames.has(`new`) || userArgNames.has(`new-from-rev`) || userArgNames.has(`new-from-patch`)) {
             throw new Error(`please, don't specify manually --new* args when requesting only new issues`);
diff --git a/dist/run/index.js b/dist/run/index.js
index b015ef3b3f..278dc9a2c6 100644
--- a/dist/run/index.js
+++ b/dist/run/index.js
@@ -89287,15 +89287,18 @@ async function runLint(lintPath, patchPath) {
         .map(([key, value]) => [key.toLowerCase(), value ?? ""]);
     const userArgsMap = new Map(userArgsList);
     const userArgNames = new Set(userArgsList.map(([key]) => key));
-    const formats = (userArgsMap.get("out-format") || "")
-        .trim()
-        .split(",")
-        .filter((f) => f.length > 0)
-        .filter((f) => !f.startsWith(`github-actions`))
-        .concat("github-actions")
-        .join(",");
-    addedArgs.push(`--out-format=${formats}`);
-    userArgs = userArgs.replace(/--out-format=\S*/gi, "").trim();
+    const annotations = core.getInput(`annotations`).trim() !== "false";
+    if (annotations) {
+        const formats = (userArgsMap.get("out-format") || "")
+            .trim()
+            .split(",")
+            .filter((f) => f.length > 0)
+            .filter((f) => !f.startsWith(`github-actions`))
+            .concat("github-actions")
+            .join(",");
+        addedArgs.push(`--out-format=${formats}`);
+        userArgs = userArgs.replace(/--out-format=\S*/gi, "").trim();
+    }
     if (isOnlyNewIssues()) {
         if (userArgNames.has(`new`) || userArgNames.has(`new-from-rev`) || userArgNames.has(`new-from-patch`)) {
             throw new Error(`please, don't specify manually --new* args when requesting only new issues`);
diff --git a/src/run.ts b/src/run.ts
index 60ff43201b..db8cf53a50 100644
--- a/src/run.ts
+++ b/src/run.ts
@@ -192,16 +192,20 @@ async function runLint(lintPath: string, patchPath: string): Promise<void> {
   const userArgsMap = new Map<string, string>(userArgsList)
   const userArgNames = new Set<string>(userArgsList.map(([key]) => key))
 
-  const formats = (userArgsMap.get("out-format") || "")
-    .trim()
-    .split(",")
-    .filter((f) => f.length > 0)
-    .filter((f) => !f.startsWith(`github-actions`))
-    .concat("github-actions")
-    .join(",")
-
-  addedArgs.push(`--out-format=${formats}`)
-  userArgs = userArgs.replace(/--out-format=\S*/gi, "").trim()
+  const annotations = core.getInput(`annotations`).trim() !== "false"
+
+  if (annotations) {
+    const formats = (userArgsMap.get("out-format") || "")
+      .trim()
+      .split(",")
+      .filter((f) => f.length > 0)
+      .filter((f) => !f.startsWith(`github-actions`))
+      .concat("github-actions")
+      .join(",")
+
+    addedArgs.push(`--out-format=${formats}`)
+    userArgs = userArgs.replace(/--out-format=\S*/gi, "").trim()
+  }
 
   if (isOnlyNewIssues()) {
     if (userArgNames.has(`new`) || userArgNames.has(`new-from-rev`) || userArgNames.has(`new-from-patch`)) {

From d9c9b53e53fa63215cdd19637cd95fbc34fdf4a1 Mon Sep 17 00:00:00 2001
From: Fernandez Ludovic <ldez@users.noreply.github.com>
Date: Sat, 4 May 2024 01:41:30 +0200
Subject: [PATCH 07/14] chore: improve marketplace style

---
 action.yml | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/action.yml b/action.yml
index b498ad078f..2318f3473b 100644
--- a/action.yml
+++ b/action.yml
@@ -1,4 +1,5 @@
-name: "Run golangci-lint"
+# https://docs.github.com/en/actions/creating-actions/metadata-syntax-for-github-actions
+name: "Golangci-lint"
 description: "Official golangci-lint action with line-attached annotations for found issues, caching and parallel execution."
 author: "golangci"
 inputs:
@@ -49,4 +50,4 @@ runs:
   post: "dist/post_run/index.js"
 branding:
   icon: "shield"
-  color: "yellow"
+  color: "white"

From ca8befdfb6b98e59b2a2d1d65070c6e66a858ae1 Mon Sep 17 00:00:00 2001
From: Fernandez Ludovic <ldez@users.noreply.github.com>
Date: Sat, 4 May 2024 02:36:06 +0200
Subject: [PATCH 08/14] doc: improve options documentation

---
 README.md | 32 ++++++++++++++++++++++++--------
 1 file changed, 24 insertions(+), 8 deletions(-)

diff --git a/README.md b/README.md
index b33532660f..abe55b532c 100644
--- a/README.md
+++ b/README.md
@@ -100,7 +100,9 @@ You will also likely need to add the following `.gitattributes` file to ensure t
 
 ## Options
 
-`version`: (required) The version of golangci-lint to use.
+### `version`
+
+(required) The version of golangci-lint to use.
 * When `install-mode` is `binary` (default) the value can be v1.2 or v1.2.3 or `latest` to use the latest version.
 * When `install-mode` is `goinstall` the value can be v1.2.3, `latest`, or the hash of a commit.
 
@@ -111,7 +113,9 @@ with:
   # ...
 ```
 
-`install-mode`: (optional) The mode to install golangci-lint.
+### `install-mode`
+
+(optional) The mode to install golangci-lint.
 It can be `binary` or `goinstall`.
 The default value is `binary`.
 
@@ -122,7 +126,9 @@ with:
   # ...
 ```
 
-`only-new-issues`: (optional) Show only new issues.
+### `only-new-issues`
+
+(optional) Show only new issues.
 If you are using `merge_group` event (merge queue) you should add the option `fetch-depth: 0` to `actions/checkout` step.
 The default value is `false`.
 
@@ -133,7 +139,9 @@ with:
   # ...
 ```
 
-`working-directory`: (optional) working directory, useful for monorepos.
+### `working-directory`
+
+(optional) working directory, useful for monorepos.
 
 ```yml
 uses: golangci/golangci-lint-action@v5
@@ -142,7 +150,9 @@ with:
   # ...
 ```
 
-`skip-cache`: (optional) If set to `true`, then all caching functionality will be completely disabled,
+### `skip-cache`
+
+(optional) If set to `true`, then all caching functionality will be completely disabled,
 takes precedence over all other caching options.
 The default value is `false`.
 
@@ -153,7 +163,9 @@ with:
   # ...
 ```
 
-`skip-save-cache`: (optional) If set to `true`, caches will not be saved, but they may still be restored, required `skip-cache: false`.
+### `skip-save-cache`
+
+(optional) If set to `true`, caches will not be saved, but they may still be restored, required `skip-cache: false`.
 The default value is `false`.
 
 ```yml
@@ -163,7 +175,9 @@ with:
   # ...
 ```
 
-`annotations`: (optional) To enable/disable GitHub Action annotations.
+### `annotations`
+
+(optional) To enable/disable GitHub Action annotations.
 If disabled (`false`), the output format(s) will follow the golangci-lint configuration file and use the same default as golangci-lint (i.e. `colored-line-number`).
 https://golangci-lint.run/usage/configuration/#output-configuration
 The default value is `true`.
@@ -175,7 +189,9 @@ with:
   # ...
 ```
 
-`args`: (optional) golangci-lint command line arguments.
+### `args`
+
+(optional) golangci-lint command line arguments.
 Note: By default, the `.golangci.yml` file should be at the root of the repository.
 The location of the configuration file can be changed by using `--config=`
 

From 046435d14ca6ec39fd472741e51e3bfbc403b4a7 Mon Sep 17 00:00:00 2001
From: Fernandez Ludovic <ldez@users.noreply.github.com>
Date: Sat, 4 May 2024 02:42:39 +0200
Subject: [PATCH 09/14] doc: improve options documentation

---
 README.md | 46 ++++++++++++++++++++++++++++++++++++----------
 1 file changed, 36 insertions(+), 10 deletions(-)

diff --git a/README.md b/README.md
index abe55b532c..b05f02e823 100644
--- a/README.md
+++ b/README.md
@@ -102,7 +102,10 @@ You will also likely need to add the following `.gitattributes` file to ensure t
 
 ### `version`
 
-(required) The version of golangci-lint to use.
+(required)
+
+The version of golangci-lint to use.
+
 * When `install-mode` is `binary` (default) the value can be v1.2 or v1.2.3 or `latest` to use the latest version.
 * When `install-mode` is `goinstall` the value can be v1.2.3, `latest`, or the hash of a commit.
 
@@ -115,8 +118,10 @@ with:
 
 ### `install-mode`
 
-(optional) The mode to install golangci-lint.
-It can be `binary` or `goinstall`.
+(optional)
+
+The mode to install golangci-lint: it can be `binary` or `goinstall`.
+
 The default value is `binary`.
 
 ```yml
@@ -128,8 +133,12 @@ with:
 
 ### `only-new-issues`
 
-(optional) Show only new issues.
+(optional)
+
+Show only new issues.
+
 If you are using `merge_group` event (merge queue) you should add the option `fetch-depth: 0` to `actions/checkout` step.
+
 The default value is `false`.
 
 ```yml
@@ -141,7 +150,9 @@ with:
 
 ### `working-directory`
 
-(optional) working directory, useful for monorepos.
+(optional)
+
+Working directory, useful for monorepos.
 
 ```yml
 uses: golangci/golangci-lint-action@v5
@@ -152,8 +163,11 @@ with:
 
 ### `skip-cache`
 
-(optional) If set to `true`, then all caching functionality will be completely disabled,
+(optional)
+
+If set to `true`, then all caching functionality will be completely disabled,
 takes precedence over all other caching options.
+
 The default value is `false`.
 
 ```yml
@@ -165,7 +179,10 @@ with:
 
 ### `skip-save-cache`
 
-(optional) If set to `true`, caches will not be saved, but they may still be restored, required `skip-cache: false`.
+(optional)
+
+If set to `true`, caches will not be saved, but they may still be restored, required `skip-cache: false`.
+
 The default value is `false`.
 
 ```yml
@@ -177,9 +194,15 @@ with:
 
 ### `annotations`
 
-(optional) To enable/disable GitHub Action annotations.
-If disabled (`false`), the output format(s) will follow the golangci-lint configuration file and use the same default as golangci-lint (i.e. `colored-line-number`).
+(optional)
+
+To enable/disable GitHub Action annotations.
+
+If disabled (`false`), the output format(s) will follow the golangci-lint configuration file (or CLI flags from `args`) 
+and use the same default as golangci-lint (i.e. `colored-line-number`).
+
 https://golangci-lint.run/usage/configuration/#output-configuration
+
 The default value is `true`.
 
 ```yml
@@ -191,7 +214,10 @@ with:
 
 ### `args`
 
-(optional) golangci-lint command line arguments.
+(optional)
+
+golangci-lint command line arguments.
+
 Note: By default, the `.golangci.yml` file should be at the root of the repository.
 The location of the configuration file can be changed by using `--config=`
 

From ecb32920c642fc861819e7d1e27c1f9fff303882 Mon Sep 17 00:00:00 2001
From: Ludovic Fernandez <ldez@users.noreply.github.com>
Date: Sat, 4 May 2024 16:54:30 +0200
Subject: [PATCH 10/14] feat: uses 2 dots compare syntax for push diff (#1030)

---
 README.md              | 7 +++++--
 dist/post_run/index.js | 5 ++---
 dist/run/index.js      | 5 ++---
 src/run.ts             | 5 ++---
 4 files changed, 11 insertions(+), 11 deletions(-)

diff --git a/README.md b/README.md
index b05f02e823..e89250977d 100644
--- a/README.md
+++ b/README.md
@@ -137,8 +137,6 @@ with:
 
 Show only new issues.
 
-If you are using `merge_group` event (merge queue) you should add the option `fetch-depth: 0` to `actions/checkout` step.
-
 The default value is `false`.
 
 ```yml
@@ -148,6 +146,11 @@ with:
   # ...
 ```
 
+* `pull_request` and `pull_request_target`: the action gets the diff of the PR content from the [GitHub API](https://docs.github.com/en/rest/pulls/pulls?apiVersion=2022-11-28#get-a-pull-request) and use it with `--new-from-patch`.
+* `push`: the action gets the diff of the push content (difference between commits before and after the push) from the [GitHub API](https://docs.github.com/en/rest/commits/commits?apiVersion=2022-11-28#compare-two-commits) and use it with `--new-from-patch`.
+* `merge_group`: the action gets the diff by using `--new-from-rev` option (relies on git).
+   You should add the option `fetch-depth: 0` to `actions/checkout` step.
+
 ### `working-directory`
 
 (optional)
diff --git a/dist/post_run/index.js b/dist/post_run/index.js
index b6746c4c53..257c21b5c5 100644
--- a/dist/post_run/index.js
+++ b/dist/post_run/index.js
@@ -89221,11 +89221,10 @@ async function fetchPushPatch(ctx) {
     const octokit = github.getOctokit(core.getInput(`github-token`, { required: true }));
     let patch;
     try {
-        const patchResp = await octokit.rest.repos.compareCommits({
+        const patchResp = await octokit.rest.repos.compareCommitsWithBasehead({
             owner: ctx.repo.owner,
             repo: ctx.repo.repo,
-            base: ctx.payload.before,
-            head: ctx.payload.after,
+            basehead: `${ctx.payload.before}..${ctx.payload.after}`,
             mediaType: {
                 format: `diff`,
             },
diff --git a/dist/run/index.js b/dist/run/index.js
index 278dc9a2c6..111a65705c 100644
--- a/dist/run/index.js
+++ b/dist/run/index.js
@@ -89221,11 +89221,10 @@ async function fetchPushPatch(ctx) {
     const octokit = github.getOctokit(core.getInput(`github-token`, { required: true }));
     let patch;
     try {
-        const patchResp = await octokit.rest.repos.compareCommits({
+        const patchResp = await octokit.rest.repos.compareCommitsWithBasehead({
             owner: ctx.repo.owner,
             repo: ctx.repo.repo,
-            base: ctx.payload.before,
-            head: ctx.payload.after,
+            basehead: `${ctx.payload.before}..${ctx.payload.after}`,
             mediaType: {
                 format: `diff`,
             },
diff --git a/src/run.ts b/src/run.ts
index db8cf53a50..2d5d3a2ae2 100644
--- a/src/run.ts
+++ b/src/run.ts
@@ -104,11 +104,10 @@ async function fetchPushPatch(ctx: Context): Promise<string> {
 
   let patch: string
   try {
-    const patchResp = await octokit.rest.repos.compareCommits({
+    const patchResp = await octokit.rest.repos.compareCommitsWithBasehead({
       owner: ctx.repo.owner,
       repo: ctx.repo.repo,
-      base: ctx.payload.before,
-      head: ctx.payload.after,
+      basehead: `${ctx.payload.before}..${ctx.payload.after}`,
       mediaType: {
         format: `diff`,
       },

From dbb7ebcd4c72c62f0b6fe5dd8f2cf90cc85c7946 Mon Sep 17 00:00:00 2001
From: Ludovic Fernandez <ldez@users.noreply.github.com>
Date: Sat, 4 May 2024 17:47:25 +0200
Subject: [PATCH 11/14] feat: add option to control cache invalidation interval
 (#1031)

---
 README.md              | 17 +++++++++++++++++
 action.yml             |  4 ++++
 dist/post_run/index.js |  7 +++++--
 dist/run/index.js      |  7 +++++--
 src/cache.ts           | 16 ++++++++++++++--
 5 files changed, 45 insertions(+), 6 deletions(-)

diff --git a/README.md b/README.md
index e89250977d..e32a5c0172 100644
--- a/README.md
+++ b/README.md
@@ -195,6 +195,23 @@ with:
   # ...
 ```
 
+### `cache-invalidation-interval`
+
+(optional)
+
+Periodically invalidate the cache every `cache-invalidation-interval` days to ensure that outdated data is removed and fresh data is loaded.
+
+The default value is `7`.
+
+```yml
+uses: golangci/golangci-lint-action@v5
+with:
+  cache-invalidation-interval: 15
+  # ...
+```
+
+If set the number is `<= 0`, the cache will be always invalidate (Not recommended).
+
 ### `annotations`
 
 (optional)
diff --git a/action.yml b/action.yml
index 2318f3473b..bb4a50870e 100644
--- a/action.yml
+++ b/action.yml
@@ -44,6 +44,10 @@ inputs:
     description: "golangci-lint command line arguments"
     default: ""
     required: false
+  cache-invalidation-interval:
+    description: "Periodically invalidate a cache because a new code being added. (number of days)"
+    default: '7'
+    required: false
 runs:
   using: "node20"
   main: "dist/run/index.js"
diff --git a/dist/post_run/index.js b/dist/post_run/index.js
index 257c21b5c5..06050ea1f0 100644
--- a/dist/post_run/index.js
+++ b/dist/post_run/index.js
@@ -88815,15 +88815,18 @@ const getLintCacheDir = () => {
 };
 const getIntervalKey = (invalidationIntervalDays) => {
     const now = new Date();
+    if (invalidationIntervalDays <= 0) {
+        return `${now.getTime()}`;
+    }
     const secondsSinceEpoch = now.getTime() / 1000;
     const intervalNumber = Math.floor(secondsSinceEpoch / (invalidationIntervalDays * 86400));
     return intervalNumber.toString();
 };
 async function buildCacheKeys() {
     const keys = [];
+    const invalidationIntervalDays = parseInt(core.getInput(`cache-invalidation-interval`, { required: true }).trim());
     // Periodically invalidate a cache because a new code being added.
-    // TODO: configure it via inputs.
-    let cacheKey = `golangci-lint.cache-${getIntervalKey(7)}-`;
+    let cacheKey = `golangci-lint.cache-${getIntervalKey(invalidationIntervalDays)}-`;
     keys.push(cacheKey);
     // Get working directory from input
     const workingDirectory = core.getInput(`working-directory`);
diff --git a/dist/run/index.js b/dist/run/index.js
index 111a65705c..b361fc4f25 100644
--- a/dist/run/index.js
+++ b/dist/run/index.js
@@ -88815,15 +88815,18 @@ const getLintCacheDir = () => {
 };
 const getIntervalKey = (invalidationIntervalDays) => {
     const now = new Date();
+    if (invalidationIntervalDays <= 0) {
+        return `${now.getTime()}`;
+    }
     const secondsSinceEpoch = now.getTime() / 1000;
     const intervalNumber = Math.floor(secondsSinceEpoch / (invalidationIntervalDays * 86400));
     return intervalNumber.toString();
 };
 async function buildCacheKeys() {
     const keys = [];
+    const invalidationIntervalDays = parseInt(core.getInput(`cache-invalidation-interval`, { required: true }).trim());
     // Periodically invalidate a cache because a new code being added.
-    // TODO: configure it via inputs.
-    let cacheKey = `golangci-lint.cache-${getIntervalKey(7)}-`;
+    let cacheKey = `golangci-lint.cache-${getIntervalKey(invalidationIntervalDays)}-`;
     keys.push(cacheKey);
     // Get working directory from input
     const workingDirectory = core.getInput(`working-directory`);
diff --git a/src/cache.ts b/src/cache.ts
index 5c73169753..75fda9f9a4 100644
--- a/src/cache.ts
+++ b/src/cache.ts
@@ -25,6 +25,11 @@ const getLintCacheDir = (): string => {
 
 const getIntervalKey = (invalidationIntervalDays: number): string => {
   const now = new Date()
+
+  if (invalidationIntervalDays <= 0) {
+    return `${now.getTime()}`
+  }
+
   const secondsSinceEpoch = now.getTime() / 1000
   const intervalNumber = Math.floor(secondsSinceEpoch / (invalidationIntervalDays * 86400))
   return intervalNumber.toString()
@@ -32,21 +37,28 @@ const getIntervalKey = (invalidationIntervalDays: number): string => {
 
 async function buildCacheKeys(): Promise<string[]> {
   const keys = []
+
+  const invalidationIntervalDays = parseInt(core.getInput(`cache-invalidation-interval`, { required: true }).trim())
+
   // Periodically invalidate a cache because a new code being added.
-  // TODO: configure it via inputs.
-  let cacheKey = `golangci-lint.cache-${getIntervalKey(7)}-`
+  let cacheKey = `golangci-lint.cache-${getIntervalKey(invalidationIntervalDays)}-`
   keys.push(cacheKey)
+
   // Get working directory from input
   const workingDirectory = core.getInput(`working-directory`)
+
   // create path to go.mod prepending the workingDirectory if it exists
   const goModPath = path.join(workingDirectory, `go.mod`)
+
   core.info(`Checking for go.mod: ${goModPath}`)
+
   if (await pathExists(goModPath)) {
     // Add checksum to key to invalidate a cache when dependencies change.
     cacheKey += await checksumFile(`sha1`, goModPath)
   } else {
     cacheKey += `nogomod`
   }
+
   keys.push(cacheKey)
 
   return keys

From dbe4fc23f1260a21484a6c6c008545afc299faa3 Mon Sep 17 00:00:00 2001
From: Fernandez Ludovic <ldez@users.noreply.github.com>
Date: Sat, 4 May 2024 17:48:42 +0200
Subject: [PATCH 12/14] chore: use getBooleanInput

---
 dist/post_run/index.js | 14 +++++---------
 dist/run/index.js      | 14 +++++---------
 src/cache.ts           |  6 +++---
 src/run.ts             | 10 ++--------
 4 files changed, 15 insertions(+), 29 deletions(-)

diff --git a/dist/post_run/index.js b/dist/post_run/index.js
index 06050ea1f0..6f07a5f73c 100644
--- a/dist/post_run/index.js
+++ b/dist/post_run/index.js
@@ -88844,7 +88844,7 @@ async function buildCacheKeys() {
     return keys;
 }
 async function restoreCache() {
-    if (core.getInput(`skip-cache`, { required: true }).trim() == "true")
+    if (core.getBooleanInput(`skip-cache`, { required: true }))
         return;
     if (!utils.isValidEvent()) {
         utils.logWarning(`Event Validation Error: The event type ${process.env[constants_1.Events.Key]} is not supported because it's not tied to a branch or tag ref.`);
@@ -88882,9 +88882,9 @@ async function restoreCache() {
 }
 exports.restoreCache = restoreCache;
 async function saveCache() {
-    if (core.getInput(`skip-cache`, { required: true }).trim() == "true")
+    if (core.getBooleanInput(`skip-cache`, { required: true }))
         return;
-    if (core.getInput(`skip-save-cache`, { required: true }).trim() == "true")
+    if (core.getBooleanInput(`skip-save-cache`, { required: true }))
         return;
     // Validate inputs, this can cause task failure
     if (!utils.isValidEvent()) {
@@ -89150,11 +89150,7 @@ const execShellCommand = (0, util_1.promisify)(child_process_1.exec);
 const writeFile = (0, util_1.promisify)(fs.writeFile);
 const createTempDir = (0, util_1.promisify)(tmp_1.dir);
 function isOnlyNewIssues() {
-    const onlyNewIssues = core.getInput(`only-new-issues`, { required: true }).trim();
-    if (onlyNewIssues !== `false` && onlyNewIssues !== `true`) {
-        throw new Error(`invalid value of "only-new-issues": "${onlyNewIssues}", expected "true" or "false"`);
-    }
-    return onlyNewIssues === `true`;
+    return core.getBooleanInput(`only-new-issues`, { required: true });
 }
 async function prepareLint() {
     const mode = core.getInput("install-mode").toLowerCase();
@@ -89289,7 +89285,7 @@ async function runLint(lintPath, patchPath) {
         .map(([key, value]) => [key.toLowerCase(), value ?? ""]);
     const userArgsMap = new Map(userArgsList);
     const userArgNames = new Set(userArgsList.map(([key]) => key));
-    const annotations = core.getInput(`annotations`).trim() !== "false";
+    const annotations = core.getBooleanInput(`annotations`);
     if (annotations) {
         const formats = (userArgsMap.get("out-format") || "")
             .trim()
diff --git a/dist/run/index.js b/dist/run/index.js
index b361fc4f25..b67b479635 100644
--- a/dist/run/index.js
+++ b/dist/run/index.js
@@ -88844,7 +88844,7 @@ async function buildCacheKeys() {
     return keys;
 }
 async function restoreCache() {
-    if (core.getInput(`skip-cache`, { required: true }).trim() == "true")
+    if (core.getBooleanInput(`skip-cache`, { required: true }))
         return;
     if (!utils.isValidEvent()) {
         utils.logWarning(`Event Validation Error: The event type ${process.env[constants_1.Events.Key]} is not supported because it's not tied to a branch or tag ref.`);
@@ -88882,9 +88882,9 @@ async function restoreCache() {
 }
 exports.restoreCache = restoreCache;
 async function saveCache() {
-    if (core.getInput(`skip-cache`, { required: true }).trim() == "true")
+    if (core.getBooleanInput(`skip-cache`, { required: true }))
         return;
-    if (core.getInput(`skip-save-cache`, { required: true }).trim() == "true")
+    if (core.getBooleanInput(`skip-save-cache`, { required: true }))
         return;
     // Validate inputs, this can cause task failure
     if (!utils.isValidEvent()) {
@@ -89150,11 +89150,7 @@ const execShellCommand = (0, util_1.promisify)(child_process_1.exec);
 const writeFile = (0, util_1.promisify)(fs.writeFile);
 const createTempDir = (0, util_1.promisify)(tmp_1.dir);
 function isOnlyNewIssues() {
-    const onlyNewIssues = core.getInput(`only-new-issues`, { required: true }).trim();
-    if (onlyNewIssues !== `false` && onlyNewIssues !== `true`) {
-        throw new Error(`invalid value of "only-new-issues": "${onlyNewIssues}", expected "true" or "false"`);
-    }
-    return onlyNewIssues === `true`;
+    return core.getBooleanInput(`only-new-issues`, { required: true });
 }
 async function prepareLint() {
     const mode = core.getInput("install-mode").toLowerCase();
@@ -89289,7 +89285,7 @@ async function runLint(lintPath, patchPath) {
         .map(([key, value]) => [key.toLowerCase(), value ?? ""]);
     const userArgsMap = new Map(userArgsList);
     const userArgNames = new Set(userArgsList.map(([key]) => key));
-    const annotations = core.getInput(`annotations`).trim() !== "false";
+    const annotations = core.getBooleanInput(`annotations`);
     if (annotations) {
         const formats = (userArgsMap.get("out-format") || "")
             .trim()
diff --git a/src/cache.ts b/src/cache.ts
index 75fda9f9a4..314647d6a7 100644
--- a/src/cache.ts
+++ b/src/cache.ts
@@ -65,7 +65,7 @@ async function buildCacheKeys(): Promise<string[]> {
 }
 
 export async function restoreCache(): Promise<void> {
-  if (core.getInput(`skip-cache`, { required: true }).trim() == "true") return
+  if (core.getBooleanInput(`skip-cache`, { required: true })) return
 
   if (!utils.isValidEvent()) {
     utils.logWarning(
@@ -107,8 +107,8 @@ export async function restoreCache(): Promise<void> {
 }
 
 export async function saveCache(): Promise<void> {
-  if (core.getInput(`skip-cache`, { required: true }).trim() == "true") return
-  if (core.getInput(`skip-save-cache`, { required: true }).trim() == "true") return
+  if (core.getBooleanInput(`skip-cache`, { required: true })) return
+  if (core.getBooleanInput(`skip-save-cache`, { required: true })) return
 
   // Validate inputs, this can cause task failure
   if (!utils.isValidEvent()) {
diff --git a/src/run.ts b/src/run.ts
index 2d5d3a2ae2..48550d4576 100644
--- a/src/run.ts
+++ b/src/run.ts
@@ -17,13 +17,7 @@ const writeFile = promisify(fs.writeFile)
 const createTempDir = promisify(dir)
 
 function isOnlyNewIssues(): boolean {
-  const onlyNewIssues = core.getInput(`only-new-issues`, { required: true }).trim()
-
-  if (onlyNewIssues !== `false` && onlyNewIssues !== `true`) {
-    throw new Error(`invalid value of "only-new-issues": "${onlyNewIssues}", expected "true" or "false"`)
-  }
-
-  return onlyNewIssues === `true`
+  return core.getBooleanInput(`only-new-issues`, { required: true })
 }
 
 async function prepareLint(): Promise<string> {
@@ -191,7 +185,7 @@ async function runLint(lintPath: string, patchPath: string): Promise<void> {
   const userArgsMap = new Map<string, string>(userArgsList)
   const userArgNames = new Set<string>(userArgsList.map(([key]) => key))
 
-  const annotations = core.getInput(`annotations`).trim() !== "false"
+  const annotations = core.getBooleanInput(`annotations`)
 
   if (annotations) {
     const formats = (userArgsMap.get("out-format") || "")

From 21e9e6b47f3490b8adcca53fe65c03e2687b503c Mon Sep 17 00:00:00 2001
From: Ludovic Fernandez <ldez@users.noreply.github.com>
Date: Sat, 4 May 2024 18:43:48 +0200
Subject: [PATCH 13/14] feat: use OS and working-directory as cache key (#1032)

---
 README.md              |  5 +++--
 dist/post_run/index.js | 13 +++++++++----
 dist/run/index.js      | 13 +++++++++----
 src/cache.ts           | 17 ++++++++++++-----
 4 files changed, 33 insertions(+), 15 deletions(-)

diff --git a/README.md b/README.md
index e32a5c0172..5e498a32a7 100644
--- a/README.md
+++ b/README.md
@@ -308,10 +308,11 @@ Inside our action, we perform 3 steps:
 ### Caching internals
 
 1. We save and restore the following directory: `~/.cache/golangci-lint`.
-2. The primary caching key looks like `golangci-lint.cache-{interval_number}-{go.mod_hash}`.
+2. The primary caching key looks like `golangci-lint.cache-{runner_os}-{working_directory}-{interval_number}-{go.mod_hash}`.
    Interval number ensures that we periodically invalidate our cache (every 7 days).
    `go.mod` hash ensures that we invalidate the cache early - as soon as dependencies have changed.
-3. We use [restore keys](https://help.github.com/en/actions/configuring-and-managing-workflows/caching-dependencies-to-speed-up-workflows#matching-a-cache-key): `golangci-lint.cache-{interval_number}-`. 
+3. We use [restore keys](https://help.github.com/en/actions/configuring-and-managing-workflows/caching-dependencies-to-speed-up-workflows#matching-a-cache-key):
+   `golangci-lint.cache-{runner_os}-{working_directory}-{interval_number}-`.
    GitHub matches keys by prefix if we have no exact match for the primary cache.
 
 This scheme is basic and needs improvements. Pull requests and ideas are welcome.
diff --git a/dist/post_run/index.js b/dist/post_run/index.js
index 6f07a5f73c..23f6c001c0 100644
--- a/dist/post_run/index.js
+++ b/dist/post_run/index.js
@@ -88824,12 +88824,17 @@ const getIntervalKey = (invalidationIntervalDays) => {
 };
 async function buildCacheKeys() {
     const keys = [];
-    const invalidationIntervalDays = parseInt(core.getInput(`cache-invalidation-interval`, { required: true }).trim());
-    // Periodically invalidate a cache because a new code being added.
-    let cacheKey = `golangci-lint.cache-${getIntervalKey(invalidationIntervalDays)}-`;
-    keys.push(cacheKey);
+    // Cache by OS.
+    let cacheKey = `golangci-lint.cache-${process.env?.RUNNER_OS}-`;
     // Get working directory from input
     const workingDirectory = core.getInput(`working-directory`);
+    if (workingDirectory) {
+        cacheKey += `${workingDirectory}-`;
+    }
+    // Periodically invalidate a cache because a new code being added.
+    const invalidationIntervalDays = parseInt(core.getInput(`cache-invalidation-interval`, { required: true }).trim());
+    cacheKey += `${getIntervalKey(invalidationIntervalDays)}-`;
+    keys.push(cacheKey);
     // create path to go.mod prepending the workingDirectory if it exists
     const goModPath = path_1.default.join(workingDirectory, `go.mod`);
     core.info(`Checking for go.mod: ${goModPath}`);
diff --git a/dist/run/index.js b/dist/run/index.js
index b67b479635..428acc2f77 100644
--- a/dist/run/index.js
+++ b/dist/run/index.js
@@ -88824,12 +88824,17 @@ const getIntervalKey = (invalidationIntervalDays) => {
 };
 async function buildCacheKeys() {
     const keys = [];
-    const invalidationIntervalDays = parseInt(core.getInput(`cache-invalidation-interval`, { required: true }).trim());
-    // Periodically invalidate a cache because a new code being added.
-    let cacheKey = `golangci-lint.cache-${getIntervalKey(invalidationIntervalDays)}-`;
-    keys.push(cacheKey);
+    // Cache by OS.
+    let cacheKey = `golangci-lint.cache-${process.env?.RUNNER_OS}-`;
     // Get working directory from input
     const workingDirectory = core.getInput(`working-directory`);
+    if (workingDirectory) {
+        cacheKey += `${workingDirectory}-`;
+    }
+    // Periodically invalidate a cache because a new code being added.
+    const invalidationIntervalDays = parseInt(core.getInput(`cache-invalidation-interval`, { required: true }).trim());
+    cacheKey += `${getIntervalKey(invalidationIntervalDays)}-`;
+    keys.push(cacheKey);
     // create path to go.mod prepending the workingDirectory if it exists
     const goModPath = path_1.default.join(workingDirectory, `go.mod`);
     core.info(`Checking for go.mod: ${goModPath}`);
diff --git a/src/cache.ts b/src/cache.ts
index 314647d6a7..dc6aa06b14 100644
--- a/src/cache.ts
+++ b/src/cache.ts
@@ -38,15 +38,22 @@ const getIntervalKey = (invalidationIntervalDays: number): string => {
 async function buildCacheKeys(): Promise<string[]> {
   const keys = []
 
-  const invalidationIntervalDays = parseInt(core.getInput(`cache-invalidation-interval`, { required: true }).trim())
-
-  // Periodically invalidate a cache because a new code being added.
-  let cacheKey = `golangci-lint.cache-${getIntervalKey(invalidationIntervalDays)}-`
-  keys.push(cacheKey)
+  // Cache by OS.
+  let cacheKey = `golangci-lint.cache-${process.env?.RUNNER_OS}-`
 
   // Get working directory from input
   const workingDirectory = core.getInput(`working-directory`)
 
+  if (workingDirectory) {
+    cacheKey += `${workingDirectory}-`
+  }
+
+  // Periodically invalidate a cache because a new code being added.
+  const invalidationIntervalDays = parseInt(core.getInput(`cache-invalidation-interval`, { required: true }).trim())
+  cacheKey += `${getIntervalKey(invalidationIntervalDays)}-`
+
+  keys.push(cacheKey)
+
   // create path to go.mod prepending the workingDirectory if it exists
   const goModPath = path.join(workingDirectory, `go.mod`)
 

From 38e1018663fa5173f3968ea0777460d3de38f256 Mon Sep 17 00:00:00 2001
From: Ludovic Fernandez <ldez@users.noreply.github.com>
Date: Sun, 5 May 2024 01:13:21 +0200
Subject: [PATCH 14/14] feat: improve log about pwd/cwd (#1033)

---
 dist/post_run/index.js | 4 ++--
 dist/run/index.js      | 4 ++--
 src/run.ts             | 5 +++--
 3 files changed, 7 insertions(+), 6 deletions(-)

diff --git a/dist/post_run/index.js b/dist/post_run/index.js
index 23f6c001c0..87f67815a6 100644
--- a/dist/post_run/index.js
+++ b/dist/post_run/index.js
@@ -89329,8 +89329,8 @@ async function runLint(lintPath, patchPath) {
                 break;
         }
     }
-    const workingDirectory = core.getInput(`working-directory`);
     const cmdArgs = {};
+    const workingDirectory = core.getInput(`working-directory`);
     if (workingDirectory) {
         if (!fs.existsSync(workingDirectory) || !fs.lstatSync(workingDirectory).isDirectory()) {
             throw new Error(`working-directory (${workingDirectory}) was not a path`);
@@ -89341,7 +89341,7 @@ async function runLint(lintPath, patchPath) {
         cmdArgs.cwd = path.resolve(workingDirectory);
     }
     const cmd = `${lintPath} run ${addedArgs.join(` `)} ${userArgs}`.trimEnd();
-    core.info(`Running [${cmd}] in [${cmdArgs.cwd || ``}] ...`);
+    core.info(`Running [${cmd}] in [${cmdArgs.cwd || process.cwd()}] ...`);
     const startedAt = Date.now();
     try {
         const res = await execShellCommand(cmd, cmdArgs);
diff --git a/dist/run/index.js b/dist/run/index.js
index 428acc2f77..11ef531598 100644
--- a/dist/run/index.js
+++ b/dist/run/index.js
@@ -89329,8 +89329,8 @@ async function runLint(lintPath, patchPath) {
                 break;
         }
     }
-    const workingDirectory = core.getInput(`working-directory`);
     const cmdArgs = {};
+    const workingDirectory = core.getInput(`working-directory`);
     if (workingDirectory) {
         if (!fs.existsSync(workingDirectory) || !fs.lstatSync(workingDirectory).isDirectory()) {
             throw new Error(`working-directory (${workingDirectory}) was not a path`);
@@ -89341,7 +89341,7 @@ async function runLint(lintPath, patchPath) {
         cmdArgs.cwd = path.resolve(workingDirectory);
     }
     const cmd = `${lintPath} run ${addedArgs.join(` `)} ${userArgs}`.trimEnd();
-    core.info(`Running [${cmd}] in [${cmdArgs.cwd || ``}] ...`);
+    core.info(`Running [${cmd}] in [${cmdArgs.cwd || process.cwd()}] ...`);
     const startedAt = Date.now();
     try {
         const res = await execShellCommand(cmd, cmdArgs);
diff --git a/src/run.ts b/src/run.ts
index 48550d4576..9cffa9c690 100644
--- a/src/run.ts
+++ b/src/run.ts
@@ -233,8 +233,9 @@ async function runLint(lintPath: string, patchPath: string): Promise<void> {
     }
   }
 
-  const workingDirectory = core.getInput(`working-directory`)
   const cmdArgs: ExecOptions = {}
+
+  const workingDirectory = core.getInput(`working-directory`)
   if (workingDirectory) {
     if (!fs.existsSync(workingDirectory) || !fs.lstatSync(workingDirectory).isDirectory()) {
       throw new Error(`working-directory (${workingDirectory}) was not a path`)
@@ -247,7 +248,7 @@ async function runLint(lintPath: string, patchPath: string): Promise<void> {
 
   const cmd = `${lintPath} run ${addedArgs.join(` `)} ${userArgs}`.trimEnd()
 
-  core.info(`Running [${cmd}] in [${cmdArgs.cwd || ``}] ...`)
+  core.info(`Running [${cmd}] in [${cmdArgs.cwd || process.cwd()}] ...`)
 
   const startedAt = Date.now()
   try {