Skip to content

feat(tools): add interactive tool and call explorer - #21

Merged
hamidi-dev merged 1 commit into
mainfrom
feat/tools-explorer
Sep 12, 2026
Merged

hamidi-dev merged 1 commit into
mainfrom
feat/tools-explorer

Conversation

@hamidi-dev

@hamidi-dev hamidi-dev commented Sep 12, 2026

Copy link
Copy Markdown
Owner

Explore tool usage in the TUI and live web report.

  • Drill into tools and MCP servers for cost, token, and model breakdowns.
  • Browse individual calls and jump to their owning turns.
  • Fix TUI scrolling and web tabs stuck loading.
  • Preserve privacy boundaries and clearly label attributed usage.

Tested: 1,884 tests passed, lint/format checks passed, and desktop/mobile browser verification completed.

Summary by CodeRabbit

  • New Features

    • Added a drillable Tools explorer to session views in the terminal and web interfaces.
    • View tool and namespace rankings, contribution totals, token composition, model attribution, and chronological calls.
    • Navigate from individual calls to their owning turns and prompts, with keyboard and mouse controls.
    • Tool treemap items are now clickable, with navigation history and preserved focus when changing cost modes.
    • Added tool-call data to session details and supported CSV exports.
  • Documentation

    • Documented Tools explorer controls, navigation, data coverage, and limitations.

@coderabbitai

coderabbitai Bot commented Sep 12, 2026

Copy link
Copy Markdown

Review Change StackReview Change Stack

No actionable comments were generated in the recent review. πŸŽ‰

ℹ️ Recent review info
βš™οΈ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Advanced

Run ID: 1d331ff4-cf89-4e3c-86bf-7d7d6319deb9

πŸ“₯ Commits

Reviewing files that changed from the base of the PR and between 25b73c1 and 4a119a1.

πŸ“’ Files selected for processing (13)
  • docs/architecture.md
  • docs/keys.md
  • docs/tui.md
  • docs/web.md
  • src/opentab/tools.py
  • src/opentab/tui/app.py
  • src/opentab/tui/keymap.py
  • src/opentab/tui/renderer.py
  • src/opentab/web.py
  • src/opentab/webpage.py
  • tests/test_tools.py
  • tests/test_tui_detail.py
  • tests/test_web.py

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


πŸ“ Walkthrough

Walkthrough

Changes

Tools explorer

Layer / File(s) Summary
Tool-call projection and session payload
src/opentab/tools.py, src/opentab/web.py, tests/test_tools.py, tests/test_web.py
Adds ordered per-call projections with proportional usage fields. Web session extras now include allowlisted tool rows and call records.
TUI projections, rendering, and navigation
src/opentab/tui/app.py, src/opentab/tui/renderer.py, src/opentab/tui/keymap.py, tests/test_tui_detail.py, docs/architecture.md, docs/keys.md, docs/tui.md
Adds cached tool rankings, drill-down views, call ledgers, pricing updates, cursor handling, and navigation to owning turns.
Web explorer and history navigation
src/opentab/webpage.py, tests/test_web.py, docs/web.md
Adds clickable tool rankings, per-tool details, call navigation, history restoration, stale-request handling, and responsive styling.

Priority: βž– Normal

Estimated code review effort: 4 (Complex) | ~60 minutes

Change: Feature

Merge Risk: βšͺ Minimal Β· up to 4a119

No merge-blocking behavior or operational risk is currently established.

πŸš₯ Pre-merge checks | βœ… 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 2.02% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 99 functions across 9 files. (4 skipped: 4… Write docstrings for the functions missing them to satisfy the coverage threshold.
βœ… Passed checks (4 passed)
Check name Status Explanation
Description Check βœ… Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check βœ… Passed The title clearly and concisely describes the main change: adding an interactive tool and call explorer to the TUI and live web report.
Linked Issues check βœ… Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check βœ… Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

Docstring coverage is 2.02% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 99 functions across 9 files. (4 skipped: 4 unsupported.)

  • Fix all pre-merge checks with AI
✨ Finishing Touches πŸ’‘ 1
πŸ“ Generate docstrings πŸ’‘
  • Create stacked PR
  • Commit on current branch
πŸ§ͺ Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/tools-explorer

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❀️ Share

Comment @coderabbitai help to get the list of available commands.

@hamidi-dev
hamidi-dev merged commit 6fa1bd6 into main Sep 12, 2026
5 checks passed
@hamidi-dev
hamidi-dev deleted the feat/tools-explorer branch September 12, 2026 18:08
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