@sweny-ai/core 0.1.88 → 0.1.89

package/dist/cli/skill.d.ts

@@ -14,14 +14,41 @@
  * and `../skills/index.js`; this CLI is a thin authoring shell over them.
  */
 import { Command } from "commander";
-declare const VALID_CATEGORIES: readonly ["general", "git", "tasks", "notification", "observability", "data"];
-type SkillCategory = (typeof VALID_CATEGORIES)[number];
+import { type SkillCategory } from "../types.js";
+declare const HARNESS_DIRS: {
+    readonly claude: ".claude/skills";
+    readonly sweny: ".sweny/skills";
+    readonly agents: ".agents/skills";
+    readonly gemini: ".gemini/skills";
+};
+type HarnessKey = keyof typeof HARNESS_DIRS;
 /** Build the SKILL.md body for a fresh scaffold. Exported for tests. */
 export declare function renderSkillTemplate(opts: {
     id: string;
     description: string;
     category: SkillCategory;
 }): string;
+interface NewOptions {
+    description?: string;
+    category?: string;
+    harness?: HarnessKey;
+    force?: boolean;
+}
+/**
+ * Action handler for `sweny skill new`. Exported for unit tests; production
+ * code goes through `registerSkillCommand`. `cwd` defaults to `process.cwd()`
+ * so the CLI works as expected; tests pass a tmp dir for isolation.
+ */
+export declare function runSkillNew(idArg: string, options: NewOptions, cwd?: string): void;
+interface ListOptions {
+    json?: boolean;
+}
+/**
+ * Action handler for `sweny skill list`. Exported for unit tests; production
+ * code goes through `registerSkillCommand`. `cwd` and `env` injection lets
+ * tests exercise the configured-badge logic without polluting global state.
+ */
+export declare function runSkillList(options: ListOptions, cwd?: string, env?: Record<string, string | undefined>): void;
 /** Register the `sweny skill` subcommand group on the parent program. */
 export declare function registerSkillCommand(program: Command): void;
 export {};

package/dist/cli/skill.js

@@ -18,9 +18,9 @@ import * as path from "node:path";
 import chalk from "chalk";
 import { builtinSkills } from "../skills/index.js";
 import { configuredSkillsWithDiagnostics, discoverSkillsWithDiagnostics } from "../skills/custom-loader.js";
+import { SKILL_CATEGORIES } from "../types.js";
 // Mirror of the loader's regex so authoring-side validation matches discovery.
 const VALID_SKILL_ID = /^[a-z0-9]([a-z0-9-]*[a-z0-9])?$/;
-const VALID_CATEGORIES = ["general", "git", "tasks", "notification", "observability", "data"];
 const HARNESS_DIRS = {
     claude: ".claude/skills",
     sweny: ".sweny/skills",
@@ -80,7 +80,12 @@ export function renderSkillTemplate(opts) {
     lines.push("");
     return lines.join("\n");
 }
-function runSkillNew(idArg, options) {
+/**
+ * Action handler for `sweny skill new`. Exported for unit tests; production
+ * code goes through `registerSkillCommand`. `cwd` defaults to `process.cwd()`
+ * so the CLI works as expected; tests pass a tmp dir for isolation.
+ */
+export function runSkillNew(idArg, options, cwd = process.cwd()) {
     const id = idArg.toLowerCase();
     if (!VALID_SKILL_ID.test(id) || id.includes("--") || id.length > 64) {
         console.error(chalk.red(`  Invalid skill id "${idArg}".\n  Skill ids must be lowercase alphanumeric + hyphens (no consecutive hyphens), max 64 chars.`));
@@ -89,8 +94,8 @@ function runSkillNew(idArg, options) {
     }
     const description = (options.description ?? `Custom ${id} skill`).trim();
     const category = (options.category ?? "general");
-    if (!VALID_CATEGORIES.includes(category)) {
-        console.error(chalk.red(`  Invalid category "${category}".\n  Allowed: ${VALID_CATEGORIES.join(", ")}`));
+    if (!SKILL_CATEGORIES.includes(category)) {
+        console.error(chalk.red(`  Invalid category "${category}".\n  Allowed: ${SKILL_CATEGORIES.join(", ")}`));
         process.exit(2);
         return;
     }
@@ -101,7 +106,6 @@ function runSkillNew(idArg, options) {
         process.exit(2);
         return;
     }
-    const cwd = process.cwd();
     const skillDir = path.join(cwd, baseDir, id);
     const skillFile = path.join(skillDir, "SKILL.md");
     if (fs.existsSync(skillFile) && !options.force) {
@@ -121,16 +125,20 @@ function runSkillNew(idArg, options) {
     console.log(chalk.dim(`    5. Run \`sweny skill list\` to confirm it discovers correctly.`));
     console.log();
 }
-function runSkillList(options) {
-    const cwd = process.cwd();
-    // We want the FULL list — built-in + custom — even when env vars
-    // aren't set. configuredSkills filters by env, which hides authoring-
-    // time skills the user just scaffolded. So combine sources directly.
+/**
+ * Action handler for `sweny skill list`. Exported for unit tests; production
+ * code goes through `registerSkillCommand`. `cwd` and `env` injection lets
+ * tests exercise the configured-badge logic without polluting global state.
+ */
+export function runSkillList(options, cwd = process.cwd(), env = process.env) {
+    // We want the FULL list (built-in + custom) even when env vars aren't set.
+    // configuredSkills filters by env, which hides authoring-time skills the
+    // user just scaffolded, so combine sources directly.
     const { skills: customSkills, warnings: customWarnings } = discoverSkillsWithDiagnostics(cwd);
     const customIds = new Set(customSkills.map((s) => s.id));
     const builtinList = builtinSkills.filter((s) => !customIds.has(s.id));
-    // Configured map drives the "configured?" badge — required env vars present.
-    const configuredIds = new Set(configuredSkillsWithDiagnostics(process.env, cwd).skills.map((s) => s.id));
+    // Configured map drives the "configured?" badge: required env vars present.
+    const configuredIds = new Set(configuredSkillsWithDiagnostics(env, cwd).skills.map((s) => s.id));
     if (options.json) {
         const data = [
             ...builtinList.map((s) => ({
@@ -188,7 +196,7 @@ export function registerSkillCommand(program) {
         .command("new <id>")
         .description("Scaffold a new SKILL.md in .claude/skills/<id>/")
         .option("-d, --description <text>", "One-line description (required field in frontmatter)")
-        .option("-c, --category <category>", `Skill category (${VALID_CATEGORIES.join("|")})`, "general")
+        .option("-c, --category <category>", `Skill category (${SKILL_CATEGORIES.join("|")})`, "general")
         .option("--harness <harness>", `Where to place the skill (${Object.keys(HARNESS_DIRS).join("|")})`, "claude")
         .option("--force", "Overwrite an existing SKILL.md")
         .action((id, options) => runSkillNew(id, options));

package/dist/schema.d.ts

@@ -241,7 +241,7 @@ export declare const skillZ: z.ZodEffects<z.ZodObject<{
     id: string;
     name: string;
     description: string;
-    category: "git" | "observability" | "tasks" | "notification" | "general";
+    category: "general" | "git" | "tasks" | "notification" | "observability";
     config: Record<string, {
         description: string;
         required?: boolean | undefined;
@@ -266,7 +266,7 @@ export declare const skillZ: z.ZodEffects<z.ZodObject<{
     id: string;
     name: string;
     description: string;
-    category: "git" | "observability" | "tasks" | "notification" | "general";
+    category: "general" | "git" | "tasks" | "notification" | "observability";
     instruction?: string | undefined;
     mcp?: {
         url?: string | undefined;
@@ -291,7 +291,7 @@ export declare const skillZ: z.ZodEffects<z.ZodObject<{
     id: string;
     name: string;
     description: string;
-    category: "git" | "observability" | "tasks" | "notification" | "general";
+    category: "general" | "git" | "tasks" | "notification" | "observability";
     config: Record<string, {
         description: string;
         required?: boolean | undefined;
@@ -316,7 +316,7 @@ export declare const skillZ: z.ZodEffects<z.ZodObject<{
     id: string;
     name: string;
     description: string;
-    category: "git" | "observability" | "tasks" | "notification" | "general";
+    category: "general" | "git" | "tasks" | "notification" | "observability";
     instruction?: string | undefined;
     mcp?: {
         url?: string | undefined;

package/dist/skills/custom-loader.js

@@ -11,6 +11,7 @@
 import { readdirSync, readFileSync, existsSync } from "node:fs";
 import { join } from "node:path";
 import YAML from "yaml";
+import { SKILL_CATEGORIES } from "../types.js";
 import { builtinSkills, isSkillConfigured } from "./index.js";
 /** Directories to scan, in ascending priority order. Last match wins. */
 const SKILL_DIRS = [".gemini/skills", ".agents/skills", ".claude/skills", ".sweny/skills"];
@@ -99,8 +100,7 @@ function parseSkillMd(content, path) {
     }
     // Narrow the loosely-typed YAML output into the fields we expect
     const fm = raw;
-    const VALID_CATEGORIES = ["general", "git", "tasks", "notification", "observability", "data"];
-    const category = typeof fm.category === "string" && VALID_CATEGORIES.includes(fm.category)
+    const category = typeof fm.category === "string" && SKILL_CATEGORIES.includes(fm.category)
         ? fm.category
         : "general";
     const id = String(fm.name);

package/dist/types.d.ts

@@ -22,8 +22,16 @@ export interface ConfigField {
     /** Default environment variable to read from */
     env?: string;
 }
-/** Skill categories — used for validation and grouping */
-export type SkillCategory = "git" | "observability" | "tasks" | "notification" | "data" | "general";
+/**
+ * Skill categories: used for per-node validation and grouping.
+ *
+ * Single source of truth for runtime + compile-time. The `as const` tuple
+ * lets the CLI iterate it for help text and the loader use it for runtime
+ * validation; `SkillCategory` is derived so adding a category requires
+ * only this line.
+ */
+export declare const SKILL_CATEGORIES: readonly ["general", "git", "tasks", "notification", "observability", "data"];
+export type SkillCategory = (typeof SKILL_CATEGORIES)[number];
 /** A skill groups related tools with shared config requirements */
 export interface Skill {
     id: string;

package/dist/types.js

@@ -1,3 +1,12 @@
+/**
+ * Skill categories: used for per-node validation and grouping.
+ *
+ * Single source of truth for runtime + compile-time. The `as const` tuple
+ * lets the CLI iterate it for help text and the loader use it for runtime
+ * validation; `SkillCategory` is derived so adding a category requires
+ * only this line.
+ */
+export const SKILL_CATEGORIES = ["general", "git", "tasks", "notification", "observability", "data"];
 export const consoleLogger = {
     info: (msg, data) => console.log(`[info] ${msg}`, data ?? ""),
     warn: (msg, data) => console.warn(`[warn] ${msg}`, data ?? ""),

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sweny-ai/core",
-  "version": "0.1.88",
+  "version": "0.1.89",
   "type": "module",
   "bin": {
     "sweny": "./dist/cli/main.js"