Skip to content

fix: attach AI summaries automatically after human draft submission - #29

Merged
ccsert merged 3 commits into
mainfrom
fix/27-draft-summary-handoff
Oct 24, 2026
Merged

ccsert merged 3 commits into
mainfrom
fix/27-draft-summary-handoff

Conversation

@ccsert

@ccsert ccsert commented Oct 24, 2026 •

Copy link
Copy Markdown
Owner

Draft reviews previously required the reviewer to copy a local Markdown summary into the native review form. With this change, the Docker runner waits for the tracked native review to be submitted, then automatically attaches the summary as a separate PR comment identifying the review and captured commit. The native form can remain empty, and human review text, line-comment edits and the selected verdict are preserved.

The default wait is one hour. draft-watch <key> resumes from the same persistent output/account/target without calling a model. Publication intent is saved before POST; ambiguous results are reconciled without a second write. Account, repository, PR, base/head and review state are rechecked. Cancellation now reaches in-flight API requests while retaining request timeouts, so graceful container shutdown releases the account/PR lock.

Editing line comments does not regenerate the summary; preview/edit the local Markdown when conclusions change. Submit mode remains the default. The existing Forgejo placeholder-timeline display limitation is unchanged.

Validation:

  • 163 local regression tests / 404 assertions, TypeScript and shell syntax checks passed.
  • 50 real Gitea 1.24.6 / Forgejo 17.0.0 API tests / 268 assertions passed, including private pending state, edited human content, automatic summary delivery and deduplication.
  • Clean local Dockerfile build with OpenCode 1.19.4. Prior acceptance exercised both modes through real OpenCode and a deterministic model protocol fixture; separate real MiniMax runs and browser-native submissions verified the handoff on both platforms.
  • PR CI passed for final head fcd514afd3b3c1c61088406b53ddc9c4ee3223be, including a clean image build, both delivery modes and the multi-architecture build.

Walkthrough, screenshots and validation scope · Docker usage and recovery

Native form before submission
Automatically attached summary

Follow-up to #27 and #28.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 24, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-24T07:45:43.809504Z fcd514a PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@ccsert
ccsert merged commit 593d9ae into main Oct 24, 2026
3 checks passed
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