Skip to content

fix: 跳过 product docs sync 中缺失的 issue 引用 - #215

Merged
Terry-Mao merged 2 commits into
mainfrom
spec/issue-214
Jun 1, 2026
Merged

fix: 跳过 product docs sync 中缺失的 issue 引用#215
Terry-Mao merged 2 commits into
mainfrom
spec/issue-214

Conversation

@github-actions

@github-actions github-actions Bot commented Jun 1, 2026

Copy link
Copy Markdown
Contributor

Closes #214

Summary

  • Product Docs Sync 准备上下文时,如果单个 linked issue 无法通过 gh issue view 读取,会跳过该 issue 并继续执行。
  • 成功读取的 linked issue 仍会进入上下文,相关 specs 只按这些已确认存在的 issue 编号读取。
  • 补充 Product Docs Sync 测试,覆盖缺失 issue 容错和上下文/spec 过滤行为。

Validation

  • PYTHONDONTWRITEBYTECODE=1 python3 -m unittest discover -s .github/aicodingflow-tests -p 'test_product_docs_sync.py'
  • PYTHONPYCACHEPREFIX=/tmp/aicodingflow-pycache python3 -m py_compile .github/scripts/prepare_product_docs_sync_context.py
  • git diff --check
  • PYTHONDONTWRITEBYTECODE=1 python3 -m unittest discover -s .github/aicodingflow-tests

@github-actions
github-actions Bot requested a review from Terry-Mao as a code owner June 1, 2026 09:10
@Terry-Mao Terry-Mao added the plan-approved Implementation plan has been approved label Jun 1, 2026
@github-actions github-actions Bot changed the title docs(spec): create issue 214 specs fix: 跳过 product docs sync 中缺失的 issue 引用 Jun 1, 2026
@Terry-Mao

Copy link
Copy Markdown
Owner

@kratos-ci-bot /review

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

未发现需要阻塞合并的问题。实现与 spec 对齐:missing linked issue 只在单个 issue fetch 路径被跳过,linked_issues 与 specs 都基于成功读取的 issue 过滤,PR/diff/docs 等其他错误路径仍保持原有失败语义。安全补充检查未发现新增风险。

@Terry-Mao
Terry-Mao merged commit 67e940c into main Jun 1, 2026
1 check passed
@Terry-Mao
Terry-Mao deleted the spec/issue-214 branch June 1, 2026 09:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

plan-approved Implementation plan has been approved

Projects

None yet

Development

Successfully merging this pull request may close these issues.

PR refs 不存在 issue,会导致 product docs sync 任务失败

1 participant