@andrian.yablonskyy/thub-coordinator 1.0.43 → 1.1.1

package/bin/thub-admin.js

@@ -34,14 +34,24 @@ const { loadConfig } = require('../src/config'),
   { createCleanupService } = require('../src/services/cleanup'),
   { generateToken } = require('../src/services/tokens'),
   { createSettingsService } = require('../src/services/settings'),
+  { roleLabel, normalizeRole } = require('../src/services/admin-users'),
   { PACKAGES, fetchLatestVersion, isNewer, isValidVersion, npmBin, formatDateTime, parseDateTime } = require('@andrian.yablonskyy/thub-common'),
   { spawnSync } = require('node:child_process'),
   { version: installedVersion } = require('../package.json');
 
 function usage(){
   console.log(`Usage:
-  thub-admin create-admin <username> <password> [--role admin|viewer]
-  thub-admin agent add <name> --kind ci|cli
+  thub-admin user add <username> --email <email> [--role user|maintainer|admin] [--password <p>] [--key]
+                                  New user (§10.3). Maintainers/admins without --password get a
+                                  temporary one (printed once); --key also creates their access key
+  thub-admin user list
+  thub-admin user block|unblock <username>
+  thub-admin user role <username> user|maintainer|admin
+  thub-admin user password <username> <password>    Set a password (no change required)
+  thub-admin user key <username>  New access key for them (printed once; the old one stops)
+  thub-admin user delete <username>
+  thub-admin create-admin <username> <password> [--role admin|maintainer]   (= user add, no email)
+  thub-admin agent add <name>      A CI token (pipelines; people get their key with: user key)
   thub-admin join-key generate
   thub-admin resource maintenance <resourceId> --on|--off
   thub-admin jobs reset --yes     Cancel every queued/assigned/preparing/running job
@@ -138,7 +148,7 @@ function main(){
     events = createEventsService(db),
     registry = createRegistryService(db, { bus, events }),
     agents = createAgentsService(db, { events }),
-    adminUsers = createAdminUsersService(db),
+    adminUsers = createAdminUsersService(db, { events }),
     jobs = createJobsService(db, { bus, events, registry, config }),
     groups = createGroupsService(db, { events, registry });
 
@@ -159,6 +169,83 @@ function main(){
     return;
   }
 
+  // Users (§10.3) — also the way back in when nobody can sign in: unblock or
+  // re-role an admin, set a password. Run as the Coordinator's user.
+  if (cmd === 'user'){
+    const { positional, flags } = parseFlags(rest),
+      [username, arg] = positional,
+      by = 'thub-admin',
+      need = (u) => {
+        const row = adminUsers.getByUsername(u || '');
+        if (!row){
+          console.error(`No user "${u}"`);
+          process.exit(4);
+        }
+        return row;
+      };
+    try {
+      if (sub === 'list'){
+        for (const u of adminUsers.list()){
+          const key = u.key ? `…${u.key.hint || '????'}` : 'none';
+          console.log(`${u.username.padEnd(24)} ${roleLabel(u.role).padEnd(11)} ${u.blocked ? 'blocked' : 'active '}  ${u.email || '(no email)'}  key: ${key}`);
+        }
+        return;
+      }
+      if (sub === 'add'){
+        if (!username || !flags.email){
+          return usage(), process.exit(4);
+        }
+        const created = adminUsers.create({ username, email: flags.email, role: flags.role || 'user', password: flags.password, requireEmail: true }, { by });
+        console.log(`Created ${roleLabel(created.role)} "${created.username}"`);
+        if (created.tempPassword){
+          console.log(`Temporary password (shown once; to change at the first sign-in): ${created.tempPassword}`);
+        }
+        if (rest.includes('--key')){
+          console.log(`Access key (shown once): ${adminUsers.issueKey(created.id, { by })}`);
+        }
+        return;
+      }
+      const target = need(username);
+      if (sub === 'block' || sub === 'unblock'){
+        adminUsers.setBlocked(target.id, sub === 'block', { by });
+        console.log(`${sub === 'block' ? 'Blocked' : 'Unblocked'} "${target.username}"`);
+      }
+      else if (sub === 'role'){
+        const { tempPassword } = adminUsers.update(target.id, { role: arg }, { by });
+        console.log(`"${target.username}" is now ${roleLabel(normalizeRole(arg))}`);
+        if (tempPassword){
+          console.log(`Temporary password (shown once): ${tempPassword}`);
+        }
+      }
+      else if (sub === 'password'){
+        if (!arg){
+          return usage(), process.exit(4);
+        }
+        adminUsers.resetPassword({ username: target.username, password: arg });
+        console.log(`Password of "${target.username}" set`);
+      }
+      else if (sub === 'key'){
+        console.log(`Access key for "${target.username}" (shown once; any previous one has stopped working): ${adminUsers.issueKey(target.id, { by })}`);
+      }
+      else if (sub === 'delete'){
+        if (!flags.yes){
+          console.error(`This deletes "${target.username}" for good (their jobs stay). Re-run with --yes to confirm.`);
+          process.exit(4);
+        }
+        adminUsers.remove(target.id, { by });
+        console.log(`Deleted "${target.username}"`);
+      }
+      else {
+        return usage(), process.exit(4);
+      }
+    }
+    catch (err){
+      console.error(`Error: ${err.message}`);
+      process.exit(1);
+    }
+    return;
+  }
+
   if (cmd === 'create-admin'){
     const { positional, flags } = parseFlags([sub, ...rest].filter(Boolean)),
       [username, password] = positional;
@@ -166,18 +253,22 @@ function main(){
       return usage(), process.exit(4);
     }
     adminUsers.create({ username, password, role: flags.role || 'admin' });
-    console.log(`Created ${flags.role || 'admin'} user "${username}"`);
+    console.log(`Created ${roleLabel(normalizeRole(flags.role || 'admin'))} user "${username}"`);
     return;
   }
 
   if (cmd === 'agent' && sub === 'add'){
     const { positional, flags } = parseFlags(rest),
       [name] = positional;
-    if (!name || !flags.kind){
+    if (!name){
       return usage(), process.exit(4);
     }
-    const { agent, token } = agents.create({ name, kind: flags.kind });
-    console.log(`Agent ${agent.id} created. Token (shown once): ${token}`);
+    if (flags.kind && flags.kind !== 'ci'){
+      console.error('Agent tokens are for CI only now; a person gets their access key with: thub-admin user key <username>');
+      process.exit(4);
+    }
+    const { agent, token } = agents.create({ name, kind: 'ci' });
+    console.log(`CI token ${agent.id} created. Token (shown once): ${token}`);
     return;
   }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@andrian.yablonskyy/thub-coordinator",
-  "version": "1.0.43",
+  "version": "1.1.1",
   "description": "TestHub Coordinator — job queue, resource registry, scheduler, heartbeat monitor, log/artifact store and web dashboard",
   "main": "src/server.js",
   "engines": {
@@ -19,7 +19,7 @@
     "lint:fix": "eslint . --fix"
   },
   "dependencies": {
-    "@andrian.yablonskyy/thub-common": "^1.0.33",
+    "@andrian.yablonskyy/thub-common": "^1.1.1",
     "better-sqlite3": "^12.4.1",
     "express": "^5.1.0",
     "express-session": "^1.18.2",

package/public/css/thub.css

@@ -86,17 +86,33 @@
   padding: 0.45rem 0.65rem;
 }
 
-/* Bootstrap hides inactive tab panes only as direct children of
-   .tab-content (`.tab-content > .tab-pane`). The client details modal's
-   capability tabs sit inside their shared <form>/<fieldset> (one Save for
-   all of them), so without this the inactive ones merely fade out and still
-   take up space, pushing the active tab's fields down. */
-.tab-content .tab-pane[data-config-pane] {
-  display: none;
+/* The client details modal keeps one size whichever tab is open — the
+   tallest tab's — instead of growing and shrinking (and, centered, jumping)
+   on every switch. Every pane sits in the same grid cell, so the tab area is
+   as tall as the tallest; inactive ones are hidden with `visibility`, which
+   keeps their height (and still takes them out of the tab order and of what
+   screen readers read). The device tabs sit inside their shared
+   <form>/<fieldset> (one Save for all of them), so that fieldset stacks its
+   panes the same way. The USB tab's output scrolls in its own box (360 px),
+   so a long lsusb doesn't make every tab that tall. */
+.thub-resource-card .tab-content,
+.thub-resource-card .tab-content > form > fieldset {
+  display: grid;
+}
+
+.thub-resource-card .tab-content > *,
+.thub-resource-card .tab-content > form > fieldset > * {
+  grid-area: 1 / 1;
+  min-width: 0;
 }
 
-.tab-content .tab-pane[data-config-pane].active {
+.thub-resource-card .tab-content .tab-pane {
   display: block;
+  visibility: hidden;
+}
+
+.thub-resource-card .tab-content .tab-pane.active {
+  visibility: visible;
 }
 
 /* Help page (views/help/): sticky table of contents, code blocks with a
@@ -233,9 +249,6 @@
    itself on short screens, and is laid out for thumbs: pages as a two-column
    grid, the tools in one row, the account in the next. public/js/actions.js
    closes it on a tap outside, Esc, or choosing a page. */
-.thub-nav-break {
-  display: none;
-}
 
 @media (max-width: 991.98px) {
   .navbar .thub-nav {
@@ -295,15 +308,19 @@
     min-height: 2.5rem;
   }
 
-  /* The account row: the profile link takes the line, sign-out at its end. */
-  .thub-nav-break {
-    display: block;
-    flex-basis: 100%;
-    height: 0;
+  /* The account row, first in the menu: you, your role, your profile, and
+     sign-out at its end. */
+  .thub-nav-me {
+    display: flex;
+    align-items: center;
+    gap: 0.75rem;
+    padding-bottom: 0.75rem;
+    margin-bottom: 0.75rem;
+    border-bottom: 1px solid var(--bs-border-color);
   }
 
-  .thub-nav-account {
-    flex: 1 1 auto;
+  .thub-nav-me .btn {
+    min-width: 2.5rem;
     min-height: 2.5rem;
   }
 }

package/public/js/actions.js

@@ -30,6 +30,16 @@
     }
   });
 
+  // <select data-describe="#hint">: show the chosen option's data-help in
+  // #hint (e.g. what a role allows, under the role picker).
+  document.addEventListener('change', (e) => {
+    const select = e.target.closest?.('select[data-describe]'),
+      hint = select && document.querySelector(select.dataset.describe);
+    if (hint){
+      hint.textContent = select.selectedOptions[0]?.dataset.help || '';
+    }
+  });
+
   // <button data-fill="#field" data-value="…">: put the value into #field.
   document.addEventListener('click', (e) => {
     const btn = e.target.closest('[data-fill]'),

package/public/js/client-config.js

@@ -34,7 +34,6 @@
       return compact({
         ...extra,
         ...device(text(tr, 'device')),
-        serial: text(tr, 'serial'),
         baudRate: num(tr, 'baudRate'),
         devpath: text(tr, 'devpath')
       });

package/src/api/agent.js

@@ -26,14 +26,17 @@ function createAgentRouter({ services, config }){
     // matching even happens, and reject them for lacking an agent token.
     auth = requireRole('agent'),
 
-    // Which jobs an agent may see (README §12): a `ci` token (pipelines) all
-    // of them; a `cli` token (a person's own `thub`) only the jobs it
-    // submitted. Anyone else's job is answered exactly like a job that
-    // doesn't exist (404), so a cli token can't tell which ids are in use.
-    canSee = (agent, job) => agent.kind === 'ci' || job.agent_id === agent.id,
+    // Which jobs a key may see (README §12, §10.3): a CI token all of them;
+    // a user's key all of them for maintainers and admins, only their own
+    // for the `user` role. Anyone else's job is answered exactly like a job
+    // that doesn't exist (404), so a user can't tell which ids are in use.
+    seesAll = (req) => req.agent.kind === 'ci' || ['maintainer', 'admin'].includes(req.user?.role),
+    canSee = (req, job) => seesAll(req) || job.agent_id === req.agent.id,
+    // Cancelling: maintainers and admins any job; anyone else their own.
+    cancelsAll = (req) => ['maintainer', 'admin'].includes(req.user?.role),
     visibleJob = (req, res, next) => {
       const job = services.jobs.get(req.params.id);
-      if (!job || !canSee(req.agent, job)){
+      if (!job || !canSee(req, job)){
         return res.status(404).json({ error: 'Unknown job' });
       }
       req.job = job;
@@ -68,8 +71,8 @@ function createAgentRouter({ services, config }){
       state: req.query.state,
       source: req.query.source,
       agentId: req.agent.id,
-      // A cli token's list is always its own jobs (canSee above).
-      mine: req.agent.kind !== 'ci' || req.query.mine === 'true' || req.query.mine === '1',
+      // A `user`'s list is always their own jobs (canSee above).
+      mine: !seesAll(req) || req.query.mine === 'true' || req.query.mine === '1',
       limit: req.query.limit ? Number(req.query.limit) : undefined
     });
     res.json({ jobs: jobs.map(services.jobs.publicJob) });
@@ -77,7 +80,7 @@ function createAgentRouter({ services, config }){
 
   router.post('/jobs/:id/cancel', auth, visibleJob, (req, res, next) => {
     try {
-      const job = services.jobs.cancel(req.params.id, { agentId: req.agent.id, isAdmin: false });
+      const job = services.jobs.cancel(req.params.id, { agentId: req.agent.id, isAdmin: cancelsAll(req) });
       res.json(services.jobs.publicJob(job));
     }
     catch (err){
@@ -101,6 +104,30 @@ function createAgentRouter({ services, config }){
     res.json({ artifacts: req.job.artifacts.map((a) => ({ ...a, url: a.link })) });
   });
 
+  // `thub whoami` / `thub key show` (§10.3): who this key belongs to.
+  router.get('/me', auth, (req, res) => {
+    if (!req.user){
+      return res.json({ kind: req.agent.kind, name: req.agent.name });
+    }
+    const { username, email, role } = req.user,
+      key = services.adminUsers.keyOf(req.user.id);
+    res.json({
+      kind: req.agent.kind,
+      user: { username, email, role },
+      key: { hint: key.token_hint, createdAt: key.token_created_at || key.created_at, lastUsedAt: key.last_used_at }
+    });
+  });
+
+  // `thub key rotate`: a user replaces their own key — the one sending this
+  // stops working at once, the new one is in the answer (only there).
+  // A CI token is replaced by an admin, on the dashboard.
+  router.post('/me/key/rotate', auth, (req, res) => {
+    if (!req.user){
+      return res.status(403).json({ error: 'A CI token is replaced by an admin on the dashboard (CI tokens)' });
+    }
+    res.json({ key: services.adminUsers.issueKey(req.user.id, { by: req.user.username }), username: req.user.username });
+  });
+
   router.get('/resources', auth, (req, res) => {
     res.json({ resources: services.registry.list().map(publicResource) });
   });

package/src/auth.js

@@ -36,11 +36,20 @@ function requireRole(role){
     const tokenHash = hashToken(token);
 
     if (role === 'agent'){
-      const agent = req.app.locals.services.agents.getByTokenHash(tokenHash);
+      const { agents, adminUsers } = req.app.locals.services,
+        agent = agents.getByTokenHash(tokenHash);
       if (!agent){
-        return res.status(401).json({ error: 'Invalid or revoked agent token' });
+        return res.status(401).json({ error: 'Invalid or revoked access key' });
       }
-      req.app.locals.services.agents.touchLastUsed(agent.id, appVersion(req, 'agent'));
+      // A user's access key (§10.3) works only while its user may use it.
+      if (agent.user_id){
+        const user = adminUsers.getById(agent.user_id);
+        if (!user || user.blocked){
+          return res.status(401).json({ error: user ? `User ${user.username} is blocked — ask an admin` : 'Invalid or revoked access key' });
+        }
+        req.user = user;
+      }
+      agents.touchLastUsed(agent.id, appVersion(req, 'agent'));
       req.agent = agent;
       return next();
     }
@@ -81,19 +90,43 @@ function requireJoinKey(config){
   };
 }
 
+// Dashboard session (§10.3): the user is read again on every request, so
+// blocking, deleting or demoting someone to `user` ends their session at
+// their next click, and a role change applies at once. A temporary password
+// must be changed (on the profile page) before anything else.
+const MUST_CHANGE_ALLOWED = ['/profile', '/profile/password', '/logout', '/live'];
 function requireAdminSession(req, res, next){
-  if (!req.session?.user){
+  const fresh = req.session?.user && req.app.locals.services.adminUsers.getById(req.session.user.id);
+  if (!fresh || fresh.blocked || !fresh.canUseDashboard){
+    const reason = !req.session?.user ? null
+      : !fresh ? 'Your account no longer exists.'
+        : fresh.blocked ? 'Your account is blocked — ask an admin.' : 'Your account has no dashboard access — use the Agent with your access key.';
+    if (req.session?.user){
+      req.session.user = null;
+      if (reason){
+        req.session.flash = [{ type: 'danger', text: reason }];
+      }
+    }
     if (req.accepts('html')){
       return res.redirect('/login');
     }
-    return res.status(401).json({ error: 'Login required' });
+    return res.status(401).json({ error: reason || 'Login required' });
+  }
+  req.session.user = fresh;
+  res.locals.user = fresh;
+  if (fresh.mustChangePassword && !MUST_CHANGE_ALLOWED.includes(req.path) && req.method === 'GET'){
+    return res.redirect('/profile');
   }
   next();
 }
 
+// The dashboard's security features (§10.3): Users, Settings, CI tokens…
 function requireAdminRole(req, res, next){
   if (req.session?.user?.role !== 'admin'){
-    return res.status(403).json({ error: 'Admin role required' });
+    const text = 'Only admins can open this — it\'s one of the Coordinator\'s security features.';
+    return req.originalUrl.startsWith('/api/') || !req.accepts('html')
+      ? res.status(403).json({ error: text })
+      : res.status(403).type('text/plain').send(text);
   }
   next();
 }

package/src/db/migrations/026_users.sql

@@ -0,0 +1,74 @@
+-- User management (README §10.3): one account table for everyone. Roles:
+--   user        submits jobs with the Agent and reads its results; no dashboard
+--   maintainer  user + the dashboard, without its security features
+--   admin       everything
+-- Each user may hold one access key: an `agents` row linked by user_id
+-- (jobs already reference agents, so a user's job history stays attached
+-- across key rotations). CI tokens stay agents rows without a user.
+--
+-- admin_users becomes users: SQLite can't change its role CHECK in place.
+CREATE TABLE users (
+  id TEXT PRIMARY KEY,
+  username TEXT NOT NULL UNIQUE,
+  email TEXT NOT NULL DEFAULT '',
+  -- NULL: no dashboard sign-in (role user, or a converted agent token).
+  password_hash TEXT,
+  role TEXT NOT NULL CHECK (role IN ('user', 'maintainer', 'admin')),
+  first_name TEXT,
+  last_name TEXT,
+  avatar_path TEXT,
+  timezone TEXT NOT NULL DEFAULT 'UTC',
+  theme TEXT NOT NULL DEFAULT 'auto' CHECK (theme IN ('auto', 'light', 'dark')),
+  session_timeout_min INTEGER NOT NULL DEFAULT 60,
+  list_prefs TEXT,
+  must_change_password INTEGER NOT NULL DEFAULT 0,
+  blocked_at TEXT,
+  created_at TEXT NOT NULL
+);
+
+-- Dashboard accounts: admin stays admin; viewer becomes maintainer.
+INSERT INTO users (id, username, email, password_hash, role, first_name, last_name, avatar_path, timezone, theme,
+                   session_timeout_min, list_prefs, created_at)
+  SELECT id, username, '', password_hash, CASE role WHEN 'viewer' THEN 'maintainer' ELSE role END,
+         first_name, last_name, avatar_path, timezone, theme, session_timeout_min, list_prefs, created_at
+  FROM admin_users;
+
+DROP TABLE admin_users;
+
+-- Every developer (cli) agent token becomes a user that owns it, so each
+-- existing key keeps working. Its name becomes the username — suffixed
+-- where that's taken (a dashboard account, an earlier token of the same
+-- name). A revoked token becomes a blocked user, kept for its job history.
+INSERT INTO users (id, username, email, password_hash, role, blocked_at, created_at)
+  SELECT 'usr_' || substr(a.id, 5),
+         a.name || CASE
+           WHEN EXISTS (SELECT 1 FROM users u WHERE u.username = a.name)
+             OR EXISTS (SELECT 1 FROM agents b WHERE b.kind = 'cli' AND b.name = a.name AND (b.created_at < a.created_at OR (b.created_at = a.created_at AND b.id < a.id)))
+           THEN '-' || substr(a.id, 5, 6) ELSE '' END,
+         '', NULL, 'user', a.revoked_at, a.created_at
+  FROM agents a
+  WHERE a.kind = 'cli';
+
+ALTER TABLE agents ADD COLUMN user_id TEXT REFERENCES users(id);
+-- The last characters of the key, to recognize it by (keys are stored hashed).
+ALTER TABLE agents ADD COLUMN token_hint TEXT;
+ALTER TABLE agents ADD COLUMN token_created_at TEXT;
+UPDATE agents SET user_id = 'usr_' || substr(id, 5) WHERE kind = 'cli';
+
+CREATE UNIQUE INDEX idx_agents_user ON agents(user_id) WHERE user_id IS NOT NULL;
+CREATE UNIQUE INDEX idx_users_email ON users(lower(email)) WHERE email <> '';
+
+-- The audit log gets user events (created, blocked, role changed, key
+-- rotated…). Same rebuild as 006: nothing has a foreign key into events.
+CREATE TABLE events_new (
+  id INTEGER PRIMARY KEY AUTOINCREMENT,
+  ts TEXT NOT NULL,
+  entity TEXT NOT NULL CHECK (entity IN ('job', 'resource', 'agent', 'group', 'user')),
+  entity_id TEXT NOT NULL,
+  type TEXT NOT NULL,
+  data TEXT
+);
+INSERT INTO events_new (id, ts, entity, entity_id, type, data) SELECT id, ts, entity, entity_id, type, data FROM events;
+DROP TABLE events;
+ALTER TABLE events_new RENAME TO events;
+CREATE INDEX idx_events_entity ON events(entity, entity_id);

package/src/server.js

@@ -68,7 +68,7 @@ function buildServices(config){
     registry = createRegistryService(db, { bus, events }),
     agents = createAgentsService(db, { events }),
     groups = createGroupsService(db, { events, registry }),
-    adminUsers = createAdminUsersService(db),
+    adminUsers = createAdminUsersService(db, { events }),
     commands = createCommandsService({ bus }),
     logs = createLogsService(db, { bus }),
     jobs = createJobsService(db, { bus, events, registry, config }),