ticketlens 0.9.16 → 0.9.18

package/bin/ticketlens.mjs

@@ -34,15 +34,26 @@ import { browserLogin } from '../skills/jtb/scripts/lib/browser-login.mjs';
 import { syncProfiles, getApiBase, getConsoleBase } from '../skills/jtb/scripts/lib/sync.mjs';
 import { promptSecret, promptText } from '../skills/jtb/scripts/lib/prompt-helpers.mjs';
 import { checkForUpdate, getUpdateHint } from '../skills/jtb/scripts/lib/update-check.mjs';
-import { incrementInvocation } from '../skills/jtb/scripts/lib/activity-counter.mjs';
+import { incrementInvocation, incrementCommand } from '../skills/jtb/scripts/lib/activity-counter.mjs';
 import { DEFAULT_CONFIG_DIR } from '../skills/jtb/scripts/lib/config.mjs';
 
+const TRACKED_COMMANDS = new Set([
+  'triage', 'fetch', 'get', 'compliance', 'review', 'standup',
+  'pr', 'ledger', 'stats', 'collisions', 'history', 'schedule',
+  'brief', 'sync',
+]);
+
 const args = process.argv.slice(2);
 const { command, args: cmdArgs } = parseCommand(args);
 
 // Best-effort invocation counter — non-fatal
 try { incrementInvocation(DEFAULT_CONFIG_DIR); } catch { /* non-fatal */ }
 
+// Per-command + per-flag tracking — non-fatal
+if (TRACKED_COMMANDS.has(command)) {
+  try { incrementCommand(DEFAULT_CONFIG_DIR, command, cmdArgs); } catch { /* non-fatal */ }
+}
+
 // Fire-and-forget: silently refresh license.json at startup if >7 days since last validation
 revalidateIfStale();
 // Fire-and-forget: refresh the cached latest npm version once per 24h

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ticketlens",
-  "version": "0.9.16",
+  "version": "0.9.18",
   "description": "Jira CLI for developers — fetch ticket context, triage your queue, and stop tab-switching. Zero dependencies, all local.",
   "type": "module",
   "bin": {

package/skills/jtb/SKILL.md

@@ -1,4 +1,4 @@
-<!-- jtb-skill-version: 0.9.16 -->
+<!-- jtb-skill-version: 0.9.18 -->
 ---
 name: jtb
 description: Fetch a Jira ticket's full context (description, comments, linked issues, code references) and assemble a structured TicketBrief for implementation planning. Use when user types /jtb, mentions a Jira ticket key, or wants to plan work from a Jira ticket.

package/skills/jtb/scripts/fetch-my-tickets.mjs

@@ -11,6 +11,7 @@ import { styleTriageSummary } from './lib/styled-assembler.mjs';
 import { resolveConnection, loadProfiles, saveProfile } from './lib/profile-resolver.mjs';
 import { resolveAdapter } from './lib/resolve-adapter.mjs';
 import { incrementTriageRun, readAndResetActivity } from './lib/activity-counter.mjs';
+import { DEFAULT_CONFIG_DIR } from './lib/config.mjs';
 import { writeFileSync, mkdirSync, statSync } from 'node:fs';
 import { resolve as resolvePath, dirname } from 'node:path';
 import { createSpinner } from './lib/spinner.mjs';
@@ -485,7 +486,7 @@ export async function run(args, envOrOpts = process.env, fetcher = globalThis.fe
     const cliToken = opts.cliToken ?? readCliToken(configDir) ?? null;
     const printFn = opts.print ?? ((s) => process.stdout.write(s));
     let cliActivity = null;
-    try { cliActivity = readAndResetActivity(configDir); } catch { /* non-fatal */ }
+    try { cliActivity = readAndResetActivity(configDir ?? DEFAULT_CONFIG_DIR); } catch { /* non-fatal */ }
     await pushFn({
       sorted,
       rawTicketMap,

package/skills/jtb/scripts/fetch-ticket.mjs

@@ -12,7 +12,7 @@ import { extractCodeReferences } from './lib/code-ref-parser.mjs';
 import { assembleBrief } from './lib/brief-assembler.mjs';
 import { styleBrief } from './lib/styled-assembler.mjs';
 import { resolveConnection, loadProfiles, loadCredentials, saveProfile } from './lib/profile-resolver.mjs';
-import { buildJiraEnv } from './lib/config.mjs';
+import { buildJiraEnv, DEFAULT_CONFIG_DIR } from './lib/config.mjs';
 import { resolveAdapter } from './lib/resolve-adapter.mjs';
 import { createSession } from './lib/banner.mjs';
 import { classifyError } from './lib/error-classifier.mjs';
@@ -24,6 +24,7 @@ import { handleUnknownFlags } from './lib/arg-validator.mjs';
 import { TICKET_KEY_PATTERN } from './lib/cli.mjs';
 import { downloadAttachments } from './lib/attachment-downloader.mjs';
 import { readBriefCache, writeBriefCache, readSummaryCache, writeSummaryCache, briefCacheAge, BRIEF_TTL_MS } from './lib/brief-cache.mjs';
+import { recordTokensSaved } from './lib/activity-counter.mjs';
 import { readCliToken } from './lib/cli-auth.mjs';
 import { resolveTemplate } from './lib/template-resolver.mjs';
 import { parseAge } from './lib/cache-manager.mjs';
@@ -1054,6 +1055,7 @@ export async function run(args, envOrOpts = process.env, fetcher = globalThis.fe
         }
       }
 
+      recordTokensSaved(configDir ?? DEFAULT_CONFIG_DIR, 'fetch', Math.round(brief.length / 4));
       printFn(brief + '\n');
       return;
     }
@@ -1241,6 +1243,7 @@ export async function run(args, envOrOpts = process.env, fetcher = globalThis.fe
     }
   }
 
+  recordTokensSaved(configDir ?? DEFAULT_CONFIG_DIR, 'fetch', Math.round(output.length / 4));
   printFn(output + '\n');
 
   // Contextual upsell: after a deep traversal with a substantial graph, nudge toward --summarize

package/skills/jtb/scripts/lib/activity-counter.mjs

@@ -24,7 +24,7 @@ function read(configDir) {
   try {
     return JSON.parse(fs.readFileSync(path.join(configDir, ACTIVITY_FILE), 'utf8'));
   } catch {
-    return { fetch_count: 0, triage_run_count: 0, invocations: 0 };
+    return { fetch_count: 0, triage_run_count: 0, invocations: 0, commands: {} };
   }
 }
 
@@ -59,12 +59,53 @@ export function incrementInvocation(configDir) {
   increment(configDir, 'invocations');
 }
 
+/**
+ * Records one invocation of a named command, plus each --flag present in
+ * flagArgs. Flag values are stripped so "--depth=2" tracks as "--depth".
+ *
+ * @param {string}   configDir
+ * @param {string}   command   - e.g. "triage", "fetch"
+ * @param {string[]} flagArgs  - the raw args passed to the command
+ */
+export function incrementCommand(configDir, command, flagArgs = []) {
+  const data = read(configDir);
+  if (!data.commands) data.commands = {};
+  if (!data.commands[command]) data.commands[command] = { count: 0 };
+
+  data.commands[command].count += 1;
+
+  for (const arg of flagArgs) {
+    if (!arg.startsWith('-')) continue;
+    const flag = arg.replace(/=.*$/, '');
+    data.commands[command][flag] = (data.commands[command][flag] ?? 0) + 1;
+  }
+
+  write(configDir, data);
+}
+
+/**
+ * Accumulates estimated tokens saved for a named command.
+ * Called from fetch-ticket.mjs after assembling the brief.
+ * Best-effort — swallows write errors like all other counters.
+ *
+ * @param {string} configDir
+ * @param {string} command   - e.g. "fetch"
+ * @param {number} tokens    - estimated tokens saved (brief.length / 4)
+ */
+export function recordTokensSaved(configDir, command, tokens) {
+  const data = read(configDir);
+  if (!data.commands) data.commands = {};
+  if (!data.commands[command]) data.commands[command] = { count: 0 };
+  data.commands[command].tokens_saved = (data.commands[command].tokens_saved ?? 0) + tokens;
+  write(configDir, data);
+}
+
 /**
  * Returns the current counters and resets them to zero.
  * Call only after a confirmed successful push.
  *
  * @param {string} configDir
- * @returns {{ fetch_count: number, triage_run_count: number, invocations: number }}
+ * @returns {{ fetch_count: number, triage_run_count: number, invocations: number, commands: object }}
  */
 export function readAndResetActivity(configDir) {
   const data = read(configDir);
@@ -72,7 +113,8 @@ export function readAndResetActivity(configDir) {
     fetch_count:      data.fetch_count      ?? 0,
     triage_run_count: data.triage_run_count ?? 0,
     invocations:      data.invocations      ?? 0,
+    commands:         data.commands         ?? {},
   };
-  write(configDir, { fetch_count: 0, triage_run_count: 0, invocations: 0 });
+  write(configDir, { fetch_count: 0, triage_run_count: 0, invocations: 0, commands: {} });
   return snapshot;
 }

package/skills/jtb/scripts/lib/browser-login.mjs

@@ -3,6 +3,7 @@ import { randomBytes }       from 'node:crypto';
 import { spawn }             from 'node:child_process';
 import { hostname as osHostname } from 'node:os';
 import { getConsoleBase }    from './sync.mjs';
+import { readLicense }       from './license.mjs';
 
 const PORT_MIN    = 49152;
 const PORT_MAX    = 65535;
@@ -102,20 +103,34 @@ export function startLocalServer(port, expectedState, timeoutMs = TIMEOUT_MS) {
 /**
  * Full browser login flow. Opens the authorize page in the default browser,
  * waits for the callback, and returns the plaintext CLI token.
+ *
+ * @param {object}   [opts]
+ * @param {Function} [opts.readLicenseFn]  - Injectable license reader (default: readLicense)
+ * @param {Function} [opts.openBrowserFn]  - Injectable browser opener (default: openBrowser)
+ * @param {Function} [opts.startServerFn]  - Injectable local server (default: startLocalServer)
  */
-export async function browserLogin() {
+export async function browserLogin({
+  readLicenseFn = () => { try { return readLicense(); } catch { return null; } },
+  openBrowserFn = openBrowser,
+  startServerFn = (port, state) => startLocalServer(port, state),
+} = {}) {
   const state    = generateState();
   const port     = pickPort();
   const hostname = osHostname();
 
   const consoleBase = getConsoleBase();
-  const url = `${consoleBase}/console/auth/cli`
+  let url = `${consoleBase}/console/auth/cli`
     + `?port=${port}`
     + `&state=${encodeURIComponent(state)}`
     + `&hostname=${encodeURIComponent(hostname)}`;
 
-  const tokenPromise = startLocalServer(port, state);
-  openBrowser(url);
+  const license = readLicenseFn();
+  if (license?.email) {
+    url += `&email=${encodeURIComponent(license.email)}`;
+  }
+
+  const tokenPromise = startServerFn(port, state);
+  openBrowserFn(url);
 
   return tokenPromise;
 }

package/skills/jtb/scripts/lib/run-collisions.mjs

@@ -2,7 +2,7 @@ import { readCliToken } from './cli-auth.mjs';
 import { formatCollisions } from './collision-reporter.mjs';
 import { DEFAULT_CONFIG_DIR } from './config.mjs';
 import { apiBase, warnIfInsecure } from './api-utils.mjs';
-import { red, yellow, bold, cyan } from './ansi.mjs';
+import { red, yellow, bold, cyan, dim } from './ansi.mjs';
 
 const COLLISIONS_PATH = '/v1/triage/collisions';
 
@@ -41,11 +41,11 @@ export async function runCollisions(args = [], opts = {}) {
 
     if (!res.ok) {
       if (res.status === 401) {
-        print(`  ${red('✗')} Session expired. Run ${cyan('ticketlens login')} to reconnect.\n`);
+        print(`  ${red('✗')} Session expired — run ${cyan('ticketlens login')} to reconnect.\n\n`);
         return { ok: false, status: 401 };
       }
       if (res.status === 403) {
-        print(`  ${red('✗')} ${bold('collisions')} requires a Team license\n`);
+        print(`  ${red('✗')} ${bold('collisions')} requires a Team license.\n  ${dim('→ Log in with a team account:')} ${cyan('ticketlens login')}\n\n`);
         return { ok: false, status: 403 };
       }
       print(`  ${yellow('⚠')} Failed to fetch collisions (${res.status})\n`);

package/skills/jtb/scripts/lib/schedule-wizard.mjs

@@ -107,7 +107,7 @@ export async function runScheduleWizard({
 
     if (!res.ok) {
       if (res.status === 401) {
-        print(`  ${red('✗')} Session expired. Run ${cyan('ticketlens login')} to reconnect.\n`);
+        print(`  ${red('✗')} Session expired — run ${cyan('ticketlens login')} to reconnect.\n\n`);
         return { ok: false, status: 401 };
       }
       if (res.status === 403) {

package/skills/jtb/scripts/lib/triage-push.mjs

@@ -8,7 +8,7 @@ import { isLicensed } from './license.mjs';
 import { DEFAULT_CONFIG_DIR } from './config.mjs';
 import { apiBase, warnIfInsecure } from './api-utils.mjs';
 import { buildTicketPayload } from './ticket-payload.mjs';
-import { red, green, yellow, bold, cyan } from './ansi.mjs';
+import { red, green, yellow, bold, cyan, dim } from './ansi.mjs';
 
 const PUSH_PATH = '/v1/triage/push';
 
@@ -109,12 +109,12 @@ export async function pushTriageSnapshot({
     }
 
     if (res.status === 401) {
-      print(`  ${red('✗')} Session expired. Run ${cyan('ticketlens login')} to reconnect.\n`);
+      print(`  ${red('✗')} Session expired.\n  ${dim('→ Run')} ${cyan('ticketlens login')} ${dim('to reconnect.')}\n\n`);
       return { ok: false, status: res.status };
     }
 
     if (res.status === 403) {
-      print(`  ${red('✗')} ${bold('--push')} requires a Team license\n`);
+      print(`  ${red('✗')} ${bold('--push')} requires a Team license.\n  ${dim('→ Log in with a team account:')} ${cyan('ticketlens login')}\n\n`);
       return { ok: false, status: res.status };
     }
 

package/skills/jtb/scripts/lib/triage-share.mjs

@@ -4,7 +4,7 @@
  */
 
 import { apiBase, warnIfInsecure } from './api-utils.mjs';
-import { red, green, yellow, bold, cyan } from './ansi.mjs';
+import { red, green, yellow, bold, cyan, dim } from './ansi.mjs';
 
 const SHARE_PATH = '/v1/triage/share';
 
@@ -77,12 +77,12 @@ export async function shareTriageSnapshot({
     }
 
     if (res.status === 401) {
-      print(`  ${red('✗')} Session expired. Run ${cyan('ticketlens login')} to reconnect.\n`);
+      print(`  ${red('✗')} Session expired — run ${cyan('ticketlens login')} to reconnect.\n\n`);
       return { ok: false, status: res.status };
     }
 
     if (res.status === 403) {
-      print(`  ${red('✗')} ${bold('--share')} requires a Team license\n`);
+      print(`  ${red('✗')} ${bold('--share')} requires a Team license.\n  ${dim('→ Log in with a team account:')} ${cyan('ticketlens login')}\n\n`);
       return { ok: false, status: res.status };
     }