Skip to content

fix(ci): skip watch-opencode-releases on forks - #11269

Merged
johnnyeric merged 3 commits into
Kilo-Org:mainfrom
vkeerthivikram:fix/watch-opencode-fork-skip-slack
Jun 15, 2026
Merged

fix(ci): skip watch-opencode-releases on forks#11269
johnnyeric merged 3 commits into
Kilo-Org:mainfrom
vkeerthivikram:fix/watch-opencode-fork-skip-slack

Conversation

@vkeerthivikram

Copy link
Copy Markdown
Contributor

Summary

  • Add if: github.repository == 'Kilo-Org/kilocode' to the check-release job in watch-opencode-releases.yml.
  • The previous vars.OPENCODE_WATCH_ENABLED != 'false' guard did not protect forks: on a fork the variable is empty, '' != 'false' is true, the job ran, and the Slack step failed because OPENCODE_WATCH_SLACK_WEBHOOK is not available on forks.
  • This matches the guard used by 18 other workflows in this repo.

Repro

  1. Fork Kilo-Org/kilocode.
  2. Wait for the 7,37 * * * * cron, or run via workflow_dispatch.
  3. slackapi/slack-github-action fails with: SlackError: Missing input! Either a token or webhook is required to take action.

Refs

@kilo-code-bot

kilo-code-bot Bot commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (1 file)
  • .github/workflows/watch-opencode-releases.yml — retained both github.repository guard and OPENCODE_WATCH_ENABLED variable gate per feedback
Previous Review Summary (commit c737850)

Current summary above is authoritative. Previous snapshots are kept for context only.

Previous review (commit c737850)

Status: No Issues Found | Recommendation: Merge

Files Reviewed (1 file)
  • .github/workflows/watch-opencode-releases.yml - Simple condition fix matching existing repo patterns

Reviewed by deepseek-v4-pro · 235,598 tokens

Review guidance: REVIEW.md from base branch main

Comment thread .github/workflows/watch-opencode-releases.yml Outdated
The schedule trigger fires on every repo where the file exists,
including contributor forks. Forks don't have access to
OPENCODE_WATCH_SLACK_WEBHOOK, so the Slack step fails. The previous
vars.OPENCODE_WATCH_ENABLED guard evaluated to true on forks (empty
!= 'false'), so it didn't protect anything.

Add the same github.repository == 'Kilo-Org/kilocode' guard used by
18 other workflows in this repo.
@vkeerthivikram
vkeerthivikram force-pushed the fix/watch-opencode-fork-skip-slack branch from c737850 to 7bf75ee Compare June 15, 2026 17:21
@johnnyeric

Copy link
Copy Markdown
Contributor

The Check shared upstream annotations / Check kilocode_change annotations validation is a false positive, this is a Kilo specific workflow.

@johnnyeric
johnnyeric merged commit 8baf3e3 into Kilo-Org:main Jun 15, 2026
15 of 16 checks passed
@johnnyeric

Copy link
Copy Markdown
Contributor

Thanks for reporting the issue and sending the fix. Merged.

@vkeerthivikram
vkeerthivikram deleted the fix/watch-opencode-fork-skip-slack branch June 15, 2026 18:06
@markijbema

Copy link
Copy Markdown
Contributor

Thanks!

@johnnyeric

Copy link
Copy Markdown
Contributor

Hey @vkeerthivikram, could you please link your GitHub account to the Kilo account? This way, we can grant you credits for the merged PRs.
Once you have done that, please message johnny[at]kilocode.ai

@vkeerthivikram

Copy link
Copy Markdown
Contributor Author

Hey @vkeerthivikram, could you please link your GitHub account to the Kilo account? This way, we can grant you credits for the merged PRs. Once you have done that, please message johnny[at]kilocode.ai

hey @johnnyeric , sorry i was on a vacation and missed this. Its already linked!! Should i mail you once with my usernames and Kilo Account email?

t7tran pushed a commit to t7tran/kilocode that referenced this pull request Aug 14, 2026
The schedule trigger fires on every repo where the file exists,
including contributor forks. Forks don't have access to
OPENCODE_WATCH_SLACK_WEBHOOK, so the Slack step fails. The previous
vars.OPENCODE_WATCH_ENABLED guard evaluated to true on forks (empty
!= 'false'), so it didn't protect anything.

Add the same github.repository == 'Kilo-Org/kilocode' guard used by
18 other workflows in this repo.
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.

fix(ci): watch-opencode-releases runs on contributor forks and fails on missing Slack webhook

3 participants