fix: add token as npx argument

BREAKING CHANGE: token is second argument and folder moved to third argument

Author: UziTech

.github/workflows/test.yml

@@ -23,20 +23,14 @@ jobs:
       - name: Build Action
         if: matrix.source == 'action'
         run: npm run build
-      - name: Install Atom with Action
-        if: matrix.source == 'action' && matrix.version == 'stable'
-        uses: ./
       - name: Install Atom ${{ matrix.version }} with Action
-        if: matrix.source == 'action' && matrix.version != 'stable'
+        if: matrix.source == 'action'
         uses: ./
         with:
           version: ${{ matrix.version }}
-      - name: Install Atom with NPM
-        if: matrix.source == 'npm' && matrix.version == 'stable'
-        run: node ./src/action.js
       - name: Install Atom ${{ matrix.version }} with NPM
-        if: matrix.source == 'npm' && matrix.version != 'stable'
-        run: node ./src/action.js ${{ matrix.version }}
+        if: matrix.source == 'npm'
+        run: node ./src/action.js ${{ matrix.version }} ${{ secrets.GITHUB_TOKEN }}
       - name: Don't download to repo directory
         run: git add . -N && git diff --name-only --exit-code -- . ':!dist'
       - name: Run Specs

src/action.js

@@ -22,8 +22,8 @@ async function run() {
 			core.error("'channel' is deprecated. Please use 'version' instead.");
 		}
 		const version = channel || (process.env.GITHUB_ACTIONS && core.getInput("version").toLowerCase()) || process.argv[2] || "stable";
-		const folder = path.resolve(process.env.RUNNER_TEMP, process.argv[3] || "./atom");
-		const token = (process.env.GITHUB_ACTIONS && core.getInput("token")) || process.argv[4] || "";
+		const token = (process.env.GITHUB_ACTIONS && core.getInput("token")) || process.argv[3] || "";
+		const folder = path.resolve(process.env.RUNNER_TEMP, process.argv[4] || "./atom");
 		core.info(`version: ${version}`);
 		core.info(`folder: ${folder}`);
 

src/setup-atom.js

@@ -21,6 +21,9 @@ const writeFileAsync = promisify(fs.writeFile);
 const CHANNELS = [
 	"stable",
 	"beta",
+];
+
+const INVALID_CHANNELS = [
 	"nightly",
 	"dev",
 ];
@@ -138,6 +141,9 @@ async function printVersions() {
 }
 
 async function findUrl(version, token) {
+	if (INVALID_CHANNELS.includes(version)) {
+		throw new Error(`'${version}' is not a valid version.`);
+	}
 	if (CHANNELS.includes(version)) {
 		const octokit = new Octokit({auth: token});
 		const {data: releases} = await octokit.rest.repos.listReleases({