Skip to content

chore(graph-utils): export GraphClient base class#2429

Closed
TalZaccai wants to merge 1 commit into
mainfrom
talzacc/graph-utils-export-graphclient
Closed

chore(graph-utils): export GraphClient base class#2429
TalZaccai wants to merge 1 commit into
mainfrom
talzacc/graph-utils-export-graphclient

Conversation

@TalZaccai
Copy link
Copy Markdown
Contributor

@TalZaccai TalZaccai commented Jun 3, 2026

Make the GraphClient base class available alongside CalendarClient / MailClient (which already extend it) so downstream agents can build new Microsoft Graph–backed clients without copying the auth/transport plumbing.

Change

ts/packages/agents/agentUtils/graphUtils/src/index.ts — add GraphClient to the existing named re-export from ./graphClient.js.

Risk

Additive-only. No existing consumer breaks because the export was previously narrower; all current callers use the more specific subclasses or ErrorResponse.

Test

  • pnpm --filter graph-utils build
  • pnpm --filter graph-utils prettier
  • Verified GraphClient shows up in dist/index.d.ts after the change.

Make the GraphClient base class available alongside CalendarClient/MailClient (which already extend it) so downstream agents can build new Microsoft Graph-backed clients without copying the auth/transport plumbing.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

Copilot AI left a 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 broadens the public API surface of the graph-utils package by exporting the shared GraphClient base class from the package entrypoint, enabling downstream agents to build additional Microsoft Graph–backed clients without duplicating auth/transport logic.

Changes:

  • Re-export GraphClient (alongside ErrorResponse) from src/index.ts so it becomes available via the package root export.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@TalZaccai
Copy link
Copy Markdown
Contributor Author

Superseded by a combined PR that adds the actual motivation for exporting GraphClient: a GraphClientOptions parameter so subclasses can target a different AAD app (custom env-var prefix, persisted auth-record filename, token-cache name, and OAuth scopes). Re-opening as a single PR from talzacc/graph-client-options-refactor since the export alone has no in-tree consumer.

@TalZaccai TalZaccai closed this Jun 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants