NotificationsYou must be signed in to change notification settings
Fork907
Star10k

feat: add app status tracking to the backend#17163

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Merged

kylecarbs merged 7 commits intomainfromstatuses

Mar 31, 2025

Merged

feat: add app status tracking to the backend#17163

kylecarbs merged 7 commits intomainfromstatuses

Mar 31, 2025

Conversation

Copy link

Member

kylecarbs commentedMar 28, 2025

This does ~95% of the backend work required to integrate the AI work.

Most left to integrate from the tasks branch is just frontend, which will be a lot smaller I believe.

The real difference between this branch and that one is the abstraction -- this now attaches statuses to apps, and returns the latest status reported as part of a workspace.

This change enables us to have a similar UX to in the tasks branch, but for agents other than Claude Code as well. Any app can report status now.

feat: add app status tracking to the backend

f0b584f

kylecarbs requested a review fromjohnstcn

March 28, 2025 20:02

kylecarbs self-assigned this

Mar 28, 2025

johnstcn reviewed

Mar 28, 2025

View reviewed changes

coderd/database/migrations/000313_workspace_app_statuses.up.sqlShow resolvedHide resolved

coderd/database/dbmem/dbmem.go

		defer q.mutex.Unlock()

		status := database.WorkspaceAppStatus{
		ID: arg.ID,

Copy link

Member

johnstcnMar 28, 2025

Choose a reason for hiding this comment

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

Given that we generate the UUID in the database, we should avoid inserting UUIDs.

Copy link

MemberAuthor

kylecarbsMar 29, 2025

Choose a reason for hiding this comment

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

Sure.

coderd/database/queries/workspaceapps.sql

Comment on lines +47 to +49

		INSERT INTO workspace_app_statuses (id, created_at, workspace_id, agent_id, app_id, state, message, needs_user_attention, uri, icon)
		VALUES ($1, $2, $3, $4, $5, $6, $7, $8, $9, $10)
		RETURNING *;

Copy link

Member