ux-toolkit 0.5.0 → 0.5.3

package/README.md

@@ -130,9 +130,9 @@ This installs to `.opencode/` in your current directory.
 | Command | Description |
 |---------|-------------|
 | `/ux-audit` | Comprehensive UX audit |
-| `/a11y-check` | Quick accessibility scan |
-| `/design-review` | Visual consistency check |
-| `/screenshot-review` | Visual review from screenshot |
+| `/ux-a11y-check` | Quick accessibility scan |
+| `/ux-design-review` | Visual consistency check |
+| `/ux-screenshot-review` | Visual review from screenshot |
 
 ## Usage Examples
 
@@ -143,7 +143,7 @@ This installs to `.opencode/` in your current directory.
 /ux-audit src/components/Button.tsx
 
 # Check accessibility
-/a11y-check src/pages/index.tsx
+/ux-a11y-check src/pages/index.tsx
 
 # Invoke agent directly
 @ux-auditor Review the login flow for usability issues

package/dist/cli.js

@@ -12,13 +12,40 @@ import { createInterface } from "readline";
 
 // src/installer.ts
 import { existsSync as existsSync2, mkdirSync, cpSync, readdirSync, rmSync, statSync } from "fs";
-import { join as join2, dirname as dirname2 } from "path";
+import { dirname as dirname2 } from "path";
 
 // src/paths.ts
 import { homedir, platform } from "os";
-import { join, dirname, resolve } from "path";
+import { dirname, resolve } from "path";
 import { fileURLToPath } from "url";
 import { existsSync } from "fs";
+
+// src/utils.ts
+import { join as pathJoin } from "path";
+function safeJoin(...paths) {
+  const stringPaths = [];
+  for (const p of paths) {
+    if (typeof p === "string") {
+      stringPaths.push(p);
+    } else if (p != null) {
+      const str = String(p);
+      if (str && str !== "[object Object]" && str !== "undefined" && str !== "null") {
+        stringPaths.push(str);
+      }
+    }
+  }
+  if (stringPaths.length === 0) {
+    return ".";
+  }
+  try {
+    return pathJoin(...stringPaths);
+  } catch {
+    const sep = stringPaths[0].includes("\\") ? "\\" : "/";
+    return stringPaths.join(sep).replace(/[/\\]+/g, sep);
+  }
+}
+
+// src/paths.ts
 function getCurrentDirname() {
   try {
     if (import.meta?.url) {
@@ -28,7 +55,7 @@ function getCurrentDirname() {
   }
   try {
     const packageJsonPath = __require.resolve("ux-toolkit/package.json");
-    return join(dirname(packageJsonPath), "dist");
+    return safeJoin(dirname(packageJsonPath), "dist");
   } catch {
   }
   const cwd = process.cwd();
@@ -47,7 +74,7 @@ function getCurrentDirname() {
 }
 var currentDirname = getCurrentDirname();
 function getPackageRoot() {
-  return join(currentDirname, "..");
+  return safeJoin(currentDirname, "..");
 }
 function getGlobalConfigDir() {
   if (process.env.UX_TOOLKIT_CONFIG_DIR) {
@@ -59,9 +86,9 @@ function getGlobalConfigDir() {
   const home = homedir();
   const currentPlatform = platform();
   if (currentPlatform === "linux" && process.env.XDG_CONFIG_HOME) {
-    return join(process.env.XDG_CONFIG_HOME, "opencode");
+    return safeJoin(process.env.XDG_CONFIG_HOME, "opencode");
   }
-  return join(home, ".config", "opencode");
+  return safeJoin(home, ".config", "opencode");
 }
 function isOpenCodeInstalled() {
   return existsSync(getGlobalConfigDir());
@@ -70,7 +97,7 @@ function getClaudeConfigDir() {
   if (process.env.CLAUDE_CONFIG_DIR) {
     return resolve(process.env.CLAUDE_CONFIG_DIR);
   }
-  return join(homedir(), ".claude");
+  return safeJoin(homedir(), ".claude");
 }
 function isClaudeInstalled() {
   return existsSync(getClaudeConfigDir());
@@ -92,7 +119,7 @@ function getPlatformInfo() {
 }
 function getProjectConfigDir(projectRoot = process.cwd(), target = "opencode") {
   const dirName = target === "claude" ? ".claude" : ".opencode";
-  return join(projectRoot, dirName);
+  return safeJoin(projectRoot, dirName);
 }
 function getDestinationPaths(global, projectRoot, target = "opencode") {
   let baseDir;
@@ -102,9 +129,9 @@ function getDestinationPaths(global, projectRoot, target = "opencode") {
     baseDir = getProjectConfigDir(projectRoot, target);
   }
   return {
-    skills: join(baseDir, "skills"),
-    agents: join(baseDir, "agents"),
-    commands: join(baseDir, "commands")
+    skills: safeJoin(baseDir, "skills"),
+    agents: safeJoin(baseDir, "agents"),
+    commands: safeJoin(baseDir, "commands")
   };
 }
 
@@ -346,15 +373,15 @@ var COMMANDS = [
     description: "Comprehensive UX audit"
   },
   {
-    name: "a11y-check",
+    name: "ux-a11y-check",
     description: "Quick accessibility scan"
   },
   {
-    name: "design-review",
+    name: "ux-design-review",
     description: "Visual consistency check"
   },
   {
-    name: "screenshot-review",
+    name: "ux-screenshot-review",
     description: "Visual review from screenshot"
   }
 ];
@@ -390,7 +417,7 @@ async function install(options = {}) {
     if (verbose) console.log(msg);
   };
   if (effectiveCategories.includes("skills")) {
-    const skillsDir = join2(packageRoot, "skills");
+    const skillsDir = safeJoin(packageRoot, "skills");
     if (existsSync2(skillsDir)) {
       let skills = readdirSync(skillsDir, { withFileTypes: true }).filter((d) => d.isDirectory()).map((d) => d.name);
       if (specificSkills?.length) {
@@ -398,8 +425,8 @@ async function install(options = {}) {
         skills = skills.filter((s) => requested.has(s.toLowerCase()));
       }
       for (const skill of skills) {
-        const src = join2(skillsDir, skill);
-        const dest = join2(destinations.skills, skill);
+        const src = safeJoin(skillsDir, skill);
+        const dest = safeJoin(destinations.skills, skill);
         try {
           if (existsSync2(dest) && !force) {
             result.skipped.push(`skill:${skill}`);
@@ -417,7 +444,7 @@ async function install(options = {}) {
     }
   }
   if (effectiveCategories.includes("agents")) {
-    const agentsDir = join2(packageRoot, "agents");
+    const agentsDir = safeJoin(packageRoot, "agents");
     if (existsSync2(agentsDir)) {
       let agents = readdirSync(agentsDir).filter((f) => f.endsWith(".md")).map((f) => f.replace(".md", ""));
       if (specificAgents?.length) {
@@ -425,8 +452,8 @@ async function install(options = {}) {
         agents = agents.filter((a) => requested.has(a.toLowerCase()));
       }
       for (const agent of agents) {
-        const src = join2(agentsDir, `${agent}.md`);
-        const dest = join2(destinations.agents, `${agent}.md`);
+        const src = safeJoin(agentsDir, `${agent}.md`);
+        const dest = safeJoin(destinations.agents, `${agent}.md`);
         try {
           if (existsSync2(dest) && !force) {
             result.skipped.push(`agent:${agent}`);
@@ -444,7 +471,7 @@ async function install(options = {}) {
     }
   }
   if (effectiveCategories.includes("commands")) {
-    const commandsDir = join2(packageRoot, "commands");
+    const commandsDir = safeJoin(packageRoot, "commands");
     if (existsSync2(commandsDir)) {
       let commands = readdirSync(commandsDir).filter((f) => f.endsWith(".md")).map((f) => f.replace(".md", ""));
       if (specificCommands?.length) {
@@ -452,8 +479,8 @@ async function install(options = {}) {
         commands = commands.filter((c) => requested.has(c.toLowerCase()));
       }
       for (const command of commands) {
-        const src = join2(commandsDir, `${command}.md`);
-        const dest = join2(destinations.commands, `${command}.md`);
+        const src = safeJoin(commandsDir, `${command}.md`);
+        const dest = safeJoin(destinations.commands, `${command}.md`);
         try {
           if (existsSync2(dest) && !force) {
             result.skipped.push(`command:${command}`);
@@ -493,13 +520,13 @@ function getTargetStatus(target, isGlobal, projectRoot) {
   const available = target === "opencode" ? isOpenCodeInstalled() : isClaudeInstalled();
   const destinations = getDestinationPaths(isGlobal, projectRoot, target);
   const skillStatuses = SKILLS.map(
-    (s) => checkComponentStatus(join2(destinations.skills, s.name), s.name)
+    (s) => checkComponentStatus(safeJoin(destinations.skills, s.name), s.name)
   );
   const agentStatuses = AGENTS.map(
-    (a) => checkComponentStatus(join2(destinations.agents, `${a.name}.md`), a.name)
+    (a) => checkComponentStatus(safeJoin(destinations.agents, `${a.name}.md`), a.name)
   );
   const commandStatuses = COMMANDS.map(
-    (c) => checkComponentStatus(join2(destinations.commands, `${c.name}.md`), c.name)
+    (c) => checkComponentStatus(safeJoin(destinations.commands, `${c.name}.md`), c.name)
   );
   return {
     target,
@@ -551,7 +578,7 @@ async function uninstall(options = {}) {
   const ourCommands = COMMANDS.map((c) => c.name);
   if (categories.includes("skills")) {
     for (const skill of ourSkills) {
-      const dest = join2(destinations.skills, skill);
+      const dest = safeJoin(destinations.skills, skill);
       try {
         if (!existsSync2(dest)) {
           result.notFound.push(`skill:${skill}`);
@@ -568,7 +595,7 @@ async function uninstall(options = {}) {
   }
   if (categories.includes("agents")) {
     for (const agent of ourAgents) {
-      const dest = join2(destinations.agents, `${agent}.md`);
+      const dest = safeJoin(destinations.agents, `${agent}.md`);
       try {
         if (!existsSync2(dest)) {
           result.notFound.push(`agent:${agent}`);
@@ -585,7 +612,7 @@ async function uninstall(options = {}) {
   }
   if (categories.includes("commands")) {
     for (const command of ourCommands) {
-      const dest = join2(destinations.commands, `${command}.md`);
+      const dest = safeJoin(destinations.commands, `${command}.md`);
       try {
         if (!existsSync2(dest)) {
           result.notFound.push(`command:${command}`);
@@ -936,13 +963,12 @@ Errors:`);
         ok.push(`Node.js ${nodeVersion}`);
       }
       const platformInfo = getPlatformInfo();
+      const { existsSync: existsSync3 } = await import("fs");
       if (platformInfo.opencode.exists) {
         ok.push(`OpenCode detected at ${platformInfo.opencode.configDir}`);
-        const { existsSync: existsSync3 } = await import("fs");
-        const { join: join3 } = await import("path");
-        const skillsDir = join3(platformInfo.opencode.configDir, "skills");
-        const agentsDir = join3(platformInfo.opencode.configDir, "agents");
-        const commandsDir = join3(platformInfo.opencode.configDir, "commands");
+        const skillsDir = safeJoin(platformInfo.opencode.configDir, "skills");
+        const agentsDir = safeJoin(platformInfo.opencode.configDir, "agents");
+        const commandsDir = safeJoin(platformInfo.opencode.configDir, "commands");
         if (!existsSync3(skillsDir)) {
           warnings.push("OpenCode skills directory missing");
         }
@@ -957,11 +983,9 @@ Errors:`);
       }
       if (platformInfo.claude.exists) {
         ok.push(`Claude Code detected at ${platformInfo.claude.configDir}`);
-        const { existsSync: existsSync3 } = await import("fs");
-        const { join: join3 } = await import("path");
-        const skillsDir = join3(platformInfo.claude.configDir, "skills");
-        const agentsDir = join3(platformInfo.claude.configDir, "agents");
-        const commandsDir = join3(platformInfo.claude.configDir, "commands");
+        const skillsDir = safeJoin(platformInfo.claude.configDir, "skills");
+        const agentsDir = safeJoin(platformInfo.claude.configDir, "agents");
+        const commandsDir = safeJoin(platformInfo.claude.configDir, "commands");
         if (!existsSync3(skillsDir)) {
           warnings.push("Claude Code skills directory missing");
         }