codex comments

Signed-off-by: Arnab Chatterjee <arnabchat2001@gmail.com>

Author: arnab2001

pkg/kit/auth.go

@@ -45,7 +45,14 @@ func Login(ctx context.Context, opts *LoginOptions) error {
 	if err != nil {
 		return err
 	}
-	reg, err := remote.NewRegistry(opts.Registry, &opts.NetworkOptions)
+	networkOpts := opts.NetworkOptions
+	if networkOpts.CredentialsPath == "" {
+		networkOpts.CredentialsPath = credentialsStorePath
+		if !networkOpts.PlainHTTP && !networkOpts.TLSVerify {
+			networkOpts.TLSVerify = true
+		}
+	}
+	reg, err := remote.NewRegistry(opts.Registry, &networkOpts)
 	if err != nil {
 		return fmt.Errorf("could not resolve registry %s: %w", opts.Registry, err)
 	}

pkg/kit/pull.go

@@ -47,12 +47,22 @@ type PullResult struct {
 }
 
 func Pull(ctx context.Context, opts *PullOptions) (*PullResult, error) {
-	storageHome := constants.StoragePath(opts.ConfigHome)
-	localRepo, err := local.NewLocalRepo(storageHome, opts.ModelRef)
+	pullOpts := *opts
+	if pullOpts.NetworkOptions.Concurrency < 1 {
+		pullOpts.NetworkOptions.Concurrency = 5
+	}
+	if pullOpts.NetworkOptions.CredentialsPath == "" {
+		pullOpts.NetworkOptions.CredentialsPath = constants.CredentialsPath(pullOpts.ConfigHome)
+		if !pullOpts.NetworkOptions.PlainHTTP && !pullOpts.NetworkOptions.TLSVerify {
+			pullOpts.NetworkOptions.TLSVerify = true
+		}
+	}
+	storageHome := constants.StoragePath(pullOpts.ConfigHome)
+	localRepo, err := local.NewLocalRepo(storageHome, pullOpts.ModelRef)
 	if err != nil {
 		return nil, err
 	}
-	desc, err := runPullRecursive(ctx, localRepo, opts, []string{})
+	desc, err := runPullRecursive(ctx, localRepo, &pullOpts, []string{})
 	if err != nil {
 		return nil, err
 	}