Skip to content

Conversation

@ssncferreira
Copy link
Contributor

No description provided.

Copy link
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@ssncferreira ssncferreira force-pushed the ssncferreira/feat-add-aiproxy-core branch from 11a58c6 to f095630 Compare December 16, 2025 20:27
@ssncferreira ssncferreira force-pushed the ssncferreira/feat-add-aiproxy-core branch from f095630 to a6abd82 Compare December 16, 2025 20:50
Flag: "aiproxy-listen-addr",
Env: "CODER_AIPROXY_LISTEN_ADDR",
Value: &c.AI.ProxyConfig.ListenAddr,
Default: ":8888",
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure if there is a better standard for a proxy port 🤔

Value: &c.AI.ProxyConfig.KeyFile,
Default: "",
Group: &deploymentGroupAIProxy,
YAML: "key_file",
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just to confirm, do we need to set this as a secret? Something like: Annotations: serpent.Annotations{}.Mark(annotationSecretKey, "true"),
This is the file path, so the content is the actual secret 🤔

Value: &c.AI.ProxyConfig.ListenAddr,
Default: ":8888",
Group: &deploymentGroupAIProxy,
YAML: "listen_addr",
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure if we have a convention for yaml between snake_case and camelCase, as I see some options of aibridge with both, for instance, rateLimit and inject_coder_mcp_tools

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants