Implement Agent Settings SDK for Node.js/TypeScript [SDK Parity] [DO NOT review] [DO NOT merge] #87

Copilot · 2025-11-24T22:46:32Z

Original prompt

This section details on the original issue you should resolve

<issue_title>[SDK Parity] Node.js/TypeScript for PR 110</issue_title>
<issue_description>## SDK Parity Update Required

This issue was automatically created by the AI-First workflow to maintain SDK parity.

Source PR Information

Original PR: #110 - Implement Agent Settings SDK for .NET [DO NOT Review, DO NOT Merge]
PR URL: microsoft/Agent365-dotnet#110
PR Author: @copilot

PR Description

Adds SDK support for managing agent settings templates by agent type and agent instance settings via the Agent 365 platform API.

New Package: `Microsoft.Agents.A365.Settings`

Models

AgentSettingsTemplate - Settings template for an agent type
AgentSettings - Settings for a specific agent instance
AgentSettingProperty - Individual setting with name, value, type, required flag, description

Service Interface

IAgentSettingsService - CRUD operations for templates and instance settings

API Endpoints

Method	Endpoint	Description
GET/PUT	`/agents/types/{agentType}/settings/template`	Template by agent type
GET/PUT	`/agents/{agentInstanceId}/settings`	Settings by instance

Usage

// Register via DI
services.AddHttpClient<IAgentSettingsService, AgentSettingsService>();

// Get template
var template = await settingsService.GetSettingsTemplateByAgentTypeAsync("custom-agent", authToken);

// Set instance settings
var settings = new AgentSettings
{
    AgentInstanceId = "instance-123",
    Properties = new List<AgentSettingProperty>
    {
        new() { Name = "maxRetries", Value = "3", Type = "integer", Required = true }
    }
};
await settingsService.SetSettingsByAgentInstanceAsync("instance-123", settings, authToken);

Configuration

MCP_PLATFORM_ENDPOINT - Override platform base URL (validated for proper URI format)

Tests

24 unit tests covering service methods, model serialization, and error handling.

Original prompt

Implementing the Agents Settings for .Net SDK.

See attached swagger file for more information on this API.

This currently API supports:

GET or SET agent setting template by agent type

GET or SET agent settings by agent instance.

https://github.com/user-attachments/files/23107469/swagger.json

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Parity Task

Changes detected in: csharp SDK
Target SDK: Node.js/TypeScript (nodejs/)

Action Required

Please apply the changes described in the original PR (#110) to the Node.js/TypeScript SDK to maintain feature parity across all SDKs.

Before implementing:

First, check if this feature already exists in the Node.js/TypeScript SDK
If the feature is already implemented, close this issue with the wontfix label and add a comment explaining that parity already exists

For bug fixes (IMPORTANT):

Bug fixes require extra caution - only act if the same bug exists in the Node.js/TypeScript SDK
Carefully analyze whether the bug described in the original PR replicates in the target platform
If the bug does NOT exist in the Node.js/TypeScript SDK, close this issue with the wontfix label and explain that the bug is not present
Stay strictly in context - only fix the specific bug described, do not make additional changes or improvements
Do not port bug fixes that are specific to the source SDK's language or platform constraints

When creating the PR for this implementation:

Assign the PR to @copilot (the author of the original PR)
Reference this issue in the PR description
Ensure the PR description explains what parity changes were made

Implementation Guidelines for Node.js/TypeScript

When implementing this feature in the TypeScript SDK:

Review existing code in nodejs/ to understand the codebase structure and patterns
Follow the coding style, naming conventions, and architectural patterns used in the TypeScript SDK
Check for configuration files (e.g., .eslintrc.json, tsconfig.json, .prettierrc) that define coding standards
Ensure your implementation is consistent with existing SDK components
Include appropriate tests following the testing patterns in the TypeScript SDK

Related Information

Workflow Run: https://github.com/microsoft/Agent365-dotnet/actions/runs/19651795880
Trigger: pull_request
Branch: 110/merge
Commit: 7b344e7625bae456cd4485a91d0fc693ed0cea0f

This is an automated issue. Please close with wontfix label if parity is not needed.</issue_descr...

Fixes [SDK Parity] Node.js/TypeScript for PR 110 #86

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Parity Task

Changes detected in: csharp SDK
Target SDK: Node.js/TypeScript (nodejs/)

Action Required

Please apply the changes described in the original PR (#110) to the Node.js/TypeScript SDK to maintain feature parity across all SDKs.

Before implementing:

First, check if this feature already exists in the Node.js/TypeScript SDK
If the feature is already implemented, close this issue with the wontfix label and add a comment explaining that parity already exists

For bug fixes (IMPORTANT):

Bug fixes require extra caution - only act if the same bug exists in the Node.js/TypeScript SDK
Carefully analyze whether the bug described in the original PR replicates in the target platform
If the bug does NOT exist in the Node.js/TypeScript SDK, close this issue with the wontfix label and explain that the bug is not present
Stay strictly in context - only fix the specific bug described, do not make additional changes or improvements
Do not port bug fixes that are specific to the source SDK's language or platform constraints

When creating the PR for this implementation:

Assign the PR to @copilot (the author of the original PR)
Reference this issue in the PR description
Ensure the PR description explains what parity changes were made

Implementation Guidelines for Node.js/TypeScript

When implementing this feature in the TypeScript SDK:

Review existing code in nodejs/ to understand the codebase structure and patterns
Follow the coding style, naming conventions, and architectural patterns used in the TypeScript SDK
Check for configuration files (e.g., .eslintrc.json, tsconfig.json, .prettierrc) that define coding standards
Ensure your implementation is consistent with existing SDK components
Include appropriate tests following the testing patterns in the TypeScript SDK

Related Information

Workflow Run: https://github.com/microsoft/Agent365-dotnet/actions/runs/19651795880
Trigger: pull_request
Branch: 110/merge
Commit: 7b344e7625bae456cd4485a91d0fc693ed0cea0f

This is an automated issue. Please close with wontfix label if parity is not needed.</issue_descr...

Fixes [SDK Parity] Node.js/TypeScript for PR 110 #86

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Parity Task

Changes detected in: csharp SDK
Target SDK: Node.js/TypeScript (nodejs/)

Action Required

Please apply the changes described in the original PR (#110) to the Node.js/TypeScript SDK to maintain feature parity across all SDKs.

Before implementing:

First, check if this feature already exists in the Node.js/TypeScript SDK
If the feature is already implemented, close this issue with the wontfix label and add a comment explaining that parity already exists

For bug fixes (IMPORTANT):

Bug fixes require extra caution - only act if the same bug exists in the Node.js/TypeScript SDK
Carefully analyze whether the bug described in the original PR replicates in the target platform
If the bug does NOT exist in the Node.js/TypeScript SDK, close this issue with the wontfix label and explain that the bug is not present
Stay strictly in context - only fix the specific bug described, do not make additional changes or improvements
Do not port bug fixes that are specific to the source SDK's language or platform constraints

When creating the PR for this implementation:

Assign the PR to @copilot (the author of the original PR)
Reference this issue in the PR description
Ensure the PR description explains what parity changes were made

Implementation Guidelines for Node.js/TypeScript

When implementing this feature in the TypeScript SDK:

Review existing code in nodejs/ to understand the codebase structure and patterns
Follow the coding style, naming conventions, and architectural patterns used in the TypeScript SDK
Check for configuration files (e.g., .eslintrc.json, tsconfig.json, .prettierrc) that define coding standards
Ensure your implementation is consistent with existing SDK components
Include appropriate tests following the testing patterns in the TypeScript SDK

Related Information

Workflow Run: https://github.com/microsoft/Agent365-dotnet/actions/runs/19651795880
Trigger: pull_request
Branch: 110/merge
Commit: 7b344e7625bae456cd4485a91d0fc693ed0cea0f

This is an automated issue. Please close with wontfix label if parity is not needed.</issue_descr...

Fixes [SDK Parity] Node.js/TypeScript for PR 110 #86

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Parity Task

Changes detected in: csharp SDK
Target SDK: Node.js/TypeScript (nodejs/)

Action Required

Please apply the changes described in the original PR (#110) to the Node.js/TypeScript SDK to maintain feature parity across all SDKs.

Before implementing:

First, check if this feature already exists in the Node.js/TypeScript SDK
If the feature is already implemented, close this issue with the wontfix label and add a comment explaining that parity already exists

For bug fixes (IMPORTANT):

Bug fixes require extra caution - only act if the same bug exists in the Node.js/TypeScript SDK
Carefully analyze whether the bug described in the original PR replicates in the target platform
If the bug does NOT exist in the Node.js/TypeScript SDK, close this issue with the wontfix label and explain that the bug is not present
Stay strictly in context - only fix the specific bug described, do not make additional changes or improvements
Do not port bug fixes that are specific to the source SDK's language or platform constraints

When creating the PR for this implementation:

Assign the PR to @copilot (the author of the original PR)
Reference this issue in the PR description
Ensure the PR description explains what parity changes were made

Implementation Guidelines for Node.js/TypeScript

When implementing this feature in the TypeScript SDK:

Review existing code in nodejs/ to understand the codebase structure and patterns
Follow the coding style, naming conventions, and architectural patterns used in the TypeScript SDK
Check for configuration files (e.g., .eslintrc.json, tsconfig.json, .prettierrc) that define coding standards
Ensure your implementation is consistent with existing SDK components
Include appropriate tests following the testing patterns in the TypeScript SDK

Related Information

Workflow Run: https://github.com/microsoft/Agent365-dotnet/actions/runs/19651795880
Trigger: pull_request
Branch: 110/merge
Commit: 7b344e7625bae456cd4485a91d0fc693ed0cea0f

This is an automated issue. Please close with wontfix label if parity is not needed.</issue_descr...

Fixes [SDK Parity] Node.js/TypeScript for PR 110 #86

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Parity Task

Changes detected in: csharp SDK
Target SDK: Node.js/TypeScript (nodejs/)

Action Required

Please apply the changes described in the original PR (#110) to the Node.js/TypeScript SDK to maintain feature parity across all SDKs.

Before implementing:

First, check if this feature already exists in the Node.js/TypeScript SDK
If the feature is already implemented, close this issue with the wontfix label and add a comment explaining that parity already exists

For bug fixes (IMPORTANT):

Bug fixes require extra caution - only act if the same bug exists in the Node.js/TypeScript SDK
Carefully analyze whether the bug described in the original PR replicates in the target platform
If the bug does NOT exist in the Node.js/TypeScript SDK, close this issue with the wontfix label and explain that the bug is not present
Stay strictly in context - only fix the specific bug described, do not make additional changes or improvements
Do not port bug fixes that are specific to the source SDK's language or platform constraints

When creating the PR for this implementation:

Assign the PR to @copilot (the author of the original PR)
Reference this issue in the PR description
Ensure the PR description explains what parity changes were made

Implementation Guidelines for Node.js/TypeScript

When implementing this feature in the TypeScript SDK:

Review existing code in nodejs/ to understand the codebase structure and patterns
Follow the coding style, naming conventions, and architectural patterns used in the TypeScript SDK
Check for configuration files (e.g., .eslintrc.json, tsconfig.json, .prettierrc) that define coding standards
Ensure your implementation is consistent with existing SDK components
Include appropriate tests following the testing patterns in the TypeScript SDK

Related Information

Workflow Run: https://github.com/microsoft/Agent365-dotnet/actions/runs/19651795880
Trigger: pull_request
Branch: 110/merge
Commit: 7b344e7625bae456cd4485a91d0fc693ed0cea0f

This is an automated issue. Please close with wontfix label if parity is not needed.</issue_descr...

Fixes [SDK Parity] Node.js/TypeScript for PR 110 #86

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: sergioescalera <8428450+sergioescalera@users.noreply.github.com>

…gs templates and instance settings Co-authored-by: sergioescalera <8428450+sergioescalera@users.noreply.github.com>

…vulnerability Co-authored-by: sergioescalera <8428450+sergioescalera@users.noreply.github.com>

sergioescalera · 2025-11-25T21:55:34Z

🔄 Original PR Updated

@copilot The original PR microsoft/Agent365-dotnet#110 has been updated with new commits.

Action Required:
Please review the updated PR and ensure this parity implementation includes all relevant changes:

📖 Review the latest changes: https://github.com/microsoft/Agent365-dotnet/pull/110/commits
🔍 Check for new features or fixes: Look for additions that need to be ported
✏️ Update this PR if needed: Add any missing functionality to maintain parity
✅ Verify completeness: Ensure all changes from the original PR are reflected here

Original PR Details:

PR: Implement Agent Settings SDK for .NET [DO NOT Review, DO NOT Merge] Agent365-dotnet#110
Author: @copilot
Latest commit: 762f791
View changes: https://github.com/microsoft/Agent365-dotnet/pull/110/files

Note: This is an automated notification triggered by new commits. If the changes are not relevant to this parity task, you can ignore this message.

Automated by AI-First Workflow

…fields Co-authored-by: sergioescalera <8428450+sergioescalera@users.noreply.github.com>

Copilot · 2025-11-25T22:01:21Z

🔄 Original PR Updated

@copilot The original PR microsoft/Agent365-dotnet#110 has been updated with new commits....

Reviewed the updated .NET PR (commit 762f791) and updated this implementation to maintain full parity. Added the following fields to models in commit 8eddb03:

AgentSettingProperty: defaultValue
AgentSettingsTemplate: id, name, description, version
AgentSettings: id, templateId, agentType, createdAt, modifiedAt

Updated README and tests to reflect these changes. All tests pass.

sergioescalera · 2025-12-01T17:27:44Z

🔄 Additional Changes Detected

New commits have been pushed to the original PR #110.

Action Required:

Review the latest changes: microsoft/Agent365-dotnet@3eb8022
Update this PR if necessary to maintain parity

Automated by AI-First Workflow v2

sergioescalera · 2025-12-01T17:57:40Z

🔄 Additional Changes Detected

New commits have been pushed to the original PR #110.

Action Required:

Review the latest changes: microsoft/Agent365-dotnet@762f791
Update this PR if necessary to maintain parity

Automated by AI-First Workflow

Copilot · 2025-12-01T23:25:15Z

🔄 Additional Changes Detected

New commits have been pushed to the original PR #110....

Reviewed commit 3eb8022 - this only modifies .github/workflows/ai-first.yml (workflow configuration), which is not part of the SDK implementation. No parity changes needed.

🔄 Additional Changes Detected

New commits have been pushed to the original PR #110....

Commit 762f791 contains a minor README.md fix that was already addressed in my previous update (commit 8eddb03). No additional changes needed.

sergioescalera · 2025-12-01T23:44:31Z

🤖 Auto-Assignment

This PR is part of a parity chain that started with a Copilot-generated PR.

Human reviewers from the original PR have been added: @sergioescalera

Reason: This is a parity implementation for issue #86, which was triggered by Copilot's PR #110.

Next Steps:

Reviewers: Please review this implementation to ensure it maintains parity
Validate the parity changes are correct and complete
Approve and merge when satisfied

Automated by AI-First Workflow

sergioescalera · 2025-12-01T23:44:35Z

🔄 Additional Changes Detected

New commits have been pushed to the original PR #110.

Action Required:

Review the latest changes: microsoft/Agent365-dotnet@8b649c5
Update this PR if necessary to maintain parity

Automated by AI-First Workflow

sergioescalera · 2025-12-02T21:41:27Z

🤖 Auto-Assignment

This PR is part of a parity chain that started with a Copilot-generated PR.

Human reviewers from the original PR have been added: @sergioescalera

Reason: This is a parity implementation for issue #86, which was triggered by Copilot's PR #110.

Next Steps:

Reviewers: Please review this implementation to ensure it maintains parity
Validate the parity changes are correct and complete
Approve and merge when satisfied

Automated by AI-First Workflow

sergioescalera · 2025-12-02T21:41:32Z

🔄 Additional Changes Detected

New commits have been pushed to the original PR #110.

Action Required:

Review the latest changes: microsoft/Agent365-dotnet@dfebd8e
Update this PR if necessary to maintain parity

Automated by AI-First Workflow

Initial plan

679189d

Copilot AI assigned Copilot and sergioescalera Nov 24, 2025

Copilot started work on behalf of sergioescalera November 24, 2025 22:46 View session

Copilot AI and others added 3 commits November 24, 2025 22:51

Initial commit - planning agent settings SDK implementation

53d9b0a

Co-authored-by: sergioescalera <8428450+sergioescalera@users.noreply.github.com>

Add @microsoft/agents-a365-settings package for managing agent settin…

9a1a999

…gs templates and instance settings Co-authored-by: sergioescalera <8428450+sergioescalera@users.noreply.github.com>

Address code review feedback: extract timeout constant and fix axios …

a2b9621

…vulnerability Co-authored-by: sergioescalera <8428450+sergioescalera@users.noreply.github.com>

Copilot AI changed the title ~~[WIP] Update SDK parity for Node.js/TypeScript~~ Implement Agent Settings SDK for Node.js/TypeScript [SDK Parity] Nov 24, 2025

Copilot AI requested a review from sergioescalera November 24, 2025 23:07

Copilot finished work on behalf of sergioescalera November 24, 2025 23:07

sergioescalera changed the title ~~Implement Agent Settings SDK for Node.js/TypeScript [SDK Parity]~~ Implement Agent Settings SDK for Node.js/TypeScript [SDK Parity] [DO NOT review] [DO NOT merge] Nov 24, 2025

Copilot started work on behalf of sergioescalera November 25, 2025 21:56 View session

Update models to maintain full parity with .NET SDK - add additional …

8eddb03

…fields Co-authored-by: sergioescalera <8428450+sergioescalera@users.noreply.github.com>

Copilot finished work on behalf of sergioescalera November 25, 2025 22:02

Copilot started work on behalf of sergioescalera December 1, 2025 23:24 View session

Copilot finished work on behalf of sergioescalera December 1, 2025 23:26

Copilot started work on behalf of sergioescalera December 1, 2025 23:31 View session

Copilot finished work on behalf of sergioescalera December 1, 2025 23:32

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Implement Agent Settings SDK for Node.js/TypeScript [SDK Parity] [DO NOT review] [DO NOT merge] #87

Implement Agent Settings SDK for Node.js/TypeScript [SDK Parity] [DO NOT review] [DO NOT merge] #87

Uh oh!

Copilot AI commented Nov 24, 2025 •

edited

Loading

Uh oh!

sergioescalera commented Nov 25, 2025

Uh oh!

Copilot AI commented Nov 25, 2025

🔄 Original PR Updated

Uh oh!

sergioescalera commented Dec 1, 2025

Uh oh!

sergioescalera commented Dec 1, 2025

Uh oh!

Copilot AI commented Dec 1, 2025 •

edited

Loading

🔄 Additional Changes Detected

🔄 Additional Changes Detected

Uh oh!

sergioescalera commented Dec 1, 2025

Uh oh!

sergioescalera commented Dec 1, 2025

Uh oh!

sergioescalera commented Dec 2, 2025

Uh oh!

sergioescalera commented Dec 2, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Implement Agent Settings SDK for Node.js/TypeScript [SDK Parity] [DO NOT review] [DO NOT merge] #87

Are you sure you want to change the base?

Implement Agent Settings SDK for Node.js/TypeScript [SDK Parity] [DO NOT review] [DO NOT merge] #87

Uh oh!

Conversation

Copilot AI commented Nov 24, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Source PR Information

PR Description

New Package: Microsoft.Agents.A365.Settings

Models

Service Interface

API Endpoints

Usage

Configuration

Tests

Parity Task

Action Required

Implementation Guidelines for Node.js/TypeScript

Related Information

Parity Task

Action Required

Implementation Guidelines for Node.js/TypeScript

Related Information

Parity Task

Action Required

Implementation Guidelines for Node.js/TypeScript

Related Information

Parity Task

Action Required

Implementation Guidelines for Node.js/TypeScript

Related Information

Parity Task

Action Required

Implementation Guidelines for Node.js/TypeScript

Related Information

Uh oh!

sergioescalera commented Nov 25, 2025

🔄 Original PR Updated

Uh oh!

Copilot AI commented Nov 25, 2025

🔄 Original PR Updated

Uh oh!

sergioescalera commented Dec 1, 2025

🔄 Additional Changes Detected

Uh oh!

sergioescalera commented Dec 1, 2025

🔄 Additional Changes Detected

Uh oh!

Copilot AI commented Dec 1, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🔄 Additional Changes Detected

🔄 Additional Changes Detected

Uh oh!

sergioescalera commented Dec 1, 2025

🤖 Auto-Assignment

Uh oh!

sergioescalera commented Dec 1, 2025

🔄 Additional Changes Detected

Uh oh!

sergioescalera commented Dec 2, 2025

🤖 Auto-Assignment

Uh oh!

sergioescalera commented Dec 2, 2025

🔄 Additional Changes Detected

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Copilot AI commented Nov 24, 2025 •

edited

Loading

New Package: `Microsoft.Agents.A365.Settings`

Copilot AI commented Dec 1, 2025 •

edited

Loading