fix: preserve sync:stop event emission for noWatch mode

Address review feedback from @zerbitx: the initial fix disabled
RestartOnError for noWatch which also prevented sync:stop from being
emitted. Add an else branch that registers a lighter goroutine for
noWatch mode that:
- Emits the sync:stop hook when context is cancelled or sync completes
- Does NOT call parent.Kill(nil) (to avoid breaking SSH injection)

This preserves the sync:stop lifecycle event while still fixing the
original bug where context cancellation killed the parent tomb.

Author: raman1236

pkg/devspace/services/sync/controller.go

@@ -226,6 +226,21 @@ func (c *controller) startWithWait(ctx devspacecontext.Context, options *Options
 			}
 			return nil
 		})
+	} else {
+		// For noWatch syncs: emit sync:stop when context is cancelled or sync
+		// completes, but do NOT kill the parent tomb (to avoid breaking SSH).
+		parent.Go(func() error {
+			select {
+			case <-ctx.Context().Done():
+			case <-onDone:
+			}
+			client.Stop(nil)
+			hook.LogExecuteHooks(ctx.WithLogger(options.SyncLog), map[string]interface{}{
+				"sync_config": options.SyncConfig,
+			}, hook.EventsForSingle("stop:sync", options.Name).With("sync.stop")...)
+			ctx.Log().Debugf("Stopped sync %s", options.SyncConfig.Path)
+			return nil
+		})
 	}
 
 	return nil