You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "potiuk (via GitHub)" <gi...@apache.org> on 2026/04/27 02:01:22 UTC

[PR] PR-triage skill: promote likely-addressed PRs with reviewer ping [airflow]

potiuk opened a new pull request, #65916:
URL: https://github.com/apache/airflow/pull/65916

   Adds a new suggested action `mark-ready-with-ping` for the case
   that today routes to plain `ping`: a PR's only outstanding
   signal is unresolved review threads from collaborators. When a
   cheap batch-query heuristic shows the author has likely
   addressed each thread (in-thread reply or post-comment commit,
   plus the latest commit post-dates the most recent unresolved
   thread), the skill proposes promoting the PR to
   `ready for maintainer review` and posting a single ping comment
   that invites the original reviewer(s) to confirm and resolve
   their threads.
   
   The heuristic is conservative: false-negatives degrade to the
   existing plain `ping` behaviour; false-positives self-correct
   on the next round-trip because the comment explicitly invites
   the reviewer to push back. The new action shares the same
   mandatory pre-mutation guard as `mark-ready` (refuse if any
   workflow run is in `action_required` for the PR's head SHA).
   
   Files touched:
   - `fetch-and-batch.md`: widen `reviewThreads.nodes.comments`
     from `first: 1` to `first: 5` so the heuristic can see
     in-thread author replies without per-PR follow-up queries.
   - `classify.md`: document the
     `unresolved_threads_only_likely_addressed` sub-flag; update
     the required-fields checklist.
   - `suggested-actions.md`: split the unresolved-threads-only
     rule into two; new "Choosing between ping and
     mark-ready-with-ping" section.
   - `actions.md`: new `mark-ready-with-ping` recipe; extend the
     order-of-operations recap.
   - `comment-templates.md`: new `mark-ready-with-ping` body;
     added to the AI-attribution-footer always-include list.
   - `SKILL.md`: extend Golden rule 1b so the workflow-approval
     guard explicitly covers `mark-ready-with-ping`; add to the
     Step 3 default-action table and Step 4 group ordering.
   - `interaction-loop.md`: insert the new group ahead of plain
     `mark-ready` so all label-add proposals run back-to-back;
     add safe override (`mark-ready-with-ping → ping`); extend
     the session summary.
   
   ##### Was generative AI tooling used to co-author this PR?
   
   - [X] Yes — Claude Opus 4.7 (1M context)
   
   Generated-by: Claude Opus 4.7 (1M context) following [the guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] PR-triage skill: promote likely-addressed PRs with reviewer ping [airflow]

Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk merged PR #65916:
URL: https://github.com/apache/airflow/pull/65916


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org