@codedrifters/configulator 0.0.212 → 0.0.214

package/docs/label-taxonomy.md

@@ -20,9 +20,10 @@ regardless of which specific agent bundles a project enables.
 
 Families in this tier:
 
-- **`status:*`** — workflow state (`ready`, `in-progress`, `blocked`,
-  `done`, `deferred`, `needs-attention`). Required by every agent that
-  transitions issues through the workflow.
+- **`status:*`** — workflow state (`ready`, `in-progress`,
+  `ready-for-review`, `blocked`, `done`, `deferred`,
+  `needs-attention`). Required by every agent that transitions issues
+  through the workflow.
 - **`priority:*`** — scheduling priority (`critical`, `high`, `medium`,
   `low`, `trivial`). Required by the orchestrator to sequence work.
 - **`type:*`** — conventional-commit work type (`feat`, `fix`, `docs`,

package/lib/index.d.mts

@@ -1146,8 +1146,15 @@ declare const requirementsAnalystBundle: AgentRuleBundle;
 declare const researchPipelineBundle: AgentRuleBundle;
 
 /**
- * Slack bundle — opt-in via `includeBundles: ["slack"]`.
- * Never auto-detected (no Slack dependency to introspect).
+ * Slack bundle — auto-activated when the consuming project declares a
+ * Slack integration via `ProjectMetadata.slack`. Consumers can still
+ * opt in manually via `includeBundles: ["slack"]` when they have no
+ * `ProjectMetadata` configured.
+ *
+ * Detection signal: any truthy value on
+ * `ProjectMetadata.of(project).metadata.slack` that carries at least one
+ * channel reference (`projectChannel`, `alertsChannel`, or a non-empty
+ * `channels` record).
  */
 declare const slackBundle: AgentRuleBundle;
 

package/lib/index.d.ts

@@ -1195,8 +1195,15 @@ declare const requirementsAnalystBundle: AgentRuleBundle;
 declare const researchPipelineBundle: AgentRuleBundle;
 
 /**
- * Slack bundle — opt-in via `includeBundles: ["slack"]`.
- * Never auto-detected (no Slack dependency to introspect).
+ * Slack bundle — auto-activated when the consuming project declares a
+ * Slack integration via `ProjectMetadata.slack`. Consumers can still
+ * opt in manually via `includeBundles: ["slack"]` when they have no
+ * `ProjectMetadata` configured.
+ *
+ * Detection signal: any truthy value on
+ * `ProjectMetadata.of(project).metadata.slack` that carries at least one
+ * channel reference (`projectChannel`, `alertsChannel`, or a non-empty
+ * `channels` record).
  */
 declare const slackBundle: AgentRuleBundle;
 

package/lib/index.js

@@ -792,9 +792,10 @@ var baseBundle = {
         "|-------|----------|",
         "| `status:ready` | The issue is fully specified, has no open blockers, and is available for a worker to pick up. |",
         "| `status:blocked` | The issue cannot be started yet \u2014 either it declares `Depends on: #N` on an open issue, or it is an epic with one or more open children. |",
-        "| `status:in-progress` | A worker has claimed the issue and a branch exists. Set when the worker starts; cleared only on completion or failure. |",
+        "| `status:in-progress` | A worker has claimed the issue and a branch exists. Set when the worker starts; cleared only when the worker opens a PR or the issue fails. |",
+        "| `status:ready-for-review` | A PR has been opened for this issue and is awaiting review and merge. Replaces `status:in-progress` at the moment the PR opens. |",
         "| `status:needs-attention` | Automated triage has flagged the issue for human review (stale, failed worker, or ambiguous state). Humans reset this label manually. |",
-        "| `status:done` | The change has shipped. Usually set when the PR merges and the issue closes. |",
+        "| `status:done` | The change has shipped. Set when the PR merges and the issue closes. |",
         "",
         "### Blocking Rules",
         "",
@@ -816,9 +817,9 @@ var baseBundle = {
         "  either blocking rule applies.",
         "- **Worker claims the issue** \u2192 remove `status:ready`, add",
         "  `status:in-progress`. A branch must exist before this transition.",
-        "- **Worker opens a PR** \u2192 leave `status:in-progress` in place until the",
-        "  PR merges. Do not pre-flip to `status:done`.",
-        "- **PR merges / issue closes** \u2192 remove `status:in-progress`, add",
+        "- **Worker opens a PR** \u2192 remove `status:in-progress`, add",
+        "  `status:ready-for-review`. The PR URL should be posted on the issue.",
+        "- **PR merges / issue closes** \u2192 remove `status:ready-for-review`, add",
         "  `status:done`.",
         "- **Dependency resolves** \u2192 if the issue was `status:blocked` solely because",
         "  of the dependency-blocking rule, remove `status:blocked` and add",
@@ -835,8 +836,9 @@ var baseBundle = {
         "  by CI. Those are expected self-healing behaviors, not failures.",
         "",
         "An issue must always carry exactly one of `status:ready`, `status:blocked`,",
-        "`status:in-progress`, or `status:done`. The `status:needs-attention` label",
-        "is additive \u2014 it coexists with whichever of those four applies."
+        "`status:in-progress`, `status:ready-for-review`, or `status:done`. The",
+        "`status:needs-attention` label is additive \u2014 it coexists with whichever of",
+        "those five applies."
       ].join("\n"),
       tags: ["workflow"]
     },
@@ -4358,11 +4360,14 @@ var issueWorkerSubAgent = {
     "",
     "## Phase 8: Update Status",
     "",
-    "After the PR is created:",
+    "After the PR is created, transition the issue to the review phase:",
     "```bash",
-    'gh issue edit <number> --remove-label "status:in-progress" --add-label "status:done"',
+    'gh issue edit <number> --remove-label "status:in-progress" --add-label "status:ready-for-review"',
     "```",
     "",
+    "Do **not** set `status:done` here \u2014 the `pr-reviewer` sub-agent is",
+    "responsible for that transition once the PR successfully merges.",
+    "",
     "## Phase 9: Branch Cleanup",
     "",
     "The PR will not auto-merge until the `pr-reviewer` enables it. Poll the PR",
@@ -5251,6 +5256,14 @@ var prReviewerSubAgent = {
     "  git fetch --prune origin",
     "  git branch -d <branch-name> 2>/dev/null || git branch -D <branch-name> 2>/dev/null || true",
     "  ```",
+    "  Transition the linked issue to `status:done` (replaces whichever of",
+    "  `status:ready-for-review` or `status:in-progress` it was carrying):",
+    "  ```bash",
+    "  gh issue edit <issue-number> \\",
+    '    --remove-label "status:ready-for-review" \\',
+    '    --remove-label "status:in-progress" \\',
+    '    --add-label "status:done"',
+    "  ```",
     "  Then check the linked issue state:",
     "  ```bash",
     "  gh issue view <issue-number> --json state --jq '.state'",
@@ -6714,11 +6727,106 @@ var researchPipelineBundle = {
   ]
 };
 
+// src/projects/project-metadata.ts
+var import_projen2 = require("projen");
+var import_javascript = require("projen/lib/javascript");
+var GITHUB_HTTPS_RE = /(?:https?:\/\/|git\+https:\/\/)github\.com\/([^/]+)\/([^/.]+)(?:\.git)?/;
+var GITHUB_SSH_RE = /git@github\.com:([^/]+)\/([^/.]+)(?:\.git)?/;
+function parseGitHubUrl(url) {
+  const httpsMatch = url.match(GITHUB_HTTPS_RE);
+  if (httpsMatch) {
+    return { owner: httpsMatch[1], name: httpsMatch[2] };
+  }
+  const sshMatch = url.match(GITHUB_SSH_RE);
+  if (sshMatch) {
+    return { owner: sshMatch[1], name: sshMatch[2] };
+  }
+  return { owner: void 0, name: void 0 };
+}
+var ProjectMetadata = class _ProjectMetadata extends import_projen2.Component {
+  /**
+   * Returns the ProjectMetadata instance for a project. Walks up the parent
+   * chain so sub-projects resolve the metadata declared on a root
+   * `MonorepoProject` without needing a duplicate declaration of their own.
+   * Returns `undefined` if no ancestor has a `ProjectMetadata` component.
+   */
+  static of(project) {
+    const isProjectMetadata = (c) => c instanceof _ProjectMetadata;
+    let current = project;
+    while (current) {
+      const found = current.components.find(isProjectMetadata);
+      if (found) {
+        return found;
+      }
+      current = current.parent;
+    }
+    return void 0;
+  }
+  constructor(project, options = {}) {
+    super(project);
+    this.metadata = this.resolveMetadata(options);
+  }
+  /**
+   * Merges explicit options with auto-detected values.
+   * Auto-detection reads the repository URL from package.json
+   * (via Projen's NodePackage manifest) and parses GitHub owner/name.
+   * Explicit options always take precedence over auto-detected values.
+   */
+  resolveMetadata(options) {
+    const autoDetected = this.autoDetectRepository();
+    return {
+      repository: {
+        owner: options.repository?.owner ?? autoDetected.owner,
+        name: options.repository?.name ?? autoDetected.name,
+        defaultBranch: options.repository?.defaultBranch ?? "main"
+      },
+      githubProject: options.githubProject,
+      organization: options.organization,
+      slack: options.slack,
+      labels: options.labels,
+      milestones: options.milestones,
+      docsPath: options.docsPath,
+      deployment: options.deployment
+    };
+  }
+  /**
+   * Attempts to auto-detect repository owner and name from the Projen
+   * project's package.json repository field.
+   */
+  autoDetectRepository() {
+    if (!(this.project instanceof import_javascript.NodeProject)) {
+      return { owner: void 0, name: void 0 };
+    }
+    const manifest = this.project.package.manifest;
+    const repoField = manifest.repository;
+    if (!repoField) {
+      return { owner: void 0, name: void 0 };
+    }
+    const url = typeof repoField === "string" ? repoField : repoField.url ?? "";
+    return parseGitHubUrl(url);
+  }
+};
+
 // src/agent/bundles/slack.ts
+function hasSlackConfig(slack) {
+  if (!slack) {
+    return false;
+  }
+  if (slack.projectChannel || slack.alertsChannel) {
+    return true;
+  }
+  if (slack.channels && Object.keys(slack.channels).length > 0) {
+    return true;
+  }
+  return false;
+}
 var slackBundle = {
   name: "slack",
-  description: "Slack MCP message formatting and best practices",
-  appliesWhen: () => false,
+  description: "Slack MCP message formatting and best practices. Auto-activates when ProjectMetadata declares a Slack integration; can still be force-included via `includeBundles: ['slack']`.",
+  appliesWhen: (project) => {
+    const pm = ProjectMetadata.of(project);
+    return hasSlackConfig(pm?.metadata.slack);
+  },
   rules: [
     {
       name: "slack-message-formatting",
@@ -6747,7 +6855,11 @@ var slackBundle = {
         "   \u2022 Feature B: <https://github.com/org/repo/pull/2|repo#2>",
         "   ```",
         "",
-        "5. **Avoid:** Plain URLs immediately after a bullet/label on the same line (e.g., `\u2022 Feature A: https://github.com/...`) when sending via API, as it can render with bullets inside or between links"
+        "5. **Avoid:** Plain URLs immediately after a bullet/label on the same line (e.g., `\u2022 Feature A: https://github.com/...`) when sending via API, as it can render with bullets inside or between links",
+        "",
+        "## Activation",
+        "",
+        "This bundle auto-activates when the consuming project declares a Slack integration through `ProjectMetadata` (any of `slack.projectChannel`, `slack.alertsChannel`, or a non-empty `slack.channels` record). If no `ProjectMetadata` is configured, opt in manually via `AgentConfigOptions.includeBundles: ['slack']`."
       ].join("\n"),
       tags: ["workflow"]
     }
@@ -7744,8 +7856,8 @@ var typescriptBundle = {
 };
 
 // src/vitest/vitest-component.ts
-var import_projen2 = require("projen");
-var import_javascript = require("projen/lib/javascript");
+var import_projen3 = require("projen");
+var import_javascript2 = require("projen/lib/javascript");
 var import_textfile = require("projen/lib/textfile");
 
 // src/versions.ts
@@ -7823,7 +7935,7 @@ var VERSION = {
 };
 
 // src/vitest/vitest-component.ts
-var Vitest = class _Vitest extends import_projen2.Component {
+var Vitest = class _Vitest extends import_projen3.Component {
   constructor(project, options = {}) {
     super(project);
     this.project = project;
@@ -7862,7 +7974,7 @@ var Vitest = class _Vitest extends import_projen2.Component {
   preSynthesize() {
     super.preSynthesize();
     for (const component of this.project.components) {
-      if (component instanceof import_javascript.Jest) {
+      if (component instanceof import_javascript2.Jest) {
         throw new Error("Vitest cannot be used together with Jest");
       }
     }
@@ -8023,86 +8135,6 @@ function prefix(rel, entry) {
   return path.posix.join(rel, entry);
 }
 
-// src/projects/project-metadata.ts
-var import_projen5 = require("projen");
-var import_javascript2 = require("projen/lib/javascript");
-var GITHUB_HTTPS_RE = /(?:https?:\/\/|git\+https:\/\/)github\.com\/([^/]+)\/([^/.]+)(?:\.git)?/;
-var GITHUB_SSH_RE = /git@github\.com:([^/]+)\/([^/.]+)(?:\.git)?/;
-function parseGitHubUrl(url) {
-  const httpsMatch = url.match(GITHUB_HTTPS_RE);
-  if (httpsMatch) {
-    return { owner: httpsMatch[1], name: httpsMatch[2] };
-  }
-  const sshMatch = url.match(GITHUB_SSH_RE);
-  if (sshMatch) {
-    return { owner: sshMatch[1], name: sshMatch[2] };
-  }
-  return { owner: void 0, name: void 0 };
-}
-var ProjectMetadata = class _ProjectMetadata extends import_projen5.Component {
-  /**
-   * Returns the ProjectMetadata instance for a project. Walks up the parent
-   * chain so sub-projects resolve the metadata declared on a root
-   * `MonorepoProject` without needing a duplicate declaration of their own.
-   * Returns `undefined` if no ancestor has a `ProjectMetadata` component.
-   */
-  static of(project) {
-    const isProjectMetadata = (c) => c instanceof _ProjectMetadata;
-    let current = project;
-    while (current) {
-      const found = current.components.find(isProjectMetadata);
-      if (found) {
-        return found;
-      }
-      current = current.parent;
-    }
-    return void 0;
-  }
-  constructor(project, options = {}) {
-    super(project);
-    this.metadata = this.resolveMetadata(options);
-  }
-  /**
-   * Merges explicit options with auto-detected values.
-   * Auto-detection reads the repository URL from package.json
-   * (via Projen's NodePackage manifest) and parses GitHub owner/name.
-   * Explicit options always take precedence over auto-detected values.
-   */
-  resolveMetadata(options) {
-    const autoDetected = this.autoDetectRepository();
-    return {
-      repository: {
-        owner: options.repository?.owner ?? autoDetected.owner,
-        name: options.repository?.name ?? autoDetected.name,
-        defaultBranch: options.repository?.defaultBranch ?? "main"
-      },
-      githubProject: options.githubProject,
-      organization: options.organization,
-      slack: options.slack,
-      labels: options.labels,
-      milestones: options.milestones,
-      docsPath: options.docsPath,
-      deployment: options.deployment
-    };
-  }
-  /**
-   * Attempts to auto-detect repository owner and name from the Projen
-   * project's package.json repository field.
-   */
-  autoDetectRepository() {
-    if (!(this.project instanceof import_javascript2.NodeProject)) {
-      return { owner: void 0, name: void 0 };
-    }
-    const manifest = this.project.package.manifest;
-    const repoField = manifest.repository;
-    if (!repoField) {
-      return { owner: void 0, name: void 0 };
-    }
-    const url = typeof repoField === "string" ? repoField : repoField.url ?? "";
-    return parseGitHubUrl(url);
-  }
-};
-
 // src/agent/renderers/claude-renderer.ts
 var import_projen6 = require("projen");
 var import_textfile2 = require("projen/lib/textfile");
@@ -9889,6 +9921,11 @@ var DEFAULT_STATUS_LABELS = [
     color: "FBCA04",
     description: "Actively being worked on"
   },
+  {
+    name: "status:ready-for-review",
+    color: "1D76DB",
+    description: "PR opened and awaiting review"
+  },
   {
     name: "status:blocked",
     color: "D93F0B",