Skip to content

fix(web): add pull request tasks to the current chat - #83

Draft
wSedlacek wants to merge 6 commits into
mainfrom
t3/fix-check-in-current-chat
Draft

wSedlacek wants to merge 6 commits into
mainfrom
t3/fix-check-in-current-chat

Conversation

@wSedlacek

Copy link
Copy Markdown

Reopened from closed upstream PR pingdotgg#30654 as a draft for t3.1code.
Idea rank: 82 of 108. Branch is as it was upstream; it likely needs a rebase (modernize) before review.

problem

the chat actions panel opened a new thread for fix instead of filling the current composer, and what those tasks added was a wall of plain text.

change

pass the current composer target through the panel and share the existing pr composer handoff. fix and resolve preserve draft text and stay in the current thread. disable these actions while editing a queued message, and require checking the pr branch before making changes. threads with no composer, such as subagent threads, still open a new thread.

fix and resolve now attach everything as chips: the pr, each failing check, each review remark and each review thread. the prompt keeps only the instruction. review chips open their full text on hover, in the composer and on sent messages. the text is still bounded to 1,000 characters.

verification

reproduced on github/main, then exercised the earlier pr head and corrections in linux chrome through the dev web client and real codex gpt-6.1-sol turns. verified repeated fix clicks, draft preservation, queued edit save/cancel, queued turn execution, provider receipt of the fix task, and resolve handoff.

for the chip and hover changes, ran the dev web client on macos with the free replay provider and read-only public prs pingdotgg#31925 (failing check) and pingdotgg#31722 (conflicts). verified fix and resolve from the actions panel keep the thread and draft and attach chips, and hovering a check chip opens its text. vp run check:changed passed format, lint, knip, typechecks, tests and react compiler checks.

native desktop/mobile, other providers, and actual remote fixes or conflict resolution were not exercised. no chip-based task was sent to a real provider. existing pull requests supplied read-only check/conflict data; none were modified.

before

main: fix opens a new thread

after

fix keeps the thread and draft and attaches the pr and check as chips

hovering a check chip shows its text

resolve stays in the thread with a pr chip

gpt-6.1-sol through the codex harness in t3 code, then claude opus 5.5 through claude code in t3 code.

🤖 Generated with Claude Code

🤖 Generated with Claude Code

wSedlacek and others added 6 commits November 9, 2026 10:07
Handoffs written to an existing composer skip checkout preparation, so
prompts now ask the agent to verify and switch branches instead of
claiming a prepared checkout. The queued-edit reason on Fix and Resolve
is reachable by hover and focus, and threads without a composer fall
back to opening a new thread.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Failing checks, review remarks and conflict tasks now arrive as chips with a
short instruction, and review chips show their full text on hover.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@github-actions github-actions Bot added vouch:trusted Author is trusted by repo permissions or the VOUCHED list. size:L labels Nov 14, 2026

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:L vouch:trusted Author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants