Skip to content

[no-ci] Clarify policy for explicitly requested upstream pushes - #2645

Merged
isVoid merged 1 commit into
NVIDIA:mainfrom
rwgk:AGENTS_md_upstream_policy
Aug 17, 2026
Merged

[no-ci] Clarify policy for explicitly requested upstream pushes#2645
isVoid merged 1 commit into
NVIDIA:mainfrom
rwgk:AGENTS_md_upstream_policy

Conversation

@rwgk

@rwgk rwgk commented Aug 16, 2026

Copy link
Copy Markdown
Contributor

Description

Follow-up to #2378.

The current policy makes the canonical upstream repository unconditionally read-only. That remains the right default for normal pull-request work, but the absolute wording creates significant friction for a small set of valid, user-directed workflows. In practice, agents have repeatedly asked me to perform a push manually even when I explicitly requested it and upstream was needed to test .github/workflows, trigger CI from a designated ref, or support similar infrastructure work.

This change keeps approved forks as the normal destination and permits an upstream push only when the user explicitly requests it and provides a rationale. It also requires verification of the complete repository and exact refspec, limits authorization to the requested ref update, and requires separate explicit authorization for higher-risk operations.

This is not intended to give agents more autonomy. The agent does not infer when an upstream push is appropriate: the user supplies the request and rationale. The instructions only allow the agent to execute that narrowly scoped request instead of requiring a manual handoff.

@rwgk rwgk added this to the cuda.bindings 13.4.0 & 12.9.8 milestone Aug 16, 2026
@rwgk rwgk added documentation Improvements or additions to documentation cuda.bindings Everything related to the cuda.bindings module labels Aug 16, 2026
@copy-pr-bot

copy-pr-bot Bot commented Aug 16, 2026

Copy link
Copy Markdown
Contributor

Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

@rwgk rwgk self-assigned this Aug 16, 2026
@rwgk rwgk added the P1 Medium priority - Should do label Aug 16, 2026
@rwgk
rwgk marked this pull request as ready for review August 16, 2026 06:16
@rwgk
rwgk requested a review from mdboom August 16, 2026 06:16
@isVoid

isVoid commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

Thanks. I also ran into CI update frictions when working with agent recently. This should help.

@isVoid
isVoid merged commit 749e104 into NVIDIA:main Aug 17, 2026
35 checks passed
@github-actions

Copy link
Copy Markdown
Doc Preview CI
Preview removed because the pull request was closed or merged.

@rwgk
rwgk deleted the AGENTS_md_upstream_policy branch August 18, 2026 23:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cuda.bindings Everything related to the cuda.bindings module documentation Improvements or additions to documentation P1 Medium priority - Should do

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants