actions/code-review
2
0
Fork 0
Code Pull Requests Actions Releases Activity

fix: streamline commitAndPush function by removing redundant code and improving error handling

Browse Source
This commit is contained in:
Jeffery
2026-05-11 10:53:59 +00:00
parent 69624a542e
commit eae73092ad
1 changed files with 17 additions and 34 deletions
Download Patch File Download Diff File Expand all files Collapse all files
app/git.js
+17 -34
View File
@@ -15,41 +15,24 @@ export async function commitAndPush(workspace) {
.replace('https://', `https://${GITEA_TOKEN}@`) .replace('https://', `https://${GITEA_TOKEN}@`)
.replace('http://', `http://${GITEA_TOKEN}@`) + `/${GITEA_REPOSITORY}.git`; .replace('http://', `http://${GITEA_TOKEN}@`) + `/${GITEA_REPOSITORY}.git`;
// try { try {
// git(['config', 'user.email', 'ai-review[bot]@gitea'], repoDir); git(['config', 'user.email', 'ai-review[bot]@gitea'], repoDir);
// git(['config', 'user.name', 'AI Review Bot'], repoDir); git(['config', 'user.name', 'AI Review Bot'], repoDir);
// git(['fetch', 'origin', PR_HEAD_BRANCH], repoDir); git(['fetch', 'origin', PR_HEAD_BRANCH], repoDir);
// git(['checkout', PR_HEAD_BRANCH], repoDir); git(['checkout', PR_HEAD_BRANCH], repoDir);
// git(['add', FINDINGS_PATH], repoDir); git(['add', FINDINGS_PATH], repoDir);
// const status = git(['status', '--porcelain'], repoDir); const status = git(['status', '--porcelain'], repoDir);
// if (!status) { if (!status) {
// console.log(' findings.json 無變更，跳過 commit'); console.log(' findings.json 無變更，跳過 commit');
// return; return;
// } }
// const out = git(['commit', '-m', 'chore: update ai-review findings [skip ci]'], repoDir); const out = git(['commit', '-m', 'chore: update ai-review findings [skip ci]'], repoDir);
// const commitHash = out.match(/\[.+ ([a-f0-9]+)\]/)?.[1] || 'unknown'; const commitHash = out.match(/\[.+ ([a-f0-9]+)\]/)?.[1] || 'unknown';
// git(['push', remoteUrl, PR_HEAD_BRANCH], repoDir); git(['push', remoteUrl, PR_HEAD_BRANCH], repoDir);
// console.log(` ✅ persisted findings commit=${commitHash} push=${PR_HEAD_BRANCH}`); console.log(` ✅ persisted findings commit=${commitHash} push=${PR_HEAD_BRANCH}`);
// } catch (e) { } catch (e) {
// console.log(` ⚠️ Runner failed: commit/push 失敗: ${e.message}`); console.log(` ⚠️ Runner failed: commit/push 失敗: ${e.message}`);
// }
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}`);
} }