review: flagOr already carries the fallback; docs name the rm recovery flags

Author: CMGS

cmd/vm/clone.go

@@ -1,7 +1,6 @@
 package vm
 
 import (
-	"cmp"
 	"errors"
 	"fmt"
 	"path/filepath"
@@ -139,8 +138,8 @@ func (h *Handler) clone(cmd *cobra.Command, srcRec *record, name string) (retErr
 	}
 	tapFlag, _ := cmd.Flags().GetString("tap")
 	r.Tap = tapFlag
-	r.CNIConfDir = cmp.Or(flagOr(cmd, "cni-conf-dir", ""), srcRec.CNIConfDir)
-	r.CNIBinDir = cmp.Or(flagOr(cmd, "cni-bin-dir", ""), srcRec.CNIBinDir)
+	r.CNIConfDir = flagOr(cmd, "cni-conf-dir", srcRec.CNIConfDir)
+	r.CNIBinDir = flagOr(cmd, "cni-bin-dir", srcRec.CNIBinDir)
 	if err = applyNet(cmd, r); err != nil {
 		return err
 	}

cmd/vm/net_linux.go

@@ -27,7 +27,7 @@ import (
 // netScope keys cocoon-macos's host TAP/netns families apart from a co-hosted cocoon's, so neither GC reclaims the other's live guests.
 const netScope = "cm"
 
-// netConf is the cocoon network config: bridge/CNI provisioning shares cocoon's forwarding plane, keyed under our own device family; the CNI dirs resolve flag, then record, then default.
+// netConf is the cocoon network config: bridge/CNI provisioning shares cocoon's forwarding plane, keyed under our own device family.
 func netConf(cmd *cobra.Command, r *record) *config.Config {
 	return &config.Config{
 		RootDir:    home.Dir(cmd),

docs/networking.md

@@ -32,7 +32,9 @@ goes straight to that IP (no port-forward).
 `--cni-conf-dir` (default `/etc/cni/net.d`) and `--cni-bin-dir` (default
 `/opt/cni/bin`) point at a non-standard CNI installation; both are ignored by
 the other net modes and are remembered in the VM record, so `rm` tears the
-NIC down without repeating them.
+NIC down without repeating them; a clone inherits them from its source. A VM
+created before the record carried them takes `rm --cni-conf-dir` and
+`rm --cni-bin-dir` once.
 
 ## Clones