Repository navigation
ci(fork): 所有改動都走同一個 PR 流程，CI 依改動的檔案決定要不要跑全套 - #80
Merged
Merged
Conversation
…obs on docs Docs went straight to develop so they would not wait 15-25 min for CI, which left no PR record and needed a rebase-lock exception. Now every change opens a PR. fork-pr.yml's new scope job classifies the changed paths with upstream's isDocsOnlyPath (plus FORK.md, never .claude/skills); the heavy jobs run only on `full`, and verify accepts skipped jobs only when scope succeeded and said `docs`. An empty path list or a failed scope runs everything. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
… inputs Gate review of #80: a docs-only PR skipped `pnpm lint`'s README link check, so a broken link would turn every later PR red, and docs/site's package.json (read by the test-runner check) counted as docs. The scope job now runs the link check itself, fork-pr-scope keeps CI, build, agent and docs/site paths full even if upstream's docs list grows, and FORK.md says a failed scope job blocks the PR. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…oves /code-review of #80: scope tests keep only paths the fork's full-scope guard rejects (docs/site) plus the skills rule; the skip rule lives in the scope script and the gate checker only; the heavy jobs drop a draft check scope already applies; the isDocsOnlyPath pin says why it imports; FORK.md's local-check line follows the scope job. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
audichuang
added a commit
that referenced
this pull request
Oct 26, 2026
audichuang
added a commit
that referenced
this pull request
Oct 26, 2026
… inputs Gate review of #80: a docs-only PR skipped `pnpm lint`'s README link check, so a broken link would turn every later PR red, and docs/site's package.json (read by the test-runner check) counted as docs. The scope job now runs the link check itself, fork-pr-scope keeps CI, build, agent and docs/site paths full even if upstream's docs list grows, and FORK.md says a failed scope job blocks the PR. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
audichuang
added a commit
that referenced
this pull request
Oct 26, 2026
…oves /code-review of #80: scope tests keep only paths the fork's full-scope guard rejects (docs/site) plus the skills rule; the skip rule lives in the scope script and the gate checker only; the heavy jobs drop a draft check scope already applies; the isDocsOnlyPath pin says why it imports; FORK.md's local-check line follows the scope job. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
audichuang
added a commit
that referenced
this pull request
Oct 27, 2026
… inputs Gate review of #80: a docs-only PR skipped `pnpm lint`'s README link check, so a broken link would turn every later PR red, and docs/site's package.json (read by the test-runner check) counted as docs. The scope job now runs the link check itself, fork-pr-scope keeps CI, build, agent and docs/site paths full even if upstream's docs list grows, and FORK.md says a failed scope job blocks the PR. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
audichuang
added a commit
that referenced
this pull request
Oct 27, 2026
…oves /code-review of #80: scope tests keep only paths the fork's full-scope guard rejects (docs/site) plus the skills rule; the skip rule lives in the scope script and the gate checker only; the heavy jobs drop a draft check scope already applies; the isDocsOnlyPath pin says why it imports; FORK.md's local-check line follows the scope job. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
audichuang
added a commit
that referenced
this pull request
Oct 28, 2026
… inputs Gate review of #80: a docs-only PR skipped `pnpm lint`'s README link check, so a broken link would turn every later PR red, and docs/site's package.json (read by the test-runner check) counted as docs. The scope job now runs the link check itself, fork-pr-scope keeps CI, build, agent and docs/site paths full even if upstream's docs list grows, and FORK.md says a failed scope job blocks the PR. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
audichuang
added a commit
that referenced
this pull request
Oct 28, 2026
…oves /code-review of #80: scope tests keep only paths the fork's full-scope guard rejects (docs/site) plus the skills rule; the skip rule lives in the scope script and the gate checker only; the heavy jobs drop a draft check scope already applies; the isDocsOnlyPath pin says why it imports; FORK.md's local-check line follows the scope job. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
audichuang
added a commit
that referenced
this pull request
Oct 29, 2026
… inputs Gate review of #80: a docs-only PR skipped `pnpm lint`'s README link check, so a broken link would turn every later PR red, and docs/site's package.json (read by the test-runner check) counted as docs. The scope job now runs the link check itself, fork-pr-scope keeps CI, build, agent and docs/site paths full even if upstream's docs list grows, and FORK.md says a failed scope job blocks the PR. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
audichuang
added a commit
that referenced
this pull request
Oct 29, 2026
…oves /code-review of #80: scope tests keep only paths the fork's full-scope guard rejects (docs/site) plus the skills rule; the skip rule lives in the scope script and the gate checker only; the heavy jobs drop a draft check scope already applies; the isDocsOnlyPath pin says why it imports; FORK.md's local-check line follows the scope job. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
audichuang
added a commit
that referenced
this pull request
Oct 30, 2026
… inputs Gate review of #80: a docs-only PR skipped `pnpm lint`'s README link check, so a broken link would turn every later PR red, and docs/site's package.json (read by the test-runner check) counted as docs. The scope job now runs the link check itself, fork-pr-scope keeps CI, build, agent and docs/site paths full even if upstream's docs list grows, and FORK.md says a failed scope job blocks the PR. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
audichuang
added a commit
that referenced
this pull request
Oct 30, 2026
…oves /code-review of #80: scope tests keep only paths the fork's full-scope guard rejects (docs/site) plus the skills rule; the skip rule lives in the scope script and the gate checker only; the heavy jobs drop a draft check scope already applies; the isDocsOnlyPath pin says why it imports; FORK.md's local-check line follows the scope job. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
audichuang
added a commit
that referenced
this pull request
Nov 2, 2026
… inputs Gate review of #80: a docs-only PR skipped `pnpm lint`'s README link check, so a broken link would turn every later PR red, and docs/site's package.json (read by the test-runner check) counted as docs. The scope job now runs the link check itself, fork-pr-scope keeps CI, build, agent and docs/site paths full even if upstream's docs list grows, and FORK.md says a failed scope job blocks the PR. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
audichuang
added a commit
that referenced
this pull request
Nov 2, 2026
…oves /code-review of #80: scope tests keep only paths the fork's full-scope guard rejects (docs/site) plus the skills rule; the skip rule lives in the scope script and the gate checker only; the heavy jobs drop a draft check scope already applies; the isDocsOnlyPath pin says why it imports; FORK.md's local-check line follows the scope job. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
audichuang
added a commit
that referenced
this pull request
Nov 3, 2026
… inputs Gate review of #80: a docs-only PR skipped `pnpm lint`'s README link check, so a broken link would turn every later PR red, and docs/site's package.json (read by the test-runner check) counted as docs. The scope job now runs the link check itself, fork-pr-scope keeps CI, build, agent and docs/site paths full even if upstream's docs list grows, and FORK.md says a failed scope job blocks the PR. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
audichuang
added a commit
that referenced
this pull request
Nov 3, 2026
…oves /code-review of #80: scope tests keep only paths the fork's full-scope guard rejects (docs/site) plus the skills rule; the skip rule lives in the scope script and the gate checker only; the heavy jobs drop a draft check scope already applies; the isDocsOnlyPath pin says why it imports; FORK.md's local-check line follows the scope job. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
audichuang
added a commit
that referenced
this pull request
Nov 4, 2026
… inputs Gate review of #80: a docs-only PR skipped `pnpm lint`'s README link check, so a broken link would turn every later PR red, and docs/site's package.json (read by the test-runner check) counted as docs. The scope job now runs the link check itself, fork-pr-scope keeps CI, build, agent and docs/site paths full even if upstream's docs list grows, and FORK.md says a failed scope job blocks the PR. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
audichuang
added a commit
that referenced
this pull request
Nov 4, 2026
…oves /code-review of #80: scope tests keep only paths the fork's full-scope guard rejects (docs/site) plus the skills rule; the skip rule lives in the scope script and the gate checker only; the heavy jobs drop a draft check scope already applies; the isDocsOnlyPath pin says why it imports; FORK.md's local-check line follows the scope job. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
audichuang
added a commit
that referenced
this pull request
Nov 5, 2026
… inputs Gate review of #80: a docs-only PR skipped `pnpm lint`'s README link check, so a broken link would turn every later PR red, and docs/site's package.json (read by the test-runner check) counted as docs. The scope job now runs the link check itself, fork-pr-scope keeps CI, build, agent and docs/site paths full even if upstream's docs list grows, and FORK.md says a failed scope job blocks the PR. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
audichuang
added a commit
that referenced
this pull request
Nov 5, 2026
…oves /code-review of #80: scope tests keep only paths the fork's full-scope guard rejects (docs/site) plus the skills rule; the skip rule lives in the scope script and the gate checker only; the heavy jobs drop a draft check scope already applies; the isDocsOnlyPath pin says why it imports; FORK.md's local-check line follows the scope job. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
audichuang
added a commit
that referenced
this pull request
Nov 5, 2026
… inputs Gate review of #80: a docs-only PR skipped `pnpm lint`'s README link check, so a broken link would turn every later PR red, and docs/site's package.json (read by the test-runner check) counted as docs. The scope job now runs the link check itself, fork-pr-scope keeps CI, build, agent and docs/site paths full even if upstream's docs list grows, and FORK.md says a failed scope job blocks the PR. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
audichuang
added a commit
that referenced
this pull request
Nov 5, 2026
…oves /code-review of #80: scope tests keep only paths the fork's full-scope guard rejects (docs/site) plus the skills rule; the skip rule lives in the scope script and the gate checker only; the heavy jobs drop a draft check scope already applies; the isDocsOnlyPath pin says why it imports; FORK.md's local-check line follows the scope job. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
audichuang
added a commit
that referenced
this pull request
Nov 6, 2026
… inputs Gate review of #80: a docs-only PR skipped `pnpm lint`'s README link check, so a broken link would turn every later PR red, and docs/site's package.json (read by the test-runner check) counted as docs. The scope job now runs the link check itself, fork-pr-scope keeps CI, build, agent and docs/site paths full even if upstream's docs list grows, and FORK.md says a failed scope job blocks the PR. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
audichuang
added a commit
that referenced
this pull request
Nov 6, 2026
…oves /code-review of #80: scope tests keep only paths the fork's full-scope guard rejects (docs/site) plus the skills rule; the skip rule lives in the scope script and the gate checker only; the heavy jobs drop a draft check scope already applies; the isDocsOnlyPath pin says why it imports; FORK.md's local-check line follows the scope job. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
audichuang
added a commit
that referenced
this pull request
Nov 7, 2026
… inputs Gate review of #80: a docs-only PR skipped `pnpm lint`'s README link check, so a broken link would turn every later PR red, and docs/site's package.json (read by the test-runner check) counted as docs. The scope job now runs the link check itself, fork-pr-scope keeps CI, build, agent and docs/site paths full even if upstream's docs list grows, and FORK.md says a failed scope job blocks the PR. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
audichuang
added a commit
that referenced
this pull request
Nov 7, 2026
…oves /code-review of #80: scope tests keep only paths the fork's full-scope guard rejects (docs/site) plus the skills rule; the skip rule lives in the scope script and the gate checker only; the heavy jobs drop a draft check scope already applies; the isDocsOnlyPath pin says why it imports; FORK.md's local-check line follows the scope job. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
audichuang
added a commit
that referenced
this pull request
Nov 8, 2026
… inputs Gate review of #80: a docs-only PR skipped `pnpm lint`'s README link check, so a broken link would turn every later PR red, and docs/site's package.json (read by the test-runner check) counted as docs. The scope job now runs the link check itself, fork-pr-scope keeps CI, build, agent and docs/site paths full even if upstream's docs list grows, and FORK.md says a failed scope job blocks the PR. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
audichuang
added a commit
that referenced
this pull request
Nov 8, 2026
…oves /code-review of #80: scope tests keep only paths the fork's full-scope guard rejects (docs/site) plus the skills rule; the skip rule lives in the scope script and the gate checker only; the heavy jobs drop a draft check scope already applies; the isDocsOnlyPath pin says why it imports; FORK.md's local-check line follows the scope job. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
audichuang
added a commit
that referenced
this pull request
Nov 8, 2026
… inputs Gate review of #80: a docs-only PR skipped `pnpm lint`'s README link check, so a broken link would turn every later PR red, and docs/site's package.json (read by the test-runner check) counted as docs. The scope job now runs the link check itself, fork-pr-scope keeps CI, build, agent and docs/site paths full even if upstream's docs list grows, and FORK.md says a failed scope job blocks the PR. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
audichuang
added a commit
that referenced
this pull request
Nov 8, 2026
…oves /code-review of #80: scope tests keep only paths the fork's full-scope guard rejects (docs/site) plus the skills rule; the skip rule lives in the scope script and the gate checker only; the heavy jobs drop a draft check scope already applies; the isDocsOnlyPath pin says why it imports; FORK.md's local-check line follows the scope job. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
為什麼
文件改動以前可以直推 develop，省掉 15～25 分鐘的 CI，但沒有 PR 紀錄，還要多一條「rebase 進行中不能直推」的例外。現在所有改動都開 PR，跑多少由 CI 依改動的檔案判斷，agent 不做任何選擇。
改了什麼
fork-pr.yml新增scopejob：用上游的方法取 PR 改動的檔案清單，交給config/scripts/fork-pr-scope.mjs判斷。全部是文件（上游的isDocsOnlyPath，加上FORK.md；.claude/skills/永遠不算）→docs，否則full。空清單或分類失敗都跑全套。full時跑。verify（「Fork PR checks passed」）只有在scope成功而且判定docs時才接受被跳過的 job，其他情況維持每個 job 都要成功。fork-pr-gate-needs.mjs固定的腳本與檢查訊息一起更新。config/fork-upstream-api.txt登記isDocsOnlyPath，上游拿掉這個 export 時 lint 會紅。FORK.md：拿掉文件直推 develop 的例外，寫明 CI 怎麼判斷。驗證
make fork-check-lint、check:code-quality:changed v1.4.240通過🤖 Generated with Claude Code