Skip to content

fix(tui): align wrapped inline tool rows#28664

Merged
kitlangton merged 10 commits into
anomalyco:devfrom
kitlangton:kit/inline-tool-wrap-snapshot
May 30, 2026
Merged

fix(tui): align wrapped inline tool rows#28664
kitlangton merged 10 commits into
anomalyco:devfrom
kitlangton:kit/inline-tool-wrap-snapshot

Conversation

@kitlangton
Copy link
Copy Markdown
Contributor

Summary

  • Render completed inline tool rows with a fixed icon column so wrapped grep/glob/read summaries hang under the tool text instead of the icon.
  • Preserve pending inline tool rendering and denied styling.
  • Add a beta snapshot fixture comparing current, stable wrap, hanging indent, detail rows, and compact layouts at a narrow terminal width.

Beta

  • Marked with the beta label for UI feedback on the new wrapping behavior.

Testing

  • bun run test -- test/cli/tui/inline-tool-wrap-snapshot.test.tsx
  • bunx oxlint packages/opencode/src/cli/cmd/tui/routes/session/index.tsx packages/opencode/test/cli/tui/inline-tool-wrap-snapshot.test.tsx (existing warnings in session/index.tsx)
  • bun typecheck
  • push hook: bun turbo typecheck

@kitlangton kitlangton force-pushed the kit/inline-tool-wrap-snapshot branch from 7669fc2 to eb5d302 Compare May 21, 2026 19:02
@kitlangton kitlangton marked this pull request as ready for review May 21, 2026 19:47
setMargin(1)
return
}
setMargin(previous?.id.startsWith("text-") || previous?.id.startsWith("tool-block-") ? 1 : 0)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This mutates state during layout; could it desync scroll until resize?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants