From 76760c4d2975c019430f955df99530bdd0348fe0 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=E5=B0=8F=E6=A9=98?= Date: Tue, 28 Apr 2026 14:16:23 +0000 Subject: [PATCH] =?UTF-8?q?fix:=20reviewer=20=E2=80=94=20remove=20'missing?= =?UTF-8?q?=20files=20from=20planner'=20check,=20planner=20designs=20roles?= =?UTF-8?q?=20not=20files?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- workflows/gitea-issue-solver/roles/reviewer/prompt.ts | 3 +-- workflows/sense-generator/roles/reviewer/prompt.ts | 3 +-- workflows/workflow-generator/roles/reviewer/prompt.ts | 3 +-- 3 files changed, 3 insertions(+), 6 deletions(-) diff --git a/workflows/gitea-issue-solver/roles/reviewer/prompt.ts b/workflows/gitea-issue-solver/roles/reviewer/prompt.ts index f12f61a..4428a5b 100644 --- a/workflows/gitea-issue-solver/roles/reviewer/prompt.ts +++ b/workflows/gitea-issue-solver/roles/reviewer/prompt.ts @@ -18,10 +18,9 @@ Run these commands and analyze the output: ### 🔴 Reject (approved: false) — tell coder exactly what to fix - **Garbage files**: node_modules/, .DS_Store, pnpm cache artifacts (e.g. \`false/\` directory), build outputs (dist/) that shouldn't be committed - **Secrets/credentials**: API keys, tokens, passwords hardcoded in the diff -- **Unrelated changes**: files modified outside the scope of the planner's design +- **Unrelated changes**: files modified outside the scope of the task - **Debug code**: console.log, debugger statements, TODO/FIXME without context - **Hardcoded paths** that should be configurable -- **Missing files** from the planner's design - **Wrong patterns**: dynamic import(), interfaces instead of types, broken routing ### ✅ Approve (approved: true) — no comment needed diff --git a/workflows/sense-generator/roles/reviewer/prompt.ts b/workflows/sense-generator/roles/reviewer/prompt.ts index f12f61a..4428a5b 100644 --- a/workflows/sense-generator/roles/reviewer/prompt.ts +++ b/workflows/sense-generator/roles/reviewer/prompt.ts @@ -18,10 +18,9 @@ Run these commands and analyze the output: ### 🔴 Reject (approved: false) — tell coder exactly what to fix - **Garbage files**: node_modules/, .DS_Store, pnpm cache artifacts (e.g. \`false/\` directory), build outputs (dist/) that shouldn't be committed - **Secrets/credentials**: API keys, tokens, passwords hardcoded in the diff -- **Unrelated changes**: files modified outside the scope of the planner's design +- **Unrelated changes**: files modified outside the scope of the task - **Debug code**: console.log, debugger statements, TODO/FIXME without context - **Hardcoded paths** that should be configurable -- **Missing files** from the planner's design - **Wrong patterns**: dynamic import(), interfaces instead of types, broken routing ### ✅ Approve (approved: true) — no comment needed diff --git a/workflows/workflow-generator/roles/reviewer/prompt.ts b/workflows/workflow-generator/roles/reviewer/prompt.ts index f12f61a..4428a5b 100644 --- a/workflows/workflow-generator/roles/reviewer/prompt.ts +++ b/workflows/workflow-generator/roles/reviewer/prompt.ts @@ -18,10 +18,9 @@ Run these commands and analyze the output: ### 🔴 Reject (approved: false) — tell coder exactly what to fix - **Garbage files**: node_modules/, .DS_Store, pnpm cache artifacts (e.g. \`false/\` directory), build outputs (dist/) that shouldn't be committed - **Secrets/credentials**: API keys, tokens, passwords hardcoded in the diff -- **Unrelated changes**: files modified outside the scope of the planner's design +- **Unrelated changes**: files modified outside the scope of the task - **Debug code**: console.log, debugger statements, TODO/FIXME without context - **Hardcoded paths** that should be configurable -- **Missing files** from the planner's design - **Wrong patterns**: dynamic import(), interfaces instead of types, broken routing ### ✅ Approve (approved: true) — no comment needed