网易首页 > 网易号 > 正文 申请入驻

你让Copilot写的PR,GitHub不认你的批准

0
分享至

你给Copilot云端代理指派了一个issue。几分钟后,一个pull request出现了。你读完diff,看着没问题,点了Approve。但到了合并规则这一步,这一下点击被无视了。


GitHub文档写得很直白:如果仓库要求pull request审批,你对Copilot提交的PR的批准,不会计入所需数量。必须另有评审人点头,它才能合进去。CI也在等——默认情况下,Copilot向PR推送改动时,GitHub Actions工作流不会自动运行,得有人先点“Approve and run workflows”。

现在把同样的活搬到你的笔记本上。你在终端里给Claude Code或Codex下指令,读完diff,敲下git merge。提出改动的人、评审改动的人、发布改动的人,全是同一个。没有任何环节数到了二。

GitHub这条规则是对的,而且不该止步于github.com。职责分离必须跟着代理工作一起,落到你的笔记本上。

GitHub对代理PR做了什么

GitHub为云端代理列出的风险与缓解措施,读起来像一份谨慎团队会提的要求清单:

  • 只有对仓库有写权限的用户才能触发代理,无写权限用户的评论永远不会传给它。
  • 新任务会拿到一个全新的copilot/分支,代理只能往这个分支推。
  • 它“不能把自己的pull request标记为Ready for review,也不能批准或合并pull request”。
  • 它“受工作仓库的任何分支保护和必需检查约束”。
  • 工作流会等待“有仓库写权限的用户点击Approve and run workflows按钮”。
  • 页面把“GitHub阻止要求Copilot云端代理创建pull request的用户批准它”列为一项缓解措施。

最后一条是有意思的选择。Copilot是提交的作者,提出改动的开发者被标记为共同作者。GitHub本可以把你当成机器人工作的外部评审人,但它没有,它把你放到了作者那一侧。

这和GitHub对真人早就有的规则一致:pull request作者不能批准自己的PR。同一页面还写明,如果指派Copilot的人是你,你就不能批准它创建的PR。写提示词的人,算作者。

工作流那一半有个退出选项。自2026年3月13日的更新日志起,仓库管理员可以关掉针对代理的“工作流运行需审批”。设置页在旁边放了警告:这么做“可能让未经评审的Copilot代码”获得仓库写权限,或触达你的Actions密钥。审批默认是开着的。

GitHub想在代理的PR上装第二把钥匙。只拿提示词作者那把,开不了门。

在笔记本上,你一个人演三个角色

NIST的术语表把职责分离浓缩成一句:任何用户都不应拥有足以独自滥用系统的权限。它举的例子是双人规则。GitHub把它用在了代理PR上,而本地的代理会话没有对应物。

在终端里跑编码代理,它的提交落在你的git身份下。它甚至可能就是GitHub的代理:Copilot CLI“可以代表你执行任务,比如执行或修改文件、运行shell命令”。同一家厂商,同一类工作。一旦这些提交算你的,而你在本地合并了它们,GitHub上没有任何规则知道有代理写过它们。

不过也得对笔记本公平一点。如果main分支受保护且要求评审,直接推送会被拒绝:开启必需评审后,“协作者只能通过pull request向受保护分支推送改动”。队友仍然得批准。缺口开在三个地方:

  • 本地合并。把agent/fix-login合进你部署用的分支,或合进一个之后打包成一个大PR、代理那部分被草草扫过的分支。
  • 管理员。“默认情况下,分支保护规则的限制不适用于拥有仓库管理员权限的人。”小团队里,跑代理的人往往就是管理员。
  • 看起来像评审的评审。你确实仔细读了代理的diff。可按GitHub对Copilot PR的规则,你的阅读不算数,因为活是你要求的。笔记本却把它当成唯一会有的那次评审。

工作流那道闸在本地也没有孪生兄弟。在GitHub上,默认情况下代理对.github/workflows/的改动要等人批准才会运行,文档提醒你对这类改动要“格外警惕”。本地呢,代理改过的测试命令,下次你或代理调用它时,就在你的shell里跑起来了,权限模式取决于你当初选的那个。

让第二把钥匙跟着代理走

规则可以这样定:代理分支以pull request的形式落地,由队友批准,再由你写的检查判定工作是否完成。

用GitHub CLI推分支、开PR,而不是直接合并:

git push -u origin agent/fix-login
gh pr create --base main --fill

--fill会从提交里取标题和正文,于是代理的提交信息成了评审人最先读到的PR描述。如果写得含糊,就改一改。

然后在GitHub侧补上缺口,改main的分支保护规则:

  • 要求pull request评审,至少一个批准。
  • 打开“不允许绕过上述设置”,去掉管理员豁免。
  • 打开“要求批准最近一次可评审的推送”:最近一次可评审的推送,必须由推送者以外的人批准。
  • 要求状态检查通过,把你的测试任务设为其中之一。

再给高风险路径指定负责人。代码负责人“在有人打开修改其所属代码的pull request时会被自动请求评审”,配合“要求代码负责人评审”,他们的批准就是强制的:

# .github/CODEOWNERS
* @acme/backend
.github/workflows/ @acme/platform

顺序有讲究:最后匹配的模式优先级最高,所以workflows那行要放在通配行之后。

代理报告说完成了。由你写的检查来拍板。

代价落在哪里

单人开发者没有第二个人。这时你写的检查就是第二方:它不知道是谁要求的改动,也不在乎代理听起来多自信。跳过必需评审,把检查设为必需状态检查,在自己仓库上关掉管理员绕过,连你自己也得走一遍。

在团队里,代价是延迟:每个代理分支现在都要等队友,就像github.com上的Copilot云端代理PR本来就要等一样。

你是让代理分支在本地直接合并,还是代理写的一切都得走一个由别人批准的PR?

特别声明:以上内容(如有图片或视频亦包括在内)为自媒体平台“网易号”用户上传并发布,本平台仅提供信息存储服务。

Notice: The content above (including the pictures and videos if any) is uploaded and posted by a user of NetEase Hao, which is a social media platform and only provides information storage services.

相关推荐
热点推荐
彻底撕破脸!乌克兰,这次走投无路了!

彻底撕破脸!乌克兰,这次走投无路了!

大嘴说天下
2026-10-17 20:40:04
以色列大选倒计时12天!贝内特放出狠话:上台第1天就彻查内塔尼亚胡

以色列大选倒计时12天!贝内特放出狠话:上台第1天就彻查内塔尼亚胡

兰妮搞笑分享
2026-10-19 09:22:01
人在监狱服刑,网贷记录多了十多万?湖北黄冈一男子称入狱20多天,就被人冒名贷款、转走资金超38万元

人在监狱服刑,网贷记录多了十多万?湖北黄冈一男子称入狱20多天,就被人冒名贷款、转走资金超38万元

台州交通广播
2026-10-18 21:13:05
蛋白粉大涨价,网友找到了“蟑螂粉”,有商家称其蛋白质含量70%,每斤还不到20元,乍一听很划算.......医生紧急提醒

蛋白粉大涨价,网友找到了“蟑螂粉”,有商家称其蛋白质含量70%,每斤还不到20元,乍一听很划算.......医生紧急提醒

都市快报橙柿互动
2026-10-18 17:43:33
4-1!熊皇被禁赛,1.25亿标王发威,姆巴佩梅开二度,皇马3连胜

4-1!熊皇被禁赛,1.25亿标王发威,姆巴佩梅开二度,皇马3连胜

我的护球最独特
2026-10-19 05:12:10
季前赛打完!东部实力一目了然:争冠档4队、季后赛档3队!

季前赛打完!东部实力一目了然:争冠档4队、季后赛档3队!

运筹帷幄的篮球
2026-10-18 11:05:40
中国为啥要下血本猛搞新能源车?说句实在话,环保就是个幌子,石油也不过是个由头。

中国为啥要下血本猛搞新能源车?说句实在话,环保就是个幌子,石油也不过是个由头。

回京历史梦
2026-10-13 09:22:14
“还不分床,等着看德国骨科?”家长晒姐弟俩午睡一幕,让过来人直言:你会后悔的

“还不分床,等着看德国骨科?”家长晒姐弟俩午睡一幕,让过来人直言:你会后悔的

妍妍教育日记
2026-09-14 21:18:41
果然不出欧洲所料!特朗普的命令不管用了,泽连斯基已解除禁令

果然不出欧洲所料!特朗普的命令不管用了,泽连斯基已解除禁令

阿器谈史
2026-10-19 09:43:50
郭士强看看吧!王俊杰NCAA爆砍31分带队赢球,为何差距那么大?

郭士强看看吧!王俊杰NCAA爆砍31分带队赢球,为何差距那么大?

萌兰聊个球
2026-10-19 09:17:33
第一集就打码,今年最敢拍的限定剧,建议收藏!

第一集就打码,今年最敢拍的限定剧,建议收藏!

绚丽的画卷
2026-10-13 14:43:52
布朗举办30岁庆生派对!詹姆斯出席并拥抱祝福 马克西等队友也来了

布朗举办30岁庆生派对!詹姆斯出席并拥抱祝福 马克西等队友也来了

追球者
2026-10-19 08:49:59
最煎熬的不是失业没了工作,而是社保年限从15年涨到了20年

最煎熬的不是失业没了工作,而是社保年限从15年涨到了20年

职场资深秘书
2026-10-16 16:36:46
或创造人类联赛历史!成都蓉城面临失冠 本周川军应请回老塔马麦罗

或创造人类联赛历史!成都蓉城面临失冠 本周川军应请回老塔马麦罗

劲爆体坛
2026-10-19 07:07:04
王俊杰轰下31分6篮板创NCAA生涯新高,率队88-86险胜布朗大学

王俊杰轰下31分6篮板创NCAA生涯新高,率队88-86险胜布朗大学

懂球帝
2026-10-19 09:42:44
4-1狂胜!就在刚刚,穆里尼奥大胆变阵出奇效!1.25亿夏窗标王一鸣惊人,姆巴佩双响破法国纪录,皇马携3连胜宣战全胜巴萨!

4-1狂胜!就在刚刚,穆里尼奥大胆变阵出奇效!1.25亿夏窗标王一鸣惊人,姆巴佩双响破法国纪录,皇马携3连胜宣战全胜巴萨!

锐评利物浦
2026-10-19 10:10:14
奢靡荒唐的白马会所:性交易泛滥,让无数富婆挥金如土

奢靡荒唐的白马会所:性交易泛滥,让无数富婆挥金如土

天启大世界
2026-10-13 17:17:54
400 多万买的闵行学区房,现在跌了一半,我女儿考试常年垫底

400 多万买的闵行学区房,现在跌了一半,我女儿考试常年垫底

来去自如的小章
2026-10-18 15:14:24
久违的御姐回来了,你会喜欢吗?

久违的御姐回来了,你会喜欢吗?

疾跑的小蜗牛
2026-10-18 16:17:16
中央定调!延迟退休继续实施,70后延后退?养老金能差多少钱?

中央定调!延迟退休继续实施,70后延后退?养老金能差多少钱?

江启
2026-10-19 09:42:56
2026-10-19 11:03:00
码上闲叙
码上闲叙
有态度网友ytd
178文章数 102关注度
往期回顾 全部

科技要闻

王晟:近千亿资金涌入后 具身智能开始退烧

头条要闻

上海阿姨心疼保险业务员离异带娃 结果5年被转走200万

头条要闻

上海阿姨心疼保险业务员离异带娃 结果5年被转走200万

体育要闻

从NBA退役10年后,他真去警局上班了

娱乐要闻

演员许君聪近况曝光 婚恋成谜让人唏嘘

财经要闻

三季度GDP,同比增长4.5%

汽车要闻

星瑞L PLUS限时售价10.97万元起 开启智混3L时代

态度原创

时尚
健康
亲子
数码
手机

重阳寻香:一炉香,一枚囊,她们以手传香

看着很健康的人,怎么会突然猝死?

亲子要闻

让孩子学搏击,不是教会孩子暴力,而是让孩子更加勇敢有担当

数码要闻

RTX Spark装进16英寸翻转本 微星尊爵N16 Flip AI+ 2026体验

手机要闻

一加Ace 7系列手机官宣本月发布

无障碍浏览 进入关怀版