shoko/kugetsu
1
0
Fork 0
Code Issues 17 Pull Requests Actions Packages Projects Releases 3 Wiki Activity

kugetsu destroy --base -y fails with "target is required" in v0.2.9 #183

New Issue
Closed
opened 2026-04-06 06:12:41 +02:00 by shoko · 0 comments
shoko commented 2026-04-06 06:12:41 +02:00
Owner
Copy Link

Bug

In v0.2.9, kugetsu destroy --base -y fails with Error: target is required.

Root cause: cmd_destroy() in kugetsu-session.sh line 482 sets target="" when = "--base". Then line 490 checks if [ -z "" ] which is now true and exits before reaching line 517 where --base is actually handled.

# Line 482-496
if [ "$target" = "--base" ]; then
    target=""      # ← sets target to empty string
fi

if [ -z "$target" ]; then
    echo "Error: target is required" >&2  # ← exits here before --base handling
    ...
    exit 1
fi

elif [ "$target" = "--base" ]; then  # ← never reached

Affected versions: v0.2.9

Reproduction

kugetsu destroy --base -y
# Error: target is required

Fix

Remove or restructure the early target="" assignment on line 482-484. The --base handling at line 517 should be reached directly.

## Bug In v0.2.9, `kugetsu destroy --base -y` fails with `Error: target is required`. **Root cause:** `cmd_destroy()` in `kugetsu-session.sh` line 482 sets `target=""` when ` = "--base"`. Then line 490 checks `if [ -z "" ]` which is now true and exits before reaching line 517 where `--base` is actually handled. ```bash # Line 482-496 if [ "$target" = "--base" ]; then target="" # ← sets target to empty string fi if [ -z "$target" ]; then echo "Error: target is required" >&2 # ← exits here before --base handling ... exit 1 fi elif [ "$target" = "--base" ]; then # ← never reached ``` **Affected versions:** v0.2.9 ## Reproduction ```bash kugetsu destroy --base -y # Error: target is required ``` ## Fix Remove or restructure the early `target=""` assignment on line 482-484. The `--base` handling at line 517 should be reached directly.
shoko added the bug label 2026-04-06 06:12:41 +02:00
shoko closed this issue 2026-04-06 07:08:43 +02:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
0.2.x-only
Available only in 0.2.x, not backportable
 0.3.x-only
Available only in 0.3.x, not backportable
 backport
Backport compatible - can merge to 0.1.x
 blocked
Issue needs clarification/discussion before it can be worked on
 bug
Bug report
 critical
Critical - blocking parallel work
 feature
New feature
 foundation
Foundation work for 0.2.x
 high
High priority - workflow improvements
 meta
Meta issue tracking multiple issues
 need-review
PR needs review before merge
 parallelization
Issues related to parallelization
 ready
Issue is clear and ready to be implemented/delegated
 ready-to-merge
PR approved and ready to merge
 task-delegation
Issues related to task-delegation
No Label bug
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
shoko
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: shoko/kugetsu#183
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?