codex-plugin-doctor 1.29.0 → 1.31.0

package/README.md

@@ -400,9 +400,9 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v5
-      - uses: Esquetta/CodexPluginDoctor@v1.29.0
+      - uses: Esquetta/CodexPluginDoctor@v1.31.0
         with:
-          version: "1.29.0"
+          version: "1.31.0"
           path: .
           runtime: "true"
           policy: codex-publish

package/dist/core/init-git-hooks.d.ts

@@ -3,6 +3,12 @@ export interface InitGitHooksResult {
     hookPaths: string[];
     preExisting: string[];
 }
+export interface RemoveGitHooksResult {
+    rootPath: string;
+    removed: string[];
+    skipped: string[];
+}
+export declare function removeGitHooks(targetPath: string): Promise<RemoveGitHooksResult>;
 export declare function initGitHooks(targetPath: string, options?: {
     force?: boolean;
 }): Promise<InitGitHooksResult>;

package/dist/core/init-git-hooks.js

@@ -54,6 +54,27 @@ async function writeHook(hookPath, content, force) {
     }
     return exists;
 }
+export async function removeGitHooks(targetPath) {
+    const rootPath = path.resolve(targetPath);
+    const hooksDir = path.join(rootPath, ".git", "hooks");
+    const hookNames = ["pre-commit", "pre-push"];
+    const removed = [];
+    const skipped = [];
+    for (const name of hookNames) {
+        const hookPath = path.join(hooksDir, name);
+        if (await fileExists(hookPath)) {
+            const content = await import("node:fs/promises").then((m) => m.readFile(hookPath, "utf8"));
+            if (content.includes("Generated by codex-plugin-doctor init git-hooks")) {
+                await import("node:fs/promises").then((m) => m.rm(hookPath));
+                removed.push(hookPath);
+            }
+            else {
+                skipped.push(hookPath);
+            }
+        }
+    }
+    return { rootPath, removed, skipped };
+}
 export async function initGitHooks(targetPath, options = {}) {
     const rootPath = path.resolve(targetPath);
     const hooksDir = path.join(rootPath, ".git", "hooks");

package/dist/index.d.ts

@@ -26,5 +26,5 @@ export { applyPolicyToDoctorConfig, applyPolicyToSecurityAudit, parsePolicyPack,
 export { buildGenericMcpDoctor, renderGenericMcpDoctor, renderGenericMcpDoctorJson, type GenericMcpDoctorReport } from "./mcp/generic-mcp-doctor.js";
 export { watchPlugin, type WatchPluginOptions, type WatchPluginResult } from "./core/watch-plugin.js";
 export { buildDepAudit, renderDepAudit, renderDepAuditJson, type DepAuditReport, type DepAuditVulnerability } from "./core/dep-audit.js";
-export { initGitHooks, type InitGitHooksResult } from "./core/init-git-hooks.js";
+export { initGitHooks, removeGitHooks, type InitGitHooksResult, type RemoveGitHooksResult } from "./core/init-git-hooks.js";
 export declare function runCheck(targetPath: string, options?: CheckOptions): Promise<CheckResult>;

package/dist/index.js

@@ -26,7 +26,7 @@ export { applyPolicyToDoctorConfig, applyPolicyToSecurityAudit, parsePolicyPack,
 export { buildGenericMcpDoctor, renderGenericMcpDoctor, renderGenericMcpDoctorJson } from "./mcp/generic-mcp-doctor.js";
 export { watchPlugin } from "./core/watch-plugin.js";
 export { buildDepAudit, renderDepAudit, renderDepAuditJson } from "./core/dep-audit.js";
-export { initGitHooks } from "./core/init-git-hooks.js";
+export { initGitHooks, removeGitHooks } from "./core/init-git-hooks.js";
 export async function runCheck(targetPath, options = {}) {
     return validatePlugin(targetPath, options);
 }

package/dist/run-cli.js

@@ -32,7 +32,7 @@ import { renderClientDoctor, renderEnvironmentDoctor, renderEnvironmentDoctorJso
 import { initCiWorkflow } from "./core/init-ci.js";
 import { watchPlugin } from "./core/watch-plugin.js";
 import { buildDepAudit, renderDepAudit, renderDepAuditJson } from "./core/dep-audit.js";
-import { initGitHooks } from "./core/init-git-hooks.js";
+import { initGitHooks, removeGitHooks } from "./core/init-git-hooks.js";
 import { initPluginPackage, initPluginTemplates, isInitPluginTemplate } from "./core/init-plugin.js";
 import { runCheck } from "./index.js";
 import { buildGenericMcpDoctor, renderGenericMcpDoctor, renderGenericMcpDoctorJson } from "./mcp/generic-mcp-doctor.js";
@@ -78,7 +78,7 @@ const defaultIo = {
     }
 };
 function printUsage(io) {
-    io.writeStderr("Usage: codex-plugin-doctor check <path|--installed> [filter] [--policy codex-publish|mcp-strict|security] [--compat] [--json|--markdown|--badge-json|--badge-markdown] [--output <path>] [--history <path>] [--runtime] [--require-runtime-approval --runtime-approval-digest <digest>] [--verbose-runtime] [--explain] [--no-animations] [--ascii]\n       codex-plugin-doctor audit --installed [filter] [--policy codex-publish|mcp-strict|security] [--security] [--compat] [--json] [--output <path>] [--cache] [--changed]\n       codex-plugin-doctor audit deps <path> [--json] [--output <path>]\n       codex-plugin-doctor mcp <path> [--json] [--output <path>]\n       codex-plugin-doctor security <path> [--policy security] [--json|--scorecard]\n       codex-plugin-doctor compat <path> [--all|--client <client>] [--json] [--scorecard] [--output <path>] [--install-preview|--apply --backup]\n       codex-plugin-doctor suppress add <path> [--fingerprint <sha256> --reason <text> --expires-at YYYY-MM-DD] [--config <path>] [--json]\n       codex-plugin-doctor suppress list <path> [--config <path>] [--json]\n       codex-plugin-doctor suppress remove <path> [--fingerprint <sha256>|--index <n>] [--config <path>] [--json]\n       codex-plugin-doctor fix <path> (--dry-run|--interactive --backup|--apply --backup)\n       codex-plugin-doctor history <history.jsonl> [--json] [--fail-on-regression]\n       codex-plugin-doctor watch <path> [--runtime] [--json] [--output <path>] [--debounce-ms <ms>]\n       codex-plugin-doctor doctor [npm <package>|contract|corpus|runtime-plan <path> [--json|--markdown] [--output <path>]|runtime-policy <path> [--json] [--output <path>]|review-bundle <path> --output <dir> --sign-key-env NAME [--json] [--allow-dirty] [--allow-untagged]|review-bundle verify <bundle-dir> --target <path> --sign-key-env NAME [--json] [--output <path>] [--failures-only]|review-bundle diff --before <dir> --after <dir> [--json]|attest <path> [--sign-key-env NAME]|attest verify <attestation.json> --target <path> --sign-key-env NAME|release-evidence <path> --sign-key-env NAME [--allow-dirty] [--allow-untagged] [--require-runtime-approval --runtime-approval-digest <digest>]|release-evidence verify <evidence.json> --target <path> --sign-key-env NAME|release-evidence asset <path> --tag <tag> --output <evidence.json> --sign-key-env NAME [--upload]|mcp <path>|inspector <path>|diff --before <path> --after <path>|recommend <path>|trust <path>|perf <path> [--max-total-ms <ms>] [--max-stage-ms stage=ms]|export --bundle <path>|snapshot|clients|--json|--update-check]\n       codex-plugin-doctor init [path] [--template skill-only|mcp-stdio|mcp-http|full-runtime]\n       codex-plugin-doctor init-ci [path]\n       codex-plugin-doctor init-git-hooks [path] [--force] [--json]\n       codex-plugin-doctor self-test\n       codex-plugin-doctor list --installed\n       codex-plugin-doctor explain <finding-id>\n       codex-plugin-doctor --version\n\nFirst run:\n       codex-plugin-doctor doctor\n       codex-plugin-doctor self-test\n       codex-plugin-doctor init my-plugin\n       codex-plugin-doctor check . --runtime --explain");
+    io.writeStderr("Usage: codex-plugin-doctor check <path|--installed> [filter] [--policy codex-publish|mcp-strict|security] [--compat] [--json|--markdown|--badge-json|--badge-markdown] [--output <path>] [--history <path>] [--runtime] [--require-runtime-approval --runtime-approval-digest <digest>] [--verbose-runtime] [--explain] [--no-animations] [--ascii]\n       codex-plugin-doctor audit --installed [filter] [--policy codex-publish|mcp-strict|security] [--security] [--compat] [--json] [--output <path>] [--cache] [--changed]\n       codex-plugin-doctor audit deps <path> [--json] [--output <path>]\n       codex-plugin-doctor mcp <path> [--json] [--output <path>]\n       codex-plugin-doctor security <path> [--policy security] [--json|--scorecard]\n       codex-plugin-doctor compat <path> [--all|--client <client>] [--json] [--scorecard] [--output <path>] [--install-preview|--apply --backup]\n       codex-plugin-doctor suppress add <path> [--fingerprint <sha256> --reason <text> --expires-at YYYY-MM-DD] [--config <path>] [--json]\n       codex-plugin-doctor suppress list <path> [--config <path>] [--json]\n       codex-plugin-doctor suppress remove <path> [--fingerprint <sha256>|--index <n>] [--config <path>] [--json]\n       codex-plugin-doctor fix <path> (--dry-run|--interactive --backup|--apply --backup)\n       codex-plugin-doctor history <history.jsonl> [--json] [--fail-on-regression]\n       codex-plugin-doctor watch <path> [--runtime] [--json] [--output <path>] [--debounce-ms <ms>]\n       codex-plugin-doctor doctor [npm <package>|contract|corpus|runtime-plan <path> [--json|--markdown] [--output <path>]|runtime-policy <path> [--json] [--output <path>]|review-bundle <path> --output <dir> --sign-key-env NAME [--json] [--allow-dirty] [--allow-untagged]|review-bundle verify <bundle-dir> --target <path> --sign-key-env NAME [--json] [--output <path>] [--failures-only]|review-bundle diff --before <dir> --after <dir> [--json]|attest <path> [--sign-key-env NAME]|attest verify <attestation.json> --target <path> --sign-key-env NAME|release-evidence <path> --sign-key-env NAME [--allow-dirty] [--allow-untagged] [--require-runtime-approval --runtime-approval-digest <digest>]|release-evidence verify <evidence.json> --target <path> --sign-key-env NAME|release-evidence asset <path> --tag <tag> --output <evidence.json> --sign-key-env NAME [--upload]|mcp <path>|inspector <path>|diff --before <path> --after <path>|recommend <path>|trust <path>|perf <path> [--max-total-ms <ms>] [--max-stage-ms stage=ms]|export --bundle <path>|snapshot|clients|--json|--update-check]\n       codex-plugin-doctor init [path] [--template skill-only|mcp-stdio|mcp-http|full-runtime]\n       codex-plugin-doctor init-ci [path]\n       codex-plugin-doctor init-git-hooks [path] [--force] [--json]\n       codex-plugin-doctor init-git-hooks [path] --remove [--json]\n       codex-plugin-doctor self-test\n       codex-plugin-doctor list --installed\n       codex-plugin-doctor explain <finding-id>\n       codex-plugin-doctor --version\n\nFirst run:\n       codex-plugin-doctor doctor\n       codex-plugin-doctor self-test\n       codex-plugin-doctor init my-plugin\n       codex-plugin-doctor check . --runtime --explain");
     io.writeStderr("Suppression governance: codex-plugin-doctor suppress list <path> [--fail-on-expired] [--fail-on-invalid] [--warn-expiring-within-days <days>]\n       codex-plugin-doctor suppress prune <path> [--apply] [--json]");
 }
 const suppressUsageText = [
@@ -1754,8 +1754,38 @@ export async function runCli(args, io = defaultIo, options = {}) {
         const initFlags = maybePath && maybePath.startsWith("--")
             ? [maybePath, ...remainingArgs]
             : remainingArgs;
+        const remove = initFlags.includes("--remove");
         const force = initFlags.includes("--force");
         const jsonOutput = initFlags.includes("--json");
+        if (remove) {
+            const removeResult = await removeGitHooks(targetPath);
+            if (jsonOutput) {
+                io.writeStdout(JSON.stringify({
+                    schemaVersion: "1.0.0",
+                    kind: "doctor.git.hooks.remove",
+                    rootPath: removeResult.rootPath,
+                    removed: removeResult.removed,
+                    skipped: removeResult.skipped
+                }, null, 2));
+                return 0;
+            }
+            const lines = ["Removed Codex Plugin Doctor git hooks", `Root: ${removeResult.rootPath}`];
+            if (removeResult.removed.length > 0) {
+                lines.push(`Removed: ${removeResult.removed.join(", ")}`);
+            }
+            if (removeResult.skipped.length > 0) {
+                lines.push(`Skipped (not generated by doctor): ${removeResult.skipped.join(", ")}`);
+            }
+            if (removeResult.removed.length === 0 && removeResult.skipped.length === 0) {
+                lines.push("No doctor-generated hooks found.");
+            }
+            io.writeStdout(lines.join("\n"));
+            return 0;
+        }
+        if (force && remove) {
+            io.writeStderr("Use either --force or --remove, not both.");
+            return 2;
+        }
         const result = await initGitHooks(targetPath, { force });
         if (jsonOutput) {
             io.writeStdout(JSON.stringify({

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "codex-plugin-doctor",
-  "version": "1.29.0",
+  "version": "1.31.0",
   "description": "CLI-first validator for Codex plugins, skills, and MCP package surfaces with runtime MCP protocol validation.",
   "type": "module",
   "main": "./dist/index.js",