Skip to content

feat(api): add workload agent_state#139

Merged
vitramir merged 1 commit into
mainfrom
feat/workload-agent-state
Apr 30, 2026
Merged

feat(api): add workload agent_state#139
vitramir merged 1 commit into
mainfrom
feat/workload-agent-state

Conversation

@emerson-gray
Copy link
Copy Markdown
Contributor

@emerson-gray emerson-gray commented Apr 30, 2026

Adds a new WorkloadAgentState enum and Workload.agent_state field (tag 19) to separate container lifecycle (status) from agent processing state.

Rationale: chat activity must reflect agent activity (processing vs idle), not just container RUNNING.

@emerson-gray emerson-gray requested a review from a team as a code owner April 30, 2026 01:12
@github-actions
Copy link
Copy Markdown

The latest Buf updates on your PR. Results from workflow buf-pr / buf (pull_request).

BuildFormatLintBreakingUpdated (UTC)
✅ passed✅ passed✅ passed✅ passedApr 30, 2026, 1:12 AM

Copy link
Copy Markdown

@noa-lucent noa-lucent left a comment

Choose a reason for hiding this comment

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

Approved. Minor doc nit left inline.

string agent_name = 17;
// Denormalized display name for runner_id.
string runner_name = 18;
WorkloadAgentState agent_state = 19;
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

[nit] Please add a short field-level comment for clarifying the intended semantics (agent processing state vs container lifecycle), similar to the note you added on .

string agent_name = 17;
// Denormalized display name for runner_id.
string runner_name = 18;
WorkloadAgentState agent_state = 19;
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

[nit] Please add a short field-level comment for agent_state clarifying the intended semantics (agent processing state vs container lifecycle), similar to the note you added on WORKLOAD_STATUS_RUNNING.

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.

3 participants