release: promote dev to main#89
Conversation
fix: migrate pi-ai package scope
Co-authored-by: Genie Automagik <genie@namastex.ai>
|
Warning Review limit reached
More reviews will be available in 49 minutes and 5 seconds. Learn how PR review limits work. Your organization has run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After more reviews become available, a review can be triggered using the 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 include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available. Please see our Fair Usage Limits Policy for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (4)
📒 Files selected for processing (3)
📝 WalkthroughWalkthroughThis PR migrates imports and docs from ChangesPackage migration and version update
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
|
Warning Review the following alerts detected in dependencies. According to your organization's Security Policy, it is recommended to resolve "Warn" alerts. Learn more about Socket for GitHub.
|
There was a problem hiding this comment.
Code Review
This pull request migrates the project's dependency from the deprecated @mariozechner/pi-ai package to the newly maintained @earendil-works/pi-ai package, updating all source files, compiled outputs, documentation, and lockfiles accordingly. It also bumps the project version to 0.260526.1 and updates the minimum Node.js engine requirement to >=22.19.0. The review feedback correctly identifies that the compiled files in dist/ and the lockfiles (package-lock.json) are out of sync with the newly bumped version in package.json. Additionally, the reviewer recommends pinning the new @earendil-works/pi-ai dependency to an exact version across package.json, package-lock.json, and bun.lock to align with the project's documented risk mitigation strategy.
| { | ||
| "name": "@automagik/rlmx", | ||
| "version": "0.260409.3", | ||
| "version": "0.260425.2", |
| "": { | ||
| "name": "@automagik/rlmx", | ||
| "version": "0.260409.3", | ||
| "version": "0.260425.2", |
| @@ -1,2 +1,2 @@ | |||
| export const VERSION = '0.260424.1'; | |||
| export const VERSION = '0.260425.2'; | |||
There was a problem hiding this comment.
The compiled version in dist/src/version.js (0.260425.2) is out of sync with the source version in src/version.ts (0.260526.1). Please rebuild the project using npm run build to ensure the built artifacts are up to date.
| export const VERSION = '0.260425.2'; | |
| export const VERSION = '0.260526.1'; |
| @@ -1,2 +1,2 @@ | |||
| export declare const VERSION = "0.260424.1"; | |||
| export declare const VERSION = "0.260425.2"; | |||
There was a problem hiding this comment.
The compiled declaration version in dist/src/version.d.ts (0.260425.2) is out of sync with the source version in src/version.ts (0.260526.1). Please rebuild the project to update the built declarations.
| export declare const VERSION = "0.260425.2"; | |
| export declare const VERSION = "0.260526.1"; |
| }, | ||
| "dependencies": { | ||
| "@mariozechner/pi-ai": "0.66.1", | ||
| "@earendil-works/pi-ai": "^0.75.5", |
There was a problem hiding this comment.
To align with the project's documented risk mitigation strategy of pinning the pi-ai dependency to an exact version (to prevent unexpected breaking changes from upstream updates), consider pinning @earendil-works/pi-ai to an exact version instead of using a caret range (^).
| "@earendil-works/pi-ai": "^0.75.5", | |
| "@earendil-works/pi-ai": "0.75.5", |
| "license": "MIT", | ||
| "dependencies": { | ||
| "@mariozechner/pi-ai": "0.66.1", | ||
| "@earendil-works/pi-ai": "^0.75.5", |
There was a problem hiding this comment.
| "name": "@automagik/rlmx", | ||
| "dependencies": { | ||
| "@mariozechner/pi-ai": "0.66.1", | ||
| "@earendil-works/pi-ai": "^0.75.5", |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: b3ffef0954
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| }, | ||
| "engines": { | ||
| "node": ">=18" | ||
| "node": ">=22.19.0" |
There was a problem hiding this comment.
Sync documented Node requirement with new engine floor
Raising engines.node to >=22.19.0 here introduces a user-facing compatibility change, but the repository still documents Node.js >= 18 in README.md requirements; users following current docs can hit EBADENGINE during install (especially with engine-strict) or run unsupported runtimes. Please update the documented/runtime prerequisites in the same release so installation guidance matches this new constraint.
Useful? React with 👍 / 👎.
Summary
Promote current dev to main after RLMX pi-ai maintenance.
Included maintenance:
Verification
Notes
Disposable Genie/Hermes wish-board reflection captured in docs/hermes/day-1-world/RLMX_PIAI_MAINTENANCE_REFLECTION.md (outside this repo).
Summary by CodeRabbit
Chores
@earendil-works/pi-aiand raised Node.js engine requirement to ≥22.19.0Documentation
Tests