Skip to content

Commit 318a1a8

Browse files
committed
fixes
Signed-off-by: Grant Linville <[email protected]>
1 parent c329336 commit 318a1a8

File tree

4 files changed

+18
-6
lines changed

4 files changed

+18
-6
lines changed

pkg/cli/credential.go

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -43,9 +43,11 @@ func (c *Credential) Run(cmd *cobra.Command, _ []string) error {
4343
return fmt.Errorf("failed to read CLI config: %w", err)
4444
}
4545

46-
ctx := c.root.CredentialContext
46+
ctxs := c.root.CredentialContext
4747
if c.AllContexts {
48-
ctx = []string{credentials.AllCredentialContexts}
48+
ctxs = []string{credentials.AllCredentialContexts}
49+
} else if len(ctxs) == 0 {
50+
ctxs = []string{credentials.DefaultCredentialContext}
4951
}
5052

5153
opts, err := c.root.NewGPTScriptOpts()
@@ -63,7 +65,7 @@ func (c *Credential) Run(cmd *cobra.Command, _ []string) error {
6365
}
6466

6567
// Initialize the credential store and get all the credentials.
66-
store, err := credentials.NewStore(cfg, opts.Runner.RuntimeManager, ctx, opts.Cache.CacheDir)
68+
store, err := credentials.NewStore(cfg, opts.Runner.RuntimeManager, ctxs, opts.Cache.CacheDir)
6769
if err != nil {
6870
return fmt.Errorf("failed to get credentials store: %w", err)
6971
}

pkg/cli/credential_delete.go

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,11 +40,16 @@ func (c *Delete) Run(cmd *cobra.Command, args []string) error {
4040
opts.Runner.RuntimeManager = runtimes.Default(opts.Cache.CacheDir)
4141
}
4242

43+
credCtx := c.root.CredentialContext
44+
if len(credCtx) == 0 {
45+
credCtx = []string{credentials.DefaultCredentialContext}
46+
}
47+
4348
if err = opts.Runner.RuntimeManager.SetUpCredentialHelpers(cmd.Context(), cfg); err != nil {
4449
return err
4550
}
4651

47-
store, err := credentials.NewStore(cfg, opts.Runner.RuntimeManager, c.root.CredentialContext, opts.Cache.CacheDir)
52+
store, err := credentials.NewStore(cfg, opts.Runner.RuntimeManager, credCtx, opts.Cache.CacheDir)
4853
if err != nil {
4954
return fmt.Errorf("failed to get credentials store: %w", err)
5055
}

pkg/cli/credential_show.go

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -42,11 +42,16 @@ func (c *Show) Run(cmd *cobra.Command, args []string) error {
4242
opts.Runner.RuntimeManager = runtimes.Default(opts.Cache.CacheDir)
4343
}
4444

45+
credCtx := c.root.CredentialContext
46+
if len(credCtx) == 0 {
47+
credCtx = []string{credentials.DefaultCredentialContext}
48+
}
49+
4550
if err = opts.Runner.RuntimeManager.SetUpCredentialHelpers(cmd.Context(), cfg); err != nil {
4651
return err
4752
}
4853

49-
store, err := credentials.NewStore(cfg, opts.Runner.RuntimeManager, c.root.CredentialContext, opts.Cache.CacheDir)
54+
store, err := credentials.NewStore(cfg, opts.Runner.RuntimeManager, credCtx, opts.Cache.CacheDir)
5055
if err != nil {
5156
return fmt.Errorf("failed to get credentials store: %w", err)
5257
}

pkg/cli/gptscript.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,7 @@ type GPTScript struct {
6464
Chdir string `usage:"Change current working directory" short:"C"`
6565
Daemon bool `usage:"Run tool as a daemon" local:"true" hidden:"true"`
6666
Ports string `usage:"The port range to use for ephemeral daemon ports (ex: 11000-12000)" hidden:"true"`
67-
CredentialContext []string `usage:"Context name(s) in which to store credentials" default:"default"`
67+
CredentialContext []string `usage:"Context name(s) in which to store credentials"`
6868
CredentialOverride []string `usage:"Credentials to override (ex: --credential-override github.com/example/cred-tool:API_TOKEN=1234)"`
6969
ChatState string `usage:"The chat state to continue, or null to start a new chat and return the state" local:"true"`
7070
ForceChat bool `usage:"Force an interactive chat session if even the top level tool is not a chat tool" local:"true"`

0 commit comments

Comments
 (0)