chore: update CI sample update script (#531)

[ymc9]

No description provided.

[coderabbitai]

Warning

Rate limit exceeded

@ymc9 has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 18 minutes and 16 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📥 Commits

Reviewing files that changed from the base of the PR and between 2966af1 and 0e14011.

📒 Files selected for processing (1)

• .github/workflows/update-samples.yml

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch dev

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[claude]

Claude finished @ymc9's task —— View job

---

Code Review for PR #531 ✅

Reviewing the CI sample update script changes...

Todo List

• Read and analyze the modified workflow file
• Review for code quality and best practices
• Check for potential bugs or issues
• Evaluate performance considerations
• Identify any security concerns
• Assess test coverage impact
• Provide comprehensive feedback

Review Summary

This PR makes a focused change to the sample repository update workflow, switching from @next to @latest package versions.

Changes Overview

The PR updates .github/workflows/update-samples.yml with the following changes:

• Step name: "Update @zenstackhq packages to next" → "Update @zenstackhq packages to latest"
• Comment: "Update each package to next tag" → "Update each package to latest tag"
• Command: npm install "$pkg@next" → npm install "$pkg@latest"
• Commit message: "chore: update @zenstackhq packages to next release" → "chore: update @zenstackhq packages to latest release"

Code Quality & Best Practices ✅

• Consistent naming: All references to package version tags are consistently updated
• Clear intent: The change clearly shifts from pre-release (@next) to stable release (@latest) distribution
• Maintainability: The workflow structure remains clean and readable

Potential Issues & Bugs ✅

• No functional bugs detected: The changes are purely cosmetic/semantic
• Logic consistency: The workflow logic remains unchanged, only the target version tag changes
• Syntax validity: All YAML syntax and bash commands remain valid

Performance Considerations ✅

• No performance impact: The change from @next to @latest doesn't affect workflow execution time
• Network efficiency: Still fetches the same number of packages, just different versions

Security Concerns ✅

• No security implications: The change doesn't introduce new security risks
• Token usage: Continues to properly use PAT_TOKEN for repository access
• Permissions: Maintains appropriate contents: read permissions

Test Coverage Impact ✅

• No test coverage changes needed: This is a CI/deployment workflow change
• Workflow testing: The workflow will be tested when it runs on the next release

Recommendations

1. Consider version pinning: While @latest is more stable than @next, consider if you want to pin to specific versions for more predictable updates
2. Release timing: Ensure this change aligns with your release strategy - sample repos will now get stable releases instead of pre-release versions
3. Documentation: Consider updating any documentation that references the sample update process

Overall Assessment

This is a low-risk, high-value change that makes the sample repository update process more stable by targeting stable releases instead of pre-release versions. The implementation is clean, consistent, and follows GitHub Actions best practices.

Approval recommendation: ✅ Approve - This change is ready to merge.

---

[Copilot]

Pull request overview

This PR updates the CI workflow that automatically updates sample repositories to use the latest tag instead of next when installing @zenstackhq packages. This changes the update strategy from pre-release versions to stable releases.

• Changed package installation from @next to @latest tag
• Updated all related comments, echo messages, and commit messages for consistency

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.