Move to @pezkuwi/dev 0.86 (eslint 10 toolchain)

- @pezkuwi/dev, dev-test, dev-ts ^0.86.2
- addrcheck.spec: optional call instead of a guard expression
  (@typescript-eslint/prefer-optional-chain)
- checkRepoMeta: drop the initial value every path overwrites
  (no-useless-assignment)
- ipfsUpload: drop a disable directive for a rule the toolchain no
  longer reports

package.json

@@ -41,7 +41,7 @@
     "dnslink-cloudflare": "^3.0.0"
   },
   "devDependencies": {
-    "@pezkuwi/dev": "^0.85.2",
+    "@pezkuwi/dev": "^0.86.2",
     "@types/node": "^22.10.5"
   },
   "resolutions": {

packages/phishing/src/addrcheck.spec.ts

@@ -22,7 +22,7 @@ async function loopSome (site: string, matcher: () => Promise<string[] | null>):
     try {
       const addresses = await matcher();
 
-      addresses && addresses.forEach((address): void => {
+      addresses?.forEach((address): void => {
         if (address && !found.includes(address)) {
           found.push(address);
         }

scripts/checkRepoMeta.mjs

@@ -87,7 +87,7 @@ check('the whole repository agrees on one host', hosts.size === 1, [...hosts]);
 
 // And that host is where the code actually is. Without this the metadata can be
 // internally consistent and still point somewhere the code has never been.
-let origin = '';
+let origin;
 
 try {
   origin = execFileSync('git', ['remote', 'get-url', 'origin'], { encoding: 'utf8' }).trim();

scripts/ipfsUpload.mjs

@@ -101,7 +101,6 @@ async function main () {
   await unpin(hash);
 }
 
-// eslint-disable-next-line promise/catch-or-return
 main()
   .catch(console.error)
   .finally(() => process.exit());