balanceofsatoshis 11.44.0 → 11.47.0

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # Versions
 
+## 11.47.0
+
+- `telegram`: Add `/graph <node public key or peer alias>` to lookup node info
+
+## 11.46.0
+
+- `telegram`: Add `/stop` command to terminate the running bot
+
+## 11.45.1
+
+- `increase-inbound-liquidity`: Add support for formulas in `--amount`
+
 ## 11.44.0
 
 - `send`: Add support for `--max-fee-rate` to limit fees paid via PPM measure

package/CONTRIBUTING.md

@@ -0,0 +1,72 @@
+# Contribution Guidelines
+
+- Feel free to open issues or pull requests
+- They may not be addressed or merged
+- You can ignore coding styles if you want
+
+## Coding Style
+
+If you want to help with style, here are some rough guidelines on style ideas:
+
+### Formatting
+
+- Spaces not tabs, 2 spaces
+- Arguments to methods are snake_case
+- Regular variables are camelCase
+- Returned attributes are snake_case
+- Minimize function nesting, make new files if nesting is required
+- No extraneous whitespace
+- A single newline should appear at the end of a file
+- Don't bother with () in functions when not needed: `const a = b => c`
+- If there are multiple things together, alphabetize them
+- Don't split up long strings over multiple lines
+- Lines should be terminated by explicit semicolons
+- Logic like ternary operators should not extend beyond a single line
+- Don't let any lines linger when they don't do anything
+- Tightly space objects, like `{attribute: value}` not `{ attribute : value }`
+- Use single '' quotes not double "" quotes, except when `` is required
+- If conditions should avoid spanning multiple lines
+- Avoid double specifying an attribute and value, `{type: type}` vs `{type}`
+
+### Control Flow
+
+- Async functions should support both cbk and Promise style
+- Use async.js methods for asynchronous control flow
+- Try to exit early from functions when possible, and note this exit in comment
+- Prefer cbk over Promise style, aside from in tests or in Promise libs
+- Use `asyncAuto` for asynchronous control flow dependency management
+- Callbacks should generally be named cbk even when redefinining in inner scope
+- Avoid mixing non-async complex logic and async control flows in the same file
+- Minimize async nesting in returned attributes and in method arguments
+- Methods should always document their arguments and their output
+
+### Variables
+
+- Generally use undefined rather than null when defining nil types
+- Prefer variable assignments on new lines rather than on a single line
+- Reduce the usage of . property access, like isArray instead of Array.isArray
+- When there is a newline in an object, always put a comma at the end of a line
+- Short properties always go first in objects: `{short, longer: type}`
+- If a statement relies on a statement above it, it should have a newline above
+- Avoid including scalar values such as strings or numbers in the code itself
+
+### JS Features
+
+- Limit usage of `let` and never use `var`, prefer `const`
+- Limit use of npm dependencies when possible, only use good dependencies
+- Target support of the oldest node.js LTS release still being supported
+- Never use class or prototype
+- Do not import more methods from an import than you actually use
+- Try to avoid passing objects in arguments as much as possible
+- Prefer using function iteration like map and forEach over for and while
+- Use arrow functions and not `function` functions whenever possible
+
+### Errors
+
+- Never ignore an error case, always deal with it as soon as possible
+- In the case of errors, do include the error string in the code
+- Add simple validations to help target simple calling mistakes
+- When throwing or returning error messages, use PascalCase for the message
+- Use HTTP status codes as a guideline: 4** is a local issue, 5** is remote
+- Return async errors as arrays: `[typeNumber, errorMsgString, extraDetails]`
+- Try to be very specific with error messages and try to not repeat one

package/bos

@@ -25,6 +25,7 @@ const {accountingCategories} = commandConstants;
 const balances = importLazy('./balances');
 const chain = importLazy('./chain');
 const commands = importLazy('./commands');
+const display = importLazy('./display');
 const encryption = importLazy('./encryption');
 const lnd = importLazy('./lnd');
 const network = importLazy('./network');
@@ -54,7 +55,6 @@ const months = [...Array(12).keys()].map(n => ++n);
 const {REPEATABLE} = prog;
 const {STRING} = prog;
 const yearMatch = /^\d{4}$/;
-
 prog
   .version(version)
 
@@ -884,10 +884,11 @@ prog
   .command('increase-inbound-liquidity', 'Increase node inbound liquidity')
   .help('Spend down a channel to get inbound. Fee is an estimate, may be more')
   .help('If you want to control chain fee increases, use show-raw-recoveries')
+  .help('Formulas supported for --amount like 5*m or 0.05*BTC for 5 million')
   .option('--address <out_address>', 'Out chain address to send funds out to')
   .option('--api-key <api_key>', 'Pre-paid API key to use')
+  .option('--amount <amount>', 'Amount to increase inbound', STRING, '500*k')
   .option('--avoid <pubkey/chan/tag>', 'Avoid forwarding through', REPEATABLE)
-  .option('--amount <amount>', 'Amount to increase liquidity', INT, 5e5)
   .option('--confs <confs>', 'Confs to consider reorg safe', INT, 1)
   .option('--dryrun', 'Only show cost estimate for increase')
   .option('--fast', 'Request swap server avoid batching delay')
@@ -930,11 +931,11 @@ prog
           spend_address: options.spendAddress || undefined,
           spend_tokens: options.spendAmount || undefined,
           timeout: 1000 * 60 * 60 * 10,
-          tokens: options.amount,
+          tokens: display.parseAmount({amount: options.amount}).tokens,
         },
         responses.returnObject({exit, logger, reject, resolve}));
       } catch (err) {
-        return reject(err);
+        return reject(logger.error({err}));
       }
     });
   })

package/package.json

@@ -28,16 +28,16 @@
     "crypto-js": "4.1.1",
     "csv-parse": "5.0.4",
     "goldengate": "11.0.0",
-    "grammy": "1.6.2",
+    "grammy": "1.7.0",
     "hot-formula-parser": "4.0.0",
     "import-lazy": "4.0.0",
     "ini": "2.0.0",
     "inquirer": "8.2.0",
     "invoices": "2.0.3",
     "ln-accounting": "5.0.5",
-    "ln-service": "53.6.0",
+    "ln-service": "53.7.0",
     "ln-sync": "3.9.0",
-    "ln-telegram": "3.12.0",
+    "ln-telegram": "3.14.0",
     "moment": "2.29.1",
     "paid-services": "3.11.0",
     "probing": "2.0.2",
@@ -52,7 +52,7 @@
   "description": "Lightning balance CLI",
   "devDependencies": {
     "@alexbosworth/tap": "15.0.10",
-    "ln-docker-daemons": "2.2.2",
+    "ln-docker-daemons": "2.2.3",
     "mock-lnd": "1.4.1",
     "secp256k1": "4.0.3"
   },
@@ -77,9 +77,8 @@
   "scripts": {
     "build-docker": "docker build -t alexbosworth/balanceofsatoshis . && docker save alexbosworth/balanceofsatoshis > balanceofsatoshis.tar && gzip balanceofsatoshis.tar",
     "integration-tests": "tap --branches=1 --functions=1 --lines=1 --statements=1 -t 120 test/integration/*.js",
-    "postpack": "PACKAGE_VERSION=$(cat package.json | grep \\\"version\\\" | head -1 | awk -F: '{ print $2 }' | sed 's/[\",]//g' | tr -d '[[:space:]]') && git tag -s v$PACKAGE_VERSION -m v$PACKAGE_VERSION && git push github --tags",
     "postpublish": "docker buildx build --platform linux/amd64,linux/arm64,linux/arm/v7 -t alexbosworth/balanceofsatoshis --push .",
     "test": "tap --branches=1 --functions=1 --lines=1 --statements=1 -t 60 test/arrays/*.js test/balances/*.js test/chain/*.js test/display/*.js test/encryption/*.js test/lnd/*.js test/network/*.js test/nodes/*.js test/peers/*.js test/responses/*.js test/routing/*.js test/services/*.js test/swaps/*.js test/tags/*.js test/wallets/*.js"
   },
-  "version": "11.44.0"
+  "version": "11.47.0"
 }

package/telegram/start_telegram_bot.js

@@ -19,12 +19,14 @@ const {handleConnectCommand} = require('ln-telegram');
 const {handleCostsCommand} = require('ln-telegram');
 const {handleEarningsCommand} = require('ln-telegram');
 const {handleEditedMessage} = require('ln-telegram');
+const {handleGraphCommand} = require('ln-telegram');
 const {handleInvoiceCommand} = require('ln-telegram');
 const {handleLiquidityCommand} = require('ln-telegram');
 const {handleMempoolCommand} = require('ln-telegram');
 const {handlePayCommand} = require('ln-telegram');
 const {handlePendingCommand} = require('ln-telegram');
 const {handleStartCommand} = require('ln-telegram');
+const {handleStopCommand} = require('ln-telegram');
 const {handleVersionCommand} = require('ln-telegram');
 const {InputFile} = require('grammy');
 const inquirer = require('inquirer');
@@ -266,12 +268,14 @@ module.exports = (args, cbk) => {
           {command: 'connect', description: 'Get connect code for the bot'},
           {command: 'costs', description: 'Show costs over the week'},
           {command: 'earnings', description: 'Show earnings over the week'},
+          {command: 'graph', description: 'Show info about a node'},
           {command: 'help', description: 'Show the list of commands'},
           {command: 'invoice', description: 'Create an invoice'},
           {command: 'liquidity', description: 'Get liquidity [with-peer]'},
           {command: 'mempool', description: 'Get info about the mempool'},
           {command: 'pay', description: 'Pay a payment request'},
           {command: 'pending', description: 'Get pending forwards, channels'},
+          {command: 'stop', description: 'Stop the bot'},
           {command: 'version', description: 'View current bot version'},
         ]);
 
@@ -355,6 +359,22 @@ module.exports = (args, cbk) => {
           return;
         });
 
+        bot.command('graph', async ctx => {
+          try {
+            await handleGraphCommand({
+              from: ctx.message.from.id,
+              id: connectedId,
+              nodes: allNodes,
+              remove: () => ctx.deleteMessage(),
+              reply: (message, options) => ctx.reply(message, options),
+              text: ctx.message.text,
+              working: () => ctx.replyWithChatAction('typing'),
+            });
+          } catch (err) {
+            logger.error({err});
+          }
+        });
+
         // Handle creation of an invoice
         bot.command('invoice', async ctx => {
           try {
@@ -457,6 +477,22 @@ module.exports = (args, cbk) => {
           });
         });
 
+        // Terminate the running bot
+        bot.command('stop', async ctx => {
+          try {
+            await handleStopCommand({
+              from: ctx.message.from.id,
+              id: connectedId,
+              quit: () => bot.stop(),
+              reply: (msg, mode) => ctx.reply(msg, mode),
+            });
+
+            process.exit();
+          } catch (err) {
+            logger.error({err});
+          }
+        });
+
         bot.command('version', async ctx => {
           try {
             await handleVersionCommand({
@@ -476,11 +512,13 @@ module.exports = (args, cbk) => {
           '/connect - Connect bot',
           '/costs - View costs over the past week',
           '/earnings - View earnings over the past week',
+          '/graph [pubkey or peer alias] - Show info about a node',
           '/invoice [amount] [memo] - Make an invoice',
           '/liquidity [with] - View node liquidity',
           '/mempool - BTC mempool report',
           '/pay - Pay an invoice',
           '/pending - View pending channels, probes, and forwards',
+          '/stop - Stop bot',
           '/version - View the current bot version',
         ];
 
@@ -599,15 +637,14 @@ module.exports = (args, cbk) => {
             return postClosedMessage({
               from,
               lnd,
-              request,
               capacity: update.capacity,
               id: connectedId,
               is_breach_close: update.is_breach_close,
               is_cooperative_close: update.is_cooperative_close,
               is_local_force_close: update.is_local_force_close,
               is_remote_force_close: update.is_remote_force_close,
-              key: apiKey.key,
               partner_public_key: update.partner_public_key,
+              send: (id, message) => bot.api.sendMessage(id, message, markdown),
             },
             err => !!err ? logger.error({node: from, closed_err: err}) : null);
           });
@@ -616,13 +653,12 @@ module.exports = (args, cbk) => {
             return postOpenMessage({
               from,
               lnd,
-              request,
               capacity: update.capacity,
               id: connectedId,
               is_partner_initiated: update.is_partner_initiated,
               is_private: update.is_private,
-              key: apiKey.key,
               partner_public_key: update.partner_public_key,
+              send: (id, message) => bot.api.sendMessage(id, message, markdown),
             },
             err => !!err ? logger.error({open_err: err}) : null);
           });
@@ -899,10 +935,9 @@ module.exports = (args, cbk) => {
 
               return await postChainTransaction({
                 from,
-                request,
                 confirmed: transaction.is_confirmed,
                 id: connectedId,
-                key: apiKey.key,
+                send: (id, message) => bot.api.sendMessage(id, message, markdown),
                 transaction: record,
               });
             } catch (err) {