From 89f8de5b652ea640158da6a1cd3dd34521995fca Mon Sep 17 00:00:00 2001 From: rcourtman <8825017+rcourtman@users.noreply.github.com> Date: Fri, 23 Oct 2026 00:57:17 +0100 Subject: [PATCH] Pin the MCP-first AI overview in the docs contract test The Pulse Intelligence overview contract still required docs/AI.md to say the Assistant is the in-app Pro surface beside pulse-mcp. That paragraph went with the chat window, so the test now pins what the doc says instead: Pulse has no in-app chat window, MCP clients connect through pulse-mcp, and capabilities land in the canonical manifest first. --- internal/api/contract_test.go | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/internal/api/contract_test.go b/internal/api/contract_test.go index c77f8dc20e..1e414d8230 100644 --- a/internal/api/contract_test.go +++ b/internal/api/contract_test.go @@ -20106,10 +20106,10 @@ func TestContract_PulseIntelligenceOverviewPinsPatrolPrimaryCore(t *testing.T) { "Pulse Assistant", "Pulse MCP", "contextual explanation, approval, and handoff surface for Patrol findings", - "Pulse Assistant and `pulse-mcp` are sibling surfaces over Pulse Intelligence, not competing implementations, and neither replaces the other", - "Assistant remains the in-app Pro surface", - "New operational capabilities should be added to the canonical API manifest first", - "MCP-only actions and Assistant-only copies of the same business logic are drift", + "Pulse has no in-app chat window", + "connect an MCP client such as Claude Code, Codex, or OpenCode to `pulse-mcp`", + "New operational capabilities are added to the canonical API manifest first", + "MCP-only copies of the same business logic are drift", } { if !strings.Contains(normalized, required) { t.Errorf("docs/AI.md must pin Pulse Intelligence Core Patrol-primary product framing; missing %q", required)