perf(ai): guard update_imported_comments_disabled_state with conversation registry

[acarl005]

Description

Part of a stack of performance fixes for #9799.

This PR: update_imported_comments_disabled_state was triggered on every BlocklistAIHistoryEvent for every AI block. Internally it called is_latest_visible_exchange_in_root_task, which scanned all exchanges of the conversation to find the latest visible one — O(n) per event. For a 3000-exchange conversation with frequent events, this added up significantly during restore.

Adds conversations_with_imported_comments: HashSet<AIConversationId> to BlocklistAIHistoryModel, maintained as conversations import/clear comments. The expensive check is now skipped entirely with an O(1) set lookup for the vast majority of conversations that have no imported comments.

Linked Issue

Fixes #9799

• The linked issue is labeled ready-to-spec or ready-to-implement.
• Where appropriate, screenshots or a short video of the implementation are included below (especially for user-visible or UI changes).

Testing

Manually loaded a conversation with 3014 AI exchanges and confirmed reduced CPU during restore.

• I have manually tested my changes locally with ./script/run

Agent Mode

• Warp Agent Mode - This PR was created via Warp's AI Agent Mode

[acarl005]

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

• perf(terminal): bulk_insert_restored_blocks to amortise BlockMetadataReceived #13116
• perf(ai): skip live-only subscriptions for restored AI blocks #13115
• perf(ai): guard update_imported_comments_disabled_state with conversation registry #13114 👈 (View in Graphite)
• perf(terminal): O(1) is_any_ai_block_focused via ancestor set #13113
• perf(ai): O(1) exchange_with_id via exchange_id_index #13111 : 1 other dependent PR (#13112 )
• master

This stack of pull requests is managed by Graphite. Learn more about stacking.

[oz-for-oss]

@acarl005

I'm starting a first review of this pull request.

You can follow along in the session on Warp.

I completed the review and no human review was requested for this pull request.

Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).

Powered by Oz

[oz-for-oss]

Overview

This PR adds a conversation-level imported-comments registry and uses it to avoid updating the Open all imported comments disabled state for conversations that have never received imported comments.

Concerns

• No blocking concerns found.

Verdict

Found: 0 critical, 0 important, 0 suggestions

Approve

Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).

Powered by Oz