Merge branch 'main' into hivanalejandro-features-bigquery-2

Author: hivanalejandro

.github/config/nodejs-dev.jsonc

@@ -217,6 +217,7 @@
     "texttospeech",
     "tpu",
     "translate",
+    "vision",
     "workflows/invoke-private-endpoint"
   ]
 }

.github/config/nodejs-prod.jsonc

@@ -95,7 +95,6 @@
     "run/idp-sql", // (untested) Error: Invalid contents in the credentials file
     "storagetransfer", // CredentialsError: Missing credentials in config, if using AWS_CONFIG_FILE, set AWS_SDK_LOAD_CONFIG=1
     "video-intelligence", // PERMISSION_DENIED: The caller does not have permission
-    "vision", // REDIS: Error: connect ECONNREFUSED 127.0.0.1:6379
     "workflows", // SyntaxError: Cannot use import statement outside a module
     "workflows/quickstart" // [ERR_MODULE_NOT_FOUND]: Cannot find package 'ts-node' imported from ...
   ]

.github/workflows/appengine-analytics.yaml

@@ -60,16 +60,51 @@ jobs:
           cat paths.txt
           echo "setups=$(./cloud-samples-tools/bin/custard setup-files .github/config/nodejs-prod.jsonc paths.txt)" >> $GITHUB_OUTPUT
 
-  nodejs-lint:
-    name: lint
+  lint:
+    needs: affected
     runs-on: ubuntu-latest
     timeout-minutes: 5
     steps:
-      - uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4
-      - uses: actions/setup-node@1d0ff469b7ec7b3cb9d8673fde0c81c44821de2a # v4
+      - name: Checkout
+        uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4
+      - name: Setup Node
+        uses: actions/setup-node@1d0ff469b7ec7b3cb9d8673fde0c81c44821de2a # v4
         with:
           node-version: 20
-      - run: make lint
+      - run: npm install
+      - name: Run lint
+        uses: actions/github-script@60a0d83039c74a4aee543508d2ffcb1c3799cdea # v7
+        with:
+          script: |
+            const { execSync } = await import("node:child_process");
+
+            const cmd = 'npx gts lint';
+            const affected = ${{ needs.affected.outputs.nodejs-paths }};
+            if (affected.length === 0) {
+              console.log("No packages were affected, nothing to lint.")
+            }
+
+            let failed = [];
+            for (const path of affected) {
+              try {
+                execSync(cmd, {cwd: path});
+                console.log(`✅ [${path}]: ${cmd}`);
+              } catch (e) {
+                failed.push(path)
+                console.log(`❌ [${path}]: ${cmd} (exit code ${e.status})`);
+                core.error(e.message);
+                console.log('--- stdout ---');
+                console.log(e.stdout.toString("utf8"));
+                console.log('--- stderr ---');
+                console.log(e.stderr.toString("utf8"));
+              }
+            }
+            console.log("=== Summary ===")
+            console.log(`  Passed: ${affected.length - failed.length}`)
+            console.log(`  Failed: ${failed.length}`)
+            if (failed.length > 0) {
+              core.setFailed(`Failed '${cmd}' on: ${failed.join(', ')}`)
+            }
 
   region-tags:
     name: region tags
@@ -82,8 +117,7 @@ jobs:
           node-version: 20
       - run: ./.github/workflows/utils/region-tags-tests.sh
 
-  nodejs-test:
-    name: test
+  test:
     needs: affected
     runs-on: ubuntu-latest
     timeout-minutes: 120 # 2 hours hard limit