Skip to content

Move RAG releases to staged CI publishing - #121

Merged
ianmacartney merged 2 commits into
mainfrom
ian/release-from-ci
Nov 10, 2026
Merged

ianmacartney merged 2 commits into
mainfrom
ian/release-from-ci

Conversation

@ianmacartney

@ianmacartney ianmacartney commented Nov 10, 2026 •

Copy link
Copy Markdown
Member

Port the merged Workflow release pattern to RAG. release.yml, version.yml, Changesets configuration, and workflow linting are copied unchanged. The shared scripts are build, fast check, and test:e2e; RAG's check retains its circular-dependency check, and test:e2e runs an anonymous Convex deployment smoke test that generates code. PR CI runs fast checks before the smoke test and verifies committed component-generated files.

Verified: npm ci, build, 114 tests, typecheck, lint, circular-dependency check, Changesets status from a clean commit, npm pack dry run, and a clean-checkout anonymous smoke test with unchanged generated files. Both GitHub PR workflows pass against current main. The existing untracked todo.md was left untouched. GitHub Actions already allows release PR creation.

Before the first staged release, configure npm trusted publishing for @convex-dev/rag with workflow filename release.yml (npm requires the first successful publish within two days of registration). This PR has no version bump, so merging it should not stage a package.

@pkg-pr-new

pkg-pr-new Bot commented Nov 10, 2026 •

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/convex/rag/@convex-dev/rag@121

commit: 910bdd1

@ianmacartney
ianmacartney marked this pull request as ready for review November 10, 2026 22:55

@cubic-dev-ai cubic-dev-ai Bot left a comment •

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

All reported issues were addressed across 9 files

Reply with feedback, questions, or to request a fix.

Re-trigger cubic

Comment thread .github/workflows/lint-workflows.yml Resolved
Comment thread CONTRIBUTING.md Resolved

Copy link
Copy Markdown
Member Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@coderabbitai

coderabbitai Bot commented Nov 10, 2026 •

Copy link
Copy Markdown

Explain Change →

No actionable comments were generated in the recent review.

Walkthrough

The changes add Changesets configuration, package scripts, and release guidance. The test workflow checks package changes and generated files. A new workflow prepares release PRs, and another selects, tests, stages, and tags package releases. The workflow lint actions are also updated.

Sequence Diagram(s)

sequenceDiagram
  participant ReleaseWorkflow
  participant PlanJob
  participant BuildJob
  participant StageJob
  participant Npm
  participant TagJob
  participant GitRepository
  ReleaseWorkflow->>PlanJob: Select and validate source commit and version
  PlanJob->>BuildJob: Export selected commit, package, version, and channel
  BuildJob->>StageJob: Upload tested package tarball and SHA-256 digest
  StageJob->>Npm: Stage verified tarball with provenance and channel
  StageJob->>TagJob: Continue after staging
  TagJob->>GitRepository: Create or verify version tag at source commit
Loading

Change: Feature

Merge Risk: Minimal · up to 49f50

This PR moves releases to staged CI publishing with Changesets. No merge-blocking defect was found. Before merging, configure npm trusted publishing for the release workflow as the PR description states.

Pre-merge checks: Passed 4

Check name Status Explanation
Title check Passed The title clearly describes the main change: moving RAG releases to staged CI publishing.
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Description check Passed The description explains the Changesets-based release workflow, CI changes, validation performed, and npm trusted-publishing setup. It is directly related to the changeset.
Finishing Touches

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

Recent review info

Run configuration

  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Central YAML; Path: .coderabbit.yaml
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 43eedf14-301f-412d-91df-879cffb7f4db
Commits

Reviewing files that changed from the base of the PR and between 0642974 and 49f5009.

Files ignored due to path filters: 1

  • package-lock.json is excluded by !**/package-lock.json
Files selected for processing: 8

  • .changeset/README.md
  • .changeset/config.json
  • .github/workflows/lint-workflows.yml
  • .github/workflows/release.yml
  • .github/workflows/test.yml
  • .github/workflows/version.yml
  • CONTRIBUTING.md
  • package.json

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

@ianmacartney
ianmacartney merged commit f592bcc into main Nov 10, 2026
8 checks passed
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