Azure / Azure/LogicAppsUX

[VS Code] Agent Loop connections are not parameterized on creation

Open
#7,696 2 comments 0 reactions 0 assignees View on GitHub
Agentic VSCode
Dominant language
TypeScript
Stars
111
Forks
109
Avg merge
1d 23h
Merged PRs (30d)
20

Description

### Severity

P2 - High (Major functionality broken)

### Describe the Bug with repro steps

**Expected Behavior: **

When a new agent loop connection is persisted to connections.json all its external dependencies are parameterized. That includes resource IDs, endpoints and and keys. The expected format for this should be:

```JSON
{
"agentConnections": {
"agent": {
"displayName": "connOpenAI",
"authentication": {
"type": "Key",
"key": "@appsetting('agent_openAIKey')"
},
"endpoint": "@appsetting('agent_openAIEndpoint')",
"resourceId": "/subscriptions/@appsetting('agent_subscriptionId')/resourceGroups/@appsetting('agent_resourceGroup')/providers/Microsoft.CognitiveServices/accounts/@appsetting('agent_openAIAccountName')",
"type": "model"
}
}
}
```

**Current Behavior: **

Agent loop connections are only partially parameterized. Although Key and endpoint are sent to appsettings, subscription, resourcegroup and names are not parameterized:

```JSON
{
"agentConnections": {
"agent": {
"displayName": "connOpenAI",
"authentication": {
"type": "Key",
"key": "@appsetting('agent_openAIKey')"
},
"endpoint": "@appsetting('agent_openAIEndpoint')",
"resourceId": "/subscriptions/00000000-0000-0000-0000-000000000000/resourceGroups/Example-ResourceGroup/providers/Microsoft.CognitiveServices/accounts/example-open-ai",
"type": "model"
}
}
}
```

**Impact: **

Customers using VS Code and already taking advantage of CI/CD improvements in the tool will have a degraded experience, as they will need to manually reconfigure this component.

Subscription IDs, resource groups and account names will be stored in source control, potentially triggering static analysis components that required no keys or IDs stored in source control.

### What type of Logic App Is this happening in?

Consumption (Portal)

### Are you experiencing a regression?

_No response_

### Which operating system are you using?

Windows

### Did you refer to the TSG before filing this issue? https://aka.ms/lauxtsg

Yes

### Workflow JSON

```json

```

### Screenshots or Videos

_No response_

### Environment

- VS Code: 1.102.0-insider
- Extension: 5.109.14 (latest)

### Additional context

There is some work already done for APIM connections in the code base that can be adapted for this, as they have similar requirements. Ping the DevEx team if required.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.