@doist/doistbot-cli 1.0.5 → 1.0.6

package/sandbox/dist/tasks/issue-fix-retry/fix-retry.js

@@ -1,9 +1,11 @@
 import { performance } from 'node:perf_hooks';
+import { SpanStatusCode } from '@opentelemetry/api';
 import { isKnownDoistbotLogin } from '../../core/bot-logins.js';
 import { recordTaskMetrics } from '../../core/datadog-metrics.js';
 import { logger } from '../../core/logging.js';
 import { removeIssueCommentReactionIfPresent } from '../../core/reactions.js';
 import { resolveIssueRepositories } from '../../core/repository-resolver.js';
+import { recordDecision, withPhase } from '../../core/tracing.js';
 import { parseIssueTriageContext } from '../issue-triage/context.js';
 import { attemptFix } from '../issue-triage/fix-attempt.js';
 import { parseFixAssessment } from '../issue-triage/output.js';
@@ -19,217 +21,240 @@ async function main() {
         repository: `${ctx.owner}/${ctx.repo}`,
         issueNumber: ctx.issueNumber,
     });
-    logger.info('Fix retry task started', {
+    // Root span for the whole run. The Codex fix attempt is wrapped in its own
+    // issue-fix-retry.fix-attempt phase below, keeping its nested model span in
+    // this task's namespace rather than triage's.
+    await withPhase('issue-fix-retry', {
         task: 'issue-fix-retry',
-        issueNumber: ctx.issueNumber,
-        commentId: ctx.commentId,
-    });
-    const { app, readOctokit, writeOctokit } = await createGitHubClients(ctx);
-    async function postComment(body) {
-        if (ctx.dryRun)
-            return;
-        await writeOctokit.rest.issues.createComment({
-            owner: ctx.owner,
-            repo: ctx.repo,
-            issue_number: ctx.issueNumber,
-            body,
-        });
-    }
-    try {
-        const [issue, comments] = await Promise.all([
-            fetchIssue(readOctokit, ctx),
-            fetchIssueComments(readOctokit, ctx),
-        ]);
-        // Find the most recent triage comment from the bot (search backwards to get the latest one)
-        let triageComment;
-        for (let i = comments.length - 1; i >= 0; i--) {
-            const comment = comments[i];
-            if (comment &&
-                isKnownDoistbotLogin(comment.user?.login) &&
-                comment.body?.includes(TRIAGE_MARKER)) {
-                triageComment = comment;
-                break;
-            }
-        }
-        if (!triageComment?.body) {
-            logger.warn('No triage comment found for fix retry', {
-                task: 'issue-fix-retry',
-                issueNumber: ctx.issueNumber,
+        'gen_ai.operation.name': 'invoke_agent',
+        'gen_ai.agent.name': 'doistbot.issue-fix-retry',
+        'issue.number': ctx.issueNumber,
+        'comment.id': ctx.commentId,
+        dry_run: ctx.dryRun,
+    }, async (span) => {
+        const { app, readOctokit, writeOctokit } = await createGitHubClients(ctx);
+        async function postComment(body) {
+            if (ctx.dryRun)
+                return;
+            await writeOctokit.rest.issues.createComment({
+                owner: ctx.owner,
+                repo: ctx.repo,
+                issue_number: ctx.issueNumber,
+                body,
             });
-            await postComment('No previous triage found on this issue. Run `@doistbot /triage` first, then retry with `@doistbot /fix`.');
-            return;
         }
-        // Strip the triage marker to get the raw triage output
-        const triageOutput = triageComment.body.replace(TRIAGE_MARKER, '').trim();
-        const fixAssessment = parseFixAssessment(triageOutput);
-        if (!fixAssessment) {
-            logger.warn('No fix assessment found in triage output', {
+        try {
+            const [issue, comments] = await Promise.all([
+                fetchIssue(readOctokit, ctx),
+                fetchIssueComments(readOctokit, ctx),
+            ]);
+            // Find the most recent triage comment from the bot (search backwards to get the latest one)
+            let triageComment;
+            for (let i = comments.length - 1; i >= 0; i--) {
+                const comment = comments[i];
+                if (comment &&
+                    isKnownDoistbotLogin(comment.user?.login) &&
+                    comment.body?.includes(TRIAGE_MARKER)) {
+                    triageComment = comment;
+                    break;
+                }
+            }
+            if (!triageComment?.body) {
+                // User-actionable (no prior triage), not a malfunction → span stays ok.
+                logger.warn('No triage comment found for fix retry', {
+                    task: 'issue-fix-retry',
+                    issueNumber: ctx.issueNumber,
+                });
+                await postComment('No previous triage found on this issue. Run `@doistbot /triage` first, then retry with `@doistbot /fix`.');
+                return;
+            }
+            // Strip the triage marker to get the raw triage output
+            const triageOutput = triageComment.body.replace(TRIAGE_MARKER, '').trim();
+            const fixAssessment = parseFixAssessment(triageOutput);
+            if (!fixAssessment) {
+                logger.warn('No fix assessment found in triage output', {
+                    task: 'issue-fix-retry',
+                    issueNumber: ctx.issueNumber,
+                });
+                await postComment("The previous triage doesn't include an automated fix assessment. Try running `@doistbot /triage` again on a repo with auto-fix enabled.");
+                return;
+            }
+            if (fixAssessment.targetRepo) {
+                metricRepository = fixAssessment.targetRepo;
+            }
+            logger.info('Fix assessment found from previous triage', {
                 task: 'issue-fix-retry',
                 issueNumber: ctx.issueNumber,
+                classification: fixAssessment.classification,
+                confidence: fixAssessment.confidence,
+                scope: fixAssessment.scope,
             });
-            await postComment("The previous triage doesn't include an automated fix assessment. Try running `@doistbot /triage` again on a repo with auto-fix enabled.");
-            return;
-        }
-        if (fixAssessment.targetRepo) {
-            metricRepository = fixAssessment.targetRepo;
-        }
-        logger.info('Fix assessment found from previous triage', {
-            task: 'issue-fix-retry',
-            issueNumber: ctx.issueNumber,
-            classification: fixAssessment.classification,
-            confidence: fixAssessment.confidence,
-            scope: fixAssessment.scope,
-        });
-        // Resolve repositories to clone. Prefer the target repo from the assessment
-        // (stable across label changes) and fall back to label-based resolution.
-        const reposDir = prepareWorkspaceReposDirectory();
-        const clonedRepositories = [];
-        if (fixAssessment.targetRepo) {
-            const resolution = resolveIssueRepositories(extractLabelNames(issue.labels));
-            const repoTarget = resolution.status === 'resolved'
-                ? resolution.repositories.find((r) => r.fullName === fixAssessment.targetRepo)
-                : undefined;
-            if (repoTarget) {
-                try {
-                    const cloneResult = await cloneRepository({
-                        app,
-                        repository: repoTarget,
-                        reposDir,
-                        fallbackInstallationId: ctx.installationId,
-                    });
-                    clonedRepositories.push(cloneResult);
-                    metricRepository = cloneResult.repository.fullName;
+            // Resolve repositories to clone. Prefer the target repo from the assessment
+            // (stable across label changes) and fall back to label-based resolution.
+            const reposDir = prepareWorkspaceReposDirectory();
+            const clonedRepositories = [];
+            if (fixAssessment.targetRepo) {
+                const resolution = resolveIssueRepositories(extractLabelNames(issue.labels));
+                const repoTarget = resolution.status === 'resolved'
+                    ? resolution.repositories.find((r) => r.fullName === fixAssessment.targetRepo)
+                    : undefined;
+                if (repoTarget) {
+                    try {
+                        const cloneResult = await cloneRepository({
+                            app,
+                            repository: repoTarget,
+                            reposDir,
+                            fallbackInstallationId: ctx.installationId,
+                        });
+                        clonedRepositories.push(cloneResult);
+                        metricRepository = cloneResult.repository.fullName;
+                    }
+                    catch (error) {
+                        logger.warn('Fix retry clone failed for assessment target repo', {
+                            task: 'issue-fix-retry',
+                            repository: fixAssessment.targetRepo,
+                            error: error instanceof Error ? error.message : String(error),
+                        });
+                    }
                 }
-                catch (error) {
-                    logger.warn('Fix retry clone failed for assessment target repo', {
+            }
+            // Fall back to label-based resolution if the assessment target wasn't cloned
+            if (clonedRepositories.length === 0) {
+                const labelNames = extractLabelNames(issue.labels);
+                const resolution = resolveIssueRepositories(labelNames);
+                if (resolution.status === 'unresolved') {
+                    // User-actionable (labels don't map to a repo) → span stays ok.
+                    logger.warn('Fix retry failed: could not resolve repositories from labels', {
                         task: 'issue-fix-retry',
-                        repository: fixAssessment.targetRepo,
-                        error: error instanceof Error ? error.message : String(error),
+                        issueNumber: ctx.issueNumber,
+                        reason: resolution.reason,
                     });
+                    await postComment(`Could not resolve target repositories from the issue labels. Please check the labels and try again.\n\nReason: ${resolution.reason}`);
+                    return;
+                }
+                if (resolution.repositories.length === 1) {
+                    metricRepository = resolution.repositories[0].fullName;
+                }
+                for (const repository of resolution.repositories) {
+                    try {
+                        const cloneResult = await cloneRepository({
+                            app,
+                            repository,
+                            reposDir,
+                            fallbackInstallationId: ctx.installationId,
+                        });
+                        clonedRepositories.push(cloneResult);
+                    }
+                    catch (error) {
+                        logger.warn('Fix retry clone failed', {
+                            task: 'issue-fix-retry',
+                            repository: repository.fullName,
+                            error: error instanceof Error ? error.message : String(error),
+                        });
+                    }
                 }
             }
-        }
-        // Fall back to label-based resolution if the assessment target wasn't cloned
-        if (clonedRepositories.length === 0) {
-            const labelNames = extractLabelNames(issue.labels);
-            const resolution = resolveIssueRepositories(labelNames);
-            if (resolution.status === 'unresolved') {
-                logger.warn('Fix retry failed: could not resolve repositories from labels', {
+            if (clonedRepositories.length === 0) {
+                // Infrastructure failure → mark the run failed.
+                const message = 'Fix retry failed: all repository clones failed';
+                logger.error(message, {
                     task: 'issue-fix-retry',
                     issueNumber: ctx.issueNumber,
-                    reason: resolution.reason,
                 });
-                await postComment(`Could not resolve target repositories from the issue labels. Please check the labels and try again.\n\nReason: ${resolution.reason}`);
+                span.setStatus({ code: SpanStatusCode.ERROR, message });
+                await postComment('Fix retry failed — could not clone the target repositories. Please try again later.');
                 return;
             }
-            if (resolution.repositories.length === 1) {
-                metricRepository = resolution.repositories[0].fullName;
-            }
-            for (const repository of resolution.repositories) {
-                try {
-                    const cloneResult = await cloneRepository({
-                        app,
-                        repository,
-                        reposDir,
-                        fallbackInstallationId: ctx.installationId,
-                    });
-                    clonedRepositories.push(cloneResult);
-                }
-                catch (error) {
-                    logger.warn('Fix retry clone failed', {
-                        task: 'issue-fix-retry',
-                        repository: repository.fullName,
-                        error: error instanceof Error ? error.message : String(error),
+            const issueInstallationAuth = (await app.octokit.auth({
+                type: 'installation',
+                installationId: ctx.installationId,
+            }));
+            // Wrap the Codex fix attempt in a retry-namespaced phase; its
+            // model span (issue-fix-retry.fix-attempt.model) nests under it.
+            const fixResult = await withPhase('issue-fix-retry.fix-attempt', {
+                'gen_ai.operation.name': 'invoke_agent',
+                'gen_ai.agent.name': 'codex.fix',
+            }, () => attemptFix({
+                ctx,
+                octokit: writeOctokit,
+                triageOutput,
+                fixAssessment,
+                clonedRepos: clonedRepositories,
+                issueTitle: issue.title ?? 'Untitled issue',
+                issueUrl: issue.html_url ?? getIssueUrl(ctx.owner, ctx.repo, ctx.issueNumber),
+                ghToken: issueInstallationAuth.token,
+                force: true,
+                modelPhaseName: 'issue-fix-retry.fix-attempt.model',
+            }));
+            metricRepository = fixResult.repository;
+            metricOutcome = fixResult.attempted && fixResult.prCreated ? 'success' : 'failure';
+            const reason = 'reason' in fixResult ? fixResult.reason : undefined;
+            const isDryRun = 'dryRun' in fixResult && fixResult.dryRun === true;
+            recordDecision('issue-fix-retry.outcome', {
+                attempted: fixResult.attempted,
+                pr_created: fixResult.attempted && fixResult.prCreated,
+                reason,
+                pr_url: 'prUrl' in fixResult ? fixResult.prUrl : undefined,
+            });
+            // No PR is a failed retry — unless it was a dry run (expected).
+            if (!fixResult.attempted || !fixResult.prCreated) {
+                if (!isDryRun) {
+                    span.setStatus({
+                        code: SpanStatusCode.ERROR,
+                        message: reason ?? 'Fix retry produced no PR',
                     });
                 }
+                await postComment(`Fix retry failed to produce a PR.\n\nReason: ${reason ?? 'Unknown reason'}`);
             }
         }
-        if (clonedRepositories.length === 0) {
-            logger.error('Fix retry failed: all repository clones failed', {
+        catch (error) {
+            const message = error instanceof Error ? error.message : String(error);
+            logger.error('Fix retry task failed', {
                 task: 'issue-fix-retry',
                 issueNumber: ctx.issueNumber,
+                error: message,
+                durationMs: performance.now() - startedAt,
             });
-            await postComment('Fix retry failed — could not clone the target repositories. Please try again later.');
-            return;
-        }
-        const issueInstallationAuth = (await app.octokit.auth({
-            type: 'installation',
-            installationId: ctx.installationId,
-        }));
-        const fixResult = await attemptFix({
-            ctx,
-            octokit: writeOctokit,
-            triageOutput,
-            fixAssessment,
-            clonedRepos: clonedRepositories,
-            issueTitle: issue.title ?? 'Untitled issue',
-            issueUrl: issue.html_url ?? getIssueUrl(ctx.owner, ctx.repo, ctx.issueNumber),
-            ghToken: issueInstallationAuth.token,
-            force: true,
-        });
-        logger.info('Fix retry result', {
-            task: 'issue-fix-retry',
-            issueNumber: ctx.issueNumber,
-            attempted: fixResult.attempted,
-            prCreated: fixResult.attempted && fixResult.prCreated,
-            reason: 'reason' in fixResult ? fixResult.reason : undefined,
-            prUrl: 'prUrl' in fixResult ? fixResult.prUrl : undefined,
-            durationMs: performance.now() - startedAt,
-        });
-        metricRepository = fixResult.repository;
-        metricOutcome = fixResult.attempted && fixResult.prCreated ? 'success' : 'failure';
-        // Inform the user if no PR was created
-        if (!fixResult.attempted || !fixResult.prCreated) {
-            const reason = 'reason' in fixResult ? fixResult.reason : 'Unknown reason';
-            await postComment(`Fix retry failed to produce a PR.\n\nReason: ${reason}`);
-        }
-    }
-    catch (error) {
-        logger.error('Fix retry task failed', {
-            task: 'issue-fix-retry',
-            issueNumber: ctx.issueNumber,
-            error: error instanceof Error ? error.message : String(error),
-            durationMs: performance.now() - startedAt,
-        });
-        try {
-            await postComment('Sorry, the fix retry encountered an error. Please try again or contact the team maintaining the bot.');
+            // The catch swallows the error (posts a user message), so set the
+            // span status explicitly — withPhase only auto-marks ERROR on rethrow.
+            span.setStatus({ code: SpanStatusCode.ERROR, message });
+            try {
+                await postComment('Sorry, the fix retry encountered an error. Please try again or contact the team maintaining the bot.');
+            }
+            catch (commentError) {
+                logger.warn('Failed to post fix retry failure response', {
+                    task: 'issue-fix-retry',
+                    issueNumber: ctx.issueNumber,
+                    error: commentError instanceof Error
+                        ? commentError.message
+                        : String(commentError),
+                });
+            }
+            exitCode = 1;
         }
-        catch (commentError) {
-            logger.warn('Failed to post fix retry failure response', {
-                task: 'issue-fix-retry',
-                issueNumber: ctx.issueNumber,
-                error: commentError instanceof Error ? commentError.message : String(commentError),
+        finally {
+            await removeIssueCommentReactionIfPresent({
+                octokit: writeOctokit,
+                owner: ctx.owner,
+                repo: ctx.repo,
+                commentId: ctx.commentId,
+                reactionId: ctx.reactionId,
+                warnMessage: 'Failed to remove fix retry reaction',
+                successMessage: 'Fix retry reaction removed',
+                logMeta: {
+                    task: 'issue-fix-retry',
+                    reactionId: ctx.reactionId,
+                },
+            });
+            await recordTaskMetrics({
+                type: 'fix',
+                repository: metricRepository,
+                model: 'openai',
+                outcome: metricOutcome,
+                durationMs: performance.now() - startedAt,
             });
         }
-        exitCode = 1;
-    }
-    finally {
-        await removeIssueCommentReactionIfPresent({
-            octokit: writeOctokit,
-            owner: ctx.owner,
-            repo: ctx.repo,
-            commentId: ctx.commentId,
-            reactionId: ctx.reactionId,
-            warnMessage: 'Failed to remove fix retry reaction',
-            successMessage: 'Fix retry reaction removed',
-            logMeta: {
-                task: 'issue-fix-retry',
-                reactionId: ctx.reactionId,
-            },
-        });
-        await recordTaskMetrics({
-            type: 'fix',
-            repository: metricRepository,
-            model: 'openai',
-            outcome: metricOutcome,
-            durationMs: performance.now() - startedAt,
-        });
-        logger.info('Fix retry task finalized', {
-            task: 'issue-fix-retry',
-            issueNumber: ctx.issueNumber,
-            durationMs: performance.now() - startedAt,
-        });
-    }
+    });
     if (exitCode !== 0) {
         process.exit(exitCode);
     }

package/sandbox/dist/tasks/issue-summarize/model.js

@@ -4,7 +4,7 @@ const ISSUE_SUMMARIZE_TIMEOUT_MS = 5 * 60 * 1000;
 const ISSUE_SUMMARIZE_MULTIMODAL_MODEL = 'google/gemini-2.5-pro';
 export function createIssueSummarizeModel(apiKey) {
     return createPiInvoker({
-        provider: 'gemini',
+        provider: 'zai',
         apiKey,
         piProviderId: 'openrouter',
         cwd: process.cwd(),

package/sandbox/dist/tasks/issue-triage/clone.js

@@ -0,0 +1,40 @@
+import { logger } from '../../core/logging.js';
+import { withPhase } from '../../core/tracing.js';
+import { cloneRepository } from './shared.js';
+export async function cloneTriageRepositories(params) {
+    return withPhase('triage.clone', { 'repos.count': params.repositories.length }, async (span) => {
+        const cloneFn = params.cloneRepositoryImpl ?? cloneRepository;
+        const clonedRepositories = [];
+        const failedRepositories = [];
+        for (const repository of params.repositories) {
+            try {
+                const cloneResult = await cloneFn({
+                    app: params.app,
+                    repository,
+                    reposDir: params.reposDir,
+                    fallbackInstallationId: params.fallbackInstallationId,
+                });
+                clonedRepositories.push(cloneResult);
+            }
+            catch (err) {
+                const message = err instanceof Error ? err.message : String(err);
+                // Keep per-repo diagnostics (auth/git errors) visible even
+                // when EVERY clone fails and the orchestrator bails early.
+                logger.warn('Repository clone failed', {
+                    repository: repository.fullName,
+                    error: message,
+                });
+                failedRepositories.push({
+                    repository: repository.fullName,
+                    error: message,
+                });
+            }
+        }
+        span.setAttributes({
+            'clone.success_count': clonedRepositories.length,
+            'clone.failed_count': failedRepositories.length,
+            'repos.count': params.repositories.length,
+        });
+        return { clonedRepositories, failedRepositories };
+    });
+}

package/sandbox/dist/tasks/issue-triage/context.js

@@ -1,4 +1,6 @@
 import { optionalPositiveInt, requiredEnv, requiredPositiveInt } from '../../core/shared.js';
+import { withPhase } from '../../core/tracing.js';
+import { extractLabelNames, fetchIssue, fetchIssueComments, } from './shared.js';
 function parseOptionalBoolean(value) {
     const normalized = value?.trim().toLowerCase();
     if (normalized === 'true')
@@ -28,6 +30,7 @@ export function parseIssueTriageContext(env = process.env) {
         privateKey: requiredEnv('PRIVATE_KEY', env),
         githubToken: requiredEnv('GITHUB_TOKEN', env),
         openaiApiKey: parseOpenAIApiKey(env),
+        geminiApiKey: env.GEMINI_API_KEY?.trim() || undefined,
         commentId,
         reactionId,
         placeholderCommentId: optionalPositiveInt(env.PLACEHOLDER_COMMENT_ID),
@@ -35,6 +38,27 @@ export function parseIssueTriageContext(env = process.env) {
         dryRun: env.DRY_RUN?.toLowerCase() === 'true',
         triggerType,
         enableAutoFixOverride: parseOptionalBoolean(env.ENABLE_AUTO_FIX),
-        openRouterApiKey: env.OPENROUTER_API_KEY?.trim() || undefined,
     };
 }
+export async function loadTriageContext(params) {
+    return withPhase('triage.context', { 'issue.number': params.issueNumber }, async (span) => {
+        const [issue, comments] = await Promise.all([
+            fetchIssue(params.octokit, {
+                owner: params.owner,
+                repo: params.repo,
+                issueNumber: params.issueNumber,
+            }),
+            fetchIssueComments(params.octokit, {
+                owner: params.owner,
+                repo: params.repo,
+                issueNumber: params.issueNumber,
+            }),
+        ]);
+        const labelNames = extractLabelNames(issue.labels);
+        span.setAttributes({
+            'comments.count': comments.length,
+            'labels.count': labelNames.length,
+        });
+        return { issue, comments, labelNames };
+    });
+}

package/sandbox/dist/tasks/issue-triage/fix-attempt.js

@@ -164,8 +164,20 @@ export function buildFollowUpCommentBody({ prUrl, targetRepoFullName, fixAssessm
         `The PR targets \`${targetRepoFullName}\`. Please review it carefully before merging.`,
     ].join('\n');
 }
-function isNonRetryableFailure(reason) {
-    return reason.includes(CODEX_CANNOT_FIX_REASON);
+// Detect the "## Cannot Fix" verdict and pull Codex's own explanation in one pass,
+// so the failure comment can tell the human *why* it bailed, not just that it did.
+// Returns null when the marker is absent (not a cannot-fix verdict), the explanation
+// string when present, or '' when the model gave the verdict with no explanation.
+// The full tail is intentional — the prompt makes "## Cannot Fix" the terminal output.
+// ponytail: 2000-char cap keeps the comment readable; raise it if reasons get truncated.
+export function extractCannotFixDetail(output) {
+    const idx = output.indexOf(CANNOT_FIX_MARKER);
+    if (idx === -1)
+        return null;
+    return output
+        .slice(idx + CANNOT_FIX_MARKER.length)
+        .trim()
+        .slice(0, 2000);
 }
 async function findSourceIssueCommentByMarker({ octokit, ctx, marker, }) {
     const comments = await octokit.paginate(octokit.rest.issues.listComments, {
@@ -187,8 +199,6 @@ export async function shouldPostAutoFixFailureComment(fixResult, { dryRun, octok
     // a "didn't produce a fix PR" comment would be misleading.
     if (fixResult.prNumber !== undefined)
         return false;
-    if (isNonRetryableFailure(fixResult.reason))
-        return false;
     if (fixResult.reason.startsWith(FIX_LOOP_FAILED_REASON_PREFIX)) {
         try {
             const hasFixLoopStartComment = await sourceIssueHasFixLoopStartComment({ octokit, ctx });
@@ -231,9 +241,11 @@ export function buildAutoFixFailureCommentBody({ reason, targetRepoFullName, })
         '',
         '**Reason:**',
         '',
-        '```',
+        // Tilde fence, not backticks: the reason often carries Codex's own
+        // backtick-fenced output (logs, CLI), which would close a ``` block early.
+        '~~~',
         reason,
-        '```',
+        '~~~',
         '',
         'You can retry by commenting `@doistbot /fix`.',
     ].join('\n');
@@ -365,6 +377,9 @@ export async function runFixCore(input) {
         ? SPECULATIVE_CODEX_TIMEOUT_MS
         : AUTO_FIX_CODEX_TIMEOUT_MS;
     const codexResult = await invokeTriageModel({
+        // Distinct phase name: this is fix generation, not triage analysis.
+        // Callers (e.g. issue-fix-retry) override to keep it in their namespace.
+        phaseName: input.modelPhaseName ?? 'triage.fix-attempt.model',
         prompt,
         apiKey: ctx.openaiApiKey,
         cwd: targetRepo.localPath,
@@ -391,13 +406,19 @@ export async function runFixCore(input) {
         outputChars: fixDescription.length,
     });
     // Check if Codex reported it cannot fix
-    if (fixDescription.includes(CANNOT_FIX_MARKER)) {
+    const cannotFixDetail = extractCannotFixDetail(fixDescription);
+    if (cannotFixDetail !== null) {
         logger.info('Fix Codex reported cannot fix', {
             task: 'issue-triage',
             issueNumber: ctx.issueNumber,
             output: fixDescription.slice(0, 500),
         });
-        return { ok: false, reason: CODEX_CANNOT_FIX_REASON };
+        return {
+            ok: false,
+            reason: cannotFixDetail
+                ? `${CODEX_CANNOT_FIX_REASON}\n\n${cannotFixDetail}`
+                : CODEX_CANNOT_FIX_REASON,
+        };
     }
     // Capture the diff
     const diff = captureGitDiff(targetRepo.localPath);
@@ -499,6 +520,7 @@ export async function attemptFix(input) {
         issueUrl: input.issueUrl,
         ghToken: input.ghToken,
         force,
+        modelPhaseName: input.modelPhaseName,
     });
     if (!coreResult.ok) {
         return {
@@ -520,6 +542,7 @@ export async function attemptFix(input) {
             repository: targetRepo.repository.fullName,
             prCreated: false,
             reason: 'Dry run — PR not created.',
+            dryRun: true,
         };
     }
     let openedPr;