local-kanban 1.6.3 → 1.7.1

package/CHANGELOG.md

@@ -3,6 +3,33 @@
 Notable changes, newest first. The format follows [Keep a Changelog](https://keepachangelog.com/en/1.1.0/)
 and the project uses [semantic versioning](https://semver.org/).
 
+## [1.7.1]
+
+### Fixed
+
+- **The update check asked the registry in a format it answers with an error.** The request carried
+  npm's abbreviated media type, which is meant for a package document and not for the `/latest`
+  address: the registry replies 406 with an empty body, so an npm install was told "cannot check"
+  again. A cached response let it through often enough to look like a flaky network — it was not.
+  Caught on a clean install of 1.7.0 minutes after it was published.
+
+## [1.7.0]
+
+### Added
+
+- **An npm install can finally see that a new version is out.** The update check asked git for the
+  current commit, and a package installed from npm has no git repository — so it answered "cannot
+  check" every single time, and the About section showed the date of a frontend file instead of a
+  version number. The board now reports its own version and compares it against the registry:
+  *update available 1.6.3 → 1.7.0*, next to the command that installs it. A clone keeps the git
+  check, which says more: the commit, the tag, and how many commits are waiting for a release.
+
+### Fixed
+
+- **A failed update check no longer sticks around for an hour.** The answer was cached for an hour
+  whether it worked or not, so a single lost request kept the board saying "cannot check" long after
+  the network came back. A failure is now remembered for a minute; a real answer still for an hour.
+
 ## [1.6.3]
 
 ### Changed

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "local-kanban",
-  "version": "1.6.3",
+  "version": "1.7.1",
   "license": "MIT",
   "author": "Dannylamao",
   "repository": {

package/public/js/core.js

@@ -86,6 +86,7 @@ const I18N_RU = {
   'Appearance': 'Оформление',
   'applies after page reload': 'применится после перезагрузки страницы',
   'Archive': 'Архивировать',
+  'ask the npm registry again': 'спросить реестр npm заново',
   'at': 'от',
   'Attach a screenshot': 'Приложить скриншот',
   'attachment': 'вложение',
@@ -116,6 +117,7 @@ const I18N_RU = {
   'Cancel the task? The issue will be closed as not planned.': 'Отменить задачу? Issue закроется как not planned.',
   'Cancelled': 'Отменено',
   'cannot check (no network, or the repo is private/not a git checkout)': 'не проверить (нет сети, либо репо приватный/без git)',
+  'cannot check — the npm registry did not answer': 'не удалось проверить — реестр npm не ответил',
   'Card actions': 'Управление карточкой',
   'change status': 'сменить статус',
   'changed — not written to the file yet': 'изменено — в файл ещё не записано',
@@ -330,6 +332,7 @@ const I18N_RU = {
   'm ago': 'м назад',
   'mark as done': 'отметить выполненным',
   'matches GitHub': 'совпадает с GitHub',
+  'matches the registry': 'совпадает с реестром',
   'matches main ✓': 'совпадает с main ✓',
   'max': 'макс',
   'MB': 'МБ',
@@ -592,6 +595,7 @@ const I18N_RU = {
   'Upload': 'Загрузить',
   'used by': 'используют',
   'Using the board at work': 'Пользуешься доской в работе',
+  'version': 'версия',
   'Waiting for you': 'Ждёт тебя',
   'Wed': 'Ср',
   'Week': 'Неделя',

package/public/js/settings.js

@@ -411,7 +411,7 @@ async function renderSection(sec) {
       + `<div class="set-row"><span class="set-lab">${tr('Code version')}<small id="about-ver">…</small></span><span class="about-fresh" id="about-fresh"></span></div>`
       + `<div class="set-row"><span class="set-lab"><span id="about-branch-lab">${tr('Board updates')}</span><small id="about-upd-sub">…</small></span><span class="about-fresh" id="about-upd"></span></div>`
       + `<div class="set-row hidden" id="about-dev-row"><span class="set-lab">${tr('dev branch')}<small id="about-dev-sub">…</small></span><span class="about-fresh" id="about-dev"></span></div>`
-      + `<div class="set-row"><span class="set-lab">${tr('Check for updates')}<small>${tr('re-read branch state from GitHub')}</small></span><button class="btn-ghost" id="about-recheck">${tr('Check')}</button></div>`
+      + `<div class="set-row"><span class="set-lab">${tr('Check for updates')}<small id="about-recheck-sub">${tr('re-read branch state from GitHub')}</small></span><button class="btn-ghost" id="about-recheck">${tr('Check')}</button></div>`
       + `<div class="set-row"><span class="set-lab">${tr('Reload the page')}<small>${tr('re-read the board code right now')}</small></span><button class="btn-ghost" id="about-reload">${tr('Reload now')}</button></div>`
       + `<div class="set-row"><span class="set-lab">${tr('Using the board at work')}<small>${tr('free for everyone, nothing switched off — if it earns you money, $12 a year per person keeps it maintained')}</small></span>`
       + `<a class="btn-ghost" href="https://github.com/lamaohub/local-kanban/blob/main/docs/COMMERCIAL.md" target="_blank" rel="noopener noreferrer">${tr('Read')}</a></div>`
@@ -432,12 +432,21 @@ async function renderSection(sec) {
       const el = $('about-upd'); const sub = $('about-upd-sub'); const lab = $('about-branch-lab');
       if (!el || !sub) return;
       if (lab && u.branch) lab.textContent = `${tr('Branch')} ${u.branch}`;
+      const rs = $('about-recheck-sub');
+      if (rs && about?.packaged) rs.textContent = tr('ask the npm registry again');
+      const what = about?.packaged ? tr('version') : tr('commit');
+      const same = about?.packaged ? tr('matches the registry') : tr('matches GitHub');
       const tag = u.tag ? ` · ${tr('tag')} ${u.tag}` : '';
       el.className = 'about-fresh';
-      if (u.update_available === null) { el.textContent = ''; sub.textContent = tr('cannot check (no network, or the repo is private/not a git checkout)'); }
-      else if (u.update_available) { el.textContent = `⟳ ${tr('update available')} ${u.local} → ${u.remote}`; el.classList.add('stale-ver'); sub.textContent = `${tr('commit')} ${u.local}${tag} · ${updateHint()}`; }
+      if (u.update_available === null) {
+        el.textContent = '';
+        sub.textContent = about?.packaged
+          ? tr('cannot check — the npm registry did not answer')
+          : tr('cannot check (no network, or the repo is private/not a git checkout)');
+      }
+      else if (u.update_available) { el.textContent = `⟳ ${tr('update available')} ${u.local} → ${u.remote}`; el.classList.add('stale-ver'); sub.textContent = `${what} ${u.local}${tag} · ${updateHint()}`; }
       else if (u.dev?.ahead) { el.textContent = `${tr('in dev')} ${u.dev.ahead} ${commitsWord(u.dev.ahead)} ${tr('awaiting release')}`; el.classList.add('stale-ver'); sub.textContent = `${tr('commit')} ${u.local}${tag} · ${tr('matches GitHub')}`; }
-      else { el.textContent = tr('up to date ✓'); sub.textContent = `${tr('commit')} ${u.local}${tag} · ${tr('matches GitHub')}`; }
+      else { el.textContent = tr('up to date ✓'); sub.textContent = `${what} ${u.local}${tag} · ${same}`; }
       const row = $('about-dev-row'); const dEl = $('about-dev'); const dSub = $('about-dev-sub');
       if (!row || !u.dev) return;
       row.classList.remove('hidden');
@@ -461,7 +470,9 @@ async function renderSection(sec) {
     try {
       const a = await api('GET', '/api/about');
       about = a;
-      $('about-ver').textContent = a.commit ? `${tr('commit')} ${a.commit}` : (a.app_mtime ? `app.js ${tr('from')} ${fmtDbTime(a.app_mtime.slice(0, 19).replace('T', ' '))}` : '—');
+      $('about-ver').textContent = (a.packaged && a.version) ? `v${a.version}`
+        : a.commit ? `${tr('commit')} ${a.commit}`
+        : (a.app_mtime ? `app.js ${tr('from')} ${fmtDbTime(a.app_mtime.slice(0, 19).replace('T', ' '))}` : '—');
       const up = a.uptime;
       const upStr = up >= 86400 ? `${Math.floor(up / 86400)}${tr('d')} ${Math.floor((up % 86400) / 3600)}${tr('h')}` : up >= 3600 ? `${Math.floor(up / 3600)}${tr('h')} ${Math.floor((up % 3600) / 60)}${tr('m')}` : `${Math.floor(up / 60)}${tr('m')}`;
       $('about-diag').innerHTML = [

package/src/routes/system.js

@@ -1,5 +1,5 @@
 import { execFile } from 'node:child_process';
-import { statSync, readdirSync, createReadStream, createWriteStream, existsSync, rmSync } from 'node:fs';
+import { statSync, readdirSync, createReadStream, createWriteStream, existsSync, readFileSync, rmSync } from 'node:fs';
 import { join, dirname, sep } from 'node:path';
 import { tmpdir } from 'node:os';
 import { fileURLToPath } from 'node:url';
@@ -14,6 +14,41 @@ import { snapshot, TASK_SELECT } from './tasks.js';
 
 const ROOT_DIR = join(dirname(fileURLToPath(import.meta.url)), '..', '..');
 
+const PACKAGED = ROOT_DIR.split(sep).includes('node_modules');
+
+let selfPkgCache;
+function selfPkg() {
+  if (selfPkgCache === undefined) {
+    try {
+      const { name, version } = JSON.parse(readFileSync(join(ROOT_DIR, 'package.json'), 'utf8'));
+      selfPkgCache = name && version ? { name, version } : null;
+    } catch { selfPkgCache = null; }
+  }
+  return selfPkgCache;
+}
+
+export function isNewer(candidate, current) {
+  const nums = (v) => String(v).split('-')[0].split('.').map((n) => parseInt(n, 10) || 0);
+  const [a, b] = [nums(candidate), nums(current)];
+  for (let i = 0; i < 3; i++) if ((a[i] || 0) !== (b[i] || 0)) return (a[i] || 0) > (b[i] || 0);
+  return false;
+}
+
+export async function registryCheck() {
+  const pkg = selfPkg();
+  if (!pkg) return null;
+  try {
+    const res = await fetch(`https://registry.npmjs.org/${encodeURIComponent(pkg.name)}/latest`, {
+      headers: { 'User-Agent': 'local-kanban', Accept: 'application/json' },
+      signal: AbortSignal.timeout(8000),
+    });
+    if (!res.ok) return null;
+    const remote = (await res.json())?.version;
+    if (!remote) return null;
+    return { update_available: isNewer(remote, pkg.version), local: pkg.version, remote, branch: null, tag: null, dev: null };
+  } catch { return null; }
+}
+
 const UPLOAD_FLOOR = 64 * 1024 * 1024;
 function uploadLimit() {
   try { return Math.max(UPLOAD_FLOOR, statSync(join(DATA_DIR, 'kanban.db')).size * 4); } catch { return UPLOAD_FLOOR; }
@@ -169,17 +204,20 @@ export default async function systemRoutes(app) {
       tasks: db.prepare('SELECT COUNT(*) AS n FROM tasks').get().n,
       projects: db.prepare('SELECT COUNT(*) AS n FROM projects').get().n,
       node: process.version,
-      packaged: ROOT.split(sep).includes('node_modules'),
+      packaged: PACKAGED,
+      version: selfPkg()?.version || null,
     };
   });
 
-  let updCache = { at: 0, data: null };
+  const CACHE_OK_MS = 3600000;
+  const CACHE_FAIL_MS = 60000;
+  let updCache = { at: 0, ttl: 0, data: null };
   let lastForced = 0;
   const FORCE_EVERY = 15000;
   app.get('/api/update-check', async (req) => {
     const forced = Boolean(req.query?.refresh) && Date.now() - lastForced >= FORCE_EVERY;
     if (forced) lastForced = Date.now();
-    if (!forced && updCache.data && Date.now() - updCache.at < 3600000) return updCache.data;
+    if (!forced && updCache.data && Date.now() - updCache.at < updCache.ttl) return updCache.data;
     const ROOT = join(dirname(fileURLToPath(import.meta.url)), '..', '..');
     const git = (args) => new Promise((resolve) => {
       execFile('git', args, { cwd: ROOT, timeout: 8000 }, (err, out) => resolve(err ? null : out.trim()));
@@ -224,7 +262,8 @@ export default async function systemRoutes(app) {
         }
       }
     } catch {  }
-    updCache = { at: Date.now(), data };
+    if (data.update_available === null) data = (await registryCheck()) || data;
+    updCache = { at: Date.now(), ttl: data.update_available === null ? CACHE_FAIL_MS : CACHE_OK_MS, data };
     return data;
   });