@oevery/rsp 3.1.0-beta.4 → 3.1.1

package/dist/{chunk-FI3SYHVH.mjs → chunk-MUNLLVVH.mjs}

@@ -29,7 +29,7 @@ import {
   resolveLanguagePolicy,
   resolveManagePolicy,
   toErrorMessage
-} from "./chunk-FJVH7ZYU.mjs";
+} from "./chunk-WJXKLLIF.mjs";
 
 // src/status/inspect.ts
 import { readFile as readFile5, stat } from "fs/promises";
@@ -666,6 +666,7 @@ function normalizeLogicalPath2(path) {
 
 // src/core/change-group.ts
 var GROUP_BRIEF_SECTIONS = getCanonicalSectionHeadings(GROUP_BRIEF_DOCUMENT_SCHEMA);
+var GROUP_REOPEN_COMPLETION_PREFIX = "Resolve reopened concern from";
 async function resolveExecutableChange(name, options = {}) {
   const ref = resolveWorkRef(name, { ...options, executable: true });
   if (ref.kind === "group-change")
@@ -711,6 +712,8 @@ async function inspectChangeGroups(options = {}) {
   const groups = [];
   const archivedSlicesByGroup = /* @__PURE__ */ new Map();
   const archivedGroups = /* @__PURE__ */ new Set();
+  const archivedGroupReopenEvidence = /* @__PURE__ */ new Map();
+  const archivedGroupBriefPaths = /* @__PURE__ */ new Map();
   const openGroupNames = new Set(workTree.briefs.map((brief) => brief.group));
   for (const path of archiveTree.files) {
     const archivePath = normalizeLogicalPath(relative2(ARCHIVES_DIR, path));
@@ -736,8 +739,16 @@ async function inspectChangeGroups(options = {}) {
           archivedSlicesByGroup.set(archiveGroup, groupSlices);
         }
       }
-      if (getDocumentTitle(parseRspDocument(content, GROUP_BRIEF_DOCUMENT_SCHEMA), "identity") === archiveGroup)
+      if (getDocumentTitle(parseRspDocument(content, GROUP_BRIEF_DOCUMENT_SCHEMA), "identity") === archiveGroup) {
         archivedGroups.add(archiveGroup);
+        const evidence = archivedGroupReopenEvidence.get(archiveGroup) ?? /* @__PURE__ */ new Set();
+        for (const item of getGroupReopenEvidence(content))
+          evidence.add(item.key);
+        archivedGroupReopenEvidence.set(archiveGroup, evidence);
+        const retainedPaths = archivedGroupBriefPaths.get(archiveGroup) ?? /* @__PURE__ */ new Set();
+        retainedPaths.add(`.rsp/archives/${archivePath}`);
+        archivedGroupBriefPaths.set(archiveGroup, retainedPaths);
+      }
     } catch (error) {
       diagnostics.push({
         severity: "error",
@@ -764,7 +775,11 @@ async function inspectChangeGroups(options = {}) {
     }
     const parsed = parseGroupBrief(brief.group, content);
     diagnostics.push(...parsed.diagnostics.map((diagnostic) => ({ ...diagnostic, path: brief.path })));
-    if (archivedGroups.has(brief.group)) {
+    if (archivedGroups.has(brief.group) && !hasExplicitGroupReopenEvidence(
+      content,
+      archivedGroupReopenEvidence.get(brief.group) ?? /* @__PURE__ */ new Set(),
+      archivedGroupBriefPaths.get(brief.group) ?? /* @__PURE__ */ new Set()
+    )) {
       diagnostics.push({
         severity: "error",
         code: "group_identity_reopened",
@@ -855,6 +870,16 @@ async function inspectChangeGroups(options = {}) {
   groups.sort((left, right) => left.name.localeCompare(right.name));
   return { groups, diagnostics };
 }
+function getGroupReopenEvidence(content) {
+  const completion = getDocumentSectionBody(parseRspDocument(content, GROUP_BRIEF_DOCUMENT_SCHEMA), "completionConditions");
+  return completion.split("\n").flatMap((line) => {
+    const match = line.trim().match(/^-\s+\[([ /xX-])\]\s+(Resolve reopened concern from `(\.rsp\/archives\/[^`]+)`: \S.*)$/);
+    return match ? [{ active: match[1] !== "-", key: match[2], sourcePath: match[3] }] : [];
+  });
+}
+function hasExplicitGroupReopenEvidence(content, retainedEvidence, retainedGroupBriefPaths) {
+  return getGroupReopenEvidence(content).some((item) => item.active && retainedGroupBriefPaths.has(item.sourcePath) && !retainedEvidence.has(item.key));
+}
 async function requireDeclaredGroupSlice(ref) {
   const briefPath = join2(dirname(ref.path), GROUP_BRIEF_FILENAME);
   let content;
@@ -975,6 +1000,69 @@ import { readFile as readFile3 } from "fs/promises";
 import { readFile as readFile2 } from "fs/promises";
 import { basename, dirname as dirname2, relative as relative3 } from "path";
 
+// src/core/issue-relationship.ts
+var ISSUE_RELATIONS = /* @__PURE__ */ new Set(["relates", "closes"]);
+var ISSUE_FIELDS = /* @__PURE__ */ new Set(["url", "relation"]);
+var IssueRelationshipError = class extends Error {
+  constructor(code, message) {
+    super(message);
+    this.code = code;
+    this.name = "IssueRelationshipError";
+  }
+  code;
+};
+function parseIssueRelationships(frontmatter) {
+  if (!frontmatter || !("issues" in frontmatter))
+    return [];
+  if (!Array.isArray(frontmatter.issues))
+    throw new IssueRelationshipError("invalid_issue_shape", 'frontmatter field "issues" must be a list');
+  const seen = /* @__PURE__ */ new Set();
+  return frontmatter.issues.map((value, index) => {
+    if (!isPlainObject(value))
+      throw new IssueRelationshipError("invalid_issue_shape", `issues[${index}] must be a mapping with url and relation`);
+    const fields = Object.keys(value);
+    const unsupported = fields.filter((field) => !ISSUE_FIELDS.has(field));
+    if (unsupported.length > 0)
+      throw new IssueRelationshipError("unsupported_issue_field", `issues[${index}] contains unsupported field(s): ${unsupported.sort().join(", ")}`);
+    if (fields.length !== 2 || !("url" in value) || !("relation" in value))
+      throw new IssueRelationshipError("invalid_issue_shape", `issues[${index}] must contain exactly url and relation`);
+    if (typeof value.url !== "string")
+      throw new IssueRelationshipError("invalid_issue_url", `issues[${index}].url must be an absolute http or https URL`);
+    if (typeof value.relation !== "string" || !ISSUE_RELATIONS.has(value.relation))
+      throw new IssueRelationshipError("unsupported_issue_relation", `issues[${index}].relation must be relates or closes`);
+    const url = normalizeIssueUrl(value.url, `issues[${index}].url`);
+    if (seen.has(url))
+      throw new IssueRelationshipError("duplicate_issue_url", `issues contains duplicate normalized URL: ${url}`);
+    seen.add(url);
+    return { url, relation: value.relation };
+  });
+}
+function createIssueRelationship(url, relation = "relates") {
+  return parseIssueRelationships({ issues: [{ url, relation }] })[0];
+}
+function normalizeIssueUrl(raw, label) {
+  if (raw.trim() !== raw || raw === "")
+    throw new IssueRelationshipError("invalid_issue_url", `${label} must be an absolute http or https URL without surrounding whitespace`);
+  let parsed;
+  try {
+    parsed = new URL(raw);
+  } catch {
+    throw new IssueRelationshipError("invalid_issue_url", `${label} must be an absolute http or https URL`);
+  }
+  if (parsed.protocol !== "http:" && parsed.protocol !== "https:")
+    throw new IssueRelationshipError("invalid_issue_url", `${label} must use http or https`);
+  if (!parsed.hostname)
+    throw new IssueRelationshipError("invalid_issue_url", `${label} must include a host`);
+  if (parsed.username || parsed.password)
+    throw new IssueRelationshipError("unsafe_issue_url", `${label} must not contain credentials`);
+  if (parsed.hash)
+    throw new IssueRelationshipError("unsafe_issue_url", `${label} must not contain a fragment`);
+  return parsed.toString();
+}
+function isPlainObject(value) {
+  return typeof value === "object" && value !== null && !Array.isArray(value);
+}
+
 // src/history/model.ts
 var HISTORY_DEFAULT_LIMIT = 20;
 var HISTORY_MAX_LIMIT = 100;
@@ -1011,6 +1099,7 @@ async function inspectArchiveHistory(options = {}) {
   }));
   const runtime = [];
   const records = [];
+  const groupBriefs = [];
   if (!tree.rootExists && tree.diagnostics.length === 0) {
     diagnostics.push({
       severity: "error",
@@ -1034,14 +1123,18 @@ async function inspectArchiveHistory(options = {}) {
       diagnostics.push(parsed.diagnostic);
     if (parsed.record)
       records.push(parsed.record);
+    if (parsed.groupBrief)
+      groupBriefs.push(parsed.groupBrief);
   }
   records.sort(compareHistoryRecords);
+  groupBriefs.sort(compareArchivedGroupBriefs);
   diagnostics.sort(compareDiagnostics);
   const diagnosticTotal = diagnostics.length;
   const boundedDiagnostics = diagnostics.slice(0, HISTORY_MAX_DIAGNOSTICS);
   return {
     rootExists: tree.rootExists,
     records,
+    groupBriefs,
     diagnostics: boundedDiagnostics,
     diagnosticSummary: {
       total: diagnosticTotal,
@@ -1082,7 +1175,15 @@ function parseArchiveEntry(sourcePath, path, archivesDir, content) {
     if (!archiveGroup || group !== archiveGroup || frontmatter.kind !== "group" || !matchesArchiveStem(archiveStem, "brief")) {
       return { diagnostic: errorDiagnostic("archive_identity_mismatch", path, `archived Change Group identity ${group} does not match its archive path`) };
     }
-    return {};
+    return { groupBrief: { date, group, path, sourcePath } };
+  }
+  let issues;
+  if ("issues" in frontmatter) {
+    try {
+      issues = parseIssueRelationships(frontmatter);
+    } catch {
+      issues = void 0;
+    }
   }
   const workRef = changeHeadings[0];
   const segments = workRef.split("/");
@@ -1107,6 +1208,7 @@ function parseArchiveEntry(sourcePath, path, archivesDir, content) {
       summary: summary.value,
       summaryTruncated: summary.truncated,
       path,
+      ...issues === void 0 ? {} : { issues },
       sourcePath,
       searchSummary: rawSummary
     }
@@ -1151,6 +1253,9 @@ function escapeRegExp(value) {
 function compareHistoryRecords(left, right) {
   return right.date.localeCompare(left.date) || left.workRef.localeCompare(right.workRef) || left.path.localeCompare(right.path);
 }
+function compareArchivedGroupBriefs(left, right) {
+  return right.date.localeCompare(left.date) || left.group.localeCompare(right.group) || left.path.localeCompare(right.path);
+}
 function compareDiagnostics(left, right) {
   return (left.path ?? "").localeCompare(right.path ?? "") || left.code.localeCompare(right.code) || left.message.localeCompare(right.message);
 }
@@ -1205,6 +1310,17 @@ function selectArchiveHistoryRecord(records, selector) {
   }
   return matches[0];
 }
+function selectArchivedGroupBrief(records, selector) {
+  const matches = "path" in selector ? records.filter((record) => record.path === selector.path) : records.filter((record) => record.group === selector.group);
+  const identity = "path" in selector ? selector.path : selector.group;
+  if (matches.length === 0)
+    throw new ArchiveHistoryError("archive_not_found", `archived Change Group not found: ${identity}`);
+  if (matches.length > 1) {
+    const candidates = matches.map((record) => record.path).sort();
+    throw new ArchiveHistoryError("archive_ambiguous", `multiple archives match Change Group ${identity}`, candidates);
+  }
+  return matches[0];
+}
 async function readArchiveHistoryDetail(record) {
   let content;
   try {
@@ -1600,6 +1716,7 @@ async function inspectProjectStatus(options = {}) {
     let done = 0;
     let total = 0;
     let progressKnown = false;
+    let issues = [];
     if (path) {
       try {
         const content = await readFile5(path, "utf-8");
@@ -1608,10 +1725,21 @@ async function inspectProjectStatus(options = {}) {
         try {
           const frontmatter = parseFrontmatter(content);
           kind = frontmatter?.kind ? String(frontmatter.kind) : "\u2014";
+          issues = parseIssueRelationships(frontmatter);
         } catch (error) {
           kind = "(invalid)";
+          const issueError = error instanceof IssueRelationshipError;
+          if (issueError) {
+            diagnostics.push({
+              severity: "error",
+              code: error.code,
+              change: name,
+              path,
+              message: error.message
+            });
+          }
           runtime.push({
-            code: "frontmatter_parse_failed",
+            code: issueError ? error.code : "frontmatter_parse_failed",
             operation: "parseFrontmatter",
             path,
             message: toErrorMessage(error)
@@ -1692,7 +1820,8 @@ async function inspectProjectStatus(options = {}) {
         ageDays,
         isFocused,
         isBlocked,
-        path: path ? normalizeLogicalPath(path) : null
+        path: path ? normalizeLogicalPath(path) : null,
+        issues
       },
       progressKnown,
       title,
@@ -1785,16 +1914,22 @@ export {
   inspectFocusTree,
   inspectArchiveTree,
   inspectWorkTree,
+  GROUP_REOPEN_COMPLETION_PREFIX,
   resolveExecutableChange,
   generateGroupBriefContent,
   inspectChangeGroups,
+  getGroupReopenEvidence,
   hasArchivedGroupBrief,
+  IssueRelationshipError,
+  parseIssueRelationships,
+  createIssueRelationship,
   HISTORY_DEFAULT_LIMIT,
   HISTORY_MAX_LIMIT,
   ArchiveHistoryError,
   inspectArchiveHistory,
   queryArchiveHistory,
   selectArchiveHistoryRecord,
+  selectArchivedGroupBrief,
   readArchiveHistoryDetail,
   historyInspectionComplete,
   inspectChangeDependencies,

package/dist/{chunk-FJVH7ZYU.mjs → chunk-WJXKLLIF.mjs}

@@ -472,10 +472,11 @@ function normalizeLogicalPath(pathValue) {
 function generateChangeContent(name, summary = "", kind, options = {}) {
   const placeholder = "<\u2026>";
   const proposalSummary = summary || placeholder;
+  const issueFrontmatter = renderIssueFrontmatter(options.issues ?? []);
   if (name === "project-setup") {
     return `---
 kind: ops
----
+${issueFrontmatter}---
 
 ${renderDocumentTitle(CHANGE_DOCUMENT_SCHEMA, "project-setup")}
 
@@ -533,11 +534,11 @@ ${changeSectionHeading("blockers")}
   }
   const frontmatterKind = kind ?? "<choose: feature | fix | refactor | docs | ops | research>";
   if (options.lite)
-    return generateLiteChangeContent(name, proposalSummary, frontmatterKind);
+    return generateLiteChangeContent(name, proposalSummary, frontmatterKind, issueFrontmatter);
   const template = getChangeTemplateByKind(kind);
   return `---
 kind: "${frontmatterKind}"
----
+${issueFrontmatter}---
 
 ${renderDocumentTitle(CHANGE_DOCUMENT_SCHEMA, name)}
 
@@ -582,11 +583,11 @@ ${changeSectionHeading("blockers")}
 - none
 `;
 }
-function generateLiteChangeContent(name, proposalSummary, frontmatterKind) {
+function generateLiteChangeContent(name, proposalSummary, frontmatterKind, issueFrontmatter) {
   const placeholder = "<\u2026>";
   return `---
 kind: "${frontmatterKind}"
----
+${issueFrontmatter}---
 
 ${renderDocumentTitle(CHANGE_DOCUMENT_SCHEMA, name)}
 
@@ -635,6 +636,14 @@ ${changeSectionHeading("blockers")}
 - none
 `;
 }
+function renderIssueFrontmatter(issues) {
+  if (issues.length === 0)
+    return "";
+  return `issues:
+${issues.map((issue) => `  - url: ${JSON.stringify(issue.url)}
+    relation: ${issue.relation}`).join("\n")}
+`;
+}
 function getChangeTemplateByKind(kind) {
   const placeholder = "<\u2026>";
   const delta = kind === "feature" || kind === "research" || kind === void 0 ? "ADDED" : "MODIFIED";
@@ -642,8 +651,7 @@ function getChangeTemplateByKind(kind) {
     why: placeholder,
     scope: placeholder,
     nonGoals: placeholder,
-    specSection: `<!-- ${placeholder} -->
-### ${delta}
+    specSection: `### ${delta}
 - Requirement: ${placeholder}
   - ${placeholder}`,
     acceptanceSection: `#### Scenario: ${placeholder}