cmd/fix: allow fix to load all packages in a directory

Unlike `cue eval`, `cue fix` should load and operate on all cue files in
a directory, regardless of whether they belong to the same package or
not.

Fixes #3405

Signed-off-by: Matthew Sackman <[email protected]>
Change-Id: If79c6d019b9496cfefc833d3fa5e5b6a98ae0ade
Reviewed-on: https://review.gerrithub.io/c/cue-lang/cue/+/1200351
TryBot-Result: CUEcueckoo <[email protected]>
Reviewed-by: Daniel Martí <[email protected]>
Unity-Result: CUE porcuepine <[email protected]>

Author: cuematthew

cmd/cue/cmd/fix.go

@@ -74,8 +74,9 @@ func runFixAll(cmd *Command, args []string) error {
 	}
 
 	instances := load.Instances(args, &load.Config{
-		Tests: true,
-		Tools: true,
+		Tests:   true,
+		Tools:   true,
+		Package: "*",
 	})
 
 	errs := fix.Instances(instances, opts...)

cmd/cue/cmd/testdata/script/fix.txtar

@@ -0,0 +1,33 @@
+exec cue fix ./...
+# Make sure we fix all files in a directory, even if they're a mix of packages (or no packages).
+cmp p/one.cue p/one.cue.fixed
+cmp p/two.cue p/two.cue.fixed
+# TODO: the following cmp fails because of a bug in fix/fix.go
+! cmp p/three.cue p/three.cue.fixed
+# TODO: the added imports have unnecessary name-mangling. https://cuelang.org/issue/3408
+-- p/one.cue --
+package one
+
+out: ["foo"] + ["bar"]
+-- p/two.cue --
+package two
+
+out: 3 * ["baz"]
+-- p/three.cue --
+out: ["a"] + ((["a"]*7) + ["gh"])
+-- p/one.cue.fixed --
+package one
+
+import list6c6973 "list"
+
+out: list6c6973.Concat([["foo"], ["bar"]])
+-- p/two.cue.fixed --
+package two
+
+import list6c6973 "list"
+
+out: list6c6973.Repeat(["baz"], 3)
+-- p/three.cue.fixed --
+import list6c6973 "list"
+
+out: list6c6973.Concat([["a"], (list6c6973.Concat([(list6c6973.Repeat(["a"], 7)), ["gh"]]))])