Fix this action since pkgx 1.1 (#3)

Author: mxcl

.github/workflows/ci.yml

@@ -2,6 +2,7 @@ on:
   push:
     branches:
       - main
+  pull_request:
 
 jobs:
   test:

action.yml

@@ -9,15 +9,19 @@ inputs:
 runs:
   using: composite
   steps:
-    - uses: pkgxdev/setup@v1
+    - uses: pkgxdev/setup@v2
 
     - run: |
-        if [ -n "${{ inputs.path }}" ]; then
-          cd "${{ inputs.path }}"
-        fi
-        pkgx --internal.activate "$PWD" > file
-        if ! command -v node >/dev/null 2>&1; then
+        TMP="$(mktemp)"
+        pkgx --internal.activate "$PWD" > "$TMP"
+        echo "file=$TMP" >> $GITHUB_OUTPUT
+      id: env
+      working-directory: ${{ inputs.path }}
+      shell: bash
+
+    - run: |
+        if ! node --version >/dev/null 2>&1; then
           eval "$(pkgx +node)"
         fi
-        node $GITHUB_ACTION_PATH/parse.js
+        node "$GITHUB_ACTION_PATH"/parse.js ${{ steps.env.outputs.file }}
       shell: bash

parse.js

@@ -2,18 +2,20 @@ const fs = require('fs');
 const readline = require('readline');
 
 const readInterface = readline.createInterface({
-    input: fs.createReadStream('file'),
+    input: fs.createReadStream(process.argv[2]),
     output: process.stdout,
     terminal: false
 });
 
 const stripQuotes = (str) => str.startsWith('"') || str.startsWith("'") ? str.slice(1, -1) : str;
 
 const replaceEnvVars = (str) => {
-  return str.replace(/(:?)\$\{([^}]+)\}/g, (match, precedingColon, varName) => {
-      const envValue = process.env[varName];
-      return envValue ? `${precedingColon}${envValue}` : (precedingColon ? '' : envValue);
-  });
+    const value = str
+      .replaceAll(/\$\{([a-zA-Z0-9_]+):\+:\$[a-zA-Z0-9_]+\}/g, (_, key) => (v => v ? `:${v}` : '')(process.env[key]))
+      .replaceAll(/\$\{([a-zA-Z0-9_]+)\}/g, (_, key) => process.env[key] ?? '')
+      .replaceAll(/\$([a-zA-Z0-9_]+)/g, (_, key) => process.env[key] ?? '')
+    console.error("FOO", str, value)
+    return value
 };
 
 readInterface.on('line', (line) => {
@@ -22,10 +24,12 @@ readInterface.on('line', (line) => {
         const [_, key, value_] = match;
         const value = stripQuotes(value_);
         if (key === 'PATH') {
-            value.split(':').forEach((path) => {
-                if (!path.startsWith("$")) {
-                  fs.appendFileSync(process.env['GITHUB_PATH'], `${path}\n`);
-                }
+            value
+              .replaceAll('${PATH:+:$PATH}', '')
+              .replaceAll('$PATH', '')
+              .replaceAll('${PATH}', '')
+              .split(':').forEach(path => {
+                fs.appendFileSync(process.env['GITHUB_PATH'], `${path}\n`);
             });
         } else {
             let v = replaceEnvVars(value);