Skip to content

fix: dev to main - Add location parameter to AI search service and update package versions, improvements to local development setup#876

Merged
Avijit-Microsoft merged 14 commits into
mainfrom
dev
Apr 20, 2026
Merged

fix: dev to main - Add location parameter to AI search service and update package versions, improvements to local development setup#876
Avijit-Microsoft merged 14 commits into
mainfrom
dev

Conversation

@Pavan-Microsoft
Copy link
Copy Markdown
Contributor

@Pavan-Microsoft Pavan-Microsoft commented Apr 16, 2026

Purpose

This pull request introduces several improvements to local development setup, infrastructure automation, dependency updates, and documentation. The main focus is on enhancing the developer experience by providing robust startup scripts for both Windows and Unix-like systems, improving environment and process management, updating dependencies for better security and compatibility, and refining infrastructure deployment scripts.

Local Development Experience Improvements:

  • Added automated startup scripts (start.cmd for Windows and start.sh for macOS/Linux/WSL) that handle environment setup, dependency installation, Azure authentication, and starting backend/frontend servers, including automatic cleanup of processes on exit or interruption. The documentation (LocalDevelopmentSetup.md) was updated to reflect these changes and provide clear instructions. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10]

Dependency Updates:

  • Updated several Python and Node.js package dependencies to address security, compatibility, and feature improvements, including aiohttp, requests, pytest, pypdf, and lodash. [1] [2] [3] [4] [5] [6]

Infrastructure and Automation Enhancements:

  • Improved Bicep and ARM templates for Azure resource deployment by adding the location parameter to the search service module, refining resource dependency ordering, and ensuring proper deployment sequencing. [1] [2] [3] [4] [5] [6] [7]

Azure Role Assignment and Access:

  • Updated Cosmos DB role assignment instructions in the documentation to use the recommended az cosmosdb sql role assignment create command and clarified variable usage for improved reliability.

These changes collectively streamline the local development workflow, improve reliability and cross-platform support, and keep dependencies and infrastructure scripts up-to-date.

Does this introduce a breaking change?

  • Yes
  • No

Golden Path Validation

  • I have tested the primary workflows (the "golden path") to ensure they function correctly without errors.

Deployment Validation

  • I have validated the deployment process successfully and all services are running as expected with this change.

@Pavan-Microsoft Pavan-Microsoft changed the title fix: Add location parameter to AI search service and update package versions fix: dev to main - Add location parameter to AI search service and update package versions Apr 17, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates application and script dependencies and adjusts the infrastructure templates to improve AI Search deployment parameterization (adding location) and deployment ordering.

Changes:

  • Bumps Python dependencies (API + infra scripts) and updates frontend overrides/lockfile for lodash.
  • Adds/passes location to the AI Search AVM module invocation in Bicep and the generated ARM template.
  • Reorders some dependsOn entries in infra/main.json and updates the generated template hash.

Reviewed changes

Copilot reviewed 7 out of 8 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
src/api/requirements.txt Updates pinned Python dependency versions (requests/types-requests/aiohttp/pytest).
infra/scripts/agent_scripts/requirements.txt Bumps aiohttp for agent scripts.
infra/scripts/index_scripts/requirements.txt Bumps pypdf for indexing scripts.
src/App/package.json Adds a lodash override version.
src/App/package-lock.json Updates resolved lodash package version in lockfile.
infra/main.bicep Passes location into searchServiceUpdate module params.
infra/main_custom.bicep Passes location into searchServiceUpdate module params.
infra/main.json Reflects Bicep changes: templateHash update, location param passed to module, dependency ordering tweaks.
Files not reviewed (1)
  • src/App/package-lock.json: Language not supported

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread infra/main_custom.bicep
@Pavan-Microsoft Pavan-Microsoft changed the title fix: dev to main - Add location parameter to AI search service and update package versions fix: dev to main - Add location parameter to AI search service and update package versions, improvements to local development setup Apr 17, 2026
@Pavan-Microsoft Pavan-Microsoft marked this pull request as ready for review April 20, 2026 05:24
@Avijit-Microsoft Avijit-Microsoft merged commit 3ba223d into main Apr 20, 2026
21 checks passed
@github-actions
Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 3.21.2 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants