@sdods/cli 0.2.0 → 0.2.1

package/dist/commands/analyze.js

@@ -169,41 +169,11 @@ function printReport(r) {
         out(`${icon} ${pc.bold(c.title)} — ${c.detail}${c.fix ? pc.dim(`\n    → ${c.fix}`) : ''}`);
     }
 }
-/**
- * Why each module exists. Modules are inferred by fusing several signals - the file that defines
- * a route, the monorepo workspace, a domain directory, an OpenAPI tag - so the winning signal and
- * its confidence are the part a reviewer actually needs to judge the proposal.
- */
-function printModules(p) {
-    const mods = p.detectedModules ?? [];
-    if (!mods.length)
-        return;
-    out(pc.dim('--- modules ---'));
-    const w = Math.max(6, ...mods.map((m) => m.name.length));
-    for (const m of mods) {
-        const targets = [
-            m.routes ? `${m.routes} route${m.routes === 1 ? '' : 's'}` : '',
-            m.endpoints ? `${m.endpoints} endpoint${m.endpoints === 1 ? '' : 's'}` : '',
-        ]
-            .filter(Boolean)
-            .join(', ');
-        const from = m.evidence[0]?.file;
-        out(`  ${m.name.padEnd(w)}  ${pc.dim(m.confidence.toFixed(2))}  ${targets.padEnd(22)}` +
-            `${pc.dim(m.signals.join('+'))}${from ? pc.dim(`  ← ${from}`) : ''}`);
-    }
-    const ignored = p.ignoredTargets ?? [];
-    if (ignored.length) {
-        out(pc.dim(`--- skipped (${ignored.length}) ---`));
-        for (const i of ignored)
-            out(pc.dim(`  ${i.target.padEnd(20)} ${i.reason}`));
-    }
-}
 function printProposal(p) {
     heading(`Proposed project: ${p.slug}`);
     out(pc.dim(`envs: ${Object.keys(p.envYamls).join(', ')} · starter features: ${Object.keys(p.starterFeatures).length} · coverage targets: ${p.coverageMap.length}`));
     for (const n of p.notes)
         warn(n);
-    printModules(p);
     out(pc.dim('--- sdods.project.yaml ---'));
     const lines = p.projectYaml.split('\n');
     out(lines.slice(0, 60).join('\n'));

package/dist/commands/browsers.js

@@ -76,7 +76,7 @@ export function register(program) {
     });
     browsers
         .command('list')
-        .description('Show which browsers are installed and where')
+        .description('Show which browsers are installed and where; exits 1 if any is missing')
         .option('-p, --project <slug>', 'limit to the browsers declared by a project')
         .action(async (opts, cmd) => {
         const ctx = createContext(cmd);

package/dist/commands/coverage.js

@@ -4,7 +4,7 @@ import { heading, json, out, table } from '../ui.js';
 export function register(program) {
     program
         .command('coverage')
-        .description('Route, endpoint and role coverage by scenarios, split by suite tag')
+        .description('Route, endpoint and role coverage by scenarios, split by suite tag; exits 1 if any module has no scenarios')
         .requiredOption('-p, --project <slug>', 'project slug')
         .option('-e, --env <name>', 'environment (for the OpenAPI spec and API base URL)')
         .option('--openapi [file]', 'include endpoints from the OpenAPI spec (env api.openapi or a file)')

package/dist/commands/tokens.js

@@ -86,21 +86,28 @@ export function register(program) {
     tokens
         .command('revoke <id>')
         .description('Revoke a token by id')
-        .action(async (id, cmd) => {
+        .action(async (id, _opts, cmd) => {
         const ctx = createContext(cmd);
         const db = await import('@sdods/db');
         const adb = await db.openDb();
         try {
-            await db.revokeApiToken(adb.db, id);
-            await db.audit(adb.db, {
-                actorType: 'cli',
-                action: 'token.revoke',
-                targetType: 'api_token',
-                targetId: id,
-            });
+            // Reporting success for an id that does not exist reads as "the token is gone" when
+            // nothing was checked at all — an operator revoking a leaked token needs the difference.
+            const revoked = await db.revokeApiToken(adb.db, id);
+            if (revoked) {
+                await db.audit(adb.db, {
+                    actorType: 'cli',
+                    action: 'token.revoke',
+                    targetType: 'api_token',
+                    targetId: id,
+                });
+            }
             if (ctx.opts.json)
-                return json({ id, revoked: true });
-            ok(`Revoked ${id}`);
+                return json({ id, revoked });
+            if (revoked)
+                ok(`Revoked ${id}`);
+            else
+                warn(`No live token with id ${id}; nothing to revoke.`);
         }
         finally {
             await adb.close();

package/dist/commands/users.js

@@ -88,7 +88,7 @@ export function register(program) {
     users
         .command('set-role <username> <role>')
         .description('Change a platform role (viewer | editor | admin)')
-        .action(async (username, role, cmd) => {
+        .action(async (username, role, _opts, cmd) => {
         const ctx = createContext(cmd);
         const adb = await openDb();
         try {
@@ -112,7 +112,7 @@ export function register(program) {
     users
         .command('deactivate <username>')
         .description('Deactivate a user (sessions and tokens stop working)')
-        .action(async (username, cmd) => {
+        .action(async (username, _opts, cmd) => {
         const ctx = createContext(cmd);
         const adb = await openDb();
         try {

package/dist/context.js

@@ -1,5 +1,12 @@
 import { ProjectRegistry, setLogJson, setLogLevel } from '@sdods/core';
 export function globalOptions(cmd) {
+    // Commander calls an action with (...positionalArgs, options, command). A callback that forgets
+    // the options parameter receives the options object here instead of the Command, and the failure
+    // surfaced as an unhelpful "root.opts is not a function".
+    if (typeof cmd?.opts !== 'function' || !('parent' in cmd)) {
+        throw new Error('createContext expected the Commander Command. An action callback takes ' +
+            '(...args, options, command) — the options parameter is probably missing.');
+    }
     let root = cmd;
     while (root.parent)
         root = root.parent;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sdods/cli",
-  "version": "0.2.0",
+  "version": "0.2.1",
   "description": "SDODS command line: run, record, analyze, agents, MCP server, scheduler and the web server.",
   "license": "Apache-2.0",
   "author": "SDODS <admin@sdods.com>",
@@ -23,8 +23,8 @@
     "node": ">=22"
   },
   "dependencies": {
-    "@sdods/contracts": "0.2.0",
-    "@sdods/core": "0.2.0",
+    "@sdods/contracts": "0.2.1",
+    "@sdods/core": "0.2.1",
     "commander": "^15.0.0",
     "execa": "^10.0.1",
     "picocolors": "^1.1.1",
@@ -33,12 +33,12 @@
     "zod": "^4.5.4",
     "@playwright/test": "^1.62.1",
     "playwright-core": "^1.62.1",
-    "@sdods/integrations": "0.2.0",
-    "@sdods/mcp": "0.2.0",
-    "@sdods/agents": "0.2.0",
-    "@sdods/db": "0.2.0",
+    "@sdods/integrations": "0.2.1",
+    "@sdods/mcp": "0.2.1",
+    "@sdods/agents": "0.2.1",
+    "@sdods/db": "0.2.1",
     "csv-parse": "^7.0.2",
-    "@sdods/server": "0.2.0"
+    "@sdods/server": "0.2.1"
   },
   "homepage": "https://sdods.com",
   "bugs": {