feat(log-detail): conditionally show balance text

[smakosh]

Summary

Updated provider cost messaging to only display "not deducted from your balance" text when using API keys mode. This clarifies that provider costs may be handled differently depending on the authentication method used.

Changes

• Modified tooltip and label text in log-detail-client.tsx to conditionally append the balance deduction disclaimer based on log.usedMode === "api-keys"
• Applied the same conditional logic to log-card.tsx for consistency across the dashboard
• Updated 4 instances of provider cost/pricing messaging across both components

Implementation Details

The changes use inline conditional rendering to append the " — not deducted from your balance" text only when the log's usedMode is "api-keys". This ensures users see accurate information about cost handling based on their authentication method, while avoiding misleading messaging for other modes.

https://claude.ai/code/session_01EiTZcynDaoX8qKXCVyuVU9

Summary by CodeRabbit

• Bug Fixes
  • Clarified billing information: Cost-related tooltips and descriptions now explicitly indicate when provider charges are not deducted from your account balance.

[coderabbitai]

Walkthrough

Two dashboard UI files updated to conditionally append " — not deducted from your balance" to Provider cost and Provider pricing tooltip texts when log.usedMode is "api-keys", indicating that API key-based logs reference provider pricing without affecting user balance.

Changes

Cohort / File(s)	Summary
Provider Pricing Tooltip Updates apps/ui/src/app/dashboard/[orgId]/[projectId]/activity/[logId]/log-detail-client.tsx, apps/ui/src/components/dashboard/log-card.tsx	Conditionally append " — not deducted from your balance" to "Provider cost" and "Provider pricing" tooltip texts when log.usedMode === "api-keys".

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• feat: enhance dashboard, log detail & costs UX #1623: Modifies the same two files to add provider pricing balance deduction clarification text in UI tooltips, directly related to this change.

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the main change: conditionally showing balance deduction text based on API key mode in the log-detail component.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch claude/conditional-note-api-keys-Vlaua

---

No actionable comments were generated in the recent review. 🎉

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}