@qvac/harness 0.7.3 → 0.8.5

package/dist-lib/lib/tools/exec/adapters/python/python-adapter.js

@@ -308,7 +308,7 @@ export function stringLiteralProgramHint(source) {
     }
     return ('command is wrapped in a string literal, so Python evaluates it as text and runs ' +
         'nothing: remove the wrapping quotes (and any escaped \\" quotes) so the first line ' +
-        'of command is the first line of the program, e.g. from fpdf import FPDF.');
+        'of command is the first line of the program, e.g. import json.');
 }
 const COLLAPSED_COMPOUND = /;\s*(?:async\s+)?(?:for|if|while|with|def|class|try|match)\b/;
 export function checkCollapsedCompound(source) {
@@ -320,159 +320,6 @@ export function checkCollapsedCompound(source) {
         'cannot follow a semicolon. Pass multi-line Python source with newline characters, ' +
         'not a single-line semicolon chain.');
 }
-const MANGLED_PPTX = [
-    ['textframe', 'text_frame'],
-    ['addslide', 'add_slide'],
-    ['slidelayouts', 'slide_layouts'],
-    ['slidewidth', 'slide_width'],
-    ['slideheight', 'slide_height'],
-    ['wordwrap', 'word_wrap'],
-    ['addparagraph', 'add_paragraph'],
-    ['addtextbox', 'add_textbox'],
-    ['addpicture', 'add_picture'],
-    ['ppalign', 'PP_ALIGN']
-];
-export function mangledPptxApiHint(source) {
-    if (!/\bpptx\b|Presentation\s*\(/.test(source))
-        return null;
-    const found = [];
-    for (const [bad, good] of MANGLED_PPTX) {
-        if (new RegExp(`\\b${bad}\\b`, 'i').test(source) && !source.includes(good)) {
-            found.push(`${bad} → ${good}`);
-        }
-    }
-    if (found.length === 0) {
-        return null;
-    }
-    return (`python-pptx names keep their underscores — found stripped forms (${found.join(', ')}). ` +
-        'Restore them (text_frame not textframe, add_slide not addslide) and keep the skill ' +
-        'sample multi-line with real newlines.');
-}
-export function wrongRgbColorHint(source) {
-    if (!/\bpptx\b|Presentation\s*\(/.test(source))
-        return null;
-    if (/\bRGBColor\b/.test(source))
-        return null;
-    const code = source.replace(/#[^\n]*/g, '').replace(/'[^'\n]*'|"[^"\n]*"/g, '');
-    if (!/\bRgbColor\b|\brgbColor\b|\brgbcolor\b|\bRGB_Color\b|\brgb_color\b/.test(code)) {
-        return null;
-    }
-    return ('python-pptx color type is RGBColor (RGB in all caps): ' +
-        'from pptx.dml.color import RGBColor — not RgbColor / rgbColor. ' +
-        'Do not catch that ImportError and continue; fix the name and rerun.');
-}
-const FPDF_SIZE_KEYWORD = /\b(cell|multi_cell)\s*\([^)]*\b(width|height)\s*=/;
-export function fpdfSizeKeywordHint(source) {
-    if (!/\bfpdf\b|\bFPDF\b/.test(source))
-        return null;
-    const match = FPDF_SIZE_KEYWORD.exec(source);
-    if (!match) {
-        return null;
-    }
-    return (`fpdf2 ${match[1]}() has no "${match[2]}" keyword — the size parameters are the positional ` +
-        'w and h: write every line (headings and body) with pdf.multi_cell(0, 6, text, ' +
-        'new_x="LMARGIN", new_y="NEXT"). Rename the arguments and keep new_x/new_y; ' +
-        'deleting keywords to silence the error prints later lines on top of each other.');
-}
-export function fpdfKeywordErrorHint(stderr) {
-    const match = /TypeError: FPDF\.(\w+)\(\) got an unexpected keyword argument '(\w+)'/.exec(stderr);
-    if (!match) {
-        return null;
-    }
-    const sized = match[2] === 'width' || match[2] === 'height';
-    const fix = sized
-        ? 'the size parameters are the positional w and h'
-        : 'copy the exact call shape from the pdf skill sample';
-    return (`fpdf2 ${match[1]}() has no "${match[2]}" keyword — ${fix}: ` +
-        'write every line (headings and body) with pdf.multi_cell(0, 6, text, ' +
-        'new_x="LMARGIN", new_y="NEXT"). Rename the arguments and keep new_x/new_y; ' +
-        'deleting keywords to silence the error prints later lines on top of each other.');
-}
-const LATIN1_SUBSTITUTES = [
-    [/[\u2010-\u2015\u2212]/gu, '-'],
-    [/[\u2022\u25e6\u25aa\u25cf]/gu, '-'],
-    [/\u2026/gu, '...'],
-    [/\u2192/gu, '->'],
-    [/\u2190/gu, '<-'],
-    [/\u2248/gu, '~'],
-    [/\u2264/gu, '<='],
-    [/\u2265/gu, '>='],
-    [/\u2122/gu, '(TM)'],
-    [/\u20ac/gu, 'EUR'],
-    [/[\u2000-\u200a]/gu, ' '],
-    [/[\u200b-\u200d\ufeff]/gu, '']
-];
-export function normalizeFpdfSource(source) {
-    if (!/\bfpdf\b|\bFPDF\b/.test(source))
-        return source;
-    let out = source;
-    for (const [from, to] of LATIN1_SUBSTITUTES) {
-        out = out.replace(from, to);
-    }
-    return out;
-}
-const CURLY_QUOTE = /[\u2018\u2019\u201a\u2032\u201c\u201d\u201e\u2033]/u;
-const TEXT_WITHOUT_COORDS = /\.text\s*\(\s*['"]/;
-const TEXT_IS_STAMP_ONLY = 'fpdf2 text(x, y, string) paints at a fixed point with no wrapping \u2014 it is only for the ' +
-    'watermark stamp. Write document text (headings, paragraphs, bullets) with ' +
-    'pdf.multi_cell(0, 6, text, new_x="LMARGIN", new_y="NEXT").';
-export function fpdfTextCallHint(source) {
-    if (!/\bfpdf\b|\bFPDF\b/.test(source))
-        return null;
-    if (!TEXT_WITHOUT_COORDS.test(source))
-        return null;
-    return TEXT_IS_STAMP_ONLY;
-}
-export function fpdfTextErrorHint(stderr) {
-    if (!/TypeError: FPDF\.text\(\)/.test(stderr))
-        return null;
-    return TEXT_IS_STAMP_ONLY;
-}
-export function fpdfNonLatin1Hint(source) {
-    if (!/\bfpdf\b|\bFPDF\b/.test(source))
-        return null;
-    if (source.includes('.replace('))
-        return null;
-    const bad = /[^\u0000-\u00ff]/u.exec(normalizeFpdfSource(source))?.[0];
-    if (!bad) {
-        return null;
-    }
-    if (CURLY_QUOTE.test(bad)) {
-        return (`fpdf2 core fonts are latin-1 only and the curly quote ${bad} is outside it — rewrite ` +
-            'the strings with straight quotes \' and " and rerun.');
-    }
-    return (`fpdf2 core fonts are latin-1 only and "${bad}" cannot be rendered — there are no Unicode ` +
-        'fonts in this runtime. Drop it or use a latin transliteration, and tell the user CJK and ' +
-        'emoji cannot appear in the PDF.');
-}
-export function fpdfUnicodeErrorHint(stderr) {
-    const encoding = /FPDFUnicodeEncodingException/.test(stderr) ||
-        (/UnicodeEncodeError: 'latin-1' codec/.test(stderr) && /fpdf/.test(stderr));
-    if (!encoding) {
-        return null;
-    }
-    return ('a character outside latin-1 reached an fpdf2 core font: replace it with ASCII ' +
-        '(bullets "- ", straight quotes, plain hyphens). CJK and emoji cannot be rendered — ' +
-        'tell the user instead of retrying with the same string.');
-}
-export function fpdfHorizontalSpaceHint(stderr) {
-    if (!/FPDFException: Not enough horizontal space/.test(stderr))
-        return null;
-    return ('an earlier cell/multi_cell left the cursor at the right margin: add ' +
-        'new_x="LMARGIN", new_y="NEXT" to every cell and multi_cell call so each ' +
-        'one starts on its own line.');
-}
-export function rgbColorImportHint(stderr) {
-    const name = /cannot import name '([^']+)' from 'pptx\.dml\.color'/.exec(stderr)?.[1];
-    if (!name || name === 'RGBColor') {
-        return null;
-    }
-    if (!/^rgb_?color$/i.test(name)) {
-        return null;
-    }
-    return (`pptx.dml.color exports RGBColor (all-caps RGB), not ${name}: ` +
-        'from pptx.dml.color import RGBColor');
-}
 export function checkPythonSource(command) {
     const source = unwrapPythonSource(command) ?? command;
     const literal = stringLiteralProgramHint(source);
@@ -483,26 +330,6 @@ export function checkPythonSource(command) {
     if (collapsed) {
         return collapsed;
     }
-    const mangled = mangledPptxApiHint(source);
-    if (mangled) {
-        return mangled;
-    }
-    const rgb = wrongRgbColorHint(source);
-    if (rgb) {
-        return rgb;
-    }
-    const fpdfKeyword = fpdfSizeKeywordHint(source);
-    if (fpdfKeyword) {
-        return fpdfKeyword;
-    }
-    const fpdfLatin1 = fpdfNonLatin1Hint(source);
-    if (fpdfLatin1) {
-        return fpdfLatin1;
-    }
-    const fpdfText = fpdfTextCallHint(source);
-    if (fpdfText) {
-        return fpdfText;
-    }
     const first = command.trimStart().split('\n')[0] ?? '';
     if (!SHELL_WRAPPER.test(first)) {
         return null;
@@ -511,7 +338,7 @@ export function checkPythonSource(command) {
         return null;
     }
     if (/^pip3?\b/.test(first) || /^(python3?|py)\s+-m\s+pip\b/.test(first)) {
-        return 'language "python" does not run shell commands: install packages via the packages argument (e.g. packages: ["python-pptx==1.0.2"]), not pip';
+        return 'language "python" does not run shell commands: install packages via the packages argument (e.g. packages: ["requests"]), not pip';
     }
     if (unwrapPythonSource(command)) {
         return null;
@@ -522,295 +349,8 @@ export function syntaxErrorHint(stderr, source) {
     if (!/SyntaxError/.test(stderr))
         return null;
     return (checkCollapsedCompound(source) ??
-        mangledPptxApiHint(source) ??
         'SyntaxError means the Python source is invalid as written — keep real newlines for for/if/with/def bodies; do not collapse a multi-line program into one semicolon-separated line.');
 }
-export function checkPptxBuildSource(source, outputs, packages) {
-    if (!outputs?.some((name) => name.toLowerCase().endsWith('.pptx'))) {
-        return null;
-    }
-    const pkgPptx = (packages ?? []).some((p) => {
-        const name = (p.split(/[=<>!~[]/)[0] ?? '').trim().toLowerCase();
-        return name === 'python-pptx' || name === 'pptx';
-    });
-    const srcPptx = /\bpptx\b|\bPresentation\s*\(/.test(source);
-    if (!pkgPptx && !srcPptx) {
-        return null;
-    }
-    const builds = /\bPresentation\s*\(/.test(source) && /\.save\s*\(/.test(source);
-    if (builds) {
-        return null;
-    }
-    return ('outputs declare a .pptx but command does not build one: pass multi-line python-pptx ' +
-        'source that constructs Presentation(...) and calls save("deck.pptx") matching an ' +
-        'outputs entry. Do not use python -c, os.listdir, pip, or a different package pin — ' +
-        'language="python" already runs bare source, and packages must stay ["python-pptx==1.0.2"].');
-}
-export function tripleQuoteAt(source, index) {
-    if (source.startsWith('"""', index))
-        return '"""';
-    return source.startsWith("'''", index) ? "'''" : null;
-}
-export function isEscaped(source, index) {
-    let slashes = 0;
-    while (index - slashes > 0 && source[index - slashes - 1] === '\\') {
-        slashes++;
-    }
-    return slashes % 2 === 1;
-}
-function splitCallArgs(source, open) {
-    const args = [];
-    let arg = '';
-    let depth = 0;
-    let quote = null;
-    for (let i = open; i < source.length; i++) {
-        const ch = source.charAt(i);
-        if (quote) {
-            arg += ch;
-            if (ch === quote && !isEscaped(source, i)) {
-                quote = null;
-            }
-            continue;
-        }
-        if (ch === '"' || ch === "'") {
-            quote = ch;
-            arg += ch;
-            continue;
-        }
-        if (ch === '(' || ch === '[' || ch === '{') {
-            depth++;
-            if (depth > 1) {
-                arg += ch;
-            }
-            continue;
-        }
-        if (ch === ')' || ch === ']' || ch === '}') {
-            depth--;
-            if (depth === 0) {
-                args.push(arg);
-                return args.map((a) => a.trim());
-            }
-            arg += ch;
-            continue;
-        }
-        if (ch === ',' && depth === 1) {
-            args.push(arg);
-            arg = '';
-            continue;
-        }
-        arg += ch;
-    }
-    return null;
-}
-const MIN_MEANT_EMU = 12700;
-function bareEmuMistake(arg) {
-    if (!/^[+-]?(?:\d+(?:\.\d*)?|\.\d+)$/.test(arg))
-        return false;
-    const value = Math.abs(Number(arg));
-    return value !== 0 && value < MIN_MEANT_EMU;
-}
-function stripCommentsAndStrings(source) {
-    let out = '';
-    let i = 0;
-    while (i < source.length) {
-        const ch = source.charAt(i);
-        const triple = tripleQuoteAt(source, i);
-        const quote = triple ?? (ch === '"' || ch === "'" ? ch : null);
-        if (quote) {
-            i += quote.length;
-            while (i < source.length && !(source.startsWith(quote, i) && !isEscaped(source, i))) {
-                i++;
-            }
-            out += '""';
-            i += quote.length;
-            continue;
-        }
-        if (ch === '#') {
-            while (i < source.length && source[i] !== '\n') {
-                i++;
-            }
-            continue;
-        }
-        out += ch;
-        i++;
-    }
-    return out;
-}
-const LENGTH_LIBRARY = {
-    pptx: {
-        module: 'pptx.util',
-        effect: 'the shape collapses to an invisible box in the top-left corner and the deck is delivered broken',
-        example: 'add_textbox(Inches(1), Inches(1), Inches(8), Inches(1.5))'
-    },
-    docx: {
-        module: 'docx.shared',
-        effect: 'the image collapses to nothing and the document is delivered broken',
-        example: 'add_picture("chart.png", width=Inches(5.5))'
-    }
-};
-function lengthLibrary(source, packages) {
-    const names = (packages ?? []).map((p) => (p.split(/[=<>!~[]/)[0] ?? '').trim().toLowerCase());
-    if (names.includes('python-pptx') || names.includes('pptx')) {
-        return 'pptx';
-    }
-    if (names.includes('python-docx') || names.includes('docx')) {
-        return 'docx';
-    }
-    if (/\bpptx\b|\bPresentation\s*\(/.test(source)) {
-        return 'pptx';
-    }
-    if (/\bdocx\b|\bDocument\s*\(/.test(source)) {
-        return 'docx';
-    }
-    return null;
-}
-const LENGTH_KEYWORDS = ['left', 'top', 'width', 'height'];
-const FIRST_LENGTH_ARG = { add_textbox: 0, add_picture: 1 };
-function callLibrary(method, before, fallback) {
-    if (method === 'add_textbox')
-        return 'pptx';
-    if (/\bshapes\s*\.\s*$/.test(before))
-        return 'pptx';
-    if (/\b(?:doc|document|run|paragraph|cell)\s*\.\s*$/.test(before)) {
-        return 'docx';
-    }
-    return fallback;
-}
-export function checkBareLengthArgs(source, packages) {
-    const clean = stripCommentsAndStrings(source);
-    const library = lengthLibrary(clean, packages);
-    if (!library) {
-        return null;
-    }
-    for (const method of ['add_textbox', 'add_picture']) {
-        const calls = new RegExp(`\\b${method}\\s*\\(`, 'g');
-        let call;
-        while ((call = calls.exec(clean)) !== null) {
-            const args = splitCallArgs(clean, calls.lastIndex - 1);
-            if (!args) {
-                continue;
-            }
-            const offender = args.find((arg, index) => {
-                const eq = arg.indexOf('=');
-                const name = eq === -1 ? '' : arg.slice(0, eq).trim();
-                if (eq !== -1 && /^[A-Za-z_]\w*$/.test(name)) {
-                    return LENGTH_KEYWORDS.includes(name) && bareEmuMistake(arg.slice(eq + 1).trim());
-                }
-                return index >= FIRST_LENGTH_ARG[method] && bareEmuMistake(arg);
-            });
-            if (offender !== undefined) {
-                const before = clean.slice(Math.max(0, call.index - 40), call.index);
-                return bareLengthError(`${method}(...)`, offender, callLibrary(method, before, library));
-            }
-        }
-    }
-    if (library === 'pptx') {
-        const assignments = /\bslide_(?:width|height)\s*=\s*([^\n]+)/g;
-        let assigned;
-        while ((assigned = assignments.exec(clean)) !== null) {
-            const value = (assigned[1] ?? '').trim();
-            if (bareEmuMistake(value)) {
-                return bareLengthError('slide_width / slide_height', value, library);
-            }
-        }
-    }
-    return null;
-}
-function bareLengthError(where, value, library) {
-    const fix = LENGTH_LIBRARY[library];
-    return (`${where} was given the bare number ${value} where a typed length is required: ` +
-        `python-${library} reads a bare number as EMU (914400 to the inch), so ${fix.effect}. ` +
-        'Nothing raises, so this never reaches you as an error. Wrap every position and size ' +
-        `with Inches(...), Pt(...) or Emu(...) from ${fix.module}, e.g. ${fix.example}. ` +
-        'Do not rescale the raw numbers.');
-}
-export function checkPdfBuildSource(source, outputs, packages) {
-    if (!outputs?.some((name) => name.toLowerCase().endsWith('.pdf')))
-        return null;
-    const pkgPdf = (packages ?? []).some((p) => {
-        const name = (p.split(/[=<>!~[]/)[0] ?? '').trim().toLowerCase();
-        return name === 'fpdf2' || name === 'pypdf';
-    });
-    const srcPdf = /\b(fpdf|FPDF|pypdf|PdfWriter|PdfReader)\b/.test(source);
-    if (!pkgPdf && !srcPdf) {
-        return null;
-    }
-    const builds = /\.output\s*\(|\.write\s*\(/.test(source);
-    if (builds) {
-        return null;
-    }
-    return ('outputs declare a .pdf but command never writes one: pass multi-line Python that ' +
-        'builds the document and saves it under the declared name — fpdf2 ends with ' +
-        'pdf.output("name.pdf"), pypdf with writer.write("name.pdf"). Do not send diagnostics ' +
-        'like os.listdir or print-only scripts while outputs still lists the file.');
-}
-export function stripCommentsAndDocstrings(source) {
-    let out = '';
-    let i = 0;
-    while (i < source.length) {
-        const ch = source.charAt(i);
-        const triple = tripleQuoteAt(source, i);
-        if (triple) {
-            i += triple.length;
-            while (i < source.length && !(source.startsWith(triple, i) && !isEscaped(source, i))) {
-                i++;
-            }
-            out += '""';
-            i += triple.length;
-            continue;
-        }
-        if (ch === '"' || ch === "'") {
-            const start = i;
-            i++;
-            while (i < source.length && !(source[i] === ch && !isEscaped(source, i))) {
-                i++;
-            }
-            i++;
-            out += source.slice(start, Math.min(i, source.length));
-            continue;
-        }
-        if (ch === '#') {
-            while (i < source.length && source[i] !== '\n') {
-                i++;
-            }
-            continue;
-        }
-        out += ch;
-        i++;
-    }
-    return out;
-}
-export const PDF_READ = /(?:PdfReader|append)\s*\(\s*['"]([\w.-]+\.pdf)['"]|clone_from\s*=\s*['"]([\w.-]+\.pdf)['"]|open\s*\(\s*['"]([\w.-]+\.pdf)['"]\s*(?:,\s*(?:mode\s*=\s*)?['"](?<mode>[rwaxb+]+)['"])?/g;
-export const OFFICE_READ = /(?:load_workbook|Document|Presentation)\s*\(\s*['"]([\w.-]+\.(?:xlsx|xlsm|docx|pptx))['"]/g;
-export function checkUnstagedRead(source, inputs) {
-    const staged = new Set((inputs ?? []).map((i) => i.path));
-    const clean = stripCommentsAndDocstrings(source);
-    for (const pattern of [PDF_READ, OFFICE_READ]) {
-        const reads = new RegExp(pattern.source, 'g');
-        let match;
-        while ((match = reads.exec(clean)) !== null) {
-            const name = match.slice(1).find((group) => group !== undefined) ?? '';
-            const mode = match.groups?.mode;
-            if (mode !== undefined && /[wax]/.test(mode)) {
-                continue;
-            }
-            if (staged.has(name)) {
-                continue;
-            }
-            const before = clean.slice(0, match.index);
-            if (before.includes(`"${name}"`) || before.includes(`'${name}'`)) {
-                continue;
-            }
-            return (`${name} does not exist in the working directory — it starts empty on every run, so ` +
-                'files from earlier calls are gone. Stage the attachment via inputs: ' +
-                `[{ "attachmentId": "<id from the tool result that produced it, or from the ` +
-                `[Attached file …] line if the user uploaded it>", "path": "${name}" }] ` +
-                'and open it by that bare name; if no id for it appears anywhere in the chat, ' +
-                'ask the user to attach the file.');
-        }
-    }
-    return null;
-}
 const ATTACHMENT_ID = /^[0-9a-f]{32}$/i;
 export function missingFileHint(stderr) {
     const name = /FileNotFoundError: \[Errno \d+\] No such file or directory: '([^']+)'/.exec(stderr)?.[1];
@@ -935,6 +475,39 @@ function prepare(args, cwd) {
     }
     return { packages, inputs, outputs, workdir };
 }
+const OFFICE_MODULES = {
+    pptx: { tool: 'pptx', skill: 'presentations' },
+    docx: { tool: 'docx', skill: 'word' },
+    openpyxl: { tool: 'xlsx', skill: 'excel' }
+};
+const OFFICE_PACKAGES = {
+    'python-pptx': 'pptx',
+    'python-docx': 'docx',
+    openpyxl: 'openpyxl'
+};
+const OFFICE_IMPORT = /^\s*(?:from\s+(pptx|docx|openpyxl)\b|import\s+(?:[\w.]+\s*(?:as\s+\w+\s*)?,\s*)*(pptx|docx|openpyxl)\b)/m;
+function officeAnswer(module, toolNames) {
+    const office = OFFICE_MODULES[module];
+    if (!office) {
+        return null;
+    }
+    const call = toolNames?.includes(office.tool)
+        ? `call the ${office.tool} tool`
+        : `load the "${office.skill}" skill and call the ${office.tool} tool it grants`;
+    return `module "${module}" is not installed: Office files are read and edited without Python here. Do not retry in Python; ${call} with the same content.`;
+}
+export function officeModuleHint(stderr, toolNames) {
+    const module = /ModuleNotFoundError: No module named '([^'.]+)/.exec(stderr)?.[1];
+    return module ? officeAnswer(module, toolNames) : null;
+}
+export function officeImportError(source, { packages, toolNames }) {
+    const imported = OFFICE_IMPORT.exec(source);
+    const declared = (packages ?? [])
+        .map((spec) => OFFICE_PACKAGES[spec.split(/[=<>!~[ ]/)[0]?.toLowerCase() ?? ''])
+        .find(Boolean);
+    const module = imported?.[1] ?? imported?.[2] ?? declared;
+    return module ? officeAnswer(module, toolNames) : null;
+}
 export function missingModuleHint(stderr, { packages, sandbox = false, network = false } = {}) {
     const module = /ModuleNotFoundError: No module named '([^']+)'/.exec(stderr)?.[1];
     if (!module) {
@@ -968,53 +541,6 @@ export function missingModuleHint(stderr, { packages, sandbox = false, network =
     }
     return `module "${root}" is not installed: add "${spec}" to packages (packages: ["${spec}"]) and run the same source again. There is no pip.`;
 }
-export function nonePlaceholderHint(stderr) {
-    if (!/AttributeError: 'NoneType' object has no attribute '(text|text_frame)'/.test(stderr)) {
-        return null;
-    }
-    return 'a placeholder the layout does not have reads as None: the blank layout has no title and no body, so shapes.title is None there. Use a layout that owns the placeholder (title+content is commonly index 1), or keep the blank layout and add every text box explicitly with shapes.add_textbox(left, top, width, height).';
-}
-export function slideShapesHint(stderr) {
-    const attribute = /AttributeError: 'Slide' object has no attribute '(add_picture|add_textbox|addpicture|addtextbox)'/.exec(stderr)?.[1];
-    if (!attribute) {
-        return null;
-    }
-    const method = attribute.startsWith('addp') ? 'add_picture' : 'add_textbox';
-    return `a Slide has no add_* methods of its own — they live on its shapes collection: slide.shapes.${method}(...), not slide.${method}(...).`;
-}
-export function shapeFontHint(stderr) {
-    const attribute = /AttributeError: '\w*(?:Placeholder|Shape|Slide)\w*' object has no attribute '(font|alignment|text_align|word_wrap|paragraphs|add_paragraph)'/.exec(stderr)?.[1];
-    if (!attribute) {
-        return null;
-    }
-    return `"${attribute}" belongs to the text frame or a paragraph inside it, not to the shape: go through shape.text_frame first (shape.text_frame.word_wrap, shape.text_frame.paragraphs[0].font.size, shape.text_frame.paragraphs[0].alignment). shape.text is the one shortcut that reads through to the frame.`;
-}
-export function shapesBackgroundHint(stderr) {
-    if (!/AttributeError: '\w*(?:Shapes|Shape|Placeholder)\w*' object has no attribute 'background'/.test(stderr)) {
-        return null;
-    }
-    return "'background' lives on the slide, not on its shapes: fill = slide.background.fill; fill.solid(); fill.fore_color.rgb = RGBColor(...). A shape is tinted through its own shape.fill the same way.";
-}
-export function paragraphFillHint(stderr) {
-    if (!/AttributeError: '(?:_Paragraph|_Run|TextFrame)' object has no attribute 'fill'/.test(stderr)) {
-        return null;
-    }
-    return "text has no 'fill': color it through the font — paragraph.font.color.rgb = RGBColor(...). A fill belongs to a shape (shape.fill) or to slide.background.fill, never to a paragraph or text frame.";
-}
-export function readOnlyPropertyHint(stderr) {
-    if (!/AttributeError: can't set attribute/.test(stderr))
-        return null;
-    if (!/pptx[/\\]util\.py/.test(stderr))
-        return null;
-    return 'a python-pptx container is read-only: font, text_frame, paragraphs, runs and shapes cannot be assigned, only reached through. Set the leaf property instead — paragraph.font.size = Pt(20) and paragraph.font.bold = True, not paragraph.font = …; and write text with text_frame.text or paragraph.text, never text_frame = …';
-}
-export function paragraphIndexHint(stderr, outputs) {
-    if (!/IndexError: tuple index out of range/.test(stderr))
-        return null;
-    if (!outputs.some((name) => name.toLowerCase().endsWith('.pptx')))
-        return null;
-    return 'a new text frame holds exactly one paragraph, so text_frame.paragraphs[1] does not exist yet: write the first through paragraphs[0] (or text_frame.text) and create each one after it with text_frame.add_paragraph(), which returns the new paragraph. A paragraph has .text, .font and .alignment — it has no .paragraphs and no .add_paragraph of its own.';
-}
 export function shellNameHint(stderr) {
     const name = /NameError: name '(python3?|py|pip3?)' is not defined/.exec(stderr)?.[1];
     if (!name) {
@@ -1060,110 +586,6 @@ export function packageFetchFailedHint(stderr) {
 function fail(stderr) {
     return { stdout: '', stderr, exitCode: 1 };
 }
-export const IMAGE_INPUT = /\.(png|jpe?g|gif|webp|bmp|heic|heif|avif|tiff?)$/i;
-const PPT_MEDIA = b4a.from('ppt/media/');
-const deckAttachedThisTurn = new WeakMap();
-export function pptxHasEmbeddedMedia(bytes) {
-    outer: for (let i = 0; i <= bytes.length - PPT_MEDIA.length; i++) {
-        for (let j = 0; j < PPT_MEDIA.length; j++) {
-            if (bytes[i + j] !== PPT_MEDIA[j])
-                continue outer;
-        }
-        return true;
-    }
-    return false;
-}
-export function softFailedDeckHint(stdout, exitCode, source) {
-    if (exitCode !== 0)
-        return null;
-    if (!/\bPresentation\s*\(/.test(source))
-        return null;
-    if (!/Image loading error|expected if file not found|cannot import name ['"]RgbColor['"]/i.test(stdout)) {
-        return null;
-    }
-    return ('the deck soft-failed (caught an image/import error and continued): remove try/except ' +
-        'around imports and add_picture, fix the error, and rerun once — a partial deck must not be saved');
-}
-export function stagedImagesMissingFromDeck(result, { inputs, outputs, source }) {
-    if (result.exitCode !== 0)
-        return null;
-    if (!/\bPresentation\s*\(/.test(source))
-        return null;
-    const images = (inputs ?? []).filter((i) => IMAGE_INPUT.test(i.path));
-    const [first] = images;
-    if (!first) {
-        return null;
-    }
-    for (const name of outputs ?? []) {
-        if (!name.toLowerCase().endsWith('.pptx'))
-            continue;
-        const bytes = result.files?.get(name);
-        if (!bytes) {
-            continue;
-        }
-        if (!pptxHasEmbeddedMedia(bytes)) {
-            return (`staged image(s) ${images.map((i) => i.path).join(', ')} were not embedded: ` +
-                `call slide.shapes.add_picture("${first.path}", …) without try/except, then save`);
-        }
-    }
-    return null;
-}
-export function checkStagedImagesUsed(source, inputs, outputs) {
-    if (!inputs?.length)
-        return null;
-    if (!outputs?.some((n) => n.toLowerCase().endsWith('.pptx')))
-        return null;
-    if (!/\bPresentation\s*\(/.test(source)) {
-        return null;
-    }
-    const images = inputs.filter((i) => IMAGE_INPUT.test(i.path));
-    if (!images.length) {
-        return null;
-    }
-    if (!/\.add_picture\s*\(/.test(source)) {
-        return ('inputs stage image(s) but command never calls slide.shapes.add_picture(...): ' +
-            'embed each staged path or drop it from inputs');
-    }
-    const missing = images.filter((i) => !source.includes(`"${i.path}"`) && !source.includes(`'${i.path}'`));
-    if (missing.length) {
-        return (`inputs stage ${missing.map((m) => m.path).join(', ')} but command never opens those files — ` +
-            `pass the bare path to slide.shapes.add_picture("…")`);
-    }
-    if (/\btry\s*:(?:(?!\bexcept\b|\bfinally\b)[\s\S]){0,1200}?\badd_picture\b/.test(source)) {
-        return 'do not wrap add_picture in try/except: if the image fails the run must fail so you can fix it';
-    }
-    return null;
-}
-export function checkTurnImagesStaged(source, { inputs, outputs, turnImages }) {
-    if (!turnImages?.length)
-        return null;
-    if (!outputs?.some((n) => n.toLowerCase().endsWith('.pptx')))
-        return null;
-    if (!/\bPresentation\s*\(/.test(source)) {
-        return null;
-    }
-    if ((inputs ?? []).some((i) => i.attachmentId === undefined)) {
-        return null;
-    }
-    return (`the user's message has ${turnImages.length} uploaded image(s) but inputs stage none of them — ` +
-        'add one id-less { "path": "…" } entry per image and embed each with slide.shapes.add_picture(...)');
-}
-export function deckDeliveredStop(attachments, exitCode, source) {
-    if (exitCode !== 0 || !attachments?.length)
-        return null;
-    if (source !== undefined && !/\bpptx\b|\bPresentation\s*\(/.test(source)) {
-        return null;
-    }
-    const decks = attachments.filter((a) => a.fileName.toLowerCase().endsWith('.pptx'));
-    if (decks.length === 0) {
-        return null;
-    }
-    const names = decks.map((a) => a.fileName).join(', ');
-    const verb = decks.length === 1 ? 'is' : 'are';
-    return (`${names} ${verb} attached to the chat. STOP: do not call exec again for this deck. ` +
-        'A second exec that rebuilds the same .pptx will be rejected. ' +
-        'Reply to the user with one line naming the file and slide count from stdout — nothing else.');
-}
 async function stage(inputs, fetchAttachment, opts = {}) {
     if (!inputs?.length)
         return undefined;
@@ -1181,37 +603,10 @@ async function collect(result, { declared, attach, runBudget, source }) {
     if (!declared?.length || !attach) {
         return base;
     }
-    const pptxBuild = source !== undefined && /\bpptx\b|\bPresentation\s*\(/.test(source);
-    const pptxNames = declared.filter((n) => n.toLowerCase().endsWith('.pptx') && result.files?.has(n));
-    const wouldAttachPptx = result.exitCode === 0 && pptxBuild && pptxNames.length > 0;
-    if (wouldAttachPptx) {
-        const attached = deckAttachedThisTurn.get(attach);
-        if (attached) {
-            const collision = pptxNames.find((n) => attached.has(n.toLowerCase()));
-            if (collision) {
-                return fail(`"${collision}" was already attached this turn — do not rebuild it; reply to the user with the file name and slide count`);
-            }
-        }
-    }
     const { attachments, missingOutputs, rejectedOutputs } = await attachOutputs(attach, (result.exitCode === 0 && result.files) || new Map(), { declared, runBudget });
     const rejected = [...(result.rejected ?? []), ...rejectedOutputs];
-    const stop = deckDeliveredStop(attachments, result.exitCode, source);
-    if (stop && result.exitCode === 0) {
-        let attached = deckAttachedThisTurn.get(attach);
-        if (!attached) {
-            attached = new Set();
-            deckAttachedThisTurn.set(attach, attached);
-        }
-        for (const n of pptxNames) {
-            attached.add(n.toLowerCase());
-        }
-    }
-    const stdout = stop && result.exitCode === 0
-        ? `${base.stdout}${base.stdout && !base.stdout.endsWith('\n') ? '\n' : ''}${stop}`
-        : base.stdout;
     return {
         ...base,
-        stdout,
         attachments: attachments.length > 0 ? attachments : undefined,
         missingOutputs: missingOutputs.length > 0 ? missingOutputs : undefined,
         rejectedOutputs: rejected.length > 0 ? rejected : undefined
@@ -1242,39 +637,11 @@ export function createPythonAdapter(disabled, loadPyodide, network = false) {
                     if (sourceError) {
                         return fail(sourceError);
                     }
-                    const source = normalizeFpdfSource(unwrapPythonSource(command) ?? command);
+                    const source = unwrapPythonSource(command) ?? command;
                     const prepared = prepare(args, options.cwd);
                     if ('error' in prepared) {
                         return fail(prepared.error);
                     }
-                    const pptxError = checkPptxBuildSource(source, prepared.outputs, prepared.packages);
-                    if (pptxError) {
-                        return fail(pptxError);
-                    }
-                    const pdfError = checkPdfBuildSource(source, prepared.outputs, prepared.packages);
-                    if (pdfError) {
-                        return fail(pdfError);
-                    }
-                    const bareLength = checkBareLengthArgs(source, prepared.packages);
-                    if (bareLength) {
-                        return fail(bareLength);
-                    }
-                    const unstagedRead = checkUnstagedRead(source, prepared.inputs);
-                    if (unstagedRead) {
-                        return fail(unstagedRead);
-                    }
-                    const stagedUse = checkStagedImagesUsed(source, prepared.inputs, prepared.outputs);
-                    if (stagedUse) {
-                        return fail(stagedUse);
-                    }
-                    const turnStage = checkTurnImagesStaged(source, {
-                        inputs: prepared.inputs,
-                        outputs: prepared.outputs,
-                        turnImages
-                    });
-                    if (turnStage) {
-                        return fail(turnStage);
-                    }
                     if (prepared.outputs?.length && !attach) {
                         return fail('this device cannot persist produced files');
                     }
@@ -1306,40 +673,16 @@ export function createPythonAdapter(disabled, loadPyodide, network = false) {
                         hosts: gated ? namedHosts(source) : undefined,
                         askConnect: gated ? confirmConnect : undefined
                     });
-                    const soft = softFailedDeckHint(result.stdout, result.exitCode, source);
-                    if (soft) {
-                        return fail(soft);
-                    }
-                    const missingPic = stagedImagesMissingFromDeck(result, {
-                        inputs: prepared.inputs,
-                        outputs: prepared.outputs,
-                        source
-                    });
-                    if (missingPic) {
-                        return fail(missingPic);
-                    }
                     const hint = packageFetchFailedHint(result.stderr) ??
                         nativeWheelHint(result.stderr) ??
+                        officeModuleHint(result.stderr, toolNames) ??
                         missingModuleHint(result.stderr, { packages, sandbox, network: reach }) ??
                         pipFromSourceHint(result.stderr, source, { packages, sandbox }) ??
                         noProcessHint(result.stderr, sandbox) ??
                         noNetworkHint(result.stderr, toolNames, reach) ??
                         missingFileHint(result.stderr) ??
-                        rgbColorImportHint(result.stderr) ??
-                        nonePlaceholderHint(result.stderr) ??
-                        slideShapesHint(result.stderr) ??
-                        shapeFontHint(result.stderr) ??
-                        shapesBackgroundHint(result.stderr) ??
-                        paragraphFillHint(result.stderr) ??
-                        readOnlyPropertyHint(result.stderr) ??
-                        paragraphIndexHint(result.stderr, prepared.outputs ?? []) ??
-                        fpdfTextErrorHint(result.stderr) ??
-                        fpdfKeywordErrorHint(result.stderr) ??
-                        fpdfHorizontalSpaceHint(result.stderr) ??
-                        fpdfUnicodeErrorHint(result.stderr) ??
                         shellNameHint(result.stderr) ??
-                        syntaxErrorHint(result.stderr, source) ??
-                        mangledPptxApiHint(source);
+                        syntaxErrorHint(result.stderr, source);
                     const hinted = hint ? { ...result, stderr: `${result.stderr}\n${hint}` } : result;
                     const installed = install && result.exitCode === 0
                         ? {