pingthings 0.2.1 → 0.3.0

package/README.md

@@ -135,6 +135,7 @@ For different sounds based on what Claude is doing, set up multiple hooks:
 | `pingthings init` | Set up Claude Code hooks automatically |
 | `pingthings create <dir>` | Create a pack from audio files |
 | `pingthings install <source>` | Install a pack from GitHub or local path |
+| `pingthings uninstall <pack>` | Remove a user-installed pack |
 
 ## Configuration
 

package/bin/pingthings.js

@@ -18,6 +18,7 @@ const commands = {
   preview: () => import('../src/cli/preview.js'),
   config: () => import('../src/cli/config.js'),
   install: () => import('../src/cli/install.js'),
+  uninstall: () => import('../src/cli/uninstall.js'),
   init: () => import('../src/cli/init.js'),
   create: () => import('../src/cli/create.js'),
   theme: () => import('../src/cli/theme.js'),
@@ -42,6 +43,7 @@ Commands:
   init               Set up Claude Code hooks automatically
   create <dir>       Create a new pack from a folder of audio files
   install <source>   Install a pack from GitHub or URL
+  uninstall <pack>   Remove a user-installed pack
 
 Options:
   --help, -h         Show this help message

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pingthings",
-  "version": "0.2.1",
+  "version": "0.3.0",
   "description": "Notification sounds for Claude Code and other CLI tools",
   "type": "module",
   "license": "GPL-2.0",

package/src/cli/uninstall.js

@@ -0,0 +1,48 @@
+import { existsSync, rmSync } from 'node:fs';
+import { join } from 'node:path';
+import { getConfigDir, readConfig, writeConfig } from '../config.js';
+
+function showHelp() {
+  console.log(`
+Usage: pingthings uninstall <pack>
+
+Remove a user-installed sound pack.
+
+Built-in packs cannot be uninstalled. Only packs in
+~/.config/pingthings/packs/ can be removed.
+
+Arguments:
+  pack    Name of the pack to remove
+`);
+}
+
+export default function uninstall(args) {
+  const packName = args[0];
+
+  if (!packName || packName === '--help' || packName === '-h') {
+    showHelp();
+    if (!packName) process.exit(1);
+    return;
+  }
+
+  const userPackDir = join(getConfigDir(), 'packs', packName);
+
+  if (!existsSync(userPackDir)) {
+    console.error(`Pack not found in user packs: ${packName}`);
+    console.error('Only user-installed packs can be uninstalled.');
+    console.error('Run "pingthings list" to see all packs.');
+    process.exit(1);
+  }
+
+  rmSync(userPackDir, { recursive: true });
+
+  // If this was the active pack, reset to default
+  const config = readConfig();
+  if (config.activePack === packName) {
+    config.activePack = '7kaa-soldiers';
+    writeConfig(config);
+    console.log(`Active pack reset to: 7kaa-soldiers`);
+  }
+
+  console.log(`Uninstalled: ${packName}`);
+}

package/src/packs.js

@@ -16,8 +16,13 @@ function readManifest(packDir) {
   const manifestPath = join(packDir, 'manifest.json');
   if (!existsSync(manifestPath)) return null;
   try {
-    return JSON.parse(readFileSync(manifestPath, 'utf8'));
-  } catch {
+    const manifest = JSON.parse(readFileSync(manifestPath, 'utf8'));
+    if (!manifest.name) {
+      console.error(`Warning: manifest.json in ${packDir} is missing "name" field`);
+    }
+    return manifest;
+  } catch (err) {
+    console.error(`Warning: Failed to parse ${manifestPath}: ${err.message}`);
     return null;
   }
 }

package/src/player.js

@@ -81,8 +81,12 @@ export function playSoundSync(filePath, volume) {
   const args = buildArgs(cmd, filePath, volume);
 
   try {
-    execFileSync(cmd, args, { stdio: 'ignore', timeout: 10000 });
-  } catch {
-    // Timeout or error — don't crash
+    execFileSync(cmd, args, { stdio: ['ignore', 'ignore', 'pipe'], timeout: 5000 });
+  } catch (err) {
+    if (err.killed) {
+      console.error(`Warning: Sound playback timed out for ${filePath}`);
+    } else if (err.stderr?.length) {
+      console.error(`Warning: Playback error: ${err.stderr.toString().trim()}`);
+    }
   }
 }