cdk-cli: added non-interactive mode #1707

Open
0xEgao wants to merge 2 commits from 0xEgao/non-interactive into main
0xEgao commented 2026-03-06 09:33:41 +00:00 (Migrated from github.com)

Description

Fixes #1701

Notes to the reviewers


Suggested CHANGELOG Updates

CHANGED

ADDED

Added Non-interactive mode for the cdk-cli

REMOVED

FIXED


Checklist

### Description <!-- Describe the purpose of this PR, what's being adding and/or fixed --> Fixes #1701 ### Notes to the reviewers <!-- In this section you can include notes directed to the reviewers, like explaining why some parts of the PR were done in a specific way --> ----- ### Suggested [CHANGELOG](https://github.com/cashubtc/cdk/blob/main/CHANGELOG.md) Updates <!-- Please do not edit the actual changelog but note what you changed here. --> #### CHANGED #### ADDED Added Non-interactive mode for the cdk-cli #### REMOVED #### FIXED ---- ### Checklist * [x] I followed the [code style guidelines](https://github.com/cashubtc/cdk/blob/main/CODE_STYLE.md) * [] I ran `just final-check` before committing
thesimplekid commented 2026-03-06 22:46:47 +00:00 (Migrated from github.com)

This is a good start but I think we need to check that each functions has the needed args that it does not require interactive prompts. For example I think on the mpp branch of melt we do not have the needed args to complete this non-interactively.

We also need to work on how the output of the command is handle and how agent friendly that is or could be made, could also be a separate pr.

We could even add some tests for this, but I won't require that as part of this pr.

This is a good start but I think we need to check that each functions has the needed args that it does not require interactive prompts. For example I think on the mpp branch of melt we do not have the needed args to complete this non-interactively. We also need to work on how the output of the command is handle and how agent friendly that is or could be made, could also be a separate pr. We could even add some tests for this, but I won't require that as part of this pr.
This pull request can be merged automatically.
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin 0xEgao/non-interactive:0xEgao/non-interactive
git switch 0xEgao/non-interactive

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch main
git merge --no-ff 0xEgao/non-interactive
git switch 0xEgao/non-interactive
git rebase main
git switch main
git merge --ff-only 0xEgao/non-interactive
git switch 0xEgao/non-interactive
git rebase main
git switch main
git merge --no-ff 0xEgao/non-interactive
git switch main
git merge --squash 0xEgao/non-interactive
git switch main
git merge --ff-only 0xEgao/non-interactive
git switch main
git merge 0xEgao/non-interactive
git push origin main
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
cashubtc/cdk!1707
No description provided.