From ac17585d5a7c8d964b9e423ea572d3b7df2566f4 Mon Sep 17 00:00:00 2001 From: Agent Date: Tue, 7 Apr 2026 18:22:30 +0000 Subject: [PATCH] =?UTF-8?q?fix:=20fix:=20entrypoint-reproduce.sh=20ignores?= =?UTF-8?q?=20DISINTO=5FFORMULA=20env=20var=20=E2=80=94=20always=20runs=20?= =?UTF-8?q?reproduce=20formula=20(#356)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- docker/reproduce/entrypoint-reproduce.sh | 218 ++++++++++++++++++++++- 1 file changed, 209 insertions(+), 9 deletions(-) diff --git a/docker/reproduce/entrypoint-reproduce.sh b/docker/reproduce/entrypoint-reproduce.sh index 723c5dc..c36192a 100644 --- a/docker/reproduce/entrypoint-reproduce.sh +++ b/docker/reproduce/entrypoint-reproduce.sh @@ -38,11 +38,20 @@ REPRODUCE_TIMEOUT="${REPRODUCE_TIMEOUT_MINUTES:-15}" LOGFILE="/home/agent/data/logs/reproduce.log" SCREENSHOT_DIR="/home/agent/data/screenshots" +# --------------------------------------------------------------------------- +# Determine agent type early for log prefix +# --------------------------------------------------------------------------- +if [ "${DISINTO_FORMULA:-reproduce}" = "triage" ]; then + AGENT_TYPE="triage" +else + AGENT_TYPE="reproduce" +fi + # --------------------------------------------------------------------------- # Logging # --------------------------------------------------------------------------- log() { - printf '[%s] reproduce: %s\n' "$(date -u '+%Y-%m-%d %H:%M:%S UTC')" "$*" | tee -a "$LOGFILE" + printf '[%s] %s: %s\n' "$(date -u '+%Y-%m-%d %H:%M:%S UTC')" "$AGENT_TYPE" "$*" | tee -a "$LOGFILE" } # --------------------------------------------------------------------------- @@ -85,7 +94,7 @@ export PROJECT_NAME PROJECT_REPO_ROOT="/home/agent/repos/${PROJECT_NAME}" -if [ "${DISINTO_FORMULA:-reproduce}" = "triage" ]; then +if [ "$AGENT_TYPE" = "triage" ]; then log "Starting triage-agent for issue #${ISSUE_NUMBER} (project: ${PROJECT_NAME})" else log "Starting reproduce-agent for issue #${ISSUE_NUMBER} (project: ${PROJECT_NAME})" @@ -198,12 +207,202 @@ elif [ -n "$FORMULA_STACK_SCRIPT" ]; then fi # --------------------------------------------------------------------------- -# Build Claude prompt for reproduction +# Build Claude prompt based on agent type # --------------------------------------------------------------------------- TIMESTAMP=$(date -u '+%Y%m%d-%H%M%S') SCREENSHOT_PREFIX="${SCREENSHOT_DIR}/issue-${ISSUE_NUMBER}-${TIMESTAMP}" -CLAUDE_PROMPT=$(cat < + e. Search for related issues or TODOs in the code: + grep -r "TODO\|FIXME\|HACK" -- + +Capture for each layer: + - The data shape flowing in and out (field names, types, nullability) + - Whether the layer's behavior matches its documented contract + - Any discrepancy found + +If a clear root cause becomes obvious during tracing, note it and continue +checking whether additional causes exist downstream. + +### Step 3: Add debug instrumentation on a throwaway branch +Use ~30% of your total turn budget here. Only instrument after tracing has +identified the most likely failure points — do not instrument blindly. + +1. Create a throwaway debug branch (NEVER commit this to main): + cd "\$PROJECT_REPO_ROOT" + git checkout -b debug/triage-\${ISSUE_NUMBER} + +2. Add targeted logging at the layer boundaries identified during tracing: + - Console.log / structured log statements around the suspicious code path + - Log the actual values flowing through: inputs, outputs, intermediate state + - Add verbose mode flags if the stack supports them + - Keep instrumentation minimal — only what confirms or refutes the hypothesis + +3. Restart the stack using the configured script (if set): + \${stack_script:-"# No stack_script configured — restart manually or connect to staging"} + +4. Re-run the reproduction steps from the reproduce-agent findings. + +5. Observe and capture new output: + - Paste relevant log lines into your working notes + - Note whether the observed values match or contradict the hypothesis + +6. If the first instrumentation pass is inconclusive, iterate: + - Narrow the scope to the next most suspicious boundary + - Re-instrument, restart, re-run + - Maximum 2-3 instrumentation rounds before declaring inconclusive + +Do NOT push the debug branch. It will be deleted in the cleanup step. + +### Step 4: Decompose root causes into backlog issues +After tracing and instrumentation, articulate each distinct root cause. + +For each root cause found: + +1. Determine the relationship to other causes: + - Layered (one causes another) → use Depends-on in the issue body + - Independent (separate code paths fail independently) → use Related + +2. Create a backlog issue for each root cause: + curl -sf -X POST "\${FORGE_API}/issues" \\ + -H "Authorization: token \${FORGE_TOKEN}" \\ + -H "Content-Type: application/json" \\ + -d '{ + "title": "fix: ", + "body": "## Root cause\\n\\n\\n## Fix suggestion\\n\\n\\n## Context\\nDecomposed from #\${ISSUE_NUMBER} (cause N of M)\\n\\n## Dependencies\\n<#X if this depends on another cause being fixed first>", + "labels": ["backlog"] + }' + +3. Note the newly created issue numbers. + +If only one root cause is found, still create a single backlog issue with +the specific code location and fix suggestion. + +If the investigation is inconclusive (no clear root cause found), skip this +step and proceed directly to link-back with the inconclusive outcome. + +### Step 5: Update original issue and relabel +Post a summary comment on the original issue and update its labels. + +#### If root causes were found (conclusive): + +Post a comment: + "## Triage findings + + Found N root cause(s): + - #X — (cause 1 of N) + - #Y — (cause 2 of N, depends on #X) + + Data flow traced: + Instrumentation: + + Next step: backlog issues above will be implemented in dependency order." + +Then swap labels: + - Remove: in-triage + - Add: in-progress + +#### If investigation was inconclusive (turn budget exhausted): + +Post a comment: + "## Triage — inconclusive + + Traced: + Tried: + Hypothesis: + + No definitive root cause identified. Leaving in-triage for supervisor + to handle as a stale triage session." + +Do NOT relabel. Leave in-triage. The supervisor monitors stale triage +sessions and will escalate or reassign. + +### Step 6: Delete throwaway debug branch +Always delete the debug branch, even if the investigation was inconclusive. + +1. Switch back to the main branch: + cd "\$PROJECT_REPO_ROOT" + git checkout "\$PRIMARY_BRANCH" + +2. Delete the local debug branch: + git branch -D debug/triage-\${ISSUE_NUMBER} + +3. Confirm no remote was pushed (if accidentally pushed, delete it too): + git push origin --delete debug/triage-\${ISSUE_NUMBER} 2>/dev/null || true + +4. Verify the worktree is clean: + git status + git worktree list + +A clean repo is a prerequisite for the next dev-agent run. Never leave +debug branches behind — they accumulate and pollute the branch list. + +## Notes +- The application is accessible at localhost (network_mode: host) +- Budget: 70% tracing data flow, 30% instrumented re-runs +- Timeout: \${FORMULA_TIMEOUT_MINUTES} minutes total (or until turn limit) +- Stack lock is held for the full run +- If stack_script is empty, connect to existing staging environment + +Begin now. +PROMPT + ) +else + # Reproduce-agent prompt: reproduce the bug and report findings + CLAUDE_PROMPT=$(cat </dev/null || echo '(no output)')\n\`\`\`" else FINDINGS="Reproduce-agent completed but did not write a findings report. Claude output:\n\`\`\`\n$(tail -100 "/tmp/reproduce-claude-output-${ISSUE_NUMBER}.txt" 2>/dev/null || echo '(no output)')\n\`\`\`" @@ -403,8 +603,8 @@ _post_comment() { BUG_REPORT_ID=$(_label_id "bug-report" "#e4e669") _remove_label "$ISSUE_NUMBER" "$BUG_REPORT_ID" -# Determine agent name for comments -if [ "${DISINTO_FORMULA:-reproduce}" = "triage" ]; then +# Determine agent name for comments (based on AGENT_TYPE set at script start) +if [ "$AGENT_TYPE" = "triage" ]; then AGENT_NAME="Triage-agent" else AGENT_NAME="Reproduce-agent"