ticketlens 0.38.52 → 0.38.54

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ticketlens",
-  "version": "0.38.52",
+  "version": "0.38.54",
   "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/scripts/fetch-ticket.mjs

@@ -21,7 +21,7 @@ import { promptSelect } from './lib/select-prompt.mjs';
 import { printFetchHelp } from './lib/help.mjs';
 import { readTextAttachments } from './lib/handoff-assembler.mjs';
 import { handleUnknownFlags } from './lib/arg-validator.mjs';
-import { TICKET_KEY_PATTERN } from './lib/cli.mjs';
+import { TICKET_KEY_PATTERN, normalizeTicketKey } 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';
@@ -552,17 +552,19 @@ export async function run(args, envOrOpts = process.env, fetcher = globalThis.fe
 
   if (args[0] === 'pr') {
     const { assemblePr } = await import('./lib/pr-assembler.mjs');
-    const ticketKeyArg = args[1];
+    let ticketKeyArg = args[1];
     if (!ticketKeyArg) {
       printErrFn('Error: "pr" requires a ticket key. Usage: ticketlens pr PROJ-123\n');
       process.exitCode = 1;
       return;
     }
-    if (!TICKET_KEY_PATTERN.test(ticketKeyArg)) {
+    const normalizedPr = normalizeTicketKey(ticketKeyArg);
+    if (!TICKET_KEY_PATTERN.test(normalizedPr)) {
       printErrFn(`Error: "${ticketKeyArg}" is not a valid ticket key. Expected format: PROJ-123\n`);
       process.exitCode = 1;
       return;
     }
+    ticketKeyArg = normalizeTicketKey(ticketKeyArg, { stream: errStream });
     const resolvedConfigDir = configDir ?? (await import('./lib/config.mjs')).DEFAULT_CONFIG_DIR;
 
     // Resolve connection (same pattern as compliance dispatch)
@@ -627,17 +629,19 @@ export async function run(args, envOrOpts = process.env, fetcher = globalThis.fe
   }
 
   if (args[0] === 'compliance') {
-    const ticketKeyArg = args[1];
+    let ticketKeyArg = args[1];
     if (!ticketKeyArg) {
       printErrFn('Error: "compliance" requires a ticket key. Usage: ticketlens compliance PROJ-123\n');
       process.exitCode = 1;
       return;
     }
-    if (!TICKET_KEY_PATTERN.test(ticketKeyArg)) {
+    const normalizedCompliance = normalizeTicketKey(ticketKeyArg);
+    if (!TICKET_KEY_PATTERN.test(normalizedCompliance)) {
       printErrFn(`Error: "${ticketKeyArg}" is not a valid ticket key. Expected format: PROJ-123\n`);
       process.exitCode = 1;
       return;
     }
+    ticketKeyArg = normalizeTicketKey(ticketKeyArg, { stream: errStream });
 
     const resolvedConfigDir = configDir ?? (await import('./lib/config.mjs')).DEFAULT_CONFIG_DIR;
 
@@ -1005,17 +1009,19 @@ export async function run(args, envOrOpts = process.env, fetcher = globalThis.fe
     return;
   }
 
-  const ticketKey = args.find(a => !a.startsWith('--'));
+  let ticketKey = args.find(a => !a.startsWith('--'));
   if (!ticketKey) {
     printFetchHelp({ stream: errStream });
     process.exitCode = 1;
     return;
   }
-  if (!TICKET_KEY_PATTERN.test(ticketKey)) {
+  const normalizedFetch = normalizeTicketKey(ticketKey);
+  if (!TICKET_KEY_PATTERN.test(normalizedFetch)) {
     printErrFn(`Error: "${ticketKey}" is not a valid ticket key. Expected format: PROJ-123\n`);
     process.exitCode = 1;
     return;
   }
+  ticketKey = normalizeTicketKey(ticketKey, { stream: errStream });
 
   // Normalize --project= alias once at entry so all recursive calls only see --profile=
   const projectArg = args.find(a => a.startsWith('--project='));

package/skills/jtb/scripts/lib/cli.mjs

@@ -5,6 +5,20 @@
 
 export const TICKET_KEY_PATTERN = /^[A-Z][A-Z0-9]+-\d+$/;
 
+/**
+ * Jira/Linear project and team keys are always created uppercase, so any
+ * lowercase input is a typo, never a distinct real key — safe to correct
+ * silently before TICKET_KEY_PATTERN ever sees it.
+ */
+export function normalizeTicketKey(ticketKey, { stream } = {}) {
+  if (!ticketKey) return ticketKey;
+  const upper = ticketKey.toUpperCase();
+  if (upper !== ticketKey && stream) {
+    stream.write(`  Note: normalized "${ticketKey}" to "${upper}"\n`);
+  }
+  return upper;
+}
+
 export function parseCommand(args) {
   const first = args[0];
 

package/skills/jtb/scripts/lib/license.mjs

@@ -130,17 +130,23 @@ export function showUpgradePrompt(requiredTier, featureFlag, { stream = process.
 const LEMONSQUEEZY_ACTIVATE_URL = 'https://api.lemonsqueezy.com/v1/licenses/activate';
 const LEMONSQUEEZY_VALIDATE_URL = 'https://api.lemonsqueezy.com/v1/licenses/validate';
 
-function resolveLicenseUrl(action, fallback) {
+// TL- keys are issued by TicketLens itself (LicenseIssuanceService::KEY_PREFIX), never by
+// LemonSqueezy — sending one to the LemonSqueezy fallback always fails, confusingly (backlog #30).
+const TL_ISSUED_KEY_RE = /^TL-/i;
+
+function resolveLicenseUrl(action, fallback, key) {
   const base = process.env.TICKETLENS_API_URL;
-  return base ? `${base.replace(/\/$/, '')}/v1/licenses/${action}` : fallback;
+  if (base) return `${base.replace(/\/$/, '')}/v1/licenses/${action}`;
+  if (TL_ISSUED_KEY_RE.test(key)) return null;
+  return fallback;
 }
 
-function resolveActivateUrl() {
-  return resolveLicenseUrl('activate', LEMONSQUEEZY_ACTIVATE_URL);
+function resolveActivateUrl(key) {
+  return resolveLicenseUrl('activate', LEMONSQUEEZY_ACTIVATE_URL, key);
 }
 
-function resolveValidateUrl() {
-  return resolveLicenseUrl('validate', LEMONSQUEEZY_VALIDATE_URL);
+function resolveValidateUrl(key) {
+  return resolveLicenseUrl('validate', LEMONSQUEEZY_VALIDATE_URL, key);
 }
 
 function extractTier(meta) {
@@ -154,8 +160,16 @@ export async function activateLicense(key, opts = {}) {
   const { configDir = DEFAULT_CONFIG_DIR, fetcher = globalThis.fetch, instanceName } = opts;
   const instance = instanceName || os.hostname();
 
+  const url = resolveActivateUrl(key);
+  if (url === null) {
+    return {
+      success: false,
+      error: 'TICKETLENS_API_URL is not set — required to activate a TicketLens-issued (TL-) license key. Set it to your TicketLens backend URL and try again.',
+    };
+  }
+
   try {
-    const res = await fetcher(resolveActivateUrl(), {
+    const res = await fetcher(url, {
       method: 'POST',
       headers: { 'Content-Type': 'application/json', Accept: 'application/json' },
       body: JSON.stringify({ license_key: key, instance_name: instance }),
@@ -194,8 +208,13 @@ export async function revalidateLicense(opts = {}) {
 
   const instance = instanceName || os.hostname();
 
+  const url = resolveValidateUrl(license.key);
+  if (url === null) {
+    return { success: true, tier: license.tier, cached: true };
+  }
+
   try {
-    const res = await fetcher(resolveValidateUrl(), {
+    const res = await fetcher(url, {
       method: 'POST',
       headers: { 'Content-Type': 'application/json', Accept: 'application/json' },
       body: JSON.stringify({ license_key: license.key, instance_name: instance }),

package/skills/jtb/scripts/lib/note-command.mjs

@@ -19,7 +19,7 @@ import { enqueueNote, isRetryableFailure, maybeAutoFlush } from './recall-queue.
 import { incrementDraftKept, incrementDraftDeleted } from './activity-counter.mjs';
 import { extractText } from './attachment-text.mjs';
 import { readAttachments, MAX_ATTACHMENTS } from './attachment-uploader.mjs';
-import { TICKET_KEY_PATTERN } from './cli.mjs';
+import { TICKET_KEY_PATTERN, normalizeTicketKey } from './cli.mjs';
 import { createStyler } from './ansi.mjs';
 import { confirmDestructive } from './confirm.mjs';
 
@@ -135,10 +135,15 @@ export async function runNoteAdd(cmdArgs, {
   // to forge a fake "## heading" line when the note is later injected into a brief.
   const title = rawTitle.replace(/[\r\n]+/g, ' ');
 
-  const ticketKey = parseFlag(cmdArgs, 'ticket');
-  if (ticketKey && !TICKET_KEY_PATTERN.test(ticketKey)) {
-    stream.write(`  Invalid --ticket value "${ticketKey}" — expected a ticket key like PROJ-123.\n`);
-    return { written: false };
+  let ticketKey = parseFlag(cmdArgs, 'ticket');
+  if (ticketKey) {
+    const normalized = normalizeTicketKey(ticketKey);
+    if (!TICKET_KEY_PATTERN.test(normalized)) {
+      stream.write(`  Invalid --ticket value "${ticketKey}" — expected a ticket key like PROJ-123.\n`);
+      return { written: false };
+    }
+    if (normalized !== ticketKey) normalizeTicketKey(ticketKey, { stream });
+    ticketKey = normalized;
   }
   const tagsArg = parseFlag(cmdArgs, 'tags');
   const tags = tagsArg ? tagsArg.split(',').map(t => t.trim()).filter(Boolean) : [];
@@ -260,10 +265,15 @@ export async function runNotePatch(cmdArgs, {
     return { patched: false };
   }
 
-  const ticketKey = parseFlag(cmdArgs, 'ticket');
-  if (ticketKey && !TICKET_KEY_PATTERN.test(ticketKey)) {
-    stream.write(`  Invalid --ticket value "${ticketKey}" — expected a ticket key like PROJ-123.\n`);
-    return { patched: false };
+  let ticketKey = parseFlag(cmdArgs, 'ticket');
+  if (ticketKey) {
+    const normalized = normalizeTicketKey(ticketKey);
+    if (!TICKET_KEY_PATTERN.test(normalized)) {
+      stream.write(`  Invalid --ticket value "${ticketKey}" — expected a ticket key like PROJ-123.\n`);
+      return { patched: false };
+    }
+    if (normalized !== ticketKey) normalizeTicketKey(ticketKey, { stream });
+    ticketKey = normalized;
   }
 
   const body = await readStdin();
@@ -339,10 +349,15 @@ export async function runNoteDelete(cmdArgs, {
     return { deleted: false };
   }
 
-  const ticketKey = parseFlag(cmdArgs, 'ticket');
-  if (ticketKey && !TICKET_KEY_PATTERN.test(ticketKey)) {
-    stream.write(`  Invalid --ticket value "${ticketKey}" — expected a ticket key like PROJ-123.\n`);
-    return { deleted: false };
+  let ticketKey = parseFlag(cmdArgs, 'ticket');
+  if (ticketKey) {
+    const normalized = normalizeTicketKey(ticketKey);
+    if (!TICKET_KEY_PATTERN.test(normalized)) {
+      stream.write(`  Invalid --ticket value "${ticketKey}" — expected a ticket key like PROJ-123.\n`);
+      return { deleted: false };
+    }
+    if (normalized !== ticketKey) normalizeTicketKey(ticketKey, { stream });
+    ticketKey = normalized;
   }
 
   const forceYes = cmdArgs.includes('--yes') || cmdArgs.includes('-y');

package/skills/jtb/scripts/lib/recall-command.mjs

@@ -11,7 +11,7 @@
  */
 
 import { DEFAULT_CONFIG_DIR } from './config.mjs';
-import { TICKET_KEY_PATTERN } from './cli.mjs';
+import { TICKET_KEY_PATTERN, normalizeTicketKey } from './cli.mjs';
 import { isLicensed, showUpgradePrompt } from './license.mjs';
 import { listNotes } from './recall-vault.mjs';
 import { readCliToken } from './cli-auth.mjs';
@@ -48,17 +48,23 @@ export async function runRecall(cmdArgs, {
     return { ok: false };
   }
 
-  const isTicketKey = TICKET_KEY_PATTERN.test(arg);
+  // A lowercase-typed ticket key (e.g. "proj-123") must not be mistaken for a
+  // free-text search phrase — normalize before classifying, but never touch
+  // a genuine search query's own casing.
+  const normalizedArg = normalizeTicketKey(arg);
+  const isTicketKey = TICKET_KEY_PATTERN.test(normalizedArg);
+  if (isTicketKey && normalizedArg !== arg) normalizeTicketKey(arg, { stream: errorStream });
+  const key = isTicketKey ? normalizedArg : arg;
 
   const cliToken = readCliTokenFn(configDir);
   if (cliToken) {
-    const profile = resolveProfileFn(isTicketKey ? arg : null, { configDir, cwd: process.cwd() });
+    const profile = resolveProfileFn(isTicketKey ? key : null, { configDir, cwd: process.cwd() });
     const recallTeamId = profile ? loadProfileRecallTeamIdFn(profile.name, configDir) : null;
     await pullNotesFn({ cliToken, configDir, ttlMs: 0, groupId: recallTeamId ?? undefined });
     await maybeAutoFlushFn({ cliToken, configDir });
   }
 
-  const filter = isTicketKey ? { ticketKey: arg } : { query: arg };
+  const filter = isTicketKey ? { ticketKey: key } : { query: key };
   const results = listNotesFn(filter, { configDir });
 
   const styled = !cmdArgs.includes('--plain') && !!stream.isTTY;

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

@@ -2,6 +2,7 @@ import { queryTicketHistory, DEFAULT_CONFIG_DIR } from './triage-history.mjs';
 import { isLicensed as defaultIsLicensed, showUpgradePrompt } from './license.mjs';
 import { createStyler } from './ansi.mjs';
 import { printHistoryHelp } from './help.mjs';
+import { normalizeTicketKey } from './cli.mjs';
 
 /**
  * Prints a ticket's local triage-history timeline. Extracted from
@@ -26,12 +27,13 @@ export async function runHistory(args = [], opts = {}) {
     return;
   }
 
-  const ticketKey = args[0];
+  let ticketKey = args[0];
   if (!ticketKey || ticketKey.startsWith('-')) {
     warn('Usage: ticketlens history TICKET-KEY\n');
     process.exitCode = 1;
     return;
   }
+  ticketKey = normalizeTicketKey(ticketKey, { stream: { write: warn } });
 
   const entries = queryFn(ticketKey, { configDir });
   if (entries.length === 0) {

package/skills/jtb/scripts/lib/ticket-command.mjs

@@ -18,7 +18,7 @@ import { checkCooldown, recordAction } from './ticket-action-cooldown.mjs';
 import { logAction } from './ticket-action-log.mjs';
 import { readMetadataCache, writeMetadataCache } from './ticket-metadata-cache.mjs';
 import { detectProjectOrTypeError, enrichCreateFailure } from './ticket-create-enrichment.mjs';
-import { TICKET_KEY_PATTERN } from './cli.mjs';
+import { TICKET_KEY_PATTERN, normalizeTicketKey } from './cli.mjs';
 import { scoreCandidates } from './duplicate-scorer.mjs';
 import { MAX_ATTACHMENTS } from './attachment-uploader.mjs';
 import { createStyler } from './ansi.mjs';
@@ -209,11 +209,17 @@ function requireLicense(isLicensedFn, configDir, commandName, stream) {
 
 function requireTicketKey(cmdArgs, usage, stream) {
   const ticketKey = cmdArgs[0];
-  if (!ticketKey || ticketKey.startsWith('--') || !TICKET_KEY_PATTERN.test(ticketKey)) {
+  if (!ticketKey || ticketKey.startsWith('--')) {
     stream.write(usage);
     return null;
   }
-  return ticketKey;
+  const normalized = normalizeTicketKey(ticketKey);
+  if (!TICKET_KEY_PATTERN.test(normalized)) {
+    stream.write(usage);
+    return null;
+  }
+  if (normalized !== ticketKey) normalizeTicketKey(ticketKey, { stream });
+  return normalized;
 }
 
 /**