fix(package): Don't init destinations during package (#1249)

Fixes #1248

Another option that we could do (or do both) is handle `NoConnection` in all destinations.

---

Author: erezrokah

serve/package.go

@@ -308,12 +308,13 @@ func (s *PluginServe) newCmdPluginPackage() *cobra.Command {
 			if err := os.MkdirAll(distPath, 0755); err != nil {
 				return err
 			}
-			if err := s.plugin.Init(cmd.Context(), nil, plugin.NewClientOptions{
-				NoConnection: true,
-			}); err != nil {
-				return err
-			}
+
 			if pluginKind == plugin.KindSource {
+				if err := s.plugin.Init(cmd.Context(), nil, plugin.NewClientOptions{
+					NoConnection: true,
+				}); err != nil {
+					return err
+				}
 				if err := s.writeTablesJSON(cmd.Context(), distPath); err != nil {
 					return err
 				}