Merge pull request #6 from pangerlkr/copilot/fix-workflow-actions

Fix copilot-setup-steps.yml YAML structure and address review feedback

.github/workflows/copilot-setup-steps.yml

@@ -1,18 +1,18 @@
-            - name: Setup Go environment
+steps:
-  uses: actions/setup-go@v6.2.0
+  - name: Setup Go environment
-  with:
+    uses: actions/setup-go@v6.2.0
-    # The Go version to download (if necessary) and use. Supports semver spec and ranges. Be sure to enclose this option in single quotation marks.
+    with:
-    go-version: # optional
+      # The Go version to download (if necessary) and use. Supports semver spec and ranges. Be sure to enclose this option in single quotation marks.
-    # Path to the go.mod, go.work, .go-version, or .tool-versions file.
+      go-version: # optional
-    go-version-file: # optional
+      # Path to the go.mod, go.work, .go-version, or .tool-versions file.
-    # Set this option to true if you want the action to always check for the latest available version that satisfies the version spec
+      go-version-file: # optional
-    check-latest: # optional
+      # Set this option to true if you want the action to always check for the latest available version that satisfies the version spec
-    # Used to pull Go distributions from go-versions. Since there's a default, this is typically not supplied by the user. When running this action on github.com, the default value is sufficient. When running on GHES, you can pass a personal access token for github.com if you are experiencing rate limiting.
+      check-latest: # optional
-    token: # optional, default is ${{ github.server_url == 'https://github.com' && github.token || '' }}
+      # Used to pull Go distributions from go-versions. Since there's a default, this is typically not supplied by the user. When running this action on github.com, the default value is sufficient. When running on GHES, you can pass a personal access token for github.com if you are experiencing rate limiting.
-    # Used to specify whether caching is needed. Set to true, if you'd like to enable caching.
+      token: # optional, default is ${{ github.server_url == 'https://github.com' && github.token || '' }}
-    cache: # optional, default is true
+      # Used to specify whether caching is needed. Set to true, if you'd like to enable caching.
-    # Used to specify the path to a dependency file - go.sum
+      cache: # optional, default is true
-    cache-dependency-path: # optional
+      # Used to specify the path to a dependency file - go.sum
-    # Target architecture for Go to use. Examples: x86, x64. Will use system architecture by default.
+      cache-dependency-path: # optional
-    architecture: # optional
+      # Target architecture for Go to use. Examples: x86, x64. Will use system architecture by default.
-          
+      architecture: # optional

package-lock.json

@@ -1,14 +1,25 @@
 {
-  "name": "everything-claude-code",
+  "name": "ecc-universal",
+  "version": "1.4.1",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {
     "": {
+      "name": "ecc-universal",
+      "version": "1.4.1",
+      "hasInstallScript": true,
+      "license": "MIT",
+      "bin": {
+        "ecc-install": "install.sh"
+      },
       "devDependencies": {
         "@eslint/js": "^9.39.2",
         "eslint": "^9.39.2",
         "globals": "^17.1.0",
         "markdownlint-cli": "^0.47.0"
+      },
+      "engines": {
+        "node": ">=18"
       }
     },
     "node_modules/@eslint-community/eslint-utils": {
@@ -294,6 +305,7 @@
       "integrity": "sha512-NZyJarBfL7nWwIq+FDL6Zp/yHEhePMNnnJ0y3qfieCrmNvYct8uvtiV41UvlSe6apAfk0fY1FbWx+NwfmpvtTg==",
       "dev": true,
       "license": "MIT",
+      "peer": true,
       "bin": {
         "acorn": "bin/acorn"
       },
@@ -599,6 +611,7 @@
       "integrity": "sha512-LEyamqS7W5HB3ujJyvi0HQK/dtVINZvd5mAAp9eT5S/ujByGjiZLCzPcHVzuXbpJDJF/cxwHlfceVUDZ2lnSTw==",
       "dev": true,
       "license": "MIT",
+      "peer": true,
       "dependencies": {
         "@eslint-community/eslint-utils": "^4.8.0",
         "@eslint-community/regexpp": "^4.12.1",
@@ -1930,6 +1943,7 @@
       "integrity": "sha512-5gTmgEY/sqK6gFXLIsQNH19lWb4ebPDLA4SdLP7dsWkIXHWlG66oPuVvXSGFPppYZz8ZDZq0dYYrbHfBCVUb1Q==",
       "dev": true,
       "license": "MIT",
+      "peer": true,
       "engines": {
         "node": ">=12"
       },

tests/lib/package-manager.test.js

@@ -1632,7 +1632,7 @@ function runTests() {
   else failed++;
 
   // Round 111: getExecCommand with newline in args
-  console.log('\nRound 111: getExecCommand (newline in args — SAFE_ARGS_REGEX \\\\\\\\s matches \\\\\\n):');
+  console.log('\n' + String.raw`Round 111: getExecCommand (newline in args — SAFE_ARGS_REGEX \s matches \n):`);
 
   if (test('getExecCommand accepts newline in args because SAFE_ARGS_REGEX includes newline', () => {
     // SAFE_ARGS_REGEX = /^[@a-zA-Z0-9\\s_.\\/:=,'\"*+-\\]+$/

tests/lib/session-aliases.test.js

@@ -787,7 +787,6 @@ function runTests() {
     // Verify the file exists
     const aliasesPath = path.join(tmpHome, '.claude', 'session-aliases.json');
     assert.ok(fs.existsSync(aliasesPath), 'Aliases file should exist');
-    const _contentBefore = fs.readFileSync(aliasesPath, 'utf8');
 
     // Attempt to save circular data — will fail
     const circular = { aliases: {}, metadata: {} };