Skip to content

fix(chat): align organization focus and loading with workspace - #7647

Merged
waleedlatif1 merged 1 commit into
stagingfrom
codex/org-chat-focus
Sep 9, 2026
Merged

waleedlatif1 merged 1 commit into
stagingfrom
codex/org-chat-focus

Conversation

@waleedlatif1

Copy link
Copy Markdown
Collaborator

Summary

  • Reuse workspace chat prefetch and guarded input focus in organization chats.
  • Reset conversation state on navigation and keep the composer visible when messages are already available.

Type of Change

  • Bug fix

Testing

902 chat and navigation regression tests pass. Manually verified organization home, cached chat switches, collapsed chat navigation, draft isolation, and workspace input focus. Type checking, lint, all 46 audits, block registry, and docs manifest checks pass.

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

@vercel

vercel Bot commented Sep 9, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated
docs Skipped Skipped Sep 9, 2026 5:15am UTC

Request Review

@greptile-apps

greptile-apps Bot commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

RetriggerView in GreptileConfidence Score: 5/5

The PR appears safe to merge, with the behavior changes matching established workspace chat patterns and covered by focused regression tests.

Summary

  • Reuses intent-aware chat-history prefetching for expanded and collapsed organization navigation.
  • Remounts organization conversation state when the organization or routed chat changes.
  • Keeps existing messages and the composer visible while background history loading continues.
  • Extracts guarded composer autofocus into a shared hook and adds regression coverage.

Diagram

%%{init: {'theme': 'neutral'}}%%
flowchart LR
    A[Organization chat link] -->|focus, hover, or pointer intent| B[ChatNavigationLink]
    B --> C[Prefetch chat-detail query]
    A -->|navigate| D[OrganizationHome]
    D --> E[Key by organization and routed chat]
    E --> F[Load cached or remote history]
    F --> G{Messages available?}
    G -->|No, history pending| H[Loading state]
    G -->|Yes| I[Render messages and composer]
    I --> J[Guarded composer autofocus]
Loading

@waleedlatif1
waleedlatif1 merged commit e7faad3 into staging Sep 9, 2026
33 checks passed
@waleedlatif1
waleedlatif1 deleted the codex/org-chat-focus branch September 9, 2026 05:41
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