@sideboard-ai/core 0.1.183 → 0.1.185

package/dist/index.js

@@ -238,7 +238,7 @@ import {
   worktreeCleanupSettings,
   wrapReviewSkillMarkdown,
   writeWorktreeFile
-} from "./chunk-26N6LKLE.js";
+} from "./chunk-G4VAMUCL.js";
 import {
   LEGACY_PLAN_FILE_REL,
   PLAN_FILE_NAME,
@@ -344,7 +344,7 @@ import {
   withEventParentId,
   withEventsParentId,
   writeInjectedMcpConfig
-} from "./chunk-VA76U3BU.js";
+} from "./chunk-5FTIOINZ.js";
 import {
   CLOUD_COORDINATOR_BUSY_REPLY,
   CLOUD_COORDINATOR_STOPPED_REPLY,
@@ -387,7 +387,7 @@ import {
   stageBuffersAsAttachments,
   syncWorkspacesFromThreads,
   takenTeamSlugsForOrchestration
-} from "./chunk-JFDCHOU7.js";
+} from "./chunk-VIODP5PT.js";
 import {
   FAMOUS_SOCCER_TEAMS,
   GITHUB_PR_BODY_MAX_CHARS,
@@ -1518,12 +1518,27 @@ var UUID_RE = /^[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}$/i;
 function assigneeFilterKey(assignee) {
   return (assignee ?? "").trim();
 }
+function applyLinearIssueQueryFilter(filter, query) {
+  const term = query?.trim();
+  if (!term) return;
+  const or = [
+    { title: { containsIgnoreCase: term } },
+    { description: { containsIgnoreCase: term } }
+  ];
+  const identifierMatch = term.match(/^[A-Za-z][\w]*-(\d+)$/);
+  const asNumber = identifierMatch ? Number(identifierMatch[1]) : /^\d+$/.test(term) ? Number(term) : NaN;
+  if (Number.isFinite(asNumber) && asNumber > 0) {
+    or.push({ number: { eq: asNumber } });
+  }
+  filter.or = or;
+}
 function buildLinearIssueFilter(input) {
   const filter = {
     state: { type: { nin: ["completed", "canceled"] } }
   };
   const since = input?.updatedSince?.trim();
   if (since) filter.updatedAt = { gte: since };
+  applyLinearIssueQueryFilter(filter, input?.query);
   const raw = assigneeFilterKey(input?.assignee) || "me";
   const key = raw.toLowerCase();
   if (key === "all" || key === "*") return filter;
@@ -1589,6 +1604,11 @@ async function listLinearIssuesFiltered(opts) {
 async function listLinearCommentsSince(opts) {
   const first = Math.max(1, Math.min(250, opts.limit ?? 200));
   const query = opts.query?.trim() ?? "";
+  const restrictToIssues = opts.issueIdentifiers != null;
+  const allowed = new Set(
+    (opts.issueIdentifiers ?? []).map((id) => id.trim()).filter(Boolean)
+  );
+  if (restrictToIssues && allowed.size === 0) return [];
   const assignee = assigneeFilterKey(opts.assignee) || (query ? "all" : "me");
   const json = await linearGraphql(
     COMMENTS_SINCE_QUERY,
@@ -1596,7 +1616,7 @@ async function listLinearCommentsSince(opts) {
       first,
       filter: {
         createdAt: { gte: opts.since },
-        issue: buildLinearIssueFilter({ assignee })
+        issue: buildLinearIssueFilter({ assignee, query: query || void 0 })
       }
     },
     opts
@@ -1604,6 +1624,7 @@ async function listLinearCommentsSince(opts) {
   const out = [];
   for (const node of json.comments?.nodes ?? []) {
     const identifier = String(node.issue?.identifier ?? "").trim();
+    if (restrictToIssues && !allowed.has(identifier)) continue;
     const body = previewIssueCommentBody(String(node.body ?? ""));
     if (!identifier && !body) continue;
     out.push({
@@ -1833,10 +1854,32 @@ function issueNumberFromApiUrl(url) {
   const number = Number(match[1]);
   return Number.isFinite(number) && number > 0 ? number : null;
 }
+var GITHUB_ISSUE_COMMENTS_PAGE_SIZE = 100;
+var GITHUB_ISSUE_COMMENTS_MAX_PAGES = 5;
+function parseGitHubIssueCommentNode(item, allowed) {
+  const rec = item && typeof item === "object" ? item : null;
+  if (!rec) return null;
+  const issueUrl = typeof rec.issue_url === "string" ? rec.issue_url : "";
+  const number = issueNumberFromApiUrl(issueUrl);
+  if (number == null) return null;
+  if (allowed && !allowed.has(number)) return null;
+  const body = previewIssueCommentBody(typeof rec.body === "string" ? rec.body : "");
+  const user = rec.user && typeof rec.user === "object" ? String(rec.user.login ?? "").trim() : "";
+  const createdAt = typeof rec.created_at === "string" ? rec.created_at : typeof rec.createdAt === "string" ? rec.createdAt : void 0;
+  const url = typeof rec.html_url === "string" ? rec.html_url : void 0;
+  return {
+    identifier: `#${number}`,
+    ...user ? { author: user } : {},
+    ...createdAt ? { createdAt } : {},
+    body,
+    ...url ? { url } : {}
+  };
+}
 async function listGitHubIssueCommentsSince(opts) {
   const limit = Math.max(1, Math.min(250, opts.limit ?? 200));
   const { cwd, slug } = await resolveGitHubIssueRepo(opts.repoPath);
   if (!slug) return [];
+  const restrictToIssues = opts.issueIdentifiers != null;
   const allowed = new Set(
     (opts.issueIdentifiers ?? []).map((id) => {
       try {
@@ -1846,36 +1889,27 @@ async function listGitHubIssueCommentsSince(opts) {
       }
     }).filter((n) => n != null)
   );
-  const path2 = `repos/${slug}/issues/comments?since=${encodeURIComponent(opts.since)}&per_page=${limit + 1}`;
-  const result = await gh(["api", path2], cwd, { reject: false });
-  if (result.exitCode !== 0 || !result.stdout.trim()) return [];
-  let parsed;
-  try {
-    parsed = JSON.parse(result.stdout);
-  } catch {
-    return [];
-  }
-  if (!Array.isArray(parsed)) return [];
+  if (restrictToIssues && allowed.size === 0) return [];
+  const pageSize = Math.min(GITHUB_ISSUE_COMMENTS_PAGE_SIZE, Math.max(1, limit));
   const out = [];
-  for (const item of parsed) {
-    const rec = item && typeof item === "object" ? item : null;
-    if (!rec) continue;
-    const issueUrl = typeof rec.issue_url === "string" ? rec.issue_url : "";
-    const number = issueNumberFromApiUrl(issueUrl);
-    if (number == null) continue;
-    if (allowed.size > 0 && !allowed.has(number)) continue;
-    const body = previewIssueCommentBody(typeof rec.body === "string" ? rec.body : "");
-    const user = rec.user && typeof rec.user === "object" ? String(rec.user.login ?? "").trim() : "";
-    const createdAt = typeof rec.created_at === "string" ? rec.created_at : typeof rec.createdAt === "string" ? rec.createdAt : void 0;
-    const url = typeof rec.html_url === "string" ? rec.html_url : void 0;
-    out.push({
-      identifier: `#${number}`,
-      ...user ? { author: user } : {},
-      ...createdAt ? { createdAt } : {},
-      body,
-      ...url ? { url } : {}
-    });
-    if (out.length >= limit) break;
+  for (let page = 1; page <= GITHUB_ISSUE_COMMENTS_MAX_PAGES; page++) {
+    const path2 = `repos/${slug}/issues/comments?since=${encodeURIComponent(opts.since)}&sort=created&direction=desc&per_page=${pageSize}&page=${page}`;
+    const result = await gh(["api", path2], cwd, { reject: false });
+    if (result.exitCode !== 0 || !result.stdout.trim()) break;
+    let parsed;
+    try {
+      parsed = JSON.parse(result.stdout);
+    } catch {
+      break;
+    }
+    if (!Array.isArray(parsed) || parsed.length === 0) break;
+    for (const item of parsed) {
+      const comment = parseGitHubIssueCommentNode(item, restrictToIssues ? allowed : null);
+      if (!comment) continue;
+      out.push(comment);
+      if (out.length >= limit) return out;
+    }
+    if (parsed.length < pageSize) break;
   }
   return out;
 }
@@ -2081,20 +2115,19 @@ async function listIssues(repoPath, opts) {
   const assignee = opts?.assignee?.trim() || void 0;
   const since = opts?.updatedSince ? parseIssueSince(opts.updatedSince) : void 0;
   if (source === "linear") {
-    const [listed, comments2] = await Promise.all([
-      listLinearIssuesFiltered({
-        assignee,
-        query,
-        limit: opts?.limit,
-        updatedSince: since
-      }),
-      since ? listLinearCommentsSince({
-        since,
-        assignee,
-        query,
-        limit: opts?.limit
-      }) : Promise.resolve(void 0)
-    ]);
+    const listed = await listLinearIssuesFiltered({
+      assignee,
+      query,
+      limit: opts?.limit,
+      updatedSince: since
+    });
+    const comments2 = since ? await listLinearCommentsSince({
+      since,
+      assignee,
+      query,
+      issueIdentifiers: query ? listed.issues.map((issue) => issue.identifier) : void 0,
+      limit: opts?.limit
+    }) : void 0;
     return {
       source,
       preferredSource,
@@ -3536,7 +3569,7 @@ function registerAbleTimeTools(server) {
   );
   server.tool(
     "abletime_comment",
-    "Add a markdown comment on an AbleTime task (id or CRM-232). When reviewing a PR or ticket, show the draft in chat and ask_user (Post this review / Keep it in chat) first \u2014 the author sees this immediately.",
+    "Add a markdown comment on an AbleTime task (id or CRM-232). When reviewing a PR or ticket, show the draft in chat and wait until they type a next step \u2014 do not ask_user after the review. The author sees this immediately.",
     { id: z4.string(), body: z4.string() },
     async (args) => {
       try {
@@ -3548,7 +3581,7 @@ function registerAbleTimeTools(server) {
   );
   server.tool(
     "abletime_update_task",
-    "Update an AbleTime task (id or CRM-232). Pass title, description, and/or state. When reviewing a PR or ticket, ask_user (Post this review / Keep it in chat) before changing the task \u2014 the author is notified.",
+    "Update an AbleTime task (id or CRM-232). Pass title, description, and/or state. When reviewing a PR or ticket, wait until they type a next step before changing the task \u2014 do not ask_user after the review. The author is notified.",
     {
       id: z4.string(),
       title: z4.string().optional(),
@@ -3677,7 +3710,7 @@ function registerGithubIssueTools(server) {
   );
   server.tool(
     "github_comment",
-    "Add a markdown comment on a GitHub issue (#123 or URL). Uses Account gh. When reviewing a PR or ticket, show the draft in chat and ask_user (Post this review / Keep it in chat) first \u2014 the author sees this immediately.",
+    "Add a markdown comment on a GitHub issue (#123 or URL). Uses Account gh. When reviewing a PR or ticket, show the draft in chat and wait until they type a next step \u2014 do not ask_user after the review. The author sees this immediately.",
     { id: z5.string(), body: z5.string(), repoPath: repoPathSchema },
     async ({ id, body, repoPath }) => {
       try {
@@ -3689,7 +3722,7 @@ function registerGithubIssueTools(server) {
   );
   server.tool(
     "github_update_issue",
-    "Update a GitHub issue (#123). Pass title, body, and/or state (open|closed). When reviewing a PR or ticket, ask_user (Post this review / Keep it in chat) before changing the issue \u2014 the author is notified.",
+    "Update a GitHub issue (#123). Pass title, body, and/or state (open|closed). When reviewing a PR or ticket, wait until they type a next step before changing the issue \u2014 do not ask_user after the review. The author is notified.",
     {
       id: z5.string(),
       title: z5.string().optional(),
@@ -3762,20 +3795,19 @@ function registerLinearTools(server) {
       try {
         const page = clampMcpIssueLimit(limit);
         const since = updatedSince ? parseIssueSince(updatedSince) : void 0;
-        const [listed, comments] = await Promise.all([
-          listLinearIssuesFiltered({
-            query,
-            assignee,
-            limit: page + 1,
-            updatedSince: since
-          }),
-          since ? listLinearCommentsSince({
-            since,
-            assignee,
-            query,
-            limit: page + 1
-          }) : Promise.resolve(void 0)
-        ]);
+        const listed = await listLinearIssuesFiltered({
+          query,
+          assignee,
+          limit: page + 1,
+          updatedSince: since
+        });
+        const comments = since ? await listLinearCommentsSince({
+          since,
+          assignee,
+          query,
+          issueIdentifiers: query ? listed.issues.map((issue) => issue.identifier) : void 0,
+          limit: page + 1
+        }) : void 0;
         return mcpJson(
           formatListedIssuesForMcp(
             {
@@ -3827,7 +3859,7 @@ function registerLinearTools(server) {
   );
   server.tool(
     "linear_update_issue",
-    "Update a Linear issue (uuid or ENG-123). Pass title, description, state, assignee, and/or priority. When reviewing a PR or ticket, ask_user (Post this review / Keep it in chat) before changing the ticket \u2014 the author is notified.",
+    "Update a Linear issue (uuid or ENG-123). Pass title, description, state, assignee, and/or priority. When reviewing a PR or ticket, wait until they type a next step before changing the ticket \u2014 do not ask_user after the review. The author is notified.",
     {
       id: z6.string(),
       title: z6.string().optional(),
@@ -3846,7 +3878,7 @@ function registerLinearTools(server) {
   );
   server.tool(
     "linear_comment",
-    "Add a markdown comment on a Linear issue (uuid or ENG-123). When reviewing a PR or ticket, show the draft in chat and ask_user (Post this review / Keep it in chat) first \u2014 the author sees this immediately.",
+    "Add a markdown comment on a Linear issue (uuid or ENG-123). When reviewing a PR or ticket, show the draft in chat and wait until they type a next step \u2014 do not ask_user after the review. The author sees this immediately.",
     {
       id: z6.string(),
       body: z6.string()
@@ -5229,7 +5261,7 @@ async function startMcpServer() {
     );
     server.tool(
       "request_review",
-      'Start a merge-readiness Review on a worktree agent thread (same as the desktop Review button). Opens a new Review chat tab, attaches .claude/skills/review/SKILL.md when present (else copies .sideboard/review.md into .context/review.md, or seeds that file from the stock template), and sends "Review changes in this workspace." Expect Approve / Approve with nits / Request changes / Needs more information in that chat. Do not comment on or update the PR or ticket until the user confirms (ask_user: Post this review / Keep it in chat) \u2014 they work through the feedback in chat first. Pass a worktree thread ref \u2014 not the orchestrator. Then wait_for_turn (loop while stillRunning) / get_turn_result on the returned review tab id.',
+      'Start a merge-readiness Review on a worktree agent thread (same as the desktop Review button). Opens a new Review chat tab, attaches .claude/skills/review/SKILL.md when present (else copies .sideboard/review.md into .context/review.md, or seeds that file from the stock template), and sends "Review changes in this workspace." Expect Approve / Approve with nits / Request changes / Needs more information in that chat. The review stays there so the user can read it and type next steps \u2014 do not comment on or update the PR or ticket, and do not ask_user after the review, until they ask. Pass a worktree thread ref \u2014 not the orchestrator. Then wait_for_turn (loop while stillRunning) / get_turn_result on the returned review tab id.',
       { ref: z9.string().describe("Worktree thread id/ref to review") },
       async ({ ref }) => {
         try {