feat!: make skill.when accept a context object

Author: Timeless0911

src/index.ts

@@ -242,7 +242,7 @@ function filterExtraSkills(
   // values are handled separately in `getSkills`.
   return extraSkills?.filter((extraSkill) => {
     const when = extraSkill.when ?? (() => true);
-    return templateName ? when(templateName, tools) : true;
+    return templateName ? when({ templateName, tools }) : true;
   });
 }
 
@@ -380,7 +380,7 @@ type ExtraSkill = {
    * selected template/tools. Explicit `--skill` values and `--help` remain
    * unfiltered so CLI input stays authoritative and help stays discoverable.
    */
-  when?: (templateName: string, tools: string[]) => boolean;
+  when?: (context: { templateName: string; tools: string[] }) => boolean;
   order?: 'pre' | 'post';
 };
 

test/help.test.ts

@@ -118,19 +118,19 @@ test('help message lists all optional skills even when template and tools are pr
           value: 'shared-docs',
           label: 'Shared Docs',
           source: 'acme/skills',
-          when: (templateName) => templateName === 'vanilla',
+          when: ({ templateName }) => templateName === 'vanilla',
         },
         {
           value: 'react-docs',
           label: 'React Docs',
           source: 'acme/skills',
-          when: (templateName) => templateName === 'react',
+          when: ({ templateName }) => templateName === 'react',
         },
         {
           value: 'rstest-best-practices',
           label: 'Rstest Best Practices',
           source: 'rstackjs/agent-skills',
-          when: (_templateName, selectedTools) => selectedTools.includes('rstest'),
+          when: ({ tools }) => tools.includes('rstest'),
         },
       ],
       argv: ['node', 'test', '--help', '--template', 'vanilla', '--tools', 'biome'],

test/skills.test.ts

@@ -746,14 +746,14 @@ test('should honor explicit --skill values even when they are hidden by template
         label: 'React Docs',
         source: 'acme/skills',
         skill: 'docs/react',
-        when: (templateName) => templateName === 'react',
+        when: ({ templateName }) => templateName === 'react',
       },
       {
         value: 'shared-docs',
         label: 'Shared Docs',
         source: 'acme/skills',
         skill: 'docs/shared',
-        when: (templateName) => templateName === 'vanilla',
+        when: ({ templateName }) => templateName === 'vanilla',
       },
     ],
     argv: [
@@ -818,7 +818,7 @@ test('should show tool-gated skills in the prompt when the required tool is sele
         value: 'rstest-best-practices',
         label: 'Rstest Best Practices',
         source: 'rstackjs/agent-skills',
-        when: (_templateName, selectedTools) => selectedTools.includes('rstest'),
+        when: ({ tools }) => tools.includes('rstest'),
       },
     ],
     argv: ['node', 'test', projectDir, '--tools', 'rstest'],
@@ -864,7 +864,7 @@ test('should honor explicit --skill values even when the required tool is not se
         value: 'rstest-best-practices',
         label: 'Rstest Best Practices',
         source: 'rstackjs/agent-skills',
-        when: (_templateName, selectedTools) => selectedTools.includes('rstest'),
+        when: ({ tools }) => tools.includes('rstest'),
       },
     ],
     argv: [