Repository navigation
Conversation
![kilo-code-bot[bot]](https://avatars.githubusercontent.com/in/2193792?s=60&v=4){kind=small}
Contributor
Code Review SummaryStatus: No Issues Found | Recommendation: Merge Executive SummaryFull re-review of the 17 workflow files that add Files Reviewed (17 files)
Verification Notes
Previous Review Summaries (7 snapshots, latest commit fafd2bd)Current summary above is authoritative. Previous snapshots are kept for context only. Previous review (commit fafd2bd)Status: No Issues Found | Recommendation: Merge Executive SummaryRe-reviewed the 17 workflow files that add Files Reviewed (17 files)
Verification Notes
Previous review (commit 46c5cbe)Status: No Issues Found | Recommendation: Merge Executive SummaryRe-reviewed the 17 workflow files that add Files Reviewed (17 files)
Verification Notes
Previous review (commit ed1bcb3)Status: No Issues Found | Recommendation: Merge Executive SummaryThe only incremental change since the prior review is a one-line comment correction in Files Reviewed (1 file)
Verification Notes
Previous review (commit cabdd36)Status: 1 Issues Found | Recommendation: Address before merge Executive SummaryThe 17 fork guards remain functionally correct and behavior-neutral for canonical-repo runs; the only new finding is an inaccurate trigger reference in a comment added by this PR. Overview
Issue Details (click to expand)SUGGESTION
Verification Notes
Fix these issues in Kilo Cloud Files Reviewed (17 files)
Previous review (commit ece9425)Status: No Issues Found | Recommendation: Merge Executive SummaryRe-reviewed the 17 workflow files that add Files Reviewed (17 files)
Previous review (commit 3c6fc28)Status: No Issues Found | Recommendation: Merge Executive SummaryThe fork-guard changes are limited to job-level Files Reviewed (17 files)
Previous review (commit a546d1f)Status: 2 Issues Found | Recommendation: Address before merge Overview
Issue Details (click to expand)SUGGESTION
Files Reviewed (17 files)
Notes
Reviewed by deepseek-v4.1-flash · Input: 0 · Output: 0 · Cached: 0 Review guidance: REVIEW.md from base branch |
maphew
force-pushed
the
fix/workflow-fork-guard
branch
4 times, most recently
from
September 22, 2026 20:39
ece9425 to
cabdd36
Compare
maphew
force-pushed
the
fix/workflow-fork-guard
branch
2 times, most recently
from
September 23, 2026 20:00
a6a40da to
fafd2bd
Compare
Adds if: github.repository == 'Kilo-Org/cloud' to the jobs of workflows that must only run in the canonical repository, combining with existing if: conditions where present. Mirrors the long-standing Kilo-Org/kilocode convention. Co-authored-by: kiloconnect[bot] <240665456+kiloconnect[bot]@users.noreply.github.com>
maphew
force-pushed
the
fix/workflow-fork-guard
branch
from
September 25, 2026 04:34
fafd2bd to
3dd3d76
Compare
Contributor
Author
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.
Summary
Adds
if: github.repository == 'Kilo-Org/cloud'to the jobs of workflows that must only run in the canonical repository, combining with existingif:conditions where present. Mirrors the long-standingKilo-Org/kilocodeconvention.Guarded (44 jobs across 17 files):
auto-docs,bump-openclaw,check-md-table-padding,codeql,deploy-production,deploy-staging,deploy-workers,deploy-kiloclaw,extension-store-release,kilo-app-release,promote-vercel-deployment,push-dev-kiloclaw,redeploy-web,sbom,setup-smoke,stage-vercel-deployment,stale-prs.Intentionally not guarded:
ci.yml,extension-ci.yml,kilo-app-ci.yml,mobile-native-build.yml,chromatic.yml,trufflehog.yml. Forpull_requesteventsgithub.repositoryis the base repo, so a guard there is a no-op for normal CI and would only turn an approved fork-PR run into a wall of skipped jobs. Kilocode likewise does not guard its test/typecheck workflows. See #6248 for the rationale and scope.The
action_requiredruns on fork PRs are GitHub's Settings → Actions → General → Fork pull request workflows gate and cannot be changed from workflow code; this PR does not affect them.Closes #6248
Verification
git diff --checkis clean and the changed files contain no tabs.if:guards plus 8 existingif:lines extended with the repo check; no structural or formatting changes.pnpm format/pnpm format:check,actionlint, or a YAML parse in this environment: there is nonode_modulesand no YAML parser available. Behavior is unchanged for every event inKilo-Org/cloud(the guard only evaluates false when the run executes inside a fork), but please let CI validate the YAML.Visual Changes
N/A
Reviewer Notes
check-md-table-padding.ymlis guarded to match its kilocode counterpart; the guard still passes for base-repo PRs.