Refactor/docs rules presets

.changeset/config.json

@@ -2,7 +2,15 @@
   "$schema": "https://unpkg.com/@changesets/config@3.1.2/schema.json",
   "changelog": ["@changesets/changelog-github", { "repo": "joachimhodana/dbt-doctor" }],
   "commit": false,
-  "fixed": [["dbt-doctor", "dbt-doctor-rules", "@dbt-doctor/types", "@dbt-doctor/project-info"]],
+  "fixed": [
+    [
+      "dbt-doctor",
+      "dbt-doctor-rules",
+      "@dbt-doctor/types",
+      "@dbt-doctor/project-info",
+      "@dbt-doctor/manifest"
+    ]
+  ],
   "linked": [],
   "access": "public",
   "baseBranch": "main",

.changeset/fast-donuts-cheat.md

@@ -0,0 +1,13 @@
+---
+"dbt-doctor": patch
+"dbt-doctor-rules": patch
+"@dbt-doctor/types": patch
+"@dbt-doctor/project-info": patch
+"@dbt-doctor/manifest": minor
+---
+
+Add Phase 1 manifest graph support with explicit manifest path configuration.
+
+- Introduce `@dbt-doctor/manifest` package for reading and normalizing graph artifacts.
+- Add manifest-backed rules (`staging-depends-on-staging`, `unused-sources`) and skip manifest-only rules when artifact is missing.
+- Add `manifestPath` config support and CLI `--manifest <path>` override for dbt Core/Fusion compatibility.

.gitignore

@@ -22,3 +22,8 @@ review-*.md
 /scripts/print-batch-input.mjs
 /scripts/rule-prompts/
 /rules.json
+packages/website/docs/rules/.rule-ids.json
+
+content/
+
+.pnpm-store/

package.json

@@ -23,8 +23,8 @@
     "changeset": "changeset",
     "version": "changeset version",
     "release": "pnpm build && changeset publish",
-    "leaderboard:update": "node --experimental-strip-types --no-warnings scripts/update-leaderboard.ts",
-    "leaderboard:sync": "node --experimental-strip-types --no-warnings scripts/sync-leaderboard-public.ts"
+    "generate:rules-docs": "node scripts/generate-rules-docs.mjs && node scripts/link-tool-parity-rules.mjs && vp fmt packages/website/docs/rules",
+    "update:tool-parity-coverage": "node scripts/update-tool-parity-coverage.mjs"
   },
   "devDependencies": {
     "@changesets/changelog-github": "^0.7.0",

packages/core/src/apply-config-preset.ts

@@ -1,30 +1,73 @@
 import type { DbtDoctorConfig, DbtDoctorPreset } from "@dbt-doctor/types";
 
-const PRESET_OVERRIDES: Record<Exclude<DbtDoctorPreset, "default">, Partial<DbtDoctorConfig>> = {
+/**
+ * Tag bundles skipped per preset via `ignore.tags`.
+ * Rules with any ignored tag are not registered for the scan.
+ */
+const IGNORE_TAGS = {
+  style: ["style", "sql-style"],
+  strict: ["strict"],
+  enterprise: ["enterprise"],
+} as const;
+
+const PRESET_OVERRIDES: Record<DbtDoctorPreset, Partial<DbtDoctorConfig>> = {
+  /**
+   * Recommended starter: core architecture, naming, SQL quality, and manifest rules
+   * without documentation-contract (`strict`), governance (`enterprise`), or
+   * formatting (`style` / `sql-style`) tiers.
+   */
+  default: {
+    ignore: {
+      tags: [...IGNORE_TAGS.enterprise, ...IGNORE_TAGS.strict, ...IGNORE_TAGS.style],
+    },
+  },
+  /**
+   * CI documentation, native SQL style, and contract discipline on top of `default`,
+   * without enterprise governance rules.
+   */
   strict: {
     failOn: "error",
-    ignore: { tags: [] },
+    ignore: { tags: [...IGNORE_TAGS.enterprise] },
+    categories: {
+      Documentation: "error",
+      Configuration: "error",
+      Architecture: "error",
+      "SQL Style": "warn",
+      Testing: "warn",
+    },
   },
+  /**
+   * Full rule catalog (all registered rules) with stricter governance and graph checks.
+   */
   enterprise: {
     scoreMode: "files",
     failOn: "warning",
-    ignore: { tags: ["style"] },
     categories: {
       Governance: "error",
       Architecture: "error",
+      Sources: "error",
+      Documentation: "warn",
+      Performance: "warn",
+      Testing: "warn",
     },
   },
 };
 
-export const applyConfigPreset = (config: DbtDoctorConfig | null): DbtDoctorConfig | null => {
-  if (!config?.preset || config.preset === "default") return config;
-  const presetDefaults = PRESET_OVERRIDES[config.preset];
+const mergePresetConfig = (preset: DbtDoctorPreset, config: DbtDoctorConfig): DbtDoctorConfig => {
+  const presetDefaults = PRESET_OVERRIDES[preset];
   return {
     ...presetDefaults,
     ...config,
+    preset,
     ignore: { ...presetDefaults.ignore, ...config.ignore },
     categories: { ...presetDefaults.categories, ...config.categories },
     surfaces: { ...presetDefaults.surfaces, ...config.surfaces },
     rules: { ...presetDefaults.rules, ...config.rules },
   };
 };
+
+export const applyConfigPreset = (config: DbtDoctorConfig | null): DbtDoctorConfig | null => {
+  if (!config) return null;
+  const preset: DbtDoctorPreset = config.preset ?? "default";
+  return mergePresetConfig(preset, config);
+};

packages/core/src/build-json-report.ts

@@ -49,6 +49,8 @@ export const buildJsonReport = (input: BuildJsonReportInput): JsonReport => {
     project: result.project,
     diagnostics: result.diagnostics,
     score: result.score,
+    ...(result.coverage ? { coverage: result.coverage } : {}),
+    ...(result.perModelScores ? { perModelScores: result.perModelScores } : {}),
     skippedChecks: result.skippedChecks,
     ...(result.skippedCheckReasons ? { skippedCheckReasons: result.skippedCheckReasons } : {}),
     elapsedMilliseconds: result.elapsedMilliseconds,

packages/core/src/index.ts

@@ -1,6 +1,5 @@
 export * from "./apply-config-preset.js";
 export * from "./apply-ignore-overrides.js";
-export * from "./baseline-diagnostics.js";
 export * from "./build-sarif-report.js";
 export * from "./apply-severity-controls.js";
 export * from "./build-rule-severity-controls.js";

packages/core/src/merge-config-with-cli-flags.ts

@@ -9,13 +9,20 @@ export interface ConfigCliOverrides {
   preset?: string;
   scoreMode?: string;
   failOn?: string;
+  manifest?: string;
 }
 
 export const mergeConfigWithCliFlags = (
   config: DbtDoctorConfig | null,
   overrides: ConfigCliOverrides,
 ): DbtDoctorConfig | null => {
-  if (!overrides.preset && !overrides.scoreMode && !overrides.failOn && !config) {
+  if (
+    !overrides.preset &&
+    !overrides.scoreMode &&
+    !overrides.failOn &&
+    !overrides.manifest &&
+    !config
+  ) {
     return config;
   }
 
@@ -51,5 +58,9 @@ export const mergeConfigWithCliFlags = (
     }
   }
 
+  if (overrides.manifest) {
+    merged.manifestPath = overrides.manifest;
+  }
+
   return merged;
 };