skiller 0.8.2 → 0.9.0

package/dist/core/SkillsUtils.js

@@ -161,17 +161,12 @@ async function walkSkillsTree(root) {
                     await walk(entryPath, entryRelativePath, depth + 1);
                 }
                 else {
-                    // Check if this directory has any .mdc files
-                    const hasMdc = await hasAnyMdc(entryPath);
-                    if (!hasMdc) {
-                        // No SKILL.md, not a grouping dir, no .mdc files - delete it
-                        try {
-                            await fs.rm(entryPath, { recursive: true, force: true });
-                            deleted.push(entryRelativePath);
-                        }
-                        catch {
-                            warnings.push(`Failed to delete empty directory '${entryRelativePath}'`);
-                        }
+                    try {
+                        await fs.rm(entryPath, { recursive: true, force: true });
+                        deleted.push(entryRelativePath);
+                    }
+                    catch {
+                        warnings.push(`Failed to delete empty directory '${entryRelativePath}'`);
                     }
                 }
             }

package/dist/core/UnifiedConfigLoader.js

@@ -41,6 +41,7 @@ const FileSystemUtils = __importStar(require("./FileSystemUtils"));
 const FileSystemUtils_1 = require("./FileSystemUtils");
 const hash_1 = require("./hash");
 const RuleProcessor_1 = require("./RuleProcessor");
+const project_paths_1 = require("./project-paths");
 /**
  * Expand environment variables in a string.
  * Supports ${VAR} syntax, replacing with process.env[VAR] or empty string if not found.
@@ -55,9 +56,9 @@ function expandEnvRecord(record) {
     return Object.fromEntries(Object.entries(record).map(([k, v]) => [k, expandEnvVars(v)]));
 }
 async function loadUnifiedConfig(options) {
-    // Resolve the effective .claude directory (local or global), mirroring the main loader behavior
+    // Resolve the effective .agents directory (local or global), mirroring the main loader behavior
     const resolvedSkillerDir = (await FileSystemUtils.findSkillerDir(options.projectRoot, true)) ||
-        path.join(options.projectRoot, '.claude');
+        path.join(options.projectRoot, project_paths_1.CANONICAL_SKILLER_DIR);
     const meta = {
         projectRoot: options.projectRoot,
         skillerDir: resolvedSkillerDir,
@@ -69,7 +70,7 @@ async function loadUnifiedConfig(options) {
     let tomlRaw = {};
     const tomlFile = options.configPath
         ? path.resolve(options.configPath)
-        : path.join(meta.skillerDir, 'skiller.toml');
+        : path.join(meta.skillerDir, project_paths_1.SKILLER_CONFIG_FILE);
     try {
         const text = await fs_1.promises.readFile(tomlFile, 'utf8');
         tomlRaw = text.trim() ? (0, toml_1.parse)(text) : {};
@@ -114,7 +115,7 @@ async function loadUnifiedConfig(options) {
                 diagnostics.push({
                     severity: 'warning',
                     code: 'SKILLS_DEPRECATED_OPTION',
-                    message: 'skills.generate_from_rules is deprecated and has no effect. Skills are now edited directly in .claude/skills/',
+                    message: 'skills.generate_from_rules is deprecated and has no effect. Local rule sources in .agents/rules/ compile automatically into .agents/skills/.',
                     file: tomlFile,
                 });
             }
@@ -122,7 +123,7 @@ async function loadUnifiedConfig(options) {
                 diagnostics.push({
                     severity: 'warning',
                     code: 'SKILLS_DEPRECATED_OPTION',
-                    message: 'skills.prune is deprecated and has no effect. Skills in .claude/skills/ are never auto-deleted.',
+                    message: 'skills.prune is deprecated and has no effect. Skills in .agents/skills/ are never auto-deleted.',
                     file: tomlFile,
                 });
             }

package/dist/core/project-paths.js

@@ -0,0 +1,8 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.SKILLER_CONFIG_FILE = exports.LEGACY_INSTRUCTIONS_FILE = exports.PROJECT_AGENTS_FILE = exports.LEGACY_SKILLER_DIR = exports.CANONICAL_SKILLER_DIR = void 0;
+exports.CANONICAL_SKILLER_DIR = '.agents';
+exports.LEGACY_SKILLER_DIR = '.claude';
+exports.PROJECT_AGENTS_FILE = 'AGENTS.md';
+exports.LEGACY_INSTRUCTIONS_FILE = 'instructions.md';
+exports.SKILLER_CONFIG_FILE = 'skiller.toml';

package/dist/lib.js

@@ -42,6 +42,8 @@ const constants_1 = require("./constants");
 const agent_selection_1 = require("./core/agent-selection");
 const apply_engine_1 = require("./core/apply-engine");
 const config_utils_1 = require("./core/config-utils");
+const SkillOwnership_1 = require("./core/SkillOwnership");
+const LegacyClaudePluginState_1 = require("./core/LegacyClaudePluginState");
 const agents = agents_1.allAgents;
 /**
  * Resolves skills enabled state based on precedence: CLI flag > skiller.toml > default (enabled)
@@ -74,6 +76,8 @@ function resolveBackupEnabled(cliFlag, configSetting) {
  * @param includedAgents Optional list of agent name filters (case-insensitive substrings)
  */
 async function applyAllAgentConfigs(projectRoot, includedAgents, configPath, cliMcpEnabled = true, cliMcpStrategy, cliGitignoreEnabled, verbose = false, dryRun = false, localOnly = false, nested = false, cliBackupEnabled, skillsEnabled) {
+    await (0, LegacyClaudePluginState_1.assertNoLegacyClaudePluginState)(projectRoot);
+    await (0, SkillOwnership_1.migrateLegacyProjectState)(projectRoot, dryRun);
     // Load configuration and rules
     (0, constants_1.logVerbose)(`Loading configuration from project root: ${projectRoot}`, verbose);
     if (configPath) {
@@ -85,7 +89,7 @@ async function applyAllAgentConfigs(projectRoot, includedAgents, configPath, cli
     if (nested) {
         const hierarchicalConfigs = await (0, apply_engine_1.loadNestedConfigurations)(projectRoot, configPath, localOnly, nested);
         if (hierarchicalConfigs.length === 0) {
-            throw new Error('No .claude directories found');
+            throw new Error('No .agents directories found');
         }
         (0, constants_1.logWarn)('Nested mode is experimental and may change in future releases.', dryRun);
         // Use the root config for agent selection (all levels share the same agent settings)
@@ -93,7 +97,7 @@ async function applyAllAgentConfigs(projectRoot, includedAgents, configPath, cli
         const rootConfig = rootConfigEntry.config;
         loadedConfig = rootConfig;
         rootConfig.cliAgents = includedAgents;
-        (0, constants_1.logVerbose)(`Loaded ${hierarchicalConfigs.length} .claude directory configurations`, verbose);
+        (0, constants_1.logVerbose)(`Loaded ${hierarchicalConfigs.length} .agents directory configurations`, verbose);
         (0, constants_1.logVerbose)(`Root configuration has ${Object.keys(rootConfig.agentConfigs).length} agent configs`, verbose);
         for (const configEntry of hierarchicalConfigs) {
             normalizeAgentConfigs(configEntry.config, agents);
@@ -102,14 +106,8 @@ async function applyAllAgentConfigs(projectRoot, includedAgents, configPath, cli
         (0, constants_1.logVerbose)(`Selected ${selectedAgents.length} agents: ${selectedAgents.map((a) => a.getName()).join(', ')}`, verbose);
         // Propagate skills (or cleanup if disabled) - do this for each nested directory
         const skillsEnabledResolved = resolveSkillsEnabled(skillsEnabled, rootConfig.skills?.enabled);
-        const { propagateSkills, migrateRulesToSkills } = await Promise.resolve().then(() => __importStar(require('./core/SkillsProcessor')));
-        // Migrate content from .claude/rules to .claude/skills (only if rules exists)
-        if (skillsEnabledResolved) {
-            for (const configEntry of hierarchicalConfigs) {
-                await migrateRulesToSkills(configEntry.skillerDir, verbose, dryRun);
-            }
-        }
-        // Propagate skills for each nested .claude directory (or cleanup if disabled)
+        const { propagateSkills } = await Promise.resolve().then(() => __importStar(require('./core/SkillsProcessor')));
+        // Propagate skills for each nested .agents directory (or cleanup if disabled)
         for (const configEntry of hierarchicalConfigs) {
             const nestedRoot = path.dirname(configEntry.skillerDir);
             (0, constants_1.logVerbose)(`Propagating skills for nested directory: ${nestedRoot}`, verbose);
@@ -124,17 +122,13 @@ async function applyAllAgentConfigs(projectRoot, includedAgents, configPath, cli
         loadedConfig = singleConfig.config;
         singleConfig.config.cliAgents = includedAgents;
         (0, constants_1.logVerbose)(`Loaded configuration with ${Object.keys(singleConfig.config.agentConfigs).length} agent configs`, verbose);
-        (0, constants_1.logVerbose)(`Found .claude directory with ${singleConfig.concatenatedRules.length} characters of rules`, verbose);
+        (0, constants_1.logVerbose)(`Found .agents directory with ${singleConfig.concatenatedRules.length} characters of rules`, verbose);
         normalizeAgentConfigs(singleConfig.config, agents);
         selectedAgents = (0, agent_selection_1.resolveSelectedAgents)(singleConfig.config, agents);
         (0, constants_1.logVerbose)(`Selected ${selectedAgents.length} agents: ${selectedAgents.map((a) => a.getName()).join(', ')}`, verbose);
         // Propagate skills (or cleanup if disabled)
         const skillsEnabledResolved = resolveSkillsEnabled(skillsEnabled, singleConfig.config.skills?.enabled);
-        const { propagateSkills, migrateRulesToSkills } = await Promise.resolve().then(() => __importStar(require('./core/SkillsProcessor')));
-        // Migrate content from .claude/rules to .claude/skills (only if rules exists)
-        if (skillsEnabledResolved) {
-            await migrateRulesToSkills(singleConfig.skillerDir, verbose, dryRun);
-        }
+        const { propagateSkills } = await Promise.resolve().then(() => __importStar(require('./core/SkillsProcessor')));
         // Always call propagateSkills - it handles cleanup when disabled
         await propagateSkills(projectRoot, selectedAgents, skillsEnabledResolved, verbose, dryRun, singleConfig.skillerDir);
         // Resolve backup setting: CLI > Config > Default (true)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "skiller",
-  "version": "0.8.2",
+  "version": "0.9.0",
   "description": "Skiller — apply the same rules to all coding agents",
   "main": "dist/lib.js",
   "publishConfig": {
@@ -50,6 +50,13 @@
   "bin": {
     "skiller": "dist/cli/index.js"
   },
+  "dependencies": {
+    "@iarna/toml": "^2.2.5",
+    "js-yaml": "^4.1.0",
+    "skills": "1.4.6",
+    "yargs": "^18.0.0",
+    "zod": "^4.1.12"
+  },
   "devDependencies": {
     "@changesets/changelog-github": "^0.5.2",
     "@changesets/cli": "2.29.8",
@@ -66,15 +73,8 @@
     "eslint-plugin-prettier": "^5.5.4",
     "jest": "^29.7.0",
     "prettier": "^3.6.2",
-    "skills": "1.4.6",
     "ts-jest": "^29.4.5",
     "typescript": "^5.9.3",
     "typescript-eslint": "^8.46.2"
-  },
-  "dependencies": {
-    "@iarna/toml": "^2.2.5",
-    "js-yaml": "^4.1.0",
-    "yargs": "^18.0.0",
-    "zod": "^4.1.12"
   }
 }