shortcutxl 0.3.89 → 0.3.90

package/BINARY-INVENTORY.json

@@ -1,14 +1,14 @@
 {
   "schemaVersion": 1,
-  "generatedAt": "2026-09-02T01:14:28.575Z",
+  "generatedAt": "2026-09-02T05:35:16.912Z",
   "package": "shortcutxl",
   "build": {
     "builder": "github-actions",
     "repository": "lyfegame/shortcut",
-    "sourceRef": "refs/heads/codex/cli-daily-credit-usage",
-    "sourceCommit": "79a4adfc21dce36c575744cbbf07df842928b610",
-    "workflowRef": "lyfegame/shortcut/.github/workflows/release-shortcut-cli-npm-package.yml@refs/heads/codex/cli-daily-credit-usage",
-    "runId": "33578448535",
+    "sourceRef": "refs/heads/codex/excel-preexecution-approval",
+    "sourceCommit": "2520273120cc4b4024d877790eacf6f41fefdd7c",
+    "workflowRef": "lyfegame/shortcut/.github/workflows/release-shortcut-cli-npm-package.yml@refs/heads/codex/excel-preexecution-approval",
+    "runId": "33595155271",
     "runnerOs": "Windows",
     "runnerImage": "win25-vs2026",
     "runnerImageVersion": "20260824.214.3"
@@ -550,7 +550,7 @@
     },
     {
       "path": "xll/python/Scripts/httpx.exe",
-      "sha256": "56ecd26f514b1582761c3403f4938d112adc5f033a63d4bc945181079ca82ead",
+      "sha256": "a7f8d80085b1d17fdf993cd7246d55f05726e3eb624859a8ba3b69b76368bd4c",
       "source": "httpx console launcher installed into embedded Python",
       "version": "see packaged httpx distribution",
       "builtBy": "third-party",
@@ -558,7 +558,7 @@
     },
     {
       "path": "xll/python/Scripts/idna.exe",
-      "sha256": "ffda0108e10dc93a09f57419aac0a52d85871bb787308bf946ac851cb11f703a",
+      "sha256": "75e2f1c65f561084f603823794d6ad5877ef0a5b1ef0b1fffff096603479642a",
       "source": "Python package console launcher installed into embedded Python",
       "version": "see owning Python package metadata in site-packages",
       "builtBy": "third-party",
@@ -566,7 +566,7 @@
     },
     {
       "path": "xll/python/Scripts/pip.exe",
-      "sha256": "ce0f86f7cf5cc2882a4821943d3af21f7c41218ff43e7a1f62b18f185aefde97",
+      "sha256": "f68ea8e144f74e0f9748750a5fc21b73e11d63486c905730aa474e28f3bab8e7",
       "source": "pip console launcher installed into embedded Python",
       "version": "see packaged pip distribution",
       "builtBy": "third-party",
@@ -574,7 +574,7 @@
     },
     {
       "path": "xll/python/Scripts/pip3.13.exe",
-      "sha256": "ce0f86f7cf5cc2882a4821943d3af21f7c41218ff43e7a1f62b18f185aefde97",
+      "sha256": "f68ea8e144f74e0f9748750a5fc21b73e11d63486c905730aa474e28f3bab8e7",
       "source": "pip console launcher installed into embedded Python",
       "version": "see packaged pip distribution",
       "builtBy": "third-party",
@@ -582,7 +582,7 @@
     },
     {
       "path": "xll/python/Scripts/pip3.exe",
-      "sha256": "ce0f86f7cf5cc2882a4821943d3af21f7c41218ff43e7a1f62b18f185aefde97",
+      "sha256": "f68ea8e144f74e0f9748750a5fc21b73e11d63486c905730aa474e28f3bab8e7",
       "source": "pip console launcher installed into embedded Python",
       "version": "see packaged pip distribution",
       "builtBy": "third-party",
@@ -590,7 +590,7 @@
     },
     {
       "path": "xll/python/Scripts/pywin32_postinstall.exe",
-      "sha256": "ff04c09d3e15a07d9a4006748c92e511f7e5c5abb2c78b68d7233e8f3ff19630",
+      "sha256": "3d5142d3d301b5b22ec9f9ba2d29c9f8a4a73bad069389a73dafbd61e2b37696",
       "source": "Python package console launcher installed into embedded Python",
       "version": "see owning Python package metadata in site-packages",
       "builtBy": "third-party",
@@ -598,7 +598,7 @@
     },
     {
       "path": "xll/python/Scripts/pywin32_testall.exe",
-      "sha256": "c0641e3d6816baddcabb155f66ffb5de7c378b8257b95ad16f0a45e26184b879",
+      "sha256": "494be3de764340437d1fb12aaaefaf8f47c5a482f14834da3a17b080c71d6e16",
       "source": "Python package console launcher installed into embedded Python",
       "version": "see owning Python package metadata in site-packages",
       "builtBy": "third-party",

package/CHANGELOG.md

@@ -1,8 +1,9 @@
 # Changelog
 
-## [0.3.89]
+## [0.3.90]
 
 - **Skill Forge** - A recursive self-improvement loop for skills. Create or improve personal skills by giving Skill Forge tasks and a grading bar to train, evaluate, and refine against.
+- **Safer, more reliable Excel changes** - Shortcut now asks for approval before making workbook changes and allows longer operations to finish without timing out.
 - **Credit usage at a glance** - See today's personal credit usage for non-teams users.
 
 ## [0.3.88]

package/dist/app/local/tools/execute-code/approval-payload.d.ts

@@ -6,6 +6,10 @@ export interface ExcelDiffApprovalPayload {
     kind: 'diff';
     description: string;
 }
+export interface ExcelWriteApprovalPayload {
+    kind: 'write';
+    description: string;
+}
 export interface OfficeDangerousApprovalPayload {
     kind: 'dangerous';
     description: string;
@@ -13,5 +17,5 @@ export interface OfficeDangerousApprovalPayload {
     summary: string;
     operations: OfficeApprovalOperation[];
 }
-export type OfficeApprovalPayload = ExcelDiffApprovalPayload | OfficeDangerousApprovalPayload;
+export type OfficeApprovalPayload = ExcelDiffApprovalPayload | ExcelWriteApprovalPayload | OfficeDangerousApprovalPayload;
 //# sourceMappingURL=approval-payload.d.ts.map

package/dist/app/local/tools/execute-code/executor.d.ts

@@ -2,9 +2,9 @@
  * COM execute_code executor — local XLL-backed execution lifecycle.
  *
  * Owns the COM-specific execution flow:
- * 1. Send code to the local XLL via xllFetch
- * 2. Parse and validate the response
- * 3. Run the approval / revert / cleanup flow
+ * 1. Classify code and request write approval before execution
+ * 2. Send approved code to the local XLL via xllFetch
+ * 3. Parse and validate the response
  *
  * Used by:
  * - `createExcelExecTool()` (COM ToolDefinition wrapper)
@@ -23,7 +23,7 @@ export interface ComExecResult {
     cfTxId?: string;
     editMode?: boolean;
 }
-export type ApprovalOutcome = 'accepted' | 'approval-unavailable' | 'failed' | 'reverted' | 'revert-failed' | 'rejected';
+export type ApprovalOutcome = 'accepted' | 'approval-unavailable' | 'failed' | 'rejected';
 export interface LocalExecResult {
     result: ComExecResult;
     outcome: ApprovalOutcome;
@@ -33,19 +33,9 @@ export interface ExcelExecApprovalObservability {
     onRequested?: (request: ApprovalRequest<OfficeApprovalPayload>) => void;
     onAccepted?: (request: ApprovalRequest<OfficeApprovalPayload>, choice: ApprovalChoice) => void;
     onRejected?: (request: ApprovalRequest<OfficeApprovalPayload>) => void;
-    onReverted?: (request: ApprovalRequest<OfficeApprovalPayload>) => void;
-    onRevertFailed?: (request: ApprovalRequest<OfficeApprovalPayload>) => void;
-}
-interface ApprovalResolution {
-    outcome: ApprovalOutcome;
-    approvalSummaries?: string[];
 }
 /** Validate that a parsed JSON value has the COM execute result shape. */
 export declare function isComExecResult(v: unknown): v is ComExecResult;
-/**
- * Resolve approval for cell changes: show UI if needed, then revert or cleanup.
- */
-export declare function resolveApproval(result: ComExecResult, approval: ApprovalGate<OfficeApprovalPayload> | undefined, postControl: (body: object) => Promise<Response>, ctx: ExtensionContext, description?: string, observability?: ExcelExecApprovalObservability, priorApprovalSummaries?: string[]): Promise<ApprovalResolution>;
 /**
  * Execute code against the local XLL and run approval.
  *
@@ -60,5 +50,4 @@ export declare function executeComLocally(excelTarget: ExcelHostEndpoint, code:
     observability?: ExcelExecApprovalObservability;
     editModeTiming?: Partial<EditModeRetryTiming>;
 }): Promise<LocalExecResult>;
-export {};
 //# sourceMappingURL=executor.d.ts.map

package/dist/app/local/tools/execute-code/executor.js

@@ -2,14 +2,13 @@
  * COM execute_code executor — local XLL-backed execution lifecycle.
  *
  * Owns the COM-specific execution flow:
- * 1. Send code to the local XLL via xllFetch
- * 2. Parse and validate the response
- * 3. Run the approval / revert / cleanup flow
+ * 1. Classify code and request write approval before execution
+ * 2. Send approved code to the local XLL via xllFetch
+ * 3. Parse and validate the response
  *
  * Used by:
  * - `createExcelExecTool()` (COM ToolDefinition wrapper)
  */
-import { log } from '../../../../shared/logging/agent-log.js';
 import { INTERACTIVE_APPROVAL_UNAVAILABLE_MESSAGE } from '../../../approvals/approval-errors.js';
 import { APPROVAL } from '../../../approvals/types.js';
 import { getApprovalSurface, resolveApprovalExecutionDecision } from '../../../permissions/approval-surface.js';
@@ -31,13 +30,6 @@ export function isComExecResult(v) {
         (r.error === undefined || typeof r.error === 'string') &&
         (r.editMode === undefined || typeof r.editMode === 'boolean'));
 }
-// ---------------------------------------------------------------------------
-// Control constants
-// ---------------------------------------------------------------------------
-const CONTROL = {
-    REVERT: 'revert',
-    CLEANUP: 'cleanup'
-};
 function describeExcelApprovalChoice(choice) {
     switch (choice) {
         case APPROVAL.ACCEPT:
@@ -97,95 +89,41 @@ async function requestAlwaysConfirmApproval(approval, ctx, description, dangerou
     return { choice, unavailable: false };
 }
 // ---------------------------------------------------------------------------
-// Approval resolution
+// Pre-execution write approval
 // ---------------------------------------------------------------------------
-/**
- * Resolve approval for cell changes: show UI if needed, then revert or cleanup.
- */
-export async function resolveApproval(result, approval, postControl, ctx, description, observability, priorApprovalSummaries) {
-    const { diff, cfTxId } = result;
-    let approvalSummaries = priorApprovalSummaries;
-    if (diff) {
-        const request = {
-            kind: 'execute_code',
-            title: 'Review Excel changes',
-            choices: [APPROVAL.ACCEPT, APPROVAL.ACCEPT_ALL, APPROVAL.REJECT],
-            payload: {
-                kind: 'diff',
-                description: description?.trim() || 'Updating Excel'
-            }
-        };
-        const decision = resolveApprovalExecutionDecision({
-            surface: getApprovalSurface(ctx),
-            gateAvailable: approval !== undefined,
-            isSafe: false,
-            isAutoApproved: approval?.isAutoApproved() ?? false,
-            alwaysRequireExplicitApproval: false,
-            allowHeadlessApproval: ctx.permissionPolicy?.office.allowHeadlessApproval ?? false
-        });
-        if (decision === 'prompt') {
-            if (!approval) {
-                observability?.onRejected?.(request);
-                try {
-                    await postControl({ control: CONTROL.REVERT, cfTxId });
-                    observability?.onReverted?.(request);
-                    return { outcome: 'reverted' };
-                }
-                catch {
-                    if (cfTxId != null) {
-                        await postControl({ control: CONTROL.CLEANUP, cfTxId }).catch(() => { });
-                    }
-                    observability?.onRevertFailed?.(request);
-                    return { outcome: 'revert-failed' };
-                }
-            }
-            observability?.onRequested?.(request);
-            const choice = await approval.requestApproval(request, ctx);
-            if (choice === APPROVAL.REJECT || choice === undefined) {
-                observability?.onRejected?.(request);
-                try {
-                    await postControl({ control: CONTROL.REVERT, cfTxId });
-                    observability?.onReverted?.(request);
-                    return { outcome: 'rejected' };
-                }
-                catch {
-                    if (cfTxId != null) {
-                        await postControl({ control: CONTROL.CLEANUP, cfTxId }).catch(() => { });
-                    }
-                    observability?.onRevertFailed?.(request);
-                    return { outcome: 'revert-failed' };
-                }
-            }
-            observability?.onAccepted?.(request, choice);
-            if (choice === APPROVAL.ACCEPT_ALL) {
-                approval.onAcceptAll();
-            }
-            approvalSummaries = appendApprovalSummary(approvalSummaries, choice);
-        }
-        else if (decision === 'deny') {
-            observability?.onRejected?.(request);
-            try {
-                await postControl({ control: CONTROL.REVERT, cfTxId });
-                observability?.onReverted?.(request);
-                return { outcome: 'reverted' };
-            }
-            catch {
-                if (cfTxId != null) {
-                    await postControl({ control: CONTROL.CLEANUP, cfTxId }).catch(() => { });
-                }
-                observability?.onRevertFailed?.(request);
-                return { outcome: 'revert-failed' };
-            }
+async function requestWriteApproval(approval, ctx, description, observability) {
+    const request = {
+        kind: 'execute_code',
+        title: 'Review Excel changes',
+        choices: [APPROVAL.ACCEPT, APPROVAL.ACCEPT_ALL, APPROVAL.REJECT],
+        payload: {
+            kind: 'write',
+            description: description?.trim() || 'Updating Excel'
         }
+    };
+    const decision = resolveApprovalExecutionDecision({
+        surface: getApprovalSurface(ctx),
+        gateAvailable: approval !== undefined,
+        isSafe: false,
+        isAutoApproved: approval?.isAutoApproved() ?? false,
+        alwaysRequireExplicitApproval: false,
+        allowHeadlessApproval: ctx?.permissionPolicy?.office.allowHeadlessApproval ?? false
+    });
+    if (decision === 'allow') {
+        return { choice: APPROVAL.ACCEPT, unavailable: false, prompted: false };
     }
-    // Accepted (explicitly or auto) — clean up CF highlights.
-    if (cfTxId != null) {
-        await postControl({ control: CONTROL.CLEANUP, cfTxId }).catch((e) => {
-            const msg = e instanceof Error ? e.message : String(e);
-            log.error('excel-exec', `CF cleanup failed for tx ${cfTxId}: ${msg}`);
-        });
+    if (decision === 'deny' || !approval || !ctx) {
+        observability?.onRejected?.(request);
+        return { choice: APPROVAL.REJECT, unavailable: true, prompted: false };
+    }
+    observability?.onRequested?.(request);
+    const choice = await approval.requestApproval(request, ctx);
+    if (choice === APPROVAL.REJECT || choice === undefined) {
+        observability?.onRejected?.(request);
+        return { choice, unavailable: false, prompted: true };
     }
-    return { outcome: 'accepted', approvalSummaries };
+    observability?.onAccepted?.(request, choice);
+    return { choice, unavailable: false, prompted: true };
 }
 // ---------------------------------------------------------------------------
 // Core executor
@@ -200,7 +138,7 @@ export async function executeComLocally(excelTarget, code, options) {
     const resolvedExcelHttpUrl = excelTarget.url;
     const { approval, ctx, description, signal, observability, editModeTiming } = options ?? {};
     let approvalSummaries;
-    const postControl = (body, sig) => xllFetch(`${resolvedExcelHttpUrl}/exec`, {
+    const postExec = (body, sig) => xllFetch(`${resolvedExcelHttpUrl}/exec`, {
         method: 'POST',
         headers: { 'Content-Type': 'application/json' },
         body: JSON.stringify(body),
@@ -224,7 +162,7 @@ export async function executeComLocally(excelTarget, code, options) {
     const sendExecRequest = async () => {
         let response;
         try {
-            response = await postControl({ code, autoApprove }, signal);
+            response = await postExec({ code, autoApprove: true }, signal);
         }
         catch (e) {
             if (signal?.aborted || isEditModeAbortError(e)) {
@@ -237,7 +175,6 @@ export async function executeComLocally(excelTarget, code, options) {
     };
     // Static analysis: skip approval for code with no detectable writes.
     const analysis = analyzeComWriteCode(code);
-    const dangerousLifecycleBypassed = analysis.alwaysRequireApproval && isDangerousExcelLifecycleBypassed(ctx);
     if (analysis.alwaysRequireApproval) {
         const approvalResult = await requestAlwaysConfirmApproval(approval, ctx, description, analysis.dangerousOperations, observability);
         const { choice } = approvalResult;
@@ -257,25 +194,26 @@ export async function executeComLocally(excelTarget, code, options) {
         }
         approvalSummaries = appendApprovalSummary(approvalSummaries, choice);
     }
-    const autoApprove = analysis.isReadOnly || analysis.alwaysRequireApproval || (approval?.isAutoApproved() ?? false);
-    const executionDecision = dangerousLifecycleBypassed
-        ? 'allow'
-        : resolveApprovalExecutionDecision({
-            surface: getApprovalSurface(ctx),
-            gateAvailable: approval !== undefined,
-            isSafe: analysis.isReadOnly,
-            isAutoApproved: approval?.isAutoApproved() ?? false,
-            alwaysRequireExplicitApproval: false,
-            allowHeadlessApproval: ctx?.permissionPolicy?.office.allowHeadlessApproval ?? false
-        });
-    if (executionDecision === 'deny') {
-        return {
-            result: {
-                ok: true,
-                output: INTERACTIVE_APPROVAL_UNAVAILABLE_MESSAGE
-            },
-            outcome: 'approval-unavailable'
-        };
+    else if (!analysis.isReadOnly) {
+        const approvalResult = await requestWriteApproval(approval, ctx, description, observability);
+        const { choice } = approvalResult;
+        if (choice !== APPROVAL.ACCEPT && choice !== APPROVAL.ACCEPT_ALL) {
+            return {
+                result: {
+                    ok: true,
+                    output: approvalResult.unavailable
+                        ? INTERACTIVE_APPROVAL_UNAVAILABLE_MESSAGE
+                        : 'Excel operation rejected by user.'
+                },
+                outcome: approvalResult.unavailable ? 'approval-unavailable' : 'rejected'
+            };
+        }
+        if (approvalResult.prompted && choice === APPROVAL.ACCEPT_ALL) {
+            approval?.onAcceptAll();
+        }
+        if (approvalResult.prompted) {
+            approvalSummaries = appendApprovalSummary(approvalSummaries, choice);
+        }
     }
     let result = await sendExecRequest();
     if (result.editMode) {
@@ -289,34 +227,6 @@ export async function executeComLocally(excelTarget, code, options) {
     if (!result.ok) {
         return { result, outcome: 'failed' };
     }
-    if (ctx) {
-        const { outcome, approvalSummaries: resolvedApprovalSummaries } = await resolveApproval(result, approval, (b) => postControl(b), ctx, description, observability, approvalSummaries);
-        return { result, outcome, approvalSummaries: resolvedApprovalSummaries };
-    }
-    if (result.diff) {
-        try {
-            await postControl({ control: CONTROL.REVERT, cfTxId: result.cfTxId });
-            return {
-                result: { ...result, output: 'Excel operation rejected by user.' },
-                outcome: 'reverted'
-            };
-        }
-        catch {
-            if (result.cfTxId != null) {
-                await postControl({ control: CONTROL.CLEANUP, cfTxId: result.cfTxId }).catch(() => { });
-            }
-            return {
-                result: { ...result, output: 'Excel operation rejected by user.' },
-                outcome: 'revert-failed'
-            };
-        }
-    }
-    if (result.cfTxId != null) {
-        await postControl({ control: CONTROL.CLEANUP, cfTxId: result.cfTxId }).catch((e) => {
-            const msg = e instanceof Error ? e.message : String(e);
-            log.error('excel-exec', `CF cleanup failed for tx ${result.cfTxId}: ${msg}`);
-        });
-    }
     return { result, outcome: 'accepted', approvalSummaries };
 }
 //# sourceMappingURL=executor.js.map

package/dist/app/local/tools/execute-code/observability.js

@@ -31,26 +31,6 @@ export function createExecuteCodeApprovalObservability(clientLogger) {
                     approval_kind: request.payload.kind
                 }
             });
-        },
-        onReverted: (request) => {
-            clientLogger.info({
-                module: 'tool',
-                event: 'tool_approval_reverted',
-                data: {
-                    tool_name: TOOL_NAME,
-                    approval_kind: request.payload.kind
-                }
-            });
-        },
-        onRevertFailed: (request) => {
-            clientLogger.error({
-                module: 'tool',
-                event: 'tool_approval_revert_failed',
-                data: {
-                    tool_name: TOOL_NAME,
-                    approval_kind: request.payload.kind
-                }
-            });
         }
     };
 }

package/dist/app/local/tools/execute-code/tool.js

@@ -81,23 +81,6 @@ export function createExcelExecTool(resolveTarget, approval, observability) {
                 ctx: ctx,
                 signal: signal ?? undefined
             });
-            if (outcome === 'revert-failed') {
-                return {
-                    content: [
-                        {
-                            type: 'text',
-                            text: 'Revert failed — changes are still applied in Excel. You may need to undo manually (Ctrl+Z).'
-                        }
-                    ],
-                    details: result
-                };
-            }
-            if (outcome === 'reverted') {
-                return {
-                    content: [{ type: 'text', text: 'Changes reverted by user.' }],
-                    details: result
-                };
-            }
             if (outcome === 'rejected') {
                 return {
                     content: [{ type: 'text', text: USER_REJECTED_APPROVAL_MESSAGE }],

package/dist/app/local/tools/execute-code/write-analyzer.js

@@ -47,8 +47,10 @@ const RAW_COM_WRITES = [
     /\.ClearComments\s*\(/,
     /\.ClearNotes\s*\(/,
     /\.Copy\s*\(/,
+    /\.Cut\s*\(/,
     /\.PasteSpecial\s*\(/,
     /\.AutoFill\s*\(/,
+    /\.Resize\s*\(/,
     /\.Merge\s*\(/,
     /\.UnMerge\s*\(/,
     /\.Move\s*\(/,

package/dist/app/local/tools/take-screenshot/com-screenshot.js

@@ -13,6 +13,7 @@ async function captureComScreenshot(host, range, signal) {
         headers: { 'Content-Type': 'application/json' },
         body: JSON.stringify({
             control: 'screenshot',
+            workbook: range.workbook,
             sheet: range.sheet,
             range: range.cell_range
         }),
@@ -43,6 +44,7 @@ export async function captureComScreenshots(ranges, signal, resolveTarget) {
         const msg = error instanceof Error ? error.message : String(error);
         console.error(`[take_screenshot] COM capture unavailable: ${msg}`);
         return ranges.map((range) => ({
+            workbook: range.workbook,
             sheet: range.sheet,
             cell_range: range.cell_range,
             data: null
@@ -58,7 +60,12 @@ export async function captureComScreenshots(ranges, signal, resolveTarget) {
             const msg = error instanceof Error ? error.message : String(error);
             console.error(`[take_screenshot] COM capture failed for ${range.sheet}!${range.cell_range}: ${msg}`);
         }
-        screenshots.push({ sheet: range.sheet, cell_range: range.cell_range, data });
+        screenshots.push({
+            workbook: range.workbook,
+            sheet: range.sheet,
+            cell_range: range.cell_range,
+            data
+        });
     }
     return screenshots;
 }

package/dist/app/local/tools/take-screenshot/mog-screenshot.d.ts

@@ -1,6 +1,5 @@
 import type { Screenshot, ScreenshotRange } from '../../../tools/take-screenshot.js';
 interface MogSheetPoolWithScreenshots {
-    activeWorkbookName: string | null | undefined;
     getWorkbook(name: string): unknown;
 }
 export declare function captureMogScreenshots(pool: MogSheetPoolWithScreenshots, ranges: ScreenshotRange[], _signal?: AbortSignal): Promise<Screenshot[]>;

package/dist/app/local/tools/take-screenshot/mog-screenshot.js

@@ -4,26 +4,14 @@ function isMogWorkbookWithScreenshots(value) {
         typeof value.captureScreenshot === 'function');
 }
 export async function captureMogScreenshots(pool, ranges, _signal) {
-    if (!pool.activeWorkbookName) {
-        return ranges.map((range) => ({
-            sheet: range.sheet,
-            cell_range: range.cell_range,
-            data: null
-        }));
-    }
-    const workbook = pool.getWorkbook(pool.activeWorkbookName);
-    if (!isMogWorkbookWithScreenshots(workbook)) {
-        console.error('[take_screenshot] Active Mog workbook does not support screenshot capture');
-        return ranges.map((range) => ({
-            sheet: range.sheet,
-            cell_range: range.cell_range,
-            data: null
-        }));
-    }
     const screenshots = [];
     for (const range of ranges) {
         let data = null;
         try {
+            const workbook = pool.getWorkbook(range.workbook);
+            if (!isMogWorkbookWithScreenshots(workbook)) {
+                throw new Error(`Workbook does not support screenshot capture: ${range.workbook}`);
+            }
             const pngBuffer = await workbook.captureScreenshot(range.sheet, range.cell_range);
             data = Buffer.from(pngBuffer).toString('base64');
         }
@@ -31,7 +19,12 @@ export async function captureMogScreenshots(pool, ranges, _signal) {
             const msg = error instanceof Error ? error.message : String(error);
             console.error(`[take_screenshot] Mog capture failed for ${range.sheet}!${range.cell_range}: ${msg}`);
         }
-        screenshots.push({ sheet: range.sheet, cell_range: range.cell_range, data });
+        screenshots.push({
+            workbook: range.workbook,
+            sheet: range.sheet,
+            cell_range: range.cell_range,
+            data
+        });
     }
     return screenshots;
 }

package/dist/app/modes/manage/prompt.js

@@ -65,7 +65,7 @@ Launch verification subagent(s) in the background to check results against requi
 
 - **NEVER** write "based on your findings, implement X" — this pushes synthesis onto the subagent. YOU must synthesize research results and write precise instructions.
 - **NEVER** delegate understanding. Read subagent outputs, extract what matters, then write specific implementation instructions.
-- **NEVER** launch a single subagent for a task that could be parallelized. If 5 sheets need independent work, launch 5 subagents.
+- **NEVER** spawn a replacement until the original subagent has stopped, and never resume the original afterward.
 - **NEVER** give vague instructions like "fix the formatting" — specify which cells, which format, which style.
 
 ## Delegation Format

package/dist/app/providers/model-resolver.js

@@ -59,6 +59,7 @@ function getDefaultModelCandidatesForProvider(provider) {
     if (provider === SHORTCUT_PROVIDER_ID) {
         return [
             SHORTCUT_OPENAI_MODEL_ID,
+            SHORTCUT_MODEL_ID.ClaudeFable51,
             SHORTCUT_MODEL_ID.ClaudeFable5,
             SHORTCUT_MODEL_ID.ClaudeOpus5
         ];

package/dist/app/session/session-models.js

@@ -34,7 +34,7 @@ export const ALL_THINKING_LEVELS = [
 /** Models shown in interactive model controls for known providers. Unlisted providers show all. */
 const INTERACTIVE_MODELS_PER_PROVIDER = {
     shortcut: new Set([
-        SHORTCUT_MODEL_ID.ClaudeFable5,
+        SHORTCUT_MODEL_ID.ClaudeFable51,
         SHORTCUT_MODEL_ID.ClaudeOpus5,
         SHORTCUT_OPENAI_MODEL_ID,
         SHORTCUT_MODEL_ID.Gpt56Luna,

package/dist/app/tools/take-screenshot.d.ts

@@ -18,10 +18,12 @@
 import type { ToolDefinition } from '../../core/core-types.js';
 import { type ShortcutLlmProxyClient } from '../providers/shortcut-llm-proxy-client.js';
 export interface ScreenshotRange {
+    workbook: string;
     sheet: string;
     cell_range: string;
 }
 export interface Screenshot {
+    workbook: string;
     sheet: string;
     cell_range: string;
     /** Base64-encoded PNG data. Null/undefined means capture failed for this range. */
@@ -49,6 +51,7 @@ export interface TakeScreenshotOperations {
 }
 declare const schema: import("@sinclair/typebox").TObject<{
     ranges: import("@sinclair/typebox").TArray<import("@sinclair/typebox").TObject<{
+        workbook: import("@sinclair/typebox").TString;
         sheet: import("@sinclair/typebox").TString;
         cell_range: import("@sinclair/typebox").TString;
     }>>;

package/dist/app/tools/take-screenshot.js

@@ -33,6 +33,9 @@ const DESCRIPTION = TAKE_SCREENSHOT_TOOL_DESCRIPTION;
 // Schema
 // ---------------------------------------------------------------------------
 const RangeSchema = Type.Object({
+    workbook: Type.String({
+        description: 'Exact workbook name to capture, such as "Model.xlsx". Use the name only, not a path.'
+    }),
     sheet: Type.String({ description: 'Sheet name to capture.' }),
     cell_range: Type.String({ description: 'Range to capture, e.g. "A1:G10".' })
 });
@@ -135,7 +138,7 @@ export function createTakeScreenshotTool(operations = defaultOperations) {
             // Filter to valid screenshots (those with data)
             const validScreenshots = screenshots.filter((ss) => ss.data);
             if (validScreenshots.length === 0) {
-                const rangesStr = ranges.map((r) => `${r.sheet}!${r.cell_range}`).join(', ');
+                const rangesStr = ranges.map((r) => `${r.workbook}:${r.sheet}!${r.cell_range}`).join(', ');
                 return {
                     content: [
                         {
@@ -151,7 +154,7 @@ export function createTakeScreenshotTool(operations = defaultOperations) {
             try {
                 const result = await operations.analyzeWithVisionLLM(validScreenshots, query, selectedModel, extCtx.shortcutLlmProxy, signal ?? undefined);
                 const rangesSummary = validScreenshots
-                    .map((ss) => `${ss.sheet}!${ss.cell_range}`)
+                    .map((ss) => `${ss.workbook}:${ss.sheet}!${ss.cell_range}`)
                     .join(', ');
                 return {
                     content: [{ type: 'text', text: result.text }],