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 :one
SELECT EXISTS (
  SELECT 1
  FROM template_versions
  WHERE id = $1 AND 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: CR
PR: coder/coder#0
File: .cursorrules:0-0
Timestamp: 2025-07-21T14:32:43.064Z
Learning: Applies to coderd/coderd.go : The REST API is defined in `coderd/coderd.go` and uses Chi for HTTP routing.

Learnt from: CR
PR: coder/coder#0
File: .cursorrules:0-0
Timestamp: 2025-07-21T14:32:43.064Z
Learning: 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: CR
PR: coder/coder#0
File: .cursorrules:0-0
Timestamp: 2025-07-21T14:32:43.064Z
Learning: Applies to coderdenttest/**/* : Enterprise features have dedicated test utilities in the `coderdenttest` package.

Learnt from: CR
PR: coder/coder#0
File: .cursorrules:0-0
Timestamp: 2025-07-21T14:32:43.064Z
Learning: 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.