From 8433c795600fa0a7ba98fed17c75dd66b2c39734 Mon Sep 17 00:00:00 2001
From: Philipp Horstenkamp <philipp@horstenkamp.de>
Date: Fri, 26 Dec 2025 23:35:21 +0000
Subject: [PATCH] chore: update devcontainer setup and pre-commit configuration

---
 .devcontainer/Dockerfile        |   7 ---
 .devcontainer/devcontainer.json |  21 +++----
 .devcontainer/setup.sh          |  10 +--
 .pre-commit-config.yaml         | 105 +++++++++++++++-----------------
 4 files changed, 62 insertions(+), 81 deletions(-)
 delete mode 100644 .devcontainer/Dockerfile

diff --git a/.devcontainer/Dockerfile b/.devcontainer/Dockerfile
deleted file mode 100644
index d696268..0000000
--- a/.devcontainer/Dockerfile
+++ /dev/null
@@ -1,7 +0,0 @@
-# .devcontainer/Dockerfile
-FROM mcr.microsoft.com/devcontainers/javascript-node:1-20-bullseye
-
-# CACHED INSTALLATION:
-# Install global NPM packages here. This creates a cached Docker layer.
-# We use 'su node' to ensure permissions are correct for the non-root user.
-RUN su node -c "npm install -g @google/gemini-cli"
diff --git a/.devcontainer/devcontainer.json b/.devcontainer/devcontainer.json
index 5c86713..2cbbbf2 100644
--- a/.devcontainer/devcontainer.json
+++ b/.devcontainer/devcontainer.json
@@ -1,17 +1,13 @@
 {
-  "name": "Gemini Agent Quality Ops",
-  "build": {
-    "dockerfile": "Dockerfile"
-  },
+  "name": "JS Dev Container",
+  "image": "mcr.microsoft.com/devcontainers/javascript-node:1-20",
   "features": {
-    "ghcr.io/devcontainers/features/python:1": {
-      "version": "3.11",
-      "installTools": true,
-      "optimize": true
-    },
     "ghcr.io/devcontainers/features/git:1": {},
     "ghcr.io/devcontainers/features/git-lfs:1": {},
-    "ghcr.io/devcontainers-extra/features/pre-commit:2": {}
+    "ghcr.io/devcontainers-extra/features/pre-commit:2": {},
+    "ghcr.io/devcontainers-extra/features/npm-packages:1": {
+      "packages": "@google/gemini-cli"
+    }
   },
   "customizations": {
     "vscode": {
@@ -29,12 +25,13 @@
     }
   },
   "postStartCommand": "bash .devcontainer/setup.sh",
-  "remoteUser": "vscode",
+  "remoteUser": "node",
   "updateRemoteUserUID": true,
   "mounts": [
     "source=pipx-venvs,target=/home/node/.local/share/pipx,type=volume",
     "source=pipx-bin,target=/home/node/.local/bin,type=volume",
     "source=pre-commit-cache,target=/home/node/.cache/pre-commit,type=volume",
-    "source=${localEnv:USERPROFILE}/.gemini,target=/home/node/.gemini,type=bind"
+    "source=${localEnv:USERPROFILE}/.gemini,target=/home/node/.gemini,type=bind",
+    "source=gvscode-extensions,target=/home/node/.cache/google-vscode-extension,type=volume"
   ]
 }
diff --git a/.devcontainer/setup.sh b/.devcontainer/setup.sh
index acbcfc5..61d9d72 100644
--- a/.devcontainer/setup.sh
+++ b/.devcontainer/setup.sh
@@ -3,9 +3,9 @@
 # 1. Fix Git Permissions (Critical in Docker)
 git config --global --add safe.directory $(pwd)
 
+# In your setup.sh or postCreateCommand
+sudo chown -R node:node /home/node/.cache/
+
 # 2. Re-connect Git Hooks
-if command -v pre-commit &> /dev/null; then
-    pre-commit install
-else
-    echo "⚠️ Warning: pre-commit not found. Skipping hook installation."
-fi
+pre-commit install
+pre-commit install-hooks
diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
index d9ba9c9..e0ec95a 100644
--- a/.pre-commit-config.yaml
+++ b/.pre-commit-config.yaml
@@ -1,57 +1,48 @@
-exclude: ^node_modules/
-repos:
-- repo: https://github.com/pre-commit/pre-commit-hooks
-  rev: v6.0.0
-  hooks:
-  - id: end-of-file-fixer
-    exclude: (.txt$|.ipynb$|README.md$|readme.mde$)
-  - id: trailing-whitespace
-    exclude: (.txt$|README.md$)
-  - id: check-yaml
-  - id: check-json
-  - id: check-toml
-  - id: check-xml
-  - id: check-added-large-files
-    args: [--enforce-all]
-  - id: name-tests-test
-  - id: detect-private-key
-  - id: check-case-conflict
-  - id: check-symlinks
-  - id: check-docstring-first
-  - id: pretty-format-json
-    args: [--autofix, --no-sort-keys, --no-ensure-ascii]
-  - id: check-merge-conflict
-  - id: no-commit-to-branch
-    args: [--branch, main]
-
-- repo: https://github.com/macisamuele/language-formatters-pre-commit-hooks
-  rev: v2.15.0
-  hooks:
-  - id: pretty-format-ini
-    args: [--autofix]
-  - id: pretty-format-toml
-    args: [--autofix]
-  - id: pretty-format-yaml
-    args: [--autofix]
-
-- repo: https://github.com/frnmst/md-toc
-  rev: 9.0.0
-  hooks:
-  - id: md-toc
-
-- repo: https://github.com/Lucas-C/pre-commit-hooks-java
-  rev: 1.3.10
-  hooks:
-  - id: validate-html
-
-- repo: https://github.com/pre-commit/mirrors-prettier
-  rev: v4.0.0-alpha.8
-  hooks:
-  - id: prettier
-    types_or: [css, javascript]
-
-- repo: https://github.com/python-jsonschema/check-jsonschema
-  rev: 0.36.0
-  hooks:
-  - id: check-renovate
-  - id: check-github-actions
+exclude: ^node_modules/
+repos:
+- repo: https://github.com/pre-commit/pre-commit-hooks
+  rev: v6.0.0
+  hooks:
+  - id: end-of-file-fixer
+    exclude: (.txt$|.ipynb$|README.md$|readme.mde$)
+  - id: trailing-whitespace
+    exclude: (.txt$|README.md$)
+  - id: check-yaml
+  - id: check-json
+  - id: check-toml
+  - id: check-xml
+  - id: check-added-large-files
+    args: [--enforce-all]
+  - id: name-tests-test
+  - id: detect-private-key
+  - id: check-case-conflict
+  - id: check-symlinks
+  - id: check-docstring-first
+  - id: pretty-format-json
+    args: [--autofix, --no-sort-keys, --no-ensure-ascii]
+  - id: check-merge-conflict
+  - id: no-commit-to-branch
+    args: [--branch, main]
+
+- repo: https://github.com/macisamuele/language-formatters-pre-commit-hooks
+  rev: v2.15.0
+  hooks:
+  - id: pretty-format-ini
+    args: [--autofix]
+  - id: pretty-format-toml
+    args: [--autofix]
+  - id: pretty-format-yaml
+    args: [--autofix]
+
+- repo: https://github.com/pre-commit/mirrors-prettier
+  rev: v4.0.0-alpha.8
+  hooks:
+  - id: prettier
+    types_or: [css, javascript]
+
+- repo: https://github.com/python-jsonschema/check-jsonschema
+  rev: 0.36.0
+  hooks:
+  - id: check-renovate
+  - id: check-github-actions
+  - id: check-github-workflows