engine: separate out Cmd creation from Cmd running (#4289)

ServerController is getting closer to what typical "user-authored" controller will look like.
It doesn't manage any work directly - it simply creates new Cmd objects that do the real work.

internal/cli/wire.go

@@ -88,6 +88,7 @@ var BaseWireSet = wire.NewSet(
 	local.ProvideExecer,
 	local.ProvideProberManager,
 	local.NewController,
+	local.NewServerController,
 	k8swatch.NewPodWatcher,
 	k8swatch.NewServiceWatcher,
 	k8swatch.NewEventWatchManager,

internal/engine/local/actions.go

@@ -1,5 +1,7 @@
 package local
 
+import "github.com/tilt-dev/tilt/internal/store"
+
 type CmdCreateAction struct {
 	Cmd *Cmd
 }
@@ -8,14 +10,36 @@ func NewCmdCreateAction(cmd *Cmd) CmdCreateAction {
 	return CmdCreateAction{Cmd: cmd.DeepCopy()}
 }
 
+var _ store.Summarizer = CmdCreateAction{}
+
 func (CmdCreateAction) Action() {}
 
-type CmdUpdateAction struct {
+func (a CmdCreateAction) Summarize(s *store.ChangeSummary) {
+	if s.CmdSpecs == nil {
+		s.CmdSpecs = make(map[string]bool)
+	}
+	s.CmdSpecs[a.Cmd.Name] = true
+}
+
+type CmdUpdateStatusAction struct {
 	Cmd *Cmd
 }
 
-func NewCmdUpdateAction(cmd *Cmd) CmdUpdateAction {
-	return CmdUpdateAction{Cmd: cmd.DeepCopy()}
+func NewCmdUpdateStatusAction(cmd *Cmd) CmdUpdateStatusAction {
+	return CmdUpdateStatusAction{Cmd: cmd.DeepCopy()}
 }
 
-func (CmdUpdateAction) Action() {}
+func (CmdUpdateStatusAction) Action() {}
+
+type CmdDeleteAction struct {
+	Name string
+}
+
+func (CmdDeleteAction) Action() {}
+
+func (a CmdDeleteAction) Summarize(s *store.ChangeSummary) {
+	if s.CmdSpecs == nil {
+		s.CmdSpecs = make(map[string]bool)
+	}
+	s.CmdSpecs[a.Name] = true
+}