taskchef 7.14.0 → 7.14.2

package/.codex-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "taskchef",
-  "version": "7.14.0",
+  "version": "7.14.2",
   "description": "Dispatch work from a data-only workspace to visible Codex project tasks.",
   "author": {
     "name": "Favo Yang",

package/docs/spec.md

@@ -138,6 +138,13 @@ the final link, preserving the delegate skill's immediate-return contract.
    TaskChef MUST atomically close it as `interrupted` before appending the new
    working turn; the executor MUST NOT report semantic `failed` for recovery.
 
+Request and result summaries are the durable source for dashboard related-link
+projection; TaskChef does not scan full Codex transcripts. When known, an
+executor MUST preserve the selected repository as a canonical GitHub repository
+URL and issues or pull requests as canonical URLs. A result spanning both a
+managed child repository and its workspace/root repository MUST include both
+pull-request URLs. Executors MUST NOT guess unresolved repository identity.
+
 If native creation fails after recording, the dispatcher MUST call
 `report_state` with `failed`, null thread/turn IDs, and a bounded summary.
 A link failure MUST remain visible and retryable; the executor MUST report it
@@ -287,8 +294,8 @@ marker, or ineligible state fails.
 | `threadId` | Matching non-empty ID for a linked task; null only for creation failure. |
 | `turnId` | Current canonical Codex UUIDv7 for a linked MCP journey; null only for creation failure. Maximum 256 characters at the MCP boundary. |
 | `status` | `working`, `needs_input`, `completed`, or `failed`. |
-| `summary` | Omitted or null for `working`; required non-empty string of at most 2,000 characters otherwise. |
-| `requestSummary` | Concise current request of at most 1,000 characters for `working`; optional for backward compatibility and omitted for semantic states. |
+| `summary` | Omitted or null for `working`; required non-empty string of at most 2,000 characters otherwise. Known GitHub issues and pull requests use canonical URLs. |
+| `requestSummary` | Concise current request of at most 1,000 characters for `working`; optional for backward compatibility and omitted for semantic states. A known selected GitHub repository uses its canonical URL. |
 
 **Structured output:** `{ task: Task }`.
 

package/docs/workflows.md

@@ -120,6 +120,13 @@ The executor obtains the turn identity from an exact native read of its own
 linked task. `report_state` records live turn state as a paired request/result
 timeline.
 
+That timeline is also the durable GitHub-link source. A working request names a
+known selected repository with its canonical GitHub URL. A terminal result
+keeps full issue and pull-request URLs, including separate child-repository and
+workspace/root pull requests. The dashboard does not recover omitted links by
+scanning the native Codex transcript, and the executor does not guess an
+ambiguous repository.
+
 ```mermaid
 sequenceDiagram
   autonumber

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "taskchef",
-  "version": "7.14.0",
+  "version": "7.14.2",
   "description": "A non-blocking interactive dispatcher for visible Codex tasks.",
   "license": "MIT",
   "author": "Favo Yang",

package/skills/taskchef-executor/SKILL.md

@@ -35,6 +35,9 @@ Complete this lifecycle setup before substantive assignment work:
 4. Call TaskChef `report_state` with the marked task ID, self-linked thread ID,
    current turn ID, `status: working`, an omitted or null result summary, and a
    concise `requestSummary` describing this turn's assignment or follow-up.
+   When the turn targets a known GitHub repository, include its canonical
+   `https://github.com/<owner>/<repository>` URL so multi-repository projects
+   retain the selected repository instead of leaving TaskChef to guess.
 
 If the preceding TaskChef turn is still unfinished because its terminal report
 was lost, this newer valid working report atomically records that predecessor
@@ -71,6 +74,30 @@ and unnecessary personal data. Identical lifecycle retries are safe; never
 replace a same-turn report with different content or let an older turn
 overwrite newer state.
 
+### Preserve repository and delivery links
+
+Treat lifecycle summaries as the durable TaskChef timeline; the dashboard does
+not scan the full Codex transcript later. Preserve known GitHub context in that
+timeline with canonical links:
+
+- Use `https://github.com/<owner>/<repository>` when identifying the repository
+  selected for a turn. Derive it from established project or Git remote
+  evidence; never guess from a similarly named directory.
+- Use the full canonical issue or pull-request URL whenever one is known. Do
+  not shorten it to bare `#123` or `PR #123` in a multi-repository project.
+- Include every delivered pull request in the result summary. If work produces
+  both a child-repository pull request and a workspace/root pull request,
+  preserve both URLs and state which repository each belongs to.
+- Keep references specific to the text being reported. Do not copy unrelated
+  links from earlier turns merely to make them visible.
+- Never invent an issue, pull request, or repository link. If repository
+  identity is genuinely unresolved, describe that ambiguity without a link.
+
+For example, a working summary may say `Update dashboard projection in
+https://github.com/favoyang/taskchef`. A terminal summary may say `Shipped the
+child change in https://github.com/favoyang/taskchef/pull/78 and the workspace
+plan in https://github.com/favoyang/skills-workspace/pull/62`.
+
 ## Compatibility
 
 Existing delegated tasks may include the former inline ownership, linking, and

package/src/dashboard/app.js

@@ -16,7 +16,7 @@ import {
   turnPresentation,
 } from "./state.js";
 import { openTaskFromControl } from "./actions.js";
-import { githubReferenceSegments } from "./github-links.js";
+import { referenceSegments } from "./github-links.js";
 import { formatRelativeTime, RelativeTimeController, parsedTimestamp } from "./time.js";
 
 const state = {
@@ -61,28 +61,33 @@ const elements = {
 };
 let notificationDescriptionSerial = 0;
 
-function githubLink(link, { compact = false } = {}) {
+function referenceLink(link, { compact = false } = {}) {
   const anchor = document.createElement("a");
   anchor.className = compact ? "github-link github-link-compact" : "github-link";
   anchor.href = link.url;
   anchor.target = "_blank";
   anchor.rel = "noopener noreferrer";
   anchor.textContent = link.label ?? link.text;
+  const githubKind = link.type === "issue"
+    ? ", GitHub issue"
+    : link.type === "pull"
+      ? ", GitHub pull request"
+      : link.provider === "github" || link.owner ? " on GitHub" : "";
   anchor.setAttribute(
     "aria-label",
-    `${link.label ?? link.text} on GitHub (opens in a new tab)`,
+    `${link.label ?? link.text}${githubKind} (opens in a new tab)`,
   );
   anchor.addEventListener("click", (event) => event.stopPropagation());
   return anchor;
 }
 
-function appendGitHubText(container, text, task) {
-  const children = githubReferenceSegments(text, {
+function appendLinkedText(container, text, task) {
+  const children = referenceSegments(text, {
     projectRepositories: task.project?.githubRepos,
     taskRepository: task.relatedGitHubRepository,
   }).map((segment) => {
     if (segment.kind === "text") return document.createTextNode(segment.text);
-    if (segment.kind === "link") return githubLink(segment);
+    if (segment.kind === "link") return referenceLink(segment);
     const ambiguous = document.createElement("span");
     ambiguous.className = "github-reference-ambiguous";
     ambiguous.textContent = segment.text;
@@ -99,7 +104,7 @@ function relatedGitHubLinks(task, { compact = false } = {}) {
   const container = document.createElement("nav");
   container.className = `github-links${compact ? " github-links-compact" : ""}`;
   container.setAttribute("aria-label", `Related GitHub links for ${task.title}`);
-  const children = links.map((link) => githubLink(link, { compact }));
+  const children = links.map((link) => referenceLink(link, { compact }));
   if (task.relatedGitHubLinksTruncated) {
     const more = document.createElement("span");
     more.className = "github-links-more";
@@ -307,7 +312,7 @@ function turnTimeline(task) {
     requestLabel.textContent = "Request";
     const request = document.createElement("p");
     request.className = "preserve-lines";
-    appendGitHubText(
+    appendLinkedText(
       request,
       turn.requestSummary ?? "Request not recorded by this TaskChef version.",
       task,
@@ -316,7 +321,7 @@ function turnTimeline(task) {
     resultLabel.textContent = "Result";
     const result = document.createElement("p");
     result.className = "preserve-lines";
-    appendGitHubText(result, presentation.summary, task);
+    appendLinkedText(result, presentation.summary, task);
     const turnMetadata = document.createElement("p");
     turnMetadata.className = "result-history-turn";
     turnMetadata.textContent = turn.turnId
@@ -424,12 +429,12 @@ function taskCard(task) {
   requestLabel.textContent = "Request";
   const request = document.createElement("span");
   request.className = "preserve-lines";
-  request.textContent = latest.requestSummary;
+  appendLinkedText(request, latest.requestSummary, task);
   const resultLabel = document.createElement("strong");
   resultLabel.textContent = "Result";
   const result = document.createElement("span");
   result.className = "preserve-lines";
-  result.textContent = latest.resultSummary;
+  appendLinkedText(result, latest.resultSummary, task);
   summary.replaceChildren(requestLabel, request, resultLabel, result);
   const relatedLinks = relatedGitHubLinks(task, { compact: true });
   const time = timestampControl(

package/src/dashboard/github-links.js

@@ -1,5 +1,6 @@
 const GITHUB_REPOSITORY = /^[A-Za-z0-9](?:[A-Za-z0-9-]{0,38})\/[A-Za-z0-9._-]+$/;
-const REFERENCE_PATTERN = /https?:\/\/(?:www\.)?github\.com\/([A-Za-z0-9](?:[A-Za-z0-9-]{0,38}))\/([A-Za-z0-9._-]+)\/(issues|pull)\/([1-9]\d*)(?![A-Za-z0-9_])|([A-Za-z0-9](?:[A-Za-z0-9-]{0,38}))\/([A-Za-z0-9._-]+)#([1-9]\d*)(?![A-Za-z0-9_])|#([1-9]\d*)(?![A-Za-z0-9_])/gi;
+const REFERENCE_PATTERN = /https?:\/\/(?:www\.)?github\.com\/([A-Za-z0-9](?:[A-Za-z0-9-]{0,38}))\/([A-Za-z0-9._-]+)\/(issues|pull)\/([1-9]\d*)(?![A-Za-z0-9_])|([A-Za-z0-9](?:[A-Za-z0-9-]{0,38}))\/([A-Za-z0-9._-]+)#([1-9]\d*)(?![A-Za-z0-9_])|#([1-9]\d*)(?![A-Za-z0-9_])|https?:\/\/(?:www\.)?github\.com\/([A-Za-z0-9](?:[A-Za-z0-9-]{0,38}))\/([A-Za-z0-9](?:[A-Za-z0-9._-]*[A-Za-z0-9_-])?)\/?(?=$|[\s#),.;:!?'"\]}>])/gi;
+const ABSOLUTE_HTTP_PATTERN = /https?:\/\/[^\s<>"'`]+/gi;
 export const MAX_RELATED_GITHUB_LINKS = 20;
 
 function canonicalRepository(owner, repository) {
@@ -23,18 +24,46 @@ export function normalizeGitHubRepository(value) {
 
 function rawReferences(text) {
   const value = String(text ?? "");
+  const absoluteRanges = absoluteHttpCandidates(value);
   const references = [];
   for (const match of value.matchAll(REFERENCE_PATTERN)) {
     const start = match.index;
     const preceding = start > 0 ? value[start - 1] : "";
+    let explicitAbsoluteUrl = null;
+    const containingAbsoluteUrl = absoluteRanges.find((range) => (
+      start >= range.start && start < range.end
+    ));
+    if (containingAbsoluteUrl) {
+      const explicitGitHubUrl = Boolean(match[1] || match[9]);
+      let absoluteHost = null;
+      try {
+        absoluteHost = new URL(containingAbsoluteUrl.text).hostname.toLowerCase();
+      } catch {
+        // Invalid absolute URL tokens still block nested shorthand recognition.
+      }
+      const completeGitHubUrl = explicitGitHubUrl
+        && start === containingAbsoluteUrl.start
+        && (absoluteHost === "github.com" || absoluteHost === "www.github.com");
+      if (!completeGitHubUrl) continue;
+      const safeUrl = safeAbsoluteHttpUrl(containingAbsoluteUrl.text);
+      if (!safeUrl) continue;
+      explicitAbsoluteUrl = {
+        end: containingAbsoluteUrl.end,
+        suffix: containingAbsoluteUrl.text.slice(match[0].length),
+        text: containingAbsoluteUrl.text,
+      };
+    }
     if (match[8] && /[\w&#/]/.test(preceding)) continue;
-    if ((match[5] || match[1]) && /[\w/]/.test(preceding)) continue;
+    if ((match[5] || match[1] || match[9]) && /[\w/]/.test(preceding)) continue;
 
-    let owner = match[1] ?? match[5] ?? null;
-    let repository = match[2] ?? match[6] ?? null;
-    const number = match[4] ?? match[7] ?? match[8];
+    let owner = match[1] ?? match[5] ?? match[9] ?? null;
+    let repository = match[2] ?? match[6] ?? match[10] ?? null;
+    if (match[9]) repository = repository.replace(/\.git$/i, "");
+    const number = match[4] ?? match[7] ?? match[8] ?? null;
     const pathType = match[3]?.toLowerCase();
-    const type = pathType === "pull" ? "pull" : pathType === "issues" ? "issue" : "generic";
+    const type = match[9]
+      ? "repository"
+      : pathType === "pull" ? "pull" : pathType === "issues" ? "issue" : "generic";
     const explicit = Boolean(owner && repository);
     if (explicit) {
       if (!validRepositoryName(repository)) continue;
@@ -42,14 +71,15 @@ function rawReferences(text) {
       [owner, repository] = normalized.split("/");
     }
     references.push({
-      end: start + match[0].length,
+      end: explicitAbsoluteUrl?.end ?? start + match[0].length,
       explicit,
       number,
       owner,
       repository,
       start,
-      text: match[0],
+      text: explicitAbsoluteUrl?.text ?? match[0],
       type,
+      urlSuffix: explicitAbsoluteUrl?.suffix ?? "",
     });
   }
   return references;
@@ -77,18 +107,114 @@ function resolvedReference(reference, repositoryContext) {
     : repositoryContext;
   if (!repository) return null;
   const [owner, repositoryName] = repository.split("/");
+  if (reference.type === "repository") {
+    return {
+      kind: "link",
+      label: `${owner}/${repositoryName}`,
+      number: null,
+      owner,
+      provider: "github",
+      repository: repositoryName,
+      text: reference.text,
+      type: reference.type,
+      url: `https://github.com/${owner}/${repositoryName}${reference.urlSuffix}`,
+    };
+  }
   const path = reference.type === "pull" ? "pull" : "issues";
   return {
     kind: "link",
+    label: `${owner}/${repositoryName}#${reference.number}`,
     number: reference.number,
     owner,
+    provider: "github",
     repository: repositoryName,
     text: reference.text,
     type: reference.type,
-    url: `https://github.com/${owner}/${repositoryName}/${path}/${reference.number}`,
+    url: `https://github.com/${owner}/${repositoryName}/${path}/${reference.number}${reference.urlSuffix}`,
   };
 }
 
+function trimAbsoluteUrl(candidate) {
+  let trimmed = candidate;
+  let previous;
+  do {
+    previous = trimmed;
+    trimmed = trimmed.replace(/[.,;:!?]+$/u, "");
+    for (const [opening, closing] of [["(", ")"], ["[", "]"], ["{", "}"]]) {
+      while (trimmed.endsWith(closing)) {
+        const openings = [...trimmed].filter((character) => character === opening).length;
+        const closings = [...trimmed].filter((character) => character === closing).length;
+        if (closings <= openings) break;
+        trimmed = trimmed.slice(0, -1);
+      }
+    }
+  } while (trimmed !== previous);
+  return trimmed;
+}
+
+function absoluteHttpCandidates(text) {
+  const candidates = [];
+  for (const match of String(text ?? "").matchAll(ABSOLUTE_HTTP_PATTERN)) {
+    const candidate = trimAbsoluteUrl(match[0]);
+    if (!candidate) continue;
+    candidates.push({
+      end: match.index + candidate.length,
+      start: match.index,
+      text: candidate,
+    });
+  }
+  return candidates;
+}
+
+function hasDotPathSegment(candidate) {
+  const authorityAndPath = candidate.slice(candidate.indexOf("//") + 2).split(/[?#]/u, 1)[0];
+  const pathStart = authorityAndPath.indexOf("/");
+  if (pathStart < 0) return false;
+  return authorityAndPath.slice(pathStart).split("/").some((segment) => {
+    try {
+      const decoded = decodeURIComponent(segment);
+      return decoded === "." || decoded === "..";
+    } catch {
+      return true;
+    }
+  });
+}
+
+function safeAbsoluteHttpUrl(candidate) {
+  try {
+    const url = new URL(candidate);
+    if (
+      !url.hostname
+      || url.username
+      || url.password
+      || candidate.includes("\\")
+      || hasDotPathSegment(candidate)
+    ) return null;
+    return url.href;
+  } catch {
+    return null;
+  }
+}
+
+function rawAbsoluteHttpReferences(text) {
+  const references = [];
+  for (const candidate of absoluteHttpCandidates(text)) {
+    const url = safeAbsoluteHttpUrl(candidate.text);
+    if (!url) continue;
+    references.push({
+      end: candidate.end,
+      kind: "link",
+      label: candidate.text,
+      provider: "web",
+      start: candidate.start,
+      text: candidate.text,
+      type: "url",
+      url,
+    });
+  }
+  return references;
+}
+
 export function githubReferenceSegments(text, {
   projectRepositories = [],
   taskRepository = null,
@@ -120,6 +246,45 @@ export function githubReferenceSegments(text, {
   return segments;
 }
 
+export function referenceSegments(text, options = {}) {
+  const value = String(text ?? "");
+  const githubSegments = githubReferenceSegments(value, options);
+  const references = [];
+  let cursor = 0;
+  for (const segment of githubSegments) {
+    const start = value.indexOf(segment.text, cursor);
+    if (start < 0) continue;
+    if (segment.kind !== "text") {
+      references.push({ end: start + segment.text.length, segment, start });
+    }
+    cursor = start + segment.text.length;
+  }
+  const absoluteReferences = rawAbsoluteHttpReferences(value).filter((reference) => (
+    !references.some(({ end, start }) => reference.start < end && reference.end > start)
+  ));
+  if (absoluteReferences.length === 0) return githubSegments;
+
+  references.push(...absoluteReferences.map((segment) => ({
+    end: segment.end,
+    segment,
+    start: segment.start,
+  })));
+  references.sort((left, right) => left.start - right.start);
+
+  const segments = [];
+  let offset = 0;
+  for (const reference of references) {
+    if (reference.start < offset) continue;
+    if (reference.start > offset) {
+      segments.push({ kind: "text", text: value.slice(offset, reference.start) });
+    }
+    segments.push(reference.segment);
+    offset = reference.end;
+  }
+  if (offset < value.length) segments.push({ kind: "text", text: value.slice(offset) });
+  return segments;
+}
+
 function taskTexts(task) {
   const texts = [task.instruction];
   for (const turn of task.turns ?? []) {
@@ -132,15 +297,9 @@ function taskTexts(task) {
   return texts.filter((text) => typeof text === "string" && text.length > 0);
 }
 
-function relatedLinkLabel(link, includeRepository, includeOwner) {
-  const reference = link.type === "pull"
-    ? `PR #${link.number}`
-    : link.type === "issue"
-      ? `Issue #${link.number}`
-      : `#${link.number}`;
-  if (!includeRepository) return reference;
-  const repository = includeOwner ? `${link.owner}/${link.repository}` : link.repository;
-  return `${repository} ${reference}`;
+function relatedLinkLabel(link) {
+  if (link.type === "repository") return `${link.owner}/${link.repository}`;
+  return `${link.owner}/${link.repository}#${link.number}`;
 }
 
 export function taskGitHubProjection(task) {
@@ -164,7 +323,7 @@ export function taskGitHubProjection(task) {
       taskRepository,
     })) {
       if (segment.kind !== "link") continue;
-      const key = segment.url.toLowerCase();
+      const key = segment.url;
       if (seen.has(key)) continue;
       if (links.length === MAX_RELATED_GITHUB_LINKS) {
         truncated = true;
@@ -174,17 +333,9 @@ export function taskGitHubProjection(task) {
       links.push(segment);
     }
   }
-  const repositories = new Set(links.map(({ owner, repository }) => `${owner}/${repository}`));
-  const repositoryNames = new Map();
-  for (const link of links) {
-    const owners = repositoryNames.get(link.repository) ?? new Set();
-    owners.add(link.owner);
-    repositoryNames.set(link.repository, owners);
-  }
-  const includeRepository = repositories.size > 1;
   return {
     relatedGitHubLinks: links.map((link) => ({
-      label: relatedLinkLabel(link, includeRepository, repositoryNames.get(link.repository).size > 1),
+      label: relatedLinkLabel(link),
       number: link.number,
       owner: link.owner,
       repository: link.repository,

package/src/mcp.js

@@ -225,7 +225,7 @@ export function createTaskChefMcpServer({
     {
       title: "Report TaskChef state",
       description:
-        "Report this self-linked executor turn's lifecycle state. Use working before substantive work in a newly linked or follow-up turn, with summary omitted or null and a concise requestSummary. Before ending the same turn, report needs_input, completed, or failed with a concise semantic summary and requestSummary omitted. Exact retries are idempotent; stale or mismatched turns are rejected.",
+        "Report this self-linked executor turn's lifecycle state. Use working before substantive work in a newly linked or follow-up turn, with summary omitted or null and a concise requestSummary. Preserve known repository context and delivered issues or pull requests as canonical GitHub URLs, including both child-repository and workspace pull requests when applicable. Before ending the same turn, report needs_input, completed, or failed with a concise semantic summary and requestSummary omitted. Exact retries are idempotent; stale or mismatched turns are rejected.",
       inputSchema: {
         taskId: z.string().min(1),
         threadId: z.string().min(1).nullable(),