-
Notifications
You must be signed in to change notification settings - Fork 0
Jabidahs/typescript-sdk #1
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
Jabidahs/typescript-sdk #1
Conversation
Co-authored-by: Felix Weinberger <3823880+felixweinberger@users.noreply.github.com>
Co-authored-by: David Soria Parra <167242713+dsp-ant@users.noreply.github.com>
Co-authored-by: David Soria Parra <167242713+dsp-ant@users.noreply.github.com>
…and add explicit resourceServerUrl (RFC 9728) (#858) Co-authored-by: Eugene <eugene@blust.ai>
…ul authentication (#945) Co-authored-by: Vinicius Costa <viniciuscsouza@yahoo.com.br> Co-authored-by: Tyler James Leonhardt <2644648+TylerLeonhardt@users.noreply.github.com> Co-authored-by: Blust.AI <159488814+blustAI@users.noreply.github.com> Co-authored-by: Eugene <eugene@blust.ai>
…ntationSchema use IconsSchema (#947)
Co-authored-by: ihrpr <inna@anthropic.com>
…f stateless HTTP, explain tools v resources v prompts (#980)
…#1216) Co-authored-by: Felix Weinberger <fweinberger@anthropic.com>
Co-authored-by: Claude <noreply@anthropic.com>
Co-authored-by: Konstantin Konstantinov <konstantin@mach5technology.com>
Co-authored-by: Konstantin Konstantinov <KKonstantinov@users.noreply.github.com>
Co-authored-by: Claude <noreply@anthropic.com>
…up across 1 directory (#1228) Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We failed to fetch the diff for pull request #1
You can try again by commenting this pull request with @sourcery-ai review, or contact us for help.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR appears to be a large-scale reformatting and modernization effort for the TypeScript SDK, focusing on code style consistency, import standardization, and the addition of new example implementations. The changes include switching to single quotes, removing unused imports, standardizing formatting, and adding comprehensive examples for elicitation, OAuth, and task-based workflows.
Key Changes:
- Standardized code formatting (single quotes, consistent spacing, arrow function syntax)
- Added extensive examples for URL and form elicitation patterns
- Introduced task-based execution examples with interactive client/server demonstrations
- Enhanced OAuth support with client credentials and upscoping capabilities
- Improved SSE polling and reconnection logic with server-provided retry intervals
Reviewed changes
Copilot reviewed 64 out of 159 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| src/experimental/tasks/client.ts | New experimental client task API with streaming support and validation |
| src/experimental/index.ts | Export entry point for experimental features |
| src/examples/shared/inMemoryEventStore.ts | Formatting changes to event store implementation |
| src/examples/server/*.ts | Multiple new server examples and formatting updates across all server examples |
| src/examples/client/*.ts | New client examples for elicitation, OAuth, and interactive tasks |
| src/client/*.ts | Formatting standardization and auth improvements across client transports |
| src/fixtures/zodTestMatrix.ts | New test fixture for Zod version compatibility testing |
| src/mocks/pkce-challenge.ts | Formatting update to mock implementation |
| src/cli.ts | File removed (deprecated CLI implementation) |
Comments suppressed due to low confidence (1)
src/client/streamableHttp.ts:1
- This import was added to simpleStreamableHttp.ts but the actual Ajv usage is not visible in the diff. If Ajv is not being used in this file, this import should be removed to avoid unused dependencies.
import { Transport, FetchLike, createFetchWithInit, normalizeHeaders } from '../shared/transport.js';
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
Now there is more lots more lots to build and Agent Army of Git's and M365 and Co Pilots, now fix , tell where you think READ ME's. I hope there sometbing good @copilot. I am building empower and engage the moment in time, perform safe and reliable repectful , be understanding to people and we are are all made accountable to the system, was built to a light , yo be understandable to all peoples. There the six principle's of responsible for AI's and humans, code of conduct. |
jarlungoodoo73
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Use all the ad much as mine as u can, unused. Resposities put into my repo's and archived.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 64 out of 159 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
Lookie's it's blowing out. Red is winning. I'II see red soon. 😅🤣 |
1 similar comment
|
Lookie's it's blowing out. Red is winning. I'II see red soon. 😅🤣 |
configuration-templates.md
contributing-to-renovate.md
docker.md
faq.md
golang.md
index.md
java.md
javascript.md
known-limitations.md
0
language-constraints-and-upgrading.md
logo-brand-guidelines.md
merge-confidence.md
node.md
noise-reduction.md
nuget.md
opentelemetry.md