microsoft / teams-ai / 7717444137

80%

push

github

[C#] chore: Bump `Azure.AI.OpenAI` to `v1.0.0-beta.12` (#1219)

## Linked issues

closes: #1218 

## Details

To use the `Kernel.Memory` library as the RAG solution for the C# Teams
Chef bot, the Teams AI Library must update it's dependency of
`Azure.AI.OpenAI` to match that of `Kernel.Memory`, i.e
`v1.0.0-beta.12`.

This PR introduces changes to migrate `Azure.AI.OpenAI` from
`v1.0.0-beta.9` to `v1.0.0-beta.12`.

#### Context
`Azure.AI.OpenAI` introduced breaking changes 

* Removed `ChatMessage` and replaced it with `ChatResponseMessage` and
`ChatRequestMessage`. `ChatRequestMessage` is an abstract class with 5
concrete types, `user`, `function`, `assistant`, `tool`, `system`.

## Change details
**Major**
* Updated internal `ChatMessage` class **without introducing any
breaking changes.**
* Updated implementation of internal mapping methods `ToChatMessage`,
`ToChatRequestMessage`.
    * Added unit tests
  * Added integration test for `OpenAIModel` class.

**Minor**
* Resolved warnings by reformatting code
* Resolved warnings by changing the severity config.

## Attestation Checklist

- [x] My code follows the style guidelines of this project

- I have checked for/fixed spelling, linting, and other errors
- I have commented my code for clarity
- I have made corresponding changes to the documentation (updating the
doc strings in the code is sufficient)
- My changes generate no new warnings
- I have added tests that validates my changes, and provides sufficient
test coverage. I have tested with:
  - Local testing
  - E2E testing in Teams
- New and existing unit tests pass locally with my changes

2518 of 3709 branches covered (0.0%)

Branch coverage included in aggregate %.

6003 of 7536 relevant lines covered (79.66%)

25.89 hits per line