chore: remove APIM and App Registration from deployment

- Comment out API Management service deployment (not used in lab)
- Comment out App Registration module (only needed for APIM)
- Remove all APIM-related outputs
- Update allowed regions to remove 'westus' (westus2, westus3, eastus2 only)

This simplifies the deployment by removing unused infrastructure components.

Author: dbarkol

infra/main.bicep

@@ -7,7 +7,7 @@ param environmentName string
 
 @minLength(1)
 @description('Primary location for all resources')
-@allowed([ 'westus', 'westus2', 'westus3', 'eastus2'])
+@allowed([ 'westus2', 'westus3', 'eastus2'])
 @metadata({
   azd: {
     type: 'location'
@@ -40,15 +40,17 @@ param chatModelName string = 'gpt-4o-mini'
 @allowed(['text-embedding-3-small'])
 param embeddingModelName string = 'text-embedding-3-small'
 
-param apimServiceName string = ''
-param apimPublisherName string = 'Snippy API'
-param apimPublisherEmail string = '[email protected]'
+// APIM is not used in the lab - commented out to reduce deployment complexity
+// param apimServiceName string = ''
+// param apimPublisherName string = 'Snippy API'
+// param apimPublisherEmail string = '[email protected]'
 
 // App registration parameters - values set by preprovision hook (scripts/setup-app-registration.sh)
-param appRegistrationDisplayName string = ''  // Set automatically: snippy-mcp-server-{resourceToken}
-param appRegistrationClientId string = ''  // Set via 'azd env set APP_REGISTRATION_CLIENT_ID <value>'
-param appRegistrationObjectId string = ''
-param appRegistrationScopeId string = ''   // Set via 'azd env set APP_REGISTRATION_SCOPE_ID <value>'
+// NOTE: App registration is only needed for APIM integration, which is currently disabled
+// param appRegistrationDisplayName string = ''  // Set automatically: snippy-mcp-server-{resourceToken}
+// param appRegistrationClientId string = ''  // Set via 'azd env set APP_REGISTRATION_CLIENT_ID <value>'
+// param appRegistrationObjectId string = ''
+// param appRegistrationScopeId string = ''   // Set via 'azd env set APP_REGISTRATION_SCOPE_ID <value>'
 
 import * as regionSelector from './app/util/region-selector.bicep'
 var abbrs = loadJsonContent('./abbreviations.json')
@@ -253,36 +255,36 @@ module api './app/api.bicep' = {
   ]
 }
 
-// API Management service
-module apim './app/apim.bicep' = {
-  name: 'apim'
-  scope: rg
-  params: {
-    name: !empty(apimServiceName) ? apimServiceName : '${abbrs.apiManagementService}${resourceToken}-${actualSuffix}'
-    location: regionSelector.getApimBasicV2Region(location)
-    tags: tags
-    publisherName: apimPublisherName
-    publisherEmail: apimPublisherEmail
-    skuName: 'BasicV2'
-    skuCapacity: 1
-    appRegistrationClientId: appRegistrationClientId
-  }
-  dependsOn: [
-    appRegistration
-  ]
-}
-
-// App Registration for MCP Server
-module appRegistration './app/app-registration.bicep' = if (!empty(appRegistrationClientId) && !empty(appRegistrationScopeId)) {
-  name: 'appRegistration'
-  scope: rg
-  params: {
-    displayName: appRegistrationDisplayName
-    applicationId: appRegistrationClientId
-    objectId: appRegistrationObjectId
-    scopeId: appRegistrationScopeId
-  }
-}
+// API Management service - Not used in the lab, commented out to simplify deployment
+// module apim './app/apim.bicep' = {
+//   name: 'apim'
+//   scope: rg
+//   params: {
+//     name: !empty(apimServiceName) ? apimServiceName : '${abbrs.apiManagementService}${resourceToken}-${actualSuffix}'
+//     location: regionSelector.getApimBasicV2Region(location)
+//     tags: tags
+//     publisherName: apimPublisherName
+//     publisherEmail: apimPublisherEmail
+//     skuName: 'BasicV2'
+//     skuCapacity: 1
+//     appRegistrationClientId: appRegistrationClientId
+//   }
+//   dependsOn: [
+//     appRegistration
+//   ]
+// }
+
+// App Registration for MCP Server - Only needed for APIM, commented out
+// module appRegistration './app/app-registration.bicep' = if (!empty(appRegistrationClientId) && !empty(appRegistrationScopeId)) {
+//   name: 'appRegistration'
+//   scope: rg
+//   params: {
+//     displayName: appRegistrationDisplayName
+//     applicationId: appRegistrationClientId
+//     objectId: appRegistrationObjectId
+//     scopeId: appRegistrationScopeId
+//   }
+// }
 
 // Durable Task Scheduler
 module dts './app/dts.bicep' = {
@@ -346,23 +348,24 @@ output AZURE_FUNCTION_NAME string = api.outputs.SERVICE_API_NAME // Function App
 @description('Connection string for the Azure Storage Account. Output name matches the AzureWebJobsStorage key in local settings.')
 output AZUREWEBJOBSSTORAGE string = storage.outputs.primaryBlobEndpoint
 
-@description('API Management gateway URL for external API access.')
-output APIM_GATEWAY_URL string = apim.outputs.gatewayUrl
+// APIM-related outputs - Not used in the lab, commented out
+// @description('API Management gateway URL for external API access.')
+// output APIM_GATEWAY_URL string = apim.outputs.gatewayUrl
 
-@description('API Management service name.')
-output APIM_SERVICE_NAME string = apim.outputs.apiManagementName
+// @description('API Management service name.')
+// output APIM_SERVICE_NAME string = apim.outputs.apiManagementName
 
-@description('API Management management API URL.')
-output APIM_MANAGEMENT_URL string = apim.outputs.managementApiUrl
+// @description('API Management management API URL.')
+// output APIM_MANAGEMENT_URL string = apim.outputs.managementApiUrl
 
-@description('App Registration Application ID (Client ID).')
-output APP_REGISTRATION_CLIENT_ID string = !empty(appRegistrationClientId) ? appRegistrationClientId : ''
+// @description('App Registration Application ID (Client ID).')
+// output APP_REGISTRATION_CLIENT_ID string = !empty(appRegistrationClientId) ? appRegistrationClientId : ''
 
-@description('App Registration Application ID URI.')
-output APP_REGISTRATION_ID_URI string = !empty(appRegistrationClientId) ? 'api://${appRegistrationClientId}' : ''
+// @description('App Registration Application ID URI.')
+// output APP_REGISTRATION_ID_URI string = !empty(appRegistrationClientId) ? 'api://${appRegistrationClientId}' : ''
 
-@description('App Registration Scope ID.')
-output APP_REGISTRATION_SCOPE_ID string = !empty(appRegistrationScopeId) ? appRegistrationScopeId : ''
+// @description('App Registration Scope ID.')
+// output APP_REGISTRATION_SCOPE_ID string = !empty(appRegistrationScopeId) ? appRegistrationScopeId : ''
 
-@description('App Registration Display Name.')
-output APP_REGISTRATION_DISPLAY_NAME string = appRegistrationDisplayName
+// @description('App Registration Display Name.')
+// output APP_REGISTRATION_DISPLAY_NAME string = appRegistrationDisplayName