Merge pull request #1763 from jlongtine/migrate-message

Create a nice error message for 0.1 plans loaded in 0.2

cmd/dagger/cmd/do.go

@@ -22,7 +22,6 @@ import (
 var doCmd = &cobra.Command{
 	Use:   "do [OPTIONS] ACTION [SUBACTION...]",
 	Short: "Execute a dagger action.",
-	// Args:  cobra.MinimumNArgs(1),
 	PreRun: func(cmd *cobra.Command, args []string) {
 		// Fix Viper bug for duplicate flags:
 		// https://github.com/spf13/viper/issues/233
@@ -32,7 +31,7 @@ var doCmd = &cobra.Command{
 	},
 	Run: func(cmd *cobra.Command, args []string) {
 		if len(args) < 1 {
-			doHelp(cmd, nil)
+			doHelpCmd(cmd, nil)
 			return
 		}
 
@@ -108,46 +107,32 @@ func getTargetPath(args []string) cue.Path {
 	return cue.MakePath(selectors...)
 }
 
-func doHelp(cmd *cobra.Command, _ []string) {
-	w := tabwriter.NewWriter(os.Stdout, 0, 0, 1, ' ', tabwriter.StripEscape)
-	defer w.Flush()
+func doHelpCmd(cmd *cobra.Command, _ []string) {
+	lg := logger.New()
 
-	planPath := viper.GetString("plan")
-
-	var (
-		errorMsg            string
-		loadedMsg           string
-		actionLookupPathMsg string
-		action              *plan.Action
-		actions             []*plan.Action
-	)
+	fmt.Printf("%s\n\n%s", cmd.Short, cmd.UsageString())
 
 	p, err := loadPlan()
 	if err != nil {
-		errorMsg = "Error: failed to load plan\n\n"
-	} else {
-		loadedMsg = "Plan loaded from " + planPath
-		actionLookupPath := getTargetPath(cmd.Flags().Args())
-		action = p.Action().FindByPath(actionLookupPath)
+		lg.Fatal().Err(err).Msg("failed to load plan")
+	}
+
+	target := getTargetPath(cmd.Flags().Args())
+	action := p.Action().FindByPath(target)
 	if action == nil {
-			errorMsg = "Error: action not found\n\n"
-		} else {
-			actions = action.Children
-			actionLookupPathMsg = fmt.Sprintf(`%s:`, actionLookupPath.String())
+		lg.Fatal().Msg(fmt.Sprintf("action %s not found", target.String()))
+		return
 	}
+
+	if len(action.Name) < 1 {
+		return
 	}
-	fmt.Printf(`%s%s
 
-%s
+	fmt.Println("")
+	w := tabwriter.NewWriter(os.Stdout, 0, 0, 1, ' ', tabwriter.StripEscape)
+	defer w.Flush()
 
-%s
-
-%s
-`, errorMsg, cmd.Short, cmd.UsageString(), loadedMsg, actionLookupPathMsg)
-
-	// fmt.Fprintln(w, "Actions\tDescription\tPackage")
-	// fmt.Fprintln(w, "\t\t")
-	for _, a := range actions {
+	for _, a := range action.Children {
 		if !a.Hidden {
 			lineParts := []string{"", a.Name, strings.TrimSpace(a.Comment)}
 			fmt.Fprintln(w, strings.Join(lineParts, "\t"))
@@ -164,7 +149,7 @@ func init() {
 	doCmd.Flags().StringArray("cache-from", []string{},
 		"External cache sources (eg. user/app:cache, type=local,src=path/to/dir)")
 
-	doCmd.SetHelpFunc(doHelp)
+	doCmd.SetHelpFunc(doHelpCmd)
 
 	if err := viper.BindPFlags(doCmd.Flags()); err != nil {
 		panic(err)

plan/plan.go

@@ -2,7 +2,9 @@ package plan
 
 import (
 	"context"
+	"errors"
 	"fmt"
+	"strings"
 
 	"cuelang.org/go/cue"
 	cueflow "cuelang.org/go/tools/flow"
@@ -16,6 +18,7 @@ import (
 )
 
 var (
+	ErrIncompatiblePlan = errors.New("attempting to load a dagger 0.1.0 project.\nPlease upgrade your config to be compatible with this version of dagger. Contact the Dagger team if you need help")
 	ActionSelector      = cue.Str("actions")
 	ClientSelector      = cue.Str("client")
 )
@@ -44,6 +47,12 @@ func Load(ctx context.Context, cfg Config) (*Plan, error) {
 
 	v, err := compiler.Build("", nil, cfg.Args...)
 	if err != nil {
+		errstring := err.Error()
+
+		if strings.Contains(errstring, "cannot find package") && strings.Contains(errstring, "alpha.dagger.io") {
+			return nil, ErrIncompatiblePlan
+		}
+
 		return nil, err
 	}
 

tests/plan.bats

@@ -38,6 +38,14 @@ setup() {
   refute_output --partial 'client.filesystem."./dependent_do".write'
 }
 
+@test "plan/do: nice error message for 0.1.0 projects" {
+  run "$DAGGER" "do" -p ./plan/do/error_message_for_0.1_projects.cue
+  assert_output --partial "attempting to load a dagger 0.1.0 project."
+
+  run "$DAGGER" "do" -p ./plan/do/error_message_for_0.1_projects.cue test
+  assert_output --partial "attempting to load a dagger 0.1.0 project."
+}
+
 @test "plan/hello" {
   # Europa loader handles the cwd differently, therefore we need to CD into the tree at or below the parent of cue.mod
   cd "$TESTDIR"

tests/plan/do/error_message_for_0.1_projects.cue

@@ -0,0 +1,60 @@
+package main
+
+import (
+	"alpha.dagger.io/dagger"
+	"alpha.dagger.io/aws/s3"
+	"alpha.dagger.io/netlify"
+	"alpha.dagger.io/os"
+)
+
+repo: dagger.#Artifact & dagger.#Input
+
+hello: {
+
+	dir: dagger.#Artifact & dagger.#Input
+
+	ctr: os.#Container & {
+		command: """
+			ls -l /src > /tmp/out
+			"""
+		mount: "/src": from: dir
+	}
+
+	f: os.#File & {
+		from: ctr
+		path: "/tmp/out"
+	}
+
+	message: f.contents & dagger.#Output
+}
+
+// Website
+web: {
+	source: os.#Dir & {
+		from: repo
+		path: "web"
+	}
+
+	url: string & dagger.#Output
+
+	// Where to host the website?
+	provider: *"s3" | "netlify" & dagger.#Input
+
+	// Deploy to AWS S3
+	if provider == "s3" {
+		url:    "\(bucket.url)index.html"
+		bucket: s3.#Put & {
+			contentType: "text/html"
+			"source":    source
+		}
+	}
+
+	// Deploy to Netlify
+	if provider == "netlify" {
+		url: site.url
+
+		site: netlify.#Site & {
+			contents: source
+		}
+	}
+}