fix: remove UnknownFlags whitelist from subcommands

cmd/attach.go

@@ -31,12 +31,11 @@ var attachConfig = config.NewAttach()
 
 // attachCmd represents the modctl command for attach.
 var attachCmd = &cobra.Command{
-	Use:                "attach [flags] <file>",
-	Short:              "Attach the file to an existing model artifact.",
-	Args:               cobra.ExactArgs(1),
-	DisableAutoGenTag:  true,
-	SilenceUsage:       true,
-	FParseErrWhitelist: cobra.FParseErrWhitelist{UnknownFlags: true},
+	Use:               "attach [flags] <file>",
+	Short:             "Attach the file to an existing model artifact.",
+	Args:              cobra.ExactArgs(1),
+	DisableAutoGenTag: true,
+	SilenceUsage:      true,
 	RunE: func(cmd *cobra.Command, args []string) error {
 		if err := attachConfig.Validate(); err != nil {
 			return err

cmd/build.go

@@ -31,12 +31,11 @@ var buildConfig = config.NewBuild()
 
 // buildCmd represents the modctl command for build.
 var buildCmd = &cobra.Command{
-	Use:                "build [flags] <path>",
-	Short:              "Build the model artifact with the context by specified path.",
-	Args:               cobra.ExactArgs(1),
-	DisableAutoGenTag:  true,
-	SilenceUsage:       true,
-	FParseErrWhitelist: cobra.FParseErrWhitelist{UnknownFlags: true},
+	Use:               "build [flags] <path>",
+	Short:             "Build the model artifact with the context by specified path.",
+	Args:              cobra.ExactArgs(1),
+	DisableAutoGenTag: true,
+	SilenceUsage:      true,
 	RunE: func(cmd *cobra.Command, args []string) error {
 		if err := buildConfig.Validate(); err != nil {
 			return err

cmd/extract.go

@@ -31,12 +31,11 @@ var extractConfig = config.NewExtract()
 
 // extractCmd represents the modctl command for extract.
 var extractCmd = &cobra.Command{
-	Use:                "extract <target> --output <output>",
-	Short:              "Extract the model artifact to the output path, which can restore the initial state of the model files.",
-	Args:               cobra.ExactArgs(1),
-	DisableAutoGenTag:  true,
-	SilenceUsage:       true,
-	FParseErrWhitelist: cobra.FParseErrWhitelist{UnknownFlags: true},
+	Use:               "extract <target> --output <output>",
+	Short:             "Extract the model artifact to the output path, which can restore the initial state of the model files.",
+	Args:              cobra.ExactArgs(1),
+	DisableAutoGenTag: true,
+	SilenceUsage:      true,
 	RunE: func(cmd *cobra.Command, args []string) error {
 		if err := extractConfig.Validate(); err != nil {
 			return err

cmd/fetch.go

@@ -31,12 +31,11 @@ var fetchConfig = config.NewFetch()
 
 // fetchCmd represents the modctl command for fetch.
 var fetchCmd = &cobra.Command{
-	Use:                "fetch [flags] <target>",
-	Short:              "Fetch can retrieve files from the remote model repository, enabling selective download of partial model files by filtering based on file path patterns.",
-	Args:               cobra.ExactArgs(1),
-	DisableAutoGenTag:  true,
-	SilenceUsage:       true,
-	FParseErrWhitelist: cobra.FParseErrWhitelist{UnknownFlags: true},
+	Use:               "fetch [flags] <target>",
+	Short:             "Fetch can retrieve files from the remote model repository, enabling selective download of partial model files by filtering based on file path patterns.",
+	Args:              cobra.ExactArgs(1),
+	DisableAutoGenTag: true,
+	SilenceUsage:      true,
 	RunE: func(cmd *cobra.Command, args []string) error {
 		if err := fetchConfig.Validate(); err != nil {
 			return err

cmd/inspect.go

@@ -32,12 +32,11 @@ var inspectConfig = config.NewInspect()
 
 // inspectCmd represents the modctl command for inspect.
 var inspectCmd = &cobra.Command{
-	Use:                "inspect [flags] <target>",
-	Short:              "Inspect can help to analyze the composition of model artifact.",
-	Args:               cobra.ExactArgs(1),
-	DisableAutoGenTag:  true,
-	SilenceUsage:       true,
-	FParseErrWhitelist: cobra.FParseErrWhitelist{UnknownFlags: true},
+	Use:               "inspect [flags] <target>",
+	Short:             "Inspect can help to analyze the composition of model artifact.",
+	Args:              cobra.ExactArgs(1),
+	DisableAutoGenTag: true,
+	SilenceUsage:      true,
 	RunE: func(cmd *cobra.Command, args []string) error {
 		return runInspect(context.Background(), args[0])
 	},

cmd/list.go

@@ -31,12 +31,11 @@ import (
 
 // listCmd represents the modctl command for list.
 var listCmd = &cobra.Command{
-	Use:                "ls",
-	Short:              "List will lists the current built model artifacts from local storage.",
-	Args:               cobra.NoArgs,
-	DisableAutoGenTag:  true,
-	SilenceUsage:       true,
-	FParseErrWhitelist: cobra.FParseErrWhitelist{UnknownFlags: true},
+	Use:               "ls",
+	Short:             "List will lists the current built model artifacts from local storage.",
+	Args:              cobra.NoArgs,
+	DisableAutoGenTag: true,
+	SilenceUsage:      true,
 	RunE: func(cmd *cobra.Command, args []string) error {
 		return runList(context.Background())
 	},

cmd/login.go

@@ -43,10 +43,9 @@ modctl login -u foo registry-1.docker.io
 # login to registry served over http:
 modctl login -u foo --plain-http registry-insecure.io
 `,
-	Args:               cobra.ExactArgs(1),
-	DisableAutoGenTag:  true,
-	SilenceUsage:       true,
-	FParseErrWhitelist: cobra.FParseErrWhitelist{UnknownFlags: true},
+	Args:              cobra.ExactArgs(1),
+	DisableAutoGenTag: true,
+	SilenceUsage:      true,
 	RunE: func(cmd *cobra.Command, args []string) error {
 		if err := loginConfig.Validate(); err != nil {
 			return err

cmd/logout.go

@@ -28,12 +28,11 @@ import (
 
 // logoutCmd represents the modctl command for logout.
 var logoutCmd = &cobra.Command{
-	Use:                "logout [flags]",
-	Short:              "Logout from a registry.",
-	Args:               cobra.ExactArgs(1),
-	DisableAutoGenTag:  true,
-	SilenceUsage:       true,
-	FParseErrWhitelist: cobra.FParseErrWhitelist{UnknownFlags: true},
+	Use:               "logout [flags]",
+	Short:             "Logout from a registry.",
+	Args:              cobra.ExactArgs(1),
+	DisableAutoGenTag: true,
+	SilenceUsage:      true,
 	RunE: func(cmd *cobra.Command, args []string) error {
 		return runLogout(context.Background(), args[0])
 	},

cmd/modelfile/generate.go

@@ -65,10 +65,9 @@ Full URLs with domain names will auto-detect the provider.`,
 
   # Generate with metadata overrides
   modctl modelfile generate ./my-model-dir --name my-custom-model --family llama3`,
-	Args:               cobra.MaximumNArgs(1),
-	DisableAutoGenTag:  true,
-	SilenceUsage:       true,
-	FParseErrWhitelist: cobra.FParseErrWhitelist{UnknownFlags: true},
+	Args:              cobra.MaximumNArgs(1),
+	DisableAutoGenTag: true,
+	SilenceUsage:      true,
 	RunE: func(cmd *cobra.Command, args []string) error {
 		// If model-url is provided, path is optional
 		workspace := "."

cmd/modelfile/modelfile.go

@@ -23,12 +23,11 @@ import (
 
 // RootCmd represents the modelfile tools command for modelfile operation.
 var RootCmd = &cobra.Command{
-	Use:                "modelfile",
-	Short:              "A command line tool for modelfile operation",
-	Args:               cobra.ExactArgs(1),
-	DisableAutoGenTag:  true,
-	SilenceUsage:       true,
-	FParseErrWhitelist: cobra.FParseErrWhitelist{UnknownFlags: true},
+	Use:               "modelfile",
+	Short:             "A command line tool for modelfile operation",
+	Args:              cobra.ExactArgs(1),
+	DisableAutoGenTag: true,
+	SilenceUsage:      true,
 	RunE: func(cmd *cobra.Command, args []string) error {
 		return nil
 	},

cmd/prune.go

@@ -31,12 +31,11 @@ var pruneConfig = config.NewPrune()
 
 // pruneCmd represents the modctl command for prune.
 var pruneCmd = &cobra.Command{
-	Use:                "prune [flags]",
-	Short:              "Prune can help to cleanup useless manifests and blobs in the local storage.",
-	Args:               cobra.NoArgs,
-	DisableAutoGenTag:  true,
-	SilenceUsage:       true,
-	FParseErrWhitelist: cobra.FParseErrWhitelist{UnknownFlags: true},
+	Use:               "prune [flags]",
+	Short:             "Prune can help to cleanup useless manifests and blobs in the local storage.",
+	Args:              cobra.NoArgs,
+	DisableAutoGenTag: true,
+	SilenceUsage:      true,
 	RunE: func(cmd *cobra.Command, args []string) error {
 		return runPrune(context.Background())
 	},

cmd/pull.go

@@ -31,12 +31,11 @@ var pullConfig = config.NewPull()
 
 // pullCmd represents the modctl command for pull.
 var pullCmd = &cobra.Command{
-	Use:                "pull [flags] <target>",
-	Short:              "Pull a model artifact from the remote registry.",
-	Args:               cobra.ExactArgs(1),
-	DisableAutoGenTag:  true,
-	SilenceUsage:       true,
-	FParseErrWhitelist: cobra.FParseErrWhitelist{UnknownFlags: true},
+	Use:               "pull [flags] <target>",
+	Short:             "Pull a model artifact from the remote registry.",
+	Args:              cobra.ExactArgs(1),
+	DisableAutoGenTag: true,
+	SilenceUsage:      true,
 	RunE: func(cmd *cobra.Command, args []string) error {
 		if err := pullConfig.Validate(); err != nil {
 			return err

cmd/push.go

@@ -31,12 +31,11 @@ var pushConfig = config.NewPush()
 
 // pushCmd represents the modctl command for push.
 var pushCmd = &cobra.Command{
-	Use:                "push [flags] <target>",
-	Short:              "Push a model artifact to the remote registry.",
-	Args:               cobra.ExactArgs(1),
-	DisableAutoGenTag:  true,
-	SilenceUsage:       true,
-	FParseErrWhitelist: cobra.FParseErrWhitelist{UnknownFlags: true},
+	Use:               "push [flags] <target>",
+	Short:             "Push a model artifact to the remote registry.",
+	Args:              cobra.ExactArgs(1),
+	DisableAutoGenTag: true,
+	SilenceUsage:      true,
 	RunE: func(cmd *cobra.Command, args []string) error {
 		if err := pushConfig.Validate(); err != nil {
 			return err

cmd/rm.go

@@ -28,12 +28,11 @@ import (
 
 // rmCmd represents the modctl command for rm.
 var rmCmd = &cobra.Command{
-	Use:                "rm [flags] <target>",
-	Short:              "Remove a model artifact from the local storage.",
-	Args:               cobra.ExactArgs(1),
-	DisableAutoGenTag:  true,
-	SilenceUsage:       true,
-	FParseErrWhitelist: cobra.FParseErrWhitelist{UnknownFlags: true},
+	Use:               "rm [flags] <target>",
+	Short:             "Remove a model artifact from the local storage.",
+	Args:              cobra.ExactArgs(1),
+	DisableAutoGenTag: true,
+	SilenceUsage:      true,
 	RunE: func(cmd *cobra.Command, args []string) error {
 		return runRm(context.Background(), args[0])
 	},

cmd/tag.go

@@ -28,12 +28,11 @@ import (
 
 // tagCmd represents the modctl command for tag.
 var tagCmd = &cobra.Command{
-	Use:                "tag [flags] <source> <target>",
-	Short:              "Tag can tag one model artifact to another one without rebuilding.",
-	Args:               cobra.ExactArgs(2),
-	DisableAutoGenTag:  true,
-	SilenceUsage:       true,
-	FParseErrWhitelist: cobra.FParseErrWhitelist{UnknownFlags: true},
+	Use:               "tag [flags] <source> <target>",
+	Short:             "Tag can tag one model artifact to another one without rebuilding.",
+	Args:              cobra.ExactArgs(2),
+	DisableAutoGenTag: true,
+	SilenceUsage:      true,
 	RunE: func(cmd *cobra.Command, args []string) error {
 		return runTag(context.Background(), args[0], args[1])
 	},

cmd/upload.go

@@ -31,12 +31,11 @@ var uploadConfig = config.NewUpload()
 
 // uploadCmd represents the modctl command for upload.
 var uploadCmd = &cobra.Command{
-	Use:                "upload [flags] <file>",
-	Short:              "Upload a file to the remote end in advance to save time in the later build, applicable to the scenario of uploading while downloading, this function needs to be used together with build.",
-	Args:               cobra.ExactArgs(1),
-	DisableAutoGenTag:  true,
-	SilenceUsage:       true,
-	FParseErrWhitelist: cobra.FParseErrWhitelist{UnknownFlags: true},
+	Use:               "upload [flags] <file>",
+	Short:             "Upload a file to the remote end in advance to save time in the later build, applicable to the scenario of uploading while downloading, this function needs to be used together with build.",
+	Args:              cobra.ExactArgs(1),
+	DisableAutoGenTag: true,
+	SilenceUsage:      true,
 	RunE: func(cmd *cobra.Command, args []string) error {
 		if err := uploadConfig.Validate(); err != nil {
 			return err

cmd/version.go

@@ -27,11 +27,10 @@ import (
 
 // versionCmd represents the modctl command for version.
 var versionCmd = &cobra.Command{
-	Use:                "version",
-	Short:              "A command line tool for modctl version",
-	DisableAutoGenTag:  true,
-	SilenceUsage:       true,
-	FParseErrWhitelist: cobra.FParseErrWhitelist{UnknownFlags: true},
+	Use:               "version",
+	Short:             "A command line tool for modctl version",
+	DisableAutoGenTag: true,
+	SilenceUsage:      true,
 	RunE: func(cmd *cobra.Command, args []string) error {
 		return runVersion()
 	},