coder · dannykopping · Jun 24, 2025 · Jun 18, 2025 · Jun 18, 2025 · Jun 18, 2025
diff --git a/CLAUDE.md b/CLAUDE.md
@@ -103,5 +103,4 @@ Read [cursor rules](.cursorrules).
 
 The frontend is contained in the site folder.
 
-For building Frontend refer to [this document](docs/contributing/frontend.md)
 For building Frontend refer to [this document](docs/about/contributing/frontend.md)
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/dbfake/dbfake.go b/coderd/database/dbfake/dbfake.go
@@ -416,6 +416,7 @@ func (t TemplateVersionBuilder) Do() TemplateVersionResponse {
 			DesiredInstances:    preset.DesiredInstances,
 			InvalidateAfterSecs: preset.InvalidateAfterSecs,
 			SchedulingTimezone:  preset.SchedulingTimezone,
+			IsDefault:           false,
 		})
 	}
 

diff --git a/coderd/database/dbgen/dbgen.go b/coderd/database/dbgen/dbgen.go
@@ -1303,6 +1303,7 @@ func Preset(t testing.TB, db database.Store, seed database.InsertPresetParams) d
 		DesiredInstances:    seed.DesiredInstances,
 		InvalidateAfterSecs: seed.InvalidateAfterSecs,
 		SchedulingTimezone:  seed.SchedulingTimezone,
+		IsDefault:           seed.IsDefault,
 	})
 	require.NoError(t, err, "insert preset")
 	return preset

diff --git a/coderd/database/dbmem/dbmem.go b/coderd/database/dbmem/dbmem.go
@@ -9198,6 +9198,7 @@ func (q *FakeQuerier) InsertPreset(_ context.Context, arg database.InsertPresetP
 			Valid: true,
 		},
 		PrebuildStatus: database.PrebuildStatusHealthy,
+		IsDefault:      arg.IsDefault,
 	}
 	q.presets = append(q.presets, preset)
 	return preset, nil

diff --git a/coderd/database/dump.sql b/coderd/database/dump.sql
diff --git a/coderd/database/migrations/000341_template_version_preset_default.down.sql b/coderd/database/migrations/000341_template_version_preset_default.down.sql
@@ -0,0 +1,2 @@
+DROP INDEX IF EXISTS idx_template_version_presets_default;
+ALTER TABLE template_version_presets DROP COLUMN IF EXISTS is_default;
diff --git a/coderd/database/migrations/000341_template_version_preset_default.up.sql b/coderd/database/migrations/000341_template_version_preset_default.up.sql
@@ -0,0 +1,6 @@
+ALTER TABLE template_version_presets ADD COLUMN is_default BOOLEAN NOT NULL DEFAULT FALSE;
+
+-- Add a unique constraint to ensure only one default preset per template version
+CREATE UNIQUE INDEX idx_template_version_presets_default 
+ON template_version_presets (template_version_id) 
+WHERE is_default = TRUE;
diff --git a/coderd/database/models.go b/coderd/database/models.go
diff --git a/coderd/database/queries.sql.go b/coderd/database/queries.sql.go
diff --git a/coderd/database/queries/presets.sql b/coderd/database/queries/presets.sql
@@ -6,7 +6,8 @@ INSERT INTO template_version_presets (
 	created_at,
 	desired_instances,
 	invalidate_after_secs,
-	scheduling_timezone
+	scheduling_timezone,
+	is_default
 )
 VALUES (
 	@id,
@@ -15,7 +16,8 @@ VALUES (
 	@created_at,
 	@desired_instances,
 	@invalidate_after_secs,
-	@scheduling_timezone
+	@scheduling_timezone,
+	@is_default
 ) RETURNING *;
 
 -- name: InsertPresetParameters :many

diff --git a/coderd/database/unique_constraint.go b/coderd/database/unique_constraint.go
diff --git a/coderd/presets.go b/coderd/presets.go
@@ -41,8 +41,9 @@ func (api *API) templateVersionPresets(rw http.ResponseWriter, r *http.Request)
 	var res []codersdk.Preset
 	for _, preset := range presets {
 		sdkPreset := codersdk.Preset{
-			ID:   preset.ID,
-			Name: preset.Name,
+			ID:      preset.ID,
+			Name:    preset.Name,
+			Default: preset.IsDefault,
 		}
 		for _, presetParam := range presetParams {
 			if presetParam.TemplateVersionPresetID != preset.ID {

diff --git a/coderd/presets_test.go b/coderd/presets_test.go
@@ -1,8 +1,10 @@
 package coderd_test
 
 import (
+	"slices"
 	"testing"
 
+	"github.com/stretchr/testify/assert"
 	"github.com/stretchr/testify/require"
 
 	"github.com/coder/coder/v2/coderd/coderdtest"
@@ -137,3 +139,93 @@ func TestTemplateVersionPresets(t *testing.T) {
 		})
 	}
 }
+
+func TestTemplateVersionPresetsDefault(t *testing.T) {
+	t.Parallel()
+
+	type expectedPreset struct {
+		name      string
+		isDefault bool
+	}
+
+	cases := []struct {
+		name     string
+		presets  []database.InsertPresetParams
+		expected []expectedPreset
+	}{
+		{
+			name:     "no presets",
+			presets:  nil,
+			expected: nil,
+		},
+		{
+			name: "single default preset",
+			presets: []database.InsertPresetParams{
+				{Name: "Default Preset", IsDefault: true},
+			},
+			expected: []expectedPreset{
+				{name: "Default Preset", isDefault: true},
+			},
+		},
+		{
+			name: "single non-default preset",
+			presets: []database.InsertPresetParams{
+				{Name: "Regular Preset", IsDefault: false},
+			},
+			expected: []expectedPreset{
+				{name: "Regular Preset", isDefault: false},
+			},
+		},
+		{
+			name: "mixed presets",
+			presets: []database.InsertPresetParams{
+				{Name: "Default Preset", IsDefault: true},
+				{Name: "Regular Preset", IsDefault: false},
+			},
+			expected: []expectedPreset{
+				{name: "Default Preset", isDefault: true},
+				{name: "Regular Preset", isDefault: false},
+			},
+		},
+	}
+
+	for _, tc := range cases {
+		tc := tc
+		t.Run(tc.name, func(t *testing.T) {
+			t.Parallel()
+			ctx := testutil.Context(t, testutil.WaitShort)
+
+			client, db := coderdtest.NewWithDatabase(t, &coderdtest.Options{IncludeProvisionerDaemon: true})
+			user := coderdtest.CreateFirstUser(t, client)
+			version := coderdtest.CreateTemplateVersion(t, client, user.OrganizationID, nil)
+
+			// Create presets
+			for _, preset := range tc.presets {
+				preset.TemplateVersionID = version.ID
+				_ = dbgen.Preset(t, db, preset)
+			}
+
+			// Get presets via API
+			userSubject, _, err := httpmw.UserRBACSubject(ctx, db, user.UserID, rbac.ScopeAll)
+			require.NoError(t, err)
+			userCtx := dbauthz.As(ctx, userSubject)
+
+			gotPresets, err := client.TemplateVersionPresets(userCtx, version.ID)
+			require.NoError(t, err)
+
+			// Verify results
+			require.Len(t, gotPresets, len(tc.expected))
+
+			for _, expected := range tc.expected {
+				found := slices.ContainsFunc(gotPresets, func(preset codersdk.Preset) bool {
+					if preset.Name != expected.name {
+						return false
+					}
+
+					return assert.Equal(t, expected.isDefault, preset.Default)
+				})
+				require.True(t, found, "Expected preset %s not found", expected.name)
+			}
+		})
+	}
+}
diff --git a/coderd/provisionerdserver/provisionerdserver.go b/coderd/provisionerdserver/provisionerdserver.go
@@ -2230,6 +2230,7 @@ func InsertWorkspacePresetAndParameters(ctx context.Context, db database.Store,
 			DesiredInstances:    desiredInstances,
 			InvalidateAfterSecs: ttl,
 			SchedulingTimezone:  schedulingTimezone,
+			IsDefault:           protoPreset.GetDefault(),
 		})
 		if err != nil {
 			return xerrors.Errorf("insert preset: %w", err)

diff --git a/codersdk/presets.go b/codersdk/presets.go
@@ -14,6 +14,7 @@ type Preset struct {
 	ID         uuid.UUID
 	Name       string
 	Parameters []PresetParameter
+	Default    bool
 }
 
 type PresetParameter struct {

diff --git a/docs/reference/api/schemas.md b/docs/reference/api/schemas.md
Original file line number	Diff line number	Diff line change
Expand Up		@@ -103,5 +103,4 @@ Read [cursor rules](.cursorrules).

		The frontend is contained in the site folder.

		For building Frontend refer to [this document](docs/contributing/frontend.md)
Copy link Contributor Author dannykopping Jun 19, 2025 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others. Learn more. Drive-by: link is dead. Copy link Member aslilac Jun 23, 2025 • edited Loading Choose a reason for hiding this comment The reason will be displayed to describe this comment to others. Learn more. ~~it's in docs/about/contributing/ now. can you leave this as is and I'll do a quick pr to fix the link?~~ sorry, I just looked at the whole file and this diff makes more sense now. the line got duplicated, with one having the out of date path and one having the correct path. you're just removing the stale one. dannykopping reacted with thumbs up emoji
		For building Frontend refer to [this document](docs/about/contributing/frontend.md)
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1,2 @@
		DROP INDEX IF EXISTS idx_template_version_presets_default;
		ALTER TABLE template_version_presets DROP COLUMN IF EXISTS is_default;