Skip to content

chore: standardize package.json scripts#220

Merged
ianmacartney merged 1 commit intomainfrom
update-package-scripts
Feb 3, 2026
Merged

chore: standardize package.json scripts#220
ianmacartney merged 1 commit intomainfrom
update-package-scripts

Conversation

@ianmacartney
Copy link
Member

@ianmacartney ianmacartney commented Feb 3, 2026

Summary

  • Add build:codegen and build:clean scripts
  • Update dev:build to use npm run build:codegen
  • Standardize predev with path-exists check
  • Add preversion and prepublishOnly hooks
  • Simplify alpha/release scripts
  • Update version script to use vim pattern

Aligning with the template-component pattern.

Summary by CodeRabbit

  • Chores
    • Reorganized build and development scripts to streamline the development workflow.
    • Updated versioning and publishing hooks to establish CI/build prerequisites.
    • Modified changelog handling in release processes.

Note: These changes are internal build and development infrastructure improvements with no impact on user-facing features or functionality.

- Add build:codegen and build:clean scripts
- Update dev:build to use npm run build:codegen
- Standardize predev with path-exists check
- Add preversion and prepublishOnly hooks
- Simplify alpha/release scripts
- Update version script to use vim pattern
@coderabbitai
Copy link

coderabbitai bot commented Feb 3, 2026

📝 Walkthrough

Walkthrough

Package.json npm scripts refactored to separate codegen from build step, introduce new build:codegen and build:clean scripts, modify predev workflow to depend on dist artifacts, and add preversion/prepublishOnly hooks for CI prerequisites and authentication management.

Changes

Cohort / File(s) Summary
Build and Dev Workflow Scripts
package.json
Separated codegen into distinct build:codegen step; added build:clean script for artifact purging; modified dev:build to run codegen separately; updated predev to require built dist artifact when local env absent; introduced preversion and prepublishOnly hooks for CI/auth prerequisites; refactored prepare, alpha, release, and version scripts for new versioning/publishing flows.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐰 Hop, hop, the build flows bright,
Codegen separated—organized right!
New scripts spring to life with care,
Clean builds and versioning everywhere,
A rabbit's dance through CI's pipeline air! 🏗️

🚥 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 'chore: standardize package.json scripts' directly and accurately describes the main change—restructuring and standardizing npm scripts in package.json.
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 unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch update-package-scripts

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 3, 2026

Open in StackBlitz

npm i https://pkg.pr.new/get-convex/agent/@convex-dev/agent@220

commit: 96242ca

@ianmacartney ianmacartney merged commit 5dfefff into main Feb 3, 2026
3 checks passed
@ianmacartney ianmacartney deleted the update-package-scripts branch February 3, 2026 05:05
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.

2 participants