Cutover report + harness Accept smoke fix

[Travis-Gilbert]

Summary

• MCP initialize smoke sends Accept application/json, text/event-stream.
• EXECUTE-REPORT updated with live 23/23 doctor evidence.

Test plan

• doctor.sh 23/23 on v2
• upstream-smokes 8/8 after Accept fix

Summary by CodeRabbit

• Bug Fixes

  • Improved authenticated smoke checks by requesting responses in supported JSON and Server-Sent Events formats.

• Documentation

  • Updated production cutover records with final validation results, health checks, completed items, remaining caveats, and merged changes.

[coderabbitai]

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 01de339e-17d5-489e-be4f-8adafbd59638

📥 Commits

Reviewing files that changed from the base of the PR and between 3658720 and 3587081.

📒 Files selected for processing (2)

• docs/plans/commonplace-production-cutover/EXECUTE-REPORT.md
• scripts/doctor-upstream-smokes.mjs

---

📝 Walkthrough

Walkthrough

The production cutover report now records merged status, acceptance evidence, final goal statuses, live observations, remaining caveats, and merged pull requests. The authenticated smoke harness now accepts JSON and Server-Sent Events responses.

Changes

Production cutover

Layer / File(s)	Summary
Final cutover evidence and smoke validation docs/plans/commonplace-production-cutover/EXECUTE-REPORT.md, scripts/doctor-upstream-smokes.mjs	The report records production-boundary acceptance, completed plan items, goal statuses, live checks, remaining caveats, and merged pull requests. The authenticated harness adds an Accept header for JSON and Server-Sent Events responses.

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related PRs

• Travis-Gilbert/CommonPlace#128: Related MCP and Server-Sent Events response transport handling.
• Travis-Gilbert/CommonPlace#165: Updated the cutover report and the smoke harness changed here.

Suggested reviewers: copilot

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch feat/production-cutover-1-0

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[Copilot]

Pull request overview

Updates the production cutover execution report with live evidence and adjusts the harness-authenticated upstream smoke to send an Accept header compatible with MCP/SSE negotiation.

Changes:

• Add Accept: application/json, text/event-stream to the MCP /mcp initialize smoke request.
• Refresh EXECUTE-REPORT with live 23/23 doctor results plus ordered GL3/doctor observations and current Plan-to-Correct status.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File	Description
scripts/doctor-upstream-smokes.mjs	Adds an Accept header for the harness MCP initialize probe to better match the server’s expected content negotiation.
docs/plans/commonplace-production-cutover/EXECUTE-REPORT.md	Updates the cutover execution report with live production evidence, revised verdict, and consolidated PR references.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.