@codedrifters/configulator 0.0.212 → 0.0.214

package/lib/index.mjs

@@ -717,9 +717,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",
         "",
@@ -741,9 +742,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",
@@ -760,8 +761,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"]
     },
@@ -4283,11 +4285,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",
@@ -5176,6 +5181,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'",
@@ -6639,11 +6652,106 @@ var researchPipelineBundle = {
   ]
 };
 
+// src/projects/project-metadata.ts
+import { Component as Component2 } from "projen";
+import { NodeProject } from "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 Component2 {
+  /**
+   * 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 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",
@@ -6672,7 +6780,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"]
     }
@@ -7297,12 +7409,12 @@ var softwareProfileBundle = {
 };
 
 // src/turbo/turbo-repo.ts
-import { Component as Component3, FileBase, JsonFile } from "projen/lib";
+import { Component as Component4, FileBase, JsonFile } from "projen/lib";
 import { JobPermission } from "projen/lib/github/workflows-model";
 
 // src/turbo/turbo-repo-task.ts
-import { Component as Component2 } from "projen/lib";
-var TurboRepoTask = class extends Component2 {
+import { Component as Component3 } from "projen/lib";
+var TurboRepoTask = class extends Component3 {
   constructor(project, options) {
     super(project);
     this.project = project;
@@ -7343,7 +7455,7 @@ var TurboRepoTask = class extends Component2 {
 // src/turbo/turbo-repo.ts
 var ROOT_TURBO_TASK_NAME = "turbo:build";
 var ROOT_CI_TASK_NAME = "build:all";
-var _TurboRepo = class _TurboRepo extends Component3 {
+var _TurboRepo = class _TurboRepo extends Component4 {
   constructor(project, options = {}) {
     super(project);
     this.project = project;
@@ -7669,7 +7781,7 @@ var typescriptBundle = {
 };
 
 // src/vitest/vitest-component.ts
-import { Component as Component4 } from "projen";
+import { Component as Component5 } from "projen";
 import { Jest } from "projen/lib/javascript";
 import { TextFile } from "projen/lib/textfile";
 
@@ -7748,7 +7860,7 @@ var VERSION = {
 };
 
 // src/vitest/vitest-component.ts
-var Vitest = class _Vitest extends Component4 {
+var Vitest = class _Vitest extends Component5 {
   constructor(project, options = {}) {
     super(project);
     this.project = project;
@@ -7948,86 +8060,6 @@ function prefix(rel, entry) {
   return path.posix.join(rel, entry);
 }
 
-// src/projects/project-metadata.ts
-import { Component as Component5 } from "projen";
-import { NodeProject as NodeProject2 } from "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 Component5 {
-  /**
-   * 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 NodeProject2)) {
-      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
 import { JsonFile as JsonFile2 } from "projen";
 import { TextFile as TextFile2 } from "projen/lib/textfile";
@@ -9823,6 +9855,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",