coderd/database/queries/templateversions.sql (1)

238-238:Rename to align with column and existing query names ("AITask" vs "AIPrompt").

The column is has_ai_task and related queries use AITask (e.g., UpdateTemplateVersionAITaskByJobID). Rename this query to avoid mixed terminology.

Apply within this file, then regenerate sqlc:

- -- name: GetTemplateVersionHasAIPrompt :one+ -- name: GetTemplateVersionHasAITask :one

site/src/api/api.ts (1)

2669-2678:Allow cancellation (AbortSignal) for better UX on long/aborted submissions.

Add optional AbortSignal support so callers can cancel in-flight requests (e.g., route changes).

-  createAITask = async (-    req: TypesGen.CreateAITasksRequest,-  ): Promise<TypesGen.Workspace> => {-    const response = await this.axios.post<TypesGen.Workspace>(-      "/api/experimental/aitasks",-      req,-    );--    return response.data;-  };+  createAITask = async (+    req: TypesGen.CreateAITasksRequest,+    signal?: AbortSignal,+  ): Promise<TypesGen.Workspace> => {+    const response = await this.axios.post<TypesGen.Workspace>(+      "/api/experimental/aitasks",+      req,+      { signal },+    );+    return response.data;+  };

Nit: consider naming the method createAITasks to mirror the plural route/type. Only do this if you can update call sites in the same PR to avoid breakage.

coderd/database/querier.go (1)

358-358:Naming consistency: prefer “HasAITask” to match schema and existing queries.

Interface name stems from SQL. If you adopt the suggested SQL rename (GetTemplateVersionHasAITask), regenerate sqlc so this becomes:

• GetTemplateVersionHasAITask(ctx context.Context, id uuid.UUID) (bool, error)

Do not hand-edit this file; re-generate from the .sql change.

coderd/database/dbmetrics/querymetrics.go (1)

1541-1546:Align wrapper and metric label with “AITask” naming if SQL is renamed.

If you rename the SQL to GetTemplateVersionHasAITask, update this wrapper and label accordingly.

-func (m queryMetricsStore) GetTemplateVersionHasAIPrompt(ctx context.Context, id uuid.UUID) (bool, error) {+func (m queryMetricsStore) GetTemplateVersionHasAITask(ctx context.Context, id uuid.UUID) (bool, error) {   start := time.Now()-  r0, r1 := m.s.GetTemplateVersionHasAIPrompt(ctx, id)-  m.queryLatencies.WithLabelValues("GetTemplateVersionHasAIPrompt").Observe(time.Since(start).Seconds())+  r0, r1 := m.s.GetTemplateVersionHasAITask(ctx, id)+  m.queryLatencies.WithLabelValues("GetTemplateVersionHasAITask").Observe(time.Since(start).Seconds())   return r0, r1 }

coderd/database/dbauthz/dbauthz_test.go (1)

1446-1459:RBAC assertion for TemplateVersion AI-prompt check looks correct

This follows the established pattern where TemplateVersion reads authorize on the parent Template via RBACObject. No issues.

Optional:

• Add a brief comment clarifying the test intent (TemplateVersion read → Template read) to improve readability.
• Consider a negative-path variant (e.g., WithNotAuthorized) similar to other tests where applicable.

coderd/database/queries.sql.go (1)

12873-12887:Nit: align naming with underlying column (has_ai_task) to avoid confusion

The method/SQL name uses “AIPrompt” while the column is has_ai_task. Since this file is generated, consider renaming the query in the source SQL to keep terminology consistent (e.g., “GetTemplateVersionHasAITask”). sqlc will then regenerate matching Go.

Example in coderd/database/queries/templateversions.sql:

-- name: GetTemplateVersionHasAITask :oneSELECT EXISTS (SELECT1FROM template_versionsWHERE id= $1AND has_ai_task);

coderd/aitasks.go (1)

95-102:Clarify error message wording (nit)

Small copy nit: “whether template version has AI Prompt” reads clearer than “if template version has ai prompt.”

-           Message: "Internal error fetching if template version has ai prompt.",+           Message: "Internal error checking whether the template version has AI Prompt.",

codersdk/aitasks.go (1)

56-64:Drop trailing slash in POST path for consistency

Standardize the path to avoid relying on router slash normalization.

-   res, err := c.Request(ctx, http.MethodPost, "/api/experimental/aitasks/", request)+   res, err := c.Request(ctx, http.MethodPost, "/api/experimental/aitasks", request)

site/src/pages/TasksPage/TasksPage.tsx (1)

742-747:Use conditional property for preset_id (minor nit)

Avoid sending undefined fields by conditionally including template_version_preset_id. Keeps payload minimal and avoids any strict decoder issues.

-    const workspace = await API.experimental.createAITask({-      name: `task-${generateWorkspaceName()}`,-      template_version_id: templateVersionId,-      template_version_preset_id: preset_id || undefined,-      prompt,-    });+    const req = {+      name: `task-${generateWorkspaceName()}`,+      template_version_id: templateVersionId,+      prompt,+      ...(preset_id ? { template_version_preset_id: preset_id } : {}),+    };+    const workspace = await API.experimental.createAITask(req);

Configuration used: .coderabbit.yaml
Review profile: CHILL
Plan: Pro

Learnt from: CRPR: coder/coder#0File: .cursorrules:0-0Timestamp: 2025-07-21T14:32:43.064ZLearning: Applies to coderd/coderd.go : The REST API is defined in `coderd/coderd.go` and uses Chi for HTTP routing.

Learnt from: CRPR: coder/coder#0File: .cursorrules:0-0Timestamp: 2025-07-21T14:32:43.064ZLearning: Applies to coderd/coderdtest/**/* : The `coderdtest` package in `coderd/coderdtest/` provides utilities for creating test instances of the Coder server, setting up test users and workspaces, and mocking external components.

Learnt from: CRPR: coder/coder#0File: .cursorrules:0-0Timestamp: 2025-07-21T14:32:43.064ZLearning: Applies to coderdenttest/**/* : Enterprise features have dedicated test utilities in the `coderdenttest` package.

Learnt from: CRPR: coder/coder#0File: .cursorrules:0-0Timestamp: 2025-07-21T14:32:43.064ZLearning: Applies to coderd/dbauthz/*.go : The database authorization (dbauthz) system enforces fine-grained access control across all database operations. All database operations must pass through this layer to ensure security.