[VS Code] Agent Loop connections are not parameterized on creation
- 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.