coder · code-asher · Jun 17, 2025 · Jun 17, 2025 · hugodutka · Jun 18, 2025
diff --git a/cli/exp_mcp.go b/cli/exp_mcp.go
@@ -585,10 +585,10 @@ func (s *mcpServer) startWatcher(ctx context.Context, inv *serpent.Invocation) {
 			case event := <-eventsCh:
 				switch ev := event.(type) {
 				case agentapi.EventStatusChange:
-					// If the screen is stable, assume complete.
+					// If the screen is stable, report idle.
 					state := codersdk.WorkspaceAppStatusStateWorking
 					if ev.Status == agentapi.StatusStable {
-						state = codersdk.WorkspaceAppStatusStateComplete
+						state = codersdk.WorkspaceAppStatusStateIdle
 					}
 					err := s.queue.Push(taskReport{
 						state: state,

diff --git a/cli/exp_mcp_test.go b/cli/exp_mcp_test.go
@@ -900,7 +900,7 @@ func TestExpMcpReporter(t *testing.T) {
 			{
 				event: makeStatusEvent(agentapi.StatusStable),
 				expected: &codersdk.WorkspaceAppStatus{
-					State:   codersdk.WorkspaceAppStatusStateComplete,
+					State:   codersdk.WorkspaceAppStatusStateIdle,
 					Message: "doing work",
 					URI:     "https://dev.coder.com",
 				},
@@ -948,7 +948,7 @@ func TestExpMcpReporter(t *testing.T) {
 			{
 				event: makeStatusEvent(agentapi.StatusStable),
 				expected: &codersdk.WorkspaceAppStatus{
-					State:   codersdk.WorkspaceAppStatusStateComplete,
+					State:   codersdk.WorkspaceAppStatusStateIdle,
 					Message: "oops",
 					URI:     "",
 				},

diff --git a/coderd/apidoc/docs.go b/coderd/apidoc/docs.go
diff --git a/coderd/apidoc/swagger.json b/coderd/apidoc/swagger.json
diff --git a/coderd/database/dump.sql b/coderd/database/dump.sql
diff --git a/coderd/database/migrations/000338_workspace_app_status_idle.down.sql b/coderd/database/migrations/000338_workspace_app_status_idle.down.sql
@@ -0,0 +1 @@
+-- It is not possible to delete enum values.
diff --git a/coderd/database/migrations/000338_workspace_app_status_idle.up.sql b/coderd/database/migrations/000338_workspace_app_status_idle.up.sql
@@ -0,0 +1 @@
+ALTER TYPE workspace_app_status_state ADD VALUE IF NOT EXISTS 'idle';
diff --git a/coderd/database/models.go b/coderd/database/models.go
diff --git a/coderd/workspaceagents.go b/coderd/workspaceagents.go
@@ -359,7 +359,10 @@ func (api *API) patchWorkspaceAgentAppStatus(rw http.ResponseWriter, r *http.Req
 	}
 
 	switch req.State {
-	case codersdk.WorkspaceAppStatusStateComplete, codersdk.WorkspaceAppStatusStateFailure, codersdk.WorkspaceAppStatusStateWorking: // valid states
+	case codersdk.WorkspaceAppStatusStateComplete,
+		codersdk.WorkspaceAppStatusStateFailure,
+		codersdk.WorkspaceAppStatusStateWorking,
+		codersdk.WorkspaceAppStatusStateIdle: // valid states
 	default:
 		httpapi.Write(ctx, rw, http.StatusBadRequest, codersdk.Response{
 			Message: "Invalid state provided.",

diff --git a/codersdk/toolsdk/toolsdk.go b/codersdk/toolsdk/toolsdk.go
@@ -191,7 +191,7 @@ Bad Tasks
 Use the "state" field to indicate your progress. Periodically report
 progress with state "working" to keep the user updated. It is not possible to send too many updates!
 
-ONLY report a "complete" or "failure" state if you have FULLY completed the task.
+ONLY report an "idle" or "failure" state if you have FULLY completed the task.
 `,
 		Schema: aisdk.Schema{
 			Properties: map[string]any{
@@ -205,10 +205,10 @@ ONLY report a "complete" or "failure" state if you have FULLY completed the task
 				},
 				"state": map[string]any{
 					"type":        "string",
-					"description": "The state of your task. This can be one of the following: working, complete, or failure. Select the state that best represents your current progress.",
+					"description": "The state of your task. This can be one of the following: working, idle, or failure. Select the state that best represents your current progress.",
 					"enum": []string{
 						string(codersdk.WorkspaceAppStatusStateWorking),
-						string(codersdk.WorkspaceAppStatusStateComplete),
+						string(codersdk.WorkspaceAppStatusStateIdle),
 						string(codersdk.WorkspaceAppStatusStateFailure),
 					},
 				},

diff --git a/codersdk/workspaceapps.go b/codersdk/workspaceapps.go
@@ -19,6 +19,7 @@ type WorkspaceAppStatusState string
 
 const (
 	WorkspaceAppStatusStateWorking  WorkspaceAppStatusState = "working"
+	WorkspaceAppStatusStateIdle     WorkspaceAppStatusState = "idle"
 	WorkspaceAppStatusStateComplete WorkspaceAppStatusState = "complete"
 	WorkspaceAppStatusStateFailure  WorkspaceAppStatusState = "failure"
 )

diff --git a/docs/reference/api/builds.md b/docs/reference/api/builds.md
diff --git a/docs/reference/api/schemas.md b/docs/reference/api/schemas.md
diff --git a/docs/reference/api/templates.md b/docs/reference/api/templates.md
diff --git a/site/src/api/typesGenerated.ts b/site/src/api/typesGenerated.ts
diff --git a/site/src/modules/apps/AppStatusStateIcon.tsx b/site/src/modules/apps/AppStatusStateIcon.tsx
@@ -5,6 +5,7 @@ import {
 	CircleAlertIcon,
 	CircleCheckIcon,
 	HourglassIcon,
+	SquareIcon,
 	TriangleAlertIcon,
 } from "lucide-react";
 import type { FC } from "react";
@@ -26,6 +27,10 @@ export const AppStatusStateIcon: FC<AppStatusStateIconProps> = ({
 	const className = cn(["size-4 shrink-0", customClassName]);
 
 	switch (state) {
+		case "idle":
+			return (
+				<SquareIcon className={cn(["text-content-secondary", className])} />
+			);
 		case "complete":
 			return (
 				<CircleCheckIcon className={cn(["text-content-success", className])} />

diff --git a/site/src/modules/workspaces/WorkspaceAppStatus/WorkspaceAppStatus.stories.tsx b/site/src/modules/workspaces/WorkspaceAppStatus/WorkspaceAppStatus.stories.tsx
@@ -39,6 +39,26 @@ export const Working: Story = {
 	},
 };
 
+export const Idle: Story = {
+	args: {
+		status: {
+			...MockWorkspaceAppStatus,
+			state: "idle",
+			message: "Done for now",
+		},
+	},
+};
+
+export const NoMessage: Story = {
+	args: {
+		status: {
+			...MockWorkspaceAppStatus,
+			state: "idle",
+			message: "",
+		},
+	},
+};
+
 export const LongMessage: Story = {
 	args: {
 		status: {

diff --git a/site/src/modules/workspaces/WorkspaceAppStatus/WorkspaceAppStatus.tsx b/site/src/modules/workspaces/WorkspaceAppStatus/WorkspaceAppStatus.tsx
@@ -5,6 +5,7 @@ import {
 	TooltipProvider,
 	TooltipTrigger,
 } from "components/Tooltip/Tooltip";
+import capitalize from "lodash/capitalize";
 import { AppStatusStateIcon } from "modules/apps/AppStatusStateIcon";
 import { cn } from "utils/cn";
 
@@ -25,6 +26,7 @@ export const WorkspaceAppStatus = ({
 		);
 	}
 
+	const message = status.message || capitalize(status.state);
 	return (
 		<div className="flex flex-col text-content-secondary">
 			<TooltipProvider>
@@ -40,11 +42,11 @@ export const WorkspaceAppStatus = ({
 								})}
 							/>
 							<span className="whitespace-nowrap max-w-72 overflow-hidden text-ellipsis text-sm text-content-primary font-medium">
-								{status.message}
+								{message}
 							</span>
 						</div>
 					</TooltipTrigger>
-					<TooltipContent>{status.message}</TooltipContent>
+					<TooltipContent>{message}</TooltipContent>
 				</Tooltip>
 			</TooltipProvider>
 			<span className="text-xs first-letter:uppercase block pl-6">

diff --git a/site/src/pages/WorkspacePage/AppStatuses.stories.tsx b/site/src/pages/WorkspacePage/AppStatuses.stories.tsx
@@ -48,6 +48,40 @@ export const WorkingState: Story = {
 	},
 };
 
+export const IdleState: Story = {
+	args: {
+		agent: mockAgent([
+			{
+				...MockWorkspaceAppStatus,
+				id: "status-8",
+				icon: "",
+				message: "Done for now",
+				created_at: createTimestamp(5, 20),
+				uri: "",
+				state: "idle" as const,
+			},
+			...MockWorkspaceAppStatuses,
+		]),
+	},
+};
+
+export const NoMessage: Story = {
+	args: {
+		agent: mockAgent([
+			{
+				...MockWorkspaceAppStatus,
+				id: "status-8",
+				icon: "",
+				message: "",
+				created_at: createTimestamp(5, 20),
+				uri: "",
+				state: "idle" as const,
+			},
+			...MockWorkspaceAppStatuses,
+		]),
+	},
+};
+
 export const LongStatusText: Story = {
 	args: {
 		agent: mockAgent([

diff --git a/site/src/pages/WorkspacePage/AppStatuses.tsx b/site/src/pages/WorkspacePage/AppStatuses.tsx
@@ -12,6 +12,7 @@ import {
 	TooltipProvider,
 	TooltipTrigger,
 } from "components/Tooltip/Tooltip";
+import capitalize from "lodash/capitalize";
 import { timeFrom } from "utils/time";
 
 import {
@@ -77,7 +78,7 @@ export const AppStatuses: FC<AppStatusesProps> = ({
 					<div className="text-sm font-medium text-content-primary flex items-center gap-2 ">
 						<AppStatusStateIcon state={latestStatus.state} latest />
 						<span className="block flex-1 whitespace-nowrap overflow-hidden text-ellipsis">
-							{latestStatus.message}
+							{latestStatus.message || capitalize(latestStatus.state)}
 						</span>
 					</div>
 					<span className="text-xs text-content-secondary first-letter:uppercase block pl-[26px]">
@@ -160,7 +161,7 @@ export const AppStatuses: FC<AppStatusesProps> = ({
 										latest={false}
 										className="size-icon-xs w-[18px]"
 									/>
-									{status.message}
+									{status.message || capitalize(status.state)}
 								</span>
 								<span className="text-2xs text-content-secondary first-letter:uppercase block pl-[26px]">
 									{formattedTimestamp}
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1 @@
		-- It is not possible to delete enum values.
Copy link Contributor hugodutka Jun 18, 2025 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others. Learn more. I don't think that's true. These steps should revert the enum to its original set of values: Replace any occurences of new enum values with different values Rename the enum to something else Re-create the original enum Update columns to use the new enum Drop the renamed enum Here are the SQL statements you'd use: https://blink.so/chat/Revert-Enum-Value-in-Postgres-5vVDbfY5A7ZOLPxyuqVgsG Copy link Member Author code-asher Jun 18, 2025 • edited Loading Choose a reason for hiding this comment The reason will be displayed to describe this comment to others. Learn more. Ah yeah I meant there is no `ALTER DELETE` to mirror `ALTER ADD` but yeah we could recreate the entire enum. But from what I saw this is how our other migrations handle enum downgrades. I am not sure if that means it is a good idea, but it is consistent at least. Although...I wonder if this causes issues with queries or any enum validation in the Go code. Maybe we need to add downgrades to all of these enums. 😢 It would also mean all the idle states would be incorrect (although maybe renaming to `complete` would not be too bad) which is unfortunate.
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1 @@
		ALTER TYPE workspace_app_status_state ADD VALUE IF NOT EXISTS 'idle';