Skip to content

feat(kiloclaw) openclaw bump notify#3779

Merged
St0rmz1 merged 2 commits into
mainfrom
feat/kiloclaw-openclaw-bump-notify
Jun 5, 2026
Merged

feat(kiloclaw) openclaw bump notify#3779
St0rmz1 merged 2 commits into
mainfrom
feat/kiloclaw-openclaw-bump-notify

Conversation

@St0rmz1
Copy link
Copy Markdown
Contributor

@St0rmz1 St0rmz1 commented Jun 5, 2026

Summary

The release check workflow used to post a Slack message the moment a new OpenClaw
release was detected. That message fired in parallel with the bump agent, so it could
not include the resulting PR link or any assessment of the upgrade, and it rendered a
link preview card. This change removes that early Slack step. The Slack notification is
now sent by the cloud agent after it opens the bump PR, so it can include the PR link
and a scored upgrade assessment. The trigger webhook also passes the current version so
the agent can report the version range.

Changes

  • Removed the Notify Slack step from .github/workflows/bump-openclaw.yml. The cloud
    agent that handles the bump now posts the Slack notification itself after opening the PR.
  • Added current_openclaw_version to the webhook payload alongside new_openclaw_version,
    so the agent has both versions without extra git work.
  • Removed continue-on-error: true from the webhook step. With the early Slack ping gone,
    a failed trigger now fails the run instead of being swallowed silently.
  • Added a comment in the step noting that Slack notification now happens downstream in the
    agent.

Verification

  • Posted the trigger payload to the webhook directly and confirmed the cloud agent
    opened or updated the bump PR and sent the new Slack message with the version range, the
    scored assessment, and links as buttons with no preview card.
  • Have not run the scheduled or manually dispatched workflow job itself yet. The change
    is config only, and CI lint covers the YAML.

Visual Changes

N/A. No application UI changes. The Slack message layout itself is produced by the cloud
agent, not by this workflow file.

@kilo-code-bot
Copy link
Copy Markdown
Contributor

kilo-code-bot Bot commented Jun 5, 2026

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Executive Summary

The workflow change correctly removes continue-on-error from the webhook step so broken triggers fail loudly, drops the now-redundant direct Slack step, and passes current_openclaw_version to the webhook payload — all changes are safe and well-commented.

Files Reviewed (1 file)
  • .github/workflows/bump-openclaw.yml

Reviewed by claude-sonnet-4.6 · 279,614 tokens

Review guidance: REVIEW.md from base branch main

@St0rmz1 St0rmz1 changed the title Feat/kiloclaw openclaw bump notify feat(kiloclaw) openclaw bump notify Jun 5, 2026
@St0rmz1 St0rmz1 merged commit c293a7c into main Jun 5, 2026
16 checks passed
@St0rmz1 St0rmz1 deleted the feat/kiloclaw-openclaw-bump-notify branch June 5, 2026 23:12
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.

2 participants