From 8ef48bdb863b047497af6002f55d70ff58cdc4bc Mon Sep 17 00:00:00 2001 From: GitHub Copilot Date: Sat, 24 Jan 2026 10:34:34 +0000 Subject: [PATCH] chore: remove completed TODO.md and REFACTORING.md --- REFACTORING.md | 40 ---------------------------------------- TODO.md | 3 --- 2 files changed, 43 deletions(-) delete mode 100644 REFACTORING.md delete mode 100644 TODO.md diff --git a/REFACTORING.md b/REFACTORING.md deleted file mode 100644 index b49ba3c..0000000 --- a/REFACTORING.md +++ /dev/null @@ -1,40 +0,0 @@ -# Refactoring & Audit Checklist - -## Tooling -- [x] Makefile restored (install, lint, format, test, coverage, check) -- [x] Coverage gate at fail_under=80 (current ~88%) - -## Dead Code Removal -- [x] Removed packets/environment protocol code -- [x] Removed unused Account model -- [x] Removed retry.py and related tests - -## LocalServer / Protocol -- [x] JSON WS protocol enforced (stdin/resize/ping/pong) -- [x] Static assets delegated to textual-serve -- [x] /screenshot.svg renders replay buffer to SVG -- [x] Disconnect triggers resize to 132x45 -- [x] Narrow WebSocket error handling (avoid bare Exception) -- [x] Consider TaskGroup/cleanup context for aiohttp runner - -## Sessions -- [x] Session.is_running() added -- [x] AppSession double JSON parse fixed; payload capped (16MB) -- [x] TerminalSession replay buffer; resize on disconnect -- [x] TerminalSession set_terminal_size is blocking; consider run_in_executor - -## Poller -- [x] OSError-only read handling; write error handling added -- [x] TwoWayDict enforces 1:1 mapping (raises on duplicate value) - -## CLI / Config -- [x] File-path detection helper deduped -- [x] Config uses tomllib (py311+) - -## Tests -- [x] 135 tests passing; coverage ~88% - -## Remaining Ideas (Low Priority) -- [x] Consolidate WS dispatch table -- [x] Simplify _get_ws_url_from_request -- [x] Normalize logging style (f-string vs %%) diff --git a/TODO.md b/TODO.md deleted file mode 100644 index 8ed6504..0000000 --- a/TODO.md +++ /dev/null @@ -1,3 +0,0 @@ -- [x] Set tab title to the container name -- [x] screenshots are still showing the tmux bar creeping up and slowly taking over the screenshot - - Fixed by maintaining pyte Screen state in TerminalSession, updated as data flows