Skip to content

Harden PR Description Updater against one-shot safe-output exhaustion - #49463

Merged
pelikhan merged 2 commits into
mainfrom
copilot/fix-pr-description-updater-issue
Aug 1, 2026
Merged

Harden PR Description Updater against one-shot safe-output exhaustion#49463
pelikhan merged 2 commits into
mainfrom
copilot/fix-pr-description-updater-issue

Conversation

Copilot AI commented Aug 1, 2026

Copy link
Copy Markdown
Contributor

The PR Description Updater could consume its single allowed update_pull_request call with probe/placeholder content, then fail to apply the real synthesized description due to max: 1. This change tightens the workflow prompt so the one-shot update is reserved for the final body only.

  • Prompt contract for one-shot PR updates

    • Step 4 now requires exactly one update_pull_request call, using only the final synthesized markdown body.
    • Explicitly forbids probe/test/placeholder updates (for example "-").
  • Failure-path behavior

    • If a final complete body is not available, the workflow now instructs the agent to emit report_incomplete instead of calling update_pull_request.
    • Clarifies no retry behavior for update_pull_request within the same run.
  • Compiled workflow sync

    • Regenerated pr-description-caveman.lock.yml to match updated markdown workflow source.
### Step 4 — Update the PR

Call `update_pull_request` exactly once with the final synthesised markdown body.

Do NOT send probe/test/placeholder content (for example `"-"`) ...
If you cannot produce the final complete body, call `report_incomplete` and do not call `update_pull_request`.

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix PR Description Updater reported incomplete result Harden PR Description Updater against one-shot safe-output exhaustion Aug 1, 2026
Copilot AI requested a review from pelikhan August 1, 2026 04:47
@pelikhan
pelikhan marked this pull request as ready for review August 1, 2026 05:31
Copilot AI review requested due to automatic review settings August 1, 2026 05:31
@pelikhan
pelikhan merged commit d1adb68 into main Aug 1, 2026
@pelikhan
pelikhan deleted the copilot/fix-pr-description-updater-issue branch August 1, 2026 05:31

Copilot AI left a comment

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.

Pull request overview

Hardens the PR Description Updater so its one permitted PR update is reserved for the final synthesized description.

Changes:

  • Prohibits probe, placeholder, and retry updates.
  • Uses report_incomplete when no complete body is available.
  • Synchronizes the compiled workflow metadata.
Show a summary per file
File Description
.github/workflows/pr-description-caveman.md Strengthens the one-shot update contract.
.github/workflows/pr-description-caveman.lock.yml Updates the generated body hash.

Review details

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

  • Files reviewed: 2/2 changed files
  • Comments generated: 0
  • Review effort level: Balanced

@github-actions

github-actions Bot commented Aug 1, 2026

Copy link
Copy Markdown
Contributor

🎉 This pull request is included in a new release.

Release: v0.84.2

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.

[aw] PR Description Updater reported incomplete result

3 participants