feat: code quality

.github/workflows/lint.yml

@@ -0,0 +1,10 @@
+name: lint
+on:
+  push:
+    branches:
+      - main
+  pull_request:
+
+jobs:
+  lint:
+    uses: lnbits/lnbits/.github/workflows/lint.yml@dev

.github/workflows/release.yml

@@ -1,10 +1,9 @@
 on:
   push:
     tags:
-      - "v[0-9]+.[0-9]+.[0-9]+"
+      - 'v[0-9]+.[0-9]+.[0-9]+'
 
 jobs:
-
   release:
     runs-on: ubuntu-latest
     steps:
@@ -34,12 +33,12 @@ jobs:
       - name: Create pull request in extensions repo
         env:
           GH_TOKEN: ${{ secrets.EXT_GITHUB }}
-          repo_name: "${{ github.event.repository.name }}"
-          tag: "${{ github.ref_name }}"
-          branch: "update-${{ github.event.repository.name }}-${{ github.ref_name }}"
-          title: "[UPDATE] ${{ github.event.repository.name }} to ${{ github.ref_name }}"
-          body: "https://github.com/lnbits/${{ github.event.repository.name }}/releases/${{ github.ref_name }}"
-          archive: "https://github.com/lnbits/${{ github.event.repository.name }}/archive/refs/tags/${{ github.ref_name }}.zip"
+          repo_name: '${{ github.event.repository.name }}'
+          tag: '${{ github.ref_name }}'
+          branch: 'update-${{ github.event.repository.name }}-${{ github.ref_name }}'
+          title: '[UPDATE] ${{ github.event.repository.name }} to ${{ github.ref_name }}'
+          body: 'https://github.com/lnbits/${{ github.event.repository.name }}/releases/${{ github.ref_name }}'
+          archive: 'https://github.com/lnbits/${{ github.event.repository.name }}/archive/refs/tags/${{ github.ref_name }}.zip'
         run: |
           cd lnbits-extensions
           git checkout -b $branch

.gitignore

@@ -1 +1,4 @@
 __pycache__
+node_modules
+.mypy_cache
+.venv

.prettierrc

@@ -0,0 +1,12 @@
+{
+  "semi": false,
+  "arrowParens": "avoid",
+  "insertPragma": false,
+  "printWidth": 80,
+  "proseWrap": "preserve",
+  "singleQuote": true,
+  "trailingComma": "none",
+  "useTabs": false,
+  "bracketSameLine": false,
+  "bracketSpacing": false
+}

Makefile

@@ -0,0 +1,47 @@
+all: format check
+
+format: prettier black ruff
+
+check: mypy pyright checkblack checkruff checkprettier
+
+prettier:
+	poetry run ./node_modules/.bin/prettier --write .
+pyright:
+	poetry run ./node_modules/.bin/pyright
+
+mypy:
+	poetry run mypy .
+
+black:
+	poetry run black .
+
+ruff:
+	poetry run ruff check . --fix
+
+checkruff:
+	poetry run ruff check .
+
+checkprettier:
+	poetry run ./node_modules/.bin/prettier --check .
+
+checkblack:
+	poetry run black --check .
+
+checkeditorconfig:
+	editorconfig-checker
+
+test:
+	PYTHONUNBUFFERED=1 \
+	DEBUG=true \
+	poetry run pytest
+install-pre-commit-hook:
+	@echo "Installing pre-commit hook to git"
+	@echo "Uninstall the hook with poetry run pre-commit uninstall"
+	poetry run pre-commit install
+
+pre-commit:
+	poetry run pre-commit run --all-files
+
+
+checkbundle:
+	@echo "skipping checkbundle"

README.md

@@ -1,63 +1,60 @@
 # Bolt Cards - <small>[LNbits](https://github.com/lnbits/lnbits) extension</small>
-<small>For more about LNBits extensions check [this tutorial](https://youtu.be/_sW7miqaXJc)</small>
 
+<small>For more about LNBits extensions check [this tutorial](https://youtu.be/_sW7miqaXJc)</small>
 
-This extension allows you to link your [Bolt Card](https://github.com/boltcard) on a NXP NTAG424 DNA tag with a LNbits hub that generated new links on each tab which allows a better privacy and security than a static LNURLw that you can also write to a NFC tag (fromon NTAG 213) in the withdraw-extension e.g. for one-time usage as a gift-card. 
+This extension allows you to link your [Bolt Card](https://github.com/boltcard) on a NXP NTAG424 DNA tag with a LNbits hub that generated new links on each tab which allows a better privacy and security than a static LNURLw that you can also write to a NFC tag (fromon NTAG 213) in the withdraw-extension e.g. for one-time usage as a gift-card.
 
 <a class="text-secondary" href="https://youtu.be/_sW7miqaXJc">Video Tutorial</a>
 
-
-**Disclaimer:** ***Use this only if you either know what you are doing or are a reckless lightning pioneer. 
-Only you are responsible for all your sats, cards and other devices. Always backup all your card keys!***
-
+**Disclaimer:** **_Use this only if you either know what you are doing or are a reckless lightning pioneer.
+Only you are responsible for all your sats, cards and other devices. Always backup all your card keys!_**
 
 For the easy way you need:
 
-* an LNbits instance in clearnet
-* opened on Android in Chrome browser 
-* Boltcard extension installed for your LNbits wallet
-* [Boltcard NFC Card Creator App](https://github.com/boltcard/bolt-nfc-android-app) from the [Apple-](https://apps.apple.com/us/app/boltcard-nfc-programmer/id6450968873) or [Play-Store](https://play.google.com/store/search?q=bolt+card+nfc+card+creator&c=apps) to write your keys to the tags once they were generated on LNbits
+- an LNbits instance in clearnet
+- opened on Android in Chrome browser
+- Boltcard extension installed for your LNbits wallet
+- [Boltcard NFC Card Creator App](https://github.com/boltcard/bolt-nfc-android-app) from the [Apple-](https://apps.apple.com/us/app/boltcard-nfc-programmer/id6450968873) or [Play-Store](https://play.google.com/store/search?q=bolt+card+nfc+card+creator&c=apps) to write your keys to the tags once they were generated on LNbits
 
 If you want to gift a Boltcard, make sure to [include the following data](https://www.figma.com/proto/OH6aGCxH45vNpKsZ2nD96S/Untitled?node-id=6%3A37&scaling=min-zoom&page-id=0%3A1) in your present, so that the user is able to make full use of it.
 
-***Always backup all keys that you're trying to write on the card. Without them you may not be able to change them in the future!***
-
+**_Always backup all keys that you're trying to write on the card. Without them you may not be able to change them in the future!_**
 
 ## Setting the card - Boltcard NFC Card Creator (easy way)
 
 - Add new card in the extension.
-    - Set a max sats per transaction. Any transaction greater than this amount will be rejected. This is usually set higher than the funds in the wallet are to prevent accidential withdraws.
-    - Set a max sats per day. After the card spends this amount of sats in a day, additional transactions will be rejected.
-    - Set a card name. This is just for your reference inside LNbits.
-    - Set the card UID. This is the unique identifier of your NFC card and is 7 bytes.
-        - If on an Android device with a newish version of Chrome, you can click the icon next to the input and tap your card to autofill this field.
-        - Otherwise read it with the Bolt-Card app (Read NFC) and paste it to the field.
-    - Advanced Options
-        - Card Keys (k0, k1, k2) will be automatically generated if not explicitly set.
-            - Set to 16 bytes of 0s (00000000000000000000000000000000) to leave the keys in default (empty) state (this is unsecure).
-            - GENERATE KEY button fill the keys randomly.
-    - Click CREATE CARD button
+  - Set a max sats per transaction. Any transaction greater than this amount will be rejected. This is usually set higher than the funds in the wallet are to prevent accidential withdraws.
+  - Set a max sats per day. After the card spends this amount of sats in a day, additional transactions will be rejected.
+  - Set a card name. This is just for your reference inside LNbits.
+  - Set the card UID. This is the unique identifier of your NFC card and is 7 bytes.
+    - If on an Android device with a newish version of Chrome, you can click the icon next to the input and tap your card to autofill this field.
+    - Otherwise read it with the Bolt-Card app (Read NFC) and paste it to the field.
+  - Advanced Options
+    - Card Keys (k0, k1, k2) will be automatically generated if not explicitly set.
+      - Set to 16 bytes of 0s (00000000000000000000000000000000) to leave the keys in default (empty) state (this is unsecure).
+      - GENERATE KEY button fill the keys randomly.
+  - Click CREATE CARD button
 - Click the QR code button next to a card to view its details. Backup the keys now! They'll be comfortable in your password manager.
-    - Now you can scan the QR code with the Boltcard app (Create Bolt Card -> SCAN QR CODE).
-    - Or the "KEYS / AUTH LINK" button to copy the auth URL to the clipboard. Then paste it into the Android app (Create Bolt Card -> PASTE AUTH URL).
+  - Now you can scan the QR code with the Boltcard app (Create Bolt Card -> SCAN QR CODE).
+  - Or the "KEYS / AUTH LINK" button to copy the auth URL to the clipboard. Then paste it into the Android app (Create Bolt Card -> PASTE AUTH URL).
 - Click WRITE CARD NOW and approach the NFC card to set it up. DO NOT REMOVE THE CARD PREMATURELY!
 
 ## Erasing the card - Boltcard NFC Card Creator
+
 Updated for v0.1.9
 
 Since v0.1.2 of Boltcard NFC Card Creator it is possible not only to reset the keys but also to disable the SUN function and do the complete erase so the card can be used again as a static tag (or set as a new Bolt Card, ofc).
 
 - In the Boltcard extension click the QR code button next to a card to view its details and select WIPE
 - OR click the red cross icon on the right side to reach the same
 - In the Boltcard app (Reset Keys)
-    - Click SCAN QR CODE to scan the QR
-    - Or click WIPE DATA in LNbits to copy and paste in to the app (PASTE KEY JSON)
+  - Click SCAN QR CODE to scan the QR
+  - Or click WIPE DATA in LNbits to copy and paste in to the app (PASTE KEY JSON)
 - Click RESET CARD NOW and approach the NFC card to erase it. DO NOT REMOVE THE CARD PREMATURELY!
 - Now if all is successful the card can be safely deleted from LNbits (but keep the keys backuped anyway; batter safe than brick).
 
 If you somehow find yourself in some non-standard state (for instance only k3 and k4 remains filled after previous unsuccessful reset), then you need to edit the key fields manually (for instance leave k0-k2 to zeroes and provide the right k3 and k4).
 
-
 ## Setting the card (advanced)
 
 A technology called [Secure Unique NFC](https://web.archive.org/web/20220706134959/https://mishka-scan.com/blog/secure-unique-nfc) is utilized in this workflow.
@@ -74,8 +71,8 @@ The key #00, K0 (also know as auth key) is used as authentification key. It is n
 
 ### The writing process
 
-There's also a more [advanced guide](https://www.whitewolftech.com/articles/payment-card/) to set cards up manually with a card reader connected to your computer. 
-Writing can also be done (without setting the keys) via the [TagWriter app by NXP](https://play.google.com/store/apps/details?id=com.nxp.nfc.tagwriter) on Android. 
+There's also a more [advanced guide](https://www.whitewolftech.com/articles/payment-card/) to set cards up manually with a card reader connected to your computer.
+Writing can also be done (without setting the keys) via the [TagWriter app by NXP](https://play.google.com/store/apps/details?id=com.nxp.nfc.tagwriter) on Android.
 
 The URI should be `lnurlw://YOUR_LNBITS_DOMAIN/boltcards/api/v1/scan/{YOUR_card_external_id}?p=00000000000000000000000000000000&c=0000000000000000`
 

__init__.py

@@ -1,10 +1,10 @@
 import asyncio
-from loguru import logger
 
 from fastapi import APIRouter
 from lnbits.db import Database
 from lnbits.helpers import template_renderer
 from lnbits.tasks import create_permanent_unique_task
+from loguru import logger
 
 db = Database("ext_boltcards")
 
@@ -22,8 +22,8 @@ def boltcards_renderer():
     return template_renderer(["boltcards/templates"])
 
 
-from .lnurl import *  # noqa: F401,F403
-from .tasks import *  # noqa: F401,F403
+from .lnurl import *  # noqa: F403
+from .tasks import *  # noqa: F403
 
 scheduled_tasks: list[asyncio.Task] = []
 
@@ -41,5 +41,5 @@ def boltcards_start():
     scheduled_tasks.append(task)
 
 
-from .views import *  # noqa: F401,F403
-from .views_api import *  # noqa: F401,F403
+from .views import *  # noqa: F403
+from .views_api import *  # noqa: F403

description.md

@@ -2,7 +2,7 @@ This extension enables you to link your Bolt Card to an NXP NTAG424 DNA tag via
 
 For a simpler setup, you will need:
 
-* An LNbits instance accessible over clearnet.
-* Google Chrome browser opened on Android.
-* Boltcard extension installed on your LNbits wallet.
-* Boltcard NFC Card Creator App, available in the Apple or Play Store, to write your keys to the tags after they have been generated on LNbits.
+- An LNbits instance accessible over clearnet.
+- Google Chrome browser opened on Android.
+- Boltcard extension installed on your LNbits wallet.
+- Boltcard NFC Card Creator App, available in the Apple or Play Store, to write your keys to the tags after they have been generated on LNbits.

lnurl.py

@@ -4,13 +4,13 @@
 from urllib.parse import urlparse
 
 from fastapi import HTTPException, Query, Request
-from lnurl import encode as lnurl_encode
-from lnurl.types import LnurlPayMetadata
-from starlette.responses import HTMLResponse
-
 from lnbits import bolt11
 from lnbits.core.services import create_invoice
 from lnbits.core.views.api import pay_invoice
+from starlette.responses import HTMLResponse
+
+from lnurl import encode as lnurl_encode
+from lnurl.types import LnurlPayMetadata
 
 from . import boltcards_ext
 from .crud import (
@@ -81,7 +81,9 @@ async def api_scan(p, c, request: Request, external_id: str):
     # bech32 encoded lnurl
     lnurlpay_bech32 = lnurl_encode(lnurlpay_raw)
     # create a lud17 lnurlp to support lud19, add to payLink field of the withdrawRequest
-    lnurlpay_nonbech32_lud17 = lnurlpay_raw.replace("https://", "lnurlp://").replace("http://","lnurlp://")
+    lnurlpay_nonbech32_lud17 = lnurlpay_raw.replace("https://", "lnurlp://").replace(
+        "http://", "lnurlp://"
+    )
 
     return {
         "tag": "withdrawRequest",

models.py

@@ -2,10 +2,11 @@
 from sqlite3 import Row
 
 from fastapi import Query, Request
+from pydantic import BaseModel
+
 from lnurl import Lnurl
 from lnurl import encode as lnurl_encode
 from lnurl.types import LnurlPayMetadata
-from pydantic import BaseModel
 
 ZERO_KEY = "00000000000000000000000000000000"
 
@@ -34,7 +35,9 @@ def from_row(cls, row: Row) -> "Card":
         return cls(**dict(row))
 
     def lnurl(self, req: Request) -> Lnurl:
-        url = str(req.url_for("boltcard.lnurl_response", device_id=self.id, _external=True))
+        url = str(
+            req.url_for("boltcard.lnurl_response", device_id=self.id, _external=True)
+        )
         return lnurl_encode(url)
 
     async def lnurlpay_metadata(self) -> LnurlPayMetadata:

package.json

@@ -0,0 +1,15 @@
+{
+  "name": "boltcards",
+  "version": "1.0.0",
+  "description": "",
+  "main": "index.js",
+  "scripts": {
+    "test": "echo \"Error: no test specified\" && exit 1"
+  },
+  "author": "",
+  "license": "ISC",
+  "dependencies": {
+    "prettier": "^3.2.5",
+    "pyright": "^1.1.358"
+  }
+}