Arch board feedback + beta.2 release preparation#48247
Open
jpalvarezl wants to merge 12 commits intomainfrom
Open
Arch board feedback + beta.2 release preparation#48247jpalvarezl wants to merge 12 commits intomainfrom
beta.2 release preparation#48247jpalvarezl wants to merge 12 commits intomainfrom
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This PR prepares the 2.0.0-beta.2 release for both azure-ai-projects and azure-ai-agents SDK libraries. It applies architecture-board-requested naming improvements (renaming classes for clarity), replaces the custom DayOfWeek enum with the JDK standard java.time.DayOfWeek, and updates both TypeSpec commit pointers to a newer API spec snapshot.
Changes:
- Renamed several model classes for clarity:
Index→AIProjectIndex,OpenAIError→ApiError,AgentObjectVersions→AgentDetailsVersions,AzureFunctionDefinitionFunction→AzureFunctionDefinitionDetails, anddefaultParamsfield →defaultParametersinOpenApiFunctionDefinition - Replaced the TSP-generated
DayOfWeekcustom enum withjava.time.DayOfWeekinWeeklyRecurrenceSchedule, adding atoPascalCaseserialization helper and a new unit-test class - Updated
CHANGELOG.mdin both libraries to stamp the2.0.0-beta.2release date
Reviewed changes
Copilot reviewed 22 out of 23 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
WeeklyRecurrenceSchedule.java |
Switches DayOfWeek from custom enum to java.time.DayOfWeek; adds toPascalCase helper |
WeeklyRecurrenceScheduleSerializationTest.java |
New unit tests covering serialization, deserialization, and round-trip for the JDK DayOfWeek |
AIProjectIndex.java (formerly Index.java) |
Renamed from Index to AIProjectIndex; default type string updated |
JsonMergePatchHelper.java |
Renames IndexAccessor → AIProjectIndexAccessor and its field/methods |
AzureAISearchIndex.java, CosmosDBIndex.java, ManagedAzureAISearchIndex.java |
Updated parent class references from Index to AIProjectIndex |
IndexesClient.java, IndexesAsyncClient.java |
Updated method signatures to use AIProjectIndex |
ApiError.java (formerly OpenAIError.java) |
Renamed from OpenAIError to ApiError |
AgentDetailsVersions.java (formerly AgentObjectVersions.java) |
Renamed; constructor moved to top |
AgentDetails.java |
Updated to use AgentDetailsVersions |
AzureFunctionDefinition.java, AzureFunctionDefinitionDetails.java |
Renamed inner type; constructor moved |
OpenApiFunctionDefinition.java |
Renamed field defaultParams → defaultParameters |
MemoryStoreUpdateResponse.java |
Updated to use ApiError instead of OpenAIError |
DayOfWeek.java |
File deleted (replaced by java.time.DayOfWeek) |
CHANGELOG.md (both libraries) |
Stamped 2.0.0-beta.2 release date, removed "Other Changes" empty section |
tsp-location.yaml (both libraries) |
Updated API spec commit hash |
| Metadata JSON files | Updated cross-language definitions to reflect renames |
Comments suppressed due to low confidence (1)
sdk/ai/azure-ai-projects/src/main/java/com/azure/ai/projects/models/AIProjectIndex.java:28
- The default
typefield value was changed fromIndexType.fromString("Index")toIndexType.fromString("AIProjectIndex")in the base class. However, this value is serialized to the wire intoJsonandtoJsonMergePatch. If the API server does not recognize"AIProjectIndex"as a valid discriminator value, this will cause errors for any code that directly instantiatesAIProjectIndex(instead of a concrete subclass). The value"AIProjectIndex"is not listed as a known discriminator value in thefromJsonmethod (which only recognizes"AzureSearch","ManagedAzureSearch", and"CosmosDBNoSqlVectorStore"). Please verify whether"AIProjectIndex"is a valid value accepted by the API, or if this default should be left as-is or removed.
...i/azure-ai-projects/src/main/java/com/azure/ai/projects/models/WeeklyRecurrenceSchedule.java
Show resolved
Hide resolved
...ure-ai-projects/src/main/java/com/azure/ai/projects/implementation/JsonMergePatchHelper.java
Show resolved
Hide resolved
dargilco
approved these changes
Mar 4, 2026
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
This pull request primarily refactors several model classes in the Azure AI Agents SDK to improve naming consistency and clarity. The changes include renaming classes and their references, updating constructors and methods, and ensuring all usages reflect the new names. No functional logic is changed, but these updates will make the codebase easier to understand and maintain.
Model class renaming and updates
Agent details and versions
AgentObjectVersionstoAgentDetailsVersionsand updated all references inAgentDetails.javaand the class itself, including constructors, getters, and JSON serialization/deserialization methods. (AgentDetails.java,AgentDetailsVersions.java) [1] [2] [3] [4] [5] [6] [7]Error model
OpenAIErrortoApiErrorthroughout the codebase, including the class name, fields, constructors, and JSON methods. All usages of the error model now use the new name. (ApiError.java) [1] [2] [3] [4] [5] [6] [7]Azure Function definition
AzureFunctionDefinitionFunctiontoAzureFunctionDefinitionDetailsand updated all references inAzureFunctionDefinition.javaand the class itself, including constructors, getters, and JSON methods. (AzureFunctionDefinition.java,AzureFunctionDefinitionDetails.java) [1] [2] [3] [4] [5] [6] [7]Documentation updates
2.0.0-beta.2and removed the "Other Changes" section. (CHANGELOG.md) [1] [2]