Skip to content

fix(ci): 修复 Markdown 解析器安装失败 - #337

Merged
Terry-Mao merged 1 commit into
mainfrom
fix/markdown-parser-venv
Nov 4, 2026
Merged

Terry-Mao merged 1 commit into
mainfrom
fix/markdown-parser-venv

Conversation

@Terry-Mao

Copy link
Copy Markdown
Owner

Summary

  • 修复 orbit PR 审查运行失败 暴露的 Python venv 依赖问题:需要安装 Markdown 解析器的 job 先通过 actions/setup-python@v6 配置 Python 3.12;CI 的 Python 测试依赖安装步骤同样配置。
  • 将 Markdown 解析器改为按需导入。普通 PR 审查和非评论触发的 Spec 工作流跳过解析器安装,评论命令仍使用完整 Markdown 解析。
  • 增加无解析器环境下的事件路径测试,并校验 workflow 步骤顺序和触发条件。

Testing

  • uv run --no-project --with PyYAML==6.0.3 --with markdown-it-py==4.2.0 --with mdurl==0.1.2 python -m unittest discover -s .github/aicodingflow-tests:526 项通过。
  • actionlint .github/workflows/review-pr.yml .github/workflows/respond-to-pr-comment.yml .github/workflows/create-spec-from-issue.yml .github/workflows/ci.yml:通过。
  • git diff --check:通过。

Risk

  • 评论触发路径现在依赖 setup-python 在 runner 上提供 Python 3.12;目标仓库同步后仍需用实际 Actions 运行验证该 runner 的工具缓存或下载能力。

How to revert

  • 在此分支或合并后的目标分支执行 git revert 5e5a4055201343421516053d46d9fc42d2581cfa,恢复原有安装步骤和导入方式。

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

审查结论

按完整 pr_diff.txt 审查:Markdown 解析器已改为按需导入,评论相关 workflow 在解析前配置 Python 3.12,非评论事件路径不再安装解析器;CI 测试环境也补齐了 setup-python。未发现需要阻止合并的正确性、安全性、测试或工作流触发问题。

验证

聚焦运行 test_prepare_issue_spec_context.py(29 项)和 test_resolve_pr_event.py(24 项)均通过;test_review_workflow_dispatch.py 因本地审查环境缺少 PyYAML 未能导入,相关 workflow 变更已通过静态检查。

@Terry-Mao
Terry-Mao merged commit 600d578 into main Nov 4, 2026
4 checks passed
@Terry-Mao
Terry-Mao deleted the fix/markdown-parser-venv branch November 4, 2026 04:46
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