chore: remove unused regex patterns and update optimization tracking

- Remove deprecated ANSI_ESCAPE_PATTERN and WHITESPACE_PATTERN from
  respawn-controller.ts (were marked as unused)
- Update optimization-todos.md with progress on completed items:
  - Item 1: BufferAccumulator implemented
  - Item 2: lastIndex resets already in place
  - Item 3: Auto-trim via BufferAccumulator
  - Item 9: Unused regex patterns removed
  - Item 13: Files not dead code (used by CLI)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This commit is contained in:
arkon
2026-01-21 04:59:15 +01:00
co-authored by Claude Opus 4.5
parent cab2b20730
commit 344b34d9bf
2 changed files with 6 additions and 19 deletions
+5 -5
View File
@@ -132,18 +132,18 @@ Each item has specific file:line references and expected impact estimates.
| Item | Status | Completed By | Notes |
|------|--------|--------------|-------|
| 1 | ☐ Pending | | |
| 2 | ☐ Pending | | |
| 3 | ☐ Pending | | |
| 1 | ✅ Done | Claude | BufferAccumulator in session.ts, respawn-controller.ts |
| 2 | ✅ Done | Claude | Already had lastIndex resets in place |
| 3 | ✅ Done | Claude | BufferAccumulator auto-trims, no manual limits needed |
| 4 | ☐ Pending | | |
| 5 | ☐ Pending | | |
| 6 | ☐ Pending | | |
| 7 | ☐ Pending | | |
| 8 | ☐ Pending | | |
| 9 | ☐ Pending | | |
| 9 | ✅ Done | Claude | Removed ANSI_ESCAPE_PATTERN, WHITESPACE_PATTERN |
| 10 | ☐ Pending | | |
| 11 | ☐ Pending | | |
| 12 | ☐ Pending | | |
| 13 | ☐ Pending | | |
| 13 | ☐ Pending | | Not unused - used by CLI |
| 14 | ☐ Pending | | |
| 15 | ☐ Pending | | |
+1 -14
View File
@@ -43,20 +43,7 @@ const MAX_RESPAWN_BUFFER_SIZE = 1024 * 1024; // 1MB
*/
const RESPAWN_BUFFER_TRIM_SIZE = 512 * 1024; // 512KB
// ========== Pre-compiled Regex Patterns ==========
/**
* Matches ANSI escape codes for terminal control sequences.
* Used to strip formatting before pattern matching.
* @deprecated Currently unused but kept for potential future use
*/
const ANSI_ESCAPE_PATTERN = /\x1b\[[0-9;]*[HJKmsu?lh]/g;
/**
* Matches whitespace sequences for normalization.
* @deprecated Currently unused but kept for potential future use
*/
const WHITESPACE_PATTERN = /\s+/g;
// ========== Constants ==========
/**
* The definitive "ready for input" indicator.