ticketlens 0.37.0 → 0.38.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ticketlens",
-  "version": "0.37.0",
+  "version": "0.38.1",
   "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-my-tickets.mjs

@@ -281,6 +281,10 @@ export async function run(args, envOrOpts = process.env, fetcher = globalThis.fe
     return;
   }
 
+  // The real resolved profile name (ticket-prefix / cwd / config.default), not just the
+  // explicit --profile= arg — used for triage-history labeling regardless of how it resolved.
+  const resolvedProfileName = conn.profileName ?? 'default';
+
   const adapter = resolveAdapter(conn, { fetcher });
 
   // Status resolution: --status flag > profile triageStatuses > defaults
@@ -432,7 +436,7 @@ export async function run(args, envOrOpts = process.env, fetcher = globalThis.fe
   // Always save a daily snapshot for history tracking (non-fatal)
   try {
     const { saveTriageSnapshot } = await import('./lib/triage-history.mjs');
-    saveTriageSnapshot(scored, { profile: profileName ?? 'default', configDir });
+    saveTriageSnapshot(scored, { profile: resolvedProfileName, configDir });
   } catch { /* non-fatal */ }
 
   // Track this triage run (best-effort, non-fatal)
@@ -453,7 +457,7 @@ export async function run(args, envOrOpts = process.env, fetcher = globalThis.fe
     try {
       const { loadYesterdaySnapshot, diffSnapshots, buildDeltaSection } =
         await import('./lib/triage-history.mjs');
-      const yesterday = loadYesterdaySnapshot({ profile: profileName ?? 'default', configDir });
+      const yesterday = loadYesterdaySnapshot({ profile: resolvedProfileName, configDir });
       if (yesterday) {
         const deltas = diffSnapshots(sorted, yesterday.tickets);
         delta = buildDeltaSection(deltas) || null;
@@ -461,7 +465,7 @@ export async function run(args, envOrOpts = process.env, fetcher = globalThis.fe
     } catch { /* non-fatal — digest still sends */ }
 
     await deliverer({
-      profile: profileName ?? 'default',
+      profile: resolvedProfileName,
       staleDays,
       summary: (() => {
         let needsResponse = 0, aging = 0;
@@ -481,7 +485,7 @@ export async function run(args, envOrOpts = process.env, fetcher = globalThis.fe
   if (exportArg) {
     const { exportTriage } = await import('./lib/triage-exporter.mjs');
     const exporterFn = opts.exporter ?? exportTriage;
-    const outputPath = await Promise.resolve(exporterFn({ tickets: sorted, format: exportArg, profile: profileName ?? 'default', configDir }));
+    const outputPath = await Promise.resolve(exporterFn({ tickets: sorted, format: exportArg, profile: resolvedProfileName, configDir }));
     const printFn = opts.print ?? ((msg) => process.stdout.write(msg + '\n'));
     printFn(`Export written to ${outputPath}`);
     return;
@@ -503,7 +507,7 @@ export async function run(args, envOrOpts = process.env, fetcher = globalThis.fe
     await pushFn({
       sorted: pushable,
       rawTicketMap,
-      profile: profileName ?? 'default',
+      profile: resolvedProfileName,
       baseUrl: conn.baseUrl,
       cliToken,
       gitBranches: scanFn(),
@@ -521,7 +525,7 @@ export async function run(args, envOrOpts = process.env, fetcher = globalThis.fe
     await shareFn({
       sorted,
       rawTicketMap,
-      profile: profileName ?? 'default',
+      profile: resolvedProfileName,
       baseUrl: conn.baseUrl,
       cliToken,
       fetcher,
@@ -564,7 +568,7 @@ export async function run(args, envOrOpts = process.env, fetcher = globalThis.fe
       metrics = metricsInjector();
     } else {
       const { computeResponseMetrics } = await import('./lib/triage-history.mjs');
-      metrics = computeResponseMetrics(profileName ?? 'default', { days: 7, configDir });
+      metrics = computeResponseMetrics(resolvedProfileName, { days: 7, configDir });
     }
     // Only show footer when there's a meaningful metric to display
     const hasAvg = metrics && metrics.avgResponseHours !== null;

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);
   }