Skip to content

chore(ci): 允许 Open-Less 组织成员触发 @claude - #461

Merged
appergb merged 1 commit into
betafrom
chore/claude-allow-org-members
May 17, 2026
Merged

chore(ci): 允许 Open-Less 组织成员触发 @claude#461
appergb merged 1 commit into
betafrom
chore/claude-allow-org-members

Conversation

@appergb

@appergbappergb commented May 17, 2026

Copy link
Copy Markdown
Collaborator

User description

背景

仓库转入 Open-Less 组织后,claude.yml 的触发门禁 author_association == 'OWNER'所有人(包括转移前的原 owner)都不再匹配 —— 因为 GitHub 的 OWNER 这个 author_association 值只对个人账号下的同名仓库有效,组织仓库下人类用户最高只会拿到 MEMBER

证据:beta 上最近 10 次 claude.yml 的 workflow run 全部 conclusion: skipped,从未真正执行过 job。

改动

把四条 if: 子表达式的 author_association == 'OWNER' 全部换成 contains(fromJson('["OWNER","MEMBER"]'), …author_association)。OWNER 保留是为了兼容仓库未来再次个人持有的情形,新增 MEMBER 才是实际生效的那条

顺手把注释里的"必须是仓库 OWNER(appergb)"同步更新为"OWNER 或 MEMBER(Open-Less 组织成员)",并新增一行说明为什么 OWNER 在 org 仓库下不会匹配(避免下次有人疑惑)。

未触碰:

  • 触发事件 (on:) 范围 —— 仍然只在评论/issue 出现 @claude 时才跑,不改变成本特征
  • 模型选择逻辑、--opus 升级逻辑、additional_permissions、OAuth token 用法
  • 任何其它 workflow / 代码

测试计划

合并后:

  • 任一 Open-Less 成员在任意 issue 评论 @claude ping
  • gh run list -w claude.yml -L 1 --repo Open-Less/openless 显示 conclusion 不再是 skipped
  • 如果 run 跑了但 Claude 没回复 → 还需要在 https://github.com/apps/claude 把 Claude App 安装到 Open-Less 组织
  • 外部 (COLLABORATOR) 评论不会触发 —— 故意保留为成员限定

不在本 PR 范围

  • 自动 PR review:仓库目前由 pr-agent.yml(Kimi + GPT-5 fallback)承担,本 PR 不动它
  • Claude App 在 Open-Less 组织的安装:那是 GitHub 网页操作,没法靠 commit 解决

PR Type

Bug fix


Description

  • 放宽 @claude 触发者门禁

  • 允许 OWNERMEMBER

  • 更新工作流注释说明


Diagram Walkthrough

flowchart LR
A["@claude comments and reviews"] --> B["Check author association"]
B --> C["Allow OWNER or MEMBER"]
C --> D["Trigger Claude workflow"]
Loading

File Walkthrough

Relevant files
Bug fix
claude.yml
Allow org members to trigger Claude

.github/workflows/claude.yml

  • Expands the author_association gate from only OWNER to OWNER or
    MEMBER.
  • Keeps the same @claude-based trigger conditions for comments, reviews,
    and issues.
  • Updates inline comments to explain Open-Less org membership behavior.
+8/-7

After the repo transfer to Open-Less, author_association for human users
is at most MEMBER (OWNER only applies to user-owned repos). The previous
gate (== OWNER) silently skipped every @claude comment. Expand the gate
to [OWNER, MEMBER] so any org member can trigger Claude on demand.
@github-actions

Copy link
Copy Markdown
Contributor

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ No major issues detected

@appergb
appergb merged commit 98dd4c7 into betaMay 17, 2026
4 checks passed
@appergb
appergb deleted the chore/claude-allow-org-members branch May 17, 2026 01:54
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

@appergb