Skip to content

feat: compact tool pill UI with auto-grouping#3

Merged
dimakis merged 1 commit into
mainfrom
feat/tool-call-ui
Mar 31, 2026
Merged

feat: compact tool pill UI with auto-grouping#3
dimakis merged 1 commit into
mainfrom
feat/tool-call-ui

Conversation

@dimakis
Copy link
Copy Markdown
Owner

@dimakis dimakis commented Mar 31, 2026

Summary

Replaces the bulky tool call cards (which dominated the viewport on mobile) with compact single-line pills that auto-group.

Before: Each tool call was a multi-line card taking ~3+ lines. 8 Read calls = entire screen filled with cards, no visible response.

After:

  • Individual pills: [dot] Read server/chat.ts — one line. Shows "Running..." while active, fades to green dot when done. Tap to expand for full input/result.
  • Auto-grouping: 3+ consecutive tool calls collapse into a single row: [dots] 5 tool calls [v]. Tap to expand.
  • Smart state: Groups stay expanded while tools are running, auto-collapse when all finish.
  • Text messages break grouping, so you always see Claude's responses immediately.

Test plan

  • Start a chat that triggers multiple tool calls (e.g., "what files are in this repo?")
  • Verify individual tool calls show as compact pills, not cards
  • Verify 3+ consecutive calls collapse into a group
  • Verify group expands on tap, shows individual pills inside
  • Verify group auto-collapses when all tools finish
  • Verify tapping a done pill expands to show the result
  • Verify text messages appear between tool groups correctly

Made with Cursor

Replace bulky tool cards with compact single-line pills:
- Running: dot + name + input + "Running..." on one line
- Done: green dot + name + input, faded. Tap to expand result.
- 3+ consecutive tool calls auto-group into collapsible summary
- Groups show dot row + "N tool calls", expand to individual pills
- Groups auto-collapse when all tools finish
- Text messages break grouping

Made-with: Cursor
@dimakis dimakis merged commit 3079a61 into main Mar 31, 2026
1 check passed
@dimakis dimakis deleted the feat/tool-call-ui branch March 31, 2026 23:47
dimakis added a commit that referenced this pull request Apr 10, 2026
- Add script existence check before exec (was #1 critical)
- Add Zod CalendarResponse schema for stdout validation (#2)
- Check r.ok before parsing fetch response in hook (#3)
- Compute today fresh in handleToday to avoid stale date (#6)
- Extract EventCard + SprintBar to separate component files (#7)
- Extract calendar CSS to dedicated calendar.css (#8)
- Add tests: Day/Week toggle, Today button, EventCard expand,
  milestone expand, non-ok HTTP response handling (#4, #5)

832 tests passing (5 new).

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
dimakis added a commit that referenced this pull request Apr 10, 2026
- Add script existence check before exec (was #1 critical)
- Add Zod CalendarResponse schema for stdout validation (#2)
- Check r.ok before parsing fetch response in hook (#3)
- Compute today fresh in handleToday to avoid stale date (#6)
- Extract EventCard + SprintBar to separate component files (#7)
- Extract calendar CSS to dedicated calendar.css (#8)
- Add tests: Day/Week toggle, Today button, EventCard expand,
  milestone expand, non-ok HTTP response handling (#4, #5)

832 tests passing (5 new).

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
dimakis added a commit that referenced this pull request Apr 10, 2026
…sprints (#164)

* feat(calendar): add calendar view with meetings, milestones, and sprints

Unified calendar page that merges Google Calendar events, release
milestones, and sprint boundaries via the mgmt calendar_api.py backend.
Mobile-first day/week view with responsive desktop grid layout.

- GET /api/calendar endpoint shells out to calendar_api.py
- useCalendarData hook with date navigation
- CalendarView page with EventCard expand/collapse
- Sprint bars, milestone badges, attendee chips
- Day/Week toggle, prev/next/today navigation
- 17 tests (5 server + 5 hook + 7 page)
- Quick action added to .mitzo.json
- Route at /calendar behind ProtectedRoute

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* fix(calendar): address all Centaur review findings

- Add script existence check before exec (was #1 critical)
- Add Zod CalendarResponse schema for stdout validation (#2)
- Check r.ok before parsing fetch response in hook (#3)
- Compute today fresh in handleToday to avoid stale date (#6)
- Extract EventCard + SprintBar to separate component files (#7)
- Extract calendar CSS to dedicated calendar.css (#8)
- Add tests: Day/Week toggle, Today button, EventCard expand,
  milestone expand, non-ok HTTP response handling (#4, #5)

832 tests passing (5 new).

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* fix(calendar): resolve lint errors and review findings

- Remove unused imports (beforeEach, writeFileSync, result)
- Fix toLocalDate to use local timezone instead of UTC
- Remove dead 'sprint' variant from CalendarEvent.type union

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* style: format global.css with prettier

Also add .claude/worktrees/ to .prettierignore to prevent
worktree artifacts from failing format checks.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
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