-
Notifications
You must be signed in to change notification settings - Fork 5
[Node.js] Align Public APIs across SDKs #221
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from all commits
Commits
Show all changes
17 commits
Select commit
Hold shift + click to select a range
ff63646
feat: align InvokeAgentDetails with .NET/Python SDKs
jsl517 c3eab6b
fix: address Copilot PR review comments and fix all test failures
jsl517 b0d099d
feat: add source files for scope API alignment
jsl517 b14fe0a
refactor: address PR review — rename types and unify Request interface
jsl517 fe81431
docs: update CHANGELOG and design doc for type renames
jsl517 3b9bb82
refactor: rename callerInfo → callerDetails on InvokeAgentScope
jsl517 8a03a7b
refactor: make request required on all scopes, rename callerInfo
jsl517 c9b838e
refactor: make request optional on Inference/ExecuteTool/OutputScope
jsl517 0e04886
fix: add tenantId check in OutputLoggingMiddleware, remove try/catch
jsl517 0bf75ea
fix: add defensive null check for InvokeAgentScope.details, add servi…
jsl517 0e4308f
docs: add CallerDetails migration JSDoc, fix baggage key table in des…
jsl517 ae8d7dd
refactor: make request first required param on OutputScope.start(), a…
jsl517 20e8508
test: populate request object in all scope tests for realistic coverage
jsl517 d91fe81
refactor: split InvokeAgentDetails into InvokeAgentScopeDetails + Age…
jsl517 476311d
refactor: flatten TenantDetails into AgentDetails, rename InvokeAgent…
jsl517 3a7bdb6
Merge remote-tracking branch 'origin/main' into users/pefan/syncsdk
jsl517 55a29bc
refactor: address PR review — remove tenantDetails from base scope, f…
jsl517 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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
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
Oops, something went wrong.
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.