ticketlens 0.36.0 → 0.38.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ticketlens",
-  "version": "0.36.0",
+  "version": "0.38.0",
   "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/lib/compliance-checker.mjs

@@ -6,15 +6,31 @@ import { findLinkedCommits } from './commit-linker.mjs';
 import { analyzeDiff } from './diff-analyzer.mjs';
 import { DEFAULT_CONFIG_DIR } from './config.mjs';
 import { appendLedger } from './ledger.mjs';
+import { createStyler } from './ansi.mjs';
 
 export const STATUS_ICON = { FOUND: '✔', PARTIAL: '~', NOT_FOUND: '✖' };
 
-function formatReport({ ticketKey, requirements, analysis, usage, isPro }) {
+export function statusColor(status, s) {
+  if (status === 'FOUND') return s.green;
+  if (status === 'PARTIAL') return s.yellow;
+  return s.dim;
+}
+
+// Shared with matchColor in ticket-command.mjs (duplicates' match-confidence
+// tiers) — same 70/50 thresholds, same green/yellow/dim vocabulary, applied
+// here to overall requirement coverage instead of a single match score.
+function coverageColor(pct, s) {
+  if (pct >= 70) return s.green;
+  if (pct >= 50) return s.yellow;
+  return s.dim;
+}
+
+function formatReport({ ticketKey, requirements, analysis, usage, isPro, s }) {
   const { results, coveragePercent } = analysis;
   const lines = [
     '',
-    `  Compliance Check — ${ticketKey}`,
-    `  ${'─'.repeat(50)}`,
+    `  Compliance Check — ${s.brand(s.bold(ticketKey))}`,
+    `  ${s.dim('─'.repeat(50))}`,
     '',
   ];
 
@@ -26,14 +42,15 @@ function formatReport({ ticketKey, requirements, analysis, usage, isPro }) {
   }
 
   for (const { requirement, status, evidence } of results) {
-    const icon = STATUS_ICON[status] ?? '?';
+    const icon = statusColor(status, s)(STATUS_ICON[status] ?? '?');
     lines.push(`  ${icon} ${requirement}`);
-    if (evidence) lines.push(`      └─ ${evidence}`);
+    if (evidence) lines.push(`      └─ ${s.dim(evidence)}`);
     lines.push('');
   }
 
   lines.push('');
-  lines.push(`  Coverage: ${coveragePercent}%  (${results.filter(r => r.status === 'FOUND').length}/${results.length} requirements found)`);
+  const found = results.filter(r => r.status === 'FOUND').length;
+  lines.push(`  Coverage: ${coverageColor(coveragePercent, s)(`${coveragePercent}%`)}  (${found}/${results.length} requirements found)`);
   lines.push('');
 
   if (!isPro) {
@@ -52,6 +69,12 @@ export async function runComplianceCheck({
   ticketKey,
   configDir = DEFAULT_CONFIG_DIR,
   stream = process.stderr,
+  // Separate from `stream` (stderr, only used for the Pro-upgrade nudge
+  // above) — the report string itself is printed to stdout by the caller
+  // (fetch-ticket.mjs), so styling must gate on stdout's TTY-ness, not
+  // stderr's, or ANSI codes would leak into `tl compliance X > out.txt`
+  // while stderr stays attached to a real terminal.
+  outStream = process.stdout,
   isLicensedFn       = isLicensed,
   showUpgradeFn      = showUpgradePrompt,
   checkUsageFn       = checkUsage,
@@ -78,7 +101,8 @@ export async function runComplianceCheck({
   const { diff } = findLinkedCommitsFn(ticketKey, { cwd: process.cwd() });
   const analysis = analyzeDiffFn(requirements, diff);
 
-  const report = formatReport({ ticketKey, requirements, analysis, usage, isPro });
+  const s = createStyler({ isTTY: outStream.isTTY });
+  const report = formatReport({ ticketKey, requirements, analysis, usage, isPro, s });
   const coveragePercent = analysis.coveragePercent;
   const missing = analysis.results
     .filter(r => r.status === 'NOT_FOUND')

package/skills/jtb/scripts/lib/pr-assembler.mjs

@@ -8,8 +8,9 @@ import { spawnSync } from 'node:child_process';
 import { fetchTicket } from './jira-client.mjs';
 import { extractRequirements } from './requirement-extractor.mjs';
 import { findLinkedCommits } from './commit-linker.mjs';
-import { runComplianceCheck, STATUS_ICON } from './compliance-checker.mjs';
+import { runComplianceCheck, STATUS_ICON, statusColor } from './compliance-checker.mjs';
 import { DEFAULT_CONFIG_DIR } from './config.mjs';
+import { createStyler } from './ansi.mjs';
 
 /**
  * Detect the git remote URL using execFn.
@@ -34,15 +35,16 @@ function detectRemoteUrl(execFn) {
  * Build the "### Linked tickets" section from a ticket's linkedIssues.
  *
  * @param {Array} linkedIssues - array of { key, summary, linkType, direction }
+ * @param {object} s - styler from createStyler()
  * @returns {string} section markdown or empty string
  */
-function buildLinkedTicketsSection(linkedIssues) {
+function buildLinkedTicketsSection(linkedIssues, s) {
   if (!linkedIssues || linkedIssues.length === 0) return '';
 
   const lines = ['', '### Linked tickets'];
   for (const issue of linkedIssues) {
     const rel = issue.linkType ?? issue.direction ?? 'Linked';
-    lines.push(`- ${issue.key}: ${rel} — ${issue.summary}`);
+    lines.push(`- ${s.brand(s.bold(issue.key))}: ${rel} — ${issue.summary}`);
   }
   return lines.join('\n');
 }
@@ -52,9 +54,10 @@ function buildLinkedTicketsSection(linkedIssues) {
  *
  * @param {object|null} complianceResult - result from runComplianceCheckFn or null
  * @param {string[]} requirements - raw requirements list
+ * @param {object} s - styler from createStyler()
  * @returns {string} section markdown
  */
-function buildCoverageSection(complianceResult, requirements) {
+function buildCoverageSection(complianceResult, requirements, s) {
   if (complianceResult === null) {
     const lines = ['', '### Requirements coverage (coverage unavailable — Pro required)'];
     for (const req of requirements) {
@@ -67,7 +70,7 @@ function buildCoverageSection(complianceResult, requirements) {
   const lines = [``, `### Requirements coverage (${coveragePercent}%)`];
 
   for (const { requirement, status, evidence } of results) {
-    const icon = STATUS_ICON[status] ?? '?';
+    const icon = statusColor(status, s)(STATUS_ICON[status] ?? '?');
     const suffix = evidence ? ` (${evidence})` : '';
     lines.push(`- ${icon} ${requirement}${suffix}`);
   }
@@ -86,7 +89,8 @@ function buildCoverageSection(complianceResult, requirements) {
  * @param {Function} [opts.findLinkedCommitsFn] - async fn(ticketKey, opts) → commit[]
  * @param {Function} [opts.runComplianceCheckFn] - async fn({ brief, ticketKey, configDir }) → result|null
  * @param {Function} [opts.execFn] - spawnSync-compatible for git remote detection
- * @param {object} [opts.stream] - stderr for progress output (optional)
+ * @param {object} [opts.outStream] - stdout — where this markdown is ultimately printed
+ *   (`tl pr KEY | pbcopy` pipes stdout), so styling gates on its TTY-ness, not stderr's
  * @returns {Promise<string>} markdown PR description
  */
 export async function assemblePr(ticketKey, {
@@ -97,8 +101,9 @@ export async function assemblePr(ticketKey, {
   findLinkedCommitsFn = findLinkedCommits,
   runComplianceCheckFn = runComplianceCheck,
   execFn = spawnSync,
-  stream,
+  outStream = process.stdout,
 } = {}) {
+  const s = createStyler({ isTTY: outStream.isTTY });
   // Fetch ticket data
   const ticket = await fetchTicketFn(ticketKey, { configDir });
 
@@ -130,7 +135,7 @@ export async function assemblePr(ticketKey, {
   );
 
   // Build output
-  const lines = [`## ${ticketKey}: ${summary}`, ''];
+  const lines = [`## ${s.brand(s.bold(ticketKey))}: ${summary}`, ''];
 
   // What changed — commits are raw git --oneline strings: "<sha> <message>"
   lines.push('### What changed');
@@ -143,7 +148,7 @@ export async function assemblePr(ticketKey, {
   }
 
   // Requirements coverage
-  lines.push(buildCoverageSection(complianceResult, requirements));
+  lines.push(buildCoverageSection(complianceResult, requirements, s));
 
   // Acceptance criteria
   lines.push('');
@@ -157,7 +162,7 @@ export async function assemblePr(ticketKey, {
   }
 
   // Linked tickets (optional section)
-  const linkedSection = buildLinkedTicketsSection(linkedIssues);
+  const linkedSection = buildLinkedTicketsSection(linkedIssues, s);
   if (linkedSection) {
     lines.push(linkedSection);
   }

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

@@ -45,10 +45,10 @@ function refuseGithubAttachments(adapter, attachPaths, stream) {
   return true;
 }
 
-function formatAttachSummary(attachResult) {
+function formatAttachSummary(attachResult, s) {
   if (!attachResult) return '';
   const lines = [];
-  for (const u of attachResult.uploaded) lines.push(`  Attached ${u.filename}${u.url ? ` (${u.url})` : ''}\n`);
+  for (const u of attachResult.uploaded) lines.push(`  ${s.green('✔')} Attached ${s.bold(u.filename)}${u.url ? ` (${u.url})` : ''}\n`);
   for (const e of attachResult.errors) lines.push(`  Failed to attach ${e.path}: ${e.message}\n`);
   if (attachResult.droppedCount > 0) lines.push(`  ${attachResult.droppedCount} attachment(s) dropped — exceeds the ${MAX_ATTACHMENTS}-file limit per call.\n`);
   return lines.join('');
@@ -190,11 +190,14 @@ function describeAppliedFields(applied) {
   return parts.join(', ');
 }
 
-function formatUpdateResult(ticketKey, { applied, errors }) {
+function formatUpdateResult(ticketKey, { applied, errors }, s) {
   const appliedText = describeAppliedFields(applied);
   const errorText = Object.entries(errors).map(([field, info]) => formatFieldError(field, info)).join('; ');
-  if (appliedText && !errorText) return `  ${ticketKey} updated: ${appliedText}.\n`;
-  if (appliedText && errorText) return `  ${ticketKey} partially updated: ${appliedText}. Failed: ${errorText}.\n`;
+  // Success/partial get the styled brand+bold key; total failure stays plain,
+  // consistent with every other refusal/error message in this file (e.g. the
+  // GitHub-priority-refusal line just above in runTicketUpdate).
+  if (appliedText && !errorText) return `  ${s.green('✔')} ${s.brand(s.bold(ticketKey))} updated: ${s.bold(appliedText)}.\n`;
+  if (appliedText && errorText) return `  ${s.yellow('~')} ${s.brand(s.bold(ticketKey))} partially updated: ${s.bold(appliedText)}. Failed: ${errorText}.\n`;
   return `  Nothing updated on ${ticketKey}. Failed: ${errorText}.\n`;
 }
 
@@ -267,6 +270,7 @@ export async function runTicketComment(cmdArgs, {
 
   const adapter = resolveTicketAdapter(ticketKey, cmdArgs, { configDir, resolveConnectionFn, resolveAdapterFn, stream });
   if (!adapter) return { ok: false };
+  const s = createStyler({ isTTY: stream.isTTY });
 
   // Uploaded BEFORE the comment write so a tracker capable of inline
   // rendering (Jira Server/DC via wiki markup, Jira Cloud via a real ADF
@@ -287,13 +291,13 @@ export async function runTicketComment(cmdArgs, {
     // actually landed) — a partial attach failure is reconstructable from
     // the difference between the two, not just silently absent from audit.
     logActionFn({ ticketKey, action: 'comment', actor, tracker: adapter.type, detail: { id: result.id, attachPaths, attachedFilenames: (attachResult?.uploaded ?? []).map(a => a.filename) } }, { configDir });
-    stream.write(`  Comment posted to ${ticketKey}${result.url ? ` (${result.url})` : ''}\n` + formatAttachSummary(attachResult));
+    stream.write(`  ${s.green('✔')} Comment posted to ${s.brand(s.bold(ticketKey))}${result.url ? ` (${result.url})` : ''}\n` + formatAttachSummary(attachResult, s));
     return { ok: true };
   } catch (err) {
     // Attachments (if any) genuinely landed on the tracker before this
     // write was attempted — formatAttachSummary is still shown here so a
     // caller retrying the whole command doesn't blindly re-upload them.
-    stream.write(formatWriteFailure(ticketKey, err) + formatAttachSummary(attachResult));
+    stream.write(formatWriteFailure(ticketKey, err) + formatAttachSummary(attachResult, s));
     return { ok: false };
   }
 }
@@ -323,13 +327,14 @@ export async function runTicketTransitionList(cmdArgs, {
 
   try {
     const options = await adapter.getTransitions(ticketKey);
+    const s = createStyler({ isTTY: stream.isTTY });
     if (options.length === 0) {
-      stream.write(`  No valid transitions available for ${ticketKey}.\n`);
+      stream.write(`  No valid transitions available for ${s.brand(s.bold(ticketKey))}.\n`);
       return { ok: true, options: [] };
     }
-    stream.write(`  Valid transitions for ${ticketKey}:\n`);
-    for (const o of options) stream.write(`    - ${o.name}\n`);
-    stream.write(`  Run again with --target="<name>" --confirm to execute.\n`);
+    stream.write(`  Valid transitions for ${s.brand(s.bold(ticketKey))}:\n\n`);
+    for (const o of options) stream.write(`    ${s.brand('●')} ${o.name}\n`);
+    stream.write(`\n  Run again with --target="<name>" --confirm to execute.\n`);
     return { ok: true, options };
   } catch (err) {
     stream.write(formatWriteFailure(ticketKey, err));
@@ -389,7 +394,8 @@ export async function runTicketTransition(cmdArgs, {
     }
     recordActionFn(ticketKey, 'transition', { configDir });
     logActionFn({ ticketKey, action: 'transition', actor, tracker: adapter.type, detail: { to: result.to } }, { configDir });
-    stream.write(`  ${ticketKey} transitioned to "${result.to}".\n`);
+    const s = createStyler({ isTTY: stream.isTTY });
+    stream.write(`  ${s.green('✔')} ${s.brand(s.bold(ticketKey))} transitioned to ${s.bold(`"${result.to}"`)}.\n`);
     return { ok: true };
   } catch (err) {
     stream.write(formatWriteFailure(ticketKey, err));
@@ -443,7 +449,8 @@ export async function runTicketAssign(cmdArgs, {
     const result = await adapter.assignToSelf(ticketKey);
     recordActionFn(ticketKey, 'assign', { configDir });
     logActionFn({ ticketKey, action: 'assign', actor, tracker: adapter.type, detail: { assignee: result.assignee } }, { configDir });
-    stream.write(`  ${ticketKey} assigned to ${result.assignee}.\n`);
+    const s = createStyler({ isTTY: stream.isTTY });
+    stream.write(`  ${s.green('✔')} ${s.brand(s.bold(ticketKey))} assigned to ${s.bold(result.assignee)}.\n`);
     return { ok: true };
   } catch (err) {
     stream.write(formatWriteFailure(ticketKey, err));
@@ -563,12 +570,14 @@ export async function runTicketLinkList(cmdArgs, {
 
   try {
     const types = await adapter.getLinkTypes();
+    const s = createStyler({ isTTY: stream.isTTY });
     if (types.length === 0) {
-      stream.write(`  No link types available for ${sourceKey} → ${targetKey} on ${adapter.type}.\n`);
+      stream.write(`  No link types available for ${s.brand(s.bold(sourceKey))} → ${s.brand(s.bold(targetKey))} on ${adapter.type}.\n`);
       return { ok: true, types: [] };
     }
-    stream.write(`  Available link types for ${sourceKey} → ${targetKey} (${adapter.type}):\n`);
-    for (const t of types) stream.write(`    - ${t}\n`);
+    stream.write(`  Available link types for ${s.brand(s.bold(sourceKey))} → ${s.brand(s.bold(targetKey))} (${adapter.type}):\n\n`);
+    for (const t of types) stream.write(`    ${s.brand('●')} ${t}\n`);
+    stream.write('\n');
     if (adapter.type === 'github') {
       stream.write(`  Note: GitHub has no generic link relationship — linking will CLOSE ${sourceKey} as a duplicate of ${targetKey}.\n`);
     }
@@ -647,10 +656,11 @@ export async function runTicketLink(cmdArgs, {
     }
     recordActionFn(cooldownKey, 'link', { configDir });
     logActionFn({ ticketKey: sourceKey, action: 'link', actor, tracker: adapter.type, detail: { targetKey, type } }, { configDir });
+    const s = createStyler({ isTTY: stream.isTTY });
     stream.write(
       adapter.type === 'github'
-        ? `  ${sourceKey} closed as a duplicate of ${targetKey}.\n`
-        : `  ${sourceKey} linked to ${targetKey} as "${type}".\n`,
+        ? `  ${s.green('✔')} ${s.brand(s.bold(sourceKey))} closed as a duplicate of ${s.brand(s.bold(targetKey))}.\n`
+        : `  ${s.green('✔')} ${s.brand(s.bold(sourceKey))} linked to ${s.brand(s.bold(targetKey))} as ${s.bold(`"${type}"`)}.\n`,
     );
     return { ok: true };
   } catch (err) {
@@ -733,7 +743,7 @@ export async function runTicketUpdate(cmdArgs, {
       recordActionFn(ticketKey, 'update', { configDir });
       logActionFn({ ticketKey, action: 'update', actor, tracker: adapter.type, detail: { ...result.applied, failed: Object.keys(result.errors) } }, { configDir });
     }
-    stream.write(formatUpdateResult(ticketKey, result));
+    stream.write(formatUpdateResult(ticketKey, result, createStyler({ isTTY: stream.isTTY })));
     return hasErrors ? { ok: false, applied: result.applied, errors: result.errors } : { ok: true, applied: result.applied };
   } catch (err) {
     stream.write(formatWriteFailure(ticketKey, err));
@@ -794,6 +804,7 @@ export async function runTicketCreate(cmdArgs, {
 
   const adapter = resolveTicketAdapter(undefined, cmdArgs, { configDir, resolveConnectionFn, resolveAdapterFn, stream });
   if (!adapter) return { ok: false };
+  const s = createStyler({ isTTY: stream.isTTY });
   const attachRefused = refuseGithubAttachments(adapter, attachPaths, stream);
 
   if (adapter.type !== 'github' && !project) {
@@ -883,6 +894,6 @@ export async function runTicketCreate(cmdArgs, {
     }
   }
 
-  stream.write(`  Created ${result.key}${result.url ? ` (${result.url})` : ''}\n` + formatAttachSummary(attachResult));
+  stream.write(`  ${s.green('✔')} Created ${s.brand(s.bold(result.key))}${result.url ? ` (${result.url})` : ''}\n` + formatAttachSummary(attachResult, s));
   return { ok: true, key: result.key };
 }