ticketlens 0.1.14 → 0.1.15

package/README.md

@@ -35,6 +35,7 @@ Zero npm dependencies. Node.js built-ins only.
 - **Scriptable** — standard CLI output: pipe to cron, git hooks, CI/CD, or any LLM tool
 - **Multi-profile** — connect multiple Jira instances simultaneously; auto-route by ticket prefix or project path
 - **Attachments included** — images, PDFs, and text files downloaded locally; Claude Code reads them as context
+- **Confluence pages** — linked Confluence pages fetched and included in the brief automatically (Jira only)
 
 ---
 
@@ -93,7 +94,7 @@ ticketlens CNV1-2 --depth=2        # + linked-of-linked (full graph)
 ticketlens CNV1-2 --plain          # Plain markdown — pipe-safe, LLM-ready
 ticketlens CNV1-2 --profile=acme   # Force a specific profile
 ticketlens CNV1-2 --no-cache       # Bypass cache, re-fetch from Jira
-ticketlens CNV1-2 --no-attachments # Skip attachment download
+ticketlens CNV1-2 --no-attachments # Skip attachment download and Confluence page fetching
 ticketlens CNV1-2 --check          # Append local VCS diff + Claude Code review instructions
 ticketlens CNV1-2 --compliance     # Check ticket requirements against local diff [Pro/Free 3/mo]
 ticketlens CNV1-2 --summarize      # AI summary via your own API key (BYOK) [Pro]
@@ -102,7 +103,7 @@ ticketlens CNV1-2 --summarize --cloud  # AI summary routed through TicketLens AP
 
 | `--depth` | Scope |
 |-----------|-------|
-| `0` | Target ticket: description, comments, attachments |
+| `0` | Target ticket: description, comments, attachments, Confluence pages |
 | `1` | + linked tickets: descriptions and comments _(default)_ |
 | `2` | + linked-of-linked: key and summary only |
 
@@ -116,6 +117,8 @@ After the first fetch, ticket data is cached to `~/.ticketlens/cache/PROFILE/TIC
 
 Attachments download to `~/.ticketlens/cache/TICKET-KEY/` (10 MB per-file cap). Claude Code reads images multimodally, extracts PDF text, and reads plain text files as context.
 
+Confluence pages linked to the ticket via Jira Remote Links are fetched automatically and included as plain text in the brief (Jira profiles only, same-origin). Use `--no-attachments` to skip both attachments and Confluence pages.
+
 ---
 
 ### Triage
@@ -455,12 +458,12 @@ npm test
 See [ROADMAP.md](ROADMAP.md) for the full plan.
 
 Recently shipped:
+- **Confluence pages** — linked Confluence pages fetched automatically and included in the brief; origin-validated, non-fatal, capped at 10 pages
 - **Linear support** — `ticketlens init` → Linear; connects via GraphQL API key, fetches tickets, triage, and statuses
 - **GitHub Issues support** — `ticketlens init` → GitHub Issues; PAT-based, same normalized ticket shape
 - **Tracker-aware config** — `ticketlens config` shows the right labels and skips irrelevant prompts per tracker type; always re-validates the connection
 
 Coming up:
-- Confluence/wiki page fetching (F45)
 - "TicketLens for PRs" — PR context assembly with linked tickets and compliance (F46)
 
 ---

package/bin/ticketlens.mjs

@@ -23,6 +23,7 @@ import {
   printActivateHelp, printLicenseHelp, printDeleteHelp,
   printProfilesHelp, printScheduleHelp,
   printInitHelp, printSwitchHelp, printConfigHelp,
+  printReviewHelp,
 } from '../skills/jtb/scripts/lib/help.mjs';
 import { createStyler } from '../skills/jtb/scripts/lib/ansi.mjs';
 import { readCliToken, saveCliToken, deleteCliToken } from '../skills/jtb/scripts/lib/cli-auth.mjs';
@@ -262,6 +263,14 @@ switch (command) {
     });
     break;
 
+  case 'review':
+    if (cmdArgs.includes('--help') || cmdArgs.includes('-h')) { printReviewHelp(); break; }
+    runFetch(['review', ...cmdArgs]).catch(err => {
+      process.stderr.write(`Error: ${err.message}\n`);
+      process.exitCode = 1;
+    });
+    break;
+
   case 'ledger':
     runFetch(['ledger', ...cmdArgs]).catch(err => {
       process.stderr.write(`Error: ${err.message}\n`);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ticketlens",
-  "version": "0.1.14",
+  "version": "0.1.15",
   "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

@@ -342,6 +342,87 @@ export async function run(args, envOrOpts = process.env, fetcher = globalThis.fe
     return;
   }
 
+  if (args[0] === 'review') {
+    const { assemblePrReview, extractTicketKeys } = await import('./lib/pr-review-assembler.mjs');
+    const { analyzeDiff } = await import('./lib/diff-analyzer.mjs');
+    const { extractRequirements } = await import('./lib/requirement-extractor.mjs');
+
+    const resolvedConfigDir = configDir ?? (await import('./lib/config.mjs')).DEFAULT_CONFIG_DIR;
+    const execFn = opts.execFn ?? spawnSync;
+    const cwd = process.cwd();
+
+    // Resolve base branch: --base=BRANCH or auto-detect main/master/develop
+    const baseArg = args.find(a => a.startsWith('--base='));
+    let baseBranch = baseArg ? baseArg.split('=')[1] : null;
+    if (!baseBranch) {
+      for (const candidate of ['main', 'master', 'develop']) {
+        const r = execFn('git', ['rev-parse', '--verify', candidate], { encoding: 'utf8', cwd, timeout: 5_000 });
+        if (r.status === 0) { baseBranch = candidate; break; }
+      }
+      baseBranch = baseBranch ?? 'main';
+    }
+
+    // Current branch name
+    const headResult = execFn('git', ['branch', '--show-current'], { encoding: 'utf8', cwd, timeout: 5_000 });
+    const headBranch = headResult.status === 0 ? (headResult.stdout.trim() || null) : null;
+
+    // Commits in range
+    const logResult = execFn('git', ['log', '--oneline', `${baseBranch}..HEAD`], { encoding: 'utf8', cwd, timeout: 10_000 });
+    const commitMessages = logResult.status === 0 ? (logResult.stdout ?? '') : '';
+
+    // Diff between base and HEAD
+    const diffResult = execFn('git', ['diff', `${baseBranch}..HEAD`], { encoding: 'utf8', cwd, timeout: 30_000 });
+    const diff = diffResult.status === 0 ? (diffResult.stdout || null) : null;
+
+    // Extract ticket keys from branch name + commit messages
+    const allText = [headBranch ?? '', commitMessages].join('\n');
+    const ticketKeys = extractTicketKeys(allText);
+
+    // Fetch tickets (best-effort — gracefully degrade if no credentials)
+    const profileArgR = args.find(a => a.startsWith('--profile='));
+    const profileNameR = profileArgR ? profileArgR.split('=')[1] : undefined;
+    const tickets = [];
+
+    if (ticketKeys.length > 0) {
+      const connR = resolveConnection(ticketKeys[0], {
+        env,
+        configDir: resolvedConfigDir,
+        profileName: profileNameR,
+        cwd,
+        onWarning: () => {},
+        onProfileNotFound: () => {},
+      });
+      const hasAuthR = connR.pat || (connR.email && connR.apiToken);
+      if (connR.baseUrl && hasAuthR) {
+        const adapterR = resolveAdapter(connR, { fetcher });
+        for (const key of ticketKeys) {
+          try {
+            tickets.push(await adapterR.fetchTicket(key, { depth: 0 }));
+          } catch { /* non-fatal — skip ticket */ }
+        }
+      } else {
+        process.stderr.write(
+          `Note: Found tickets [${ticketKeys.join(', ')}] but no profile configured. Run 'ticketlens init' to set up.\n`
+        );
+      }
+    }
+
+    const isLic = opts.isLicensedFn ?? ((tier) => isLicensed(tier, resolvedConfigDir));
+    const assembleFn = opts.assemblePrReviewFn ?? assemblePrReview;
+
+    const md = await assembleFn({
+      diff,
+      tickets,
+      baseBranch,
+      headBranch,
+      isLicensedFn: isLic,
+      analyzeDiffFn: analyzeDiff,
+      extractRequirementsFn: extractRequirements,
+    });
+    printFn(md + '\n');
+    return;
+  }
+
   const ticketKey = args.find(a => !a.startsWith('--'));
   if (!ticketKey) {
     printFetchHelp({ stream: process.stderr });

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

@@ -38,6 +38,7 @@ export function printHelp({ stream = process.stdout } = {}) {
     `    ${s.brand('ticketlens')} ${s.dim('<TICKET-KEY>')} ${s.dim('[options]')}   Fetch a ticket brief`,
     `    ${s.brand('ticketlens')} get ${s.dim('<TICKET-KEY>')}         Same as above ${s.dim('(explicit alias)')}`,
     `    ${s.brand('ticketlens')} triage ${s.dim('[options]')}         Scan your assigned tickets`,
+    `    ${s.brand('ticketlens')} review ${s.dim('[--base=BRANCH]')}    Code-review context brief from current branch`,
     `    ${s.brand('ticketlens')} compliance ${s.dim('<TICKET-KEY>')}    Check requirements coverage  ${s.dim('[Pro/Free 3/mo]')}`,
     '',
     `    ${s.brand('ticketlens')} delete ${s.dim('<PROFILE-NAME>')}     Remove a profile`,
@@ -539,3 +540,37 @@ export function printTriageHelp({ stream = process.stdout } = {}) {
   ];
   stream.write(lines.join('\n') + '\n');
 }
+
+export function printReviewHelp({ stream = process.stdout } = {}) {
+  const s = createStyler({ isTTY: stream.isTTY });
+  const lines = [
+    '',
+    `  ${s.bold(s.brand('ticketlens'))} ${s.bold('review')} ${s.dim('[--base=BRANCH] [--profile=NAME]')}`,
+    '',
+    `  Assemble a code-review context brief from your current branch.`,
+    `  Extracts linked ticket keys from the branch name and commit messages,`,
+    `  fetches each ticket, and outputs a markdown brief for AI-assisted review.`,
+    '',
+    `  Requirements coverage analysis against the diff is available on ${s.dim('[Pro]')}.`,
+    '',
+    `  ${s.bold('OPTIONS')}`,
+    '',
+    `    ${s.brand('--base')}=${s.dim('BRANCH')}    Compare against this branch ${s.dim('(default: auto-detect main/master/develop)')}`,
+    `    ${s.brand('--profile')}=${s.dim('NAME')}  Use a specific tracker profile for ticket fetching`,
+    `    ${s.brand('-h')}, ${s.brand('--help')}       Show this help`,
+    '',
+    `  ${s.bold('EXAMPLES')}`,
+    '',
+    `    ${s.dim('$')} ticketlens review`,
+    `    ${s.dim('$')} ticketlens review --base=main`,
+    `    ${s.dim('$')} ticketlens review --base=main --profile=myteam`,
+    `    ${s.dim('$')} ticketlens review --base=main | pbcopy ${s.dim('# copy brief to clipboard')}`,
+    '',
+    `  ${s.bold('OUTPUT SECTIONS')}`,
+    '',
+    `    Branch, Changed files, Ticket context`,
+    `    Requirements coverage ${s.dim('[Pro]')}, Review focus ${s.dim('[Pro]')}`,
+    '',
+  ];
+  stream.write(lines.join('\n') + '\n');
+}

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

@@ -0,0 +1,164 @@
+/**
+ * PR Review Context Assembler
+ * Given a git diff + pre-fetched tickets, produces a markdown brief
+ * suitable for AI-assisted code review.
+ */
+
+import { analyzeDiff } from './diff-analyzer.mjs';
+import { extractRequirements } from './requirement-extractor.mjs';
+
+const TICKET_KEY_RE = /\b([A-Z][A-Z0-9]+-\d+)\b/g;
+
+/**
+ * Extract all unique ticket keys from a string (branch name, commit messages, etc.).
+ * Returns a sorted, deduplicated array.
+ *
+ * @param {string} text
+ * @returns {string[]}
+ */
+export function extractTicketKeys(text) {
+  if (!text || typeof text !== 'string') return [];
+  const found = new Set(text.match(TICKET_KEY_RE) ?? []);
+  return [...found].sort();
+}
+
+/**
+ * Build the "### Changed files" section from a unified diff string.
+ * Parses "+++ b/<path>" headers.
+ *
+ * @param {string|null} diff
+ * @returns {string} markdown section or empty string
+ */
+function buildChangedFilesSection(diff) {
+  if (!diff) return '';
+  const files = [];
+  for (const line of diff.split('\n')) {
+    if (line.startsWith('+++ b/')) {
+      const path = line.slice(6).trim();
+      if (path && path !== '/dev/null') files.push(path);
+    }
+  }
+  if (files.length === 0) return '';
+  const lines = ['', `### Changed files (${files.length})`];
+  for (const f of files) lines.push(`- ${f}`);
+  return lines.join('\n');
+}
+
+/**
+ * Build the coverage + review-focus sections for all tickets (Pro-gated).
+ * Returns empty string when no requirements exist across all tickets.
+ *
+ * @param {object[]} tickets
+ * @param {string} diff
+ * @param {Function} extractRequirementsFn
+ * @param {Function} analyzeDiffFn
+ * @returns {string}
+ */
+function buildCoverageSections(tickets, diff, extractRequirementsFn, analyzeDiffFn) {
+  const coverageEntries = [];
+  const focusEntries = [];
+
+  for (const ticket of tickets) {
+    const reqs = extractRequirementsFn(ticket.description ?? '');
+    if (reqs.length === 0) continue;
+
+    const { results, coveragePercent } = analyzeDiffFn(reqs, diff);
+    coverageEntries.push({ ticket, results, coveragePercent });
+
+    const uncovered = results
+      .filter(r => r.status === 'NOT_FOUND')
+      .map(r => r.requirement);
+    if (uncovered.length > 0) focusEntries.push({ ticket, uncovered });
+  }
+
+  if (coverageEntries.length === 0) return '';
+
+  const lines = ['', '### Requirements coverage'];
+  for (const { ticket, results, coveragePercent } of coverageEntries) {
+    lines.push('', `#### ${ticket.key} (${coveragePercent}%)`);
+    for (const r of results) {
+      if (r.status === 'NOT_FOUND') {
+        lines.push(`- ✖ ${r.requirement}`);
+      } else {
+        const loc = r.evidence ? ` — ${r.evidence}` : '';
+        const mark = r.status === 'FOUND' ? '✔' : '~';
+        lines.push(`- ${mark} ${r.requirement}${loc}`);
+      }
+    }
+  }
+
+  if (focusEntries.length > 0) {
+    lines.push('', '### Review focus');
+    for (const { ticket, uncovered } of focusEntries) {
+      lines.push('', `#### ${ticket.key} — uncovered requirements`);
+      for (const req of uncovered) lines.push(`- ${req}`);
+    }
+  }
+
+  return lines.join('\n');
+}
+
+/**
+ * Assemble a markdown PR code-review context brief.
+ *
+ * @param {object} opts
+ * @param {string|null}  [opts.diff]               - git diff string
+ * @param {object[]}     [opts.tickets]             - pre-fetched ticket objects
+ * @param {string}       [opts.baseBranch]          - e.g. 'main'
+ * @param {string|null}  [opts.headBranch]          - current branch name
+ * @param {Function}     [opts.isLicensedFn]        - fn(tier) → bool
+ * @param {Function}     [opts.analyzeDiffFn]       - injectable (tests)
+ * @param {Function}     [opts.extractRequirementsFn] - injectable (tests)
+ * @returns {Promise<string>}
+ */
+export async function assemblePrReview({
+  diff = null,
+  tickets = [],
+  baseBranch = 'main',
+  headBranch = null,
+  isLicensedFn = () => false,
+  analyzeDiffFn = analyzeDiff,
+  extractRequirementsFn = extractRequirements,
+} = {}) {
+  const lines = ['## PR Review Context'];
+
+  // Branch
+  lines.push('', '### Branch');
+  if (headBranch) {
+    lines.push(`\`${headBranch}\` → \`${baseBranch}\``);
+  } else {
+    lines.push(`→ \`${baseBranch}\``);
+  }
+
+  // Changed files (from diff)
+  const filesSection = buildChangedFilesSection(diff);
+  if (filesSection) lines.push(filesSection);
+
+  if (tickets.length === 0) {
+    lines.push('', '_No linked tickets found in branch name or commits._');
+  } else {
+    // Ticket context
+    lines.push('', '### Ticket context');
+    for (const ticket of tickets) {
+      const reqs = extractRequirementsFn(ticket.description ?? '');
+      lines.push('', `#### ${ticket.key}: ${ticket.summary ?? ''}`);
+      if (reqs.length === 0) {
+        lines.push('_No requirements found._');
+      } else {
+        for (const req of reqs) lines.push(`- ${req}`);
+      }
+    }
+
+    // Requirements coverage (Pro-gated, requires diff)
+    if (isLicensedFn('pro') && diff) {
+      const coverageOut = buildCoverageSections(tickets, diff, extractRequirementsFn, analyzeDiffFn);
+      if (coverageOut) lines.push(coverageOut);
+    } else if (!isLicensedFn('pro')) {
+      lines.push('', '_Requirements coverage analysis requires a Pro license — `ticketlens activate <KEY>`_');
+    }
+  }
+
+  lines.push('', '---', `_Generated by TicketLens · \`ticketlens review --base=${baseBranch}\`_`);
+
+  return lines.join('\n');
+}