proyectos/simplenote-cli
2
0
Fork 0
Code Issues 2 Pull Requests Actions Packages Projects Releases Wiki Activity

[Bug] doc delete confirmation prompt is dead code #1

New Issue
Open
opened 2026-03-28 03:33:09 +00:00 by openclaw · 0 comments
openclaw commented 2026-03-28 03:33:09 +00:00
Owner
Copy Link

Bug Description

In src/commands/doc.js, the doc delete command imports inquirer but never uses it for confirmation:

const { Confirm } = await import("inquirer");
// For simplicity, just confirm

The --yes flag is defined but does nothing - the deletion proceeds immediately without user confirmation.

Impact

Users running simplenote doc delete <id> without --yes expect a confirmation prompt but none appears.

Suggested Fix

Either implement the confirmation prompt or remove the --yes flag and dead import.

## Bug Description In `src/commands/doc.js`, the `doc delete` command imports `inquirer` but never uses it for confirmation: ```js const { Confirm } = await import("inquirer"); // For simplicity, just confirm ``` The `--yes` flag is defined but does nothing - the deletion proceeds immediately without user confirmation. ## Impact Users running `simplenote doc delete <id>` without `--yes` expect a confirmation prompt but none appears. ## Suggested Fix Either implement the confirmation prompt or remove the `--yes` flag and dead import.
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
darroyo openclaw
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: proyectos/simplenote-cli#1
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?