[pull] main from Azure-Samples:main#37
Open
pull[bot] wants to merge 409 commits intodigital-power:mainfrom
Open
[pull] main from Azure-Samples:main#37pull[bot] wants to merge 409 commits intodigital-power:mainfrom
pull[bot] wants to merge 409 commits intodigital-power:mainfrom
Conversation
Check Broken URLsWe have automatically detected the following broken URLs in your files. Review and fix the paths to resolve this issue. Check the file paths and associated broken URLs inside them. For more details, check our Contributing Guide.
|
Less overhead makes this easier to maintain and enables using hardlinking on Windows runners
* Add evaluation workflow * Trying to trigger workflow * Remove conditional * Update workflow * Add back old python file * New branch for eval * Fix uv * Remove python tests for now * New PR for eval * Add debug * Add workflow dispatch * Add workflow dispatch * Remove comment for now * Add workflow push * Add checkout * Try azd env new first * Try refresh * Add env config * Fix the action vars * Fix local server start * Fix app run * logs pos * Run app directly * nohup * Log more * Logger calls * Fix log calls * Remove empty string values * Ask less questions * Evaluate all questions * Base on comment * Base on comment * Revert unneeded changes * Add note, link eval docs in more places, link to videos
Bumps [cryptography](https://github.com/pyca/cryptography) from 44.0.0 to 44.0.1. - [Changelog](https://github.com/pyca/cryptography/blob/main/CHANGELOG.rst) - [Commits](pyca/cryptography@44.0.0...44.0.1) --- updated-dependencies: - dependency-name: cryptography dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* Improve the logic in the locust tests * ask for follow up questions * With the GPT-V test * reformatted file
* Fix screenshot * Update results_dir * Results dir
…2366) Bumps the github-actions group with 1 update: [Azure/setup-azd](https://github.com/azure/setup-azd). Updates `Azure/setup-azd` from 2.0.0 to 2.1.0 - [Release notes](https://github.com/azure/setup-azd/releases) - [Changelog](https://github.com/Azure/setup-azd/blob/main/CHANGELOG.md) - [Commits](Azure/setup-azd@v2...v2.1.0) --- updated-dependencies: - dependency-name: Azure/setup-azd dependency-type: direct:production update-type: version-update:semver-minor dependency-group: github-actions ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* First attempt with infra * Evaluate the simulated users * Revert launch.json changes * Remove unneeded infra * Add links and progress tracking * Update evals/safety_evaluation.py Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Reword arg, add comment on time needed * Use the enum * Fix one more zerodiv error --------- Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
* Default to 0125 * Update to latest API version
Bumps [jinja2](https://github.com/pallets/jinja) from 3.1.5 to 3.1.6. - [Release notes](https://github.com/pallets/jinja/releases) - [Changelog](https://github.com/pallets/jinja/blob/main/CHANGES.rst) - [Commits](pallets/jinja@3.1.5...3.1.6) --- updated-dependencies: - dependency-name: jinja2 dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Upgrade Search SDK and add support for query rewriting by setting AZURE_SEARCH_QUERY_REWRITING --------- Co-authored-by: Pamela Fox <pamela.fox@gmail.com>
* Adjust container app parameters to reduce costs * Change back to enough CPU/memory
Bumps [dompurify](https://github.com/cure53/DOMPurify) from 3.2.0 to 3.2.4. - [Release notes](https://github.com/cure53/DOMPurify/releases) - [Commits](cure53/DOMPurify@3.2.0...3.2.4) --- updated-dependencies: - dependency-name: dompurify dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [react-i18next](https://github.com/i18next/react-i18next) from 15.1.1 to 15.4.1. - [Changelog](https://github.com/i18next/react-i18next/blob/master/CHANGELOG.md) - [Commits](i18next/react-i18next@v15.1.1...v15.4.1) --- updated-dependencies: - dependency-name: react-i18next dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* Port to gpt-4o-mini as default * Update snapshot for 128K model * Fix markdown
* Upgrade MS Graph SDK dependencies to remove pendulum dependency * Remove pin from .in file
* WIP * WIP * ruff, black * adding usage * mypy * ruff, black * mypy, ruff, black, and update generate thought steps * fix comments, set answer thought tag on streaming approaches * fixing frontend * fixing backend + frontend * token graph fixup * fix token usage for non-streaming response * re-style token graph * updates * adddressing feedback * ruff, black * prettify * fixing typing errors * ruff, black * mypy, ruff * prettier * trying to fix test failures * fixing test failures * fix streaming allowed setup * black * try to fix e2e test * add tests; updating env vars * adding tests * rerecording * more recording * add tests; ruff, black * run prettier * update docs * fix test * fix linter * Fleshed out readme * Adding reasoning to deploy featureS * Make changes from pamelas feedback * undangle that comma --------- Co-authored-by: Pamela Fox <pamela.fox@gmail.com>
…3024) Bumps [@fluentui/react-table](https://github.com/microsoft/fluentui) from 9.19.11 to 9.19.14. - [Release notes](https://github.com/microsoft/fluentui/releases) - [Commits](https://github.com/microsoft/fluentui/compare/@fluentui/react-table_v9.19.11...@fluentui/react-table_v9.19.14) --- updated-dependencies: - dependency-name: "@fluentui/react-table" dependency-version: 9.19.14 dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
--- updated-dependencies: - dependency-name: aiohttp dependency-version: 3.13.4 dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* Bump prettier from 3.3.3 to 3.8.1 in /app/frontend Bumps [prettier](https://github.com/prettier/prettier) from 3.3.3 to 3.8.1. - [Release notes](https://github.com/prettier/prettier/releases) - [Changelog](https://github.com/prettier/prettier/blob/main/CHANGELOG.md) - [Commits](prettier/prettier@3.3.3...3.8.1) --- updated-dependencies: - dependency-name: prettier dependency-version: 3.8.1 dependency-type: direct:development update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> * Run prettier on AnalysisPanel.tsx to fix CI formatting check Agent-Logs-Url: https://github.com/Azure-Samples/azure-search-openai-demo/sessions/f66637a5-8f85-4cff-9927-212ca63c82d9 Co-authored-by: pamelafox <297042+pamelafox@users.noreply.github.com> --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: pamelafox <297042+pamelafox@users.noreply.github.com>
Bumps [tenacity](https://github.com/jd/tenacity) from 9.1.2 to 9.1.4. - [Release notes](https://github.com/jd/tenacity/releases) - [Commits](jd/tenacity@9.1.2...9.1.4) --- updated-dependencies: - dependency-name: tenacity dependency-version: 9.1.4 dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [cryptography](https://github.com/pyca/cryptography) from 46.0.6 to 46.0.7. - [Changelog](https://github.com/pyca/cryptography/blob/main/CHANGELOG.rst) - [Commits](pyca/cryptography@46.0.6...46.0.7) --- updated-dependencies: - dependency-name: cryptography dependency-version: 46.0.7 dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* Migrate from Chat Completions to Responses API
- Switch all OpenAI calls from chat.completions.create to responses.create
- Replace AzureOpenAI/AsyncAzureOpenAI types with standard OpenAI SDK types
- Update message format: messages -> input, max_tokens -> max_output_tokens
- Flatten tool definitions to Responses API format
- Update streaming from ChatCompletionChunk to ResponseStreamEvent
- Remove seed parameter (not supported in Responses API)
- Remove o-series model support, use prefix-based GPT-5 detection
- Add store=False on all responses.create calls
- Add dynamic reasoning effort options from backend /config endpoint
- Hide temperature for reasoning models, show reasoning effort dropdown
- Update all test mocks, fixtures, and snapshots
- Update image content types: text -> input_text, image_url -> input_image
* Migrate from Chat Completions to Responses API
## Purpose
Migrate the entire backend from the legacy OpenAI Chat Completions API (`chat.completions.create`) to the unified Responses API (`responses.create`). This is required for GPT-5+ models and aligns with OpenAI's recommended API surface going forward.
### Backend changes
- **API migration**: All `chat.completions.create` calls → `responses.create` with the new parameter shapes (`messages` → `input`, `max_tokens` → `max_output_tokens`, `store=False`)
- **Streaming**: Rewritten from `ChatCompletionChunk` events to `response.output_text.delta` / `response.completed` events
- **Tool definitions**: Flattened from nested `{"type": "function", "function": {"name": ...}}` to `{"type": "function", "name": ...}`
- **Image content types**: `"text"` → `"input_text"`, `"image_url"` → `"input_image"`, flattened `image_url` from nested object to plain string
- **Response access**: `choices[0].message.content` → `response.output_text`
- **Token usage**: Added `ResponseUsage` handling alongside `CompletionUsage`
### Model support changes
- **Dropped o-series support** (o1, o3, o4-mini) — replaced with prefix-based GPT-5 family detection (`model.startswith("gpt-5")`)
- **Dynamic reasoning effort options**: Backend computes valid options per model and sends them via `/config` endpoint; frontend renders dynamically
- `gpt-5`: `minimal`, `low`, `medium`, `high`
- `gpt-5.1–5.2`: `none`, `low`, `medium`, `high`
- `gpt-5.4+`: `none`, `low`, `medium`, `high`, `xhigh`
- **Temperature**: Only sent for non-reasoning models (reasoning models reject it)
### Frontend changes
- **Removed seed setting** from UI, API model, and all 10 locale files
- **Reasoning effort dropdown** is now dynamically populated from backend options
- **Temperature field** hidden when a reasoning model is deployed
- **Reasoning effort dropdown** moved to LLM Settings section
### Test changes
- All mock fixtures updated from `ChatCompletion`/`ChatCompletionChunk` to `Response`/`ResponseStreamEvent`
- Monkeypatch paths: `chat.completions.create` → `responses.create`
- Test reasoning model changed from `o3-mini` to `gpt-5`
- Removed `test_app_config_for_reasoning_without_streaming` (o1-specific)
- All snapshot files regenerated
## Does this introduce a breaking change?
```
[x] Yes
[ ] No
```
Users on o-series models (o1, o3, o3-mini, o4-mini) will need to switch to a GPT-5 family model. The seed developer setting is removed. The streaming output shape changed slightly (fewer intermediate chunks).
## Does this require changes to learn.microsoft.com docs?
```
[ ] Yes
[x] No
```
## Type of change
```
[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[x] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:
```
## Code quality checklist
- [x] The current tests all pass (`python -m pytest`).
- [x] I added tests that prove my fix is effective or that my feature works
- [x] I ran `python -m pytest --cov` to verify 100% coverage of added lines
- [x] I ran `ty check` to check for type errors
- [x] I either used the pre-commit hooks or ran `ruff` and `black` manually on my code.
* Fix ty check: use isinstance for stream event type narrowing
* Update streaming snapshots after isinstance event check change
* Revert isinstance to event.type checks for mock compatibility
The isinstance checks broke test mocks since MockResponseEvent is not a
real SDK event class. Use event.type string checks with type: ignore
comments for ty instead.
* Use real SDK event classes in mocks for isinstance compatibility
Replace MockResponseEvent with ResponseTextDeltaEvent and
ResponseCompletedEvent from the OpenAI SDK. This enables isinstance
type checks in production code without type: ignore comments.
* Fix E2E tests: remove seed field, add reasoningEffortOptions to config mocks
* Address PR review comments
- Fix temperature=0.0 falsy bug: use 'is not None' check
- Fix t() fallback: use defaultValue param for missing translations
- Translate 'none' and 'xhigh' reasoning options in all locales
* Mock configure_azure_monitor in test to prevent local hang
The test was calling configure_azure_monitor() with a fake connection
string, which hangs locally because the SDK tries to connect to Azure.
Mock it and assert it was called instead.
* Remove manual test plan, add results to PR description
* Update backend-frontend contract to Responses API shape
Streaming:
- {"type": "context", "context": {...}, "session_state": ...}
- {"type": "delta", "delta": "text chunk"}
Non-streaming:
- {"output_text": "full text", "context": {...}, "session_state": ...}
Removes Chat Completions shape (message.content, delta.content, delta.role)
from the backend-frontend contract.
* Use Responses API event type names in contract
- 'delta' -> 'response.output_text.delta' (matches ResponseTextDeltaEvent.type)
- 'context' -> 'response.context' (custom, follows naming convention)
* Address PR review comments
- Fix nested context.context in follow-up questions event
- Use explicit undefined check for empty-string deltas
- Make ChatAppResponseOrError fields optional for error responses
* Fix content filter error handling for Responses API
- Extract error code from body.error.code (new SDK location)
- Add HttpResponseError check for agentic retrieval content filter errors
- Update test mocks to match real Responses API error shape
Fixes #3030
* Parse error body from non-2xx responses before throwing
Previously the frontend threw a generic 'Request failed with status 400'
without reading the error message from the response body.
* Linting on error
* Update documentation
* Fix error mocks and types
Check Broken URLsWe have automatically detected the following broken URLs in your files. Review and fix the paths to resolve this issue. Check the file paths and associated broken URLs inside them.
|
Bumps [pillow](https://github.com/python-pillow/Pillow) from 12.1.1 to 12.2.0. - [Release notes](https://github.com/python-pillow/Pillow/releases) - [Changelog](https://github.com/python-pillow/Pillow/blob/main/CHANGES.rst) - [Commits](python-pillow/Pillow@12.1.1...12.2.0) --- updated-dependencies: - dependency-name: pillow dependency-version: 12.2.0 dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Check Broken URLsWe have automatically detected the following broken URLs in your files. Review and fix the paths to resolve this issue. Check the file paths and associated broken URLs inside them.
|
…3036) Safe dependency bump: actions/github-script 8→9, CI passes (1/1)
Safe dependency bump: click 8.3.1→8.3.2, werkzeug 3.1.6→3.1.8, CI passes (21/21)
Safe dependency bump: react-helmet-async 2.0.5→3.0.0, CI passes (22/22). v3 release contains only CI changes, no API breakage.
Safe dependency bump: dompurify 3.3.2→3.3.3, @types/dompurify 3.0.5→3.2.0, CI passes (22/22)
Safe dependency bump: pydantic 2.12.5→2.13.0, pydantic-core 2.41.5→2.46.0, CI passes (21/21). Transitive dep only.
Safe dependency bump: pypdf 6.9.2→6.10.0, CI passes (21/21). Tested via LocalPdfParser in test_prepdocslib_textsplitter.py.
Check Broken URLsWe have automatically detected the following broken URLs in your files. Review and fix the paths to resolve this issue. Check the file paths and associated broken URLs inside them.
|
5 similar comments
Check Broken URLsWe have automatically detected the following broken URLs in your files. Review and fix the paths to resolve this issue. Check the file paths and associated broken URLs inside them.
|
Check Broken URLsWe have automatically detected the following broken URLs in your files. Review and fix the paths to resolve this issue. Check the file paths and associated broken URLs inside them.
|
Check Broken URLsWe have automatically detected the following broken URLs in your files. Review and fix the paths to resolve this issue. Check the file paths and associated broken URLs inside them.
|
Check Broken URLsWe have automatically detected the following broken URLs in your files. Review and fix the paths to resolve this issue. Check the file paths and associated broken URLs inside them.
|
Check Broken URLsWe have automatically detected the following broken URLs in your files. Review and fix the paths to resolve this issue. Check the file paths and associated broken URLs inside them.
|
Bumps [pypdf](https://github.com/py-pdf/pypdf) from 6.10.0 to 6.10.1. - [Release notes](https://github.com/py-pdf/pypdf/releases) - [Changelog](https://github.com/py-pdf/pypdf/blob/main/CHANGELOG.md) - [Commits](py-pdf/pypdf@6.10.0...6.10.1) --- updated-dependencies: - dependency-name: pypdf dependency-version: 6.10.1 dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [dompurify](https://github.com/cure53/DOMPurify) from 3.3.3 to 3.4.0. - [Release notes](https://github.com/cure53/DOMPurify/releases) - [Commits](cure53/DOMPurify@3.3.3...3.4.0) --- updated-dependencies: - dependency-name: dompurify dependency-version: 3.4.0 dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Check Broken URLsWe have automatically detected the following broken URLs in your files. Review and fix the paths to resolve this issue. Check the file paths and associated broken URLs inside them.
|
1 similar comment
Check Broken URLsWe have automatically detected the following broken URLs in your files. Review and fix the paths to resolve this issue. Check the file paths and associated broken URLs inside them.
|
Bumps [pypdf](https://github.com/py-pdf/pypdf) from 6.10.1 to 6.10.2. - [Release notes](https://github.com/py-pdf/pypdf/releases) - [Changelog](https://github.com/py-pdf/pypdf/blob/main/CHANGELOG.md) - [Commits](py-pdf/pypdf@6.10.1...6.10.2) --- updated-dependencies: - dependency-name: pypdf dependency-version: 6.10.2 dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Check Broken URLsWe have automatically detected the following broken URLs in your files. Review and fix the paths to resolve this issue. Check the file paths and associated broken URLs inside them.
|
Bumps [python-dotenv](https://github.com/theskumar/python-dotenv) from 1.1.1 to 1.2.2. - [Release notes](https://github.com/theskumar/python-dotenv/releases) - [Changelog](https://github.com/theskumar/python-dotenv/blob/main/CHANGELOG.md) - [Commits](theskumar/python-dotenv@v1.1.1...v1.2.2) --- updated-dependencies: - dependency-name: python-dotenv dependency-version: 1.2.2 dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Check Broken URLsWe have automatically detected the following broken URLs in your files. Review and fix the paths to resolve this issue. Check the file paths and associated broken URLs inside them.
|
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.
See Commits and Changes for more details.
Created by
pull[bot] (v2.0.0-alpha.4)
Can you help keep this open source service alive? 💖 Please sponsor : )