@forgesworn/moneyer 0.3.0 → 0.3.1

package/CHANGELOG.md

@@ -1,5 +1,22 @@
 # Changelog
 
+## [0.3.1] - 2026-08-22
+
+- **Rounding the fee to a whole sat now really is the default.** 0.3.0
+  documented `roundFeeToSat` as on unless turned off, and made it so for a
+  mint started from the CLI, where the environment reader fills the
+  default in. A mint built by calling `createMoneyer` with a config object
+  passes the field through as absent, and the server read absent as off -
+  so an embedder got a mint that kept fractions of a sat while the same
+  mint from the binary rounded them away. Absent now means on in both,
+  and only an explicit `false` opts out.
+
+  It hid because every test that cared about rounding set the flag
+  explicitly, so nothing exercised the default through the library at all.
+  Found from the other side: notecase's cross-implementation suite began
+  asserting what a wallet is actually credited, and the mint it builds in
+  process disagreed with the mint on the wire.
+
 ## [0.3.0] - 2026-08-22
 
 - **The mint fee is now ceilinged to a whole sat by default.**

package/dist/server.js

@@ -92,7 +92,13 @@ export const createMoneyer = async (config, deps = {}) => {
         const exact = grossMsat - applyMintFee(grossMsat, config.mintFee);
         // Both readings sit inside lnurlcash-kit's mintFeeBand, so a wallet
         // is not misled either way - it is told the range up front.
-        return config.roundFeeToSat === true ? Math.ceil(exact / 1000) * 1000 : exact;
+        //
+        // Absent means ON, the same as DEFAULTS and MONEYER_ROUND_FEE_TO_SAT:
+        // only an explicit false opts out. Reading it as `=== true` made the
+        // library default differ from the binary's, so a mint built by calling
+        // createMoneyer kept fractions of a sat that the same mint started from
+        // the CLI would have rounded.
+        return config.roundFeeToSat !== false ? Math.ceil(exact / 1000) * 1000 : exact;
     };
     // Every quote and every credit goes through here, so the fee posture
     // cannot drift between what is advertised and what is withheld.
@@ -116,7 +122,7 @@ export const createMoneyer = async (config, deps = {}) => {
     const mintFeeLine = config.mintFee ? `Mint fees: ${config.mintFee.baseFeeMsat},${config.mintFee.feePpm}` : null;
     // The same fee for a person: "Mint fees: 5000,1000" is for wallets that
     // parse LUD-25, and reads as nonsense to anyone who does not.
-    const feeInWords = config.mintFee ? describeFee(config.mintFee, config.roundFeeToSat === true) : null;
+    const feeInWords = config.mintFee ? describeFee(config.mintFee, config.roundFeeToSat !== false) : null;
     const nostr = config.zap ? (deps.nostr ?? poolTransport()) : null;
     const zap = config.zap && nostr
         ? createZapBridge({

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@forgesworn/moneyer",
-  "version": "0.3.0",
+  "version": "0.3.1",
   "description": "An LNURLcash (LUD-25) mint - strikes Lightning bearer notes. Independent implementation, cln/lnd funding sources, SQLite, zero HTTP framework.",
   "author": "TheCryptoDonkey",
   "license": "MIT",