Merge pull request #226 from ibuildthecloud/disable-cache

change: switch --cache to --disable-cache

Author: ibuildthecloud

pkg/cache/cache.go

@@ -18,17 +18,14 @@ type Client struct {
 }
 
 type Options struct {
-	Cache    *bool  `usage:"Disable caching" default:"true"`
-	CacheDir string `usage:"Directory to store cache (default: $XDG_CACHE_HOME/gptscript)"`
+	DisableCache bool   `usage:"Disable caching of LLM API responses"`
+	CacheDir     string `usage:"Directory to store cache (default: $XDG_CACHE_HOME/gptscript)"`
 }
 
 func Complete(opts ...Options) (result Options) {
 	for _, opt := range opts {
 		result.CacheDir = types.FirstSet(opt.CacheDir, result.CacheDir)
-		result.Cache = types.FirstSet(opt.Cache, result.Cache)
-	}
-	if result.Cache == nil {
-		result.Cache = &[]bool{true}[0]
+		result.DisableCache = types.FirstSet(opt.DisableCache, result.DisableCache)
 	}
 	if result.CacheDir == "" {
 		result.CacheDir = filepath.Join(xdg.CacheHome, version.ProgramName)
@@ -54,7 +51,7 @@ func New(opts ...Options) (*Client, error) {
 	}
 	return &Client{
 		dir:  opt.CacheDir,
-		noop: !*opt.Cache,
+		noop: opt.DisableCache,
 	}, nil
 }
 

pkg/openai/client.go

@@ -68,7 +68,7 @@ func complete(opts ...Options) (result Options, err error) {
 
 	if result.Cache == nil {
 		result.Cache, err = cache.New(cache.Options{
-			Cache: new(bool),
+			DisableCache: true,
 		})
 	}