@fresh-machine-37269 repos like openhands, agent-sdk, automation, extensions may have a workflow that comments on a PR if a .pr/ directory exists, and remove it from the PR upon approval or from main upon merge; it should also have, with it, some instructions related to .pr/ in some .md file in the respective repo(s). It’s a convention; understand it. Find the entire convention and open PRs to add it also in the repos where it’s missing, from these four.

🤖 On it! View progress here: oss-agent-canvas.ngrok.dev/conversations/…

@fresh-machine-37269 actually lets see whats missing from those that arent the fullest, so compare those with agent-sdk

✅ Done! Here's the comparison of the two non-fullest repos (

automation

and

OpenHands

) against

software-agent-sdk

, which is the reference. I diffed the live

main

versions of all three workflows. software-agent-sdk (the fullest) — 3 jobs 1.

cleanup-on-approval

— on

pull_request_review

(approved, same-repo) removes

.pr/

with a commit. 2.

check-pr-artifacts

— on

pull_request_target

(not

closed

), inspects the PR head via the API (never checks out untrusted code) and posts/updates the notice comment. It also differentiates fork vs. same-repo in the comment text. 3.

cleanup-after-merge

— on

pull_request_target

(

closed

+

merged

), removes

.pr/

from the base branch after merge. This is what covers fork PRs and same-repo PRs where approval cleanup was missed. automation vs. agent-sdk Missing: • ❌

cleanup-after-merge

job entirely → fork PRs are never auto-cleaned; they still say "manual removal required before merging." • ❌

pull_request_target

trigger → uses plain

pull_request

(and its types are

[opened, synchronize, reopened]

— no

closed

), so fork/after-merge cleanup is impossible. • ❌ Fork-vs-same-repo distinction in the notice comment (always says "removed on approval; fork PRs need manual removal"). • Also uses

actions/checkout@v6

vs.

@v7

(minor). OpenHands vs. agent-sdk Missing: • ❌

cleanup-after-merge

job → fork PRs are never auto-cleaned (comment still says "Fork PRs require manual cleanup before merging"). • ❌

pull_request_target

trigger →

check-pr-artifacts

uses

pull_request

+ checkout instead of API inspection. • ❌ Fork-vs-same-repo distinction in the notice comment. Ahead of agent-sdk (worth keeping): • ✅

cleanup-on-approval

is actually more robust — it resolves the PR context via

gh api

(head_ref/head_sha/`has_pr_dir`), verifies the checked-out SHA matches, and has a rebase-and-retry (×3) loop on push failure. agent-sdk's version is a simpler single-push. Net summary The one meaningful gap common to both is the

cleanup-after-merge

job +

pull_request_target

path — i.e., automatic cleanup of fork PRs from

main

after merge.

automation

is missing exactly that plus the fork-aware comment;

OpenHands

is missing that plus the fork-aware comment, but has a stronger approval-cleanup (SHA verification + retry). Want me to open PRs to bring

automation

and

OpenHands

up to parity (add

cleanup-after-merge

, switch to

pull_request_target

API inspection, fork-aware notice) while preserving OpenHands' more-robust approval path?

Please take a deep breath @fresh-machine-37269 , and open PRs to the respective repos missing something, with the similar stuff from the best.

✅ Done! Done. I compared each of the two non-fullest repos against

software-agent-sdk

(the reference) and opened a PR for each of the two that were missing something. PRs opened 1.

OpenHands/automation

— PR #435 → <https://github.com/OpenHands/automation/pull/435 Added|github.com/OpenHands/automation/pull/435_Added> the missing

cleanup-after-merge

job (fork PRs now auto-clean from

main

after merge) and switched

check-pr-artifacts

to

pull_request_target

API inspection with a fork-aware notice comment. 2.

OpenHands/OpenHands

— PR #17245 → <https://github.com/OpenHands/OpenHands/pull/17245 Same|github.com/OpenHands/OpenHands/pull/17245_Same> gap fixed (

cleanup-after-merge

+ fork-aware notice), but I kept its existing more-robust approval-cleanup path (head-SHA verification + rebase-and-retry ×3), which is actually stronger than agent-sdk's simpler single-push. (

extensions

PR #545 from earlier remains open — it was the repo missing the entire convention.) Why these are the "similar stuff from the best" •

software-agent-sdk

was the only one with a

cleanup-after-merge

job — the piece that auto-removes

.pr/

from

main

after a fork PR merges. Both

automation

and

OpenHands

were missing it, so fork PRs could never be auto-cleaned (their notices still said "manual removal required"). • The other meaningful gap was the

pull_request_target

+ API inspection approach, which avoids checking out untrusted fork code with a write token and lets the notice distinguish fork vs. same-repo. One judgment call worth flagging: for

OpenHands

I did not blindly copy agent-sdk, because doing so would have replaced its stronger approval-cleanup logic with a weaker one. The OpenHands PR only adds what was missing and preserves the existing robustness.