@packmind/cli 0.20.0 → 0.21.0

package/main.cjs

@@ -3852,7 +3852,7 @@ var require_package = __commonJS({
   "apps/cli/package.json"(exports2, module2) {
     module2.exports = {
       name: "@packmind/cli",
-      version: "0.20.0",
+      version: "0.21.0",
       description: "A command-line interface for Packmind linting and code quality checks",
       private: false,
       bin: {
@@ -3887,6 +3887,7 @@ var require_package = __commonJS({
       },
       dependencies: {
         "@types/inquirer": "^9.0.9",
+        diff: "^8.0.3",
         inquirer: "^13.0.2",
         zod: "^4.3.5",
         semver: "^6.3.1"
@@ -5060,6 +5061,39 @@ var ChangeProposalSubmittedEvent = class extends UserEvent {
   }
 };
 
+// packages/types/src/playbookChangeManagement/applier/DiffService.ts
+var import_diff = require("diff");
+
+// packages/types/src/playbookChangeManagement/applier/types.ts
+var STANDARD_CHANGE_TYPES = [
+  "updateStandardName" /* updateStandardName */,
+  "updateStandardScope" /* updateStandardScope */,
+  "updateStandardDescription" /* updateStandardDescription */,
+  "addRule" /* addRule */,
+  "updateRule" /* updateRule */,
+  "deleteRule" /* deleteRule */,
+  "removeStandard" /* removeStandard */
+];
+var RECIPE_CHANGE_TYPES = [
+  "updateCommandName" /* updateCommandName */,
+  "updateCommandDescription" /* updateCommandDescription */,
+  "removeCommand" /* removeCommand */
+];
+var SKILL_CHANGE_TYPES = [
+  "updateSkillName" /* updateSkillName */,
+  "updateSkillDescription" /* updateSkillDescription */,
+  "updateSkillPrompt" /* updateSkillPrompt */,
+  "updateSkillMetadata" /* updateSkillMetadata */,
+  "updateSkillLicense" /* updateSkillLicense */,
+  "updateSkillCompatibility" /* updateSkillCompatibility */,
+  "updateSkillAllowedTools" /* updateSkillAllowedTools */,
+  "addSkillFile" /* addSkillFile */,
+  "updateSkillFileContent" /* updateSkillFileContent */,
+  "updateSkillFilePermissions" /* updateSkillFilePermissions */,
+  "deleteSkillFile" /* deleteSkillFile */,
+  "removeSkill" /* removeSkill */
+];
+
 // apps/cli/src/application/useCases/ExecuteSingleFileAstUseCase.ts
 var ExecuteSingleFileAstUseCase = class _ExecuteSingleFileAstUseCase {
   constructor(linterExecutionUseCase) {
@@ -5123,30 +5157,30 @@ var GitService = class {
     this.gitRunner = gitRunner;
     this.logger = logger2;
   }
-  getGitRepositoryRoot(path21) {
+  getGitRepositoryRoot(path22) {
     try {
       const { stdout } = this.gitRunner("rev-parse --show-toplevel", {
-        cwd: path21
+        cwd: path22
       });
       const gitRoot = stdout.trim();
       this.logger.debug("Resolved git repository root", {
-        inputPath: path21,
+        inputPath: path22,
         gitRoot
       });
       return gitRoot;
     } catch (error) {
       if (error instanceof Error) {
         throw new Error(
-          `Failed to get Git repository root. The path '${path21}' does not appear to be inside a Git repository.
+          `Failed to get Git repository root. The path '${path22}' does not appear to be inside a Git repository.
 ${error.message}`
         );
       }
       throw new Error("Failed to get Git repository root: Unknown error");
     }
   }
-  tryGetGitRepositoryRoot(path21) {
+  tryGetGitRepositoryRoot(path22) {
     try {
-      return this.getGitRepositoryRoot(path21);
+      return this.getGitRepositoryRoot(path22);
     } catch {
       return null;
     }
@@ -6400,10 +6434,10 @@ var PackmindHttpClient = class {
       return null;
     }
   }
-  async request(path21, options = {}) {
+  async request(path22, options = {}) {
     const { host } = this.getAuthContext();
     const { method = "GET", body } = options;
-    const url = `${host}${path21}`;
+    const url = `${host}${path22}`;
     try {
       const response = await fetch(url, {
         method,
@@ -9924,10 +9958,10 @@ async function defaultPromptForCode() {
     input: process.stdin,
     output: process.stdout
   });
-  return new Promise((resolve8) => {
+  return new Promise((resolve9) => {
     rl.question("Enter the login code from the browser: ", (answer) => {
       rl.close();
-      resolve8(answer.trim());
+      resolve9(answer.trim());
     });
   });
 }
@@ -9961,7 +9995,7 @@ async function defaultExchangeCodeForApiKey(code, host) {
   return await response.json();
 }
 function defaultStartCallbackServer() {
-  return new Promise((resolve8, reject) => {
+  return new Promise((resolve9, reject) => {
     let timeoutId = null;
     const server = http.createServer((req, res) => {
       res.setHeader("Access-Control-Allow-Origin", "*");
@@ -9974,7 +10008,7 @@ function defaultStartCallbackServer() {
         if (timeoutId) {
           clearTimeout(timeoutId);
         }
-        resolve8(code);
+        resolve9(code);
         setImmediate(() => {
           server.close();
         });
@@ -10051,8 +10085,8 @@ var LogoutUseCase = class {
   constructor(deps) {
     this.deps = {
       getCredentialsPath: deps?.getCredentialsPath ?? getCredentialsPath,
-      fileExists: deps?.fileExists ?? ((path21) => fs7.existsSync(path21)),
-      deleteFile: deps?.deleteFile ?? ((path21) => fs7.unlinkSync(path21)),
+      fileExists: deps?.fileExists ?? ((path22) => fs7.existsSync(path22)),
+      deleteFile: deps?.deleteFile ?? ((path22) => fs7.unlinkSync(path22)),
       hasEnvVar: deps?.hasEnvVar ?? (() => !!process.env[ENV_VAR_NAME2])
     };
   }
@@ -10865,7 +10899,7 @@ var UploadSkillUseCase = class {
 };
 
 // apps/cli/src/application/useCases/diffStrategies/CommandDiffStrategy.ts
-var import_diff = require("diff");
+var import_diff2 = require("diff");
 var fs11 = __toESM(require("fs/promises"));
 var path12 = __toESM(require("path"));
 
@@ -10903,7 +10937,7 @@ var CommandDiffStrategy = class {
     }
     const serverBody = stripFrontmatter(file.content);
     const localBody = stripFrontmatter(localContent);
-    const changes = (0, import_diff.diffLines)(serverBody, localBody);
+    const changes = (0, import_diff2.diffLines)(serverBody, localBody);
     const hasDifferences = changes.some(
       (change) => change.added || change.removed
     );
@@ -10928,7 +10962,7 @@ var CommandDiffStrategy = class {
 };
 
 // apps/cli/src/application/useCases/diffStrategies/SkillDiffStrategy.ts
-var import_diff2 = require("diff");
+var import_diff3 = require("diff");
 var fs12 = __toESM(require("fs/promises"));
 var path13 = __toESM(require("path"));
 var SkillDiffStrategy = class {
@@ -11000,7 +11034,7 @@ var SkillDiffStrategy = class {
     if (!serverParsed || !localParsed) {
       const serverBody = stripFrontmatter(file.content);
       const localBody = stripFrontmatter(localContent);
-      const changes = (0, import_diff2.diffLines)(serverBody, localBody);
+      const changes = (0, import_diff3.diffLines)(serverBody, localBody);
       const hasDifferences = changes.some(
         (change) => change.added || change.removed
       );
@@ -12261,7 +12295,7 @@ function isNotLoggedInError(error) {
 }
 async function lintHandler(args2, deps) {
   const {
-    path: path21,
+    path: path22,
     draft,
     rule,
     language,
@@ -12282,7 +12316,7 @@ async function lintHandler(args2, deps) {
     throw new Error("option --rule is required to use --draft mode");
   }
   const startedAt = Date.now();
-  const targetPath = path21 ?? ".";
+  const targetPath = path22 ?? ".";
   const absolutePath = resolvePath(targetPath);
   if (diff) {
     const gitRoot = await packmindCliHexa.tryGetGitRepositoryRoot(absolutePath);
@@ -12565,7 +12599,7 @@ function extractWasmFiles() {
 // apps/cli/src/main.ts
 var import_dotenv = require("dotenv");
 var fs19 = __toESM(require("fs"));
-var path20 = __toESM(require("path"));
+var path21 = __toESM(require("path"));
 
 // apps/cli/src/infra/commands/InstallCommand.ts
 var import_cmd_ts2 = __toESM(require_cjs());
@@ -13736,7 +13770,7 @@ async function promptAgentsWithReadline(choices) {
   output.write("\n");
   const preselected = choices.map((c, i) => c.checked ? i + 1 : null).filter((i) => i !== null);
   const defaultValue = preselected.length > 0 ? preselected.join(",") : "1,2,3";
-  return new Promise((resolve8) => {
+  return new Promise((resolve9) => {
     rl.question(
       `Enter numbers separated by commas (default: ${defaultValue}): `,
       (answer) => {
@@ -13747,7 +13781,7 @@ async function promptAgentsWithReadline(choices) {
         const numbersStr = trimmed === "" ? defaultValue : trimmed;
         const numbers = numbersStr.split(",").map((s) => parseInt(s.trim(), 10)).filter((n) => !isNaN(n) && n >= 1 && n <= choices.length);
         const selectedAgents = numbers.map((n) => choices[n - 1].value);
-        resolve8(selectedAgents);
+        resolve9(selectedAgents);
       }
     );
   });
@@ -14660,10 +14694,10 @@ var import_fs20 = require("fs");
 var import_cmd_ts19 = __toESM(require_cjs());
 
 // apps/cli/src/infra/utils/diffFormatter.ts
-var import_diff3 = require("diff");
+var import_diff4 = require("diff");
 init_source();
 function formatContentDiff(oldContent, newContent) {
-  const changes = (0, import_diff3.diffLines)(oldContent, newContent);
+  const changes = (0, import_diff4.diffLines)(oldContent, newContent);
   const lines = [];
   for (const change of changes) {
     const trimmedValue = change.value.replace(/\n$/, "");
@@ -15299,6 +15333,9 @@ function parseSkillDirectory(files) {
   if (typeof properties.allowedTools === "string") {
     payload.allowedTools = properties.allowedTools;
   }
+  if (properties.metadata != null && typeof properties.metadata === "object") {
+    payload.metadata = properties.metadata;
+  }
   const supportingFiles = files.filter(
     (f) => f.relativePath !== SKILL_MD_FILENAME
   );
@@ -15427,7 +15464,7 @@ async function diffAddHandler(deps) {
   };
   const result = await packmindCliHexa.submitDiffs([[diff]], message);
   for (const err of result.errors) {
-    logErrorConsole(`Failed to submit "${err.name}": ${err.message}`);
+    logErrorConsole(`Failed to submit addition "${err.name}": ${err.message}`);
   }
   const summaryParts = [];
   if (result.submitted > 0) {
@@ -15524,6 +15561,213 @@ function isErrnoException(err) {
   return err instanceof Error && "code" in err;
 }
 
+// apps/cli/src/infra/commands/diffRemoveHandler.ts
+var path18 = __toESM(require("path"));
+var ARTIFACT_TYPE_LABELS2 = {
+  command: "command",
+  standard: "standard",
+  skill: "skill"
+};
+async function diffRemoveHandler(deps) {
+  const {
+    packmindCliHexa,
+    filePath,
+    message: messageFlag,
+    exit,
+    getCwd,
+    existsSync: existsSync25,
+    unlinkSync: unlinkSync5,
+    rmSync: rmSync2
+  } = deps;
+  if (!filePath) {
+    logErrorConsole(
+      'Missing file path. Usage: packmind-cli diff remove <path> -m "message"'
+    );
+    exit(1);
+    return;
+  }
+  const cwd = getCwd();
+  const absolutePath = path18.resolve(cwd, filePath);
+  const artefactResult = resolveArtefactFromPath(absolutePath);
+  if (!artefactResult) {
+    logErrorConsole(
+      `Unsupported file path: ${absolutePath}. File must be in a recognized artefact directory (command, standard, or skill).`
+    );
+    exit(1);
+    return;
+  }
+  const matchPath = artefactResult.artifactType === "skill" ? absolutePath.endsWith("SKILL.md") ? absolutePath : path18.join(absolutePath, "SKILL.md") : absolutePath;
+  const skillDirPath = artefactResult.artifactType === "skill" ? absolutePath.endsWith("SKILL.md") ? path18.dirname(absolutePath) : absolutePath : void 0;
+  const existsCheckPath = skillDirPath ?? absolutePath;
+  if (!existsSync25(existsCheckPath)) {
+    logErrorConsole(`File or directory does not exist: ${filePath}`);
+    exit(1);
+    return;
+  }
+  let configPackages;
+  let configAgents;
+  try {
+    const fullConfig = await packmindCliHexa.readFullConfig(cwd);
+    if (fullConfig) {
+      configPackages = Object.keys(fullConfig.packages);
+      configAgents = fullConfig.agents;
+    } else {
+      configPackages = [];
+    }
+  } catch (err) {
+    logErrorConsole("Failed to parse packmind.json");
+    if (err instanceof Error) {
+      logErrorConsole(err.message);
+    } else {
+      logErrorConsole(String(err));
+    }
+    logErrorConsole(
+      "\n\u{1F4A1} Please fix the packmind.json file or delete it to continue."
+    );
+    exit(1);
+    return;
+  }
+  if (configPackages.length === 0) {
+    logErrorConsole(
+      "No packages configured. Configure packages in packmind.json first."
+    );
+    exit(1);
+    return;
+  }
+  let gitRemoteUrl;
+  let gitBranch;
+  let relativePath;
+  const gitRoot = await packmindCliHexa.tryGetGitRepositoryRoot(cwd);
+  if (gitRoot) {
+    try {
+      gitRemoteUrl = packmindCliHexa.getGitRemoteUrlFromPath(gitRoot);
+      gitBranch = packmindCliHexa.getCurrentBranch(gitRoot);
+      relativePath = cwd.startsWith(gitRoot) ? cwd.slice(gitRoot.length) : "/";
+      if (!relativePath.startsWith("/")) {
+        relativePath = "/" + relativePath;
+      }
+      if (!relativePath.endsWith("/")) {
+        relativePath = relativePath + "/";
+      }
+    } catch (err) {
+      logErrorConsole(
+        `Failed to collect git info: ${err instanceof Error ? err.message : String(err)}`
+      );
+    }
+  }
+  if (!gitRemoteUrl || !gitBranch || !relativePath) {
+    logErrorConsole(
+      "\n\u274C Could not determine git repository info. The diff command requires a git repository with a remote configured."
+    );
+    exit(1);
+    return;
+  }
+  const deployedContent = await packmindCliHexa.getPackmindGateway().deployment.getDeployed({
+    packagesSlugs: configPackages,
+    gitRemoteUrl,
+    gitBranch,
+    relativePath,
+    agents: configAgents
+  });
+  const relativeToGitRoot = matchPath.startsWith(gitRoot) ? matchPath.slice(gitRoot.length) : matchPath;
+  let normalizedFilePath = normalizePath(relativeToGitRoot);
+  if (normalizedFilePath.startsWith("/")) {
+    normalizedFilePath = normalizedFilePath.slice(1);
+  }
+  const deployedFile = deployedContent.fileUpdates.createOrUpdate.find(
+    (file) => normalizePath(file.path) === normalizedFilePath
+  );
+  if (!deployedFile) {
+    const artifactTypeLabel = ARTIFACT_TYPE_LABELS2[artefactResult.artifactType];
+    logErrorConsole(`This ${artifactTypeLabel} does not come from Packmind`);
+    exit(1);
+    return;
+  }
+  if (!deployedFile.artifactId || !deployedFile.spaceId) {
+    logErrorConsole(
+      "Missing artifact metadata. Cannot create change proposal for removal."
+    );
+    exit(1);
+    return;
+  }
+  if (!deployedContent.targetId || !deployedContent.packageIds) {
+    logErrorConsole(
+      "Missing target or package information. Cannot create change proposal for removal."
+    );
+    exit(1);
+    return;
+  }
+  let message;
+  if (messageFlag !== void 0) {
+    const validation = validateMessage(messageFlag);
+    if (!validation.valid) {
+      logErrorConsole(validation.error);
+      exit(1);
+      return;
+    }
+    message = validation.message;
+  } else if (process.stdin.isTTY) {
+    const editorMessage = openEditorForMessage();
+    const validation = validateMessage(editorMessage);
+    if (!validation.valid) {
+      logErrorConsole(
+        "Aborting submission: empty message. Use -m to provide a message."
+      );
+      exit(1);
+      return;
+    }
+    message = validation.message;
+  } else {
+    logErrorConsole(
+      'Non-interactive mode requires -m flag. Usage: packmind-cli diff remove <path> -m "your message"'
+    );
+    exit(1);
+    return;
+  }
+  const changeProposalType = artefactResult.artifactType === "standard" ? "removeStandard" /* removeStandard */ : artefactResult.artifactType === "command" ? "removeCommand" /* removeCommand */ : "removeSkill" /* removeSkill */;
+  const diff = {
+    filePath: absolutePath,
+    type: changeProposalType,
+    payload: {
+      targetId: deployedContent.targetId,
+      packageIds: deployedContent.packageIds
+    },
+    artifactName: deployedFile.artifactName || artefactResult.artifactType,
+    artifactType: artefactResult.artifactType,
+    artifactId: deployedFile.artifactId,
+    spaceId: deployedFile.spaceId
+  };
+  const result = await packmindCliHexa.submitDiffs([[diff]], message);
+  for (const err of result.errors) {
+    logErrorConsole(`Failed to submit removal "${err.name}": ${err.message}`);
+  }
+  if (result.errors.length > 0) {
+    exit(1);
+    return;
+  }
+  if (result.submitted > 0) {
+    logSuccessConsole("Change proposal for removal submitted successfully");
+  } else if (result.alreadySubmitted > 0) {
+    logWarningConsole("Change proposal for removal already submitted");
+  }
+  try {
+    if (skillDirPath) {
+      rmSync2(skillDirPath, { recursive: true });
+      logSuccessConsole(`Directory deleted: ${skillDirPath}`);
+    } else {
+      unlinkSync5(absolutePath);
+      logSuccessConsole(`File deleted: ${filePath}`);
+    }
+  } catch (err) {
+    logErrorConsole(
+      `Failed to delete file: ${err instanceof Error ? err.message : String(err)}`
+    );
+    exit(1);
+    return;
+  }
+  exit(0);
+}
+
 // apps/cli/src/infra/commands/DiffCommand.ts
 var diffCommand = (0, import_cmd_ts19.command)({
   name: "diff",
@@ -15546,7 +15790,7 @@ var diffCommand = (0, import_cmd_ts19.command)({
     positionals: (0, import_cmd_ts19.restPositionals)({
       type: import_cmd_ts19.string,
       displayName: "args",
-      description: "Subcommand and arguments (e.g., add <path>)"
+      description: "Subcommand and arguments (e.g., add <path>, remove <path>)"
     })
   },
   handler: async ({ submit, includeSubmitted, message, positionals }) => {
@@ -15564,6 +15808,19 @@ var diffCommand = (0, import_cmd_ts19.command)({
       });
       return;
     }
+    if (positionals[0] === "remove" || positionals[0] === "rm") {
+      await diffRemoveHandler({
+        packmindCliHexa,
+        filePath: positionals[1],
+        message,
+        exit: process.exit,
+        getCwd: () => process.cwd(),
+        existsSync: (p) => (0, import_fs20.existsSync)(p),
+        unlinkSync: (p) => (0, import_fs20.unlinkSync)(p),
+        rmSync: (p, opts) => (0, import_fs20.rmSync)(p, opts)
+      });
+      return;
+    }
     await diffArtefactsHandler({
       packmindCliHexa,
       exit: process.exit,
@@ -15951,7 +16208,7 @@ var import_cmd_ts24 = __toESM(require_cjs());
 
 // apps/cli/src/application/services/AgentArtifactDetectionService.ts
 var fs18 = __toESM(require("fs/promises"));
-var path18 = __toESM(require("path"));
+var path19 = __toESM(require("path"));
 var AGENT_ARTIFACT_CHECKS = [
   { agent: "claude", paths: [".claude"] },
   { agent: "cursor", paths: [".cursor"] },
@@ -15969,7 +16226,7 @@ var AgentArtifactDetectionService = class {
     const detected = [];
     for (const check of AGENT_ARTIFACT_CHECKS) {
       for (const relativePath of check.paths) {
-        const fullPath = path18.join(baseDirectory, relativePath);
+        const fullPath = path19.join(baseDirectory, relativePath);
         const exists = await this.pathExists(fullPath);
         if (exists) {
           detected.push({
@@ -16066,7 +16323,7 @@ async function promptAgentsWithReadline2(choices) {
   output.write("\n");
   const preselected = choices.map((c, i) => c.checked ? i + 1 : null).filter((i) => i !== null);
   const defaultValue = preselected.length > 0 ? preselected.join(",") : "1,2,3";
-  return new Promise((resolve8) => {
+  return new Promise((resolve9) => {
     rl.question(
       `Enter numbers separated by commas (default: ${defaultValue}): `,
       (answer) => {
@@ -16075,7 +16332,7 @@ async function promptAgentsWithReadline2(choices) {
         const numbersStr = trimmed === "" ? defaultValue : trimmed;
         const numbers = numbersStr.split(",").map((s) => parseInt(s.trim(), 10)).filter((n) => !isNaN(n) && n >= 1 && n <= choices.length);
         const selectedAgents = numbers.map((n) => choices[n - 1].value);
-        resolve8(selectedAgents);
+        resolve9(selectedAgents);
       }
     );
   });
@@ -16334,6 +16591,14 @@ function isLocalNpmPackage(scriptPath) {
   if (!scriptPath) return false;
   return scriptPath.includes(import_path5.default.join("node_modules", "@packmind", "cli"));
 }
+function isHomebrewInstall(executablePath) {
+  try {
+    const realPath = (0, import_fs21.realpathSync)(executablePath);
+    return realPath.includes("/Cellar/");
+  } catch {
+    return false;
+  }
+}
 async function updateHandler(deps) {
   const execBasename = import_path5.default.basename(deps.executablePath).replace(/\.exe$/, "");
   const jsRuntimes = ["node", "bun", "deno"];
@@ -16350,6 +16615,13 @@ async function updateHandler(deps) {
     process.exit(1);
     return;
   }
+  if (isHomebrewInstall(deps.executablePath)) {
+    logInfoConsole(
+      "This CLI was installed via Homebrew.\nTo update, run: brew upgrade packmind-cli"
+    );
+    process.exit(0);
+    return;
+  }
   logInfoConsole(
     `Current version: ${deps.currentVersion} (${deps.isExecutableMode ? "standalone executable" : "npm package"})`
   );
@@ -16431,12 +16703,12 @@ function findEnvFile() {
   const currentDir = process.cwd();
   const gitService = new GitService();
   const gitRoot = gitService.getGitRepositoryRootSync(currentDir);
-  const filesystemRoot = path20.parse(currentDir).root;
+  const filesystemRoot = path21.parse(currentDir).root;
   const stopDir = gitRoot ?? filesystemRoot;
   let searchDir = currentDir;
-  let parentDir = path20.dirname(searchDir);
+  let parentDir = path21.dirname(searchDir);
   while (searchDir !== parentDir) {
-    const envPath2 = path20.join(searchDir, ".env");
+    const envPath2 = path21.join(searchDir, ".env");
     if (fs19.existsSync(envPath2)) {
       return envPath2;
     }
@@ -16444,7 +16716,7 @@ function findEnvFile() {
       return null;
     }
     searchDir = parentDir;
-    parentDir = path20.dirname(searchDir);
+    parentDir = path21.dirname(searchDir);
   }
   return null;
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@packmind/cli",
-  "version": "0.20.0",
+  "version": "0.21.0",
   "description": "A command-line interface for Packmind linting and code quality checks",
   "private": false,
   "bin": {
@@ -47,9 +47,9 @@
     "axios": "1.13.5",
     "bcrypt": "6.0.0",
     "bullmq": "5.63.0",
-    "diff": "8.0.3",
+    "diff": "^8.0.3",
     "dotenv": "16.4.7",
-    "express": "5.1.0",
+    "express": "5.2.1",
     "inquirer": "^13.0.2",
     "minimatch": "9.0.3",
     "open": "11.0.0",