ticketlens 0.42.3 → 0.42.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ticketlens",
-  "version": "0.42.3",
+  "version": "0.42.4",
   "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/hooks/recall-auto-capture.mjs

@@ -29,7 +29,7 @@ import { appendFileSync } from 'node:fs';
 import { join } from 'node:path';
 import { isLicensed } from '../scripts/lib/license.mjs';
 import { readCliToken } from '../scripts/lib/cli-auth.mjs';
-import { buildCaptureExcerpt, privateTmpDir } from './recall-nudge-lib.mjs';
+import { buildCaptureExcerpt, privateTmpDir, writeLastCaptureAt } from './recall-nudge-lib.mjs';
 import { autoCapture } from '../scripts/lib/summarizer.mjs';
 import { runNoteAdd } from '../scripts/lib/note-command.mjs';
 import { DEFAULT_CONFIG_DIR } from '../scripts/lib/config.mjs';
@@ -56,17 +56,20 @@ function logLine(message) {
  * @param {string} opts.transcriptPath
  * @param {string} [opts.ticketKey]
  * @param {string} [opts.configDir]
+ * @param {string} [opts.cwd] - directory whose shared capture marker gets updated on a real capture (backlog #44: makes a successful background capture visible to a later sync Stop check, which otherwise has no way to know this ran)
  * @returns {Promise<{outcome: 'skipped'|'captured'|'not-written'|'error', reason?: string, title?: string, error?: string}>}
  */
 export async function runAutoCapture({
   transcriptPath,
   ticketKey,
   configDir = DEFAULT_CONFIG_DIR,
+  cwd = process.cwd(),
   isLicensedFn = isLicensed,
   readCliTokenFn = readCliToken,
   buildCaptureExcerptFn = buildCaptureExcerpt,
   autoCaptureFn = autoCapture,
   runNoteAddFn = runNoteAdd,
+  writeLastCaptureAtFn = writeLastCaptureAt,
 } = {}) {
   if (!isLicensedFn('pro', configDir)) {
     logLine('skipped: not licensed');
@@ -112,10 +115,16 @@ export async function runAutoCapture({
   });
 
   logLine(written ? `captured: ${result.title}` : `not-written: ${result.title}`);
+  if (written) writeLastCaptureAtFn(cwd, Date.now());
   return written ? { outcome: 'captured', title: result.title } : { outcome: 'not-written', title: result.title };
 }
 
 if (import.meta.url === `file://${process.argv[1]}`) {
-  const [, , transcriptPath, ticketKeyArg] = process.argv;
-  runAutoCapture({ transcriptPath, ticketKey: ticketKeyArg || undefined }).catch(() => { /* never escape — nothing is watching this process */ });
+  // cwdArg (backlog #44 code review): passed explicitly by the spawning Stop
+  // hook rather than relying on this child's own process.cwd() — Node
+  // resolves process.cwd() through symlinks, so a cwd with a symlinked
+  // component would otherwise hash to a different marker file than the one
+  // a later Stop's hasRecentCapture(cwd) reads, silently breaking the bridge.
+  const [, , transcriptPath, ticketKeyArg, cwdArg] = process.argv;
+  runAutoCapture({ transcriptPath, ticketKey: ticketKeyArg || undefined, cwd: cwdArg || undefined }).catch(() => { /* never escape — nothing is watching this process */ });
 }

package/skills/jtb/hooks/recall-nudge-stop.mjs

@@ -89,11 +89,31 @@ if (!sessionId || !transcriptPath) process.exit(0);
 
 const { sawFetch, sawMutatingAction, sawRecallFlag, sawNoteAdd, ticketKey } = scanTranscript(transcriptPath);
 const cliToken = readCliToken();
+// Backlog #44 (9th/10th reports): entitled accounts already run the async
+// auto-capture judge below — throttled to at most once per CAPTURE_FRESHNESS_MS
+// per cwd (hasRecentAutoCaptureAttempt below), not on every single Stop. Once
+// it succeeds, its outcome is now visible via writeLastCaptureAt
+// (recall-auto-capture.mjs) to a LATER Stop's hasRecentCapture() check — but
+// THIS Stop's own decision below still can't see an attempt spawned moments
+// ago (detached, still running), nor is a fresh one spawned if the window's
+// one attempt already ran and moved on. For that residual window, a hard
+// block duplicates a safety net that's already in flight or already spent:
+// exit 0 instead of 2 so this never blocks the turn for these accounts. Free
+// tier has no such net, so it keeps the hard block unchanged — this is its
+// only enforcement.
+const hasSafetyNet = isLicensed('pro') && Boolean(cliToken);
 
-if (isLicensed('pro') && cliToken && !hasRecentAutoCaptureAttempt(cwd)) {
+if (hasSafetyNet && !hasRecentAutoCaptureAttempt(cwd)) {
   try {
     writeLastAutoCaptureAttemptAt(cwd, Date.now());
-    const child = spawn(process.execPath, [AUTO_CAPTURE_SCRIPT, transcriptPath, ticketKey ?? ''], {
+    // cwd is passed explicitly as argv (not left to the child's own
+    // process.cwd()) — code review finding: Node resolves process.cwd()
+    // through symlinks, so a cwd with any symlink component (macOS os.tmpdir()
+    // itself, an iCloud-synced Desktop, a Docker bind mount) would make the
+    // child's own process.cwd() differ from the string used here, hashing to
+    // a different marker file and silently breaking the bridge this fix exists
+    // to build.
+    const child = spawn(process.execPath, [AUTO_CAPTURE_SCRIPT, transcriptPath, ticketKey ?? '', cwd], {
       cwd,
       detached: true,
       stdio: 'ignore',
@@ -156,4 +176,7 @@ if (sawRecallFlag) {
     '`ticketlens note add`. If genuinely nothing qualified, just say so — then finish.\n',
   );
 }
-process.exit(2);
+// hasSafetyNet: the message above is still written (a diagnostic trail if
+// the async job silently fails) but exit 0 means Claude Code never surfaces
+// it or blocks the turn — only exit 2 feeds stderr back (backlog #44).
+process.exit(hasSafetyNet ? 0 : 2);

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 = 'http://api.ticketlens.test';
+export const DEFAULT_API_BASE = 'https://api.ticketlens.app';
 export const DEFAULT_SITE_BASE = 'https://ticketlens.app';
 
 // Matches localhost, 127.0.0.1, and any hostname ending in .test or .local,