shortcutxl 0.2.19 → 0.2.21

package/skills/sec-edgar/sec_to_pdf.py

@@ -33,52 +33,50 @@ async def convert_sec_filing_to_pdf(url: str, output_path: str) -> dict:
     """
     async with async_playwright() as p:
         # MUST use channel='chrome' - regular Chromium gets 403 blocked by SEC
-        browser = await p.chromium.launch(headless=True, channel="chrome")
-        context = await browser.new_context(
-            user_agent="ShortcutAgent/1.0 (support@shortcut.ai)"
-        )
+        browser = await p.chromium.launch(headless=True, channel='chrome')
+        context = await browser.new_context(user_agent='ShortcutAgent/1.0 (support@shortcut.ai)')
         page = await context.new_page()
 
         # Navigate to filing (networkidle waits for no network activity for 500ms)
-        response = await page.goto(url, wait_until="networkidle", timeout=20000)
+        response = await page.goto(url, wait_until='networkidle', timeout=20000)
 
         if response is None or response.status != 200:
             await browser.close()
             return {
-                "status": "error",
-                "code": response.status if response else 0,
-                "message": "SEC blocked request or navigation failed",
+                'status': 'error',
+                'code': response.status if response else 0,
+                'message': 'SEC blocked request or navigation failed',
             }
 
         # Count images for verification
-        images = await page.query_selector_all("img")
+        images = await page.query_selector_all('img')
         image_count = len(images)
 
         # Generate PDF
-        pdf_bytes = await page.pdf(format="A4", print_background=True)
+        pdf_bytes = await page.pdf(format='A4', print_background=True)
 
         # Ensure output directory exists and save
         output_dir = os.path.dirname(output_path)
         if output_dir:
             os.makedirs(output_dir, exist_ok=True)
-        with open(output_path, "wb") as f:
+        with open(output_path, 'wb') as f:
             f.write(pdf_bytes)
 
         await browser.close()
 
         return {
-            "status": "success",
-            "pdf_path": output_path,
-            "pdf_size_bytes": len(pdf_bytes),
-            "image_count": image_count,
+            'status': 'success',
+            'pdf_path': output_path,
+            'pdf_size_bytes': len(pdf_bytes),
+            'image_count': image_count,
         }
 
 
 async def main():
     if len(sys.argv) != 3:
-        print("Usage: python sec_to_pdf.py <filing_url> <output_path>")
+        print('Usage: python sec_to_pdf.py <filing_url> <output_path>')
         print()
-        print("Example:")
+        print('Example:')
         print(
             '  python sec_to_pdf.py "https://www.sec.gov/Archives/edgar/data/320193/000032019324000123/aapl-20240928.htm" "/workspace/aapl_10k.pdf"'
         )
@@ -87,23 +85,23 @@ async def main():
     url = sys.argv[1]
     output_path = sys.argv[2]
 
-    print("Converting SEC filing to PDF...")
-    print(f"  URL: {url}")
-    print(f"  Output: {output_path}")
+    print('Converting SEC filing to PDF...')
+    print(f'  URL: {url}')
+    print(f'  Output: {output_path}')
     print()
 
     result = await convert_sec_filing_to_pdf(url, output_path)
 
-    if result["status"] == "success":
-        size_mb = result["pdf_size_bytes"] / (1024 * 1024)
-        print("Success!")
-        print(f"  PDF saved to: {result['pdf_path']}")
-        print(f"  PDF size: {result['pdf_size_bytes']:,} bytes ({size_mb:.2f} MB)")
-        print(f"  Images rendered: {result['image_count']}")
+    if result['status'] == 'success':
+        size_mb = result['pdf_size_bytes'] / (1024 * 1024)
+        print('Success!')
+        print(f'  PDF saved to: {result["pdf_path"]}')
+        print(f'  PDF size: {result["pdf_size_bytes"]:,} bytes ({size_mb:.2f} MB)')
+        print(f'  Images rendered: {result["image_count"]}')
     else:
-        print(f"Error: {result['message']} (code: {result['code']})")
+        print(f'Error: {result["message"]} (code: {result["code"]})')
         sys.exit(1)
 
 
-if __name__ == "__main__":
+if __name__ == '__main__':
     asyncio.run(main())

package/xll/modules/debug_render.py

@@ -0,0 +1,289 @@
+"""
+debug_render.py — Progressive rendering tests for isolating display issues.
+
+Type these in Excel cells to test each layer of the rendering pipeline:
+
+  =debug_write_test()     Layer 1: Can we write to a cell at all via COM?
+  =debug_batch_test()     Layer 2: Does xl_batch() work?
+  =debug_grid_test()      Layer 3: Can we write a 2D grid?
+  =debug_format_pong()    Layer 4: Set column/row sizes for pong display
+  =debug_frame()          Layer 5: Render one static pong frame
+  =debug_counter()        Layer 6: Animated counter (1 cell, 1Hz)
+  =debug_stop()           Stop any running animation
+"""
+
+import shortcut_xl
+from shortcut_xl import xl_func, xl_log
+
+_counter_running = False
+
+
+# === Layer 1: Basic COM write ===
+
+
+@xl_func
+def debug_write_test():
+    """Write a value to Z1 via xl_app() COM. Simplest possible test."""
+    try:
+
+        def _do(app):
+            app.ActiveSheet.Range('Z1').Value = 'WRITE_OK'
+
+        shortcut_xl.xl_batch(_do)
+        return "Wrote 'WRITE_OK' to Z1 — check Z1"
+    except Exception as e:
+        xl_log(f'debug_write_test FAILED: {e}')
+        return f'FAILED: {e}'
+
+
+# === Layer 2: xl_batch with ScreenUpdating toggle ===
+
+
+@xl_func
+def debug_batch_test():
+    """Write 3 cells in one batch. Tests ScreenUpdating toggle + PumpWaitingMessages."""
+    try:
+
+        def _do(app):
+            sheet = app.ActiveSheet
+            sheet.Range('Z1').Value = 'BATCH'
+            sheet.Range('Z2').Value = 'TEST'
+            sheet.Range('Z3').Value = 'OK'
+
+        shortcut_xl.xl_batch(_do)
+        return 'Wrote Z1:Z3 — check those cells'
+    except Exception as e:
+        xl_log(f'debug_batch_test FAILED: {e}')
+        return f'FAILED: {e}'
+
+
+# === Layer 3: 2D grid write via Resize().Value ===
+
+
+@xl_func
+def debug_grid_test():
+    """Write a small 5x5 grid using the same Resize().Value pattern as pong."""
+    try:
+        grid = []
+        for r in range(5):
+            row = []
+            for c in range(5):
+                if r == 0 or r == 4 or c == 0 or c == 4:
+                    row.append('#')  # border
+                elif r == 2 and c == 2:
+                    row.append('X')  # center
+                else:
+                    row.append('')  # empty (same as pong does)
+            grid.append(row)
+
+        def _do(app):
+            sheet = app.ActiveSheet
+            sheet.Range('Z5:AD9').Value = grid
+
+        shortcut_xl.xl_batch(_do)
+        xl_log(f'debug_grid_test: wrote 5x5 grid to Z5:AD9, data={grid}')
+        return 'Wrote 5x5 grid to Z5:AD9 — check those cells'
+    except Exception as e:
+        xl_log(f'debug_grid_test FAILED: {e}')
+        return f'FAILED: {e}'
+
+
+# === Layer 3b: Grid with tuples (test COM conversion) ===
+
+
+@xl_func
+def debug_grid_tuple_test():
+    """Same as grid_test but converts to tuple-of-tuples for COM.
+    If this works but debug_grid_test doesn't, it's a list-vs-tuple issue."""
+    try:
+        grid = []
+        for r in range(5):
+            row = []
+            for c in range(5):
+                if r == 0 or r == 4 or c == 0 or c == 4:
+                    row.append('#')
+                elif r == 2 and c == 2:
+                    row.append('X')
+                else:
+                    row.append('')
+            grid.append(tuple(row))
+        grid = tuple(grid)
+
+        def _do(app):
+            sheet = app.ActiveSheet
+            sheet.Range('Z5:AD9').Value = grid
+
+        shortcut_xl.xl_batch(_do)
+        xl_log('debug_grid_tuple_test: wrote tuple grid to Z5:AD9')
+        return 'Wrote tuple-of-tuples 5x5 to Z5:AD9'
+    except Exception as e:
+        xl_log(f'debug_grid_tuple_test FAILED: {e}')
+        return f'FAILED: {e}'
+
+
+# === Layer 4: Format cells for pong display ===
+
+
+@xl_func
+def debug_format_pong():
+    """Set up column widths and row heights so pong grid looks right.
+    This is what pong.py is missing — without it, the grid is spread across
+    the entire screen and looks like nothing."""
+    try:
+
+        def _do(app):
+            sheet = app.ActiveSheet
+            # Columns A-AD (1-30): narrow width for character grid
+            cols = sheet.Range('A:AD')
+            cols.ColumnWidth = 2.0
+            cols.HorizontalAlignment = -4108  # xlCenter
+
+            # Rows 3-22 (pong grid area): compact height
+            rows = sheet.Range('3:22')
+            rows.RowHeight = 15
+
+            # Monospace font for the grid
+            grid_range = sheet.Range('A3:AD22')
+            grid_range.Font.Name = 'Consolas'
+            grid_range.Font.Size = 11
+
+            # Scoreboard formatting
+            sheet.Range('A1').Font.Name = 'Consolas'
+            sheet.Range('A1').Font.Size = 14
+            sheet.Range('A1').Font.Bold = True
+
+        shortcut_xl.xl_batch(_do)
+        return 'Formatted A:AD for pong display'
+    except Exception as e:
+        xl_log(f'debug_format_pong FAILED: {e}')
+        return f'FAILED: {e}'
+
+
+# === Layer 5: Static pong frame ===
+
+
+@xl_func
+def debug_frame():
+    """Render exactly one pong frame. No animation, no schedule_call.
+    If this shows up, the rendering works and the issue is in the animation loop."""
+    try:
+        WIDTH, HEIGHT, PADDLE_H = 30, 20, 4
+        grid = []
+        ball_x, ball_y = 15, 10
+        paddle_l, paddle_r = 8, 8
+
+        for r in range(HEIGHT):
+            row = []
+            for c in range(WIDTH):
+                ch = ''
+                if c == 0 and paddle_l <= r < paddle_l + PADDLE_H:
+                    ch = chr(9608)  # full block
+                elif c == WIDTH - 1 and paddle_r <= r < paddle_r + PADDLE_H:
+                    ch = chr(9608)
+                elif r == ball_y and c == ball_x:
+                    ch = chr(9679)  # black circle
+                elif c == WIDTH // 2 and r % 2 == 0:
+                    ch = chr(9474)  # light vertical
+                row.append(ch)
+            grid.append(row)
+
+        def _do(app):
+            sheet = app.ActiveSheet
+            sheet.Range('A3:AD22').Value = grid
+            sheet.Range('A1').Value = 'PONG   0  :  0'
+            sheet.Range('A2').Value = 'Static frame test'
+
+        shortcut_xl.xl_batch(_do)
+        xl_log('debug_frame: wrote static pong frame')
+        return 'Wrote static pong frame to A3:AD22'
+    except Exception as e:
+        xl_log(f'debug_frame FAILED: {e}')
+        return f'FAILED: {e}'
+
+
+# === Layer 5b: Static frame with ASCII (no Unicode) ===
+
+
+@xl_func
+def debug_frame_ascii():
+    """Same as debug_frame but uses ASCII chars only.
+    If this renders but debug_frame doesn't, it's a Unicode/font issue."""
+    try:
+        WIDTH, HEIGHT, PADDLE_H = 30, 20, 4
+        grid = []
+        ball_x, ball_y = 15, 10
+        paddle_l, paddle_r = 8, 8
+
+        for r in range(HEIGHT):
+            row = []
+            for c in range(WIDTH):
+                ch = '.'  # visible empty (not "")
+                if (
+                    c == 0
+                    and paddle_l <= r < paddle_l + PADDLE_H
+                    or c == WIDTH - 1
+                    and paddle_r <= r < paddle_r + PADDLE_H
+                ):
+                    ch = '|'
+                elif r == ball_y and c == ball_x:
+                    ch = 'O'
+                elif c == WIDTH // 2 and r % 2 == 0:
+                    ch = ':'
+                row.append(ch)
+            grid.append(row)
+
+        def _do(app):
+            sheet = app.ActiveSheet
+            sheet.Range('A3:AD22').Value = grid
+            sheet.Range('A1').Value = 'PONG   0  :  0'
+            sheet.Range('A2').Value = 'Static ASCII frame test'
+
+        shortcut_xl.xl_batch(_do)
+        xl_log('debug_frame_ascii: wrote ASCII pong frame')
+        return 'Wrote ASCII pong frame to A3:AD22'
+    except Exception as e:
+        xl_log(f'debug_frame_ascii FAILED: {e}')
+        return f'FAILED: {e}'
+
+
+# === Layer 6: Animated counter (simplest animation) ===
+
+
+@xl_func
+def debug_counter():
+    """Increment a single cell once per second. Simplest animation test.
+    If this works, schedule_call + xl_batch animation pipeline is fine."""
+    global _counter_running
+    if _counter_running:
+        return 'Already running! =debug_stop() first'
+
+    _counter_running = True
+    _count = [0]
+
+    def _tick():
+        if not _counter_running:
+            return
+        _count[0] += 1
+        xl_log(f'debug_counter tick: {_count[0]}')
+        try:
+
+            def _do(app):
+                app.ActiveSheet.Range('Z1').Value = f'Count: {_count[0]}'
+
+            shortcut_xl.xl_batch(_do)
+        except Exception as e:
+            xl_log(f'debug_counter error: {e}')
+
+        if _counter_running and _count[0] < 30:
+            shortcut_xl.schedule_call(_tick, 1.0)
+
+    shortcut_xl.schedule_call(_tick, 0.1)
+    return 'Counting in Z1 (1/sec, 30 max)'
+
+
+@xl_func
+def debug_stop():
+    """Stop the debug counter."""
+    global _counter_running
+    _counter_running = False
+    return 'Stopped'

package/xll/modules/shortcut_xl/_com.py

@@ -9,9 +9,9 @@ from shortcut_xl._log import xl_log
 _OBJID_NATIVEOM = -16
 
 # Excel window class names (hierarchy: XLMAIN → XLDESK → EXCEL7)
-_WC_XLMAIN = "XLMAIN"
-_WC_XLDESK = "XLDESK"
-_WC_EXCEL7 = "EXCEL7"
+_WC_XLMAIN = 'XLMAIN'
+_WC_XLDESK = 'XLDESK'
+_WC_EXCEL7 = 'EXCEL7'
 
 
 class _GUID(ctypes.Structure):
@@ -24,9 +24,7 @@ class _GUID(ctypes.Structure):
 
 
 # IID_IDispatch = {00020400-0000-0000-C000-000000000046}
-_IID_IDISPATCH = _GUID(
-    0x00020400, 0x0000, 0x0000,
-    (ctypes.c_byte * 8)(0xC0, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x46))
+_IID_IDISPATCH = _GUID(0x00020400, 0x0000, 0x0000, (ctypes.c_byte * 8)(0xC0, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x46))
 
 
 _user32 = ctypes.windll.user32
@@ -64,6 +62,7 @@ def _find_own_xlmain():
 def _ensure_com_init():
     """Initialize COM on the current thread (idempotent)."""
     import pythoncom
+
     pythoncom.CoInitialize()
 
 
@@ -91,7 +90,8 @@ def xl_app():
         if excel7:
             punk = ctypes.c_void_p()
             hr = oleacc.AccessibleObjectFromWindow(
-                excel7, _OBJID_NATIVEOM, ctypes.byref(_IID_IDISPATCH), ctypes.byref(punk))
+                excel7, _OBJID_NATIVEOM, ctypes.byref(_IID_IDISPATCH), ctypes.byref(punk)
+            )
             if hr == 0 and punk.value:
                 iunk = pythoncom.ObjectFromAddress(punk.value)
                 disp = iunk.QueryInterface(pythoncom.IID_IDispatch)
@@ -99,11 +99,10 @@ def xl_app():
                 return window.Application
 
     # Fallback — try ROT, but NEVER create a new instance.
-    xl_log("xl_app: HWND approach failed, trying GetActiveObject")
+    xl_log('xl_app: HWND approach failed, trying GetActiveObject')
     try:
-        return win32com.client.GetActiveObject("Excel.Application")
+        return win32com.client.GetActiveObject('Excel.Application')
     except Exception as e:
         raise RuntimeError(
-            "Could not find a running Excel instance. "
-            "Make sure Excel is open before loading the XLL."
+            'Could not find a running Excel instance. Make sure Excel is open before loading the XLL.'
         ) from e

package/xll/modules/shortcut_xl/_diff_highlight.py

@@ -19,26 +19,27 @@ from shortcut_xl._log import xl_log
 from shortcut_xl._tracking import DirtyCell
 
 # Excel constants
-_XL_EXPRESSION = 2          # xlExpression CF type
-_XL_CALC_MANUAL = -4135     # xlCalculationManual
+_XL_EXPRESSION = 2  # xlExpression CF type
+_XL_CALC_MANUAL = -4135  # xlCalculationManual
 _HIGHLIGHT_COLOR = 0x00FFFF  # yellow (BGR for #FFFF00) — Track Changes color
 
 # Thresholds
 _MAX_HIGHLIGHT_CELLS = 10_000  # skip CF above this — too many COM calls
-_ADDR_BATCH_LIMIT = 255        # max chars per comma-separated address string
-_MAX_TX_ENTRIES = 10           # evict oldest if exceeded (guards against leaked entries)
+_ADDR_BATCH_LIMIT = 255  # max chars per comma-separated address string
+_MAX_TX_ENTRIES = 10  # evict oldest if exceeded (guards against leaked entries)
 
 
 # ---------------------------------------------------------------------------
 # Stateless helpers
 # ---------------------------------------------------------------------------
 
+
 def _group_by_sheet(dirty_cells: list[DirtyCell]):
     by_sheet = {}
     for cell in dirty_cells:
         sheet = cell.get('sheet', '')
         if not sheet:
-            xl_log(f"_group_by_sheet: skipping cell with no sheet key: {cell}")
+            xl_log(f'_group_by_sheet: skipping cell with no sheet key: {cell}')
             continue
         by_sheet.setdefault(sheet, []).append(cell)
     return by_sheet
@@ -49,9 +50,9 @@ def _build_union(app, ws, addresses):
     if not addresses:
         return None
     batches = []
-    current = ""
+    current = ''
     for addr in addresses:
-        candidate = f"{current},{addr}" if current else addr
+        candidate = f'{current},{addr}' if current else addr
         if len(candidate) > _ADDR_BATCH_LIMIT and current:
             batches.append(current)
             current = addr
@@ -75,11 +76,11 @@ def _apply_cf_highlights(app, by_sheet):
             rng = _build_union(app, ws, [c['address'] for c in cells])
             if rng is None:
                 continue
-            rule = rng.FormatConditions.Add(_XL_EXPRESSION, None, "=TRUE")
+            rule = rng.FormatConditions.Add(_XL_EXPRESSION, None, '=TRUE')
             rule.Interior.Color = _HIGHLIGHT_COLOR
             rules.append(rule)
         except Exception as e:
-            xl_log(f"_apply_cf_highlights({sheet_name}): {e}")
+            xl_log(f'_apply_cf_highlights({sheet_name}): {e}')
     return rules
 
 
@@ -113,9 +114,9 @@ def revert_cells(app, dirty_cells: list[DirtyCell]):
                     else:
                         ws.Range(addr).Value = cell.get('oldValue')
                 except Exception as e:
-                    xl_log(f"revert_cells({sheet_name}!{addr}): {e}")
+                    xl_log(f'revert_cells({sheet_name}!{addr}): {e}')
         except Exception as e:
-            xl_log(f"revert_cells({sheet_name}): {e}")
+            xl_log(f'revert_cells({sheet_name}): {e}')
 
     try:
         app.ScreenUpdating = True
@@ -132,6 +133,7 @@ def revert_cells(app, dirty_cells: list[DirtyCell]):
 # Transaction store
 # ---------------------------------------------------------------------------
 
+
 class DiffHighlightStore:
     """Owns CF highlight state and revert data keyed by transaction ID.
 
@@ -141,8 +143,8 @@ class DiffHighlightStore:
 
     def __init__(self, max_tx_entries: int = _MAX_TX_ENTRIES):
         self._next_tx_id = 0
-        self._cf_state = {}      # tx_id -> list of CF rule COM objects
-        self._revert_data = {}   # tx_id -> list of dirty cell dicts
+        self._cf_state = {}  # tx_id -> list of CF rule COM objects
+        self._revert_data = {}  # tx_id -> list of dirty cell dicts
         self._max_tx_entries = max_tx_entries
 
     def highlight_changes(self, app, dirty_cells: list[DirtyCell]):
@@ -198,7 +200,7 @@ class DiffHighlightStore:
             try:
                 rule.Delete()
             except Exception as e:
-                xl_log(f"cleanup_by_tx: {e}")
+                xl_log(f'cleanup_by_tx: {e}')
 
     def revert_by_tx(self, app, tx_id):
         """Look up revert data by transaction ID and restore old values.
@@ -213,7 +215,7 @@ class DiffHighlightStore:
             try:
                 rule.Delete()
             except Exception as e:
-                xl_log(f"revert_by_tx cleanup: {e}")
+                xl_log(f'revert_by_tx cleanup: {e}')
         if not dirty_cells:
             return 0
         revert_cells(app, dirty_cells)

package/xll/modules/shortcut_xl/_exec_entry.py

@@ -35,14 +35,25 @@ from shortcut_xl._sandbox import sandbox_flag as _sandbox_flag
 # audit hook can't distinguish those from user-initiated imports.
 # User code imports are already blocked by removing __import__ from
 # builtins in _managed.py._SAFE_BUILTINS.
-_BLOCKED_AUDIT_EVENTS = frozenset({
-    'socket.connect', 'socket.bind', 'socket.sendto', 'socket.listen',
-    'subprocess.Popen',
-    'os.system', 'os.exec', 'os.spawn', 'os.popen',
-    'os.remove', 'os.rename', 'os.mkdir', 'os.rmdir',
-    'ctypes.dlopen',
-    'webbrowser.open',
-})
+_BLOCKED_AUDIT_EVENTS = frozenset(
+    {
+        'socket.connect',
+        'socket.bind',
+        'socket.sendto',
+        'socket.listen',
+        'subprocess.Popen',
+        'os.system',
+        'os.exec',
+        'os.spawn',
+        'os.popen',
+        'os.remove',
+        'os.rename',
+        'os.mkdir',
+        'os.rmdir',
+        'ctypes.dlopen',
+        'webbrowser.open',
+    }
+)
 
 
 def _sandbox_audit(event, args):
@@ -64,6 +75,7 @@ from shortcut_xl._diff_highlight import DiffHighlightStore
 from shortcut_xl._managed import run_managed
 from shortcut_xl._threading import _run_on_main
 from shortcut_xl._tracking import DirtyCell
+from shortcut_xl.api.categorize import com_error_to_str
 from shortcut_xl.api.utils.helpers import address_to_index, build_cell_ref
 
 _MAX_DIFF_JSON_CELLS = 1000
@@ -80,6 +92,10 @@ def _format_cell_value(v):
     if isinstance(v, bool):
         return 'TRUE' if v else 'FALSE'
     if isinstance(v, int):
+        # COM error codes are large negative ints — display as error string
+        err = com_error_to_str(v)
+        if err is not None:
+            return err
         return v
     if isinstance(v, str):
         return 'empty' if len(v) == 0 else v
@@ -150,7 +166,7 @@ def dispatch_request(body_json):
         return json.dumps({'ok': False, 'error': f'Invalid JSON: {e}'})
 
     if not isinstance(request, dict):
-        return json.dumps({'ok': False, 'error': "request must be a JSON object"})
+        return json.dumps({'ok': False, 'error': 'request must be a JSON object'})
 
     if 'code' in request:
         return exec_and_respond(request['code'], auto_approve=request.get('autoApprove', False))
@@ -189,7 +205,7 @@ def exec_and_respond(code_str, *, auto_approve=False):
     except Exception:
         output = sio.getvalue()
         tb = traceback.format_exc()
-        error = f"{output}{tb}" if output else tb
+        error = f'{output}{tb}' if output else tb
         return json.dumps({'ok': False, 'error': error})
     finally:
         sys.stdout, sys.stderr = old_out, old_err
@@ -206,11 +222,13 @@ def handle_control(request):
 
     if action == 'revert':
         if cf_tx_id is None:
-            return json.dumps({'ok': False, 'error': "revert requires cfTxId"})
+            return json.dumps({'ok': False, 'error': 'revert requires cfTxId'})
         n = _run_on_main(lambda: _store.revert_by_tx(xl_app(), cf_tx_id))
         if n == 0:
-            return json.dumps({'ok': True, 'output': 'Nothing to revert (transaction not found or already cleaned up).'})
-        return json.dumps({'ok': True, 'output': f"Reverted {n} cell{'s' if n != 1 else ''}."})
+            return json.dumps(
+                {'ok': True, 'output': 'Nothing to revert (transaction not found or already cleaned up).'}
+            )
+        return json.dumps({'ok': True, 'output': f'Reverted {n} cell{"s" if n != 1 else ""}.'})
 
     elif action == 'cleanup':
         if cf_tx_id is not None:
@@ -218,4 +236,4 @@ def handle_control(request):
         return json.dumps({'ok': True, 'output': ''})
 
     else:
-        return json.dumps({'ok': False, 'error': f"Unknown control action: {action}"})
+        return json.dumps({'ok': False, 'error': f'Unknown control action: {action}'})

package/xll/modules/shortcut_xl/_log.py

@@ -6,7 +6,7 @@ import os
 
 def xl_log(msg):
     """Append a timestamped message to %TEMP%\\shortcutxl.log."""
-    log_path = os.path.join(os.environ.get("TEMP", "."), "shortcutxl.log")
-    timestamp = datetime.datetime.now().strftime("%Y-%m-%d %H:%M:%S.%f")[:-3]
-    with open(log_path, "a", encoding="utf-8") as f:
-        f.write(f"[{timestamp}] {msg}\n")
+    log_path = os.path.join(os.environ.get('TEMP', '.'), 'shortcutxl.log')
+    timestamp = datetime.datetime.now().strftime('%Y-%m-%d %H:%M:%S.%f')[:-3]
+    with open(log_path, 'a', encoding='utf-8') as f:
+        f.write(f'[{timestamp}] {msg}\n')