pingthings 0.2.0 → 0.3.0

package/LICENSE

@@ -6,6 +6,25 @@
  Everyone is permitted to copy and distribute verbatim copies
  of this license document, but changing it is not allowed.
 
-Audio assets are from Seven Kingdoms: Ancient Adversaries
-Copyright 1997,1998 Enlight Software Ltd.
-Licensed under GPL v2.
+This project is licensed under GPL v2. Audio assets are sourced from
+multiple open source projects under compatible licenses:
+
+7kaa-soldiers: Seven Kingdoms: Ancient Adversaries
+  Copyright 1997,1998 Enlight Software Ltd. — GPL v2
+
+wesnoth-combat: Battle for Wesnoth
+  Copyright Wesnoth contributors — GPL v2+
+
+openarena-announcer: OpenArena
+  Copyright OpenArena contributors — GPL v2
+
+freedoom-arsenal: Freedoom
+  Copyright Freedoom contributors — BSD-3-Clause
+
+warzone2100-command: Warzone 2100
+  Copyright Warzone 2100 Project — GPL v2
+
+0ad-civilizations: 0 A.D.
+  Copyright Wildfire Games — CC-BY-SA 3.0
+  Distributed as part of this aggregate work alongside GPL v2 code.
+  See packs/0ad-civilizations/manifest.json for full attribution.

package/README.md

@@ -126,10 +126,16 @@ For different sounds based on what Claude is doing, set up multiple hooks:
 |---------|-------------|
 | `pingthings play [sound] [--event type]` | Play a sound (random, specific, or event-based) |
 | `pingthings list` | Show available sound packs |
+| `pingthings select` | Interactive pack selector |
 | `pingthings use <pack>` | Set the active sound pack |
 | `pingthings preview <pack>` | Preview a random sound from a pack |
+| `pingthings test-events [pack]` | Play all event sounds to hear each one |
+| `pingthings theme [name]` | Apply a sound theme |
 | `pingthings config [key] [val]` | Show or update configuration |
-| `pingthings install <pack>` | Install a sound pack (coming soon) |
+| `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
 
@@ -139,13 +145,17 @@ Config lives at `~/.config/pingthings/config.json`:
 {
   "activePack": "7kaa-soldiers",
   "mode": "random",
-  "specificSound": null
+  "specificSound": null,
+  "volume": 100,
+  "eventPacks": {}
 }
 ```
 
 - **activePack** — which sound pack to use
 - **mode** — `"random"` (default), `"specific"`, or `"informational"`
 - **specificSound** — sound name to always play when mode is `"specific"`
+- **volume** — playback volume, 0-100 (default: 100)
+- **eventPacks** — per-event pack overrides (e.g. `{"error": "freedoom-arsenal"}`)
 
 Set values via CLI:
 

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.0",
+  "version": "0.3.0",
   "description": "Notification sounds for Claude Code and other CLI tools",
   "type": "module",
   "license": "GPL-2.0",

package/src/cli/init.js

@@ -1,5 +1,5 @@
 import { createInterface } from 'node:readline';
-import { existsSync, readFileSync, writeFileSync } from 'node:fs';
+import { existsSync, readFileSync, writeFileSync, renameSync } from 'node:fs';
 import { join } from 'node:path';
 import { homedir } from 'node:os';
 
@@ -59,7 +59,9 @@ function readSettings() {
 
 function writeSettings(settings) {
   const path = getSettingsPath();
-  writeFileSync(path, JSON.stringify(settings, null, 2) + '\n', 'utf8');
+  const tmpPath = path + '.tmp';
+  writeFileSync(tmpPath, JSON.stringify(settings, null, 2) + '\n', 'utf8');
+  renameSync(tmpPath, path);
 }
 
 function applyHooks(mode) {

package/src/cli/install.js

@@ -1,4 +1,4 @@
-import { existsSync, mkdirSync, writeFileSync, readdirSync } from 'node:fs';
+import { existsSync, mkdirSync, cpSync } from 'node:fs';
 import { join, basename } from 'node:path';
 import { execFileSync } from 'node:child_process';
 import { getConfigDir } from '../config.js';
@@ -75,6 +75,10 @@ function installFromLocal(source) {
   }
 
   const packName = basename(source);
+  if (!packName) {
+    console.error('Could not determine pack name from path.');
+    process.exit(1);
+  }
   const packsDir = join(getConfigDir(), 'packs');
   const destDir = join(packsDir, packName);
 
@@ -87,7 +91,7 @@ function installFromLocal(source) {
   mkdirSync(packsDir, { recursive: true });
 
   try {
-    execFileSync('cp', ['-r', source, destDir], { stdio: 'inherit' });
+    cpSync(source, destDir, { recursive: true });
   } catch {
     console.error('Failed to copy pack.');
     process.exit(1);

package/src/cli/play.js

@@ -128,5 +128,10 @@ export default function play(args) {
     soundFile = pickRandom(sounds);
   }
 
+  if (!soundFile) {
+    console.error(`No sounds available in pack: ${packName}`);
+    process.exit(1);
+  }
+
   playSound(soundFile, config.volume);
 }

package/src/cli/select.js

@@ -2,6 +2,7 @@ import { createInterface } from 'node:readline';
 import { readConfig, writeConfig } from '../config.js';
 import { listPacks, getPackSounds } from '../packs.js';
 import { playSound } from '../player.js';
+import { basename } from 'node:path';
 
 function showHelp() {
   console.log(`
@@ -68,7 +69,7 @@ export default async function select(args) {
       const sounds = getPackSounds(chosen.name);
       if (sounds.length > 0) {
         const sample = sounds[Math.floor(Math.random() * sounds.length)];
-        playSound(sample);
+        playSound(sample, config.volume);
       }
 
       console.log(`\nActive pack set to: ${chosen.name}`);

package/src/cli/test-events.js

@@ -1,6 +1,6 @@
 import { readConfig, VALID_EVENTS } from '../config.js';
 import { getEventSounds, getPackSounds, pickRandom, resolvePack } from '../packs.js';
-import { playSound } from '../player.js';
+import { playSoundSync } from '../player.js';
 import { basename } from 'node:path';
 
 function showHelp() {
@@ -17,10 +17,6 @@ Events played: done, permission, complete, error, blocked
 `);
 }
 
-function sleep(ms) {
-  return new Promise(resolve => setTimeout(resolve, ms));
-}
-
 export default async function testEvents(args) {
   if (args.includes('--help') || args.includes('-h')) {
     showHelp();
@@ -48,10 +44,7 @@ export default async function testEvents(args) {
 
     const sound = pickRandom(sounds);
     console.log(`  ${event.padEnd(12)} ${basename(sound)}`);
-    playSound(sound);
-
-    // Wait between sounds so they don't overlap
-    await sleep(1500);
+    playSoundSync(sound, config.volume);
   }
 
   console.log('\nDone.\n');

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()}`);
+    }
   }
 }