@maccesar/aiskills 1.10.0 → 1.12.0

package/README.md

@@ -40,7 +40,7 @@ claude   # or gemini, or codex
 
 Installed files:
 - All skills to `~/.agents/skills/`
-- Platform symlinks in `~/.claude/skills/`, `~/.gemini/skills/`, or `~/.codex/skills/`
+- Platform symlinks in `~/.claude/skills/` and `~/.gemini/skills/` (Codex CLI auto-discovers from `~/.agents/skills/` — no symlink needed)
 
 ### Which option should I use?
 
@@ -59,7 +59,7 @@ Installed files:
 | --------------------------------------------------------- | --------- | ------------------- |
 | [Claude Code](https://claude.ai/claude-code)              | Supported | `~/.claude/skills/` |
 | [Gemini CLI](https://github.com/google-gemini/gemini-cli) | Supported | `~/.gemini/skills/` |
-| [Codex CLI](https://developers.openai.com/codex/cli/)     | Supported | `~/.codex/skills/`  |
+| [Codex CLI](https://developers.openai.com/codex/cli/)     | Supported | `~/.agents/skills/` (auto-discovered) |
 
 All three platforms use the same Agent Skills format: a `SKILL.md` file with YAML frontmatter that tells the assistant when to use the skill and what to do.
 

package/lib/cleanup.js

@@ -9,6 +9,7 @@ import {
   LEGACY_COMMANDS,
   getAgentsSkillsDir,
   getClaudeCommandsDir,
+  getCodexSkillsDir,
 } from './config.js';
 import { detectPlatforms } from './platform.js';
 import { existsSync, lstatSync, rmSync } from 'fs';
@@ -53,6 +54,14 @@ export function removeLegacySkillSymlinks(platformSkillsDir) {
   return removeSkillSymlinks(platformSkillsDir, { legacyOnly: true });
 }
 
+// Codex reads skills from the canonical ~/.agents/skills/, so any aiskills-managed
+// symlinks at ~/.codex/skills/ from earlier versions are redundant and should be
+// cleaned up. Targets both active and legacy skill names.
+export function removeCodexRedundantSymlinks(baseDir) {
+  const codexSkillsDir = getCodexSkillsDir(baseDir);
+  return removeSkillSymlinks(codexSkillsDir);
+}
+
 export function removeSkills(baseDir, options = {}) {
   const skillsDir = getAgentsSkillsDir(baseDir);
   const skillList = getSkillList(options);
@@ -74,6 +83,8 @@ export function cleanupLegacyArtifacts(baseDir) {
     removeLegacySkillSymlinks(platform.skillsDir);
   }
 
+  removeCodexRedundantSymlinks(baseDir);
+
   if (baseDir) {
     removeSkills(undefined, { legacyOnly: true });
     removeCommands(undefined, { legacyOnly: true });
@@ -81,6 +92,7 @@ export function cleanupLegacyArtifacts(baseDir) {
     for (const platform of globalPlatforms) {
       removeLegacySkillSymlinks(platform.skillsDir);
     }
+    removeCodexRedundantSymlinks(undefined);
   }
 }
 
@@ -89,6 +101,7 @@ export default {
   getCommandList,
   removeSkillSymlinks,
   removeLegacySkillSymlinks,
+  removeCodexRedundantSymlinks,
   removeSkills,
   removeCommands,
   cleanupLegacyArtifacts,

package/lib/commands/skills.js

@@ -86,7 +86,8 @@ export async function skillsCommand(options) {
 
   if (detectedPlatforms.length === 0 && !options.path && !isLocal) {
     console.log(chalk.yellow('No AI coding assistants detected globally.'));
-    console.log('Install one of: Claude Code, Gemini CLI, or Codex CLI');
+    console.log('Install one of: Claude Code or Gemini CLI.');
+    console.log('(Codex CLI is supported via the canonical ~/.agents/skills/ — no platform-specific symlink needed.)');
     console.log('Or use: aiskills install --local');
     process.exit(1);
   }

package/lib/config.js

@@ -68,12 +68,9 @@ export const getPlatforms = (baseDir = os.homedir()) => [
     skillsDir: getGeminiSkillsDir(baseDir),
     configDir: path.join(baseDir, '.gemini'),
   },
-  {
-    name: 'codex',
-    displayName: 'Codex CLI',
-    skillsDir: getCodexSkillsDir(baseDir),
-    configDir: path.join(baseDir, '.codex'),
-  },
+  // Codex CLI is intentionally omitted: it auto-discovers skills from the
+  // canonical ~/.agents/skills/ per the agentskills.io standard, so a
+  // platform-specific symlink at ~/.codex/skills/ would be redundant.
 ];
 
 // API configuration

package/lib/platform.js

@@ -26,7 +26,7 @@ export function detectPlatforms(baseDir) {
 
 /**
  * Get platform by name
- * @param {string} name - Platform name (claude, gemini, codex)
+ * @param {string} name - Platform name (claude, gemini)
  * @param {string} baseDir - Optional base directory
  * @returns {Object|null} Platform object or null
  */

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@maccesar/aiskills",
-  "version": "1.10.0",
+  "version": "1.12.0",
   "description": "AI coding assistant skills for Claude Code, Gemini CLI, and Codex CLI",
   "type": "module",
   "bin": {

package/skills/stitch-showcase/references/index.html

@@ -186,13 +186,13 @@
       // Update badge text
       viewBadge.textContent = currentView === 'mobile' ? 'Mobile' : 'WEB';
 
-      // Update toggle icon: show the icon of the OTHER mode (what you'll switch to)
+      // Update toggle icon: show the icon of the CURRENT mode (matches the badge)
       if (currentView === 'mobile') {
-        iconPhone.classList.add('hidden');
-        iconMonitor.classList.remove('hidden');
-      } else {
         iconPhone.classList.remove('hidden');
         iconMonitor.classList.add('hidden');
+      } else {
+        iconPhone.classList.add('hidden');
+        iconMonitor.classList.remove('hidden');
       }
     }
     applyViewMode();

package/skills/stitch-showcase/references/viewer.html

@@ -219,6 +219,21 @@
       var frameMobile = document.getElementById('viewer-frame-mobile');
       var frameWeb = document.getElementById('viewer-frame-web');
 
+      // Inject scrollbar-hiding CSS into iframes after load (cosmetic, viewer-level only).
+      // Keeps source HTMLs untouched so they're clean for downstream handoff.
+      function injectScrollbarCSS(frame) {
+        try {
+          var doc = frame.contentDocument || frame.contentWindow.document;
+          if (!doc || doc.getElementById('__no-scrollbar-style')) return;
+          var style = doc.createElement('style');
+          style.id = '__no-scrollbar-style';
+          style.textContent = '*::-webkit-scrollbar{display:none!important}*{scrollbar-width:none!important;-ms-overflow-style:none!important}';
+          doc.head.appendChild(style);
+        } catch (e) { /* cross-origin or not yet ready */ }
+      }
+      frameMobile.addEventListener('load', function () { injectScrollbarCSS(frameMobile); });
+      frameWeb.addEventListener('load', function () { injectScrollbarCSS(frameWeb); });
+
       var screens = SCREENS;
       var currentIndex = 0;
       var VIEW_KEY = 'showcase-view-mode-' + '{{PROJECT_SLUG}}';
@@ -230,13 +245,13 @@
         root.classList.remove('view-mobile', 'view-web');
         root.classList.add('view-' + currentView);
 
-        // Toggle icon shows what you'll switch TO
+        // Toggle icon shows the CURRENT mode (matches the badge)
         if (currentView === 'mobile') {
-          iconPhone.classList.add('hidden');
-          iconMonitor.classList.remove('hidden');
-        } else {
           iconPhone.classList.remove('hidden');
           iconMonitor.classList.add('hidden');
+        } else {
+          iconPhone.classList.add('hidden');
+          iconMonitor.classList.remove('hidden');
         }
 
         // Recalc phone scaling when switching to mobile

package/skills/stitch-showcase/scripts/build_showcase.py

@@ -106,6 +106,18 @@ def build(source_dir: str, project_type: str = None, project_name: str = None) -
         print("Error: no screens found (no zips or folders with code.html).", file=sys.stderr)
         sys.exit(1)
 
+    # Copy shared asset folders (images, fonts, css, js) referenced by relative paths in HTMLs.
+    # Searches in source/, then source.parent/ (project root). Skips if already copied.
+    for asset_name in ('images', 'fonts', 'css', 'js'):
+        for candidate_root in (source, source.parent):
+            candidate = candidate_root / asset_name
+            if candidate.is_dir():
+                dest = assets_dir / asset_name
+                if not dest.exists():
+                    shutil.copytree(candidate, dest)
+                    print(f"  ✓ Copied shared {asset_name}/ from {candidate_root.name}/")
+                break
+
     # Enrich with DESIGN.md metadata
     screens = _enrich_screens(screens_raw, metadata, source)
 
@@ -160,6 +172,18 @@ def build_context(source_dir: str, project_type: str = None, project_name: str =
         print("Error: no screens found (no zips or folders with code.html).", file=sys.stderr)
         sys.exit(1)
 
+    # Copy shared asset folders (images, fonts, css, js) referenced by relative paths in HTMLs.
+    # Searches in source/, then source.parent/ (project root). Skips if already copied.
+    for asset_name in ('images', 'fonts', 'css', 'js'):
+        for candidate_root in (source, source.parent):
+            candidate = candidate_root / asset_name
+            if candidate.is_dir():
+                dest = assets_dir / asset_name
+                if not dest.exists():
+                    shutil.copytree(candidate, dest)
+                    print(f"  ✓ Copied shared {asset_name}/ from {candidate_root.name}/")
+                break
+
     # Enrich with DESIGN.md metadata
     screens = _enrich_screens(screens_raw, metadata, source)
 

package/skills/stitch-showcase/scripts/extract_zips.py

@@ -136,16 +136,11 @@ def _process_dir(dir_path: Path, slug: str, assets: Path) -> dict | None:
     }
 
 
-_NO_SCROLLBAR_CSS = '<style>*::-webkit-scrollbar{display:none!important}*{scrollbar-width:none!important;-ms-overflow-style:none!important}</style>'
-
 def _copy_html(src: Path, dst: Path) -> None:
-    """Copy HTML injecting scrollbar-hiding CSS into <head>."""
-    text = src.read_text(encoding="utf-8", errors="ignore")
-    if "</head>" in text:
-        text = text.replace("</head>", f"{_NO_SCROLLBAR_CSS}</head>", 1)
-    elif "<body" in text:
-        text = text.replace("<body", f"{_NO_SCROLLBAR_CSS}<body", 1)
-    dst.write_text(text, encoding="utf-8")
+    """Copy HTML verbatim. Scrollbar-hiding CSS is now injected at viewer level
+    (see references/viewer.html), keeping the source HTML untouched so it can be
+    handed off cleanly to downstream tools (e.g., Laravel Blade conversion)."""
+    shutil.copy2(src, dst)
 
 
 def _find_file(root: Path, filename: str) -> Path | None: