feat: migrate generated packages to root ESLint config and ignore current violations

.eslintignore

@@ -4,4 +4,4 @@ test/fixtures
 build/
 docs/
 protos/
-packages/
+handwritten/

.eslintrc.json

@@ -0,0 +1,11 @@
+{
+  "overrides": [
+    {
+      "files": ["packages/**/*.ts"],
+      "extends": "./node_modules/gts",
+      "parserOptions": {
+        "project": ["packages/*/tsconfig.json"]
+      }
+    }
+  ]
+}

.github/workflows/eslint.yaml

@@ -0,0 +1,49 @@
+name: ESLint Check
+
+on:
+  pull_request:
+    paths:
+      - 'packages/**'
+
+permissions:
+  contents: read
+
+jobs:
+  lint-changed:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+        with:
+          fetch-depth: 0 # Required to get all history for diffs
+
+      - name: Use Node.js
+        uses: actions/setup-node@v4
+        with:
+          node-version: 18
+
+      - name: Install shared linting dependencies
+        run: |
+          npm install --save-dev eslint@8 gts@6.0.2 stream-json
+
+      - name: Get changed packages
+        id: changed-packages
+        run: |
+          # Get files changed in the PR relative to the base branch
+          git fetch origin ${{ github.base_ref }}
+          CHANGED_FILES=$(git diff --name-only origin/${{ github.base_ref }}...HEAD)
+
+          # Extract unique package names from packages/ directory
+          CHANGED_PACKAGES=$(echo "$CHANGED_FILES" | grep '^packages/' | cut -d/ -f2 | sort -u | tr '\n' ' ')
+
+          echo "packages=$CHANGED_PACKAGES" >> $GITHUB_OUTPUT
+          echo "Changed packages: $CHANGED_PACKAGES"
+
+      - name: Run ESLint on changed packages
+        if: steps.changed-packages.outputs.packages != ''
+        env:
+          CHANGED_PACKAGES: ${{ steps.changed-packages.outputs.packages }}
+        run: |
+          for pkg in $CHANGED_PACKAGES; do
+            echo "Processing package: $pkg"
+            node packages/run-eslint-root.mjs "$pkg" --fail-on-error
+          done

core/generator/gapic-generator-typescript/package.json

@@ -58,6 +58,7 @@
     "proto3-json-serializer": "^3.0.3",
     "protobufjs": "^7.5.4",
     "protobufjs-cli": "^1.2.0",
+    "prettier": "^3.2.5",
     "yargs": "^17.7.2"
   },
   "devDependencies": {

core/generator/gapic-generator-typescript/typescript/src/templater.ts

@@ -16,6 +16,7 @@ import * as fs from 'fs';
 import nunjucks from 'nunjucks';
 import * as path from 'path';
 import * as util from 'util';
+import * as prettier from 'prettier';
 
 import type * as protos from '../../protos/index.js';
 
@@ -183,7 +184,7 @@ async function countRegionTagLines(
   return {start, end};
 }
 
-function renderFile(
+async function renderFile(
   targetFilename: string,
   templateName: string,
   renderParameters: {},
@@ -206,6 +207,28 @@ function renderFile(
       }
     }
   }
+
+  // Format generated TypeScript files using Prettier
+  if (targetFilename.match(/\.ts$/i)) {
+    try {
+      processed = await prettier.format(processed, {
+        parser: 'typescript',
+        filepath: targetFilename,
+        singleQuote: true,
+        trailingComma: 'all',
+        semi: true,
+      });
+    } catch (err) {
+      if (err instanceof Error) {
+        console.warn(
+          `Failed to format TypeScript file ${targetFilename}: ${err.toString()}`,
+        );
+      } else {
+        throw err;
+      }
+    }
+  }
+
   const output =
     {} as protos.google.protobuf.compiler.CodeGeneratorResponse.File;
   output.name = targetFilename;
@@ -254,7 +277,7 @@ async function processOneTemplate(
           .replace(/\$service/, service.name!.toSnakeCase());
 
         result.push(
-          renderFile(pushFilename, relativeTemplateName, {
+          await renderFile(pushFilename, relativeTemplateName, {
             method,
             api,
             commonParameters,
@@ -274,7 +297,7 @@ async function processOneTemplate(
         continue;
       }
       result.push(
-        renderFile(
+        await renderFile(
           outputFilename.replace(/\$service/, service.name!.toSnakeCase()),
           relativeTemplateName,
           {api, commonParameters, service, id},
@@ -283,7 +306,7 @@ async function processOneTemplate(
     }
   } else {
     result.push(
-      renderFile(outputFilename, relativeTemplateName, {
+      await renderFile(outputFilename, relativeTemplateName, {
         api,
         commonParameters,
         id,