Prevent prebuild deletion from failing because old params were passed in

dannykopping · dannykopping · commit 8cd6a6fb353c · 2025-02-14T12:17:12.000Z
Signed-off-by: Danny Kopping &lt;danny@coder.com&gt;
diff --git a/coderd/prebuilds/controller.go b/coderd/prebuilds/controller.go
@@ -417,8 +417,13 @@ func (c Controller) provision(ctx context.Context, db database.Store, prebuildID
 		Initiator(PrebuildOwnerUUID).
 		ActiveVersion().
 		VersionID(template.ActiveVersionID).
-		MarkPrebuild().
-		RichParameterValues(params)
+		MarkPrebuild()
+
+	// We only inject the required params when the prebuild is being created.
+	// This mirrors the behaviour of regular workspace deletion (see cli/delete.go).
+	if transition != database.WorkspaceTransitionDelete {
+		builder = builder.RichParameterValues(params)
+	}
 
 	_, provisionerJob, _, err := builder.Build(
 		ctx,
