ppcos 1.3.1 → 1.4.0

package/bin/ppcos.js

@@ -81,6 +81,8 @@ program
   .description('Update base skills in all clients while preserving custom work')
   .option('--client <name>', 'Update only this client')
   .option('--dry-run', 'Show changes without applying')
+  .option('--skip-modified', 'Non-interactive: keep locally modified managed files')
+  .option('--keep-local', 'Alias for --skip-modified')
   .action(gated(update));
 
 // status

package/lib/commands/update-hub.js

@@ -27,6 +27,8 @@ import { promptConflictResolution, backupFiles } from '../utils/conflicts.js';
  * @param {string} basePath - Path to downloaded/local base template directory
  * @param {object} options - Command options
  * @param {boolean} options.dryRun - Dry run mode
+ * @param {boolean} options.skipModified - Keep modified managed files without prompting
+ * @param {boolean} options.keepLocal - Alias for skipModified
  */
 export default async function updateHub(basePath, options = {}) {
   const hubDir = process.cwd();
@@ -113,9 +115,14 @@ export default async function updateHub(basePath, options = {}) {
 
   // Handle modified files
   let resolution = null;
+  const skipModified = options.skipModified || options.keepLocal;
   if (mods.modified.length > 0) {
     console.log(`Updating Hub Skills (v${currentVersion} → v${packageVersion})`);
-    resolution = await promptConflictResolution(mods.modified, hubDir, hubBasePath);
+    if (skipModified) {
+      resolution = 'skip';
+    } else {
+      resolution = await promptConflictResolution(mods.modified, hubDir, hubBasePath);
+    }
     if (resolution === 'cancel') {
       console.log('  Hub update cancelled');
       console.log('');

package/lib/commands/update.js

@@ -41,6 +41,8 @@ const CLIENT_TEMPLATE_EXCLUDE_DIRS = new Set(['tmp', 'hub-base']);
  * @param {string|object} basePathOrOptions - Path to template files (temp dir or local), or options object for backward compat
  * @param {object} options - Command options
  * @param {boolean} options.dryRun - Dry run mode
+ * @param {boolean} options.skipModified - Keep modified managed files without prompting
+ * @param {boolean} options.keepLocal - Alias for skipModified
  * @returns {Promise<{status: 'updated'|'up-to-date'|'skipped'|'cancelled', details?: object}>}
  */
 async function updateClient(clientName, basePathOrOptions = {}, options = {}) {
@@ -116,10 +118,15 @@ async function updateClient(clientName, basePathOrOptions = {}, options = {}) {
   // Handle modified files (excluding orphans, which are handled separately)
   let resolution = null;
   let backedUpFiles = [];
+  const skipModified = options.skipModified || options.keepLocal;
 
   if (modifiedNonOrphans.length > 0) {
     console.log(`Updating ${clientName} (v${currentVersion} → v${packageVersion})`);
-    resolution = await promptConflictResolution(modifiedNonOrphans, clientDir, basePath);
+    if (skipModified) {
+      resolution = 'skip';
+    } else {
+      resolution = await promptConflictResolution(modifiedNonOrphans, clientDir, basePath);
+    }
 
     if (resolution === 'cancel') {
       return { status: 'cancelled' };
@@ -273,6 +280,8 @@ async function updateClient(clientName, basePathOrOptions = {}, options = {}) {
  * @param {object} options - Command options
  * @param {string} [options.client] - Update only this client
  * @param {boolean} [options.dryRun] - Show changes without applying
+ * @param {boolean} [options.skipModified] - Keep modified managed files without prompting
+ * @param {boolean} [options.keepLocal] - Alias for skipModified
  */
 export default async function update(options = {}) {
   const packageVersion = getPackageVersion();
@@ -303,6 +312,9 @@ export default async function update(options = {}) {
 
       if (error.message.includes('Not authenticated') || error.message.includes('401') || error.message.includes('expired')) {
         logger.info('Try: ppcos login');
+      } else if (error.status === 429 && error.retryAfter) {
+        const minutes = Math.ceil(error.retryAfter / 60);
+        logger.info(`Try again in ${minutes} minute${minutes !== 1 ? 's' : ''}.`);
       } else {
         logger.info('Check your internet connection and try again.');
       }

package/lib/utils/api-client.js

@@ -108,7 +108,11 @@ export async function downloadSkills(sessionToken) {
 
     if (!response.ok) {
       const error = await response.json().catch(() => ({ error: 'Download failed' }));
-      throw new Error(error.error || 'Failed to download skills');
+      const message = error.error || 'Failed to download skills';
+      const downloadError = new Error(message);
+      downloadError.status = response.status;
+      downloadError.retryAfter = error.retryAfter;
+      throw downloadError;
     }
 
     return response;

package/lib/utils/migrations.js

@@ -87,18 +87,6 @@ export function migrateSettingsHooks(clientDir) {
           }
         ]
       }
-    ],
-    PostCompact: [
-      {
-        hooks: [
-          {
-            type: 'command',
-            command: '.claude/hooks/post-compact-reminder.sh',
-            timeout: 30,
-            statusMessage: 'Restoring context after compaction...'
-          }
-        ]
-      }
     ]
   };
 
@@ -123,6 +111,69 @@ export function migrateSettingsHooks(clientDir) {
   }
 }
 
+/**
+ * One-time migration: repair the post-compaction context hook.
+ *
+ * Earlier versions shipped a `PostCompact` hook event. That key is not a valid
+ * hook event in Claude Code's settings schema, so the whole settings.local.json
+ * file is rejected — silently disabling permissions and every other hook. It
+ * also never worked for its purpose: PostCompact does not inject stdout back
+ * into the session. The correct mechanism is a SessionStart hook with
+ * matcher "compact", whose stdout is added to context after compaction.
+ *
+ * This removes any `PostCompact` event and ensures the SessionStart "compact"
+ * group exists, pointing at the same post-compact-reminder.sh script.
+ * settings.local.json is config-only (never overwritten by update), so existing
+ * clients can only be repaired here.
+ */
+export function migrateCompactSessionHook(clientDir) {
+  const settingsPath = join(clientDir, '.claude', 'settings.local.json');
+  if (!existsSync(settingsPath)) return;
+
+  const COMPACT_SCRIPT = '.claude/hooks/post-compact-reminder.sh';
+  const compactGroup = {
+    matcher: 'compact',
+    hooks: [
+      {
+        type: 'command',
+        command: COMPACT_SCRIPT,
+        timeout: 30,
+        statusMessage: 'Restoring context after compaction...'
+      }
+    ]
+  };
+
+  try {
+    const settings = JSON.parse(readFileSync(settingsPath, 'utf8'));
+    if (!settings.hooks) settings.hooks = {};
+
+    let changed = false;
+
+    // Remove the invalid PostCompact event if present.
+    if (settings.hooks.PostCompact) {
+      delete settings.hooks.PostCompact;
+      changed = true;
+    }
+
+    // Ensure a SessionStart "compact" group running the reminder script exists.
+    const sessionStart = settings.hooks.SessionStart || [];
+    const hasCompact = sessionStart.some(group =>
+      (group.hooks || []).some(h => h.command === COMPACT_SCRIPT)
+    );
+    if (!hasCompact) {
+      settings.hooks.SessionStart = [...sessionStart, compactGroup];
+      changed = true;
+    }
+
+    if (!changed) return;
+
+    writeFileSync(settingsPath, JSON.stringify(settings, null, 2) + '\n');
+    logger.info('  Repaired post-compaction hook (PostCompact -> SessionStart "compact")');
+  } catch {
+    // Don't fail update over migration
+  }
+}
+
 /**
  * One-time migration: normalize backslash manifest keys to forward slashes.
  * On Windows, getAllFiles() returned backslash paths which were stored as manifest keys.
@@ -198,6 +249,7 @@ export async function runAllMigrations(clientDir, manifest) {
   await ensureMemoryFolder(clientDir);
   migrateSettingsDenyRules(clientDir);
   migrateSettingsHooks(clientDir);
+  migrateCompactSessionHook(clientDir);
   migrateManifestKeys(manifest);
   migrateConfigManagedTypes(manifest);
   migrateTmpDataFiles(clientDir, manifest);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ppcos",
-  "version": "1.3.1",
+  "version": "1.4.0",
   "description": "CLI tool to manage Google Ads AI workflow skills and agents for Claude Code",
   "type": "module",
   "bin": {