- }
- />
- Archive
-
- )
+ }
+ />
+ Archive
+
+ ) : null}
+
) : null}
diff --git a/rev/BR-t3code-show-provider-icon-on-hover-cy-review.md b/rev/BR-t3code-show-provider-icon-on-hover-cy-review.md
new file mode 100644
index 00000000000..766c5a61ba9
--- /dev/null
+++ b/rev/BR-t3code-show-provider-icon-on-hover-cy-review.md
@@ -0,0 +1,42 @@
+# Cy-review: provider icon on sidebar thread hover
+
+- Target: `t3code/show-provider-icon-on-hover`
+- Base: `dev`
+- Date: 2026-07-19
+- Diff base: uncommitted working-tree diff from `HEAD`
+- Round: 1
+
+## Review fleet
+
+Two reviewers were sufficient for this narrow, single-file presentation change:
+
+- Skeptical Code Reviewer: correctness, state gaps, running-thread regressions, and missing-provider behavior.
+- Accessibility and Design/Reuse Reviewer: keyboard access, tooltip semantics, responsive behavior, and reuse of existing provider abstractions.
+
+Both reviewers were instructed to use GPT-5.6 Sol at medium reasoning effort.
+
+## Summary
+
+- Raw findings: 4
+- Deduplicated findings: 3
+- Fix now: 2
+- Deferred: 1
+- Discarded: 0
+
+## Combined findings
+
+| ID | File:line | Sources | Severity | Disposition | Rationale |
+| ----- | ----------------------------------------- | ------------- | -------- | ----------- | -------------------------------------------------------------------------------------------------------------------- |
+| CR-01 | `apps/web/src/components/Sidebar.tsx:484` | SC-001, AR-01 | Medium | Fix now | Running rows without a resolved provider could fade their metadata while rendering no hover replacement. |
+| CR-02 | `apps/web/src/components/Sidebar.tsx:814` | AR-02 | Medium | Fix now | The provider/model tooltip trigger was not focusable, leaving the exact model unavailable to keyboard users. |
+| CR-03 | `apps/web/src/components/Sidebar.tsx:409` | SC-002 | Medium | Defer | Deleted or unavailable historical provider instances have no trustworthy driver kind for branded fallback rendering. |
+
+## Deferred candidates
+
+| ID | Scope | Reason |
+| ----- | ------ | -------------------------------------------------------------------------------------------------------------------------------------------------------------------- |
+| CR-03 | Medium | A durable fallback requires choosing whether to persist provider presentation metadata, infer it from instance ids, or show a generic historical-provider indicator. |
+
+## Discarded summary
+
+No findings were discarded.
diff --git a/rev/check/BR-t3code-show-provider-icon-on-hover-cy-review-deferred.md b/rev/check/BR-t3code-show-provider-icon-on-hover-cy-review-deferred.md
new file mode 100644
index 00000000000..87c78dca009
--- /dev/null
+++ b/rev/check/BR-t3code-show-provider-icon-on-hover-cy-review-deferred.md
@@ -0,0 +1,7 @@
+# Deferred items: provider icon on sidebar thread hover
+
+## CR-03: Historical provider fallback
+
+If a thread's provider instance is removed or its environment configuration is unavailable, the thread still retains the instance id and model but no longer has a trustworthy driver kind or display metadata for a branded icon. This means the hover affordance can be absent for those historical or disconnected rows. The current patch does not infer branding from an arbitrary instance id because custom ids do not reliably encode their driver and a generic fallback is a product choice. Follow up by deciding whether thread shells should persist provider presentation metadata or whether the sidebar should show a generic historical-provider indicator.
+
+> cy-review complete — 2026-07-19T13:31:11+02:00 — rounds: 1