mirrors/hermes-agent
1
0
Fork 0
mirror of https://github.com/NousResearch/hermes-agent.git synced 2026-05-07 02:51:50 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 7

fix(tui): clamp copied selection bounds

Browse source 
Clamp copied selection columns to the screen width before scanning rendered cells.
This commit is contained in:
Brooklyn Nicholson 2026-04-25 15:32:45 -05:00
parent b1c18e5a41
commit 31d7f1951a
2 changed files with 12 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

10
ui-tui/packages/hermes-ink/src/ink/selection.test.ts
View file

@ -55,6 +55,16 @@ describe('selection whitespace handling', () => {
expect(getSelectedText(selection, screen)).toBe(' x')
})
it('clamps copied selection bounds to screen width', () => {
const { screen } = screenWithText()
const selection = createSelectionState()
startSelection(selection, 0, 1)
updateSelection(selection, 99, 1)
expect(getSelectedText(selection, screen)).toBe('hi')
})
it('does not paint selection background on leading/trailing empty cells or empty rows', () => {
const { screen, styles } = screenWithText()
const selection = createSelectionState()

4
ui-tui/packages/hermes-ink/src/ink/selection.ts
View file

@ -969,8 +969,8 @@ export function getSelectedText(s: SelectionState, screen: Screen): string {
}
for (let row = start.row; row <= end.row; row++) {
const rowStart = row === start.row ? start.col : 0
const rowEnd = row === end.row ? end.col : screen.width - 1
const rowStart = Math.max(0, row === start.row ? start.col : 0)
const rowEnd = Math.min(row === end.row ? end.col : screen.width - 1, screen.width - 1)
const bounds = selectionContentBounds(screen, row, rowStart, rowEnd)
joinRows(lines, bounds ? extractRowText(screen, row, bounds.first, bounds.last) : '', sw[row]! > 0)