@allanpk716/work-skills-setup 1.4.0 → 1.4.1

package/installer/package-lock.json

@@ -1,12 +1,12 @@
 {
   "name": "@allanpk716/work-skills-setup",
-  "version": "0.1.0",
+  "version": "1.4.1",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {
     "": {
       "name": "@allanpk716/work-skills-setup",
-      "version": "0.1.0",
+      "version": "1.4.1",
       "license": "MIT",
       "dependencies": {
         "boxen": "^5.1.2",

package/installer/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@allanpk716/work-skills-setup",
-  "version": "0.1.0",
+  "version": "1.4.1",
   "description": "Work Skills installer for Windows developers",
   "main": "src/index.js",
   "bin": {

package/installer/src/hooks/hooks-installer.js

@@ -287,17 +287,11 @@ function installHooks(options = {}) {
 }
 
 /**
- * Check if global hooks are already registered
+ * Check if global hooks are already registered in settings.json.
+ * Does NOT check script content - scripts are always re-copied on install.
  * @returns {boolean}
  */
-function isHooksInstalled() {
-  const hooksDir = getHooksDir();
-  const hasScripts = SCRIPT_MAPPINGS.every(
-    ({ target }) => fs.existsSync(path.join(hooksDir, target))
-  );
-
-  if (!hasScripts) return false;
-
+function isHooksRegistered() {
   const settings = readSettings();
   if (!settings.hooks) return false;
 
@@ -311,6 +305,17 @@ function isHooksInstalled() {
   return !!(stopEntry && notifyEntry);
 }
 
+/**
+ * Check if hook scripts exist on disk (regardless of settings.json registration).
+ * @returns {boolean}
+ */
+function isHooksInstalled() {
+  const hooksDir = getHooksDir();
+  return SCRIPT_MAPPINGS.every(
+    ({ target }) => fs.existsSync(path.join(hooksDir, target))
+  );
+}
+
 /**
  * Get the global commands directory path
  * @returns {string} ~/.claude/commands/
@@ -500,6 +505,7 @@ module.exports = {
   findScriptsSourceDir,
   installHooks,
   isHooksInstalled,
+  isHooksRegistered,
   cleanMarketplaceCache,
   installCommands,
   isCommandsInstalled,

package/installer/src/hooks/index.js

@@ -2,7 +2,7 @@
 
 const chalk = require('chalk');
 const { t } = require('../i18n/index.js');
-const { installHooks, isHooksInstalled, installCommands, isCommandsInstalled } = require('./hooks-installer.js');
+const { installHooks, isHooksInstalled, isHooksRegistered, installCommands, isCommandsInstalled } = require('./hooks-installer.js');
 
 /**
  * Run hooks installation step in the npx installer pipeline.
@@ -13,33 +13,40 @@ const { installHooks, isHooksInstalled, installCommands, isCommandsInstalled } =
 async function runHooksInstallation() {
   console.log(chalk.bold.blue('\n=== Notification Hooks Setup ===\n'));
 
-  if (isHooksInstalled()) {
-    console.log(chalk.green('  ✓ Notification hooks already registered globally'));
+  // Always re-copy scripts to ensure deployed versions match source
+  const alreadyRegistered = isHooksRegistered();
+
+  if (alreadyRegistered && isHooksInstalled()) {
+    console.log(chalk.gray('  Updating notification hook scripts...'));
   } else {
     console.log(chalk.gray('  Registering notification hooks globally...'));
+  }
 
-    const result = installHooks({
-      onProgress: (stage) => {
-        const messages = {
-          locating: '  Locating notification scripts...',
-          copying: '  Copying scripts to ~/.claude/hooks/...',
-          registering: '  Registering hooks in settings.json...',
-          cleaning: '  Cleaning up marketplace cache...'
-        };
-        if (messages[stage]) {
-          console.log(chalk.gray(messages[stage]));
-        }
+  const result = installHooks({
+    onProgress: (stage) => {
+      const messages = {
+        locating: '  Locating notification scripts...',
+        copying: '  Copying scripts to ~/.claude/hooks/...',
+        registering: '  Registering hooks in settings.json...',
+        cleaning: '  Cleaning up marketplace cache...'
+      };
+      if (messages[stage]) {
+        console.log(chalk.gray(messages[stage]));
       }
-    });
+    }
+  });
 
-    if (result.success) {
-      console.log(chalk.green('  ✓ Notification hooks registered successfully'));
-      result.copied.forEach(script => {
-        console.log(chalk.gray(`    - ${script}`));
-      });
+  if (result.success) {
+    if (alreadyRegistered) {
+      console.log(chalk.green('  ✓ Notification hook scripts updated'));
     } else {
-      console.log(chalk.yellow('  ⊘ Notification hooks skipped: ' + (result.error || 'unknown error')));
+      console.log(chalk.green('  ✓ Notification hooks registered successfully'));
     }
+    result.copied.forEach(script => {
+      console.log(chalk.gray(`    - ${script}`));
+    });
+  } else {
+    console.log(chalk.yellow('  ⊘ Notification hooks skipped: ' + (result.error || 'unknown error')));
   }
 
   // Install slash commands globally

package/installer/src/marketplace/index.js

@@ -96,6 +96,13 @@ async function runMarketplaceIntegration() {
     // Step 3: Display plugins and prompt for selection
     displayPluginTable(plugins);
 
+    // Check if all plugins are already installed
+    const allInstalled = plugins.every(p => isPluginInstalled(p.name));
+    if (allInstalled) {
+      console.log(chalk.green('\n  ✓ All plugins are already installed'));
+      return { success: true, installed: 0, skipped: plugins.length, failed: 0 };
+    }
+
     const multiselect = new MultiSelect({
       name: 'plugins',
       message: t('marketplace.select_plugins'),

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@allanpk716/work-skills-setup",
-  "version": "1.4.0",
+  "version": "1.4.1",
   "description": "Work Skills installer for Windows developers",
   "main": "installer/src/index.js",
   "bin": {