screentinker/server/test/billing-token-mint.test.js
ScreenTinker 385eda3cb1 feat(#146): owner-only CLI to mint billing:read tokens (scripts/mint-billing-token.js)
The billing:read scope + dual-path gate were built but there was no way to MINT a token
(and it must NOT go in the workspace-scoped, self-service API-Tokens UI). Adds a server-side,
owner-only CLI — no new UI, no network endpoint. Owner-only BY CONSTRUCTION: it's a
host-side script, so filesystem/shell access = the platform owner.

- server/lib/billing-token.js (testable): mintBillingToken/revokeBillingToken/
  listBillingTokens. Reuses the EXACT existing token path — same secret (st_ + 32 bytes
  base64url), same SHA-256 hashing (hashToken), same api_tokens columns — no second format.
  Resolves the platform OWNER (oldest platform_admin/superadmin; #14 collapsed superadmin ->
  platform_admin so that's the top tier) and binds to their workspace. api_tokens.user_id +
  workspace_id are BOTH NOT NULL (no platform-level token exists); the workspace binding is
  VESTIGIAL for billing (billing:read is off-ladder -> can't reach any workspace router;
  billing is platform-global), documented in-file rather than loosening NOT NULL pre-release.
- scripts/mint-billing-token.js: thin CLI wrapper. --name mints and prints the secret ONCE
  (+ id, + "run as owner on host" warning), --list, --revoke <id> (soft revoke, mirrors the
  dashboard DELETE).

Tests (4, test/billing-token-mint.test.js): minted row is scope EXACTLY billing:read with a
matching SHA-256 hash and no read/write/full/agency scope; the token reads GET
/api/billing/usage (200) but is refused on /api/devices (403) and /api/admin (401) — scope
isolation; revocation -> 401; mint requires a name; revoke refuses a non-billing id. CLI
smoked live (mint/list/revoke). Suite 310/310.

SPEC-vs-REALITY (again): spec said bcrypt + JSON `scopes`; this codebase uses SHA-256 + a
single `scope` TEXT column. Built to the real system.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
2026-07-01 21:31:11 -05:00

85 lines
4.5 KiB
JavaScript

'use strict';
// #146 — owner-only billing:read token minting (server/lib/billing-token.js + the CLI it
// backs). Booted server + in-process mint function. Verifies the minted row's exact shape
// (scope + SHA-256 hash, nothing else), that the token reads billing but is refused
// elsewhere (scope isolation), and that revocation takes effect.
const { test, before, after } = require('node:test');
const assert = require('node:assert/strict');
const { spawn } = require('node:child_process');
const path = require('node:path');
const os = require('node:os');
const fs = require('node:fs');
const crypto = require('node:crypto');
const Database = require('better-sqlite3');
const PORT = 4021;
const BASE = `http://127.0.0.1:${PORT}`;
const DATA_DIR = path.join(os.tmpdir(), 'st-billmint-' + crypto.randomBytes(4).toString('hex'));
process.env.DATA_DIR = DATA_DIR; // so requiring lib/billing-token's deps resolves this db too
const { mintBillingToken, revokeBillingToken, listBillingTokens } = require('../lib/billing-token');
const { hashToken } = require('../middleware/apiToken');
let proc, db, minted;
before(async () => {
const logFd = fs.openSync(path.join(os.tmpdir(), 'st-billmint.log'), 'w');
proc = spawn('node', ['server.js'], {
cwd: path.join(__dirname, '..'),
env: { ...process.env, DATA_DIR, SELF_HOSTED: 'true', PORT: String(PORT), NODE_ENV: 'test' },
stdio: ['ignore', logFd, logFd],
});
let up = false;
for (let i = 0; i < 80; i++) { try { const r = await fetch(BASE + '/api/status'); if (r.ok) { up = true; break; } } catch { /* */ } await new Promise(r => setTimeout(r, 250)); }
if (!up) throw new Error('server did not boot');
db = new Database(path.join(DATA_DIR, 'db', 'remote_display.db'));
// Register a user and promote to platform_admin so an OWNER + a workspace exist.
const email = 'own' + crypto.randomBytes(4).toString('hex') + '@x.local';
await (await fetch(BASE + '/api/auth/register', { method: 'POST', headers: { 'Content-Type': 'application/json' }, body: JSON.stringify({ email, password: 'Passw0rd123' }) })).json();
db.prepare("UPDATE users SET role = 'platform_admin' WHERE email = ?").run(email);
minted = mintBillingToken(db, { name: 'Bold invoicing' }); // the function the CLI wraps
});
after(() => { try { db && db.close(); } catch { /* */ } try { proc.kill('SIGKILL'); } catch { /* */ } });
const S = (r) => r.status;
const bearer = (t) => ({ headers: { Authorization: 'Bearer ' + t } });
test('mint creates an api_tokens row: scope EXACTLY billing:read, correct SHA-256 hash', () => {
const row = db.prepare('SELECT * FROM api_tokens WHERE id = ?').get(minted.id);
assert.ok(row, 'row exists');
assert.equal(row.scope, 'billing:read', 'scope is exactly billing:read');
assert.equal(row.token_hash, hashToken(minted.secret), 'stored hash matches the SHA-256 verification path');
assert.ok(minted.secret.startsWith('st_'), 'same secret format as existing tokens');
// does NOT carry any workspace-level scope
for (const s of ['read', 'write', 'full', 'agency']) assert.notEqual(row.scope, s);
assert.ok(row.user_id && row.workspace_id, 'bound to owner + a workspace (FK satisfied)');
assert.equal(row.revoked_at, null, 'not revoked at mint');
// listBillingTokens surfaces it
assert.ok(listBillingTokens(db).some((t) => t.id === minted.id));
});
test('minted token reads billing (200) but is REFUSED elsewhere (scope isolation)', async () => {
assert.equal(S(await fetch(BASE + '/api/billing/usage', bearer(minted.secret))), 200, 'reads billing');
const body = await (await fetch(BASE + '/api/billing/usage', bearer(minted.secret))).json();
assert.equal(typeof body.billable_screens, 'number');
assert.equal(S(await fetch(BASE + '/api/devices', bearer(minted.secret))), 403, 'refused on a workspace router');
assert.equal(S(await fetch(BASE + '/api/admin/orgs', bearer(minted.secret))), 401, 'refused on an admin router');
});
test('revocation: a revoked minted token is refused', async () => {
assert.equal(S(await fetch(BASE + '/api/billing/usage', bearer(minted.secret))), 200, 'valid before revoke');
const res = revokeBillingToken(db, minted.id);
assert.equal(res.ok, true);
assert.equal(S(await fetch(BASE + '/api/billing/usage', bearer(minted.secret))), 401, 'refused after revoke');
});
test('mint requires a name; revoke refuses a non-billing token id', () => {
assert.throws(() => mintBillingToken(db, { name: '' }), /name is required/);
// revoke guard: a made-up id / non-billing scope is refused, not silently applied
assert.equal(revokeBillingToken(db, 'no-such-id').ok, false);
});