Skip to content

ci: post canvas deploy reminder after every main merge - #38

Merged
HongmingWang-Rabbit merged 1 commit into
mainfrom
fix/ci-canvas-deploy-reminder
Apr 14, 2026
Merged

ci: post canvas deploy reminder after every main merge#38
HongmingWang-Rabbit merged 1 commit into
mainfrom
fix/ci-canvas-deploy-reminder

Conversation

@HongmingWang-Rabbit

Copy link
Copy Markdown
Contributor

Problem

5 consecutive merge cycles (PRs #21, #25, #30, #32, #34) went undeployed because the canvas container is not auto-rebuilt after CI passes and the manual rebuild step was silently forgotten each time.

Solution

Adds a canvas-deploy-reminder job to ci.yml. After every push to main where canvas-build passes, it posts a commit comment via the built-in GITHUB_TOKEN (no new secrets needed) with the exact commands to rebuild the container.

The comment looks like:

Canvas build passed ✅ — deploy required

The canvas container is not auto-deployed. Run this on the host machine to apply:

cd /g/personal_programs/molecule-monorepo
git pull origin main
docker compose build canvas && docker compose up -d canvas

Properties

  • No new secrets required — uses GITHUB_TOKEN with contents: write (minimum scope for commit comments)
  • Only fires on main pushesif: github.event_name == 'push' && github.ref == 'refs/heads/main'; PR builds are unaffected
  • Only fires after a clean buildneeds: canvas-build means a failing build produces no comment
  • Each merge gets one comment — on the merge commit SHA, not a PR comment, so it's always findable via git log
  • Body written via heredoc + file — avoids shell escaping issues with embedded backticks

Test plan

  • CI YAML is syntactically valid (reviewed full file)
  • After this merges to main, the next canvas-touching merge should produce a commit comment on that merge SHA
  • Verify the comment appears under the merge commit in GitHub's commit view

🤖 Generated with Claude Code

Adds a `canvas-deploy-reminder` job to ci.yml that fires on every
push to main once `canvas-build` passes. It posts a commit comment via
the built-in GITHUB_TOKEN (no new secrets needed) reminding whoever
monitors CI to run:

  cd /g/personal_programs/molecule-monorepo
  git pull origin main
  docker compose build canvas && docker compose up -d canvas

The comment includes the commit SHA and a direct link to the build log.

Rationale: 5 consecutive merge cycles (PRs #21, #25, #30, #32, #34)
went undeployed because there is no auto-deploy hook and the manual
step was silently forgotten. A commit comment on the merge commit is
the lowest-friction reminder that requires no external secrets or infra.

Does NOT run on PRs — only on direct pushes to main (i.e. post-merge).
Uses `needs: canvas-build` so the reminder only fires after build+tests
pass; a failing build produces no comment.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@HongmingWang-Rabbit

Copy link
Copy Markdown
Contributor Author

7-gate verification passed. CI 6/6 green + 1 expected skip (the new reminder job itself, only fires on push to main). Diff is workflow-only. Security: minimal permissions (contents: write for commit comments), GITHUB_TOKEN via env, heredoc body uses quoted 'BODY' delimiter preventing expansion, no user-input interpolation into run:. Design: posts commit comment only, no auto-deploy. Merging.

@HongmingWang-Rabbit
HongmingWang-Rabbit merged commit 8b33b37 into main Apr 14, 2026
7 checks passed
@HongmingWang-Rabbit
HongmingWang-Rabbit deleted the fix/ci-canvas-deploy-reminder branch April 14, 2026 09:20
molecule-ai Bot pushed a commit that referenced this pull request Apr 21, 2026
ci: post canvas deploy reminder after every main merge
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