Skip to content

fix: add health checks to migration page fixture examples#80

Merged
jpr5 merged 1 commit intomainfrom
fix/migration-health-checks
Apr 6, 2026
Merged

fix: add health checks to migration page fixture examples#80
jpr5 merged 1 commit intomainfrom
fix/migration-health-checks

Conversation

@jpr5
Copy link
Copy Markdown
Contributor

@jpr5 jpr5 commented Apr 6, 2026

The Python and Kotlin fixture examples on migration pages used fragile time.sleep or no wait at all. Replaced with health check loops against /__aimock/health. Also fixed Python npx example using stderr=subprocess.PIPE which can deadlock.

- Python Docker: time.sleep(2) → health check loop on /__aimock/health
- Python npx: stderr=subprocess.PIPE → subprocess.STDOUT (deadlock fix) + health check
- Mokksy Kotlin: add health check wait after docker run -d
@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new bot commented Apr 6, 2026

Open in StackBlitz

npm i https://pkg.pr.new/CopilotKit/aimock/@copilotkit/aimock@80

commit: fdcb92a

@jpr5 jpr5 merged commit e1bcaaf into main Apr 6, 2026
10 checks passed
@jpr5 jpr5 deleted the fix/migration-health-checks branch April 6, 2026 16:34
@jpr5 jpr5 mentioned this pull request Apr 6, 2026
jpr5 added a commit that referenced this pull request Apr 6, 2026
## v1.8.0

### Features
- `requestTransform` — deterministic matching and recording (#79, based
on @iskhakovt's #63)
- Reasoning/thinking for OpenAI Chat Completions (#62 by @erezcor)
- Reasoning for Gemini, Bedrock, Bedrock Converse, Ollama (#81)
- Web search events for OpenAI Responses API (#62)

### Fixes
- Migration page health checks (#80)
- Stream collapse reasoning event handling
- GitHub URLs, OG image, drift docs reframe

### Versions bumped
- `package.json` → 1.8.0
- `plugin.json` → 1.8.0
- `marketplace.json` → ^1.8.0
- `Chart.yaml` appVersion → 1.8.0
- `aimock-pytest/_version.py` AIMOCK_VERSION → 1.8.0
- `CHANGELOG.md` — full v1.8.0 + retroactive v1.7.0 entries

2,090 tests across 59 files. Build clean.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant