prettify+eslint codebase (#1387)

* added prettier and ran `npx prettier ./samples/ -w`

* clean up library typecasting + fix prettier errors + remove some ts/prettier ignore lines

* remove typecasts from querySelector calls

* clean up library typecasting + fix prettier errors + remove some ts/prettier ignore lines

* added prettier and ran `npx prettier ./samples/ -w`

* add Prettier Check to test action

* for all ts file in /samples/ for all queryselector calls use ! to narrow the returned type to non-nullable

* remove more typecasting

* manually fix ts errors

* fix json

* exclude fewer files from prettifying and run

* ran `npx eslint --fix`

* manual fixes for eslint

* fix or disable remaining eslint errors

* comment spacing enforcement

* remove and cleanup for @ts-nocheck

* check eslint test action

* prettify output code too
make newlines preserve in TS->JS
make build-all fail on any failures
simplify cross-platform sed calls
other misc prettifying
start enforcing no google.maps usage in runtime

* more google.maps.* removal

* fix status code check
do simple tests first

* fix even more build/typings issues

* bootstrap normalization

* further build fixes

* enforce maps js async

* temporarily skip certain checks

* fix: Adds exclude-dir for node_modules.

* DRY up build_single.sh

---------

Co-authored-by: William French <dubnewt@gmail.com>

Author: chrisjshull

.github/blunderbuss.yml

@@ -15,4 +15,4 @@
 assign_issues:
   - googlemaps/devrel
 assign_prs:
-  - googlemaps/devrel
+  - googlemaps/devrel

.github/dependabot.yml

@@ -20,4 +20,4 @@ updates:
       interval: "weekly"
     ignore:
       - dependency-name: "three"
-      - dependency-name: "@types/three"
+      - dependency-name: "@types/three"

.github/header-checker-lint.yml

@@ -14,4 +14,4 @@ sourceFileExtensions:
   - .sh
   - .swift
   - .ts
-  - .yml
+  - .yml

.github/snippet-bot.yml

@@ -1,3 +1,3 @@
 alwaysCreateStatusCheck: true
 ignoreFiles:
-- "samples/*.sh"
+  - "samples/*.sh"

.github/workflows/approve-merge.yml

@@ -36,7 +36,7 @@ jobs:
         with:
           ref: ${{ github.event.pull_request.head.ref }} # Checkout the PR's head commit
           token: ${{ secrets.GH_MERGE_TOKEN }}
-          
+
       - name: Approve PR
         run: gh pr review --approve "$PR_URL"
         env:

.github/workflows/dist-pr.yml

@@ -18,7 +18,7 @@ on:
   workflow_dispatch:
     inputs:
       source_run_id:
-        description: 'ID of the upstream workflow run that triggered this'
+        description: "ID of the upstream workflow run that triggered this"
         required: false
         type: string
 

.github/workflows/release.yml

@@ -19,7 +19,7 @@ on:
     branches:
       - main
     paths:
-      - 'samples/**'
+      - "samples/**"
 
 jobs:
   release:
@@ -51,7 +51,7 @@ jobs:
 
       - uses: actions/setup-node@v4
         with:
-          node-version: '24.x'
+          node-version: "24.x"
 
       - run: npm i
 
@@ -93,7 +93,7 @@ jobs:
             echo "  - samples/$workspace"
             rm -rf dist/samples/$workspace
           done
-          
+
       - name: Generate Index
         run: bash samples/generate-index.sh
 
@@ -130,6 +130,6 @@ jobs:
         uses: benc-uk/workflow-dispatch@v1
         with:
           workflow: dist-pr.yml
-          ref: 'temp-build-branch'
+          ref: "temp-build-branch"
           token: ${{ secrets.GH_APPROVER_TOKEN }} # Use the same token that pushed, it needs 'workflows: write' permissions
           inputs: '{ "source_run_id": "${{ github.run_id }}" }' # Optional: Pass the run ID for traceability

.github/workflows/playwright.yml .github/workflows/tests.yml.github/workflows/playwright.yml renamed to .github/workflows/tests.yml

@@ -12,24 +12,24 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-name: Playwright Tests
+name: Tests
 
 on:
   pull_request:
     types: [opened, synchronize, edited]
     paths:
-      - 'samples/**'
-      - 'e2e/samples.spec.ts'
-      - 'playwright.config.ts'
-      - 'package.json'
+      - "samples/**"
+      - "e2e/samples.spec.ts"
+      - "playwright.config.ts"
+      - "package.json"
   push:
     branches:
       - main
     paths:
-      - 'samples/**'
-      - 'e2e/samples.spec.ts'
-      - 'playwright.config.ts'
-      - 'package.json'
+      - "samples/**"
+      - "e2e/samples.spec.ts"
+      - "playwright.config.ts"
+      - "package.json"
   schedule:
     - cron: "0 12 * * *"
 
@@ -74,7 +74,7 @@ jobs:
         if: env.HAS_CHANGES == 'true'
         uses: actions/setup-node@v4
         with:
-          node-version: '24.x'
+          node-version: "24.x"
 
       - name: Cache npm dependencies
         if: env.HAS_CHANGES == 'true'
@@ -108,17 +108,29 @@ jobs:
         if: env.HAS_CHANGES == 'true'
         run: npm install
 
+      - name: Prettier Check
+        if: env.HAS_CHANGES == 'true'
+        run: npx prettier ./samples/ --check --log-level warn
+        env:
+          CI: true
+
+      - name: ESLint Check
+        if: env.HAS_CHANGES == 'true'
+        run: npx eslint
+        env:
+          CI: true
+
       - name: Install Playwright Browsers
         if: steps.playwright-cache.outputs.cache-hit != 'true' && env.HAS_CHANGES == 'true'
         run: npx playwright install chromium --with-deps
-      
+
       - name: Build Changed Workspaces
         if: env.HAS_CHANGES == 'true'
         run: |
           IFS=$'\n'
           CHANGED_WORKSPACES_ARRAY=(${STEPS_GET_WORKSPACES_OUTPUTS_CHANGED_WORKSPACES})
           echo "Changed Workspaces Array: ${CHANGED_WORKSPACES_ARRAY[@]}"
-          
+
           echo "Building changed workspaces:"
           for workspace in "${CHANGED_WORKSPACES_ARRAY[@]}"; do
             echo "  - samples/$workspace"
@@ -162,7 +174,7 @@ jobs:
       - name: Setup Node.js
         uses: actions/setup-node@v4
         with:
-          node-version: '24.x'
+          node-version: "24.x"
 
       - name: Cache npm dependencies
         uses: actions/cache@v4

.gitignore

@@ -6,7 +6,7 @@ samples/.env
 
 # Ignore dist files generated by build.
 samples/*/dist/
-samples/**/*.js
+samples/*/*.js
 
 # Ignore playwright outputs
 /test-results/

.prettierignore

@@ -0,0 +1,5 @@
+/index.html
+package.json
+package-lock.json
+dist
+samples/*/*.js