Skip to content

ref(explorer): update hook to use useMutation#113146

Merged
aliu39 merged 29 commits intomasterfrom
aliu/timeout-ui
Apr 17, 2026
Merged

ref(explorer): update hook to use useMutation#113146
aliu39 merged 29 commits intomasterfrom
aliu/timeout-ui

Conversation

@aliu39
Copy link
Copy Markdown
Member

@aliu39 aliu39 commented Apr 16, 2026

Refactor useSeerExplorer to use useMutation for the 4 POST requests it can make. The previous style of api.requestPromise is deprecated and complicates on-success/error handling, while this is built into useMutation.

Temporarily removes the wasInterrupted state which showed a placeholder msg after interrupt requests. This state is hard to manage correctly and a low prio feature, especially since the current system for interrupts often fails. We can bring it back after reworking interrupts on the backend

@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Apr 16, 2026
@aliu39 aliu39 changed the title Aliu/usemutate ref(explorer): update hook to use useMutation Apr 16, 2026
@aliu39 aliu39 marked this pull request as ready for review April 16, 2026 05:25
@aliu39 aliu39 requested a review from a team as a code owner April 16, 2026 05:25
Comment thread static/app/views/seerExplorer/hooks/useSeerExplorer.tsx Outdated
Comment thread static/app/views/seerExplorer/hooks/useSeerExplorer.tsx Outdated
Comment thread static/app/views/seerExplorer/hooks/useSeerExplorer.tsx
Comment thread static/app/views/seerExplorer/hooks/useSeerExplorer.tsx
Comment thread static/app/views/seerExplorer/explorerPanel.tsx
Comment thread static/app/views/seerExplorer/hooks/useSeerExplorer.tsx Outdated
Comment thread static/app/views/seerExplorer/hooks/useSeerExplorer.tsx
Comment thread static/app/views/seerExplorer/hooks/useSeerExplorer.tsx
Comment thread static/app/views/seerExplorer/hooks/useSeerExplorer.tsx
Copy link
Copy Markdown
Member

@shruthilayaj shruthilayaj left a comment

Choose a reason for hiding this comment

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

Image

Just a note, when we interrupt chat, could we remove this "thinking" block ("profiling your project" in this case)

@aliu39
Copy link
Copy Markdown
Member Author

aliu39 commented Apr 17, 2026

Image Just a note, when we interrupt chat, could we remove this "thinking" block ("profiling your project" in this case)

the clearOptimistic useEffect ("on partial load") should handle this once the interrupt succeeds. Unfortunately once we start processing the current system isn't able to interrupt before at least one assistant block is loaded

Comment thread static/app/views/seerExplorer/hooks/useSeerExplorer.tsx
Copy link
Copy Markdown
Contributor

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 6ed04c8. Configure here.

onChange={onInputChange}
onKeyDown={onKeyDown}
onClick={onInputClick}
placeholder={getPlaceholder()}
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Stale interrupted prop on styled component now unused

Low Severity

StyledInputGroup still declares an interrupted prop in its type (<{interrupted?: boolean}>) and uses it to conditionally style placeholder text color (p.interrupted ? p.theme.tokens.content.warning : undefined), but this prop is no longer passed from any call site after removing wasJustInterrupted. This is dead styling logic that will never activate.

Additional Locations (1)
Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 6ed04c8. Configure here.

@aliu39 aliu39 enabled auto-merge (squash) April 17, 2026 18:37
@aliu39 aliu39 merged commit 1df395e into master Apr 17, 2026
65 checks passed
@aliu39 aliu39 deleted the aliu/timeout-ui branch April 17, 2026 18:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants