Skip to content

chore(ci): update workflows for pull request and main branch snapshots#71

Open
Adammatthiesen wants to merge 3 commits intomainfrom
adam/new-snapshot-ci
Open

chore(ci): update workflows for pull request and main branch snapshots#71
Adammatthiesen wants to merge 3 commits intomainfrom
adam/new-snapshot-ci

Conversation

@Adammatthiesen
Copy link
Member

@Adammatthiesen Adammatthiesen commented Feb 28, 2026

Replace current workflows for snapshots with matching ones from main repo

Summary by CodeRabbit

  • Chores
    • Updated CI/CD workflow configurations to streamline preview release processes.
    • Refined workflow triggers and job conditions for pull requests and main branch deployments.
    • Adjusted build configuration and dependency handling settings.

@changeset-bot
Copy link

changeset-bot bot commented Feb 28, 2026

⚠️ No Changeset found

Latest commit: cd457b7

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@coderabbitai
Copy link

coderabbitai bot commented Feb 28, 2026

📝 Walkthrough

Walkthrough

The PR refactors the CI/CD pipeline to adopt an external preview workflow system from withstudiocms/automations, replacing the previous snapshot release approach. Changes include updating the changeset base branch reference, restructuring PR workflows with conditional bot-based routing, adding a preview job to the main push workflow, and adjusting dependency ignore configurations accordingly.

Changes

Cohort / File(s) Summary
Changeset Configuration
.changeset/config.json
Updated baseBranch reference from "main" to "origin/main" to align with external workflow expectations.
Workflow Pipeline Refactoring
.github/workflows/ci-pr-snapshots.yml, .github/workflows/ci-push-main.yml
Consolidated PR snapshot workflow into two conditional bot-aware jobs (preview-changeset-release and preview-pr) and added preview-main job to push workflow. Both delegate to external preview.yml workflow with build-cmd configuration.
Configuration & Script Updates
knip.config.ts, package.json
Removed legacy ci:snapshot script from package.json and updated knip config to ignore 'pkg-pr-new' dependency while removing ignoreBinaries configuration.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~22 minutes

Possibly related PRs

Suggested labels

ci

Suggested reviewers

  • louisescher
  • dreyfus92

Poem

🐰 Hopping through workflows with glee,
External previews dance wild and free,
Bot-aware branching, so clean and bright,
Snapshots reborn in automation's light!

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main changes: updating CI workflows for pull request and main branch snapshots, which is the primary focus across multiple workflow files.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch adam/new-snapshot-ci

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 and usage tips.

@pkg-pr-new
Copy link

pkg-pr-new bot commented Feb 28, 2026

npm i https://pkg.pr.new/@studiocms/cfetch@71

commit: 157cc97

@Adammatthiesen Adammatthiesen marked this pull request as ready for review February 28, 2026 07:00
@Adammatthiesen Adammatthiesen requested a review from a team as a code owner February 28, 2026 07:00
@coderabbitai coderabbitai bot added the ci label Feb 28, 2026
Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 2

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In @.github/workflows/ci-pr-snapshots.yml:
- Around line 5-9: Edit the paths-ignore list in the ci-pr-snapshots.yml
workflow: remove the ".github/**/**" entry so workflow-only PRs are not skipped,
and replace the incorrect glob "**/**.md" with the correct "**/*.md"; keep the
other ignore entries intact and ensure YAML indentation/format remains valid.

In @.github/workflows/ci-push-main.yml:
- Around line 75-85: The preview-main job currently lacks a fork-guard and will
run on fork pushes; mirror the mergebot guard by adding the same "if" condition
used there to the preview-main job (identify the job by name preview-main) so it
only runs for the upstream owner (e.g., github.repository_owner ==
'withstudiocms' or the same expression used in mergebot); update the
preview-main job's top-level config to include that if-condition and ensure
secrets/uses remain unchanged.

ℹ️ Review info

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 149aeae and cd457b7.

📒 Files selected for processing (5)
  • .changeset/config.json
  • .github/workflows/ci-pr-snapshots.yml
  • .github/workflows/ci-push-main.yml
  • knip.config.ts
  • package.json
💤 Files with no reviewable changes (1)
  • package.json

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant