Skip to content

docs: add a paired server and CLI update skill - #295

Closed
vincenzopalazzo wants to merge 2 commits into
lightningdevkit:mainfrom
vincenzopalazzo:feat/ldk-server-update-skill-upstream
Closed

vincenzopalazzo wants to merge 2 commits into
lightningdevkit:mainfrom
vincenzopalazzo:feat/ldk-server-update-skill-upstream

Conversation

@vincenzopalazzo

Copy link
Copy Markdown
Member

Summary

  • Add a goose skill for updating a running ldk-server to a specific commit or tag.
  • Require installing ldk-server and ldk-server-cli from the same build. Since 819bed8 the server HMAC covers the request body, so an older CLI parses but fails with Invalid credentials.
  • Restart only when asked. Do not replace keys_seed, keys_mnemonic, or channel storage as part of a binary update.

The skill does not name a production host.

Written with AI assistance (Goose).

Test plan

  • Skill file is documentation only; no code or CI change.
  • git diff origin/main contains only .goose/skills/ldk-server-update/SKILL.md.

Record the host update path so a server bump always installs the matching
CLI. The July CLI signs only the timestamp and fails against the body HMAC
with Invalid credentials.

Written with AI assistance (Goose).
An old CLI signs only the timestamp. Current servers also sign the request
body, so a server-only upgrade fails closed with Invalid credentials.

Keep the procedure generic: no host addresses, install both binaries from
the same revision, and restart only when asked.

Written with AI assistance (Goose).
@ldk-reviews-bot

Copy link
Copy Markdown

👋 Hi! Please choose at least one reviewer by assigning them on the right bar.
If no reviewers are assigned within 10 minutes, I'll automatically assign one.
Once the first reviewer has submitted a review, a second will be assigned if required.

@vincenzopalazzo

Copy link
Copy Markdown
Member Author

ops wrong repo :P

@vincenzopalazzo
vincenzopalazzo deleted the feat/ldk-server-update-skill-upstream branch September 22, 2026 03:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants