Skip to content

Port upstream 0.62.0: detail-section visibility via detailSection IDs - #745

Merged
Finesssee merged 1 commit into
mainfrom
port/micro-0.62.0-detail-section-visibility
Oct 3, 2026
Merged

Finesssee merged 1 commit into
mainfrom
port/micro-0.62.0-detail-section-visibility

Conversation

@Finesssee

Copy link
Copy Markdown
Collaborator

Summary

Ports the 0.62.0 gap: provider detail-section visibility via detailSection: IDs (upstream steipete#3638). The usage-item visibility system previously covered only metric: rows; now the Settings > Providers visibility list and the card rendering handle detail sections too.

  • New detailSection: namespace (rust/src/settings.rs USAGE_ITEM_DETAIL_SECTION_PREFIX, mirrored in lib/usageItemVisibility.ts as detailSectionItemId / isDetailSectionVisible).
  • usage_items.rs: one descriptor per distinct display-detail title (redacted when Hide Personal Info is on); hidden detail sections the provider no longer reports come back as available: false placeholders with the title recovered from the stored ID.
  • MenuCardDetails.tsx: the provider-details section filters rows by isDetailSectionVisible, and the duplicated-section artifact is fixed (one gated copy).
  • Detail-section rows hide independently of metric rows; un-hiding restores the row without a new provider detail section.

Validation

  • cargo +1.98.0 fmt --all -- --check clean; cargo +1.98.0 clippy --workspace --all-targets -- -D warnings clean; cargo +1.98.0 test -p codexbar --lib 2160 passed / 0 failed.
  • cargo +1.98.0 test -p codexbar-desktop-tauri: 460 passed / 1 failed — the documented Isolate bootstrap payload test from real settings #684 bootstrap_payload_exposes_every_provider_variant baseline. New test usage_item_descriptors_include_detail_sections covers: distinct-title dedupe, Hide-Personal-Info redaction of titles and stored IDs, and the unavailable placeholder path.
  • Frontend: pnpm test 65 files / 393 passed; pnpm run check/build OK (locale 871 keys).

Upstream reference

steipete/CodexBar 0.62.0 steipete#3638 (ProviderUsageItemVisibility.swift, MenuCardView+ProviderDetailVisibility.swift) — tag-pinned read-only.

@coderabbitai

coderabbitai Bot commented Oct 2, 2026

Copy link
Copy Markdown

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Autopilot is currently an internal CodeRabbit preview.


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.

Finesssee added a commit that referenced this pull request Oct 2, 2026
Finesssee added a commit that referenced this pull request Oct 2, 2026
…e HEAD imports plus the detailSection visibility import
@Finesssee
Finesssee merged commit 9441091 into main Oct 3, 2026
4 checks passed
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