Skip to content

Make PR Description Updater use triggering PR context for update_pull_request - #53301

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

Make PR Description Updater use triggering PR context for update_pull_request#53301
pelikhan merged 2 commits into
mainfrom
copilot/aw-fix-pr-description-updater

Conversation

Copilot AI commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

The PR Description Updater workflow reported structured incompletion because its single allowed update_pull_request call was consumed during recovery from argument/target mismatch, before final content submission. The workflow config required explicit PR targeting while the prompt instructed identifier-less calls.

  • Root cause

    • safe-outputs.update-pull-request was configured with target: '*', which requires pull_request_number.
    • Prompt guidance directed the agent to avoid passing PR identifiers, creating a conflicting contract and retry path that exhausted max: 1.
  • Change made

    • Removed wildcard targeting from .github/workflows/pr-description-caveman.md so update_pull_request resolves to the triggering PR context.
    • Recompiled the workflow, updating .github/workflows/pr-description-caveman.lock.yml to reflect the new safe-outputs contract.
  • Behavioral impact

    • update_pull_request no longer requires explicit pull_request_number in this workflow.
    • The final synthesized description can be submitted in the single allowed call, avoiding quota loss to corrective retries.
safe-outputs:
  update-pull-request:
    body: true
    title: false
    operation: replace
    max: 1

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix PR Description Updater to handle call limits Make PR Description Updater use triggering PR context for update_pull_request Aug 17, 2026
Copilot AI requested a review from pelikhan August 17, 2026 05:31
@github-actions

Copy link
Copy Markdown
Contributor

PR Triage

Category: bug | Risk: low | Score: 30/100

  • Impact: 12/50 | Urgency: 8/30 | Quality: 10/20

Recommended action: batch_review (grouped with #53302, #53300 as small workflow/agent fixes)

Generated by 🔧 PR Triage Agent · auto · 43 AIC · ⌖ 2.45 AIC · ⊞ 8.1K ·

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

Aligns the PR Description Updater’s safe-output configuration with its triggering pull-request context.

Changes:

  • Removes wildcard PR targeting so identifiers are inferred from the event.
  • Regenerates the compiled workflow configuration.
Show a summary per file
File Description
.github/workflows/pr-description-caveman.md Removes target: '*'.
.github/workflows/pr-description-caveman.lock.yml Updates generated metadata and safe-output configuration.

Review details

💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.

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

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[aw] PR Description Updater reported incomplete result

3 participants