claudecode-omc 5.5.2 → 5.6.0

package/src/cli/source.js

@@ -4,7 +4,8 @@ const path = require('path');
 const fs = require('fs');
 const fsp = require('fs/promises');
 const { readConfig, setActiveSource, recordSync, addSource, removeSource } = require('../config/sources');
-const { getProjectRoot, getSourceArtifactDir, getSyncTargetDir, getSyncTempDir } = require('../config/paths');
+const { getProjectRoot, getSourceArtifactDir, getSyncTargetDir, getSyncTempDir, getSourceMetadataDir } = require('../config/paths');
+const { buildSourceCatalog } = require('../catalog/source-catalog');
 
 async function copyDirRecursive(src, dest) {
   await fsp.mkdir(dest, { recursive: true });
@@ -20,6 +21,11 @@ async function copyDirRecursive(src, dest) {
   }
 }
 
+function copyFileRecursive(src, dest) {
+  return fsp.mkdir(path.dirname(dest), { recursive: true })
+    .then(() => fsp.copyFile(src, dest));
+}
+
 function parseMappingFlag(mappingFlag) {
   if (!mappingFlag) return {};
 
@@ -82,6 +88,30 @@ async function syncRemoteSource(sourceName, sourceConfig, root) {
       }
     }
 
+    const metadataDir = getSourceMetadataDir(sourceName, root);
+    await fsp.rm(path.join(metadataDir, 'manifests'), { recursive: true, force: true });
+    const manifestFiles = [];
+    for (const manifestPath of sourceConfig.manifests || []) {
+      const srcPath = path.join(tmpDir, manifestPath);
+      if (!fs.existsSync(srcPath) || !fs.statSync(srcPath).isFile()) continue;
+      const destPath = path.join(metadataDir, 'manifests', manifestPath);
+      await copyFileRecursive(srcPath, destPath);
+      manifestFiles.push(manifestPath);
+    }
+
+    await fsp.mkdir(metadataDir, { recursive: true });
+    await fsp.writeFile(path.join(metadataDir, 'bundle.json'), JSON.stringify({
+      syncedAt: new Date().toISOString(),
+      sourceName,
+      remote: sourceConfig.remote,
+      ref: sourceConfig.ref,
+      kind: sourceConfig.kind || 'content-repo',
+      harnesses: sourceConfig.harnesses || ['claude'],
+      artifacts: sourceConfig.artifacts || [],
+      manifests: manifestFiles,
+      profiles: sourceConfig.profiles || [],
+    }, null, 2) + '\n', 'utf8');
+
     return true;
   } finally {
     if (fs.existsSync(tmpDir)) {
@@ -105,6 +135,19 @@ async function source(args, flags = {}) {
         console.log(`${marker} ${name} (priority ${src.priority})`);
         console.log(`  ${location}`);
         console.log(`  artifacts: ${(src.artifacts || []).join(', ')}`);
+        console.log(`  kind: ${src.kind || 'content-repo'}`);
+        console.log(`  installMode: ${src.installMode || 'auto'}`);
+        console.log(`  harnesses: ${(src.harnesses || ['claude']).join(', ')}`);
+        console.log(`  profiles: ${(src.profiles || []).join(', ')}`);
+        if (src.appliedProfile) {
+          console.log(`  appliedProfile: ${src.appliedProfile}`);
+        }
+        if (src.allowlist && Object.keys(src.allowlist).length > 0) {
+          const allowlist = Object.entries(src.allowlist)
+            .map(([artifactType, names]) => `${artifactType}(${names.length})`)
+            .join(', ');
+          console.log(`  allowlist: ${allowlist}`);
+        }
         if (src.role) {
           console.log(`  role: ${src.role}`);
         }
@@ -114,6 +157,9 @@ async function source(args, flags = {}) {
             .join(', ');
           console.log(`  mapping: ${mapping}`);
         }
+        if (src.manifests && src.manifests.length > 0) {
+          console.log(`  manifests: ${src.manifests.join(', ')}`);
+        }
         console.log('');
       }
       break;
@@ -123,7 +169,7 @@ async function source(args, flags = {}) {
       const name = args[1];
       const remote = args[2];
       if (!name || !remote) {
-        throw new Error('Usage: omc-manage source add <name> <remote-url> [--ref main] [--priority N] [--artifacts skills,agents,guidelines] [--mapping guidelines=CLAUDE.md] [--role guidelines]');
+        throw new Error('Usage: omc-manage source add <name> <remote-url> [--ref main] [--priority N] [--artifacts skills,agents,guidelines] [--mapping guidelines=CLAUDE.md] [--role guidelines] [--kind distribution-repo] [--install-mode planned] [--harnesses claude,codex] [--manifests package.json,agent.yaml] [--profiles claude-runtime,reference-only]');
       }
       await addSource(name, remote, {
         ref: flags.ref,
@@ -131,6 +177,11 @@ async function source(args, flags = {}) {
         artifacts: flags.artifacts,
         mapping: parseMappingFlag(flags.mapping),
         role: flags.role,
+        kind: flags.kind,
+        installMode: flags.installMode,
+        harnesses: flags.harnesses,
+        manifests: flags.manifests,
+        profiles: flags.profiles,
       });
       console.log(`Source "${name}" added.`);
       console.log(`Run "omc-manage source sync ${name}" to fetch artifacts.`);
@@ -198,6 +249,17 @@ async function source(args, flags = {}) {
 
       for (const [name, src] of Object.entries(config.sources)) {
         console.log(`[${name}] (priority ${src.priority})`);
+        console.log(`  kind: ${src.kind || 'content-repo'}`);
+        console.log(`  installMode: ${src.installMode || 'auto'}`);
+        if (src.appliedProfile) {
+          console.log(`  appliedProfile: ${src.appliedProfile}`);
+        }
+        if (src.allowlist && Object.keys(src.allowlist).length > 0) {
+          const allowlist = Object.entries(src.allowlist)
+            .map(([artifactType, names]) => `${artifactType}(${names.length})`)
+            .join(', ');
+          console.log(`  allowlist: ${allowlist}`);
+        }
         if (src.role) {
           console.log(`  role: ${src.role}`);
         }
@@ -224,8 +286,53 @@ async function source(args, flags = {}) {
       break;
     }
 
+    case 'inspect': {
+      const name = args[1];
+      if (!name || !config.sources[name]) {
+        const validNames = Object.keys(config.sources).join(', ');
+        throw new Error(`Invalid source. Use: ${validNames}`);
+      }
+
+      const catalog = await buildSourceCatalog(name, getProjectRoot());
+      if (flags.json) {
+        console.log(JSON.stringify(catalog, null, 2));
+        break;
+      }
+
+      console.log(`Source Inspection: ${name}`);
+      console.log('='.repeat(40));
+      console.log(`kind: ${catalog.kind}`);
+      console.log(`installMode: ${catalog.installMode}`);
+      console.log(`role: ${catalog.role || 'installable'}`);
+      console.log(`harnesses: ${catalog.harnesses.join(', ')}`);
+      console.log(`profiles: ${catalog.profiles.join(', ')}`);
+      console.log(`manifests discovered: ${catalog.manifests.filter(m => m.present).length}/${catalog.manifests.length}`);
+      console.log('');
+
+      console.log('Surfaces:');
+      for (const surface of catalog.surfaces) {
+        const bits = [
+          surface.category,
+          surface.harness,
+          surface.installable ? 'installable' : 'non-installable',
+        ];
+        if (surface.artifactType) bits.push(`artifact=${surface.artifactType}`);
+        if (surface.count != null) bits.push(`count=${surface.count}`);
+        console.log(`  ${surface.name} — ${bits.join(', ')}`);
+      }
+
+      if (catalog.warnings.length > 0) {
+        console.log('');
+        console.log('Warnings:');
+        for (const warning of catalog.warnings) {
+          console.log(`  - ${warning}`);
+        }
+      }
+      break;
+    }
+
     default:
-      throw new Error(`Unknown subcommand: ${cmd}. Use: list, add, remove, set, sync, or status`);
+      throw new Error(`Unknown subcommand: ${cmd}. Use: list, add, remove, set, sync, status, or inspect`);
   }
 }
 

package/src/config/paths.js

@@ -9,6 +9,35 @@ function getProjectRoot() {
   return path.resolve(__dirname, '..', '..');
 }
 
+function getSourceRootDir(sourceName, root) {
+  root = root || getProjectRoot();
+
+  if (isDistributionMode()) {
+    if (sourceName === 'local') {
+      const userLocal = path.join(USER_DATA_DIR, 'local');
+      if (fs.existsSync(userLocal)) {
+        return userLocal;
+      }
+      return path.join(root, '.local');
+    }
+
+    const synced = path.join(USER_DATA_DIR, 'upstream', sourceName);
+    if (fs.existsSync(synced)) {
+      return synced;
+    }
+    return path.join(root, 'bundled', 'upstream', sourceName);
+  }
+
+  if (sourceName === 'local') {
+    return path.join(root, '.local');
+  }
+  return path.join(root, '.upstream', sourceName);
+}
+
+function getSourceMetadataDir(sourceName, root) {
+  return path.join(getSourceRootDir(sourceName, root), '.omc-source');
+}
+
 /**
  * Detect if running from an npm-installed package (no .git dir, has bundled/)
  */
@@ -222,6 +251,8 @@ function getReportDir(root) {
 module.exports = {
   getProjectRoot,
   isDistributionMode,
+  getSourceRootDir,
+  getSourceMetadataDir,
   getSourceArtifactDir,
   getSyncTargetDir,
   getSyncTempDir,

package/src/config/sources.js

@@ -6,6 +6,14 @@ const os = require('os');
 const CONFIG_DIR = path.join(os.homedir(), '.omc-manage');
 const CONFIG_PATH = path.join(CONFIG_DIR, 'sources.json');
 
+const DEFAULT_DISTRIBUTION_MANIFESTS = [
+  'package.json',
+  '.claude-plugin/plugin.json',
+  'agent.yaml',
+];
+
+const DEFAULT_INSTALL_PROFILES = ['claude-runtime', 'reference-only'];
+
 function getDefaultConfig() {
   return {
     active: 'local',
@@ -14,6 +22,9 @@ function getDefaultConfig() {
         path: '.local',
         priority: 1,
         artifacts: ['skills', 'agents', 'hooks', 'commands', 'guidelines', 'settings', 'hud'],
+        kind: 'content-repo',
+        harnesses: ['claude'],
+        profiles: DEFAULT_INSTALL_PROFILES,
       },
       'oh-my-claudecode': {
         remote: 'https://github.com/Yeachan-Heo/oh-my-claudecode.git',
@@ -25,6 +36,9 @@ function getDefaultConfig() {
           agents: 'agents',
           hooks: 'hooks',
         },
+        kind: 'content-repo',
+        harnesses: ['claude'],
+        profiles: DEFAULT_INSTALL_PROFILES,
       },
       superpowers: {
         remote: 'https://github.com/obra/superpowers.git',
@@ -37,6 +51,9 @@ function getDefaultConfig() {
           hooks: 'hooks',
           commands: 'commands',
         },
+        kind: 'content-repo',
+        harnesses: ['claude'],
+        profiles: DEFAULT_INSTALL_PROFILES,
       },
       'anthropic-skills': {
         remote: 'https://github.com/anthropics/skills.git',
@@ -49,6 +66,9 @@ function getDefaultConfig() {
           template: 'template',
         },
         role: 'reference',
+        kind: 'content-repo',
+        harnesses: ['claude'],
+        profiles: ['reference-only'],
       },
     },
     lastSync: null,
@@ -60,19 +80,55 @@ function dedupeArtifacts(artifacts) {
   return [...new Set((artifacts || []).filter(Boolean))];
 }
 
+function dedupeStrings(values) {
+  return [...new Set((values || []).filter(Boolean))];
+}
+
+function normalizeAllowlist(allowlist) {
+  if (!allowlist || typeof allowlist !== 'object') return undefined;
+  const next = {};
+  for (const [artifactType, names] of Object.entries(allowlist)) {
+    const uniqueNames = dedupeStrings(Array.isArray(names) ? names : []);
+    if (uniqueNames.length > 0) {
+      next[artifactType] = uniqueNames.sort();
+    }
+  }
+  return Object.keys(next).length > 0 ? next : undefined;
+}
+
+function normalizeSourceConfig(name, source) {
+  source.artifacts = dedupeArtifacts(source.artifacts);
+  source.mapping = source.mapping || {};
+
+  // Migrate local prompt guidance from legacy claude-md to guidelines.
+  if (name === 'local' && source.artifacts.includes('claude-md') && !source.artifacts.includes('guidelines')) {
+    source.artifacts = source.artifacts.map((artifact) => (
+      artifact === 'claude-md' ? 'guidelines' : artifact
+    ));
+  }
+
+  source.kind = source.kind || 'content-repo';
+  source.installMode = source.installMode || (source.kind === 'distribution-repo' ? 'planned' : 'auto');
+  source.harnesses = dedupeStrings(source.harnesses || ['claude']);
+  source.manifests = dedupeStrings(source.manifests || (source.kind === 'distribution-repo'
+    ? DEFAULT_DISTRIBUTION_MANIFESTS
+    : []));
+  source.profiles = dedupeStrings(source.profiles || DEFAULT_INSTALL_PROFILES);
+  source.allowlist = normalizeAllowlist(source.allowlist);
+
+  if (source.role === 'reference' && !source.profiles.includes('reference-only')) {
+    source.profiles.push('reference-only');
+  }
+
+  return source;
+}
+
 function normalizeConfig(config) {
   const normalized = config || getDefaultConfig();
   normalized.sources = normalized.sources || {};
 
   for (const [name, source] of Object.entries(normalized.sources)) {
-    source.artifacts = dedupeArtifacts(source.artifacts);
-
-    // Migrate local prompt guidance from legacy claude-md to guidelines.
-    if (name === 'local' && source.artifacts.includes('claude-md') && !source.artifacts.includes('guidelines')) {
-      source.artifacts = source.artifacts.map((artifact) => (
-        artifact === 'claude-md' ? 'guidelines' : artifact
-      ));
-    }
+    normalizeSourceConfig(name, source);
   }
 
   return normalized;
@@ -128,14 +184,19 @@ async function addSource(name, remote, options = {}) {
   if (config.sources[name]) {
     throw new Error(`Source "${name}" already exists`);
   }
-  config.sources[name] = {
+  config.sources[name] = normalizeSourceConfig(name, {
     remote,
     ref: options.ref || 'main',
     priority: options.priority || Object.keys(config.sources).length + 1,
     artifacts: options.artifacts || ['skills'],
     mapping: options.mapping || {},
     role: options.role,
-  };
+    kind: options.kind,
+    installMode: options.installMode,
+    harnesses: options.harnesses,
+    manifests: options.manifests,
+    profiles: options.profiles,
+  });
   await writeConfig(config);
 }
 
@@ -151,6 +212,32 @@ async function removeSource(name) {
   await writeConfig(config);
 }
 
+async function updateSource(name, mutator) {
+  const config = readConfig();
+  if (!config.sources[name]) {
+    throw new Error(`Source "${name}" not found`);
+  }
+
+  const current = { ...config.sources[name] };
+  const next = mutator ? (await mutator(current, config)) : current;
+  config.sources[name] = normalizeSourceConfig(name, next || current);
+  await writeConfig(config);
+  return config.sources[name];
+}
+
+function getSourceAllowlist(source, artifactType) {
+  if (!source || !source.allowlist) return null;
+  const names = source.allowlist[artifactType];
+  if (!Array.isArray(names) || names.length === 0) return null;
+  return new Set(names);
+}
+
+function filterItemsByAllowlist(source, artifactType, items) {
+  const allowlist = getSourceAllowlist(source, artifactType);
+  if (!allowlist) return items;
+  return items.filter(item => allowlist.has(item.name));
+}
+
 module.exports = {
   readConfig,
   writeConfig,
@@ -159,7 +246,13 @@ module.exports = {
   recordSync,
   addSource,
   removeSource,
+  updateSource,
+  getSourceAllowlist,
+  filterItemsByAllowlist,
   normalizeConfig,
+  normalizeSourceConfig,
+  DEFAULT_DISTRIBUTION_MANIFESTS,
+  DEFAULT_INSTALL_PROFILES,
   CONFIG_DIR,
   CONFIG_PATH,
 };

package/src/merge/hook-merger.js

@@ -14,6 +14,7 @@ function loadHookFilesFromSource(sourceDir, sourceName) {
   for (const entry of entries) {
     if (!entry.isFile()) continue;
     if (entry.name === 'hooks.json') continue;
+    if (entry.name.toLowerCase().endsWith('.md')) continue;
     const fullPath = path.join(sourceDir, entry.name);
     items.push({
       name: entry.name,