Skip to content

fix: auto-review task poll 403#9

Merged
github-actions[bot] merged 1 commit into
mainfrom
fix/auto-review-task-poll-403
Mar 7, 2026
Merged

fix: auto-review task poll 403#9
github-actions[bot] merged 1 commit into
mainfrom
fix/auto-review-task-poll-403

Conversation

@liuxiaotong

Copy link
Copy Markdown
Owner

Summary

  • 提交 review 任务时在 payload 中传入 user_id: github-actions,使任务有 owner
  • 轮询任务状态的 URL 加 ?user_id=github-actions 查询参数,解决 403 拒绝

Root Cause

auto-review 提交 review 任务时没传 user_id,导致任务无 owner,轮询 /tasks/{task_id} 时被 403 拒绝。

提交 review 任务时传 user_id,轮询时带 user_id 查询参数,
解决无 owner 任务被 403 拒绝的问题。

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@github-actions

github-actions Bot commented Mar 7, 2026

Copy link
Copy Markdown

Auto Review (Round 1) — ✅ Approved

第 1 轮审查:Approve with Suggestions。核心问题修复正确,逻辑合理。发现 2 个 Warning(硬编码 user_id 可能导致权限混乱、错误处理缺失)和 2 个 Suggestion(环境变量配置、日志补充)。

核心改进:1) 提交任务时传入 user_id: github-actions 使任务有 owner;2) 轮询 URL 加 ?user_id=github-actions 查询参数解决 403。修复方案直接有效,符合 API 权限设计。

@github-actions github-actions Bot enabled auto-merge (squash) March 7, 2026 07:41
@github-actions github-actions Bot merged commit f234460 into main Mar 7, 2026
5 checks passed
Sign up for free to 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