fix: add package root entrypoint - #186
Open
xianzuyang9-blip wants to merge 1 commit into
Open
Conversation
xianzuyang9-blip
had a problem deploying
to
manual-approval
June 9, 2026 02:39 — with
GitHub Actions
Failure
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
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.
Fixes #185.
The package root export points to
dist/src/index.js, but the repository did not havesrc/index.ts, sonpm run buildproduced subpath entrypoints likeruntimewhile leaving the root export missing.This adds a root entrypoint that exposes the existing public modules as namespaces (
identity,runtime,browser, andcodeInterpreter). Using namespace exports avoids collisions between shared names such asSessionInfo,DEFAULT_TIMEOUT, andWebSocketConnectionwhile making the existing root export resolve to real JS and type files.Verification:
npm run buildcompleted, butdist/src/index.jswas missing whiledist/src/runtime/index.jsexisted.npm run buildcompletes and emitsdist/src/index.js.dist/src/index.jsanddist/src/index.d.tsexist andimport('bedrock-agentcore')exposesruntime,identity,browser, andcodeInterpreter.npm test: 16 test files / 398 tests passed.Note: the local pre-commit hook ran tests and lint successfully, then stopped on an existing Prettier warning in
tests_integ/otel-no-user-content.test.ts, which this PR does not touch. I committed with--no-verifyto keep the change scoped.