fix: destroy --base -y fails with 'target is required' (issue #183) #184

shoko · 2026-04-06T06:14:17+02:00

shoko commented

2026-04-06 06:14:17 +02:00

Summary

Fixed kugetsu destroy --base -y which was failing with "Error: target is required" in v0.2.9.

Bug

The cmd_destroy() function had code that set target="" when target="--base", causing the early validation check if [ -z "$target" ] to exit before reaching the actual --base handling.

Fix

Removed the erroneous early assignment (lines 482-484) that was clearing the target variable before the --base case could be handled.

Changes

kugetsu-session.sh: Removed 4 lines that incorrectly cleared target when it was "--base"

## Summary Fixed `kugetsu destroy --base -y` which was failing with "Error: target is required" in v0.2.9. ## Bug The `cmd_destroy()` function had code that set `target=""` when `target="--base"`, causing the early validation check `if [ -z "$target" ]` to exit before reaching the actual `--base` handling. ## Fix Removed the erroneous early assignment (lines 482-484) that was clearing the target variable before the `--base` case could be handled. ## Changes - `kugetsu-session.sh`: Removed 4 lines that incorrectly cleared `target` when it was `"--base"`

shoko added 1 commit 2026-04-06 06:14:18 +02:00

fix: destroy --base -y fails with 'target is required' (issue #183 ) 836fde07fc

Removed erroneous code that set target="" when target="--base".
This caused the early exit at 'if [ -z "$target" ]' to trigger
before reaching the actual --base handling at line 517.

han approved these changes 2026-04-06 06:16:01 +02:00

han left a comment

First-time contributor

lgtm

shoko merged commit 3a2095861f into main

2026-04-06 06:16:30 +02:00

shoko referenced this issue from a commit

2026-04-06 06:16:32 +02:00

Merge pull request 'fix: destroy --base -y fails with 'target is required' (issue #183)' (#184) from fix/issue-183-destroy-base-requires-target into main

Sign in to join this conversation.

2 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: shoko/kugetsu#184