ticketlens 0.42.2 → 0.42.3

package/README.md

@@ -1121,6 +1121,7 @@ npm test
 See [ROADMAP.md](ROADMAP.md) for the full plan.
 
 Recently shipped:
+- **`review`/`compliance` fixes** — branch diff now uses merge-base semantics, no longer picks up unrelated unmerged-main files; AC parser now recognizes Jira wiki `Requirements`/`How to test` headers, not just literal "Acceptance Criteria"
 - **`ticketlens update --priority`** — a bad priority name now tells you the project's real, current valid options instead of a bare tracker error
 - **`ticketlens assign --to="name or email"`** — assign a ticket to another developer, not just yourself. Jira (Cloud + Server/DC); searches real assignable users first, lists matches, executes only on one confirmed match. Pro tier
 - **Console: Recall "select all N matching" bulk delete** — Gmail-style banner deletes every note matching your search/filters across all pages, not just the current one

package/package.json

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

@@ -800,7 +800,7 @@ export async function run(args, envOrOpts = process.env, fetcher = globalThis.fe
     const logResult = execFn('git', ['log', '--oneline', `${baseBranch}..HEAD`], { encoding: 'utf8', cwd, timeout: 10_000 });
     const commitMessages = logResult.status === 0 ? (logResult.stdout ?? '') : '';
 
-    const diffResult = execFn('git', ['diff', `${baseBranch}..HEAD`], { encoding: 'utf8', cwd, timeout: 30_000 });
+    const diffResult = execFn('git', ['diff', `${baseBranch}...HEAD`], { encoding: 'utf8', cwd, timeout: 30_000 });
     const diff = diffResult.status === 0 ? (diffResult.stdout || null) : null;
 
     const allText = [headBranch ?? '', commitMessages].join('\n');

package/skills/jtb/scripts/lib/api-utils.mjs

@@ -3,7 +3,7 @@
  * Centralised here to avoid triplicating the regex and warning logic.
  */
 
-export const DEFAULT_API_BASE = 'https://api.ticketlens.app';
+export const DEFAULT_API_BASE = 'http://api.ticketlens.test';
 export const DEFAULT_SITE_BASE = 'https://ticketlens.app';
 
 // Matches localhost, 127.0.0.1, and any hostname ending in .test or .local,

package/skills/jtb/scripts/lib/branch-scanner.mjs

@@ -43,7 +43,7 @@ export function scanCurrentBranch({ cwd = process.cwd(), execFn = spawnSync, fsC
 
   let files = [];
   if (base) {
-    const diffOut = run(execFn, ['diff', '--name-only', `${base}..HEAD`], cwd);
+    const diffOut = run(execFn, ['diff', '--name-only', `${base}...HEAD`], cwd);
     if (diffOut) {
       files = diffOut.trim().split('\n').filter(Boolean).slice(0, MAX_FILES);
     }

package/skills/jtb/scripts/lib/requirement-extractor.mjs

@@ -9,8 +9,17 @@
 const RE_GWT           = /^\s*(given|when|then)\s+(.+)/i;
 const RE_MUST_ITEM     = /^\s*[-*•]\s+((?:must|should|shall|ensure|verify)\b.+|.+(?:must|should|shall|ensure|verify).+)/i;
 const RE_NUM_MUST      = /^\s*\d+[.)]\s+((?:must|should|shall|ensure|verify)\b.+|.+(?:must|should|shall|ensure|verify).+)/i;
-const RE_AC_HEADER     = /^\s*(?:#+\s*|h[1-6]\.\s*)?acceptance criteria\s*:?\s*$/i;
-const RE_HEADING       = /^\s*(?:#+|h[1-6]\.)\s+/i;
+// Matches "Acceptance Criteria" plus the two real-world Jira Server/DC section headers
+// (backlog #43) that carry the same content without that literal phrase: "Requirements"
+// and "How to test". A nested subheading (e.g. "h3. Validation range") does NOT match
+// this and still exits the section — known limitation, filed as a follow-up.
+const RE_AC_HEADER     = /^\s*(?:#+\s*|h[1-6]\.\s*)?(?:acceptance criteria|requirements|how to test)\s*:?\s*$/i;
+// Wiki-only (h1.-h6.) for section-exit detection — backlog #43: a leading markdown "#"
+// is indistinguishable from Jira Server/DC's "#" numbered-list marker (e.g. "# Run the
+// test suite."), so including it here closed real AC/how-to-test sections on their own
+// first list item. Entry detection (RE_AC_HEADER/RE_NON_AC_HEADER) is unaffected — those
+// still accept a markdown "#" prefix on the header line itself.
+const RE_HEADING       = /^\s*h[1-6]\.\s+/i;
 const RE_BULLET        = /^\s*[-*•]\s+(.+)/;
 const RE_NUM_ITEM      = /^\s*\d+[.)]\s+(.+)/;
 const RE_NUM_MARKER    = /^\s*(\d+)[.)]/;
@@ -101,7 +110,7 @@ export function extractRequirements(text) {
 
     if (RE_AC_HEADER.test(line)) { inAcSection = true; continue; }
 
-    // Exit on any explicit heading (markdown ## or wiki h2.) that is not the AC header
+    // Exit on any explicit heading (wiki h1.-h6. only, see RE_HEADING) that is not the AC header
     if (inAcSection && RE_HEADING.test(line) && !RE_AC_HEADER.test(line)) {
       inAcSection = false;
     }