phewsh 0.15.43 → 0.15.45

package/commands/update.js

@@ -112,9 +112,24 @@ async function main() {
     if (code === 0) {
       console.log(`  ${green('✓')} Updated to v${latest}. New shells pick it up immediately.`);
     } else {
-      console.log(`  ${yellow('Update failed')} (npm exited ${code}).`);
-      console.log(`  ${g('If it was a permissions error, fix your npm prefix or run:')}`);
-      console.log(`  ${w(`sudo npm install -g ${pkg.name}@latest`)}`);
+      // NEVER suggest `sudo npm i -g` — that root-owns the package dir and
+      // poisons every future non-sudo update (the exact trap users hit). The
+      // correct fix is to OWN the package, once, then update normally.
+      let prefix = '';
+      try {
+        prefix = require('child_process').execFileSync('npm', ['config', 'get', 'prefix'],
+          { encoding: 'utf-8', timeout: 3000 }).trim();
+      } catch { /* unknown prefix */ }
+      const pkgDir = prefix ? `${prefix}/lib/node_modules/${pkg.name}` : `$(npm prefix -g)/lib/node_modules/${pkg.name}`;
+      const binLink = prefix ? `${prefix}/bin/${pkg.name}` : `$(npm prefix -g)/bin/${pkg.name}`;
+      console.log(`  ${yellow('Update failed')} (npm exited ${code}) — almost certainly a permissions issue.`);
+      console.log(`  ${g('The phewsh package is probably root-owned from a past `sudo` install.')}`);
+      console.log(`  ${b('Own it once, then updates never need sudo again:')}`);
+      console.log(`    ${w(`sudo chown -R $(whoami) "${pkgDir}"`)}`);
+      console.log(`    ${w(`sudo chown -h $(whoami) "${binLink}"`)}`);
+      console.log(`    ${w(`phewsh update`)}`);
+      console.log(`  ${g('Or reinstall with the installer (handles this for you):')} ${w('curl -fsSL phewsh.com/install.sh | sh')}`);
+      console.log(`  ${g('Do NOT run `sudo npm i -g` — that is what caused this.')}`);
     }
     console.log('');
     process.exit(code ?? 0);

package/lib/slash-commands.js

@@ -28,10 +28,14 @@ If there is no \`.intent/\` directory here, say so and tell me I can create one
 
 // Tools that support file-based custom slash commands, and how each wants them.
 // parentDir must already exist (the tool is installed) before we write.
+//
+// Claude Code is deliberately EXCLUDED: it already ships a phewsh `/intent`
+// SKILL (the artifact generator) and gets the SessionStart hook brief, so a
+// global `/intent` command here just collides with the canonical one (two
+// `/intent` entries). We add `/intent` only where it's genuinely absent.
 const SLASH_TARGETS = [
-  { id: 'claude-code', parent: '.claude', sub: 'commands', file: 'intent.md', fmt: 'claude' },
-  { id: 'codex',       parent: '.codex',  sub: 'prompts',  file: 'intent.md', fmt: 'codex' },
-  { id: 'gemini',      parent: '.gemini', sub: 'commands', file: 'intent.toml', fmt: 'gemini' },
+  { id: 'codex',  parent: '.codex',  sub: 'prompts',  file: 'intent.md', fmt: 'codex' },
+  { id: 'gemini', parent: '.gemini', sub: 'commands', file: 'intent.toml', fmt: 'gemini' },
 ];
 
 function bodyFor(fmt) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "phewsh",
-  "version": "0.15.43",
+  "version": "0.15.45",
   "description": "Turn intent into action. Structure your thinking, execute your next step.",
   "bin": {
     "phewsh": "bin/phewsh.js"