refactor: revert interface changes

coder · evgeniy-scherbina · Apr 24, 2025 · Apr 17, 2025 · Apr 22, 2025 · Apr 21, 2025
commit e7b7f80287d598afe41664d375525e94a219a2a7
diff --git a/coderd/prebuilds/api.go b/coderd/prebuilds/api.go
@@ -26,35 +26,11 @@ type ReconciliationOrchestrator interface {
 	Stop(ctx context.Context, cause error)
 }
 
-// Reconciler defines the core operations for managing prebuilds.
-// It provides both high-level orchestration (ReconcileAll) and lower-level operations
-// for more fine-grained control (SnapshotState, ReconcilePreset, CalculateActions).
-// All database operations must be performed within repeatable-read transactions
-// to ensure consistency.
 type Reconciler interface {
 	// ReconcileAll orchestrates the reconciliation of all prebuilds across all templates.
 	// It takes a global snapshot of the system state and then reconciles each preset
 	// in parallel, creating or deleting prebuilds as needed to reach their desired states.
-	// For more fine-grained control, you can use the lower-level methods SnapshotState
-	// and ReconcilePreset directly.
 	ReconcileAll(ctx context.Context) error
-
-	// SnapshotState captures the current state of all prebuilds across templates.
-	// It creates a global database snapshot that can be viewed as a collection of PresetSnapshots,
-	// each representing the state of prebuilds for a specific preset.
-	// MUST be called inside a repeatable-read transaction.
-	SnapshotState(ctx context.Context, store database.Store) (*GlobalSnapshot, error)
-
-	// ReconcilePreset handles a single PresetSnapshot, determining and executing
-	// the required actions (creating or deleting prebuilds) based on the current state.
-	// MUST be called inside a repeatable-read transaction.
-	ReconcilePreset(ctx context.Context, snapshot PresetSnapshot) error
-
-	// CalculateActions determines what actions are needed to reconcile a preset's prebuilds
-	// to their desired state. This includes creating new prebuilds, deleting excess ones,
-	// or waiting due to backoff periods.
-	// MUST be called inside a repeatable-read transaction.
-	CalculateActions(ctx context.Context, state PresetSnapshot) (*ReconciliationActions, error)
 }
 
 type Claimer interface {