@codedrifters/configulator 0.0.407 → 0.0.408

package/lib/index.mjs

@@ -5027,7 +5027,15 @@ var createRuleSkill = {
     "- Use `ruleExtensions` to add project-specific content to existing bundle rules instead of replacing them"
   ].join("\n")
 };
-function buildBaseBundle(paths = DEFAULT_AGENT_PATHS) {
+var DEFAULT_BASE_CONVENTIONS = {
+  progressFiles: resolveProgressFiles(),
+  sharedEditing: resolveSharedEditing(),
+  temporalFraming: resolveTemporalFraming(),
+  skillEvals: resolveSkillEvals(),
+  issueTemplates: resolveIssueTemplates(),
+  hasDownstreamIssueKindBundles: true
+};
+function buildBaseBundle(paths = DEFAULT_AGENT_PATHS, conventions = DEFAULT_BASE_CONVENTIONS) {
   return {
     name: "base",
     description: "Core rules: project overview, interaction style, and general coding conventions",
@@ -5696,7 +5704,7 @@ function buildBaseBundle(paths = DEFAULT_AGENT_PATHS) {
         // them via `agentConfig.additionalRulePaths`; other consumers
         // get a clean default.
         filePatterns: [".claude/agents/*.md", ".claude/procedures/**"],
-        content: renderProgressFilesRuleContent(resolveProgressFiles()),
+        content: renderProgressFilesRuleContent(conventions.progressFiles),
         platforms: {
           cursor: { exclude: true }
         },
@@ -5707,7 +5715,7 @@ function buildBaseBundle(paths = DEFAULT_AGENT_PATHS) {
         description: "Shared-editing safety: single-entry deterministic-sort inserts on index files, commit-path verification, and the merge-conflict resolution recipe that keeps concurrent agent sessions from dropping each other's rows on shared registries and feature matrices.",
         scope: AGENT_RULE_SCOPE.FILE_PATTERN,
         filePatterns: DEFAULT_SHARED_INDEX_PATHS,
-        content: renderSharedEditingRuleContent(resolveSharedEditing()),
+        content: renderSharedEditingRuleContent(conventions.sharedEditing),
         platforms: {
           cursor: { exclude: true }
         },
@@ -5725,7 +5733,7 @@ function buildBaseBundle(paths = DEFAULT_AGENT_PATHS) {
           "docs/src/content/docs/standards-research/**/*.md",
           "docs/src/content/docs/customer-research/**/*.md"
         ],
-        content: renderTemporalFramingRuleContent(resolveTemporalFraming()),
+        content: renderTemporalFramingRuleContent(conventions.temporalFraming),
         platforms: {
           cursor: { exclude: true }
         },
@@ -5738,7 +5746,7 @@ function buildBaseBundle(paths = DEFAULT_AGENT_PATHS) {
         // Bundle defaults exclude paths into configulator's own
         // source — see comment on progress-file-convention above.
         filePatterns: [".claude/skills/**"],
-        content: renderSkillEvalsRuleContent(resolveSkillEvals()),
+        content: renderSkillEvalsRuleContent(conventions.skillEvals),
         platforms: {
           cursor: { exclude: true }
         },
@@ -5755,7 +5763,10 @@ function buildBaseBundle(paths = DEFAULT_AGENT_PATHS) {
           ".claude/agents/*.md",
           `${paths.docsRoot}/agents/issue-templates.md`
         ],
-        content: renderIssueTemplatesRuleContent(resolveIssueTemplates()),
+        content: renderIssueTemplatesRuleContent(
+          conventions.issueTemplates,
+          conventions.hasDownstreamIssueKindBundles
+        ),
         platforms: {
           cursor: { exclude: true }
         },
@@ -19462,37 +19473,7 @@ var scanCommand = {
     ""
   ].join("\n")
 };
-var orchestratorBundle = {
-  name: "orchestrator",
-  description: "Pipeline orchestrator agent for issue triage, PR review, and queue management",
-  // Always included by default
-  appliesWhen: () => true,
-  rules: [
-    {
-      name: "orchestrator-conventions",
-      description: "Guidelines for orchestrator agent behavior and pipeline management, including the funnel-tier dispatch sort, scope gate, and per-agent scheduled-task layout",
-      scope: AGENT_RULE_SCOPE.FILE_PATTERN,
-      // Bundle defaults exclude paths into configulator's own source
-      // (only meaningful when configulator is a workspace package).
-      // codedrifters/packages restores them via
-      // `agentConfig.additionalRulePaths`.
-      filePatterns: [
-        ".claude/agents/orchestrator.md",
-        ".claude/scheduled-tasks/**"
-      ],
-      content: buildOrchestratorConventionsContent(
-        DEFAULT_AGENT_TIERS,
-        resolveScopeGate(),
-        resolveRunRatio(),
-        resolveScheduledTasks(),
-        resolveUnblockDependents()
-      ),
-      platforms: {
-        cursor: { exclude: true }
-      },
-      tags: ["workflow"]
-    }
-  ],
+var ORCHESTRATOR_BUNDLE_STATIC = {
   subAgents: [orchestratorSubAgent, issueWorkerSubAgent],
   procedures: [
     checkBlockedProcedure,
@@ -19508,6 +19489,51 @@ var orchestratorBundle = {
     ]
   }
 };
+var DEFAULT_ORCHESTRATOR_CONVENTIONS = {
+  tiers: DEFAULT_AGENT_TIERS,
+  scopeGate: resolveScopeGate(),
+  runRatio: resolveRunRatio(),
+  scheduledTasks: resolveScheduledTasks(),
+  unblockDependents: resolveUnblockDependents(),
+  excludeBundles: []
+};
+function buildOrchestratorBundle(conventions = DEFAULT_ORCHESTRATOR_CONVENTIONS) {
+  return {
+    name: "orchestrator",
+    description: "Pipeline orchestrator agent for issue triage, PR review, and queue management",
+    // Always included by default
+    appliesWhen: () => true,
+    rules: [
+      {
+        name: "orchestrator-conventions",
+        description: "Guidelines for orchestrator agent behavior and pipeline management, including the funnel-tier dispatch sort, scope gate, and per-agent scheduled-task layout",
+        scope: AGENT_RULE_SCOPE.FILE_PATTERN,
+        // Bundle defaults exclude paths into configulator's own source
+        // (only meaningful when configulator is a workspace package).
+        // codedrifters/packages restores them via
+        // `agentConfig.additionalRulePaths`.
+        filePatterns: [
+          ".claude/agents/orchestrator.md",
+          ".claude/scheduled-tasks/**"
+        ],
+        content: buildOrchestratorConventionsContent(
+          conventions.tiers,
+          conventions.scopeGate,
+          conventions.runRatio,
+          conventions.scheduledTasks,
+          conventions.unblockDependents,
+          conventions.excludeBundles
+        ),
+        platforms: {
+          cursor: { exclude: true }
+        },
+        tags: ["workflow"]
+      }
+    ],
+    ...ORCHESTRATOR_BUNDLE_STATIC
+  };
+}
+var orchestratorBundle = buildOrchestratorBundle();
 
 // src/agent/bundles/people-profile.ts
 function buildPeopleProfileAnalystSubAgent(paths, issueDefaults, tier) {
@@ -33981,10 +34007,14 @@ function renderPriorityRulesSection(rules) {
 }
 
 // src/agent/bundles/index.ts
-function buildBuiltInBundles(paths = DEFAULT_AGENT_PATHS, issueDefaults = DEFAULT_RESOLVED_ISSUE_DEFAULTS, defaultAgentTier = AGENT_MODEL.BALANCED, bundleAgentTiers = /* @__PURE__ */ new Map(), prReviewPolicy = resolvePrReviewPolicy(), buildPolicy = DEFAULT_BUILD_POLICY) {
+var DEFAULT_RULE_CONVENTIONS = {
+  base: DEFAULT_BASE_CONVENTIONS,
+  orchestrator: DEFAULT_ORCHESTRATOR_CONVENTIONS
+};
+function buildBuiltInBundles(paths = DEFAULT_AGENT_PATHS, issueDefaults = DEFAULT_RESOLVED_ISSUE_DEFAULTS, defaultAgentTier = AGENT_MODEL.BALANCED, bundleAgentTiers = /* @__PURE__ */ new Map(), prReviewPolicy = resolvePrReviewPolicy(), buildPolicy = DEFAULT_BUILD_POLICY, conventions = DEFAULT_RULE_CONVENTIONS) {
   const tierFor = (bundle) => bundleAgentTiers.get(bundle) ?? defaultAgentTier;
   return [
-    buildBaseBundle(paths),
+    buildBaseBundle(paths, conventions.base),
     upstreamConfigulatorDocsBundle,
     typescriptBundle,
     vitestBundle,
@@ -33997,7 +34027,7 @@ function buildBuiltInBundles(paths = DEFAULT_AGENT_PATHS, issueDefaults = DEFAUL
     slackBundle,
     buildMeetingAnalysisBundle(tierFor("meeting-analysis")),
     agendaBundle,
-    orchestratorBundle,
+    buildOrchestratorBundle(conventions.orchestrator),
     buildPrReviewBundle(prReviewPolicy),
     buildRequirementsAnalystBundle(paths, issueDefaults),
     buildRequirementsWriterBundle(paths, issueDefaults),
@@ -35085,6 +35115,13 @@ var AgentConfig = class _AgentConfig extends Component8 {
    * credential requirement when a remote cache actually exists. The
    * getter is lazy by design — `TurboRepo` must already be attached
    * to the project when the bundles are first read.
+   *
+   * Every **config-driven convention rule** is likewise resolved here
+   * and seeded into its owning bundle, so the rule enters the rule map
+   * already carrying the consumer's settings. Rewriting those rules
+   * after the map was assembled — the previous approach — silently
+   * discarded any `ruleExtensions` append or same-name `rules`
+   * override that had already been merged in.
    */
   get pathAwareBundles() {
     if (!this.cachedBundles) {
@@ -35094,11 +35131,52 @@ var AgentConfig = class _AgentConfig extends Component8 {
         resolveDefaultAgentTier(this.options),
         resolveBundleAgentTiers(this.options),
         resolvePrReviewPolicy(this.options.prReviewPolicy),
-        resolveBuildPolicy(this.project)
+        resolveBuildPolicy(this.project),
+        this.resolvedRuleConventions
       );
     }
     return this.cachedBundles;
   }
+  /**
+   * Resolved settings for every config-driven convention rule, derived
+   * from this project's options. Consumed by `buildBuiltInBundles` so
+   * the `base` and `orchestrator` bundles seed final rule content.
+   *
+   * `excludeBundles` feeds two of these: the orchestrator's rendered
+   * tier table / scope-gate overrides / scheduled-tasks registry drop
+   * rows owned by excluded bundles, and the issue-templates rule falls
+   * back to its disabled stub once every downstream-issue-kind bundle
+   * has been excluded.
+   */
+  get resolvedRuleConventions() {
+    const excludeBundles = this.options.excludeBundles ?? [];
+    const orchestratorAssets = resolveOrchestratorAssets(
+      this.options.tiers,
+      this.options.scopeGate,
+      this.options.runRatio,
+      this.options.scheduledTasks,
+      this.options.unblockDependents,
+      excludeBundles
+    );
+    return {
+      base: {
+        progressFiles: resolveProgressFiles(this.options.progressFiles),
+        sharedEditing: resolveSharedEditing(this.options.sharedEditing),
+        temporalFraming: resolveTemporalFraming(this.options.temporalFraming),
+        skillEvals: resolveSkillEvals(this.options.skillEvals),
+        issueTemplates: resolveIssueTemplates(this.options.issueTemplates),
+        hasDownstreamIssueKindBundles: hasAnyDownstreamIssueKindBundle(excludeBundles)
+      },
+      orchestrator: {
+        tiers: orchestratorAssets.tiers,
+        scopeGate: orchestratorAssets.scopeGate,
+        runRatio: orchestratorAssets.runRatio,
+        scheduledTasks: orchestratorAssets.scheduledTasks,
+        unblockDependents: orchestratorAssets.unblockDependents,
+        excludeBundles
+      }
+    };
+  }
   /**
    * Returns the bundles that are active for this project: auto-detected
    * bundles (when `autoDetectBundles !== false`) plus force-included
@@ -35413,43 +35491,10 @@ ${section}`
       }
     }
     const excludedBundleNames = this.options.excludeBundles ?? [];
-    if (this.options.tiers || this.options.scopeGate || this.options.runRatio || this.options.scheduledTasks || this.options.unblockDependents || excludedBundleNames.length > 0) {
-      const orchestratorRule = ruleMap.get("orchestrator-conventions");
-      if (orchestratorRule) {
-        const { conventionsContent } = resolveOrchestratorAssets(
-          this.options.tiers,
-          this.options.scopeGate,
-          this.options.runRatio,
-          this.options.scheduledTasks,
-          this.options.unblockDependents,
-          excludedBundleNames
-        );
-        if (conventionsContent !== orchestratorRule.content) {
-          ruleMap.set("orchestrator-conventions", {
-            ...orchestratorRule,
-            content: conventionsContent
-          });
-        }
-      }
-    }
     const injectBundleHooks = this.options.claudeMd?.injectBundleHooks ?? true;
     const resolvedProgressFiles = resolveProgressFiles(
       this.options.progressFiles
     );
-    if (this.options.progressFiles) {
-      const progressRule = ruleMap.get("progress-file-convention");
-      if (progressRule) {
-        const progressContent = renderProgressFilesRuleContent(
-          resolvedProgressFiles
-        );
-        if (progressContent !== progressRule.content) {
-          ruleMap.set("progress-file-convention", {
-            ...progressRule,
-            content: progressContent
-          });
-        }
-      }
-    }
     if (injectBundleHooks && resolvedProgressFiles.enabled) {
       for (const [ruleName, label] of PROGRESS_FILE_BUNDLE_HOOKS) {
         const existing = ruleMap.get(ruleName);
@@ -35476,20 +35521,6 @@ ${hook}`
     const resolvedSharedEditingForRules = resolveSharedEditing(
       this.options.sharedEditing
     );
-    if (this.options.sharedEditing) {
-      const sharedEditingRule = ruleMap.get("shared-editing-safety");
-      if (sharedEditingRule) {
-        const sharedEditingContent = renderSharedEditingRuleContent(
-          resolvedSharedEditingForRules
-        );
-        if (sharedEditingContent !== sharedEditingRule.content) {
-          ruleMap.set("shared-editing-safety", {
-            ...sharedEditingRule,
-            content: sharedEditingContent
-          });
-        }
-      }
-    }
     if (injectBundleHooks && resolvedSharedEditingForRules.enabled) {
       for (const [ruleName, label] of SHARED_EDITING_BUNDLE_HOOKS) {
         const existing = ruleMap.get(ruleName);
@@ -35516,20 +35547,6 @@ ${hook}`
     const resolvedSkillEvalsForRules = resolveSkillEvals(
       this.options.skillEvals
     );
-    if (this.options.skillEvals) {
-      const skillEvalsRule = ruleMap.get("skill-evals");
-      if (skillEvalsRule) {
-        const skillEvalsContent = renderSkillEvalsRuleContent(
-          resolvedSkillEvalsForRules
-        );
-        if (skillEvalsContent !== skillEvalsRule.content) {
-          ruleMap.set("skill-evals", {
-            ...skillEvalsRule,
-            content: skillEvalsContent
-          });
-        }
-      }
-    }
     if (injectBundleHooks && resolvedSkillEvalsForRules.enabled) {
       for (const [ruleName, label] of SKILL_EVALS_BUNDLE_HOOKS) {
         const existing = ruleMap.get(ruleName);
@@ -35557,21 +35574,6 @@ ${hook}`
       this.options.issueTemplates
     );
     const hasDownstreamBundles = hasAnyDownstreamIssueKindBundle(excludedBundleNames);
-    if (this.options.issueTemplates || !hasDownstreamBundles) {
-      const issueTemplatesRule = ruleMap.get("issue-templates-convention");
-      if (issueTemplatesRule) {
-        const issueTemplatesContent = renderIssueTemplatesRuleContent(
-          resolvedIssueTemplatesForRules,
-          hasDownstreamBundles
-        );
-        if (issueTemplatesContent !== issueTemplatesRule.content) {
-          ruleMap.set("issue-templates-convention", {
-            ...issueTemplatesRule,
-            content: issueTemplatesContent
-          });
-        }
-      }
-    }
     if (injectBundleHooks && resolvedIssueTemplatesForRules.enabled && hasDownstreamBundles) {
       for (const [ruleName, label] of ISSUE_TEMPLATES_BUNDLE_HOOKS) {
         const existing = ruleMap.get(ruleName);
@@ -35595,23 +35597,6 @@ ${hook}`
         });
       }
     }
-    const resolvedTemporalFramingForRules = resolveTemporalFraming(
-      this.options.temporalFraming
-    );
-    if (this.options.temporalFraming) {
-      const temporalFramingRule = ruleMap.get("temporal-framing-convention");
-      if (temporalFramingRule) {
-        const temporalFramingContent = renderTemporalFramingRuleContent(
-          resolvedTemporalFramingForRules
-        );
-        if (temporalFramingContent !== temporalFramingRule.content) {
-          ruleMap.set("temporal-framing-convention", {
-            ...temporalFramingRule,
-            content: temporalFramingContent
-          });
-        }
-      }
-    }
     const tierExamples = this.options.features?.sourceTierExamples;
     if (_AgentConfig.hasActiveTierExamples(tierExamples)) {
       const sourceRule = ruleMap.get("source-quality-verification");
@@ -41836,6 +41821,7 @@ export {
   DEFAULT_API_EXTRACTOR_REPORT_FILENAME,
   DEFAULT_API_EXTRACTOR_REPORT_FOLDER,
   DEFAULT_AUDIT_REPORT_DIR,
+  DEFAULT_BASE_CONVENTIONS,
   DEFAULT_BUILD_POLICY,
   DEFAULT_BUNDLE_OVERRIDES,
   DEFAULT_DECOMPOSITION_TEMPLATE,
@@ -41852,6 +41838,7 @@ export {
   DEFAULT_ISSUE_TEMPLATES_PATH,
   DEFAULT_ISSUE_TEMPLATES_REQUIRE_REFERENCE,
   DEFAULT_OFF_PEAK_CRON_EXAMPLE,
+  DEFAULT_ORCHESTRATOR_CONVENTIONS,
   DEFAULT_PARTIAL_UNBLOCK_COMMENT_TEMPLATE,
   DEFAULT_PATHS_EXEMPT_FROM_SIZE,
   DEFAULT_PRIORITY_LABELS,
@@ -41864,6 +41851,7 @@ export {
   DEFAULT_REQUIREMENT_CATEGORY_DIRS,
   DEFAULT_REQUIRE_PRODUCT_CONTEXT,
   DEFAULT_RESOLVED_ISSUE_DEFAULTS,
+  DEFAULT_RULE_CONVENTIONS,
   DEFAULT_SAMPLE_COMPILER_OPTIONS,
   DEFAULT_SCHEDULED_TASKS_ROOT,
   DEFAULT_SCHEDULED_TASK_ENTRIES,
@@ -41967,6 +41955,7 @@ export {
   buildIndustryDiscoveryBundle,
   buildMaintenanceAuditBundle,
   buildMeetingAnalysisBundle,
+  buildOrchestratorBundle,
   buildOrchestratorConventionsContent,
   buildPeopleProfileBundle,
   buildPrReviewBundle,