@ghl-ai/aw 0.1.58-beta.0 → 0.1.59

package/commands/push.mjs

@@ -662,6 +662,10 @@ async function publishProjectAwDocs(cwd, home, dryRun, scope = null) {
 // Auto-generate a branch name from the files being pushed.
 function generateBranchName(files, extraPaths = []) {
   const shortId = Date.now().toString(36).slice(-5);
+  const branchSafe = (value) => String(value || '')
+    .replace(/\//g, '-')
+    .replace(/[^A-Za-z0-9._-]+/g, '-')
+    .replace(/^-+|-+$/g, '') || 'item';
 
   if (files.length === 0 && extraPaths.length > 0) {
     return `${managedBranchPrefix(extraPaths)}-${shortId}`;
@@ -675,8 +679,8 @@ function generateBranchName(files, extraPaths = []) {
 
   if (files.length === 1) {
     const f = files[0];
-    const nsSlug = f.namespace.replace(/\//g, '-');
-    return `${prefix}/${nsSlug}-${f.type}-${f.slug}-${shortId}`;
+    const nsSlug = branchSafe(f.namespace);
+    return `${prefix}/${nsSlug}-${f.type}-${branchSafe(f.slug)}-${shortId}`;
   }
 
   if (namespaces.length === 1) {
@@ -698,7 +702,7 @@ function singular(type, count) {
 function parseCommitFiles(commits) {
   const result = [];
   for (const c of commits) {
-    const m = c.message.match(/registry:\s+(add|remove|sync)\s+(agents|skills|commands|evals)\/([\w-]+)\s+(?:to|from)\s+([\w/]+)/);
+    const m = c.message.match(/registry:\s+(add|remove|sync)\s+(agents|skills|commands|evals)\/([\w/-]+)\s+(?:to|from)\s+([\w/]+)/);
     if (m) result.push({ verb: m[1], type: m[2], slug: m[3], namespace: m[4] });
   }
   return result;
@@ -900,13 +904,40 @@ function collectBatchFiles(folderAbsPath, registrySubDir) {
 
 // ── Parse type/namespace/slug from a registry-relative path ──────────
 
-function parseRegistryPath(relPath) {
+function findSkillSlugParts(namespaceParts, skillParts, registrySubDir) {
+  if (skillParts.length === 0) return [];
+
+  if (registrySubDir) {
+    for (let i = skillParts.length; i > 0; i--) {
+      const candidate = join(registrySubDir, ...namespaceParts, 'skills', ...skillParts.slice(0, i));
+      if (existsSync(join(candidate, 'SKILL.md'))) return skillParts.slice(0, i);
+    }
+  }
+
+  const skillMdIdx = skillParts.findIndex(part => part === 'SKILL' || part === 'SKILL.md');
+  if (skillMdIdx > 0) return skillParts.slice(0, skillMdIdx);
+
+  return [skillParts[0]];
+}
+
+function parseRegistryPath(relPath, registrySubDir = null) {
   const parts = relPath.split('/');
   for (let i = 0; i < parts.length; i++) {
     if (PUSHABLE_TYPES.includes(parts[i]) && i + 1 < parts.length) {
+      const namespaceParts = parts.slice(0, i);
+      if (parts[i] === 'skills') {
+        const skillParts = parts.slice(i + 1).map(part => part.replace(/\.md$/, ''));
+        const slugParts = findSkillSlugParts(namespaceParts, skillParts, registrySubDir);
+        return {
+          type: parts[i],
+          namespace: namespaceParts.join('/'),
+          slug: slugParts.join('/'),
+        };
+      }
+
       return {
         type: parts[i],
-        namespace: parts.slice(0, i).join('/'),
+        namespace: namespaceParts.join('/'),
         slug: parts[i + 1].replace(/\.md$/, ''),
       };
     }
@@ -1198,7 +1229,7 @@ export async function pushCommand(args) {
 
     if (staged.length > 0 || extraStaged.length > 0) {
       const files = staged.map(f => {
-        const meta = parseRegistryPath(f.registryPath);
+        const meta = parseRegistryPath(f.registryPath, registrySubDir);
         const parts = f.registryPath.split('/');
         return {
           absPath: join(awHome, f.path),
@@ -1264,7 +1295,7 @@ export async function pushCommand(args) {
 
     const files = allEntries
       .map(f => {
-        const meta = parseRegistryPath(f.registryPath);
+        const meta = parseRegistryPath(f.registryPath, registrySubDir);
         const parts = f.registryPath.split('/');
         return {
           absPath: join(awHome, f.path),
@@ -1369,7 +1400,7 @@ export async function pushCommand(args) {
     const deletedInFolder = folderChanges.deleted
       .filter(e => !folderPrefix || e.registryPath.startsWith(folderPrefix))
       .map(e => {
-        const meta = parseRegistryPath(e.registryPath);
+        const meta = parseRegistryPath(e.registryPath, registrySubDir);
         const parts = e.registryPath.split('/');
         return {
           absPath: join(awHome, e.path),
@@ -1396,13 +1427,8 @@ export async function pushCommand(args) {
   }
 
   // Single file input
-  const regParts = resolved.registryPath.split('/');
-  let typeIdx = -1;
-  for (let i = regParts.length - 1; i >= 0; i--) {
-    if (PUSHABLE_TYPES.includes(regParts[i])) { typeIdx = i; break; }
-  }
-
-  if (typeIdx === -1 || typeIdx + 1 >= regParts.length) {
+  const meta = parseRegistryPath(resolved.registryPath, registrySubDir);
+  if (!meta?.type || !meta.slug) {
     fmt.cancel([
       `Invalid push path: ${chalk.red(resolved.registryPath)}`,
       '',
@@ -1416,14 +1442,16 @@ export async function pushCommand(args) {
     return;
   }
 
-  const namespaceParts = regParts.slice(0, typeIdx);
-  const parentDir = regParts[typeIdx];
-  const slug = regParts[typeIdx + 1];
-  const namespacePath = namespaceParts.join('/');
+  const parentDir = meta.type;
+  const slug = meta.slug;
+  const namespacePath = meta.namespace;
   const isDir = !isDeletedFile && statSync(absPath).isDirectory();
+  const registryPathWithExt = resolved.registryPath.endsWith('.md')
+    ? resolved.registryPath
+    : `${resolved.registryPath}.md`;
   const registryTarget = isDir
     ? `${REGISTRY_DIR}/${namespacePath}/${parentDir}/${slug}`
-    : `${REGISTRY_DIR}/${namespacePath}/${parentDir}/${slug}.md`;
+    : `${REGISTRY_DIR}/${registryPathWithExt}`;
 
   // Check if the file actually has changes before trying to commit
   const singleFileChanges = detectChanges(awHome, REGISTRY_DIR);

package/ecc.mjs

@@ -12,7 +12,12 @@ import { applyStoredStartupPreferences } from "./startup.mjs";
 
 const AW_ECC_REPO_SSH = "git@github.com:shreyansh-ghl/aw-ecc.git";
 const AW_ECC_REPO_HTTPS = "https://github.com/shreyansh-ghl/aw-ecc.git";
-export const AW_ECC_TAG = "v1.4.62";
+export const AW_ECC_TAG = "v1.4.63";
+const REQUIRED_ECC_FILES = [
+  "package.json",
+  "scripts/install-apply.js",
+  "scripts/sync-ecc-to-codex.sh",
+];
 
 const MARKETPLACE_NAME = "aw-marketplace";
 const PLUGIN_KEY = `aw@${MARKETPLACE_NAME}`;
@@ -85,11 +90,15 @@ async function cloneWithRefAsync(url, ref, dest) {
   }
 }
 
+function hasRequiredEccFiles(dest) {
+  return REQUIRED_ECC_FILES.every((relPath) => existsSync(join(dest, relPath)));
+}
+
 async function cloneOrUpdateAsync(tag, dest) {
   const overrideUrl = process.env.AW_ECC_CLONE_URL;
   const overrideRef = process.env.AW_ECC_CLONE_REF?.trim();
 
-  if (existsSync(join(dest, ".git"))) {
+  if (existsSync(join(dest, ".git")) && hasRequiredEccFiles(dest)) {
     try {
       if (!overrideUrl && !overrideRef) {
         await runA(`git -C ${dest} fetch --quiet --depth 1 origin tag ${tag}`);
@@ -102,6 +111,7 @@ async function cloneOrUpdateAsync(tag, dest) {
       }
       // Restore working tree — pruneStaleHooks may have deleted tracked files
       await runA(`git -C ${dest} checkout -- .`);
+      if (!hasRequiredEccFiles(dest)) throw new Error("aw-ecc checkout is missing required files");
       return;
     } catch { /* fall through to fresh clone */ }
   }
@@ -188,7 +198,7 @@ function cloneOrUpdate(tag, dest) {
   // AW_ECC_CLONE_REF lets aw init install ECC from a non-default branch or tag.
   const overrideRef = process.env.AW_ECC_CLONE_REF?.trim();
 
-  if (existsSync(join(dest, ".git"))) {
+  if (existsSync(join(dest, ".git")) && hasRequiredEccFiles(dest)) {
     try {
       if (!overrideUrl && !overrideRef) {
         run(`git -C ${dest} fetch --quiet --depth 1 origin tag ${tag}`);
@@ -201,6 +211,7 @@ function cloneOrUpdate(tag, dest) {
       }
       // Restore working tree — pruneStaleHooks may have deleted tracked files
       run(`git -C ${dest} checkout -- .`);
+      if (!hasRequiredEccFiles(dest)) throw new Error("aw-ecc checkout is missing required files");
       return;
     } catch { /* fall through to fresh clone */ }
   }

package/git.mjs

@@ -453,6 +453,7 @@ export function detectChanges(awHome, registryDir) {
     if (!line || line.length < 3) continue;
     const xy = line.slice(0, 2);
     const filePath = line.slice(3).trim();
+    if (filePath === `${registryDir}/.aw-upgrade.log`) continue;
     const registryPath = filePath.startsWith(registryDir + '/') ? filePath.slice(registryDir.length + 1) : filePath;
     const entry = { path: filePath, registryPath };
 
@@ -485,6 +486,7 @@ export function getStagedFiles(awHome, registryDir) {
       if (!line) continue;
       const [status, filePath] = line.split('\t');
       if (!filePath?.startsWith(registryDir + '/')) continue;
+      if (filePath === `${registryDir}/.aw-upgrade.log`) continue;
       result.push({
         path: filePath,
         registryPath: filePath.slice(registryDir.length + 1),