fix: Standardize principal variables, improve resource group handling, and enhance local Teams dev setup

[Pavan-Microsoft]

Purpose

This pull request updates Azure principal environment variable names throughout the CI workflow and infrastructure configuration, improves deployment outputs, and enhances documentation for local Teams development. It also includes minor Makefile and deployment script fixes. The most important changes are grouped below:

CI Workflow and Environment Variable Standardization:

• Renamed Azure principal environment variables from AZURE_PRINCIPAL_ID, AZURE_PRINCIPAL_NAME, and AZURE_PRINCIPAL_TYPE to PRINCIPAL_ID, PRINCIPAL_NAME, and PRINCIPAL_TYPE in .github/workflows/ci.yml and updated all references accordingly. This standardizes variable naming across the workflow and infrastructure templates. [1] [2] [3] [4]
• Added new outputs to the deploy job in .github/workflows/ci.yml, including resource_group and solution_suffix, and updated downstream jobs to consume these outputs for improved environment management. [1] [2] [3]

Documentation Improvements:

• Updated docs/TEAMS_LOCAL_DEPLOYMENT.md to clarify how to set local development URLs and enable multitenant mode for Teams Toolkit, making local setup instructions more explicit and user-friendly.

Makefile and Deployment Script Fixes:

• Removed duplicate azd-login target and cleaned up hardcoded PostgreSQL credential output in the Makefile for clarity and maintainability. [1] [2]
• Ensured the correct Azure environment is selected before running the destroy command in the Makefile for reliable resource cleanup.

Does this introduce a breaking change?

• Yes
• No

How to Test

• Get the code

git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
npm install

What to Check

Verify the pipeline