fix(llm.do): fix broken npm entrypoint by adding files and type module#2430
Open
codeboost-tr wants to merge 2 commits into
Open
fix(llm.do): fix broken npm entrypoint by adding files and type module#2430codeboost-tr wants to merge 2 commits into
codeboost-tr wants to merge 2 commits into
Conversation
This change adds the 'files' field to ensure that source and build artifacts are included in the published npm package. It also adds 'type': 'module' to correctly identify the package as an ES module, matching the published exports metadata.
|
@codeboost-tr is attempting to deploy a commit to the do Team on Vercel. A member of the Team first needs to authorize it. |
Author
|
Hello! This PR fixes the broken npm entrypoint issue. It's ready for review and deployment authorization. Thanks! |
- Fixed broken npm entrypoint by pointing to dist/index.js - Added missing dependencies (ai-sdk, zod, language-models) - Added build and build:packages scripts using tsc - Created tsconfig.json extending the project's base sdk config - Fixed absolute imports and portable type errors - Added README.md and LICENSE to the package
Author
|
@nathanclevenger — quick ping: the Vercel deployment check on this PR is blocked because the do team hasn't authorized the GitHub app. Could a team member click through https://vercel.com/git/authorize?team=do&slug=dotdo so the deploy can run? The npm entrypoint fix is ready for review once CI is unblocked. Thanks! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR fixes a broken root entrypoint in the published
llm.donpm package.The issue was that the published tarball was missing the
dist/(orsrc/) directory because thefilesfield was missing inpackage.json. Additionally, adding"type": "module"ensures the package is correctly treated as ESM.Changes:
filesarray to includesrc,dist,README.md, andLICENSE."type": "module".Linked Issue: charles-openclaw/charles-microbounties#628