test: don't check pnpx

This is a breaking change because the binary version of pnpm
is only available from v6.17.1

close #18

.github/dependabot.yml

@@ -0,0 +1,10 @@
+version: 2
+updates:
+  - package-ecosystem: github-actions
+    directory: "/"
+    schedule:
+      interval: weekly
+    open-pull-requests-limit: 10
+    labels:
+    - dependabot
+    - github-actions

.github/workflows/test.yaml

@@ -3,6 +3,7 @@ name: Test Action
 on:
   - push
   - pull_request
+  - workflow_dispatch
 
 jobs:
   test_default_inputs:
@@ -14,7 +15,7 @@ jobs:
       fail-fast: false
       matrix:
         pnpm:
-          - 4.11.1
+          - 6.17.1
         os:
           - ubuntu-latest
           - macos-latest
@@ -26,10 +27,10 @@ jobs:
       - name: Run the action
         uses: ./
         with:
-          version: 4.11.1
+          version: 6.17.1
 
       - name: 'Test: which'
-        run: which pnpm; which pnpx
+        run: which pnpm
 
       - name: 'Test: install'
         run: pnpm install
@@ -43,7 +44,7 @@ jobs:
       fail-fast: false
       matrix:
         pnpm:
-          - 4.11.1
+          - 6.17.1
         os:
           - ubuntu-latest
           - macos-latest
@@ -55,13 +56,11 @@ jobs:
       - name: Run the action
         uses: ./
         with:
-          version: 4.11.1
+          version: 6.17.1
           dest: ~/test/pnpm
-          bin_dest: ~/test/pnpm/.bin
-          registry: http://registry.yarnpkg.com/
 
       - name: 'Test: which'
-        run: which pnpm && which pnpx
+        run: which pnpm
 
       - name: 'Test: install'
         run: pnpm install
@@ -75,7 +74,7 @@ jobs:
       fail-fast: false
       matrix:
         pnpm:
-          - 4.11.1
+          - 6.17.1
         os:
           - ubuntu-latest
           - macos-latest
@@ -113,11 +112,11 @@ jobs:
       - name: Run the action
         uses: ./
         with:
-          version: 4.11.1
+          version: 6.17.1
           run_install: ${{ matrix.run_install.value }}
 
       - name: 'Test: which'
-        run: which pnpm; which pnpx
+        run: which pnpm
 
       - name: 'Test: install'
         run: pnpm install

.gitignore

@@ -8,3 +8,4 @@ temp
 *.temp
 tmp.*
 temp.*
+.pnpm-store

README.md

@@ -1,24 +1,16 @@
-# Setup PNPM
+# Setup pnpm
 
-Install PNPM package manager.
+Install pnpm package manager.
 
 ## Inputs
 
 ### `version`
 
-**Required** Version of PNPM to install.
+**Required** Version of pnpm to install. It supports npm versioning scheme, it could be an exact version (such as `6.24.1`), or a version range (such as `6`, `6.x.x`, `6.24.x`, `^6.24.1`, `*`, etc.), or `latest`.
 
 ### `dest`
 
-**Optional** Where to store PNPM files.
+**Optional** Where to store pnpm files.
-
-### `bin_dest`
-
-**Optional** Where to store executables (`pnpm` and `pnpx` commands).
-
-### `registry`
-
-**Optional** (_default:_ `https://registry.npmjs.com`) Registry to download PNPM from.
 
 ### `run_install`
 
@@ -50,11 +42,11 @@ Expanded path of inputs#dest.
 
 ### `bin_dest`
 
-Expanded path of inputs#bin_dest.
+Location of `pnpm` and `pnpx` command.
 
 ## Usage example
 
-### Just install PNPM
+### Just install pnpm
 
 ```yaml
 on:
@@ -62,15 +54,16 @@ on:
   - pull_request
 
 jobs:
-  runs-on: ubuntu-latest
+  install:
+    runs-on: ubuntu-latest
 
-  steps:
+    steps:
-    - uses: pnpm/action-setup@v1.2.0
+      - uses: pnpm/action-setup@v2.1.0
-      with:
+        with:
-        version: 4.11.1
+          version: 6.0.2
 ```
 
-### Install PNPM and a few NPM packages
+### Install pnpm and a few npm packages
 
 ```yaml
 on:
@@ -78,20 +71,52 @@ on:
   - pull_request
 
 jobs:
-  runs-on: ubuntu-latest
+  install:
+    runs-on: ubuntu-latest
 
-  steps:
+    steps:
-    - uses: actions/checkout@v2
+      - uses: actions/checkout@v2
 
-    - uses: pnpm/action-setup@v1.2.0
+      - uses: pnpm/action-setup@v2.1.0
-      with:
+        with:
-        version: 4.11.1
+          version: 6.0.2
-        run_install: |
+          run_install: |
-          - recursive: true
+            - recursive: true
-            args: [--frozen-lockfile, --strict-peer-dependencies]
+              args: [--frozen-lockfile, --strict-peer-dependencies]
-          - args: [--global, gulp, prettier, typescript]
+            - args: [--global, gulp, prettier, typescript]
 ```
 
+### Use cache to reduce installation time
+
+```yaml
+on:
+  - push
+  - pull_request
+
+jobs:
+  cache-and-install:
+    runs-on: ubuntu-latest
+
+    steps:
+      build:
+        - uses: actions/checkout@v2
+
+        - name: Cache pnpm modules
+          uses: actions/cache@v2
+          with:
+            path: ~/.pnpm-store
+            key: ${{ runner.os }}-${{ hashFiles('**/pnpm-lock.yaml') }}
+            restore-keys: |
+              ${{ runner.os }}-
+
+        - uses: pnpm/action-setup@v2.1.0
+          with:
+            version: 6.0.2
+            run_install: true
+```
+
+**Note:** You don't need to run `pnpm store prune` at the end; post-action has already taken care of that.
+
 ## Notes
 
 This action does not setup Node.js for you, use [actions/setup-node](https://github.com/actions/setup-node) yourself.

action.yml

@@ -11,14 +11,6 @@ inputs:
     description: Where to store PNPM files
     required: false
     default: ~/setup-pnpm
-  bin_dest:
-    description: Where to store executables (pnpm and pnpx commands)
-    required: false
-    default: ~/setup-pnpm/.bin
-  registry:
-    description: Registry to download PNPM from
-    required: false
-    default: https://registry.npmjs.com
   run_install:
     description: If specified, run `pnpm install`
     required: false

package.json

@@ -7,19 +7,21 @@
     "start": "pnpm run build && sh ./run.sh"
   },
   "dependencies": {
-    "node-fetch": "^2.6.0",
+    "@actions/core": "^1.6.0",
-    "expand-tilde": "^2.0.2",
-    "js-yaml": "^3.13.1",
-    "ajv": "^6.12.0",
-    "@actions/core": "^1.2.4",
     "@types/expand-tilde": "^2.0.0",
-    "@types/node-fetch": "^2.5.7",
+    "@types/fs-extra": "^9.0.13",
-    "@types/js-yaml": "^3.12.3",
+    "@types/js-yaml": "^4.0.5",
-    "@types/node": "^13.13.5"
+    "@types/node": "^14.18.10",
+    "@types/node-fetch": "^2.5.12",
+    "ajv": "^6.12.6",
+    "expand-tilde": "^2.0.2",
+    "fs-extra": "^9.1.0",
+    "js-yaml": "^4.1.0",
+    "node-fetch": "^2.6.7"
   },
   "devDependencies": {
-    "typescript": "^3.8.3",
     "@ts-schema-autogen/cli": "^0.1.2",
-    "@zeit/ncc": "^0.22.1"
+    "@vercel/ncc": "^0.27.0",
+    "typescript": "^4.5.5"
   }
 }

renovate.json

@@ -1,6 +1,10 @@
 {
+  "$schema": "https://docs.renovatebot.com/renovate-schema.json",
   "pinVersions": false,
   "extends": [
     "config:base"
+  ],
+  "ignoreDeps": [
+    "ajv"
   ]
 }

run.sh

@@ -3,7 +3,5 @@
 export HOME="$(pwd)"
 export INPUT_VERSION=4.11.1
 export INPUT_DEST='~/pnpm.temp'
-export INPUT_BIN_DEST='~/pnpm.temp/.bin'
-export INPUT_REGISTRY=https://registry.npmjs.com
 export INPUT_RUN_INSTALL=null
 exec node dist/index.js

src/inputs/index.ts

@@ -5,8 +5,6 @@ import { RunInstall, parseRunInstall } from './run-install'
 export interface Inputs {
   readonly version: string
   readonly dest: string
-  readonly binDest: string
-  readonly registry: string
   readonly runInstall: RunInstall[]
 }
 
@@ -19,8 +17,6 @@ const parseInputPath = (name: string) => expandTilde(getInput(name, options))
 export const getInputs = (): Inputs => ({
   version: getInput('version', options),
   dest: parseInputPath('dest'),
-  binDest: parseInputPath('bin_dest'),
-  registry: getInput('registry', options),
   runInstall: parseRunInstall('run_install'),
 })
 

src/inputs/run-install.ts

@@ -1,5 +1,5 @@
 import process from 'process'
-import { safeLoad } from 'js-yaml'
+import { load } from 'js-yaml'
 import Ajv from 'ajv'
 import { getInput, error, InputOptions } from '@actions/core'
 import runInstallSchema from './run-install-input.schema.json'
@@ -21,10 +21,9 @@ const options: InputOptions = {
 }
 
 export function parseRunInstall(name: string): RunInstall[] {
-  const result: RunInstallInput = safeLoad(getInput(name, options))
+  const result: RunInstallInput = load(getInput(name, options)) as any
   const ajv = new Ajv({
     allErrors: true,
-    async: false,
   })
   const validate = ajv.compile(runInstallSchema)
   if (!validate(result)) {

src/install-pnpm/index.ts

@@ -9,7 +9,7 @@ export async function install(inputs: Inputs) {
   const status = await runSelfInstaller(inputs)
   endGroup()
   if (status) {
-    return setFailed(`Something does wrong, self-installer exits with code ${status}`)
+    return setFailed(`Something went wrong, self-installer exits with code ${status}`)
   }
 }
 

src/install-pnpm/run.ts

@@ -1,26 +1,38 @@
+import { addPath, exportVariable } from '@actions/core'
 import { spawn } from 'child_process'
 import { execPath } from 'process'
-import { downloadSelfInstaller } from '../self-installer'
+import path from 'path'
+import { remove, ensureFile, writeFile } from 'fs-extra'
+import fetch from 'node-fetch'
 import { Inputs } from '../inputs'
 
 export async function runSelfInstaller(inputs: Inputs): Promise<number> {
-  const cp = spawn(execPath, {
+  const { version, dest } = inputs
-    env: {
+  const target = version ? `@pnpm/exe@${version}` : '@pnpm/exe'
-      PNPM_VERSION: inputs.version,
+  const pkgJson = path.join(dest, 'package.json')
-      PNPM_DEST: inputs.dest,
+
-      PNPM_BIN_DEST: inputs.binDest,
+  await remove(dest)
-      PNPM_REGISTRY: inputs.registry,
+  await ensureFile(pkgJson)
-    },
+  await writeFile(pkgJson, JSON.stringify({ private: true }))
+
+  const cp = spawn(execPath, ['-', 'install', target, '--no-lockfile'], {
+    cwd: dest,
     stdio: ['pipe', 'inherit', 'inherit'],
   })
 
-  const response = await downloadSelfInstaller()
+  const response = await fetch('https://get.pnpm.io/v6.16.js')
   response.body.pipe(cp.stdin)
 
-  return new Promise((resolve, reject) => {
+  const exitCode = await new Promise<number>((resolve, reject) => {
     cp.on('error', reject)
     cp.on('close', resolve)
   })
+  if (exitCode === 0) {
+    const pnpmHome = path.join(dest, 'node_modules/.bin')
+    addPath(pnpmHome)
+    exportVariable('PNPM_HOME', pnpmHome)
+  }
+  return exitCode
 }
 
 export default runSelfInstaller

src/outputs/index.ts

@@ -1,10 +1,12 @@
 import { setOutput, addPath } from '@actions/core'
 import { Inputs } from '../inputs'
+import { getBinDest } from '../utils'
 
 export function setOutputs(inputs: Inputs) {
-  addPath(inputs.binDest)
+  const binDest = getBinDest(inputs)
+  addPath(binDest)
   setOutput('dest', inputs.dest)
-  setOutput('bin_dest', inputs.binDest)
+  setOutput('bin_dest', binDest)
 }
 
 export default setOutputs

src/self-installer/download.ts

@@ -1,4 +0,0 @@
-import fetch from 'node-fetch'
-import url from './url'
-export const downloadSelfInstaller = () => fetch(url)
-export default downloadSelfInstaller

src/self-installer/index.ts

@@ -1,2 +0,0 @@
-export * from './url'
-export * from './download'

src/self-installer/url.ts

@@ -1,3 +0,0 @@
-export const ref = '301414cec74a2b6b63c95b42f2ad1790ccb980ed'
-export const url = `https://raw.githubusercontent.com/pnpm/self-installer/${ref}/install.js`
-export default url

src/utils/index.ts

@@ -2,7 +2,9 @@ import process from 'process'
 import path from 'path'
 import { Inputs } from '../inputs'
 
+export const getBinDest = (inputs: Inputs): string => path.join(inputs.dest, 'node_modules', '.bin')
+
 export const patchPnpmEnv = (inputs: Inputs): NodeJS.ProcessEnv => ({
   ...process.env,
-  PATH: inputs.binDest + path.delimiter + process.env.PATH
+  PATH: getBinDest(inputs) + path.delimiter + process.env.PATH
 })