Apply reviews

- default.yaml: Fix comment placing
- pkg/qemuimgutil: Change `QemuImageUtil.Convert` to returns error on specifying other than `raw.Type`

Signed-off-by: Norio Nomura <[email protected]>

Author: norio-nomura

pkg/qemuimgutil/qemuimgutil.go

@@ -15,6 +15,7 @@ import (
 	"strconv"
 
 	"github.com/lima-vm/go-qcow2reader/image"
+	"github.com/lima-vm/go-qcow2reader/image/raw"
 	"github.com/sirupsen/logrus"
 )
 
@@ -201,8 +202,11 @@ func GetInfo(ctx context.Context, path string) (*Info, error) {
 }
 
 // Convert converts a disk image to raw format.
-// Specified imageType is ignored.
-func (q *QemuImageUtil) Convert(ctx context.Context, _ image.Type, source, dest string, size *int64, allowSourceWithBackingFile bool) error {
+// Currently only raw.Type is supported.
+func (q *QemuImageUtil) Convert(ctx context.Context, imageType image.Type, source, dest string, size *int64, allowSourceWithBackingFile bool) error {
+	if imageType != raw.Type {
+		return fmt.Errorf("QemuImageUtil.Convert only supports raw.Type, got %q", imageType)
+	}
 	if !allowSourceWithBackingFile {
 		info, err := getInfo(ctx, source)
 		if err != nil {

templates/default.yaml

@@ -370,11 +370,11 @@ vmOpts:
     #   riscv64: "max" # (or "host" when running on riscv64 host)
     #   x86_64:  "max" # (or "host" when running on x86_64 host; additional options are appended on Intel Mac)
   vz:
-    diskImageFormat: null
     # Specify the disk image format: "raw" or "asif".
     # Currently only applies to the primary disk image.
     # "asif" requires macOS 26+, and does not support converting back to "raw".
     # 🟢 Builtin default: "raw"
+    diskImageFormat: null
     rosetta:
       # Enable Rosetta inside the VM; needs `vmType: vz`
       # Hint: try `softwareupdate --install-rosetta` if Lima gets stuck at `Installing rosetta...`