Skip to content

ci(release): dispatch ic-mops bump to caffeinelabs/app#620

Merged
Kamirus merged 2 commits into
mainfrom
chore/dispatch-ic-mops-bump
Jul 22, 2026
Merged

ci(release): dispatch ic-mops bump to caffeinelabs/app#620
Kamirus merged 2 commits into
mainfrom
chore/dispatch-ic-mops-bump

Conversation

@Kamirus

@Kamirus Kamirus commented Jul 22, 2026

Copy link
Copy Markdown
Collaborator

After each CLI release is published to npm and GitHub, notify private caffeinelabs/app so it can open the usual two-pin ic-mops bump PR.

Dispatch runs immediately after gh release create, before canister deploy / artifacts. Notify steps use continue-on-error so a dispatch hiccup cannot skip on-chain deploy.

Merge app first (#6933), then this, so the listener exists before the next cli-v* release.

After publishing a CLI release, notify app to open a PR that pins the
new ic-mops version before canister deploy can block the signal.

Co-authored-by: Cursor <cursoragent@cursor.com>
@github-actions

github-actions Bot commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Cursor AI review

👍 APPROVE — looks safe to merge

Category Assessment Details
Summary After a successful gh release create, the CLI release workflow best-effort dispatches mops-cli-release to caffeinelabs/app so it can open the usual ic-mops pin bump PR.
Code Quality Reuses the existing GitHub App credential pattern already used later in the same workflow; no speculative helpers.
Consistency Matches the file’s existing create-github-app-token usage, SHA-pinned actions, and release-step layout.
Security No trigger/permission broadening; reuses existing app secrets; new token is scoped to repositories: app; new action is SHA-pinned; payload values come from already-validated version/tag outputs.
Tests Workflow-only change; no new test surface expected.
Maintainability Clear comments and continue-on-error so notify failures cannot skip on-chain deploy/artifacts.

Verdict

Decision: APPROVE
Risk: Very Low
Reason: Additive, best-effort CI notify after npm/GitHub release using already-present app credentials and SHA-pinned actions, with no publish/deploy behavior change and failures explicitly non-blocking.


Generated for commit 243065c

Notify steps stay best-effort so a token/dispatch hiccup after npm
publish cannot skip on-chain canister deploy.

Co-authored-by: Cursor <cursoragent@cursor.com>
@Kamirus
Kamirus merged commit af3a303 into main Jul 22, 2026
28 checks passed
@Kamirus
Kamirus deleted the chore/dispatch-ic-mops-bump branch July 22, 2026 15:29
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.

1 participant