Add chat history API for OpenAI by pontemonti · Pull Request #157 · microsoft/Agent365-nodejs

pontemonti · 2026-01-23T07:17:52Z

No description provided.

Copilot

Pull request overview

This pull request adds chat history APIs for the OpenAI Agents SDK integration, enabling developers to send conversation history to the MCP platform for real-time threat protection without manual type conversion.

Changes:

Implements two new public methods: sendChatHistoryAsync (session-based) and sendChatHistoryMessagesAsync (message array-based) in McpToolRegistrationService
Adds automatic conversion from OpenAI AgentInputItem types to platform-native ChatHistoryMessage format with role pass-through, content extraction, UUID generation for missing IDs, and timestamp generation
Updates OpenAI Agents SDK packages from version 0.1.5 to 0.4.0 and adds uuid package dependency

Reviewed changes

Copilot reviewed 12 out of 13 changed files in this pull request and generated 7 comments.

Show a summary per file

File	Description
packages/agents-a365-tooling-extensions-openai/src/McpToolRegistrationService.ts	Core implementation of chat history APIs with conversion logic
packages/agents-a365-tooling-extensions-openai/src/index.ts	Re-exports OpenAIConversationsSession for convenience
packages/agents-a365-tooling-extensions-openai/package.json	Adds uuid and @types/uuid dependencies
packages/agents-a365-tooling-extensions-openai/docs/design.md	Documents the new chat history APIs with usage examples
tests/tooling-extensions-openai/sendChatHistoryAsync.test.ts	Comprehensive tests for session-based API (241 lines)
tests/tooling-extensions-openai/sendChatHistoryMessagesAsync.test.ts	Comprehensive tests for message array-based API (286 lines)
tests/tooling-extensions-openai/messageConversion.test.ts	Tests for message conversion logic (459 lines)
tests/tooling-extensions-openai/fixtures/mockOpenAITypes.ts	Mock types and helper functions for testing (231 lines)
tests/package.json	Updates test script paths for better reliability
pnpm-workspace.yaml	Updates OpenAI Agents SDK packages to v0.4.0
pnpm-lock.yaml	Lockfile updates for dependency changes
docs/prd/prd-openai-chat-history-api.md	Product requirements document (741 lines)
docs/tasks/prd-openai-chat-history-api.tasks.md	Implementation task breakdown (361 lines)

Files not reviewed (1)

pnpm-lock.yaml: Language not supported

… API

…ion in McpToolRegistrationService

Copilot

Pull request overview

Copilot reviewed 11 out of 12 changed files in this pull request and generated 2 comments.

Files not reviewed (1)

pnpm-lock.yaml: Language not supported

…remove validation

Copilot

Pull request overview

Copilot reviewed 11 out of 12 changed files in this pull request and generated 2 comments.

Files not reviewed (1)

pnpm-lock.yaml: Language not supported

Copilot

Pull request overview

Copilot reviewed 11 out of 12 changed files in this pull request and generated no new comments.

Files not reviewed (1)

pnpm-lock.yaml: Language not supported

Copilot · 2026-01-27T17:07:20Z

@pontemonti I've opened a new pull request, #158, to work on those changes. Once the pull request is ready, I'll request review from you.

Copilot · 2026-01-27T17:14:18Z

@pontemonti I've opened a new pull request, #159, to work on those changes. Once the pull request is ready, I'll request review from you.

#158) * Initial plan * fix: ensure MCP call is made even when messages array is empty Co-authored-by: pontemonti <7850950+pontemonti@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: pontemonti <7850950+pontemonti@users.noreply.github.com>

* Initial plan * Remove fragile 'is required' check from sendChatHistory methods Co-authored-by: pontemonti <7850950+pontemonti@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: pontemonti <7850950+pontemonti@users.noreply.github.com>

pontemonti · 2026-01-27T17:44:48Z

@copilot - after the last couple of updates, the tests are now failing. Please run all the checks just like what the PR workflow does, and then address the failures.

Copilot · 2026-01-27T17:44:58Z

@pontemonti I've opened a new pull request, #160, to work on those changes. Once the pull request is ready, I'll request review from you.

Copilot

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

…tations (#160) * Initial plan * Fix test failures: wrap session.getItems() in try-catch and update test expectations for empty arrays Co-authored-by: pontemonti <7850950+pontemonti@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: pontemonti <7850950+pontemonti@users.noreply.github.com> Co-authored-by: Johan Broberg <johan@pontemonti.net>

Copilot

Pull request overview

Copilot reviewed 11 out of 12 changed files in this pull request and generated 3 comments.

Files not reviewed (1)

pnpm-lock.yaml: Language not supported

Copilot · 2026-01-27T19:48:18Z

@pontemonti I've opened a new pull request, #163, to work on those changes. Once the pull request is ready, I'll request review from you.

…sync (#163) * Initial plan * Add error handling for convertToChatHistoryMessages and test Co-authored-by: pontemonti <7850950+pontemonti@users.noreply.github.com> * Address code review feedback: safer error handling and remove extra blank line Co-authored-by: pontemonti <7850950+pontemonti@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: pontemonti <7850950+pontemonti@users.noreply.github.com>

…s as Error type

Copilot

Pull request overview

Copilot reviewed 11 out of 12 changed files in this pull request and generated no new comments.

Files not reviewed (1)

pnpm-lock.yaml: Language not supported

Add chat history API for OpenAI

9d703b9

pontemonti requested a review from a team as a code owner January 23, 2026 07:17

Copilot AI review requested due to automatic review settings January 23, 2026 07:17

Copilot started reviewing on behalf of pontemonti January 23, 2026 07:18 View session

Copilot AI reviewed Jan 23, 2026

View reviewed changes

Johan Broberg and others added 4 commits January 26, 2026 15:48

fix: update uuid and @types/uuid dependencies to use catalog

1632fbb

fix: correct header formatting in PRD for OpenAI sendChatHistoryAsync…

b8cbce2

… API

refactor: remove unnecessary logging and improve message role validat…

d78ab70

…ion in McpToolRegistrationService

Merge branch 'main' into users/johanb/RTP_OpenAI

dfdf42f

Copilot AI review requested due to automatic review settings January 26, 2026 23:56

Copilot started reviewing on behalf of pontemonti January 26, 2026 23:57 View session

Copilot AI reviewed Jan 27, 2026

View reviewed changes

Comment thread packages/agents-a365-tooling-extensions-openai/src/McpToolRegistrationService.ts

Comment thread packages/agents-a365-tooling-extensions-openai/src/index.ts

Johan Broberg and others added 2 commits January 26, 2026 17:27

refactor: simplify role extraction in McpToolRegistrationService and …

71cd9b2

…remove validation

Merge branch 'main' into users/johanb/RTP_OpenAI

36836e9

Copilot AI review requested due to automatic review settings January 27, 2026 05:33

Copilot started reviewing on behalf of pontemonti January 27, 2026 05:34 View session

Copilot AI reviewed Jan 27, 2026

View reviewed changes

Comment thread pnpm-lock.yaml

Comment thread pnpm-lock.yaml

Add @types/uuid and uuid dependencies

77c1a5f

Copilot AI review requested due to automatic review settings January 27, 2026 06:33

Copilot started reviewing on behalf of pontemonti January 27, 2026 06:33 View session

Copilot AI reviewed Jan 27, 2026

View reviewed changes

pontemonti commented Jan 27, 2026

View reviewed changes

Comment thread packages/agents-a365-tooling-extensions-openai/src/McpToolRegistrationService.ts Outdated

Copilot AI mentioned this pull request Jan 27, 2026

Remove early return for empty messages in sendChatHistoryMessagesAsync #158

Merged

pontemonti commented Jan 27, 2026

View reviewed changes

Comment thread packages/agents-a365-tooling-extensions-openai/src/McpToolRegistrationService.ts Outdated

Copilot AI mentioned this pull request Jan 27, 2026

Remove fragile error handling from sendChatHistory methods #159

Merged

Copilot AI and others added 2 commits January 27, 2026 09:15

Copilot AI review requested due to automatic review settings January 27, 2026 17:36

Copilot AI mentioned this pull request Jan 27, 2026

Fix error handling in sendChatHistoryAsync and empty array test expectations #160

Merged

Copilot AI reviewed Jan 27, 2026

View reviewed changes

Update tests/tooling-extensions-openai/sendChatHistoryAsync.test.ts

cfa66ec

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Copilot AI review requested due to automatic review settings January 27, 2026 17:59

Copilot started reviewing on behalf of pontemonti January 27, 2026 18:00 View session

Copilot AI reviewed Jan 27, 2026

View reviewed changes

Comment thread packages/agents-a365-tooling-extensions-openai/src/McpToolRegistrationService.ts Outdated

Comment thread packages/agents-a365-tooling-extensions-openai/src/McpToolRegistrationService.ts

Comment thread packages/agents-a365-tooling-extensions-openai/src/McpToolRegistrationService.ts

pontemonti commented Jan 27, 2026

View reviewed changes

Comment thread packages/agents-a365-tooling-extensions-openai/src/McpToolRegistrationService.ts Outdated

Copilot AI mentioned this pull request Jan 27, 2026

Add error handling for message conversion in sendChatHistoryMessagesAsync #163

Merged

pontemonti mentioned this pull request Jan 27, 2026

Claude SDK: Monitor for chat history API availability #164

Open

Copilot AI mentioned this pull request Jan 27, 2026

feat: add Chat History API documentation and recommended usage for Claude SDK #165

Merged

pontemonti enabled auto-merge (squash) January 27, 2026 21:57

Refactor error handling in convertToChatHistoryMessages to cast error…

287b971

…s as Error type

Copilot AI review requested due to automatic review settings January 27, 2026 22:59

Copilot started reviewing on behalf of pontemonti January 27, 2026 22:59 View session

Copilot AI reviewed Jan 27, 2026

View reviewed changes

tmlsousa reviewed Jan 27, 2026

View reviewed changes

Comment thread packages/agents-a365-tooling-extensions-openai/src/McpToolRegistrationService.ts

mrunalhirve128 previously approved these changes Jan 28, 2026

View reviewed changes

tmlsousa previously approved these changes Jan 28, 2026

View reviewed changes

Comment thread packages/agents-a365-tooling-extensions-openai/src/McpToolRegistrationService.ts

Comment thread packages/agents-a365-tooling-extensions-openai/src/McpToolRegistrationService.ts

Merge branch 'main' into users/johanb/RTP_OpenAI

ffd7354

pontemonti dismissed stale reviews from tmlsousa and mrunalhirve128 via ffd7354 January 28, 2026 00:06

mrunalhirve128 approved these changes Jan 28, 2026

View reviewed changes

tmlsousa approved these changes Jan 28, 2026

View reviewed changes

pontemonti merged commit a598a7c into main Jan 28, 2026
7 checks passed

pontemonti deleted the users/johanb/RTP_OpenAI branch January 28, 2026 00:25

Conversation

pontemonti commented Jan 23, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Copilot AI commented Jan 27, 2026

Uh oh!

Uh oh!

Copilot AI commented Jan 27, 2026

Uh oh!

pontemonti commented Jan 27, 2026

Uh oh!

Copilot AI commented Jan 27, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI commented Jan 27, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants