ppcos 1.0.4 → 1.0.6

package/lib/commands/update.js

@@ -218,6 +218,28 @@ function migrateConfigManagedTypes(manifest) {
   }
 }
 
+/**
+ * One-time migration: remove leaked data files from skill tmp/ directories.
+ * Data files were accidentally included in .claude-base.zip and distributed
+ * to clients. This removes them and cleans up the manifest.
+ */
+function migrateTmpDataFiles(clientDir, manifest) {
+  const keepFiles = new Set(['.gitkeep', '.gitignore']);
+  let removed = 0;
+  for (const [path] of Object.entries(manifest.managedFiles)) {
+    if (!/\/tmp\//.test(path)) continue;
+    const filename = path.split('/').pop();
+    if (keepFiles.has(filename)) continue;
+    const fullPath = join(clientDir, path);
+    if (existsSync(fullPath)) unlinkSync(fullPath);
+    delete manifest.managedFiles[path];
+    removed++;
+  }
+  if (removed > 0) {
+    logger.info(`  Removed ${removed} tmp data file${removed !== 1 ? 's' : ''} from tmp/`);
+  }
+}
+
 /**
  * Find all client directories with .managed.json
  * @returns {string[]} Array of client names
@@ -383,6 +405,7 @@ async function updateClient(clientName, basePathOrOptions = {}, options = {}) {
   migrateSettingsHooks(clientDir);
   migrateManifestKeys(manifest);
   migrateConfigManagedTypes(manifest);
+  migrateTmpDataFiles(clientDir, manifest);
 
   // Check if already up to date
   if (currentVersion === packageVersion) {

package/lib/utils/fs-helpers.js

@@ -2,11 +2,16 @@
  * File system helper utilities
  */
 
-import { readdir, stat, mkdir, copyFile, access } from 'node:fs/promises';
-import { readdirSync, statSync, mkdirSync, copyFileSync, existsSync } from 'node:fs';
+import { readdir, stat, mkdir, copyFile, access, chmod } from 'node:fs/promises';
+import { readdirSync, statSync, mkdirSync, copyFileSync, existsSync, chmodSync } from 'node:fs';
 import { join, relative, dirname } from 'node:path';
 import { constants } from 'node:fs';
 
+// Directories to skip during template file enumeration.
+// tmp/ dirs hold generated runtime data (analysis output, caches)
+// that must never be distributed to clients.
+const EXCLUDED_DIR_NAMES = new Set(['tmp']);
+
 /**
  * Check if a file exists (async)
  * @param {string} filePath - Path to check
@@ -36,14 +41,15 @@ export function fileExistsSync(filePath) {
  * @param {string} [baseDir] - Base directory for relative paths
  * @returns {Promise<string[]>} Array of relative file paths
  */
-export async function getAllFiles(dir, baseDir = dir) {
+export async function getAllFiles(dir, baseDir = dir, excludeDirs = EXCLUDED_DIR_NAMES) {
   const files = [];
   const entries = await readdir(dir, { withFileTypes: true });
 
   for (const entry of entries) {
     const fullPath = join(dir, entry.name);
     if (entry.isDirectory()) {
-      files.push(...await getAllFiles(fullPath, baseDir));
+      if (excludeDirs.has(entry.name)) continue;
+      files.push(...await getAllFiles(fullPath, baseDir, excludeDirs));
     } else {
       files.push(relative(baseDir, fullPath).replace(/\\/g, '/'));
     }
@@ -58,14 +64,15 @@ export async function getAllFiles(dir, baseDir = dir) {
  * @param {string} [baseDir] - Base directory for relative paths
  * @returns {string[]} Array of relative file paths
  */
-export function getAllFilesSync(dir, baseDir = dir) {
+export function getAllFilesSync(dir, baseDir = dir, excludeDirs = EXCLUDED_DIR_NAMES) {
   const files = [];
   const entries = readdirSync(dir, { withFileTypes: true });
 
   for (const entry of entries) {
     const fullPath = join(dir, entry.name);
     if (entry.isDirectory()) {
-      files.push(...getAllFilesSync(fullPath, baseDir));
+      if (excludeDirs.has(entry.name)) continue;
+      files.push(...getAllFilesSync(fullPath, baseDir, excludeDirs));
     } else {
       files.push(relative(baseDir, fullPath).replace(/\\/g, '/'));
     }
@@ -83,6 +90,8 @@ export async function copyFileWithDirs(src, dest) {
   const destDir = dirname(dest);
   await mkdir(destDir, { recursive: true });
   await copyFile(src, dest);
+  const srcStats = await stat(src);
+  await chmod(dest, srcStats.mode);
 }
 
 /**
@@ -96,6 +105,8 @@ export function copyFileWithDirsSync(src, dest) {
     mkdirSync(destDir, { recursive: true });
   }
   copyFileSync(src, dest);
+  const srcStats = statSync(src);
+  chmodSync(dest, srcStats.mode);
 }
 
 /**

package/package.json

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