Skip to content

perf(cli): memoize console preview rows - #156

Merged
codeaholicguy merged 1 commit into
mainfrom
feature-console-preview-row-memoization
Aug 14, 2026
Merged

perf(cli): memoize console preview rows#156
codeaholicguy merged 1 commit into
mainfrom
feature-console-preview-row-memoization

Conversation

@codeaholicguy

Copy link
Copy Markdown
Owner

Summary

  • extract a pure flattened preview row-model helper
  • memoize preview rows by messages and slice the viewport from that model
  • preserve the existing message-based viewport helper while avoiding duplicate scans in PreviewPane
  • add a deterministic Ink rerender regression test for scroll-only updates

Validation

  • npm test --workspace packages/cli -- src/tests/tui/console/PreviewPane.test.ts (11 passed)
  • npm test --workspace packages/cli (960 passed)
  • npm run lint --workspace packages/cli (exit 0; 5 existing unrelated warnings)
  • npm run build --workspace packages/cli (exit 0)

Risks

  • None known; viewport, continuation indicator, append-offset adjustment, multiline, timestamp, and scrolling behavior continue through the existing tested row-slicing path.

@codeaholicguy
codeaholicguy merged commit ef0184f into main Aug 14, 2026
7 checks passed
@codeaholicguy
codeaholicguy deleted the feature-console-preview-row-memoization branch August 14, 2026 12:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant