From dc5a62ae5359b8ec27ac19c3d037045d194a8048 Mon Sep 17 00:00:00 2001
From: Rhuan Barreto <rhuan@barreto.work>
Date: Thu, 26 Feb 2026 02:34:38 +0100
Subject: [PATCH] fix: move bundled deps to devDependencies to avoid
 unnecessary installs

The binary bundles all dependencies, so consumers installing archgate
as a dev dependency (for the `./rules` export) were pulling in
commander, inquirer, MCP SDK, and zod unnecessarily. Move all deps
to devDependencies and relocate RuleSetSchema from the public rules
export to the internal loader where it's actually used.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 bun.lock             | 10 ++++------
 package.json         | 12 +++++-------
 src/engine/loader.ts | 18 +++++++++++++++++-
 src/formats/rules.ts | 16 ----------------
 4 files changed, 26 insertions(+), 30 deletions(-)

diff --git a/bun.lock b/bun.lock
index 5f79ad47..981032d6 100644
--- a/bun.lock
+++ b/bun.lock
@@ -4,23 +4,21 @@
   "workspaces": {
     "": {
       "name": "dasolve",
-      "dependencies": {
-        "@commander-js/extra-typings": "14.0.0",
-        "@modelcontextprotocol/sdk": "1.26.0",
-        "inquirer": "12.9.4",
-        "zod": "4.3.6",
-      },
       "devDependencies": {
+        "@commander-js/extra-typings": "14.0.0",
         "@commitlint/cli": "19.8.1",
         "@commitlint/config-conventional": "19.8.1",
         "@commitlint/cz-commitlint": "19.8.1",
+        "@modelcontextprotocol/sdk": "1.26.0",
         "@simple-release/npm": "2.3.0",
         "@types/bun": "1.3.9",
         "commitizen": "4.3.1",
         "conventional-changelog": "7.1.1",
         "conventional-changelog-angular": "8.0.0",
+        "inquirer": "12.9.4",
         "oxlint": "1.14.0",
         "prettier": "3.6.2",
+        "zod": "4.3.6",
       },
       "optionalDependencies": {
         "archgate-darwin-arm64": ">=0.1.1",
diff --git a/package.json b/package.json
index 6232becf..05b74584 100644
--- a/package.json
+++ b/package.json
@@ -50,23 +50,21 @@
     "changelog": "conventional-changelog -p angular -i CHANGELOG.md -s -r 0"
   },
   "type": "module",
-  "dependencies": {
-    "@commander-js/extra-typings": "14.0.0",
-    "@modelcontextprotocol/sdk": "1.26.0",
-    "inquirer": "12.9.4",
-    "zod": "4.3.6"
-  },
   "devDependencies": {
+    "@commander-js/extra-typings": "14.0.0",
     "@commitlint/cli": "19.8.1",
     "@commitlint/config-conventional": "19.8.1",
     "@commitlint/cz-commitlint": "19.8.1",
+    "@modelcontextprotocol/sdk": "1.26.0",
     "@simple-release/npm": "2.3.0",
     "@types/bun": "1.3.9",
     "commitizen": "4.3.1",
     "conventional-changelog": "7.1.1",
     "conventional-changelog-angular": "8.0.0",
+    "inquirer": "12.9.4",
     "oxlint": "1.14.0",
-    "prettier": "3.6.2"
+    "prettier": "3.6.2",
+    "zod": "4.3.6"
   },
   "peerDependencies": {
     "typescript": "^5"
diff --git a/src/engine/loader.ts b/src/engine/loader.ts
index ce9d8997..a01c9c20 100644
--- a/src/engine/loader.ts
+++ b/src/engine/loader.ts
@@ -2,7 +2,23 @@ import { readdirSync } from "node:fs";
 import { join, basename } from "node:path";
 import { parseAdr } from "../formats/adr";
 import type { AdrDocument } from "../formats/adr";
-import { RuleSetSchema, type RuleSet } from "../formats/rules";
+import { type RuleSet } from "../formats/rules";
+import { z } from "zod";
+import type { RuleContext } from "../formats/rules";
+
+const RuleSetSchema = z.object({
+  rules: z.record(
+    z.string(),
+    z.object({
+      description: z.string(),
+      severity: z.enum(["error", "warning", "info"]).optional(),
+      check: z.custom<(ctx: RuleContext) => Promise<void>>(
+        (val) => typeof val === "function",
+        "Expected a function"
+      ),
+    })
+  ),
+});
 import { projectPaths } from "../helpers/paths";
 import { logDebug, logWarn } from "../helpers/log";
 
diff --git a/src/formats/rules.ts b/src/formats/rules.ts
index 522b3f3e..52f7ccae 100644
--- a/src/formats/rules.ts
+++ b/src/formats/rules.ts
@@ -1,5 +1,3 @@
-import { z } from "zod";
-
 // --- Severity ---
 
 export type Severity = "error" | "warning" | "info";
@@ -63,20 +61,6 @@ export type RuleSet = {
   rules: Record<string, RuleConfig>;
 };
 
-export const RuleSetSchema = z.object({
-  rules: z.record(
-    z.string(),
-    z.object({
-      description: z.string(),
-      severity: z.enum(["error", "warning", "info"]).optional(),
-      check: z.custom<(ctx: RuleContext) => Promise<void>>(
-        (val) => typeof val === "function",
-        "Expected a function"
-      ),
-    })
-  ),
-});
-
 /**
  * Define rules in a .rules.ts companion file.
  * Keys become rule IDs, preventing duplicates.