agent-skills-standard 1.3.2 → 1.4.1

package/README.md

@@ -33,6 +33,36 @@ Modern AI coding agents are powerful, but they have major flaws:
 
 ---
 
+## 🛠 The Solution: Digital DNA for AI
+
+Agent Skills Standard treats instructions as **versioned dependencies**, much like software libraries.
+
+- **🎯 Smart Loading**: We use a "Search-on-Demand" pattern. The AI only looks at detailed examples when it specifically needs them, saving its memory for your actual code.
+- **🚀 High-Density Language**: We use a specialized "Compressed Syntax" that is **40% more efficient** than normal English. This means the AI understands more while using fewer resources.
+- **🔁 One-Click Sync**: A single command ensures your AI tool stays up-to-date with your team's latest standards.
+
+---
+
+## ✨ Features
+
+- **🛡️ Multi-Agent Support**: Out-of-the-box mapping for Cursor, Claude Dev, GitHub Copilot, and more.
+- **📦 Modular Registry**: Don't load everything. Only enable the skills your project actually uses (e.g., `Flutter + BLoC + Clean Architecture`).
+- **🔄 Dynamic Re-detection**: Automatically re-enables excluded skills if matching dependencies (like `Retrofit`, `Room`, or `Navigation`) are added to your project.
+- **🔒 Secure Overrides**: Lock specific files in your project so they never get overwritten by the central registry.
+- **📊 Semantic Tagging**: Skills are tagged with triggers that tell the AI _exactly_ when to apply them.
+
+---
+
+## � Token Economy & Optimization
+
+To ensure AI efficiency, this project follows a strict **Token Economy**. Every skill is audited for its footprint in the AI's context window.
+
+### 📏 Our Standards
+
+- **High-Density**: Core rules in `SKILL.md` are kept under **70 lines**.
+- **Efficiency**: Target **< 500 tokens** per primary skill file.
+- **Progressive Disclosure**: Heavy examples, checklists, and implementation guides are moved to the `references/` directory and are only loaded by the agent when specific context matches.
+
 ## 🚀 Installation
 
 You can run the tool instantly without installing, or install it globally for convenience:
@@ -83,15 +113,24 @@ skills:
     ref: flutter-v1.1.0
     # 🚫 Exclude specific sub-skills from being synced
     exclude: ['getx-navigation']
+    # ➕ Include specific skills (supports cross-category 'category/skill' or 'category/*' syntax)
+    include:
+      - 'bloc-state-management'
+      - 'react/hooks'
+      - 'common/*'
     # 🔒 Protect local modifications from being overwritten
     custom_overrides: ['bloc-state-management']
 ```
 
 ### Key Options
 
-- **`exclude`**: A list of skill IDs to skip during synchronization. Useful if you want the main framework guidelines but use a different library for a specific sub-task (e.g., excluding `auto-route` if you use `go_router`).
-- **`custom_overrides`**: A list of skill IDs that the CLI should **never** overwrite. Use this when you have modified a standard skill locally to fit your project's unique needs. The CLI will detect these and preserve your local version.
-- **`ref`**: Specify a specific version or tag of the skills from the registry.
+- **`exclude`**: A list of skill IDs to skip during synchronization.
+- **`include`**: A list of skill IDs to fetch. Supports:
+  - **Relative Path**: `bloc-state-management` (from current category)
+  - **Absolute Path**: `react/hooks` (pull specific skill from another category)
+  - **Glob Path**: `common/*` (pull ALL skills from another category)
+- **`custom_overrides`**: A list of skill IDs that the CLI should **never** overwrite.
+- **`ref`**: Specify a specific version or tag for the skills.
 
 ---
 
@@ -115,7 +154,12 @@ The CLI connects to the [Official Skills Registry](https://github.com/HoangNguye
 - **React**: Hooks, Patterns, Performance.
 - **NestJS**: Architecture, Microservices, Security.
 - **Next.js**: App Router, RSC, FSD Architecture.
-- **Golang**: (Coming Soon)
+- **Golang**: Clean Architecture, API Design, Concurrency.
+- **Angular**: Standalone, Signals, Control Flow.
+- **Kotlin**: Idiomatic Patterns, Coroutines, Flow.
+- **Spring Boot**: Architecture, Security, Data, Test.
+- **Android Native**: Architecture, Compose, DI, Performance, Testing, WorkManager.
+- **iOS Native**: Architecture, UI/Layout, Lifecycle, Security, Performance.
 
 ---
 

package/dist/commands/__tests__/InitCommand.spec.d.ts

@@ -0,0 +1 @@
+export {};

package/dist/commands/__tests__/InitCommand.spec.js

@@ -0,0 +1,66 @@
+"use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+const fs_extra_1 = __importDefault(require("fs-extra"));
+const inquirer_1 = __importDefault(require("inquirer"));
+const vitest_1 = require("vitest");
+const init_1 = require("../init");
+vitest_1.vi.mock('fs-extra');
+vitest_1.vi.mock('inquirer');
+(0, vitest_1.describe)('InitCommand', () => {
+    let initCommand;
+    let mockInitService;
+    let mockRegistryService;
+    (0, vitest_1.beforeEach)(() => {
+        vitest_1.vi.clearAllMocks();
+        // Create mock instances
+        mockInitService = {
+            getInitializationContext: vitest_1.vi.fn().mockResolvedValue({}),
+            getPromptChoices: vitest_1.vi.fn().mockReturnValue({
+                frameworkChoices: [],
+                agentChoices: [],
+                defaultFramework: 'flutter',
+            }),
+            buildAndSaveConfig: vitest_1.vi.fn().mockResolvedValue(undefined),
+        };
+        mockRegistryService = {
+            discoverRegistry: vitest_1.vi
+                .fn()
+                .mockResolvedValue({ categories: [], metadata: {} }),
+        };
+        initCommand = new init_1.InitCommand(mockInitService, mockRegistryService);
+        vitest_1.vi.mocked(inquirer_1.default.prompt).mockImplementation(async (questions) => {
+            const q = Array.isArray(questions) ? questions[0] : questions;
+            if (q.name === 'overwrite')
+                return { overwrite: true };
+            return { framework: 'flutter', agents: ['cursor'], registry: 'url' };
+        });
+        // Fix pathExists mock to avoid 'void' assignment lints
+        vitest_1.vi.mocked(fs_extra_1.default.pathExists).mockImplementation(() => Promise.resolve(false));
+    });
+    (0, vitest_1.describe)('run', () => {
+        (0, vitest_1.it)('should run initialization flow', async () => {
+            const consoleSpy = vitest_1.vi.spyOn(console, 'log').mockImplementation(() => { });
+            await initCommand.run();
+            (0, vitest_1.expect)(mockInitService.getInitializationContext).toHaveBeenCalled();
+            (0, vitest_1.expect)(mockInitService.buildAndSaveConfig).toHaveBeenCalled();
+            (0, vitest_1.expect)(consoleSpy).toHaveBeenCalledWith(vitest_1.expect.stringContaining('Initialized .skillsrc'));
+            consoleSpy.mockRestore();
+        });
+        (0, vitest_1.it)('should abort if config exists and user says no', async () => {
+            vitest_1.vi.mocked(fs_extra_1.default.pathExists).mockImplementation(() => Promise.resolve(true));
+            vitest_1.vi.mocked(inquirer_1.default.prompt).mockResolvedValue({ overwrite: false });
+            const consoleSpy = vitest_1.vi.spyOn(console, 'log').mockImplementation(() => { });
+            await initCommand.run();
+            (0, vitest_1.expect)(consoleSpy).toHaveBeenCalledWith(vitest_1.expect.stringContaining('Aborted'));
+            (0, vitest_1.expect)(mockInitService.buildAndSaveConfig).not.toHaveBeenCalled();
+            consoleSpy.mockRestore();
+        });
+        (0, vitest_1.it)('should use default services if not provided', () => {
+            const command = new init_1.InitCommand();
+            (0, vitest_1.expect)(command).toBeDefined();
+        });
+    });
+});

package/dist/commands/__tests__/ListSkillsCommand.spec.d.ts

@@ -0,0 +1 @@
+export {};

package/dist/commands/__tests__/ListSkillsCommand.spec.js

@@ -0,0 +1,51 @@
+"use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+const inquirer_1 = __importDefault(require("inquirer"));
+const vitest_1 = require("vitest");
+const list_skills_1 = require("../list-skills");
+vitest_1.vi.mock('inquirer');
+(0, vitest_1.describe)('ListSkillsCommand', () => {
+    let listSkillsCommand;
+    let mockConfigService;
+    let mockDetectionService;
+    let mockSkillService;
+    (0, vitest_1.beforeEach)(() => {
+        vitest_1.vi.clearAllMocks();
+        mockConfigService = {
+            getRegistryUrl: vitest_1.vi
+                .fn()
+                .mockResolvedValue('https://github.com/owner/repo'),
+        };
+        mockDetectionService = {
+            getProjectDeps: vitest_1.vi.fn().mockResolvedValue(new Set(['dep1'])),
+        };
+        mockSkillService = {
+            getSkillsWithStatus: vitest_1.vi.fn().mockResolvedValue([
+                { name: 'skill1', status: 'detected' },
+                { name: 'skill2', status: 'not-detected' },
+                { name: 'skill3', status: 'no-rule' },
+            ]),
+        };
+        listSkillsCommand = new list_skills_1.ListSkillsCommand(mockConfigService, mockDetectionService, mockSkillService);
+        vitest_1.vi.mocked(inquirer_1.default.prompt).mockResolvedValue({ framework: 'flutter' });
+    });
+    (0, vitest_1.describe)('run', () => {
+        (0, vitest_1.it)('should prompt user and display skills', async () => {
+            const consoleSpy = vitest_1.vi.spyOn(console, 'log').mockImplementation(() => { });
+            await listSkillsCommand.run();
+            (0, vitest_1.expect)(inquirer_1.default.prompt).toHaveBeenCalled();
+            (0, vitest_1.expect)(mockSkillService.getSkillsWithStatus).toHaveBeenCalledWith('flutter', vitest_1.expect.any(String), vitest_1.expect.any(Set));
+            (0, vitest_1.expect)(consoleSpy).toHaveBeenCalledWith(vitest_1.expect.stringContaining('- skill1'));
+            (0, vitest_1.expect)(consoleSpy).toHaveBeenCalledWith(vitest_1.expect.stringContaining('- skill2'));
+            (0, vitest_1.expect)(consoleSpy).toHaveBeenCalledWith(vitest_1.expect.stringContaining('- skill3'));
+            consoleSpy.mockRestore();
+        });
+        (0, vitest_1.it)('should use default services if not provided', () => {
+            const command = new list_skills_1.ListSkillsCommand();
+            (0, vitest_1.expect)(command).toBeDefined();
+        });
+    });
+});

package/dist/commands/__tests__/SyncCommand.spec.d.ts

@@ -0,0 +1 @@
+export {};

package/dist/commands/__tests__/SyncCommand.spec.js

@@ -0,0 +1,70 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+const vitest_1 = require("vitest");
+const sync_1 = require("../sync");
+(0, vitest_1.describe)('SyncCommand', () => {
+    let syncCommand;
+    let mockConfigService;
+    let mockDetectionService;
+    let mockSyncService;
+    (0, vitest_1.beforeEach)(() => {
+        vitest_1.vi.clearAllMocks();
+        mockConfigService = {
+            loadConfig: vitest_1.vi.fn(),
+            saveConfig: vitest_1.vi.fn(),
+        };
+        mockDetectionService = {
+            getProjectDeps: vitest_1.vi.fn().mockResolvedValue(new Set()),
+        };
+        mockSyncService = {
+            reconcileConfig: vitest_1.vi.fn(),
+            assembleSkills: vitest_1.vi.fn(),
+            writeSkills: vitest_1.vi.fn(),
+            checkForUpdates: vitest_1.vi.fn().mockImplementation((c) => Promise.resolve(c)),
+        };
+        syncCommand = new sync_1.SyncCommand(mockConfigService, mockDetectionService, mockSyncService);
+    });
+    (0, vitest_1.describe)('run', () => {
+        (0, vitest_1.it)('should fail if .skillsrc is not found', async () => {
+            vitest_1.vi.mocked(mockConfigService.loadConfig).mockResolvedValue(null);
+            const consoleSpy = vitest_1.vi.spyOn(console, 'log').mockImplementation(() => { });
+            await syncCommand.run();
+            (0, vitest_1.expect)(consoleSpy).toHaveBeenCalledWith(vitest_1.expect.stringContaining('.skillsrc not found'));
+            consoleSpy.mockRestore();
+        });
+        (0, vitest_1.it)('should run successfully if config is valid', async () => {
+            const config = { registry: 'url', skills: { flutter: {} } };
+            vitest_1.vi.mocked(mockConfigService.loadConfig).mockResolvedValue(config);
+            vitest_1.vi.mocked(mockSyncService.assembleSkills).mockResolvedValue([]);
+            const consoleSpy = vitest_1.vi.spyOn(console, 'log').mockImplementation(() => { });
+            await syncCommand.run();
+            (0, vitest_1.expect)(mockSyncService.reconcileConfig).toHaveBeenCalled();
+            (0, vitest_1.expect)(mockSyncService.assembleSkills).toHaveBeenCalled();
+            (0, vitest_1.expect)(mockSyncService.writeSkills).toHaveBeenCalled();
+            (0, vitest_1.expect)(consoleSpy).toHaveBeenCalledWith(vitest_1.expect.stringContaining('All skills synced successfully!'));
+            consoleSpy.mockRestore();
+        });
+        (0, vitest_1.it)('should catch and log errors', async () => {
+            vitest_1.vi.mocked(mockConfigService.loadConfig).mockRejectedValue(new Error('Test error'));
+            const consoleSpy = vitest_1.vi
+                .spyOn(console, 'error')
+                .mockImplementation(() => { });
+            await syncCommand.run();
+            (0, vitest_1.expect)(consoleSpy).toHaveBeenCalledWith(vitest_1.expect.stringContaining('Sync failed:'), 'Test error');
+            consoleSpy.mockRestore();
+        });
+        (0, vitest_1.it)('should catch and log non-Error objects', async () => {
+            vitest_1.vi.mocked(mockConfigService.loadConfig).mockRejectedValue('String error');
+            const consoleSpy = vitest_1.vi
+                .spyOn(console, 'error')
+                .mockImplementation(() => { });
+            await syncCommand.run();
+            (0, vitest_1.expect)(consoleSpy).toHaveBeenCalledWith(vitest_1.expect.stringContaining('Sync failed:'), 'String error');
+            consoleSpy.mockRestore();
+        });
+        (0, vitest_1.it)('should use default services if not provided', () => {
+            const command = new sync_1.SyncCommand();
+            (0, vitest_1.expect)(command).toBeDefined();
+        });
+    });
+});

package/dist/commands/init.d.ts

@@ -1,6 +1,8 @@
+import { InitService } from '../services/InitService';
+import { RegistryService } from '../services/RegistryService';
 export declare class InitCommand {
-    private detectionService;
+    private initService;
     private registryService;
-    private configService;
+    constructor(initService?: InitService, registryService?: RegistryService);
     run(): Promise<void>;
 }

package/dist/commands/init.js

@@ -6,32 +6,19 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.InitCommand = void 0;
 const fs_extra_1 = __importDefault(require("fs-extra"));
 const inquirer_1 = __importDefault(require("inquirer"));
-const js_yaml_1 = __importDefault(require("js-yaml"));
 const path_1 = __importDefault(require("path"));
 const picocolors_1 = __importDefault(require("picocolors"));
 const constants_1 = require("../constants");
-const ConfigService_1 = require("../services/ConfigService");
-const DetectionService_1 = require("../services/DetectionService");
+const InitService_1 = require("../services/InitService");
 const RegistryService_1 = require("../services/RegistryService");
 class InitCommand {
-    constructor() {
-        this.detectionService = new DetectionService_1.DetectionService();
-        this.registryService = new RegistryService_1.RegistryService();
-        this.configService = new ConfigService_1.ConfigService();
+    constructor(initService, registryService) {
+        this.initService = initService || new InitService_1.InitService();
+        this.registryService = registryService || new RegistryService_1.RegistryService();
     }
     async run() {
         const configPath = path_1.default.join(process.cwd(), '.skillsrc');
-        const detectionResults = await this.detectionService.detectFrameworks();
-        const agentDetection = await this.detectionService.detectAgents();
-        const anyAgentDetected = Object.values(agentDetection).some((d) => d);
-        const shouldCheck = (id) => anyAgentDetected ? agentDetection[id] : true;
-        const { categories: supportedCategories, metadata } = await this.registryService.discoverRegistry(constants_1.DEFAULT_REGISTER);
-        const choices = constants_1.SUPPORTED_FRAMEWORKS.map((f) => ({
-            name: supportedCategories.includes(f.id)
-                ? f.name
-                : `${f.name} (Coming Soon)`,
-            value: f.id,
-        }));
+        // 1. Check for existing config
         if (await fs_extra_1.default.pathExists(configPath)) {
             const { overwrite } = await inquirer_1.default.prompt([
                 {
@@ -46,24 +33,25 @@ class InitCommand {
                 return;
             }
         }
+        // 2. Gather context data
+        const context = await this.initService.getInitializationContext();
+        const { categories, metadata } = await this.registryService.discoverRegistry(constants_1.DEFAULT_REGISTER);
+        // 3. Prepare Choices
+        const { frameworkChoices, agentChoices, defaultFramework } = this.initService.getPromptChoices(context, categories);
+        // 4. Prompt User
         const answers = await inquirer_1.default.prompt([
             {
                 type: 'list',
                 name: 'framework',
                 message: 'Select Framework:',
-                choices: choices,
-                default: constants_1.SUPPORTED_FRAMEWORKS.find((f) => detectionResults[f.id])?.id ||
-                    'flutter',
+                choices: frameworkChoices,
+                default: defaultFramework,
             },
             {
                 type: 'checkbox',
                 name: 'agents',
                 message: 'Select AI Agents you use:',
-                choices: constants_1.SUPPORTED_AGENTS.map((a) => ({
-                    name: `${a.name} (${a.path}/)`,
-                    value: a.id,
-                    checked: shouldCheck(a.id),
-                })),
+                choices: agentChoices,
             },
             {
                 type: 'input',
@@ -72,23 +60,10 @@ class InitCommand {
                 default: constants_1.DEFAULT_REGISTER,
             },
         ]);
-        const frameworkId = answers.framework;
-        const frameworkDef = constants_1.SUPPORTED_FRAMEWORKS.find((f) => f.id === frameworkId);
-        const config = this.configService.buildInitialConfig(frameworkId, answers.agents, answers.registry, metadata, frameworkDef?.languages || []);
-        const projectDeps = await this.detectionService.getProjectDeps();
-        this.configService.applyDependencyExclusions(config, frameworkId, projectDeps);
-        const commentHeader = `# Auto-detected configuration generated by agent-skills-standard init
-#
-# Presence in 'skills' list = Active. To disable a skill, remove it from the list.
-# 'exclude': IDs of sub-skills to skip during sync (auto-populated with undetected skills).
-# 'custom_overrides': IDs of skills to PROTECT. Use this if you have modified a standard 
-# skill locally and don't want the CLI to overwrite it.
-#
-# Run 'ags list-skills' to view all available skills.
-`;
-        await fs_extra_1.default.writeFile(configPath, commentHeader + js_yaml_1.default.dump(config));
+        // 5. Build and Save
+        await this.initService.buildAndSaveConfig(answers, metadata);
         console.log(picocolors_1.default.green('\n✅ Initialized .skillsrc with your preferences!'));
-        console.log(picocolors_1.default.gray(`   Selected framework: ${frameworkId}`));
+        console.log(picocolors_1.default.gray(`   Selected framework: ${answers.framework}`));
         console.log(picocolors_1.default.cyan('\nNext step: Run `agent-skills-standard sync` to generate rule files.'));
     }
 }

package/dist/commands/list-skills.d.ts

@@ -1,6 +1,10 @@
+import { ConfigService } from '../services/ConfigService';
+import { DetectionService } from '../services/DetectionService';
+import { SkillService } from '../services/SkillService';
 export declare class ListSkillsCommand {
+    private configService;
     private detectionService;
-    private githubService;
+    private skillService;
+    constructor(configService?: ConfigService, detectionService?: DetectionService, skillService?: SkillService);
     run(): Promise<void>;
-    private parseRegistryUrl;
 }

package/dist/commands/list-skills.js

@@ -4,20 +4,20 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
 };
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.ListSkillsCommand = void 0;
-const fs_extra_1 = __importDefault(require("fs-extra"));
 const inquirer_1 = __importDefault(require("inquirer"));
-const js_yaml_1 = __importDefault(require("js-yaml"));
-const path_1 = __importDefault(require("path"));
 const picocolors_1 = __importDefault(require("picocolors"));
 const constants_1 = require("../constants");
+const ConfigService_1 = require("../services/ConfigService");
 const DetectionService_1 = require("../services/DetectionService");
-const GithubService_1 = require("../services/GithubService");
+const SkillService_1 = require("../services/SkillService");
 class ListSkillsCommand {
-    constructor() {
-        this.detectionService = new DetectionService_1.DetectionService();
-        this.githubService = new GithubService_1.GithubService(process.env.GITHUB_TOKEN);
+    constructor(configService, detectionService, skillService) {
+        this.configService = configService || new ConfigService_1.ConfigService();
+        this.detectionService = detectionService || new DetectionService_1.DetectionService();
+        this.skillService = skillService || new SkillService_1.SkillService();
     }
     async run() {
+        // 1. Get framework choice from user
         const choices = constants_1.SUPPORTED_FRAMEWORKS.map((f) => ({
             name: f.name,
             value: f.id,
@@ -31,64 +31,22 @@ class ListSkillsCommand {
                 default: choices.find((c) => c.value === constants_1.Framework.Flutter)?.value,
             },
         ]);
+        // 2. Load context
         const projectDeps = await this.detectionService.getProjectDeps();
+        const registryUrl = await this.configService.getRegistryUrl();
         console.log(picocolors_1.default.green(`\nAvailable skills for ${framework}:`));
-        // Determine registry URL (from .skillsrc or default)
-        const defaultRegistry = 'https://github.com/HoangNguyen0403/agent-skills-standard';
-        let registryUrl = defaultRegistry;
-        const configPath = path_1.default.join(process.cwd(), '.skillsrc');
-        if (await fs_extra_1.default.pathExists(configPath)) {
-            const raw = await fs_extra_1.default.readFile(configPath, 'utf8');
-            const conf = js_yaml_1.default.load(raw);
-            if (conf && conf.registry)
-                registryUrl = conf.registry;
-        }
-        // Try to fetch the repo tree to list all skills under skills/<framework>/
-        let skillFolders = [];
-        try {
-            const parsed = this.parseRegistryUrl(registryUrl);
-            if (parsed) {
-                // We assume 'main' branch if not specified (legacy behavior of fetchRepoTree handled default,
-                // but for listing skills, 'main' is a safe default for now or we could fetch repo info)
-                // I'll stick to 'main' to keep it simple as in RegistryService refactor.
-                const treeResult = await this.githubService.getRepoTree(parsed.owner, parsed.repo, 'main');
-                if (treeResult && Array.isArray(treeResult.tree)) {
-                    const allFiles = treeResult.tree || [];
-                    skillFolders = Array.from(new Set(allFiles
-                        .filter((f) => f.path.startsWith(`skills/${framework}/`))
-                        .map((f) => {
-                        const parts = f.path.split('/');
-                        return parts[2];
-                    })
-                        .filter(Boolean)));
-                }
-            }
-        }
-        catch {
-            // ignore network errors and fall back to detection registry
-        }
-        // Fallback: use detection registry skill ids if we couldn't fetch tree
-        const detectSkills = constants_1.SKILL_DETECTION_REGISTRY[framework] || [];
-        if (!skillFolders || skillFolders.length === 0) {
-            skillFolders = detectSkills.map((s) => s.id);
-        }
-        // For each discovered skill, show whether it would be detected (if we have rules)
-        for (const skillName of skillFolders.sort()) {
-            const detectRule = detectSkills.find((x) => x.id === skillName);
-            let mark = picocolors_1.default.yellow('no-rule');
-            if (detectRule) {
-                const present = detectRule.packages.some((p) => projectDeps.has(p));
-                mark = present ? picocolors_1.default.green('detected') : picocolors_1.default.gray('not-detected');
-            }
-            console.log(`- ${skillName} (${mark})`);
+        // 3. Fetch skills with status
+        const skills = await this.skillService.getSkillsWithStatus(framework, registryUrl, projectDeps);
+        // 4. Output results
+        for (const skill of skills) {
+            const mark = skill.status === 'no-rule'
+                ? picocolors_1.default.yellow('no-rule')
+                : skill.status === 'detected'
+                    ? picocolors_1.default.green('detected')
+                    : picocolors_1.default.gray('not-detected');
+            console.log(`- ${skill.name} (${mark})`);
         }
         console.log('\nTip: Use the .skillsrc exclude array to disable/enable sub-skills before running sync.');
     }
-    parseRegistryUrl(registryUrl) {
-        const m = registryUrl.match(/github\.com\/([^/]+)\/([^/]+)/i);
-        if (!m)
-            return null;
-        return { owner: m[1], repo: m[2].replace(/\.git$/, '') };
-    }
 }
 exports.ListSkillsCommand = ListSkillsCommand;

package/dist/commands/sync.d.ts

@@ -1,8 +1,10 @@
+import { ConfigService } from '../services/ConfigService';
+import { DetectionService } from '../services/DetectionService';
+import { SyncService } from '../services/SyncService';
 export declare class SyncCommand {
     private configService;
-    private githubService;
+    private detectionService;
+    private syncService;
+    constructor(configService?: ConfigService, detectionService?: DetectionService, syncService?: SyncService);
     run(): Promise<void>;
-    private checkForUpdates;
-    private assembleFromRemote;
-    private writeToTargets;
 }