fix(azure.ai.agents): skip postdeploy hook when project has no hosted agents (#7984)

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: JeffreyCA <9157833+JeffreyCA@users.noreply.github.com>

Author: Copilot

cli/azd/extensions/azure.ai.agents/internal/cmd/listen.go

@@ -160,6 +160,19 @@ func isHostedAgentService(svc *azdext.ServiceConfig, proj *azdext.ProjectConfig)
 }
 
 func postdeployHandler(ctx context.Context, azdClient *azdext.AzdClient, args *azdext.ProjectEventArgs) error {
+	// Skip when the project has no hosted agent services. `postdeploy` fires on every
+	// `azd deploy`, so without this guard the AZURE_AI_PROJECT_ENDPOINT/AZURE_TENANT_ID
+	// reads below would fail for projects that don't use this extension. See #7373.
+	var hostedAgents []*azdext.ServiceConfig
+	for _, svc := range args.Project.Services {
+		if svc.Host == AiAgentHost && isHostedAgentService(svc, args.Project) {
+			hostedAgents = append(hostedAgents, svc)
+		}
+	}
+	if len(hostedAgents) == 0 {
+		return nil
+	}
+
 	// Collect agent identities from hosted agent services that were deployed.
 	// After deploy, each hosted agent's name/version is stored as AGENT_{SERVICE_KEY}_NAME/VERSION.
 	// We fetch the full agent version object from the API to get the instance identity principal ID,
@@ -209,10 +222,7 @@ func postdeployHandler(ctx context.Context, azdClient *azdext.AzdClient, args *a
 
 	// Build name→principalID map by fetching the agent version for each hosted service.
 	agentIdentities := make(map[string]string)
-	for _, svc := range args.Project.Services {
-		if svc.Host != AiAgentHost || !isHostedAgentService(svc, args.Project) {
-			continue
-		}
+	for _, svc := range hostedAgents {
 		serviceKey := toServiceKey(svc.Name)
 
 		versionResp, err := azdClient.Environment().GetValue(ctx, &azdext.GetEnvRequest{

cli/azd/extensions/azure.ai.agents/internal/cmd/listen_test.go

@@ -7,8 +7,33 @@ import (
 	"testing"
 
 	"azureaiagent/internal/project"
+
+	"github.com/azure/azure-dev/cli/azd/pkg/azdext"
 )
 
+// TestPostdeployHandler_NoAgentService_NoOp verifies postdeployHandler returns nil
+// without any RPC calls when the project has no hosted agent services. Regression
+// guard for #7373.
+func TestPostdeployHandler_NoAgentService_NoOp(t *testing.T) {
+	t.Parallel()
+
+	// Use a temp dir + explicit RelativePath so isHostedAgentService deterministically
+	// returns false (no agent.yaml present) regardless of the test working directory.
+	args := &azdext.ProjectEventArgs{
+		Project: &azdext.ProjectConfig{
+			Path: t.TempDir(),
+			Services: map[string]*azdext.ServiceConfig{
+				"teams-bot": {Name: "teams-bot", Host: "containerapp", RelativePath: "."},
+			},
+		},
+	}
+
+	// nil azdClient — the early return must fire before any RPC call.
+	if err := postdeployHandler(t.Context(), nil, args); err != nil {
+		t.Fatalf("expected no error for project without agent services, got: %v", err)
+	}
+}
+
 func TestParseConnectionIDs(t *testing.T) {
 	t.Parallel()