Skip to content

Harden against over-eager PR action - #28

Merged
scode merged 1 commit into
mainfrom
no-eager-pr-ops
Feb 26, 2026
Merged

scode merged 1 commit into
mainfrom
no-eager-pr-ops

Conversation

@scode

@scode scode commented Feb 26, 2026 •

Copy link
Copy Markdown
Owner

Hopefully improving upon (a) agents over eagerly updating a PR even when not asked, and (b) updating the current PR instead of creating a new one even when told to "make a PR".

scode commented Feb 26, 2026 •

Copy link
Copy Markdown
Owner Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@scode scode changed the title Require per-message explicit consent for PR operations Harden against over-eager PR action Feb 26, 2026

scode commented Feb 26, 2026 •

Copy link
Copy Markdown
Owner Author

Merge activity

  • Feb 26, 4:35 AM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Feb 26, 4:36 AM UTC: Graphite rebased this pull request as part of a merge.
  • Feb 26, 4:37 AM UTC: @scode merged this pull request with Graphite.

@scode
scode changed the base branch from sandbox-instructions to graphite-base/28 February 26, 2026 04:35
@scode
scode changed the base branch from graphite-base/28 to main February 26, 2026 04:35
Agents were eagerly updating PRs after code changes if a prior message
had requested a PR create/update. Rewrite rule 6 to make clear that each
gt create/modify/submit requires a fresh explicit request in the current
message — prior requests do not carry forward.
@scode
scode merged commit 4372d22 into main Feb 26, 2026
2 checks passed
@scode
scode deleted the no-eager-pr-ops branch February 26, 2026 04:37
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.

1 participant