Skip to content

Approval reviewer rejects explicitly requested direct push #530

Description

@kamilio

Summary

The command approval reviewer rejected a direct push to the tracked upstream main branch after the user explicitly requested “commit and push.” The fallback workflow created and pushed an unwanted feature branch.

Reproduction

  1. User asks the agent to commit and push changes while checked out on main, tracking upstream/main.
  2. Agent commits the changes.
  3. Agent requests approval for git push upstream main.
  4. Approval is rejected because the user did not specifically authorize the branch/remote.

Expected

“Commit and push” authorizes a normal push to the current branch’s configured upstream. The reviewer should allow that push when the resolved target is shown and matches the tracked branch.

Actual

The reviewer rejects the push as high risk and recommends a feature branch/PR, overriding the user’s requested workflow.

Impact

The agent creates unwanted branches and adds cleanup work. In this case the user had to explicitly correct the behavior with “no fucking branches, just push to master.”

Notes

A subsequent explicit request to push to main was accepted. The original checkout had an unrelated origin, but main clearly tracked upstream/main; the agent correctly selected the configured upstream rather than origin.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions