fix: update import statement in elizaos.js to use package alias#6389
fix: update import statement in elizaos.js to use package alias#6389
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the ✨ 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 |
|
Claude encountered an error —— View job I'll analyze this and get back to you. |
|
Claude encountered an error —— View job I'll analyze this and get back to you. |
|
I've updated the doc to ensure we use cli (as we always did): elizaOS/docs#83 |
Greptile Overview
Greptile Summary
Updated the import statement in
elizaos.jsto use the package alias (@elizaos/cli) instead of the direct path (@elizaos/cli/dist/index.js). This follows Node.js best practices by relying on the package's exports field defined in@elizaos/cli/package.json, which maps.to./dist/index.js.Changes:
Confidence Score: 5/5
@elizaos/clipackage correctly defines its exports in package.json, mapping the default export to dist/index.js. This is a clean improvement that enhances maintainability without changing behavior.Important Files Changed
Sequence Diagram
sequenceDiagram participant User participant elizaos as elizaos package participant cli as @elizaos/cli package participant index as dist/index.js User->>elizaos: Run `elizaos` command elizaos->>elizaos: Load bin/elizaos.js Note over elizaos: import '@elizaos/cli' elizaos->>cli: Resolve package alias cli->>cli: Check package.json exports Note over cli: "." → "./dist/index.js" cli->>index: Execute CLI entry point index->>User: Run CLI command