Skip to content

Add collapsible activity-sorted thread tree#85

Merged
adamfgr merged 1 commit into
mainfrom
codex/qol-tree-collapse
Jul 6, 2026
Merged

Add collapsible activity-sorted thread tree#85
adamfgr merged 1 commit into
mainfrom
codex/qol-tree-collapse

Conversation

@wizzoapp

@wizzoapp wizzoapp Bot commented Jul 6, 2026

Copy link
Copy Markdown

Summary

  • add persisted chevron expansion for parent thread groups in the sidebar; chevron toggles only the group while parent row click still opens the thread
  • sort active threads first, then by sidebar thread activity order; parent groups float by their most active descendant and children use the same ordering
  • cover sort behavior and expansion-state persistence in web tests

Verification

  • vp run --filter @t3tools/web test -- src/components/Sidebar.logic.test.ts src/uiStateStore.test.ts
  • vp run --filter @t3tools/web build
  • vp run typecheck
  • vp check (passes; existing unrelated mobile unused-code warnings remain)
  • factory pre-commit gate passed with clean Codex autoreview

Visual verification

  • T3 preview could not attach: preview_open returned PreviewAutomationNoAvailableHostError. Reported as ADA-81.

Goes live on the next web redeploy after merge.

@wizzoapp wizzoapp Bot marked this pull request as ready for review July 6, 2026 11:01

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 422986f182

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread apps/web/src/components/Sidebar.tsx
@wizzoapp wizzoapp Bot marked this pull request as draft July 6, 2026 11:29
@wizzoapp wizzoapp Bot force-pushed the codex/qol-tree-collapse branch from 422986f to 578bffe Compare July 6, 2026 11:52
@wizzoapp wizzoapp Bot marked this pull request as ready for review July 6, 2026 11:53

@adamfgr adamfgr left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Auto-approved by wizzo-approve: all policy conditions verified against HEAD 578bffe.

@adamfgr adamfgr merged commit 4c60ad2 into main Jul 6, 2026
13 checks passed
@adamfgr adamfgr deleted the codex/qol-tree-collapse branch July 6, 2026 12:11
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