Skip to content

refactor sending prompt to agent from UI#1539

Open
adboio wants to merge 1 commit intochore/relative-time-utilsfrom
04-07-refactor_sending_prompt_to_agent_from_ui
Open

refactor sending prompt to agent from UI#1539
adboio wants to merge 1 commit intochore/relative-time-utilsfrom
04-07-refactor_sending_prompt_to_agent_from_ui

Conversation

@adboio
Copy link
Copy Markdown
Contributor

@adboio adboio commented Apr 7, 2026

Problem

trying to add more things that prompt the agent automatically, and more annotations to the review panel, was getting messy

Changes

small refactor:

  • adds sendPromptToAgent util that:
    • takes in a prompt+task
    • puts the review panel back to split view if it's open in full view
    • focuses the main session tab
    • sends the prompt to the agent
  • broke out the prompt builders into a new util reviewPrompts.ts
  • made CommentAnnotationself-contained -- now it handles building prompt and sending to agent -- so we don't have to thread a million things through a million components

i still don't like how task/taskId are getting drilled a million levels deep everywhere, but that happens throughout the whole codebase... i considered a small task context, but it's pretty deeps, i'll revisit that later... for now this just avoids drilling allll the annotation stuff

How did you test this?

manually

Copy link
Copy Markdown
Contributor Author

adboio commented Apr 7, 2026

@adboio adboio force-pushed the 04-07-refactor_sending_prompt_to_agent_from_ui branch from 04dfd84 to 97e2be9 Compare April 7, 2026 20:42
@adboio adboio requested a review from a team April 7, 2026 21:42
@adboio adboio marked this pull request as ready for review April 7, 2026 21:42
@adboio adboio force-pushed the chore/relative-time-utils branch from 3068737 to 906775e Compare April 8, 2026 01:49
@adboio adboio force-pushed the 04-07-refactor_sending_prompt_to_agent_from_ui branch from 97e2be9 to 2d9e0a7 Compare April 8, 2026 01:49
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