chief-clancy 0.5.4 → 0.5.5

package/dist/scripts/shared/pull-request/gitlab/gitlab.js

@@ -1,4 +1,15 @@
+/**
+ * GitLab merge request creation and review state checking.
+ *
+ * Uses the GitLab REST API v4 to create merge requests, check review state,
+ * and fetch review comments.
+ * Supports both gitlab.com and self-hosted instances.
+ *
+ * Auth: `PRIVATE-TOKEN` header (personal access token with `api` scope).
+ */
+import { gitlabDiscussionsSchema, gitlabMrListSchema, } from '../../../../schemas/gitlab-mr.js';
 import { postPullRequest } from '../post-pr/post-pr.js';
+import { extractReworkContent, isReworkComment, } from '../rework-comment/rework-comment.js';
 /**
  * Create a merge request on GitLab.
  *
@@ -24,4 +35,120 @@ export async function createMergeRequest(token, apiBase, projectPath, sourceBran
         return { url: data.web_url ?? '', number: data.iid ?? 0 };
     }, (status, text) => status === 409 && text.includes('already exists'));
 }
+/**
+ * Check the review state of an open MR for a given branch.
+ *
+ * Finds the open MR matching the source branch, fetches all discussions.
+ * Inline notes (type `DiffNote`) always trigger rework. General conversation
+ * notes only trigger rework when prefixed with `Rework:`.
+ *
+ * @param token - The GitLab personal access token.
+ * @param apiBase - The API base URL (e.g. `https://gitlab.com/api/v4`).
+ * @param projectPath - The raw project path (e.g. `group/subgroup/project`). URL-encoded internally.
+ * @param branch - The source branch to look up.
+ * @param since - ISO 8601 timestamp; only notes created after this time trigger rework.
+ * @returns The review state if an open MR exists, otherwise `undefined`.
+ */
+export async function checkMrReviewState(token, apiBase, projectPath, branch, since) {
+    try {
+        const encodedPath = encodeURIComponent(projectPath);
+        const mrUrl = `${apiBase}/projects/${encodedPath}/merge_requests?source_branch=${branch}&state=opened`;
+        const mrRes = await fetch(mrUrl, {
+            headers: { 'PRIVATE-TOKEN': token },
+        });
+        if (!mrRes.ok)
+            return undefined;
+        const data = gitlabMrListSchema.parse(await mrRes.json());
+        if (data.length === 0)
+            return undefined;
+        const mr = data[0];
+        const discussionsUrl = `${apiBase}/projects/${encodedPath}/merge_requests/${mr.iid}/discussions?per_page=100`;
+        const discussionsRes = await fetch(discussionsUrl, {
+            headers: { 'PRIVATE-TOKEN': token },
+        });
+        if (!discussionsRes.ok)
+            return undefined;
+        const discussions = gitlabDiscussionsSchema.parse(await discussionsRes.json());
+        let hasRework = false;
+        for (const discussion of discussions) {
+            for (const note of discussion.notes) {
+                if (note.system)
+                    continue;
+                if (since && note.created_at && note.created_at <= since)
+                    continue;
+                if (note.type === 'DiffNote' &&
+                    note.resolvable !== false &&
+                    note.resolved !== true) {
+                    hasRework = true;
+                    break;
+                }
+                if (isReworkComment(note.body)) {
+                    hasRework = true;
+                    break;
+                }
+            }
+            if (hasRework)
+                break;
+        }
+        return {
+            changesRequested: hasRework,
+            prNumber: mr.iid,
+            prUrl: mr.web_url,
+        };
+    }
+    catch {
+        return undefined;
+    }
+}
+/**
+ * Fetch feedback comments from an MR's discussions.
+ *
+ * Inline notes (type `DiffNote`) are always included — they inherently
+ * represent change requests. General conversation notes are only included
+ * when they start with `Rework:` (case-insensitive), with the prefix
+ * stripped. Inline notes are prefixed with `[new_path]` when available.
+ *
+ * @param token - The GitLab personal access token.
+ * @param apiBase - The API base URL (e.g. `https://gitlab.com/api/v4`).
+ * @param projectPath - The raw project path. URL-encoded internally.
+ * @param mrIid - The MR internal ID (iid).
+ * @param since - ISO 8601 timestamp; only notes created after this time are returned.
+ * @returns An array of feedback descriptions.
+ */
+export async function fetchMrReviewComments(token, apiBase, projectPath, mrIid, since) {
+    try {
+        const encodedPath = encodeURIComponent(projectPath);
+        const url = `${apiBase}/projects/${encodedPath}/merge_requests/${mrIid}/discussions?per_page=100`;
+        const res = await fetch(url, {
+            headers: { 'PRIVATE-TOKEN': token },
+        });
+        if (!res.ok)
+            return [];
+        const discussions = gitlabDiscussionsSchema.parse(await res.json());
+        const comments = [];
+        for (const discussion of discussions) {
+            for (const note of discussion.notes) {
+                if (note.system)
+                    continue;
+                if (since && note.created_at && note.created_at <= since)
+                    continue;
+                if (note.type === 'DiffNote' &&
+                    note.resolvable !== false &&
+                    note.resolved !== true) {
+                    const prefix = note.position?.new_path
+                        ? `[${note.position.new_path}] `
+                        : '';
+                    comments.push(`${prefix}${note.body}`);
+                }
+                else if (isReworkComment(note.body)) {
+                    comments.push(extractReworkContent(note.body));
+                }
+            }
+        }
+        return comments;
+    }
+    catch {
+        return [];
+    }
+}
 //# sourceMappingURL=gitlab.js.map

package/dist/scripts/shared/pull-request/gitlab/gitlab.js.map

@@ -1 +1 @@
-{"version":3,"file":"gitlab.js","sourceRoot":"","sources":["../../../../../src/scripts/shared/pull-request/gitlab/gitlab.ts"],"names":[],"mappings":"AAUA,OAAO,EAAE,eAAe,EAAE,MAAM,uBAAuB,CAAC;AAExD;;;;;;;;;;;GAWG;AACH,MAAM,CAAC,KAAK,UAAU,kBAAkB,CACtC,KAAa,EACb,OAAe,EACf,WAAmB,EACnB,YAAoB,EACpB,YAAoB,EACpB,KAAa,EACb,WAAmB;IAEnB,MAAM,WAAW,GAAG,kBAAkB,CAAC,WAAW,CAAC,CAAC;IAEpD,OAAO,eAAe,CACpB,GAAG,OAAO,aAAa,WAAW,iBAAiB,EACnD,EAAE,eAAe,EAAE,KAAK,EAAE,EAC1B;QACE,aAAa,EAAE,YAAY;QAC3B,aAAa,EAAE,YAAY;QAC3B,KAAK;QACL,WAAW;QACX,oBAAoB,EAAE,IAAI;KAC3B,EACD,CAAC,IAAI,EAAE,EAAE;QACP,MAAM,IAAI,GAAG,IAA0C,CAAC;QACxD,OAAO,EAAE,GAAG,EAAE,IAAI,CAAC,OAAO,IAAI,EAAE,EAAE,MAAM,EAAE,IAAI,CAAC,GAAG,IAAI,CAAC,EAAE,CAAC;IAC5D,CAAC,EACD,CAAC,MAAM,EAAE,IAAI,EAAE,EAAE,CAAC,MAAM,KAAK,GAAG,IAAI,IAAI,CAAC,QAAQ,CAAC,gBAAgB,CAAC,CACpE,CAAC;AACJ,CAAC"}
+{"version":3,"file":"gitlab.js","sourceRoot":"","sources":["../../../../../src/scripts/shared/pull-request/gitlab/gitlab.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AACH,OAAO,EACL,uBAAuB,EACvB,kBAAkB,GACnB,MAAM,wBAAwB,CAAC;AAGhC,OAAO,EAAE,eAAe,EAAE,MAAM,uBAAuB,CAAC;AACxD,OAAO,EACL,oBAAoB,EACpB,eAAe,GAChB,MAAM,qCAAqC,CAAC;AAE7C;;;;;;;;;;;GAWG;AACH,MAAM,CAAC,KAAK,UAAU,kBAAkB,CACtC,KAAa,EACb,OAAe,EACf,WAAmB,EACnB,YAAoB,EACpB,YAAoB,EACpB,KAAa,EACb,WAAmB;IAEnB,MAAM,WAAW,GAAG,kBAAkB,CAAC,WAAW,CAAC,CAAC;IAEpD,OAAO,eAAe,CACpB,GAAG,OAAO,aAAa,WAAW,iBAAiB,EACnD,EAAE,eAAe,EAAE,KAAK,EAAE,EAC1B;QACE,aAAa,EAAE,YAAY;QAC3B,aAAa,EAAE,YAAY;QAC3B,KAAK;QACL,WAAW;QACX,oBAAoB,EAAE,IAAI;KAC3B,EACD,CAAC,IAAI,EAAE,EAAE;QACP,MAAM,IAAI,GAAG,IAA0C,CAAC;QACxD,OAAO,EAAE,GAAG,EAAE,IAAI,CAAC,OAAO,IAAI,EAAE,EAAE,MAAM,EAAE,IAAI,CAAC,GAAG,IAAI,CAAC,EAAE,CAAC;IAC5D,CAAC,EACD,CAAC,MAAM,EAAE,IAAI,EAAE,EAAE,CAAC,MAAM,KAAK,GAAG,IAAI,IAAI,CAAC,QAAQ,CAAC,gBAAgB,CAAC,CACpE,CAAC;AACJ,CAAC;AAED;;;;;;;;;;;;;GAaG;AACH,MAAM,CAAC,KAAK,UAAU,kBAAkB,CACtC,KAAa,EACb,OAAe,EACf,WAAmB,EACnB,MAAc,EACd,KAAc;IAEd,IAAI,CAAC;QACH,MAAM,WAAW,GAAG,kBAAkB,CAAC,WAAW,CAAC,CAAC;QACpD,MAAM,KAAK,GAAG,GAAG,OAAO,aAAa,WAAW,iCAAiC,MAAM,eAAe,CAAC;QAEvG,MAAM,KAAK,GAAG,MAAM,KAAK,CAAC,KAAK,EAAE;YAC/B,OAAO,EAAE,EAAE,eAAe,EAAE,KAAK,EAAE;SACpC,CAAC,CAAC;QAEH,IAAI,CAAC,KAAK,CAAC,EAAE;YAAE,OAAO,SAAS,CAAC;QAEhC,MAAM,IAAI,GAAG,kBAAkB,CAAC,KAAK,CAAC,MAAM,KAAK,CAAC,IAAI,EAAE,CAAC,CAAC;QAC1D,IAAI,IAAI,CAAC,MAAM,KAAK,CAAC;YAAE,OAAO,SAAS,CAAC;QAExC,MAAM,EAAE,GAAG,IAAI,CAAC,CAAC,CAAC,CAAC;QAEnB,MAAM,cAAc,GAAG,GAAG,OAAO,aAAa,WAAW,mBAAmB,EAAE,CAAC,GAAG,2BAA2B,CAAC;QAC9G,MAAM,cAAc,GAAG,MAAM,KAAK,CAAC,cAAc,EAAE;YACjD,OAAO,EAAE,EAAE,eAAe,EAAE,KAAK,EAAE;SACpC,CAAC,CAAC;QAEH,IAAI,CAAC,cAAc,CAAC,EAAE;YAAE,OAAO,SAAS,CAAC;QAEzC,MAAM,WAAW,GAAG,uBAAuB,CAAC,KAAK,CAC/C,MAAM,cAAc,CAAC,IAAI,EAAE,CAC5B,CAAC;QAEF,IAAI,SAAS,GAAG,KAAK,CAAC;QAEtB,KAAK,MAAM,UAAU,IAAI,WAAW,EAAE,CAAC;YACrC,KAAK,MAAM,IAAI,IAAI,UAAU,CAAC,KAAK,EAAE,CAAC;gBACpC,IAAI,IAAI,CAAC,MAAM;oBAAE,SAAS;gBAC1B,IAAI,KAAK,IAAI,IAAI,CAAC,UAAU,IAAI,IAAI,CAAC,UAAU,IAAI,KAAK;oBAAE,SAAS;gBACnE,IACE,IAAI,CAAC,IAAI,KAAK,UAAU;oBACxB,IAAI,CAAC,UAAU,KAAK,KAAK;oBACzB,IAAI,CAAC,QAAQ,KAAK,IAAI,EACtB,CAAC;oBACD,SAAS,GAAG,IAAI,CAAC;oBACjB,MAAM;gBACR,CAAC;gBACD,IAAI,eAAe,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC;oBAC/B,SAAS,GAAG,IAAI,CAAC;oBACjB,MAAM;gBACR,CAAC;YACH,CAAC;YACD,IAAI,SAAS;gBAAE,MAAM;QACvB,CAAC;QAED,OAAO;YACL,gBAAgB,EAAE,SAAS;YAC3B,QAAQ,EAAE,EAAE,CAAC,GAAG;YAChB,KAAK,EAAE,EAAE,CAAC,OAAO;SAClB,CAAC;IACJ,CAAC;IAAC,MAAM,CAAC;QACP,OAAO,SAAS,CAAC;IACnB,CAAC;AACH,CAAC;AAED;;;;;;;;;;;;;;GAcG;AACH,MAAM,CAAC,KAAK,UAAU,qBAAqB,CACzC,KAAa,EACb,OAAe,EACf,WAAmB,EACnB,KAAa,EACb,KAAc;IAEd,IAAI,CAAC;QACH,MAAM,WAAW,GAAG,kBAAkB,CAAC,WAAW,CAAC,CAAC;QACpD,MAAM,GAAG,GAAG,GAAG,OAAO,aAAa,WAAW,mBAAmB,KAAK,2BAA2B,CAAC;QAElG,MAAM,GAAG,GAAG,MAAM,KAAK,CAAC,GAAG,EAAE;YAC3B,OAAO,EAAE,EAAE,eAAe,EAAE,KAAK,EAAE;SACpC,CAAC,CAAC;QAEH,IAAI,CAAC,GAAG,CAAC,EAAE;YAAE,OAAO,EAAE,CAAC;QAEvB,MAAM,WAAW,GAAG,uBAAuB,CAAC,KAAK,CAAC,MAAM,GAAG,CAAC,IAAI,EAAE,CAAC,CAAC;QACpE,MAAM,QAAQ,GAAa,EAAE,CAAC;QAE9B,KAAK,MAAM,UAAU,IAAI,WAAW,EAAE,CAAC;YACrC,KAAK,MAAM,IAAI,IAAI,UAAU,CAAC,KAAK,EAAE,CAAC;gBACpC,IAAI,IAAI,CAAC,MAAM;oBAAE,SAAS;gBAC1B,IAAI,KAAK,IAAI,IAAI,CAAC,UAAU,IAAI,IAAI,CAAC,UAAU,IAAI,KAAK;oBAAE,SAAS;gBAEnE,IACE,IAAI,CAAC,IAAI,KAAK,UAAU;oBACxB,IAAI,CAAC,UAAU,KAAK,KAAK;oBACzB,IAAI,CAAC,QAAQ,KAAK,IAAI,EACtB,CAAC;oBACD,MAAM,MAAM,GAAG,IAAI,CAAC,QAAQ,EAAE,QAAQ;wBACpC,CAAC,CAAC,IAAI,IAAI,CAAC,QAAQ,CAAC,QAAQ,IAAI;wBAChC,CAAC,CAAC,EAAE,CAAC;oBACP,QAAQ,CAAC,IAAI,CAAC,GAAG,MAAM,GAAG,IAAI,CAAC,IAAI,EAAE,CAAC,CAAC;gBACzC,CAAC;qBAAM,IAAI,eAAe,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC;oBACtC,QAAQ,CAAC,IAAI,CAAC,oBAAoB,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC;gBACjD,CAAC;YACH,CAAC;QACH,CAAC;QAED,OAAO,QAAQ,CAAC;IAClB,CAAC;IAAC,MAAM,CAAC;QACP,OAAO,EAAE,CAAC;IACZ,CAAC;AACH,CAAC"}

package/dist/scripts/shared/pull-request/pr-body/pr-body.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"pr-body.d.ts","sourceRoot":"","sources":["../../../../../src/scripts/shared/pull-request/pr-body/pr-body.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AACH,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,EAAE,MAAM,kBAAkB,CAAC;AAE5D;;;;;;;;;GASG;AACH,wBAAgB,WAAW,CAAC,MAAM,EAAE,WAAW,EAAE,MAAM,EAAE,MAAM,GAAG,MAAM,CA8BvE"}
+{"version":3,"file":"pr-body.d.ts","sourceRoot":"","sources":["../../../../../src/scripts/shared/pull-request/pr-body/pr-body.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AACH,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,EAAE,MAAM,kBAAkB,CAAC;AAE5D;;;;;;;;;GASG;AACH,wBAAgB,WAAW,CAAC,MAAM,EAAE,WAAW,EAAE,MAAM,EAAE,MAAM,GAAG,MAAM,CA2CvE"}

package/dist/scripts/shared/pull-request/pr-body/pr-body.js

@@ -30,6 +30,13 @@ export function buildPrBody(config, ticket) {
     }
     lines.push('---');
     lines.push('*Created by [Clancy](https://github.com/Pushedskydiver/clancy)*');
+    lines.push('');
+    lines.push('---');
+    lines.push('**Rework instructions:** To request changes:');
+    lines.push('- **Code comments** — leave inline comments on specific lines. These are always picked up automatically.');
+    lines.push('- **General feedback** — reply with a comment starting with `Rework:` followed by what needs fixing. Comments without the `Rework:` prefix are treated as discussion.');
+    lines.push('');
+    lines.push("Example: `Rework: The form validation doesn't handle empty passwords`");
     return lines.join('\n');
 }
 //# sourceMappingURL=pr-body.js.map

package/dist/scripts/shared/pull-request/pr-body/pr-body.js.map

@@ -1 +1 @@
-{"version":3,"file":"pr-body.js","sourceRoot":"","sources":["../../../../../src/scripts/shared/pull-request/pr-body/pr-body.ts"],"names":[],"mappings":"AAQA;;;;;;;;;GASG;AACH,MAAM,UAAU,WAAW,CAAC,MAAmB,EAAE,MAAc;IAC7D,MAAM,KAAK,GAAa,EAAE,CAAC;IAE3B,QAAQ,MAAM,CAAC,QAAQ,EAAE,CAAC;QACxB,KAAK,QAAQ;YACX,KAAK,CAAC,IAAI,CAAC,UAAU,MAAM,CAAC,GAAG,EAAE,CAAC,CAAC;YACnC,MAAM;QACR,KAAK,MAAM;YACT,KAAK,CAAC,IAAI,CACR,cAAc,MAAM,CAAC,GAAG,KAAK,MAAM,CAAC,GAAG,CAAC,aAAa,WAAW,MAAM,CAAC,GAAG,GAAG,CAC9E,CAAC;YACF,MAAM;QACR,KAAK,QAAQ;YACX,KAAK,CAAC,IAAI,CAAC,eAAe,MAAM,CAAC,GAAG,EAAE,CAAC,CAAC;YACxC,MAAM;IACV,CAAC;IAED,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;IAEf,IAAI,MAAM,CAAC,WAAW,EAAE,CAAC;QACvB,KAAK,CAAC,IAAI,CAAC,gBAAgB,CAAC,CAAC;QAC7B,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;QACf,KAAK,CAAC,IAAI,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC;QAC/B,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;IACjB,CAAC;IAED,KAAK,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;IAClB,KAAK,CAAC,IAAI,CAAC,iEAAiE,CAAC,CAAC;IAE9E,OAAO,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AAC1B,CAAC"}
+{"version":3,"file":"pr-body.js","sourceRoot":"","sources":["../../../../../src/scripts/shared/pull-request/pr-body/pr-body.ts"],"names":[],"mappings":"AAQA;;;;;;;;;GASG;AACH,MAAM,UAAU,WAAW,CAAC,MAAmB,EAAE,MAAc;IAC7D,MAAM,KAAK,GAAa,EAAE,CAAC;IAE3B,QAAQ,MAAM,CAAC,QAAQ,EAAE,CAAC;QACxB,KAAK,QAAQ;YACX,KAAK,CAAC,IAAI,CAAC,UAAU,MAAM,CAAC,GAAG,EAAE,CAAC,CAAC;YACnC,MAAM;QACR,KAAK,MAAM;YACT,KAAK,CAAC,IAAI,CACR,cAAc,MAAM,CAAC,GAAG,KAAK,MAAM,CAAC,GAAG,CAAC,aAAa,WAAW,MAAM,CAAC,GAAG,GAAG,CAC9E,CAAC;YACF,MAAM;QACR,KAAK,QAAQ;YACX,KAAK,CAAC,IAAI,CAAC,eAAe,MAAM,CAAC,GAAG,EAAE,CAAC,CAAC;YACxC,MAAM;IACV,CAAC;IAED,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;IAEf,IAAI,MAAM,CAAC,WAAW,EAAE,CAAC;QACvB,KAAK,CAAC,IAAI,CAAC,gBAAgB,CAAC,CAAC;QAC7B,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;QACf,KAAK,CAAC,IAAI,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC;QAC/B,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;IACjB,CAAC;IAED,KAAK,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;IAClB,KAAK,CAAC,IAAI,CAAC,iEAAiE,CAAC,CAAC;IAC9E,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;IACf,KAAK,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;IAClB,KAAK,CAAC,IAAI,CAAC,8CAA8C,CAAC,CAAC;IAC3D,KAAK,CAAC,IAAI,CACR,0GAA0G,CAC3G,CAAC;IACF,KAAK,CAAC,IAAI,CACR,uKAAuK,CACxK,CAAC;IACF,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;IACf,KAAK,CAAC,IAAI,CACR,uEAAuE,CACxE,CAAC;IAEF,OAAO,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AAC1B,CAAC"}

package/dist/scripts/shared/pull-request/rework-comment/rework-comment.d.ts

@@ -0,0 +1,23 @@
+/**
+ * Rework comment detection and extraction utilities.
+ *
+ * Comments starting with "Rework:" (case-insensitive) are treated as
+ * rework requests. All other comments are treated as discussion and ignored.
+ */
+/**
+ * Check if a comment is a rework request (starts with "Rework:" case-insensitive).
+ *
+ * @param body - The comment body to check.
+ * @returns `true` if the comment starts with "Rework:".
+ */
+export declare function isReworkComment(body: string): boolean;
+/**
+ * Extract the rework description from a "Rework: ..." comment.
+ *
+ * Strips the "Rework:" prefix and returns the remaining content trimmed.
+ *
+ * @param body - The full comment body starting with "Rework:".
+ * @returns The description after the prefix.
+ */
+export declare function extractReworkContent(body: string): string;
+//# sourceMappingURL=rework-comment.d.ts.map

package/dist/scripts/shared/pull-request/rework-comment/rework-comment.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"rework-comment.d.ts","sourceRoot":"","sources":["../../../../../src/scripts/shared/pull-request/rework-comment/rework-comment.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH;;;;;GAKG;AACH,wBAAgB,eAAe,CAAC,IAAI,EAAE,MAAM,GAAG,OAAO,CAErD;AAED;;;;;;;GAOG;AACH,wBAAgB,oBAAoB,CAAC,IAAI,EAAE,MAAM,GAAG,MAAM,CAKzD"}

package/dist/scripts/shared/pull-request/rework-comment/rework-comment.js

@@ -0,0 +1,30 @@
+/**
+ * Rework comment detection and extraction utilities.
+ *
+ * Comments starting with "Rework:" (case-insensitive) are treated as
+ * rework requests. All other comments are treated as discussion and ignored.
+ */
+/**
+ * Check if a comment is a rework request (starts with "Rework:" case-insensitive).
+ *
+ * @param body - The comment body to check.
+ * @returns `true` if the comment starts with "Rework:".
+ */
+export function isReworkComment(body) {
+    return body.trim().toLowerCase().startsWith('rework:');
+}
+/**
+ * Extract the rework description from a "Rework: ..." comment.
+ *
+ * Strips the "Rework:" prefix and returns the remaining content trimmed.
+ *
+ * @param body - The full comment body starting with "Rework:".
+ * @returns The description after the prefix.
+ */
+export function extractReworkContent(body) {
+    return body
+        .trim()
+        .replace(/^rework:\s*/i, '')
+        .trim();
+}
+//# sourceMappingURL=rework-comment.js.map

package/dist/scripts/shared/pull-request/rework-comment/rework-comment.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"rework-comment.js","sourceRoot":"","sources":["../../../../../src/scripts/shared/pull-request/rework-comment/rework-comment.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH;;;;;GAKG;AACH,MAAM,UAAU,eAAe,CAAC,IAAY;IAC1C,OAAO,IAAI,CAAC,IAAI,EAAE,CAAC,WAAW,EAAE,CAAC,UAAU,CAAC,SAAS,CAAC,CAAC;AACzD,CAAC;AAED;;;;;;;GAOG;AACH,MAAM,UAAU,oBAAoB,CAAC,IAAY;IAC/C,OAAO,IAAI;SACR,IAAI,EAAE;SACN,OAAO,CAAC,cAAc,EAAE,EAAE,CAAC;SAC3B,IAAI,EAAE,CAAC;AACZ,CAAC"}

package/dist/types/index.d.ts

@@ -1,4 +1,4 @@
 export type { BoardProvider, Ticket } from './board.js';
-export type { GitPlatform, PrCreationResult, ProgressStatus, RemoteInfo, } from './remote.js';
+export type { GitPlatform, PrCreationResult, PrReviewState, ProgressStatus, RemoteInfo, } from './remote.js';
 export type { BoardConfig, GitHubEnv, JiraEnv, LinearEnv, SharedEnv, } from '../schemas/env.js';
 //# sourceMappingURL=index.d.ts.map

package/dist/types/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../src/types/index.ts"],"names":[],"mappings":"AAAA,YAAY,EAAE,aAAa,EAAE,MAAM,EAAE,MAAM,YAAY,CAAC;AACxD,YAAY,EACV,WAAW,EACX,gBAAgB,EAChB,cAAc,EACd,UAAU,GACX,MAAM,aAAa,CAAC;AACrB,YAAY,EACV,WAAW,EACX,SAAS,EACT,OAAO,EACP,SAAS,EACT,SAAS,GACV,MAAM,kBAAkB,CAAC"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../src/types/index.ts"],"names":[],"mappings":"AAAA,YAAY,EAAE,aAAa,EAAE,MAAM,EAAE,MAAM,YAAY,CAAC;AACxD,YAAY,EACV,WAAW,EACX,gBAAgB,EAChB,aAAa,EACb,cAAc,EACd,UAAU,GACX,MAAM,aAAa,CAAC;AACrB,YAAY,EACV,WAAW,EACX,SAAS,EACT,OAAO,EACP,SAAS,EACT,SAAS,GACV,MAAM,kBAAkB,CAAC"}

package/dist/types/remote.d.ts

@@ -39,6 +39,15 @@ export type PrCreationResult = {
     error: string;
     alreadyExists?: boolean;
 };
+/** Result of checking PR/MR review state. */
+export type PrReviewState = {
+    /** Whether changes have been requested by a reviewer. */
+    changesRequested: boolean;
+    /** The PR/MR number/ID (needed to fetch comments). */
+    prNumber: number;
+    /** The PR/MR URL (for logging). */
+    prUrl: string;
+};
 /** Progress log status values. */
 export type ProgressStatus = 'DONE' | 'SKIPPED' | 'PR_CREATED' | 'PUSHED' | 'PUSH_FAILED' | 'LOCAL' | 'PLAN' | 'APPROVE' | 'REWORK';
 //# sourceMappingURL=remote.d.ts.map

package/dist/types/remote.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"remote.d.ts","sourceRoot":"","sources":["../../src/types/remote.ts"],"names":[],"mappings":"AAAA;;GAEG;AAEH,uCAAuC;AACvC,MAAM,MAAM,WAAW,GACnB,QAAQ,GACR,QAAQ,GACR,WAAW,GACX,kBAAkB,GAClB,OAAO,GACP,SAAS,CAAC;AAEd,qDAAqD;AACrD,MAAM,MAAM,UAAU,GAClB;IACE,IAAI,EAAE,QAAQ,CAAC;IACf,KAAK,EAAE,MAAM,CAAC;IACd,IAAI,EAAE,MAAM,CAAC;IACb,QAAQ,EAAE,MAAM,CAAC;CAClB,GACD;IACE,IAAI,EAAE,QAAQ,CAAC;IACf,WAAW,EAAE,MAAM,CAAC;IACpB,QAAQ,EAAE,MAAM,CAAC;CAClB,GACD;IACE,IAAI,EAAE,WAAW,CAAC;IAClB,SAAS,EAAE,MAAM,CAAC;IAClB,QAAQ,EAAE,MAAM,CAAC;IACjB,QAAQ,EAAE,MAAM,CAAC;CAClB,GACD;IACE,IAAI,EAAE,kBAAkB,CAAC;IACzB,UAAU,EAAE,MAAM,CAAC;IACnB,QAAQ,EAAE,MAAM,CAAC;IACjB,QAAQ,EAAE,MAAM,CAAC;CAClB,GACD;IACE,IAAI,EAAE,OAAO,GAAG,SAAS,CAAC;IAC1B,GAAG,EAAE,MAAM,CAAC;CACb,GACD;IACE,IAAI,EAAE,MAAM,CAAC;CACd,CAAC;AAEN,0CAA0C;AAC1C,MAAM,MAAM,gBAAgB,GACxB;IAAE,EAAE,EAAE,IAAI,CAAC;IAAC,GAAG,EAAE,MAAM,CAAC;IAAC,MAAM,EAAE,MAAM,CAAA;CAAE,GACzC;IAAE,EAAE,EAAE,KAAK,CAAC;IAAC,KAAK,EAAE,MAAM,CAAC;IAAC,aAAa,CAAC,EAAE,OAAO,CAAA;CAAE,CAAC;AAE1D,kCAAkC;AAClC,MAAM,MAAM,cAAc,GACtB,MAAM,GACN,SAAS,GACT,YAAY,GACZ,QAAQ,GACR,aAAa,GACb,OAAO,GACP,MAAM,GACN,SAAS,GACT,QAAQ,CAAC"}
+{"version":3,"file":"remote.d.ts","sourceRoot":"","sources":["../../src/types/remote.ts"],"names":[],"mappings":"AAAA;;GAEG;AAEH,uCAAuC;AACvC,MAAM,MAAM,WAAW,GACnB,QAAQ,GACR,QAAQ,GACR,WAAW,GACX,kBAAkB,GAClB,OAAO,GACP,SAAS,CAAC;AAEd,qDAAqD;AACrD,MAAM,MAAM,UAAU,GAClB;IACE,IAAI,EAAE,QAAQ,CAAC;IACf,KAAK,EAAE,MAAM,CAAC;IACd,IAAI,EAAE,MAAM,CAAC;IACb,QAAQ,EAAE,MAAM,CAAC;CAClB,GACD;IACE,IAAI,EAAE,QAAQ,CAAC;IACf,WAAW,EAAE,MAAM,CAAC;IACpB,QAAQ,EAAE,MAAM,CAAC;CAClB,GACD;IACE,IAAI,EAAE,WAAW,CAAC;IAClB,SAAS,EAAE,MAAM,CAAC;IAClB,QAAQ,EAAE,MAAM,CAAC;IACjB,QAAQ,EAAE,MAAM,CAAC;CAClB,GACD;IACE,IAAI,EAAE,kBAAkB,CAAC;IACzB,UAAU,EAAE,MAAM,CAAC;IACnB,QAAQ,EAAE,MAAM,CAAC;IACjB,QAAQ,EAAE,MAAM,CAAC;CAClB,GACD;IACE,IAAI,EAAE,OAAO,GAAG,SAAS,CAAC;IAC1B,GAAG,EAAE,MAAM,CAAC;CACb,GACD;IACE,IAAI,EAAE,MAAM,CAAC;CACd,CAAC;AAEN,0CAA0C;AAC1C,MAAM,MAAM,gBAAgB,GACxB;IAAE,EAAE,EAAE,IAAI,CAAC;IAAC,GAAG,EAAE,MAAM,CAAC;IAAC,MAAM,EAAE,MAAM,CAAA;CAAE,GACzC;IAAE,EAAE,EAAE,KAAK,CAAC;IAAC,KAAK,EAAE,MAAM,CAAC;IAAC,aAAa,CAAC,EAAE,OAAO,CAAA;CAAE,CAAC;AAE1D,6CAA6C;AAC7C,MAAM,MAAM,aAAa,GAAG;IAC1B,yDAAyD;IACzD,gBAAgB,EAAE,OAAO,CAAC;IAC1B,sDAAsD;IACtD,QAAQ,EAAE,MAAM,CAAC;IACjB,mCAAmC;IACnC,KAAK,EAAE,MAAM,CAAC;CACf,CAAC;AAEF,kCAAkC;AAClC,MAAM,MAAM,cAAc,GACtB,MAAM,GACN,SAAS,GACT,YAAY,GACZ,QAAQ,GACR,aAAa,GACb,OAAO,GACP,MAAM,GACN,SAAS,GACT,QAAQ,CAAC"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "chief-clancy",
-  "version": "0.5.4",
+  "version": "0.5.5",
   "description": "Autonomous, board-driven development for Claude Code — scaffolds docs, integrates Kanban boards, runs tickets in a loop.",
   "keywords": [
     "claude",

package/src/roles/setup/workflows/init.md

@@ -406,6 +406,21 @@ Same storage logic as Jira above.
 
 ---
 
+### Q3e (all boards): Max rework cycles
+
+PR-based rework detection is automatic — no configuration needed. This setting controls the safety limit.
+
+Output:
+
+```
+Max rework cycles before flagging for human intervention? [3]
+```
+
+If a number is entered: store as `CLANCY_MAX_REWORK` in `.clancy/.env`.
+If enter is pressed with no value: use default 3 — store `CLANCY_MAX_REWORK=3` in `.clancy/.env`.
+
+---
+
 ### Q4: Base branch (auto-detect)
 
 Silently detect the base branch — do not ask unless detection fails:

package/src/roles/setup/workflows/scaffold.md

@@ -396,6 +396,12 @@ MAX_ITERATIONS=5
 # CLANCY_GIT_PLATFORM=gitlab               # override auto-detection (github/gitlab/bitbucket)
 # CLANCY_GIT_API_URL=https://gitlab.example.com/api/v4  # self-hosted git API base URL
 
+# ─── Optional: Rework loop ──────────────────────────────────────────────────
+# PR-based rework is automatic — when a reviewer leaves inline comments or
+# a conversation comment prefixed with "Rework:", Clancy picks it up on the
+# next run. No configuration needed.
+# CLANCY_MAX_REWORK=3                    # Max rework cycles before human intervention (default: 3)
+
 # ─── Optional: Planner queue ─────────────────────────────────────────────────
 # Status for backlog tickets that /clancy:plan fetches from (default: Backlog)
 # Only used if Planner role is enabled via CLANCY_ROLES
@@ -457,6 +463,12 @@ MAX_ITERATIONS=20
 # PLAYWRIGHT_STORYBOOK_PORT=6006
 # PLAYWRIGHT_STARTUP_WAIT=15
 
+# ─── Optional: Rework loop ──────────────────────────────────────────────────
+# PR-based rework is automatic — when a reviewer leaves inline comments or
+# a conversation comment prefixed with "Rework:", Clancy picks it up on the
+# next run. No configuration needed.
+# CLANCY_MAX_REWORK=3                    # Max rework cycles before human intervention (default: 3)
+
 # ─── Optional: Notifications ──────────────────────────────────────────────────
 # Webhook URL for Slack or Teams notifications on ticket completion
 # CLANCY_NOTIFY_WEBHOOK=https://hooks.slack.com/services/your/webhook/url
@@ -513,6 +525,12 @@ MAX_ITERATIONS=20
 # CLANCY_STATUS_DONE="Done"
 # CLANCY_STATUS_REVIEW="In Review"        # used when creating a PR instead of merging locally
 
+# ─── Optional: Rework loop ──────────────────────────────────────────────────
+# PR-based rework is automatic — when a reviewer leaves inline comments or
+# a conversation comment prefixed with "Rework:", Clancy picks it up on the
+# next run. No configuration needed.
+# CLANCY_MAX_REWORK=3                    # Max rework cycles before human intervention (default: 3)
+
 # ─── Optional: Git host (PR creation) ───────────────────────────────────────
 # When an issue has no parent, Clancy pushes the feature branch and creates a
 # pull request instead of squash-merging locally. Requires a git host token.

package/src/roles/setup/workflows/settings.md

@@ -51,6 +51,7 @@ General
   [G1] Max iterations    {MAX_ITERATIONS:-5}          tickets per /clancy:run session
   [G2] Claude model      {CLANCY_MODEL:-default}     model used for each ticket session
   [G3] Base branch       {CLANCY_BASE_BRANCH:-main}
+  [G4] Max rework        {CLANCY_MAX_REWORK:-3}
 
 {If Jira:}
 Jira
@@ -61,16 +62,12 @@ Jira
   [B5] Done status       {CLANCY_STATUS_DONE if set, else off}
   [B6] Review status     {CLANCY_STATUS_REVIEW if set, else "uses Done status"}
 
-{If GitHub:}
-GitHub
-  (No board-specific settings — labels are managed in GitHub directly)
-
 {If Linear:}
 Linear
   [B1] Label filter      {CLANCY_LABEL if set, else off}
   [B2] Pickup status     {CLANCY_STATUS_IN_PROGRESS if set, else off}
   [B3] Done status       {CLANCY_STATUS_DONE if set, else off}
-  [B4] Review status     {CLANCY_STATUS_REVIEW if set, else "uses Done status"}
+  [B4] Review status     {CLANCY_STATUS_REVIEW if set, else "uses Done state"}
 
 Roles
   [R1] Planner           {✅ enabled / ─ disabled}
@@ -156,6 +153,24 @@ Write `CLANCY_BASE_BRANCH=<value>` to `.clancy/.env`.
 
 ---
 
+### [G4] Max rework cycles
+
+```
+Max rework cycles — current: {value or 3}
+After this many rework cycles on a single ticket, Clancy flags it for human intervention.
+
+[1] 3 (default)
+[2] Enter a different number
+[3] Cancel
+```
+
+Validate the input is a positive integer between 1 and 20. If invalid, re-prompt.
+
+If [1]: remove `CLANCY_MAX_REWORK` from `.clancy/.env` (uses default).
+If [2]: prompt `How many rework cycles before human intervention?` then write `CLANCY_MAX_REWORK=<value>` to `.clancy/.env`.
+
+---
+
 ### [B1] Jira status filter (Jira only)
 
 ```