Skip to content

Comments

gl-plugin: Use call_typed for regular invoice creation#660

Merged
cdecker merged 2 commits intomainfrom
2026w06-lsp-invoice-typed-rpc
Feb 23, 2026
Merged

gl-plugin: Use call_typed for regular invoice creation#660
cdecker merged 2 commits intomainfrom
2026w06-lsp-invoice-typed-rpc

Conversation

@cdecker
Copy link
Collaborator

@cdecker cdecker commented Feb 11, 2026

Replace call_raw with call_typed when creating a regular invoice in the LSP invoice handler. This provides better type safety by using the generated cln_rpc::model::requests::InvoiceRequest and cln_rpc::model::responses::InvoiceResponse types.

Benefits:

  • Compile-time type checking for request/response fields
  • No manual hex decoding needed for payment_hash/payment_secret
  • Access to created_index field from the response
  • Cleaner code without custom request/response structs

Depends on #658

Base automatically changed from 2026w06-lsp-invoice-capacity-check to main February 11, 2026 12:35
Before creating an LSPS2 invoice that requires JIT channel negotiation,
check if the node already has sufficient incoming capacity to receive
the payment directly.

When the receivable capacity (with a 5% buffer for fees) exceeds the
invoice amount, create a regular invoice instead of negotiating with
the LSP. This avoids unnecessary LSP fees and channel opening costs
when the node can already receive the payment.

The capacity check:
- Sums receivable_msat across all CHANNELD_NORMAL channels with
  connected peers
- Applies a 5% buffer to account for routing fees
- Falls back to JIT negotiation if capacity is insufficient or
  for 'any amount' invoices
Replace call_raw with call_typed when creating a regular invoice in the
LSP invoice handler. This provides better type safety by using the
generated cln_rpc::model::requests::InvoiceRequest and
cln_rpc::model::responses::InvoiceResponse types.

Benefits:
- Compile-time type checking for request/response fields
- No manual hex decoding needed for payment_hash/payment_secret
- Access to created_index field from the response
- Cleaner code without custom request/response structs
@cdecker cdecker force-pushed the 2026w06-lsp-invoice-typed-rpc branch from e9692a6 to c6d2dff Compare February 23, 2026 12:11
@cdecker cdecker merged commit 7185343 into main Feb 23, 2026
16 checks passed
@cdecker cdecker deleted the 2026w06-lsp-invoice-typed-rpc branch February 23, 2026 12:46
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.

1 participant