feat: allow to skip golangci-lint installation (#1079)

Co-authored-by: Fernandez Ludovic <ldez@users.noreply.github.com>

.github/CONTRIBUTING.md

@@ -0,0 +1,25 @@
+## How to contribute
+
+### Did you find a bug?
+
+* **Ensure the bug was not already reported** by searching on GitHub under [Issues](https://github.com/golangci/golangci-lint-action/issues).
+
+* If you're unable to find an open issue addressing the problem, [open a new one](https://github.com/golangci/golangci-lint-action/issues/new).
+  Be sure to include a **title and clear description**, as much relevant information as possible,
+  and a **code sample** or an **executable test case** demonstrating the expected behavior that is not occurring.
+
+* **Do not open up a GitHub issue if the bug is a security vulnerability**,
+  and instead to refer to our [security policy](https://github.com/golangci/golangci-lint-action?tab=security-ov-file).
+
+### Do you intend to add a new feature or change an existing one?
+
+* Suggest your change inside an [issue](https://github.com/golangci/golangci-lint-action/issues).
+
+* Do not open a pull request on GitHub until you have collected positive feedback about the change.
+
+### Did you write a patch that fixes a bug?
+
+* Open a new GitHub pull request with the patch.
+
+* Ensure the PR description clearly describes the problem and solution.
+  Include the relevant issue number if applicable.

.github/ISSUE_TEMPLATE/bug_report.yml

@@ -1,4 +1,4 @@
-name: Bug Report
+name: 🐞 Bug Report
 description: "Create a report to help us improve."
 body:
   - type: checkboxes
@@ -36,14 +36,29 @@ body:
       required: true
 
   - type: textarea
-    id: config
+    id: workflow-file
     attributes:
       label: Workflow file
       value: |-
         <details>
 
+        ```yml
+        <add your file content here>
         ```
-        <add you file here>
+
+        </details>
+    validations:
+      required: true
+
+  - type: textarea
+    id: config
+    attributes:
+      label: Golangci-lint configuration
+      value: |-
+        <details>
+
+        ```yml
+        <add your file content here>
         ```
 
         </details>

.github/ISSUE_TEMPLATE/config.yml

@@ -1,11 +1,14 @@
 blank_issues_enabled: false
 contact_links:
-  - name: Questions
+  - name: 📖 Golangci-lint documentation
-    url: https://github.com/golangci/golangci-lint-action/discussions
-    about: If you have a question, or are looking for advice, please post on our Discussions forum!
-  - name: golangci-lint main repository
-    url: https://github.com/golangci/golangci-lint
-    about: The main repository of golangci-lint.
-  - name: golangci-lint documentation
     url: https://golangci-lint.run
     about: Please take a look to our documentation.
+  - name: ❓ Questions
+    url: https://github.com/golangci/golangci-lint-action/discussions
+    about: If you have a question, or are looking for advice, please post on our Discussions forum!
+  - name: 💬 Chat on Slack
+    url: https://gophers.slack.com/archives/CS0TBRKPC
+    about: Maybe chatting with the community can help
+  - name: 🏡 Golangci-lint main repository
+    url: https://github.com/golangci/golangci-lint
+    about: The main repository of golangci-lint.

.github/ISSUE_TEMPLATE/feature_request.yml

@@ -1,4 +1,4 @@
-name: Feature request
+name: 💡 Feature request
 description: "Suggest an idea for this project."
 body:
   - type: checkboxes

.github/dependabot.yml

@@ -6,5 +6,10 @@ updates:
       interval: weekly
   - package-ecosystem: npm
     directory: "/"
+    groups:
+      dev-dependencies:
+        dependency-type: development
+      dependencies:
+        dependency-type: production
     schedule:
       interval: weekly

README.md

@@ -8,19 +8,28 @@ The action runs [golangci-lint](https://github.com/golangci/golangci-lint) and r
 
 ![GitHub Annotations](./static/annotations.png)
 
-## Compatibility
+![Logs](./static/colored-line-number.png)
 
-* `v5.0.0+` removes `skip-pkg-cache` and `skip-build-cache` because the cache related to Go itself is already handled by `actions/setup-go`.
+## Supporting Us
-* `v4.0.0+` requires an explicit `actions/setup-go` installation step before using this action: `uses: actions/setup-go@v5`.
+
-  The `skip-go-installation` option has been removed.
+<!-- [![GitHub Sponsors](https://img.shields.io/badge/GitHub-Donate-blue?logo=github&style=for-the-badge)](https://github.com/sponsors/golangci) -->
-* `v2.0.0+` works with `golangci-lint` version >= `v1.28.3`
+[![Open Collective backers and sponsors](https://img.shields.io/badge/OpenCollective-Donate-blue?logo=opencollective&style=for-the-badge)](https://opencollective.com/golangci-lint)
-* `v1.2.2` is deprecated due to we forgot to change the minimum version of `golangci-lint` to `v1.28.3` ([issue](https://github.com/golangci/golangci-lint-action/issues/39))
+[![Linter Authors](https://img.shields.io/badge/Linter_Authors-Donate-blue?style=for-the-badge)](https://golangci-lint.run/product/thanks/)
-* `v1.2.1` works with `golangci-lint` version >= `v1.14.0` ([issue](https://github.com/golangci/golangci-lint-action/issues/39))
+
+`golangci-lint` is a free and open-source project built by volunteers.
+
+If you value it, consider supporting us, we appreciate it! :heart:
 
 ## How to use
 
+We recommend running this action in a job separate from other jobs (`go test`, etc.)
+because different jobs [run in parallel](https://help.github.com/en/actions/getting-started-with-github-actions/core-concepts-for-github-actions#job).
+
 Add `.github/workflows/golangci-lint.yml` with the following contents:
 
+<details>
+<summary>Simple Example</summary>
+
 ```yaml
 name: golangci-lint
 on:
@@ -43,21 +52,17 @@ jobs:
       - uses: actions/checkout@v4
       - uses: actions/setup-go@v5
         with:
-          go-version: '1.22'
+          go-version: stable
       - name: golangci-lint
-        uses: golangci/golangci-lint-action@v5
+        uses: golangci/golangci-lint-action@v6
         with:
-          version: latest
+          version: v1.59
 ```
 
-We recommend running this action in a job separate from other jobs (`go test`, etc.)
+</details>
-because different jobs [run in parallel](https://help.github.com/en/actions/getting-started-with-github-actions/core-concepts-for-github-actions#job).
 
-### Multiple OS Support
+<details>
-
+<summary>Multiple OS Example</summary>
-If you need to run linters for specific operating systems, you will need to use the action `>=v2`.
-
-Here is a sample configuration file:
 
 ```yaml
 name: golangci-lint
@@ -77,7 +82,7 @@ jobs:
   golangci:
     strategy:
       matrix:
-        go: ['1.22']
+        go: [stable]
         os: [ubuntu-latest, macos-latest, windows-latest]
     name: lint
     runs-on: ${{ matrix.os }}
@@ -87,9 +92,9 @@ jobs:
         with:
           go-version: ${{ matrix.go }}
       - name: golangci-lint
-        uses: golangci/golangci-lint-action@v5
+        uses: golangci/golangci-lint-action@v6
         with:
-          version: latest
+          version: v1.59
 ```
 
 You will also likely need to add the following `.gitattributes` file to ensure that line endings for Windows builds are properly formatted:
@@ -98,93 +103,373 @@ You will also likely need to add the following `.gitattributes` file to ensure t
 *.go text eol=lf
 ```
 
+</details>
+
+<details>
+<summary>Go Workspace Example</summary>
+
+```yaml
+name: golangci-lint
+
+on:
+  pull_request:
+  push:
+    branches:
+      - "main"
+      - "master"
+
+env:
+  GO_VERSION: stable
+  GOLANGCI_LINT_VERSION: v1.59
+
+jobs:
+  detect-modules:
+    runs-on: ubuntu-latest
+    outputs:
+      modules: ${{ steps.set-modules.outputs.modules }}
+    steps:
+      - uses: actions/checkout@v4
+      - uses: actions/setup-go@v5
+        with:
+          go-version: ${{ env.GO_VERSION }}
+      - id: set-modules
+        run: echo "modules=$(go list -m -json | jq -s '.' | jq -c '[.[].Dir]')" >> $GITHUB_OUTPUT
+
+  golangci-lint:
+    needs: detect-modules
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        modules: ${{ fromJSON(needs.detect-modules.outputs.modules) }}
+    steps:
+      - uses: actions/checkout@v4
+      - uses: actions/setup-go@v5
+        with:
+          go-version: ${{ env.GO_VERSION }}
+      - name: golangci-lint ${{ matrix.modules }}
+        uses: golangci/golangci-lint-action@v6
+        with:
+          version: ${{ env.GOLANGCI_LINT_VERSION }}
+          working-directory: ${{ matrix.modules }}
+```
+
+</details>
+
+<details>
+<summary>Go Workspace Example (Multiple OS)</summary>
+
+```yaml
+# ./.github/workflows/golangci-lint.yml
+name: golangci-lint (multi OS)
+
+on:
+  pull_request:
+  push:
+    branches:
+      - "main"
+      - "master"
+
+jobs:
+  golangci-lint:
+    strategy:
+      matrix:
+        go-version: [ stable, oldstable ]
+        os: [ubuntu-latest, macos-latest, windows-latest]
+    uses: ./.github/workflows/.golangci-lint-reusable.yml
+    with:
+      os: ${{ matrix.os }}
+      go-version: ${{ matrix.go-version }}
+      golangci-lint-version: v1.59
+```
+
+```yaml
+# ./.github/workflows/.golangci-lint-reusable.yml
+name: golangci-lint-reusable
+
+on:
+  workflow_call:
+    inputs:
+      os:
+        description: 'OS'
+        required: true
+        type: string
+      go-version:
+        description: 'Go version'
+        required: true
+        type: string
+        default: stable
+      golangci-lint-version:
+        description: 'Golangci-lint version'
+        type: string
+        default: 'v1.59.1'
+
+jobs:
+  detect-modules:
+    runs-on: ${{ inputs.os }}
+    outputs:
+      modules: ${{ steps.set-modules.outputs.modules }}
+    steps:
+      - uses: actions/checkout@v4
+      - uses: actions/setup-go@v5
+        with:
+          go-version: ${{ inputs.go-version }}
+      - id: set-modules
+        shell: bash # require for Windows to be able to use $GITHUB_OUTPUT https://github.com/actions/runner/issues/2224
+        run: echo "modules=$(go list -m -json | jq -s '.' | jq -c '[.[].Dir]')" >> $GITHUB_OUTPUT
+
+  golangci-lint:
+    needs: detect-modules
+    runs-on: ${{ inputs.os }}
+    strategy:
+      matrix:
+        modules: ${{ fromJSON(needs.detect-modules.outputs.modules) }}
+    steps:
+      - uses: actions/checkout@v4
+      - uses: actions/setup-go@v5
+        with:
+          go-version: ${{ inputs.go-version }}
+      - name: golangci-lint ${{ matrix.modules }}
+        uses: golangci/golangci-lint-action@v6
+        with:
+          version: ${{ inputs.golangci-lint-version }}
+          working-directory: ${{ matrix.modules }}
+```
+
+You will also likely need to add the following `.gitattributes` file to ensure that line endings for Windows builds are properly formatted:
+
+```.gitattributes
+*.go text eol=lf
+```
+
+</details>
+
+## Compatibility
+
+* `v6.0.0+` removes `annotations` option, removes the default output format (`github-actions`).
+* `v5.0.0+` removes `skip-pkg-cache` and `skip-build-cache` because the cache related to Go itself is already handled by `actions/setup-go`.
+* `v4.0.0+` requires an explicit `actions/setup-go` installation step before using this action: `uses: actions/setup-go@v5`.
+  The `skip-go-installation` option has been removed.
+* `v2.0.0+` works with `golangci-lint` version >= `v1.28.3`
+* `v1.2.2` is deprecated due to we forgot to change the minimum version of `golangci-lint` to `v1.28.3` ([issue](https://github.com/golangci/golangci-lint-action/issues/39))
+* `v1.2.1` works with `golangci-lint` version >= `v1.14.0` ([issue](https://github.com/golangci/golangci-lint-action/issues/39))
+
 ## Options
 
-`version`: (required) The version of golangci-lint to use.
+### `version`
-* 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)
+
+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.
+* `goinstall`: the value can be v1.2.3, `latest`, or the hash of a commit.
+* `none`: the value is ignored.
+
+<details>
+<summary>Example</summary>
 
 ```yml
-uses: golangci/golangci-lint-action@v5
+uses: golangci/golangci-lint-action@v6
 with:
-  version: latest
+  version: v1.58
   # ...
 ```
 
-`install-mode`: (optional) The mode to install golangci-lint.
+</details>
-It can be `binary` or `goinstall`.
+
+### `install-mode`
+
+(optional)
+
+The mode to install golangci-lint: it can be `binary`, `goinstall`, or `none`.
+
 The default value is `binary`.
 
+<details>
+<summary>Example</summary>
+
 ```yml
-uses: golangci/golangci-lint-action@v5
+uses: golangci/golangci-lint-action@v6
 with:
   install-mode: "goinstall"
   # ...
 ```
 
-`only-new-issues`: (optional) Show only new issues.
+</details>
-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`.
+### `github-token`
+
+(optional)
+
+When using `only-new-issues` option, the GitHub API is used, so a token is required.
+
+By default, it uses the `github.token` from the action.
+
+<details>
+<summary>Example</summary>
 
 ```yml
-uses: golangci/golangci-lint-action@v5
+uses: golangci/golangci-lint-action@v6
+with:
+  github-token: xxx
+  # ...
+```
+
+</details>
+
+### `only-new-issues`
+
+(optional)
+
+Show only new issues.
+
+The default value is `false`.
+
+* `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.
+
+<details>
+<summary>Example</summary>
+
+```yml
+uses: golangci/golangci-lint-action@v6
 with:
   only-new-issues: true
   # ...
 ```
 
-`working-directory`: (optional) working directory, useful for monorepos.
+</details>
+
+### `working-directory`
+
+(optional)
+
+Working directory, useful for monorepos.
+
+<details>
+<summary>Example</summary>
 
 ```yml
-uses: golangci/golangci-lint-action@v5
+uses: golangci/golangci-lint-action@v6
 with:
   working-directory: somedir
   # ...
 ```
 
-`skip-cache`: (optional) If set to `true`, then all caching functionality will be completely disabled,
+</details>
-takes precedence over all other caching options.
+
-The default value is `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=`
+
+<details>
+<summary>Example</summary>
 
 ```yml
-uses: golangci/golangci-lint-action@v5
+uses: golangci/golangci-lint-action@v6
+with:
+  args: --timeout=30m --config=/my/path/.golangci.yml --issues-exit-code=0
+  # ...
+```
+
+</details>
+
+### `problem-matchers`
+
+(optional)
+
+Force the usage of the embedded problem matchers.
+
+By default, the [problem matcher of Go (`actions/setup-go`)](https://github.com/actions/setup-go/blob/main/matchers.json) already handles the golangci-lint output (`colored-line-number`).
+
+Works only with `colored-line-number` (the golangci-lint default).
+
+https://golangci-lint.run/usage/configuration/#output-configuration
+
+The default value is `false`.
+
+<details>
+<summary>Example</summary>
+
+```yml
+uses: golangci/golangci-lint-action@v6
+with:
+  problem-matchers: true
+  # ...
+```
+
+</details>
+
+### `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`.
+
+<details>
+<summary>Example</summary>
+
+```yml
+uses: golangci/golangci-lint-action@v6
 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`.
+</details>
+
+### `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`.
 
+<details>
+<summary>Example</summary>
+
 ```yml
-uses: golangci/golangci-lint-action@v5
+uses: golangci/golangci-lint-action@v6
 with:
   skip-save-cache: true
   # ...
 ```
 
-`annotations`: (optional) To enable/disable GitHub Action annotations.
+</details>
-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
+### `cache-invalidation-interval`
-The default value is `true`.
+
+(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`.
+
+If set the number is `<= 0`, the cache will be always invalidate (Not recommended).
+
+<details>
+<summary>Example</summary>
 
 ```yml
-uses: golangci/golangci-lint-action@v5
+uses: golangci/golangci-lint-action@v6
 with:
-  annotations: false
+  cache-invalidation-interval: 15
   # ...
 ```
 
-`args`: (optional) golangci-lint command line arguments.
+</details>
-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
 
@@ -246,10 +531,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.

action.yml

@@ -6,11 +6,13 @@ inputs:
   version:
     description: |
       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:
-      When `install-mode` is `goinstall` the value can be v1.2.3, `latest`, or the hash of a commit.
+      - `binary` (default): the value can be v1.2 or v1.2.3 or `latest` to use the latest version.
+      - `goinstall`: the value can be v1.2.3, `latest`, or the hash of a commit.
+      - `none`: the value is ignored.
     required: false
   install-mode:
-    description: "The mode to install golangci-lint. It can be 'binary' or 'goinstall'."
+    description: "The mode to install golangci-lint. It can be 'binary', 'goinstall', or 'none'."
     default: "binary"
     required: false
   working-directory:
@@ -36,14 +38,18 @@ inputs:
       restore existing caches, subject to other options.
     default: 'false'
     required: false
-  annotations:
+  problem-matchers:
-    description: "To Enable/disable GitHub Action annotations"
+    description: "Force the usage of the embedded problem matchers"
-    default: 'true'
+    default: 'false'
     required: false
   args:
     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"

package.json

@@ -30,21 +30,23 @@
     "@actions/github": "^6.0.0",
     "@actions/http-client": "^2.2.1",
     "@actions/tool-cache": "^2.0.1",
-    "@types/node": "^20.12.7",
+    "@types/node": "^22.0.0",
     "@types/semver": "^7.5.8",
     "@types/tmp": "^0.2.6",
-    "tmp": "^0.2.3"
+    "@types/which": "^3.0.4",
+    "tmp": "^0.2.3",
+    "which": "^4.0.0"
   },
   "devDependencies": {
-    "@typescript-eslint/eslint-plugin": "^7.7.1",
+    "@typescript-eslint/eslint-plugin": "^7.17.0",
-    "@typescript-eslint/parser": "^7.7.1",
+    "@typescript-eslint/parser": "^7.17.0",
     "@vercel/ncc": "^0.38.1",
     "eslint": "^8.57.0",
     "eslint-config-prettier": "^9.1.0",
     "eslint-plugin-import": "^2.29.1",
-    "eslint-plugin-prettier": "^5.1.3",
+    "eslint-plugin-prettier": "^5.2.1",
-    "eslint-plugin-simple-import-sort": "^12.1.0",
+    "eslint-plugin-simple-import-sort": "^12.1.1",
-    "prettier": "^3.2.5",
+    "prettier": "^3.3.3",
-    "typescript": "^5.4.5"
+    "typescript": "^5.5.4"
   }
 }

problem-matchers.json

@@ -0,0 +1,17 @@
+{
+  "problemMatcher": [
+    {
+      "owner": "golangci-lint-colored-line-number",
+      "severity": "error",
+      "pattern": [
+        {
+          "regexp": "^([^:]+):(\\d+):(?:(\\d+):)?\\s+(.+ \\(.+\\))$",
+          "file": 1,
+          "line": 2,
+          "column": 3,
+          "message": 4
+        }
+      ]
+    }
+  ]
+}

src/cache.ts

@@ -20,11 +20,18 @@ function checksumFile(hashName: string, path: string): Promise<string> {
 const pathExists = async (path: string): Promise<boolean> => !!(await fs.promises.stat(path).catch(() => false))
 
 const getLintCacheDir = (): string => {
-  return path.resolve(`${process.env.HOME}/.cache/golangci-lint`)
+  const home = process.platform === "win32" ? process.env.USERPROFILE : process.env.HOME
+
+  return path.resolve(`${home}`, `.cache`, `golangci-lint`)
 }
 
 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,28 +39,42 @@ const getIntervalKey = (invalidationIntervalDays: number): string => {
 
 async function buildCacheKeys(): Promise<string[]> {
   const keys = []
-  // Periodically invalidate a cache because a new code being added.
+
-  // TODO: configure it via inputs.
+  // Cache by OS.
-  let cacheKey = `golangci-lint.cache-${getIntervalKey(7)}-`
+  let cacheKey = `golangci-lint.cache-${process.env?.RUNNER_OS}-`
-  keys.push(cacheKey)
+
   // 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`)
+
   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
 }
 
 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(
@@ -95,8 +116,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.getBooleanInput(`skip-cache`, { required: true })) return
-  if (core.getInput(`skip-save-cache`, { required: true }).trim() == "true") return
+  if (core.getBooleanInput(`skip-save-cache`, { required: true })) return
 
   // Validate inputs, this can cause task failure
   if (!utils.isValidEvent()) {

src/install.ts

@@ -38,6 +38,7 @@ const getAssetURL = (versionConfig: VersionConfig): string => {
 export enum InstallMode {
   Binary = "binary",
   GoInstall = "goinstall",
+  None = "none",
 }
 
 type ExecRes = {

src/run.ts

@@ -6,6 +6,7 @@ import * as fs from "fs"
 import * as path from "path"
 import { dir } from "tmp"
 import { promisify } from "util"
+import which from "which"
 
 import { restoreCache, saveCache } from "./cache"
 import { installLint, InstallMode } from "./install"
@@ -17,17 +18,20 @@ const writeFile = promisify(fs.writeFile)
 const createTempDir = promisify(dir)
 
 function isOnlyNewIssues(): boolean {
-  const onlyNewIssues = core.getInput(`only-new-issues`, { required: true }).trim()
+  return core.getBooleanInput(`only-new-issues`, { required: true })
-
-  if (onlyNewIssues !== `false` && onlyNewIssues !== `true`) {
-    throw new Error(`invalid value of "only-new-issues": "${onlyNewIssues}", expected "true" or "false"`)
-  }
-
-  return onlyNewIssues === `true`
 }
 
 async function prepareLint(): Promise<string> {
   const mode = core.getInput("install-mode").toLowerCase()
+
+  if (mode === InstallMode.None) {
+    const bin = await which("golangci-lint", { nothrow: true })
+    if (!bin) {
+      throw new Error("golangci-lint binary not found in the PATH")
+    }
+    return bin
+  }
+
   const versionConfig = await findLintVersion(<InstallMode>mode)
 
   return await installLint(versionConfig, <InstallMode>mode)
@@ -47,7 +51,6 @@ async function fetchPatch(): Promise<string> {
     case `push`:
       return await fetchPushPatch(ctx)
     case `merge_group`:
-      core.info(JSON.stringify(ctx.payload))
       return ``
     default:
       core.info(`Not fetching patch for showing only new issues because it's not a pull request context: event name is ${ctx.eventName}`)
@@ -104,11 +107,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,
+      basehead: `${ctx.payload.before}...${ctx.payload.after}`,
-      head: ctx.payload.after,
       mediaType: {
         format: `diff`,
       },
@@ -192,21 +194,32 @@ 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 problemMatchers = core.getBooleanInput(`problem-matchers`)
 
-  if (annotations) {
+  if (problemMatchers) {
-    const formats = (userArgsMap.get("out-format") || "")
+    const matchersPath = path.join(__dirname, "../..", "problem-matchers.json")
-      .trim()
+    if (fs.existsSync(matchersPath)) {
-      .split(",")
+      // Adds problem matchers.
-      .filter((f) => f.length > 0)
+      // https://github.com/actions/setup-go/blob/cdcb36043654635271a94b9a6d1392de5bb323a7/src/main.ts#L81-L83
-      .filter((f) => !f.startsWith(`github-actions`))
+      core.info(`##[add-matcher]${matchersPath}`)
-      .concat("github-actions")
+    }
-      .join(",")
-
-    addedArgs.push(`--out-format=${formats}`)
-    userArgs = userArgs.replace(/--out-format=\S*/gi, "").trim()
   }
 
+  const formats = (userArgsMap.get("out-format") || "")
+    .trim()
+    .split(",")
+    .filter((f) => f.length > 0)
+    .filter((f) => !f.startsWith(`github-actions`)) // Removes `github-actions` format.
+    .join(",")
+
+  if (formats) {
+    // Adds formats but without `github-actions` format.
+    addedArgs.push(`--out-format=${formats}`)
+  }
+
+  // Removes `--out-format` from the user flags because it's already inside `addedArgs`.
+  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`)
@@ -240,8 +253,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`)
@@ -254,7 +268,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 {