Skip to content

chore(ci): use canonical CarryCtx snapshot publish script - #16

Merged
Xuepoo merged 1 commit into
mainfrom
cmd/workflow-publish-2290
Oct 20, 2026
Merged

Xuepoo merged 1 commit into
mainfrom
cmd/workflow-publish-2290

Conversation

@Xuepoo

@Xuepoo Xuepoo commented Oct 20, 2026 •

Copy link
Copy Markdown
Contributor

Priority: P2 | Area: release | Labels: chore,P2,area:release | Milestone: 0.0.25 | RFC: — | Task: bitty#2290

Summary

just workflow-publish inlined a bare carryctx export --publication plus git push, so a clone whose local refs/heads/carryctx-snapshots lagged the remote could produce a non-fast-forward push, and a stale checkout could publish a source revision the Snapshot source gate rejects.

  • Vendors the canonical scripts/workflow-publish.sh (byte-identical to bitty-ai/scripts/workflow-publish.sh, sha256 4c3338457588b6988d9b0dfb4f049c117d39c74a8c29552352c8f9678be5247e).
  • Points just workflow-publish at the script; the recipe no longer embeds publication logic.
  • Leaves .github/workflows/snapshot-source.yml unchanged.

The script aligns local refs/heads/carryctx-snapshots to refs/remotes/origin/carryctx-snapshots before exporting, refuses a HEAD that is not reachable from origin/main, verifies manifest.redacted, and pushes only when the ref advanced.

Verification

  • bash -n, shellcheck -S warning, and shfmt -d pass.
  • actionlint -shellcheck= passes.
  • scripts/workflow-publish.sh --dry-run from a detached origin/main checkout passes (no ref write, no push).
  • just check passes.

Refs bitty#2290

Summary by CodeRabbit

  • New Features
    • Added a publication workflow for redacted CarryCtx snapshots.
    • Added validation for required tools, project settings, checkout provenance, and snapshot metadata.
    • Added dry-run support to preview publication without updating or pushing references.
    • Added clear reporting for successful updates, no-op publications, and failures.
  • Refactor
    • Moved snapshot publication steps into a dedicated workflow script for more consistent execution.

@Xuepoo Xuepoo added this to the 0.0.25 milestone Oct 20, 2026
@Xuepoo Xuepoo added chore area:release Release preparation P2 Priority 2 labels Oct 20, 2026
@coderabbitai

coderabbitai Bot commented Oct 20, 2026

Copy link
Copy Markdown

Review in Change Stack →

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: f9f75ffd-06e3-45b3-9662-b88128da4478
📥 Commits

Reviewing files that changed from the base of the PR and between fed101a and fa72840.

📒 Files selected for processing: 2
  • justfile
  • scripts/workflow-publish.sh
+-----------------------------------------------------------------------+
| Simplicity does not precede complexity, but follows it. - Alan Perlis |
+-----------------------------------------------------------------------+
  \
   \     /)/)
        ( '')
        /  |
      o(___)
✨ Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@Xuepoo
Xuepoo merged commit 1c51609 into main Oct 20, 2026
9 of 10 checks passed
@Xuepoo
Xuepoo deleted the cmd/workflow-publish-2290 branch October 20, 2026 11:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:release Release preparation chore P2 Priority 2

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant