Skip to content

Commit 477c6df

Browse files
tnsardesaiclaude
andauthored
fix: correct log message when releasing browser with reuse=false (#112)
## Summary - When releasing a browser with `reuse=false`, the CLI now logs "Deleted browser X from pool Y" instead of the misleading "Released browser X back to pool Y" - The original "Released" message is preserved when `reuse` is not set or is `true` ## Test plan - [x] Run `kernel browser-pools release <pool> --session-id <id> --reuse=false` and verify it says "Deleted browser ... from pool ..." - [x] Run `kernel browser-pools release <pool> --session-id <id>` (no reuse flag) and verify it says "Released browser ... back to pool ..." 🤖 Generated with [Claude Code](https://claude.com/claude-code) <!-- CURSOR_SUMMARY --> --- > [!NOTE] > **Low Risk** > Only changes CLI logging text based on an existing flag; no behavior, API calls, or data handling are modified. > > **Overview** > Adjusts the `browser-pools release` CLI success output to reflect actual behavior when `--reuse=false`. > > When `reuse` is explicitly set to `false`, the command now prints *"Deleted browser … from pool …"*; otherwise it keeps the existing *"Released browser … back to pool …"* message. > > <sup>Written by [Cursor Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit 43ab9a8. This will update automatically on new commits. Configure [here](https://cursor.com/dashboard?tab=bugbot).</sup> <!-- /CURSOR_SUMMARY --> Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
1 parent c730c0c commit 477c6df

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed

cmd/browser_pools.go

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -373,7 +373,11 @@ func (c BrowserPoolsCmd) Release(ctx context.Context, in BrowserPoolsReleaseInpu
373373
if err != nil {
374374
return util.CleanedUpSdkError{Err: err}
375375
}
376-
pterm.Success.Printf("Released browser %s back to pool %s\n", in.SessionID, in.IDOrName)
376+
if in.Reuse.Set && !in.Reuse.Value {
377+
pterm.Success.Printf("Deleted browser %s from pool %s\n", in.SessionID, in.IDOrName)
378+
} else {
379+
pterm.Success.Printf("Released browser %s back to pool %s\n", in.SessionID, in.IDOrName)
380+
}
377381
return nil
378382
}
379383

0 commit comments

Comments
 (0)