@opengsd/gsd-pi 1.1.1-dev.2034b16 → 1.1.1-dev.2de7ea0

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
Files changed (142) hide show
  1. package/dist/cli.js +3 -2
  2. package/dist/help-text.js +10 -6
  3. package/dist/resources/.managed-resources-content-hash +1 -1
  4. package/dist/resources/extensions/browser-tools/engine/managed-gsd-browser.js +495 -0
  5. package/dist/resources/extensions/browser-tools/engine/selection.js +16 -0
  6. package/dist/resources/extensions/browser-tools/extension-manifest.json +2 -2
  7. package/dist/resources/extensions/browser-tools/index.js +57 -9
  8. package/dist/resources/extensions/browser-tools/package.json +5 -1
  9. package/dist/resources/extensions/gsd/auto-post-unit.js +21 -3
  10. package/dist/resources/extensions/gsd/auto-prompts.js +15 -6
  11. package/dist/resources/extensions/gsd/bootstrap/db-tools.js +2 -2
  12. package/dist/resources/extensions/gsd/browser-evidence.js +29 -2
  13. package/dist/resources/extensions/gsd/commands/handlers/ops.js +2 -2
  14. package/dist/resources/extensions/gsd/commands-handlers.js +76 -11
  15. package/dist/resources/extensions/gsd/commands-mcp-status.js +2 -1
  16. package/dist/resources/extensions/gsd/docs/preferences-reference.md +8 -0
  17. package/dist/resources/extensions/gsd/doctor-runtime-checks.js +2 -2
  18. package/dist/resources/extensions/gsd/mcp-project-config.js +9 -76
  19. package/dist/resources/extensions/gsd/post-unit-hooks.js +9 -0
  20. package/dist/resources/extensions/gsd/preferences-validation.js +39 -0
  21. package/dist/resources/extensions/gsd/prompt-loader.js +7 -0
  22. package/dist/resources/extensions/gsd/prompts/run-uat.md +40 -22
  23. package/dist/resources/extensions/gsd/prompts/validate-milestone.md +3 -3
  24. package/dist/resources/extensions/gsd/rule-registry.js +428 -52
  25. package/dist/resources/extensions/gsd/tools/validate-milestone.js +46 -16
  26. package/dist/resources/extensions/gsd/tools/workflow-tool-executors.js +29 -14
  27. package/dist/resources/extensions/gsd/verdict-parser.js +59 -15
  28. package/dist/resources/extensions/shared/gsd-browser-cli.js +145 -0
  29. package/dist/rtk.d.ts +7 -1
  30. package/dist/rtk.js +27 -11
  31. package/dist/update-check.d.ts +15 -1
  32. package/dist/update-check.js +87 -12
  33. package/dist/update-cmd.d.ts +1 -0
  34. package/dist/update-cmd.js +53 -2
  35. package/dist/web/standalone/.next/BUILD_ID +1 -1
  36. package/dist/web/standalone/.next/app-path-routes-manifest.json +6 -6
  37. package/dist/web/standalone/.next/build-manifest.json +2 -2
  38. package/dist/web/standalone/.next/prerender-manifest.json +3 -3
  39. package/dist/web/standalone/.next/server/app/_global-error.html +1 -1
  40. package/dist/web/standalone/.next/server/app/_global-error.rsc +1 -1
  41. package/dist/web/standalone/.next/server/app/_global-error.segments/_full.segment.rsc +1 -1
  42. package/dist/web/standalone/.next/server/app/_global-error.segments/_global-error/__PAGE__.segment.rsc +1 -1
  43. package/dist/web/standalone/.next/server/app/_global-error.segments/_global-error.segment.rsc +1 -1
  44. package/dist/web/standalone/.next/server/app/_global-error.segments/_head.segment.rsc +1 -1
  45. package/dist/web/standalone/.next/server/app/_global-error.segments/_index.segment.rsc +1 -1
  46. package/dist/web/standalone/.next/server/app/_global-error.segments/_tree.segment.rsc +1 -1
  47. package/dist/web/standalone/.next/server/app/_not-found.html +1 -1
  48. package/dist/web/standalone/.next/server/app/_not-found.rsc +1 -1
  49. package/dist/web/standalone/.next/server/app/_not-found.segments/_full.segment.rsc +1 -1
  50. package/dist/web/standalone/.next/server/app/_not-found.segments/_head.segment.rsc +1 -1
  51. package/dist/web/standalone/.next/server/app/_not-found.segments/_index.segment.rsc +1 -1
  52. package/dist/web/standalone/.next/server/app/_not-found.segments/_not-found/__PAGE__.segment.rsc +1 -1
  53. package/dist/web/standalone/.next/server/app/_not-found.segments/_not-found.segment.rsc +1 -1
  54. package/dist/web/standalone/.next/server/app/_not-found.segments/_tree.segment.rsc +1 -1
  55. package/dist/web/standalone/.next/server/app/api/update/route.js +1 -1
  56. package/dist/web/standalone/.next/server/app/index.html +1 -1
  57. package/dist/web/standalone/.next/server/app/index.rsc +1 -1
  58. package/dist/web/standalone/.next/server/app/index.segments/__PAGE__.segment.rsc +1 -1
  59. package/dist/web/standalone/.next/server/app/index.segments/_full.segment.rsc +1 -1
  60. package/dist/web/standalone/.next/server/app/index.segments/_head.segment.rsc +1 -1
  61. package/dist/web/standalone/.next/server/app/index.segments/_index.segment.rsc +1 -1
  62. package/dist/web/standalone/.next/server/app/index.segments/_tree.segment.rsc +1 -1
  63. package/dist/web/standalone/.next/server/app-paths-manifest.json +6 -6
  64. package/dist/web/standalone/.next/server/chunks/8357.js +1 -1
  65. package/dist/web/standalone/.next/server/middleware-build-manifest.js +1 -1
  66. package/dist/web/standalone/.next/server/pages/404.html +1 -1
  67. package/dist/web/standalone/.next/server/pages/500.html +1 -1
  68. package/dist/web/standalone/.next/server/server-reference-manifest.json +1 -1
  69. package/package.json +3 -2
  70. package/packages/cloud-mcp-gateway/package.json +2 -2
  71. package/packages/contracts/package.json +1 -1
  72. package/packages/daemon/package.json +4 -4
  73. package/packages/gsd-agent-core/dist/session/agent-session-compaction.d.ts +2 -0
  74. package/packages/gsd-agent-core/dist/session/agent-session-compaction.d.ts.map +1 -1
  75. package/packages/gsd-agent-core/dist/session/agent-session-compaction.js +8 -2
  76. package/packages/gsd-agent-core/dist/session/agent-session-compaction.js.map +1 -1
  77. package/packages/gsd-agent-core/package.json +5 -5
  78. package/packages/gsd-agent-modes/package.json +7 -7
  79. package/packages/mcp-server/dist/remote-questions.d.ts.map +1 -1
  80. package/packages/mcp-server/dist/remote-questions.js +23 -9
  81. package/packages/mcp-server/dist/remote-questions.js.map +1 -1
  82. package/packages/mcp-server/dist/workflow-tools.js +1 -1
  83. package/packages/mcp-server/dist/workflow-tools.js.map +1 -1
  84. package/packages/mcp-server/package.json +3 -3
  85. package/packages/native/package.json +1 -1
  86. package/packages/pi-agent-core/package.json +1 -1
  87. package/packages/pi-ai/dist/models.generated.d.ts +17 -0
  88. package/packages/pi-ai/dist/models.generated.d.ts.map +1 -1
  89. package/packages/pi-ai/dist/models.generated.js +19 -2
  90. package/packages/pi-ai/dist/models.generated.js.map +1 -1
  91. package/packages/pi-ai/package.json +1 -1
  92. package/packages/pi-coding-agent/package.json +7 -7
  93. package/packages/pi-tui/package.json +1 -1
  94. package/packages/rpc-client/package.json +2 -2
  95. package/pkg/package.json +1 -1
  96. package/src/resources/extensions/browser-tools/engine/managed-gsd-browser.ts +579 -0
  97. package/src/resources/extensions/browser-tools/engine/selection.ts +19 -0
  98. package/src/resources/extensions/browser-tools/extension-manifest.json +2 -2
  99. package/src/resources/extensions/browser-tools/index.ts +60 -9
  100. package/src/resources/extensions/browser-tools/package.json +5 -1
  101. package/src/resources/extensions/browser-tools/tests/browser-engine-selection.test.mjs +35 -0
  102. package/src/resources/extensions/browser-tools/tests/managed-gsd-browser-tools.test.mjs +33 -0
  103. package/src/resources/extensions/gsd/auto-post-unit.ts +28 -2
  104. package/src/resources/extensions/gsd/auto-prompts.ts +16 -6
  105. package/src/resources/extensions/gsd/bootstrap/db-tools.ts +2 -2
  106. package/src/resources/extensions/gsd/browser-evidence.ts +26 -2
  107. package/src/resources/extensions/gsd/commands/handlers/ops.ts +2 -2
  108. package/src/resources/extensions/gsd/commands-handlers.ts +76 -11
  109. package/src/resources/extensions/gsd/commands-mcp-status.ts +2 -1
  110. package/src/resources/extensions/gsd/docs/preferences-reference.md +8 -0
  111. package/src/resources/extensions/gsd/doctor-runtime-checks.ts +2 -2
  112. package/src/resources/extensions/gsd/mcp-project-config.ts +13 -78
  113. package/src/resources/extensions/gsd/post-unit-hooks.ts +14 -1
  114. package/src/resources/extensions/gsd/preferences-validation.ts +36 -0
  115. package/src/resources/extensions/gsd/prompt-loader.ts +8 -0
  116. package/src/resources/extensions/gsd/prompts/run-uat.md +40 -22
  117. package/src/resources/extensions/gsd/prompts/validate-milestone.md +3 -3
  118. package/src/resources/extensions/gsd/rule-registry.ts +558 -58
  119. package/src/resources/extensions/gsd/rule-types.ts +2 -0
  120. package/src/resources/extensions/gsd/tests/browser-evidence.test.ts +142 -0
  121. package/src/resources/extensions/gsd/tests/complete-milestone-excerpt.test.ts +30 -0
  122. package/src/resources/extensions/gsd/tests/doctor-runtime-checks.test.ts +27 -0
  123. package/src/resources/extensions/gsd/tests/integration/auto-recovery.test.ts +4 -4
  124. package/src/resources/extensions/gsd/tests/integration/run-uat.test.ts +66 -10
  125. package/src/resources/extensions/gsd/tests/mcp-project-config.test.ts +32 -0
  126. package/src/resources/extensions/gsd/tests/mcp-status.test.ts +2 -0
  127. package/src/resources/extensions/gsd/tests/post-unit-hooks.test.ts +157 -0
  128. package/src/resources/extensions/gsd/tests/post-unit-retry-on-orchestrator-bridge.test.ts +179 -0
  129. package/src/resources/extensions/gsd/tests/preferences.test.ts +29 -0
  130. package/src/resources/extensions/gsd/tests/prompt-contracts.test.ts +22 -1
  131. package/src/resources/extensions/gsd/tests/prompt-loader-extension-dir.test.ts +14 -0
  132. package/src/resources/extensions/gsd/tests/rule-registry.test.ts +75 -0
  133. package/src/resources/extensions/gsd/tests/validate-milestone-prompt-verification-classes.test.ts +6 -3
  134. package/src/resources/extensions/gsd/tests/validate-milestone-write-order.test.ts +133 -0
  135. package/src/resources/extensions/gsd/tests/workflow-tool-executors.test.ts +74 -0
  136. package/src/resources/extensions/gsd/tools/validate-milestone.ts +46 -15
  137. package/src/resources/extensions/gsd/tools/workflow-tool-executors.ts +31 -14
  138. package/src/resources/extensions/gsd/types.ts +63 -0
  139. package/src/resources/extensions/gsd/verdict-parser.ts +54 -13
  140. package/src/resources/extensions/shared/gsd-browser-cli.ts +172 -0
  141. /package/dist/web/standalone/.next/static/{StOMnvtgGiBHrBOZJZ1Gr → JdwzU6IGLVBZPf84PIaJQ}/_buildManifest.js +0 -0
  142. /package/dist/web/standalone/.next/static/{StOMnvtgGiBHrBOZJZ1Gr → JdwzU6IGLVBZPf84PIaJQ}/_ssgManifest.js +0 -0
@@ -44,19 +44,12 @@ function getRequiredVerificationClasses(milestoneId) {
44
44
  required.push("UAT");
45
45
  return required;
46
46
  }
47
- async function collectPersistedBrowserEvidence(basePath, milestoneId) {
48
- const chunks = [];
49
- for (const slice of getMilestoneSlices(milestoneId)) {
50
- const artifactPath = `milestones/${milestoneId}/slices/${slice.id}/${slice.id}-ASSESSMENT.md`;
51
- const artifact = getArtifact(artifactPath);
52
- if (artifact?.full_content)
53
- chunks.push(artifact.full_content);
54
- const assessmentPath = resolveSliceFile(basePath, milestoneId, slice.id, "ASSESSMENT");
55
- const assessmentContent = assessmentPath ? await loadFile(assessmentPath) : null;
56
- if (assessmentContent)
57
- chunks.push(assessmentContent);
58
- }
59
- return chunks.join("\n\n");
47
+ function hasRuntimeExecutableUatEvidenceText(text) {
48
+ if (!/\buatType:\s*runtime-executable\b/i.test(text))
49
+ return false;
50
+ if (!/\bverdict:\s*PASS\b/i.test(text))
51
+ return false;
52
+ return /^\|\s*[^|\n]+\s*\|\s*runtime\s*\|\s*PASS\s*\|[^|\n]*\bgsd_uat_exec\b/mi.test(text);
60
53
  }
61
54
  async function browserEvidenceGateRequiresAttention(params, basePath) {
62
55
  if (params.verdict !== "pass")
@@ -77,7 +70,36 @@ async function browserEvidenceGateRequiresAttention(params, basePath) {
77
70
  ]);
78
71
  if (!hasBrowserRequiredText(requirementText))
79
72
  return false;
80
- const persistedEvidence = await collectPersistedBrowserEvidence(basePath, params.milestoneId);
73
+ // Collect per-slice evidence so the runtime bypass is checked independently
74
+ // for each slice. Concatenating all slices before checking would allow runtime
75
+ // evidence from one slice to cover another slice's browser requirements.
76
+ const sliceEvidencePairs = [];
77
+ for (const slice of slices) {
78
+ const chunks = [];
79
+ const artifactPath = `milestones/${params.milestoneId}/slices/${slice.id}/${slice.id}-ASSESSMENT.md`;
80
+ const artifact = getArtifact(artifactPath);
81
+ if (artifact?.full_content)
82
+ chunks.push(artifact.full_content);
83
+ const assessmentPath = resolveSliceFile(basePath, params.milestoneId, slice.id, "ASSESSMENT");
84
+ const assessmentContent = assessmentPath ? await loadFile(assessmentPath) : null;
85
+ if (assessmentContent)
86
+ chunks.push(assessmentContent);
87
+ sliceEvidencePairs.push({
88
+ sliceRequirementText: compactTextParts([slice.demo, slice.goal, slice.success_criteria]),
89
+ evidenceText: chunks.join("\n\n"),
90
+ });
91
+ }
92
+ const persistedEvidence = sliceEvidencePairs.map((s) => s.evidenceText).join("\n\n");
93
+ // Runtime bypass: each slice whose own requirement text has browser-observable
94
+ // criteria must have its own runtime-executable UAT evidence. When no individual
95
+ // slice has slice-level browser requirements (e.g., they come from milestone-level
96
+ // fields only), fall back to checking whether any slice has runtime evidence.
97
+ const browserRequiringSlices = sliceEvidencePairs.filter((s) => hasBrowserRequiredText(s.sliceRequirementText));
98
+ const runtimeBypasses = browserRequiringSlices.length > 0
99
+ ? browserRequiringSlices.every((s) => hasRuntimeExecutableUatEvidenceText(s.evidenceText))
100
+ : sliceEvidencePairs.some((s) => hasRuntimeExecutableUatEvidenceText(s.evidenceText));
101
+ if (runtimeBypasses)
102
+ return false;
81
103
  const validationEvidence = compactTextParts([
82
104
  params.successCriteriaChecklist,
83
105
  params.verificationClasses,
@@ -138,12 +160,20 @@ export async function handleValidateMilestone(params, basePath, opts) {
138
160
  const requiredClasses = getRequiredVerificationClasses(params.milestoneId);
139
161
  if (requiredClasses.length > 0) {
140
162
  const verificationClasses = params.verificationClasses ?? "";
141
- const missingClass = requiredClasses.find((className) => !new RegExp(`\\b${className}\\b`, "i").test(verificationClasses));
142
- if (missingClass) {
163
+ const missingClasses = requiredClasses.filter((className) => !new RegExp(`\\b${className}\\b`, "i").test(verificationClasses));
164
+ if (missingClasses.length === 1) {
165
+ const missingClass = missingClasses[0];
143
166
  return {
144
167
  error: `verificationClasses must include canonical row "${missingClass}" because this milestone planned ${missingClass.toLowerCase()} verification`,
145
168
  };
146
169
  }
170
+ if (missingClasses.length > 1) {
171
+ const quotedClasses = missingClasses.map((className) => `"${className}"`).join(", ");
172
+ const plannedClasses = missingClasses.map((className) => className.toLowerCase()).join(", ");
173
+ return {
174
+ error: `verificationClasses must include canonical rows ${quotedClasses} because this milestone planned ${plannedClasses} verification`,
175
+ };
176
+ }
147
177
  }
148
178
  const artifactBasePath = resolveCanonicalMilestoneRoot(basePath, params.milestoneId);
149
179
  const shouldApplyBrowserEvidenceGate = !opts?.skipBrowserEvidenceGate &&
@@ -954,6 +954,9 @@ function validateUatChecks(basePath, params) {
954
954
  function validateUatMode(params) {
955
955
  const modes = new Set(params.checks.map((check) => check.mode));
956
956
  const hasHuman = params.checks.some((check) => check.result === "NEEDS-HUMAN");
957
+ if (params.uatType === "artifact-driven" && hasHuman && params.verdict === "PASS") {
958
+ return "artifact-driven UAT cannot PASS with human-only checks";
959
+ }
957
960
  if (hasHuman &&
958
961
  params.verdict === "PASS" &&
959
962
  !["human-experience", "mixed", "live-runtime"].includes(params.uatType) &&
@@ -969,11 +972,11 @@ function validateUatMode(params) {
969
972
  if (params.uatType === "live-runtime" && !modes.has("runtime") && !modes.has("browser")) {
970
973
  return "live-runtime UAT requires runtime or browser evidence";
971
974
  }
972
- if (params.uatType === "artifact-driven" && hasHuman && params.verdict === "PASS") {
973
- return "artifact-driven UAT cannot PASS with human-only checks";
974
- }
975
975
  return null;
976
976
  }
977
+ function quoteToolNames(toolNames) {
978
+ return toolNames.map((toolName) => `"${toolName}"`).join(", ");
979
+ }
977
980
  function validateCanonicalPresentation(params) {
978
981
  const aliasHints = {
979
982
  gsd_save_summary: "gsd_summary_save",
@@ -981,34 +984,46 @@ function validateCanonicalPresentation(params) {
981
984
  gsd_complete_slice: "gsd_slice_complete",
982
985
  gsd_milestone_complete: "gsd_complete_milestone",
983
986
  };
987
+ const errors = [];
984
988
  for (const toolName of params.presentation.presentedTools) {
985
989
  const baseName = parseMcpToolName(toolName)?.tool ?? toolName;
986
990
  const canonical = aliasHints[baseName];
987
991
  if (canonical)
988
- return `presentation tool "${toolName}" uses an alias; use canonical "${canonical}"`;
992
+ errors.push(`presentation tool "${toolName}" uses an alias; use canonical "${canonical}"`);
989
993
  }
990
994
  const presentedCanonical = new Set(params.presentation.presentedTools.map((toolName) => canonicalWorkflowToolName(parseMcpToolName(toolName)?.tool ?? toolName)));
991
- for (const requiredTool of RUN_UAT_WORKFLOW_TOOL_NAMES) {
992
- if (!presentedCanonical.has(requiredTool)) {
993
- return `presentation is missing required UAT tool "${requiredTool}"`;
994
- }
995
+ const missingRequiredTools = RUN_UAT_WORKFLOW_TOOL_NAMES.filter((requiredTool) => !presentedCanonical.has(requiredTool));
996
+ if (missingRequiredTools.length === 1) {
997
+ errors.push(`presentation is missing required UAT tool "${missingRequiredTools[0]}"`);
998
+ }
999
+ else if (missingRequiredTools.length > 1) {
1000
+ errors.push(`presentation is missing required UAT tools ${quoteToolNames(missingRequiredTools)}`);
995
1001
  }
996
1002
  const forbiddenCanonical = new Set(RUN_UAT_FORBIDDEN_TOOL_NAMES
997
1003
  .filter((toolName) => !toolName.includes("*"))
998
1004
  .map((toolName) => canonicalWorkflowToolName(parseMcpToolName(toolName)?.tool ?? toolName)));
1005
+ const forbiddenPresentedTools = [];
999
1006
  for (const toolName of params.presentation.presentedTools) {
1000
1007
  const canonical = canonicalWorkflowToolName(parseMcpToolName(toolName)?.tool ?? toolName);
1001
1008
  if (toolName === "mcp__gsd-workflow__*" || forbiddenCanonical.has(canonical)) {
1002
- return `presentation includes forbidden run-uat tool "${toolName}"`;
1009
+ forbiddenPresentedTools.push(toolName);
1003
1010
  }
1004
1011
  }
1012
+ if (forbiddenPresentedTools.length === 1) {
1013
+ errors.push(`presentation includes forbidden run-uat tool "${forbiddenPresentedTools[0]}"`);
1014
+ }
1015
+ else if (forbiddenPresentedTools.length > 1) {
1016
+ errors.push(`presentation includes forbidden run-uat tools ${quoteToolNames(forbiddenPresentedTools)}`);
1017
+ }
1005
1018
  const blockedCanonical = new Set(params.presentation.blockedTools.map((entry) => canonicalWorkflowToolName(parseMcpToolName(entry.name)?.tool ?? entry.name)));
1006
- for (const blockedTool of ["gsd_exec", "gsd_summary_save", "gsd_save_gate_result"]) {
1007
- if (!blockedCanonical.has(blockedTool)) {
1008
- return `presentation must record "${blockedTool}" as blocked during run-uat`;
1009
- }
1019
+ const missingBlockedTools = ["gsd_exec", "gsd_summary_save", "gsd_save_gate_result"].filter((blockedTool) => !blockedCanonical.has(blockedTool));
1020
+ if (missingBlockedTools.length === 1) {
1021
+ errors.push(`presentation must record "${missingBlockedTools[0]}" as blocked during run-uat`);
1010
1022
  }
1011
- return null;
1023
+ else if (missingBlockedTools.length > 1) {
1024
+ errors.push(`presentation must record ${quoteToolNames(missingBlockedTools)} as blocked during run-uat`);
1025
+ }
1026
+ return errors.length > 0 ? errors.join("; ") : null;
1012
1027
  }
1013
1028
  function nextUatAttempt(basePath, milestoneId, sliceId) {
1014
1029
  const contract = resolveGsdPathContract(basePath);
@@ -5,7 +5,62 @@
5
5
  * (e.g. `passed` → `pass`) are applied consistently across the codebase.
6
6
  */
7
7
  import { extractUatType } from "./files.js";
8
+ import { splitFrontmatter, parseFrontmatterMap } from "../shared/frontmatter.js";
9
+ import { parse as parseYaml } from "yaml";
10
+ function normalizeVerdict(value) {
11
+ if (typeof value !== "string")
12
+ return undefined;
13
+ let verdict = value.trim().toLowerCase();
14
+ if (!verdict)
15
+ return undefined;
16
+ if (verdict === "passed")
17
+ verdict = "pass";
18
+ return verdict;
19
+ }
20
+ function getCaseInsensitive(obj, key) {
21
+ const lowerKey = key.toLowerCase();
22
+ for (const [candidate, value] of Object.entries(obj)) {
23
+ if (candidate.toLowerCase() === lowerKey)
24
+ return value;
25
+ }
26
+ return undefined;
27
+ }
8
28
  // ── Verdict extraction ──────────────────────────────────────────────────
29
+ /**
30
+ * Extract and normalize the frontmatter `verdict` value.
31
+ *
32
+ * Supports both top-level `verdict` and the hook outcome shape
33
+ * `outcome.verdict`. Returns `undefined` when frontmatter is absent or has no
34
+ * verdict field.
35
+ */
36
+ export function extractFrontmatterVerdict(content) {
37
+ const [frontmatterLines] = splitFrontmatter(content);
38
+ if (!frontmatterLines)
39
+ return undefined;
40
+ try {
41
+ const parsed = parseYaml(frontmatterLines.join("\n"));
42
+ if (parsed && typeof parsed === "object") {
43
+ const root = parsed;
44
+ const topLevel = normalizeVerdict(getCaseInsensitive(root, "verdict"));
45
+ if (topLevel)
46
+ return topLevel;
47
+ const outcome = getCaseInsensitive(root, "outcome");
48
+ if (outcome && typeof outcome === "object") {
49
+ const nested = normalizeVerdict(getCaseInsensitive(outcome, "verdict"));
50
+ if (nested)
51
+ return nested;
52
+ }
53
+ }
54
+ }
55
+ catch {
56
+ // Fall through to the permissive parser used by legacy frontmatter paths.
57
+ }
58
+ const frontmatter = parseFrontmatterMap(frontmatterLines);
59
+ const topLevel = normalizeVerdict(getCaseInsensitive(frontmatter, "verdict"));
60
+ if (topLevel)
61
+ return topLevel;
62
+ return undefined;
63
+ }
9
64
  /**
10
65
  * Extract and normalize the `verdict` value from YAML frontmatter.
11
66
  *
@@ -17,25 +72,14 @@ import { extractUatType } from "./files.js";
17
72
  */
18
73
  export function extractVerdict(content) {
19
74
  // Primary: YAML frontmatter verdict (canonical format)
20
- const fmMatch = content.match(/^---\n([\s\S]*?)\n---/);
21
- if (fmMatch) {
22
- const verdictMatch = fmMatch[1].match(/verdict:\s*([\w-]+)/i);
23
- if (verdictMatch) {
24
- let v = verdictMatch[1].toLowerCase();
25
- if (v === "passed")
26
- v = "pass";
27
- return v;
28
- }
29
- return undefined;
30
- }
75
+ const [frontmatterLines] = splitFrontmatter(content);
76
+ if (frontmatterLines)
77
+ return extractFrontmatterVerdict(content);
31
78
  // Fallback: detect verdict in markdown body (LLM manual writes, #2960).
32
79
  // Matches patterns like: **Verdict:** PASS, **Verdict:** ✅ PASS, **Verdict** needs-remediation
33
80
  const bodyMatch = content.match(/\*\*Verdict:?\*\*\s*(?:✅\s*)?(\w[\w-]*)/i);
34
81
  if (bodyMatch) {
35
- let v = bodyMatch[1].toLowerCase();
36
- if (v === "passed")
37
- v = "pass";
38
- return v;
82
+ return normalizeVerdict(bodyMatch[1]);
39
83
  }
40
84
  return undefined;
41
85
  }
@@ -0,0 +1,145 @@
1
+ import { createHash } from "node:crypto";
2
+ import { execFileSync } from "node:child_process";
3
+ import { existsSync, readFileSync } from "node:fs";
4
+ import { createRequire } from "node:module";
5
+ import { basename, resolve } from "node:path";
6
+ import { fileURLToPath } from "node:url";
7
+ export const GSD_BROWSER_MCP_SERVER_NAME = "gsd-browser";
8
+ function parseJsonEnv(env, name) {
9
+ const raw = env[name];
10
+ if (!raw)
11
+ return undefined;
12
+ try {
13
+ return JSON.parse(raw);
14
+ }
15
+ catch {
16
+ throw new Error(`Invalid JSON in ${name}`);
17
+ }
18
+ }
19
+ function sanitizeSessionSegment(value) {
20
+ return value
21
+ .replace(/[^a-zA-Z0-9._-]+/g, "-")
22
+ .replace(/^-+|-+$/g, "")
23
+ .slice(0, 40);
24
+ }
25
+ function compareSemverLocal(a, b) {
26
+ const left = a.split(".").map(Number);
27
+ const right = b.split(".").map(Number);
28
+ for (let index = 0; index < Math.max(left.length, right.length); index++) {
29
+ const leftValue = left[index] || 0;
30
+ const rightValue = right[index] || 0;
31
+ if (leftValue > rightValue)
32
+ return 1;
33
+ if (leftValue < rightValue)
34
+ return -1;
35
+ }
36
+ return 0;
37
+ }
38
+ function parseGsdBrowserVersion(output) {
39
+ return output.match(/\b(\d+\.\d+\.\d+)\b/)?.[1] ?? null;
40
+ }
41
+ function resolveBundledGsdBrowserPackageVersion() {
42
+ try {
43
+ const requireFromHere = createRequire(import.meta.url);
44
+ const packageJsonPath = requireFromHere.resolve("@opengsd/gsd-browser/package.json");
45
+ const pkg = JSON.parse(readFileSync(packageJsonPath, "utf-8"));
46
+ return typeof pkg.version === "string" ? parseGsdBrowserVersion(pkg.version) : null;
47
+ }
48
+ catch {
49
+ return null;
50
+ }
51
+ }
52
+ function resolvePathGsdBrowserVersion(env) {
53
+ const explicit = env.GSD_BROWSER_PATH_VERSION?.trim();
54
+ if (explicit)
55
+ return parseGsdBrowserVersion(explicit);
56
+ try {
57
+ return parseGsdBrowserVersion(execFileSync("gsd-browser", ["--version"], {
58
+ encoding: "utf-8",
59
+ env,
60
+ stdio: ["ignore", "pipe", "ignore"],
61
+ timeout: 2000,
62
+ }));
63
+ }
64
+ catch {
65
+ return null;
66
+ }
67
+ }
68
+ function shouldPreferPathGsdBrowser(env) {
69
+ const pathVersion = resolvePathGsdBrowserVersion(env);
70
+ if (!pathVersion)
71
+ return false;
72
+ const bundledVersion = resolveBundledGsdBrowserPackageVersion();
73
+ return !bundledVersion || compareSemverLocal(pathVersion, bundledVersion) > 0;
74
+ }
75
+ export function resolveBundledGsdBrowserCliPath(env = process.env) {
76
+ const explicit = env.GSD_BROWSER_CLI_PATH?.trim() || env.GSD_BROWSER_BIN_PATH?.trim();
77
+ if (explicit)
78
+ return explicit;
79
+ try {
80
+ const requireFromHere = createRequire(import.meta.url);
81
+ const packageJsonPath = requireFromHere.resolve("@opengsd/gsd-browser/package.json");
82
+ const candidate = resolve(packageJsonPath, "..", "bin", "gsd-browser");
83
+ if (existsSync(candidate))
84
+ return candidate;
85
+ }
86
+ catch {
87
+ // Fall through to path candidates for source/dist layouts.
88
+ }
89
+ const candidates = [
90
+ resolve(fileURLToPath(new URL("../../../../node_modules/@opengsd/gsd-browser/bin/gsd-browser", import.meta.url))),
91
+ resolve(fileURLToPath(new URL("../../../../node_modules/.bin/gsd-browser", import.meta.url))),
92
+ ];
93
+ for (const candidate of candidates) {
94
+ if (existsSync(candidate))
95
+ return candidate;
96
+ }
97
+ return null;
98
+ }
99
+ export function buildGsdBrowserSessionName(projectRoot, suffix) {
100
+ const resolvedProjectRoot = resolve(projectRoot);
101
+ const base = sanitizeSessionSegment(basename(resolvedProjectRoot)) || "project";
102
+ const hash = createHash("sha1").update(resolvedProjectRoot).digest("hex").slice(0, 8);
103
+ const cleanSuffix = suffix ? sanitizeSessionSegment(suffix) : "";
104
+ return cleanSuffix ? `gsd-${base}-${hash}-${cleanSuffix}` : `gsd-${base}-${hash}`;
105
+ }
106
+ export function resolveGsdBrowserMcpLaunchConfig(projectRoot, env = process.env, options = {}) {
107
+ const resolvedProjectRoot = resolve(projectRoot);
108
+ const serverName = env.GSD_BROWSER_MCP_NAME?.trim() || GSD_BROWSER_MCP_SERVER_NAME;
109
+ const explicitArgs = parseJsonEnv(env, "GSD_BROWSER_MCP_ARGS");
110
+ const explicitEnv = parseJsonEnv(env, "GSD_BROWSER_MCP_ENV");
111
+ const explicitCommand = env.GSD_BROWSER_MCP_COMMAND?.trim();
112
+ const explicitCliPath = env.GSD_BROWSER_CLI_PATH?.trim() || env.GSD_BROWSER_BIN_PATH?.trim();
113
+ const preferPathCli = !explicitCommand && !explicitCliPath && shouldPreferPathGsdBrowser(env);
114
+ const bundledCliPath = !explicitCommand && !explicitCliPath && !preferPathCli
115
+ ? resolveBundledGsdBrowserCliPath(env)
116
+ : null;
117
+ const sessionName = options.sessionName?.trim() || buildGsdBrowserSessionName(resolvedProjectRoot, options.sessionSuffix);
118
+ const command = explicitCommand
119
+ || explicitCliPath
120
+ || (preferPathCli ? "gsd-browser" : undefined)
121
+ || (bundledCliPath ? process.execPath : undefined)
122
+ || "gsd-browser";
123
+ const args = Array.isArray(explicitArgs) && explicitArgs.length > 0
124
+ ? explicitArgs.map(String)
125
+ : [
126
+ ...(bundledCliPath ? [bundledCliPath] : []),
127
+ "mcp",
128
+ "--session",
129
+ sessionName,
130
+ "--identity-scope",
131
+ "project",
132
+ "--identity-project",
133
+ resolvedProjectRoot,
134
+ ];
135
+ const cwd = env.GSD_BROWSER_MCP_CWD?.trim() || resolvedProjectRoot;
136
+ return {
137
+ serverName,
138
+ command,
139
+ args,
140
+ cwd,
141
+ ...(explicitEnv ? { env: explicitEnv } : {}),
142
+ projectRoot: resolvedProjectRoot,
143
+ sessionName,
144
+ };
145
+ }
package/dist/rtk.d.ts CHANGED
@@ -40,6 +40,12 @@ export interface ValidateRtkBinaryOptions {
40
40
  spawnSyncImpl?: typeof spawnSync;
41
41
  env?: NodeJS.ProcessEnv;
42
42
  }
43
- export declare function validateRtkBinary(binaryPath: string, options?: ValidateRtkBinaryOptions): boolean;
43
+ export type ValidateRtkBinaryResult = {
44
+ valid: true;
45
+ } | {
46
+ valid: false;
47
+ error: string;
48
+ };
49
+ export declare function validateRtkBinary(binaryPath: string, options?: ValidateRtkBinaryOptions): ValidateRtkBinaryResult;
44
50
  export declare function ensureRtkAvailable(options?: EnsureRtkOptions): Promise<EnsureRtkResult>;
45
51
  export declare function bootstrapRtk(options?: EnsureRtkOptions): Promise<EnsureRtkResult>;
package/dist/rtk.js CHANGED
@@ -162,19 +162,28 @@ export function rewriteCommandWithRtk(command, options = {}) {
162
162
  const rewritten = (result.stdout ?? "").trimEnd();
163
163
  return rewritten || command;
164
164
  }
165
+ function trimSpawnOutput(output) {
166
+ return output?.toString().trim() ?? "";
167
+ }
165
168
  export function validateRtkBinary(binaryPath, options = {}) {
166
169
  const run = options.spawnSyncImpl ?? spawnSync;
167
170
  const result = run(binaryPath, ["rewrite", "git status"], {
168
171
  encoding: "utf-8",
169
172
  env: buildRtkEnv(options.env ?? process.env),
170
- stdio: ["ignore", "pipe", "ignore"],
173
+ stdio: ["ignore", "pipe", "pipe"],
171
174
  timeout: RTK_REWRITE_TIMEOUT_MS,
172
175
  });
173
176
  if (result.error)
174
- return false;
175
- if (result.status !== 0)
176
- return false;
177
- return (result.stdout ?? "").trim() === "rtk git status";
177
+ return { valid: false, error: result.error.message };
178
+ if (result.status !== 0) {
179
+ const stderr = trimSpawnOutput(result.stderr);
180
+ return { valid: false, error: stderr || `exit code ${result.status ?? "unknown"}` };
181
+ }
182
+ const stdout = trimSpawnOutput(result.stdout);
183
+ if (stdout !== "rtk git status") {
184
+ return { valid: false, error: stdout ? `unexpected output: ${stdout}` : "unexpected empty output" };
185
+ }
186
+ return { valid: true };
178
187
  }
179
188
  export async function ensureRtkAvailable(options = {}) {
180
189
  const env = options.env ?? process.env;
@@ -190,12 +199,18 @@ export async function ensureRtkAvailable(options = {}) {
190
199
  }
191
200
  const targetDir = options.targetDir ?? getManagedRtkDir(env);
192
201
  const managedPath = getManagedRtkPath(process.platform, targetDir);
193
- if (existsSync(managedPath) && validateRtkBinary(managedPath, { env })) {
194
- return { enabled: true, supported: true, available: true, source: "managed", binaryPath: managedPath };
202
+ if (existsSync(managedPath)) {
203
+ const managedValidation = validateRtkBinary(managedPath, { env });
204
+ if (managedValidation.valid) {
205
+ return { enabled: true, supported: true, available: true, source: "managed", binaryPath: managedPath };
206
+ }
195
207
  }
196
208
  const systemPath = resolveSystemRtkPath(options.pathValue ?? getPathValue(env));
197
- if (systemPath && validateRtkBinary(systemPath, { env })) {
198
- return { enabled: true, supported: true, available: true, source: "system", binaryPath: systemPath };
209
+ if (systemPath) {
210
+ const systemValidation = validateRtkBinary(systemPath, { env });
211
+ if (systemValidation.valid) {
212
+ return { enabled: true, supported: true, available: true, source: "system", binaryPath: systemPath };
213
+ }
199
214
  }
200
215
  const version = options.releaseVersion ?? RTK_VERSION;
201
216
  const assetName = resolveRtkAssetName(process.platform, osArch(), version);
@@ -241,9 +256,10 @@ export async function ensureRtkAvailable(options = {}) {
241
256
  if (process.platform !== "win32") {
242
257
  chmodSync(managedPath, 0o755);
243
258
  }
244
- if (!validateRtkBinary(managedPath, { env })) {
259
+ const downloadedValidation = validateRtkBinary(managedPath, { env });
260
+ if (!downloadedValidation.valid) {
245
261
  rmSync(managedPath, { force: true });
246
- throw new Error("downloaded RTK binary failed validation");
262
+ throw new Error(`downloaded RTK binary failed validation: ${downloadedValidation.error}`);
247
263
  }
248
264
  options.log?.(`installed RTK ${version} to ${managedPath}`);
249
265
  return { enabled: true, supported: true, available: true, source: "downloaded", binaryPath: managedPath };
@@ -1,5 +1,9 @@
1
1
  import { isPnpmInstall } from './resources/shared/package-manager-detection.js';
2
2
  export { isPnpmInstall };
3
+ export declare const GSD_PI_PACKAGE_NAME = "@opengsd/gsd-pi";
4
+ export declare const GSD_BROWSER_PACKAGE_NAME = "@opengsd/gsd-browser";
5
+ export declare const DEFAULT_REGISTRY_URL = "https://registry.npmjs.org/@opengsd%2fgsd-pi/latest";
6
+ export declare const GSD_BROWSER_REGISTRY_URL = "https://registry.npmjs.org/@opengsd%2fgsd-browser/latest";
3
7
  interface UpdateCheckCache {
4
8
  lastCheck: number;
5
9
  latestVersion: string;
@@ -13,6 +17,14 @@ export declare function readUpdateCache(cachePath?: string, packageName?: string
13
17
  export declare function writeUpdateCache(cache: Omit<UpdateCheckCache, 'packageName'> & {
14
18
  packageName?: string;
15
19
  }, cachePath?: string, packageName?: string): void;
20
+ export declare function resolveInstalledPackageVersion(packageName: string): string | null;
21
+ /**
22
+ * Resolves the gsd-browser version from PATH (via `gsd-browser --version`).
23
+ * Respects GSD_BROWSER_PATH_VERSION env override for testing.
24
+ * Returns null if gsd-browser is not on PATH or times out.
25
+ */
26
+ export declare function resolveGsdBrowserPathVersion(env?: NodeJS.ProcessEnv): string | null;
27
+ export declare function pickHigherVersion(a: string | null, b: string | null): string | null;
16
28
  export declare function fetchLatestVersionFromRegistry(registryUrl?: string, fetchTimeoutMs?: number): Promise<string | null>;
17
29
  /**
18
30
  * Detects whether the currently-running gsd binary was installed via `bun add -g`.
@@ -29,18 +41,20 @@ export declare function resolveInstallCommand(pkg: string, options?: {
29
41
  env?: NodeJS.ProcessEnv;
30
42
  }): string;
31
43
  export interface UpdateCheckOptions {
44
+ packageName?: string;
32
45
  currentVersion?: string;
33
46
  cachePath?: string;
34
47
  registryUrl?: string;
35
48
  checkIntervalMs?: number;
36
49
  fetchTimeoutMs?: number;
37
- onUpdate?: (current: string, latest: string) => void;
50
+ onUpdate?: (current: string, latest: string, packageName: string) => void;
38
51
  }
39
52
  /**
40
53
  * Non-blocking update check. Queries npm registry at most once per 24h,
41
54
  * caches the result, and prints a banner if a newer version is available.
42
55
  */
43
56
  export declare function checkForUpdates(options?: UpdateCheckOptions): Promise<void>;
57
+ export declare function checkForGsdBrowserUpdates(options?: UpdateCheckOptions): Promise<void>;
44
58
  /**
45
59
  * Interactive update prompt shown at startup when a newer version is available.
46
60
  * Fetches the latest version (with cache), then asks the user whether to