docs: reconcile living docs to MIE-only deployment and post-Sprint-7 state#59
Merged
Conversation
Add JOURNAL entries for the two post-Sprint-8 efforts:
- MIE Container Manager deploy fix (v1 API migration: /api/v1 base, {data}
envelope, array-shaped create services) — PRs #55/#56, deploy green.
- CI test suite 44min -> 11m30s via fixing per-test full-population reruns
(@BeforeAll once-per-class) plus 8-way test sharding — PR #57, all 239 pass.
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
…state Bring CLAUDE.md, AGENTS.md, README, DEPLOY, the sprint index, CHANGELOG, .env.example, JOURNAL, and the demo/MCP guides into agreement with the current codebase and the single live MIE TWH deployment. - Stack facts: Next.js 16 + React 19; Spring AI OpenAI starter (gpt-5.4-nano / gpt-4o-mini); measure catalog 60 total / 49 CMS eCQM; SendGrid env var name corrected. - Deployment: MIE Create-a-Container documented as the sole live stack; the decommissioned Vercel + Fly.io stack moved to a labeled historical appendix in DEPLOY.md and bannered in DEPLOY_OS_MIEWEB.md; README surfaces, .env.example CORS origin, and guide hostnames updated. - Status: Sprint 7 closed + Sprint 8 scoped-run parity, CI test-sharding speedup, and the MIE v1 deploy migration reflected across CLAUDE.md, README, sprint index, and CHANGELOG. - DEMO_RUNBOOK refactored from hardcoded (ephemeral) UUIDs to an auth + capture procedure verified against the live API response shapes. No code changes; documentation only. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: c824b467e8
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Address Codex review on #59: the capture commands used query params the backend silently ignores. /api/runs has no measureId filter (status, scopeType, triggerType, site, from, to, limit only) and /api/cases filters by measureId, not measureName — so a demo where the first row was not the Audiogram scenario captured unrelated IDs and broke the deterministic get_run_summary / explain_outcome evidence trail. - runs: post-filter the returned JSON by measureName (Audiogram) instead of passing an unsupported measureId param. - cases: filter by the supported measureId (reusing $MEASURE_ID from step 2) instead of the unsupported measureName. - mirror both fixes in the pre-flight smoke check. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Documentation-only PR that brings the living docs into agreement with the current codebase and the single live MIE TWH deployment, removing facts that had drifted since the 2026-05-21 focus snapshot.
Stack facts corrected
Next.js 14+→ Next.js 16 + React 19 (frontend/package.json)Spring AI (Anthropic)→ Spring AI (OpenAI starter), modelsgpt-5.4-nano/gpt-4o-mini(build.gradle.kts,application.yml)WORKWELL_EMAIL_SENDGRID_API_KEYDeployment reconciled to MIE-only
DEPLOY.mdrewritten: MIE Create-a-Container is the sole current stack; all Fly.io/Vercel provisioning, rollback, and troubleshooting moved into a clearly-labeled historical appendix (env-var names preserved)DEPLOY_OS_MIEWEB.mdgets a historical/superseded banner that neutralizes its now-invalid Fly/Vercel rollback path.env.example(incl. the stale CORS origin), and the demo/MCP guide hostnames updated totwh.os.mieweb.org/twh-api.os.mieweb.orgStatus refreshed
Best-practice cleanups (the two review caveats)
{token}; list endpoints return bare arrays)Intentionally left as historical
docs/archive/**,docs/new instructions/**,docs/superpowers/**, per-sprint specs,LIVE_APP_QA_REPORT.md, andPOST_MERGE_STATUS.md(dated, already annotated).Notes
main.🤖 Generated with Claude Code