fix: refactor commitAndPush function to improve clarity and maintainability

Dockerfile

@@ -1,9 +1,9 @@
-FROM node:20-slim
+FROM alpine
 
-# force rebuild to ensure git is installed
+RUN apk add --no-cache bash nodejs npm git \
-RUN apt-get update && apt-get install -y --no-install-recommends \
+    && node --version \
-    git \
+    && npm --version \
-    && rm -rf /var/lib/apt/lists/*
+    && git --version
 
 WORKDIR /action
 

app/git.js

@@ -11,9 +11,31 @@ function git(args, cwd) {
 
 export async function commitAndPush(workspace) {
   const repoDir = path.join(workspace, GITEA_REPOSITORY);
-  const remoteUrl = GITEA_SERVER_URL.replace(/\/$/, '').replace('https://', `https://${GITEA_TOKEN}@`).replace('http://', `http://${GITEA_TOKEN}@`) + `/${GITEA_REPOSITORY}.git`;
+  const remoteUrl = GITEA_SERVER_URL.replace(/\/$/, '')
+    .replace('https://', `https://${GITEA_TOKEN}@`)
+    .replace('http://', `http://${GITEA_TOKEN}@`) + `/${GITEA_REPOSITORY}.git`;
+
+  // try {
+  //   git(['config', 'user.email', 'ai-review[bot]@gitea'], repoDir);
+  //   git(['config', 'user.name', 'AI Review Bot'], repoDir);
+  //   git(['fetch', 'origin', PR_HEAD_BRANCH], repoDir);
+  //   git(['checkout', PR_HEAD_BRANCH], repoDir);
+  //   git(['add', FINDINGS_PATH], repoDir);
+
+  //   const status = git(['status', '--porcelain'], repoDir);
+  //   if (!status) {
+  //     console.log('  findings.json 無變更，跳過 commit');
+  //     return;
+  //   }
+
+  //   const out = git(['commit', '-m', 'chore: update ai-review findings [skip ci]'], repoDir);
+  //   const commitHash = out.match(/\[.+ ([a-f0-9]+)\]/)?.[1] || 'unknown';
+  //   git(['push', remoteUrl, PR_HEAD_BRANCH], repoDir);
+  //   console.log(`  ✅ persisted findings  commit=${commitHash}  push=${PR_HEAD_BRANCH}`);
+  // } catch (e) {
+  //   console.log(`  ⚠️  Runner failed: commit/push 失敗: ${e.message}`);
+  // }
 
-  try {
   git(['config', 'user.email', 'ai-review[bot]@gitea'], repoDir);
   git(['config', 'user.name', 'AI Review Bot'], repoDir);
   git(['fetch', 'origin', PR_HEAD_BRANCH], repoDir);
@@ -30,7 +52,4 @@ export async function commitAndPush(workspace) {
   const commitHash = out.match(/\[.+ ([a-f0-9]+)\]/)?.[1] || 'unknown';
   git(['push', remoteUrl, PR_HEAD_BRANCH], repoDir);
   console.log(`  ✅ persisted findings  commit=${commitHash}  push=${PR_HEAD_BRANCH}`);
-  } catch (e) {
-    console.log(`  ⚠️  Runner failed: commit/push 失敗: ${e.message}`);
-  }
 }