@timmy6942025/cli-timer 1.1.0 → 1.1.2

package/README.md

@@ -102,6 +102,7 @@ This launches a Bubble Tea based screen where you can change:
 - Tick rate (50-1000 ms)
 - Completion message
 - System notification on completion (default On)
+- Completion sound/alarm on completion (default Off)
 - Pause key / pause alt key
 - Restart key
 - Exit key / exit alt key
@@ -113,3 +114,5 @@ Controls in settings UI:
 - `/`: filter fonts in font picker
 - `/`: filter keys in key picker
 - `Esc`/`q`: back/cancel
+
+Note for macOS: If system notifications are inconsistent with built-in AppleScript notifications, install `terminal-notifier` (`brew install terminal-notifier`) for improved reliability.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@timmy6942025/cli-timer",
-  "version": "1.1.0",
+  "version": "1.1.2",
   "description": "Simple customizable terminal timer and stopwatch",
   "main": "src/index.js",
   "bin": {

package/settings-ui/main.go

@@ -39,14 +39,15 @@ var defaultKeybindings = keybindings{
 }
 
 type config struct {
-	Font              string      `json:"font"`
-	CenterDisplay     bool        `json:"centerDisplay"`
-	ShowHeader        bool        `json:"showHeader"`
-	ShowControls      bool        `json:"showControls"`
-	TickRateMs        int         `json:"tickRateMs"`
-	CompletionMessage string      `json:"completionMessage"`
-	NotifyOnComplete  bool        `json:"notifyOnComplete"`
-	Keybindings       keybindings `json:"keybindings"`
+	Font                string      `json:"font"`
+	CenterDisplay       bool        `json:"centerDisplay"`
+	ShowHeader          bool        `json:"showHeader"`
+	ShowControls        bool        `json:"showControls"`
+	TickRateMs          int         `json:"tickRateMs"`
+	CompletionMessage   string      `json:"completionMessage"`
+	NotifyOnComplete    bool        `json:"notifyOnComplete"`
+	PlaySoundOnComplete bool        `json:"playSoundOnComplete"`
+	Keybindings         keybindings `json:"keybindings"`
 }
 
 type statePayload struct {
@@ -151,6 +152,7 @@ func buildMenuItems(cfg config) []list.Item {
 		menuEntry{id: "tickRate", title: "Tick rate", description: fmt.Sprintf("%d ms", cfg.TickRateMs)},
 		menuEntry{id: "message", title: "Completion message", description: summarizeMessage(cfg.CompletionMessage)},
 		menuEntry{id: "notify", title: "System notification", description: boolText(cfg.NotifyOnComplete)},
+		menuEntry{id: "sound", title: "Completion sound/alarm", description: boolText(cfg.PlaySoundOnComplete)},
 		menuEntry{id: "pauseKey", title: "Pause key", description: keyTokenLabel(cfg.Keybindings.PauseKey)},
 		menuEntry{id: "pauseAltKey", title: "Pause alt key", description: keyTokenLabel(cfg.Keybindings.PauseAltKey)},
 		menuEntry{id: "restartKey", title: "Restart key", description: keyTokenLabel(cfg.Keybindings.RestartKey)},
@@ -226,14 +228,15 @@ func normalizeKeybindings(cfg keybindings) keybindings {
 
 func normalizeConfig(cfg config) config {
 	result := config{
-		Font:              defaultFont,
-		CenterDisplay:     true,
-		ShowHeader:        true,
-		ShowControls:      true,
-		TickRateMs:        defaultTickRateMs,
-		CompletionMessage: defaultCompletionMessage,
-		NotifyOnComplete:  true,
-		Keybindings:       defaultKeybindings,
+		Font:                defaultFont,
+		CenterDisplay:       true,
+		ShowHeader:          true,
+		ShowControls:        true,
+		TickRateMs:          defaultTickRateMs,
+		CompletionMessage:   defaultCompletionMessage,
+		NotifyOnComplete:    true,
+		PlaySoundOnComplete: false,
+		Keybindings:         defaultKeybindings,
 	}
 
 	if strings.TrimSpace(cfg.Font) != "" {
@@ -249,6 +252,7 @@ func normalizeConfig(cfg config) config {
 		result.CompletionMessage = normalizeCompletionMessage(cfg.CompletionMessage)
 	}
 	result.NotifyOnComplete = cfg.NotifyOnComplete
+	result.PlaySoundOnComplete = cfg.PlaySoundOnComplete
 	result.Keybindings = normalizeKeybindings(cfg.Keybindings)
 	return result
 }
@@ -462,6 +466,10 @@ func (m *model) applyMenuAction() tea.Cmd {
 		m.payload.Config.NotifyOnComplete = !m.payload.Config.NotifyOnComplete
 		m.refreshMenu()
 		return nil
+	case "sound":
+		m.payload.Config.PlaySoundOnComplete = !m.payload.Config.PlaySoundOnComplete
+		m.refreshMenu()
+		return nil
 	case "pauseKey":
 		m.openKeyPicker("pauseKey", "Select Pause Key")
 		return nil

package/settings-ui/main_test.go

@@ -10,14 +10,15 @@ func testPayload() statePayload {
 	return statePayload{
 		ConfigPath: "/tmp/cli-timer-test-config.json",
 		Config: config{
-			Font:              "Standard",
-			CenterDisplay:     true,
-			ShowHeader:        true,
-			ShowControls:      true,
-			TickRateMs:        100,
-			CompletionMessage: "Time is up!",
-			NotifyOnComplete:  false,
-			Keybindings:       defaultKeybindings,
+			Font:                "Standard",
+			CenterDisplay:       true,
+			ShowHeader:          true,
+			ShowControls:        true,
+			TickRateMs:          100,
+			CompletionMessage:   "Time is up!",
+			NotifyOnComplete:    false,
+			PlaySoundOnComplete: false,
+			Keybindings:         defaultKeybindings,
 		},
 		Fonts: []string{"Standard", "Big"},
 	}

package/src/index.js

@@ -39,6 +39,7 @@ const DEFAULT_CONFIG = Object.freeze({
   tickRateMs: 100,
   completionMessage: "Time is up!",
   notifyOnComplete: true,
+  playSoundOnComplete: false,
   keybindings: { ...DEFAULT_KEYBINDINGS }
 });
 
@@ -203,6 +204,7 @@ function normalizeConfig(raw) {
     tickRateMs: DEFAULT_CONFIG.tickRateMs,
     completionMessage: DEFAULT_CONFIG.completionMessage,
     notifyOnComplete: DEFAULT_CONFIG.notifyOnComplete,
+    playSoundOnComplete: DEFAULT_CONFIG.playSoundOnComplete,
     keybindings: { ...DEFAULT_KEYBINDINGS }
   };
 
@@ -225,6 +227,9 @@ function normalizeConfig(raw) {
     if (typeof raw.notifyOnComplete === "boolean") {
       next.notifyOnComplete = raw.notifyOnComplete;
     }
+    if (typeof raw.playSoundOnComplete === "boolean") {
+      next.playSoundOnComplete = raw.playSoundOnComplete;
+    }
     next.keybindings = normalizeKeybindings(raw.keybindings);
     if (typeof raw.font === "string") {
       const normalizedFont = normalizeFontName(raw.font);
@@ -485,11 +490,11 @@ function sendSystemNotification({ title, message }) {
 
   try {
     if (process.platform === "darwin") {
-      const script = `display notification "${escapeAppleScriptString(safeMessage)}" with title "${escapeAppleScriptString(safeTitle)}"`;
-      if (spawnOk("osascript", ["-e", script])) {
+      if (spawnOk("terminal-notifier", ["-title", safeTitle, "-message", safeMessage])) {
         return true;
       }
-      if (spawnOk("terminal-notifier", ["-title", safeTitle, "-message", safeMessage])) {
+      const script = `display notification "${escapeAppleScriptString(safeMessage)}" with title "${escapeAppleScriptString(safeTitle)}"`;
+      if (spawnOk("osascript", ["-e", script])) {
         return true;
       }
       return false;
@@ -568,19 +573,28 @@ function sendSystemNotification({ title, message }) {
   return false;
 }
 
+function playCompletionAlarm(config) {
+  if (!config || !config.playSoundOnComplete) {
+    return;
+  }
+  try {
+    process.stderr.write("\x07\x07\x07");
+  } catch (_error) {
+  }
+}
+
 function notifyTimerFinished(config, initialSeconds) {
-  if (!config || !config.notifyOnComplete) {
+  if (!config) {
     return;
   }
-  const message = config.completionMessage || "Time is up!";
-  const title = initialSeconds ? `Timer finished (${formatHms(initialSeconds)})` : "Timer finished";
-  const notified = sendSystemNotification({ title, message });
-  if (!notified) {
-    try {
-      process.stderr.write("\x07");
-    } catch (_error) {
-    }
+
+  if (config.notifyOnComplete) {
+    const message = config.completionMessage || "Time is up!";
+    const title = initialSeconds ? `Timer finished (${formatHms(initialSeconds)})` : "Timer finished";
+    sendSystemNotification({ title, message });
   }
+
+  playCompletionAlarm(config);
 }
 
 function runNonInteractiveTimer(initialSeconds, tickRateMs) {