myskill 1.2.2 → 1.2.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "myskill",
-  "version": "1.2.2",
+  "version": "1.2.3",
   "description": "CLI tool for creating and managing AI agent skills",
   "author": "Sarfraz Ahmed <sarfraznawaz2005@gmail.com>",
   "license": "MIT",

package/src/commands/convert.js

@@ -65,6 +65,12 @@ export async function convert(sourcePath, options = {}) {
   } else if (targetPlatform.id === "claude") {
     if (sourceFm["allowed-tools"])
       newFm["allowed-tools"] = sourceFm["allowed-tools"];
+  } else if (targetPlatform.id === "copilot") {
+    if (sourceFm.license) newFm.license = sourceFm.license;
+    if (sourceFm.compatibility) newFm.compatibility = sourceFm.compatibility;
+    if (sourceFm.metadata) newFm.metadata = sourceFm.metadata;
+    if (sourceFm["allowed-tools"])
+      newFm["allowed-tools"] = sourceFm["allowed-tools"];
   }
 
   const newContent = generateSkill(newFm, markdownBody);

package/src/commands/create.js

@@ -168,7 +168,9 @@ export async function create(options = {}) {
     const localBase =
       answers.platform === "opencode"
         ? ".opencode/skill"
-        : `.${answers.platform}/skills`;
+        : answers.platform === "copilot"
+          ? ".github/skills"
+          : `.${answers.platform}/skills`;
     targetDir = path.join(process.cwd(), localBase, answers.name);
   }
 

package/src/commands/list.js

@@ -25,7 +25,11 @@ export async function list(options = {}) {
     const locations = [{ name: "Global", path: globalPath }];
 
     const localBase =
-      platform.id === "opencode" ? ".opencode/skill" : `.${platform.id}/skills`;
+      platform.id === "opencode"
+        ? ".opencode/skill"
+        : platform.id === "copilot"
+          ? ".github/skills"
+          : `.${platform.id}/skills`;
     locations.push({
       name: "Project",
       path: path.join(process.cwd(), localBase),

package/src/platforms/baseSchema.js

@@ -0,0 +1,17 @@
+import { z } from "zod";
+
+export const agentSkillsBaseSchema = z.object({
+  name: z
+    .string()
+    .min(1)
+    .max(64)
+    .regex(
+      /^[a-z0-9]+(-[a-z0-9]+)*$/,
+      "Name must be lowercase alphanumeric with single hyphens, no start/end hyphen",
+    ),
+  description: z.string().min(1).max(1024),
+  license: z.string().optional(),
+  compatibility: z.string().max(500).optional(),
+  metadata: z.record(z.string()).optional(),
+  "allowed-tools": z.union([z.string(), z.array(z.string())]).optional(),
+});

package/src/platforms/claude.js

@@ -1,48 +1,14 @@
 import { z } from "zod";
 import path from "path";
 import os from "os";
-import { getConfig } from "../utils/config.js";
-
-// We need to make this async or a function to read config
-// But index.js exports static objects.
-// Let's change the platforms to be functions or getters?
-// This is a breaking change for internal architecture.
-// Or we can load config synchronously? No, fs-extra is async.
-// Best approach: Keep the export structure but make defaultPath a getter or load it once at startup.
-// Since `bin/myskill.js` is async, we can initialize platforms there?
-// Or just reading config synchronously if we must?
-// Node.js supports top-level await in modules.
-
-// Let's try top-level await for config loading since we are in ESM.
-let config = {};
-try {
-  // We can't easily do top level await here without fs/promises and maybe it slows down startup.
-  // Instead, let's make `defaultPath` a property we resolve when needed, or check config inside commands.
-  // BUT the prompt definitions and validation logic depend on platform definition.
-  // The path is mostly used in commands (create, list, etc).
-  // So let's change `defaultPath` to a function `getDefaultPath()`.
-} catch (e) {}
-
-// For now, I will modify the definitions to export functions or include a resolution helper.
-// Actually, simplest is to just expose the hardcoded default as fallback,
-// and utility function `getPlatformPath(platformId)` in `utils/config.js` or `platforms/index.js`.
+import { agentSkillsBaseSchema } from "./baseSchema.js";
 
 export const claude = {
   id: "claude",
   name: "Claude Code",
   docsUrl: "https://code.claude.com/docs/en/skills",
   defaultPath: path.join(os.homedir(), ".claude", "skills"),
-  schema: z.object({
-    name: z
-      .string()
-      .min(1)
-      .max(64)
-      .regex(
-        /^[a-z0-9-]+$/,
-        "Name must be lowercase alphanumeric with hyphens",
-      ),
-    description: z.string().min(1).max(1024),
-    "allowed-tools": z.union([z.string(), z.array(z.string())]).optional(),
+  schema: agentSkillsBaseSchema.extend({
     model: z.string().optional(),
     context: z.enum(["fork"]).optional(),
     agent: z.string().optional(),

package/src/platforms/copilot.js

@@ -0,0 +1,24 @@
+import { z } from "zod";
+import path from "path";
+import os from "os";
+import { agentSkillsBaseSchema } from "./baseSchema.js";
+
+export const copilot = {
+  id: "copilot",
+  name: "GitHub Copilot CLI",
+  docsUrl: "https://docs.github.com/copilot/concepts/agents/about-agent-skills",
+  defaultPath: path.join(os.homedir(), ".copilot", "skills"),
+  schema: agentSkillsBaseSchema,
+  prompts: [
+    {
+      type: "input",
+      name: "license",
+      message: "License (e.g., MIT):",
+    },
+    {
+      type: "input",
+      name: "compatibility",
+      message: "Compatibility (e.g., github-copilot):",
+    },
+  ],
+};

package/src/platforms/index.js

@@ -1,14 +1,16 @@
 import { claude } from "./claude.js";
-import { opencode } from "./opencode.js";
+import { copilot } from "./copilot.js";
 import { codex } from "./codex.js";
 import { gemini } from "./gemini.js";
+import { opencode } from "./opencode.js";
 import { getConfig } from "../utils/config.js";
 
 export const platforms = {
   claude,
-  opencode,
+  copilot,
   codex,
   gemini,
+  opencode,
 };
 
 export function getPlatform(id) {

package/src/utils/skills.js

@@ -5,7 +5,6 @@ import { platforms, getPlatformPath } from "../platforms/index.js";
 
 export function detectPlatform(frontmatter) {
   if (
-    frontmatter["allowed-tools"] ||
     frontmatter.context ||
     frontmatter.hooks ||
     frontmatter.agent ||
@@ -14,6 +13,16 @@ export function detectPlatform(frontmatter) {
     return platforms.claude;
   }
 
+  if (
+    frontmatter["allowed-tools"] &&
+    !frontmatter.context &&
+    !frontmatter.hooks &&
+    !frontmatter.agent &&
+    !frontmatter["user-invocable"]
+  ) {
+    return platforms.copilot;
+  }
+
   if (frontmatter.license || frontmatter.compatibility) {
     return platforms.opencode;
   }
@@ -78,7 +87,11 @@ export async function getAllInstalledSkills() {
   for (const platform of Object.values(platforms)) {
     const globalPath = await getPlatformPath(platform.id);
     const localBase =
-      platform.id === "opencode" ? ".opencode/skill" : `.${platform.id}/skills`;
+      platform.id === "opencode"
+        ? ".opencode/skill"
+        : platform.id === "copilot"
+          ? ".github/skills"
+          : `.${platform.id}/skills`;
     const locations = [
       { name: "Global", path: globalPath },
       { name: "Project", path: path.join(process.cwd(), localBase) },