Skip to content

feat(agent/proto): add user secrets to agent manifest#24252

Merged
zedkipp merged 1 commit into
mainfrom
zedkipp/plat-75-agent-secrets-proto
Apr 10, 2026
Merged

feat(agent/proto): add user secrets to agent manifest#24252
zedkipp merged 1 commit into
mainfrom
zedkipp/plat-75-agent-secrets-proto

Conversation

@zedkipp

@zedkipp zedkipp commented Apr 10, 2026

Copy link
Copy Markdown
Contributor

Add workspace secrets as a field in the agent manifest protobuf schema. This allows the control plane to pass user secrets to agents for runtime injection into workspace sessions.

Message fields:

  • env_name: environment variable name (empty for file-only secrets)
  • file_path: file path (empty for env-only secrets)
  • value: the decrypted secret value as bytes

Related to user secrets RFC: https://www.notion.so/coderhq/User-Secrets-32cd579be59280a98192dec7b3b5daf1?source=copy_link#32dd579be59280ae80dad718afdb961a

Extend the agent protocol to include user secrets in the manifest.
This allows the control plane to pass decrypted secret values to
agents for runtime injection into workspaces as environment variables
and files.
@github-actions github-actions Bot added the community Pull Requests and issues created by the community. label Apr 10, 2026
@zedkipp zedkipp removed the community Pull Requests and issues created by the community. label Apr 10, 2026
@zedkipp
zedkipp marked this pull request as ready for review April 10, 2026 18:41
Comment thread agent/proto/agent.proto
// written to a file.
string file_path = 2;
// The decrypted secret value.
bytes value = 3;

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

a word of warning: you can't just set environment variables to arbitrary binary data. This needs to be validated before setting (as well as on insert of the secret).

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

ack, we'll also do validation on file path values

@zedkipp
zedkipp merged commit 2f52de7 into main Apr 10, 2026
41 checks passed
@zedkipp
zedkipp deleted the zedkipp/plat-75-agent-secrets-proto branch April 10, 2026 20:57
@github-actions github-actions Bot locked and limited conversation to collaborators Apr 10, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants