@hasna/recordings 0.1.24 → 0.1.26

package/dist/cli/index.js

@@ -5098,7 +5098,7 @@ var init_pg_migrate = __esm(() => {
 var require_package = __commonJS((exports, module) => {
   module.exports = {
     name: "@hasna/recordings",
-    version: "0.1.24",
+    version: "0.1.26",
     type: "module",
     description: "Speech-to-text recording tool with MCP and CLI \u2014 records, transcribes, and optionally enhances text using AI",
     repository: {
@@ -6081,7 +6081,7 @@ async function processText(rawText, config, systemPrompt) {
 }
 
 // src/version.ts
-var VERSION = "0.1.24";
+var VERSION = "0.1.26";
 
 // src/cli/storage.ts
 import chalk from "chalk";
@@ -6442,6 +6442,14 @@ function registerStorageCommands(program) {
   });
 }
 
+// src/cli/options.ts
+function applyEnhancementOptions(config, opts) {
+  if (opts.enhance === false || opts.noEnhance === false) {
+    config.auto_enhance = false;
+  }
+  return config;
+}
+
 // src/cli/index.ts
 var program = new Command;
 program.name("recordings").description("Speech-to-text recording tool \u2014 record, transcribe, and enhance with AI").version(VERSION).option("--json", "Output as JSON").option("--agent <name>", "Agent name or ID").option("--project <name>", "Project name or ID").option("--session <id>", "Session ID");
@@ -6451,8 +6459,7 @@ program.command("record").description("Record from microphone, transcribe, and o
   ensureDataDir(config);
   if (opts.language)
     config.language = opts.language;
-  if (opts.noEnhance === false)
-    config.auto_enhance = false;
+  applyEnhancementOptions(config, opts);
   const deps = await checkRecordingDeps();
   if (!deps.available) {
     console.error(chalk2.red(`Error: ${deps.message}`));
@@ -6518,8 +6525,7 @@ Saved as ${recording.id.slice(0, 8)}`));
 program.command("transcribe <file>").description("Transcribe an existing audio file").option("--no-enhance", "Skip AI enhancement").option("--stream", "Stream transcription deltas while the file is processed").option("-t, --tags <tags>", "Comma-separated tags").option("--prompt <prompt>", "Vocabulary/context prompt for transcription").option("--system-prompt <prompt>", "System prompt for enhancement context").action(async (file, opts) => {
   const config = loadConfig();
   ensureDataDir(config);
-  if (opts.noEnhance === false)
-    config.auto_enhance = false;
+  applyEnhancementOptions(config, opts);
   const parentOpts = program.opts();
   if (!parentOpts.json) {
     console.log(chalk2.blue("Transcribing..."));
@@ -6883,8 +6889,7 @@ program.command("listen").description("Push-to-talk mode \u2014 press Space to s
   ensureDataDir(config);
   if (opts.language)
     config.language = opts.language;
-  if (opts.noEnhance === false)
-    config.auto_enhance = false;
+  applyEnhancementOptions(config, opts);
   const deps = await checkRecordingDeps();
   if (!deps.available) {
     console.error(chalk2.red(`Error: ${deps.message}`));
@@ -7388,4 +7393,8 @@ function findPackageRoot() {
     current = parent;
   }
 }
-program.parse();
+program.parseAsync().catch((error) => {
+  const msg = error instanceof Error ? error.message : String(error);
+  console.error(`ERROR: ${msg}`);
+  process.exit(1);
+});

package/dist/cli/options.d.ts

@@ -0,0 +1,7 @@
+import type { RecordingsConfig } from "../types/index.js";
+export type EnhancementOptionBag = {
+    enhance?: boolean;
+    noEnhance?: boolean;
+};
+export declare function applyEnhancementOptions(config: RecordingsConfig, opts: EnhancementOptionBag): RecordingsConfig;
+//# sourceMappingURL=options.d.ts.map

package/dist/cli/options.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"options.d.ts","sourceRoot":"","sources":["../../src/cli/options.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,mBAAmB,CAAC;AAE1D,MAAM,MAAM,oBAAoB,GAAG;IACjC,OAAO,CAAC,EAAE,OAAO,CAAC;IAClB,SAAS,CAAC,EAAE,OAAO,CAAC;CACrB,CAAC;AAEF,wBAAgB,uBAAuB,CACrC,MAAM,EAAE,gBAAgB,EACxB,IAAI,EAAE,oBAAoB,GACzB,gBAAgB,CAKlB"}

package/dist/mcp/index.js

@@ -4920,7 +4920,7 @@ var require_lib2 = __commonJS((exports, module) => {
 var require_package = __commonJS((exports, module) => {
   module.exports = {
     name: "@hasna/recordings",
-    version: "0.1.24",
+    version: "0.1.26",
     type: "module",
     description: "Speech-to-text recording tool with MCP and CLI \u2014 records, transcribes, and optionally enhances text using AI",
     repository: {
@@ -9797,7 +9797,7 @@ async function processText(rawText, config, systemPrompt) {
 }
 
 // src/version.ts
-var VERSION = "0.1.24";
+var VERSION = "0.1.26";
 
 // src/db/storage-config.ts
 import { existsSync as existsSync2, readFileSync as readFileSync2 } from "fs";

package/dist/version.d.ts

@@ -1,2 +1,2 @@
-export declare const VERSION = "0.1.24";
+export declare const VERSION = "0.1.26";
 //# sourceMappingURL=version.d.ts.map

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hasna/recordings",
-  "version": "0.1.24",
+  "version": "0.1.26",
   "type": "module",
   "description": "Speech-to-text recording tool with MCP and CLI — records, transcribes, and optionally enhances text using AI",
   "repository": {

package/src/native/Recordings/RecordingsLib/FnKeyMonitor.swift

@@ -12,6 +12,8 @@ final class FnKeyMonitor: @unchecked Sendable {
     private var healthCheckTimer: Timer?
     private var fnIsDown = false
 
+    var isRunning: Bool { eventTap != nil }
+
     private static let fnKeyCode: UInt16 = 63
 
     /// Start monitoring. Returns true if successful.

package/src/native/Recordings/RecordingsLib/RecordingEngine.swift

@@ -44,7 +44,7 @@ public struct TranscriptionResult: Sendable {
     var displayText: String { processedText ?? rawText }
 }
 
-public enum RecordingTrigger {
+public enum RecordingTrigger: Equatable, Sendable {
     case manual
     case fnKey
     case keyboardShortcut
@@ -121,6 +121,7 @@ public final class RecordingEngine: ObservableObject {
     private var recordingTimer: Timer?
     private var activeTrigger: RecordingTrigger?
     private var keyboardShortcutIsDown = false
+    private var fnKeyIsDown = false
     private var targetAppBundleIdentifier: String?
     private var targetAppPid: pid_t?
     public var projectStore: ProjectStore?
@@ -137,6 +138,7 @@ public final class RecordingEngine: ObservableObject {
 
     // fn key monitor (CGEventTap-based, swallows fn to prevent emoji picker)
     private let fnMonitor = FnKeyMonitor()
+    private var permissionRetryTimer: Timer?
 
     let home = FileManager.default.homeDirectoryForCurrentUser.path
     private var audioDir: String { "\(home)/.hasna/recordings/audio" }
@@ -164,13 +166,23 @@ public final class RecordingEngine: ObservableObject {
         // Set up fn key monitor — hold fn to record, release to stop (like WisprFlow)
         fnMonitor.onFnKeyDown = { [weak self] in
             Task { @MainActor [weak self] in
-                guard let self, self.useFnKey, !self.isRecording else { return }
+                guard let self else { return }
+                self.fnKeyIsDown = true
+                guard self.useFnKey, Self.canBeginRecording(isRecording: self.isRecording, isTranscribing: self.isTranscribing) else { return }
                 self.startRecording(trigger: .fnKey)
             }
         }
         fnMonitor.onFnKeyUp = { [weak self] in
             Task { @MainActor [weak self] in
-                guard let self, self.useFnKey, self.isRecording, self.activeTrigger == .fnKey else { return }
+                guard let self else { return }
+                self.fnKeyIsDown = false
+                guard self.useFnKey, self.activeTrigger == .fnKey else { return }
+                guard self.isRecording else {
+                    self.log("fn released before recording started; cancelling pending start")
+                    self.resetRecordingIntent()
+                    self.updateStatus()
+                    return
+                }
                 self.stopAndTranscribe()
             }
         }
@@ -180,7 +192,7 @@ public final class RecordingEngine: ObservableObject {
             Task { @MainActor [weak self] in
                 guard let self, !self.keyboardShortcutIsDown else { return }
                 self.keyboardShortcutIsDown = true
-                guard !self.isRecording else { return }
+                guard Self.canBeginRecording(isRecording: self.isRecording, isTranscribing: self.isTranscribing) else { return }
                 self.startRecording(trigger: .keyboardShortcut)
             }
         }
@@ -188,11 +200,27 @@ public final class RecordingEngine: ObservableObject {
             Task { @MainActor [weak self] in
                 guard let self, self.keyboardShortcutIsDown else { return }
                 self.keyboardShortcutIsDown = false
-                guard self.isRecording, self.activeTrigger == .keyboardShortcut else { return }
+                guard self.activeTrigger == .keyboardShortcut else { return }
+                guard self.isRecording else {
+                    self.log("shortcut released before recording started; cancelling pending start")
+                    self.resetRecordingIntent()
+                    self.updateStatus()
+                    return
+                }
                 self.stopAndTranscribe()
             }
         }
 
+        // Granting Accessibility does not revive a tap that failed to create,
+        // so retry until permissions arrive instead of requiring a relaunch.
+        permissionRetryTimer = Timer.scheduledTimer(withTimeInterval: 5.0, repeats: true) { [weak self] _ in
+            Task { @MainActor [weak self] in
+                guard let self, self.useFnKey, !self.fnMonitor.isRunning, AXIsProcessTrusted() else { return }
+                self.log("accessibility granted — retrying fn monitor")
+                self.updateFnMonitor()
+            }
+        }
+
         updateStatus()
     }
 
@@ -278,7 +306,12 @@ public final class RecordingEngine: ObservableObject {
     // MARK: - Start Recording (Streaming)
 
     public func startRecording(trigger: RecordingTrigger = .manual) {
-        guard !isRecording else { return }
+        guard Self.canBeginRecording(isRecording: isRecording, isTranscribing: isTranscribing) else {
+            if isTranscribing {
+                statusMessage = "Finish transcribing before recording again"
+            }
+            return
+        }
         log("startRecording trigger=\(trigger) microphoneStatus=\(AVCaptureDevice.authorizationStatus(for: .audio).rawValue) accessibility=\(AXIsProcessTrusted())")
         activeTrigger = trigger
         keyboardShortcutIsDown = trigger == .keyboardShortcut
@@ -309,6 +342,12 @@ public final class RecordingEngine: ObservableObject {
                     guard let self else { return }
                     self.log("microphone access response granted=\(granted)")
                     if granted {
+                        guard self.shouldContinueStarting(trigger: trigger) else {
+                            self.log("recording start cancelled before microphone permission completed trigger=\(trigger)")
+                            self.resetRecordingIntent()
+                            self.updateStatus()
+                            return
+                        }
                         self.startNativeRecording()
                     } else {
                         self.resetRecordingIntent()
@@ -326,6 +365,33 @@ public final class RecordingEngine: ObservableObject {
         }
     }
 
+    nonisolated static func canBeginRecording(isRecording: Bool, isTranscribing: Bool) -> Bool {
+        !isRecording && !isTranscribing
+    }
+
+    nonisolated static func shouldContinueStartingAfterPermission(
+        trigger: RecordingTrigger,
+        keyboardShortcutIsDown: Bool,
+        fnKeyIsDown: Bool
+    ) -> Bool {
+        switch trigger {
+        case .manual:
+            return true
+        case .keyboardShortcut:
+            return keyboardShortcutIsDown
+        case .fnKey:
+            return fnKeyIsDown
+        }
+    }
+
+    private func shouldContinueStarting(trigger: RecordingTrigger) -> Bool {
+        activeTrigger == trigger && Self.shouldContinueStartingAfterPermission(
+            trigger: trigger,
+            keyboardShortcutIsDown: keyboardShortcutIsDown,
+            fnKeyIsDown: fnKeyIsDown
+        )
+    }
+
     private func startNativeRecording() {
         let streamState = PCMStreamState()
         pcmStreamState = streamState
@@ -656,6 +722,7 @@ public final class RecordingEngine: ObservableObject {
     private func resetRecordingIntent() {
         activeTrigger = nil
         keyboardShortcutIsDown = false
+        fnKeyIsDown = false
         targetAppBundleIdentifier = nil
         targetAppPid = nil
     }

package/src/native/Recordings/RecordingsTests/RecordingStartGateTests.swift

@@ -0,0 +1,48 @@
+import Testing
+@testable import RecordingsLib
+
+struct RecordingStartGateTests {
+    @Test("recording cannot begin while already recording or transcribing")
+    func cannotBeginWhenBusy() {
+        #expect(RecordingEngine.canBeginRecording(isRecording: false, isTranscribing: false) == true)
+        #expect(RecordingEngine.canBeginRecording(isRecording: true, isTranscribing: false) == false)
+        #expect(RecordingEngine.canBeginRecording(isRecording: false, isTranscribing: true) == false)
+    }
+
+    @Test("manual recording can continue after permission grant")
+    func manualPermissionContinuation() {
+        #expect(RecordingEngine.shouldContinueStartingAfterPermission(
+            trigger: .manual,
+            keyboardShortcutIsDown: false,
+            fnKeyIsDown: false
+        ))
+    }
+
+    @Test("keyboard shortcut recording is cancelled when released before permission grant")
+    func keyboardPermissionContinuationRequiresHeldKey() {
+        #expect(RecordingEngine.shouldContinueStartingAfterPermission(
+            trigger: .keyboardShortcut,
+            keyboardShortcutIsDown: true,
+            fnKeyIsDown: false
+        ))
+        #expect(RecordingEngine.shouldContinueStartingAfterPermission(
+            trigger: .keyboardShortcut,
+            keyboardShortcutIsDown: false,
+            fnKeyIsDown: false
+        ) == false)
+    }
+
+    @Test("fn recording is cancelled when released before permission grant")
+    func fnPermissionContinuationRequiresHeldKey() {
+        #expect(RecordingEngine.shouldContinueStartingAfterPermission(
+            trigger: .fnKey,
+            keyboardShortcutIsDown: false,
+            fnKeyIsDown: true
+        ))
+        #expect(RecordingEngine.shouldContinueStartingAfterPermission(
+            trigger: .fnKey,
+            keyboardShortcutIsDown: false,
+            fnKeyIsDown: false
+        ) == false)
+    }
+}