Skip to content

Stabilize the operator integration test server - #355

Merged
mintuhouse merged 1 commit into
mainfrom
mintuhouse/fix-main-ci-startup
Oct 23, 2026
Merged

mintuhouse merged 1 commit into
mainfrom
mintuhouse/fix-main-ci-startup

Conversation

@mintuhouse

Copy link
Copy Markdown
Contributor
  • Stabilize the operator-authorization integration test behind repeated main CI failures by binding Vite and its requests to the same IPv4 loopback address.
  • Preserve server output and report early process exits during readiness, reusing the exit promise for cleanup.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 23, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-23T17:20:31.467128Z 6f64648 PR opened
🔒 Security Review ✅ Completed 2026-10-23T17:21:33.289484Z 6f64648 PR opened
ℹ️ 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" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

mintuhouse commented Oct 23, 2026 •

Copy link
Copy Markdown
Contributor Author

Validation results for 6f6464875c9b176b1ad6a1ab31e27f3ac606873c:

Check Result
Platform formatting, OpenAPI, lint/type checking, unused-code checks Passed
Local unit tests 538 passed
Local integration tests 138 passed, 2 intentional skips, 57.78s
Production build and binary compilation Passed
Platform browser suite 14 passed, 4.1m
Todos app browser suite 66 passed, 2.6m
Hosted CI All 10 jobs passed
Fresh-run integration verification 138 passed, 2 skipped, 61.53s
Code and security reviews Completed with no findings

The previously flaky configuration test passed on two hosted runners in 17.520s and 23.649s. Locally, the original localhost setup reproduced ECONNREFUSED even after Vite reported ready. Pinning the listener and requests to 127.0.0.1 passed the complete operator-security scenario without changing its assertions or timeouts.

The initial combined local ready invocation used VITEST_MAX_WORKERS=2, which overrides integration serialization in the installed Vitest version and caused fixture collisions. Removing that override produced the passing CI-equivalent integration result above. Cold local runs also encountered dev-server compilation timeouts before the successful runs. Future startup failures now retain Vite output and report early process exit directly.

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
www-astralbeam-ai 6f64648 Commit Preview URL

Branch Preview URL
Oct 23 2026, 05:20 PM

@mintuhouse
mintuhouse merged commit 388daac into main Oct 23, 2026
22 checks passed
@mintuhouse
mintuhouse deleted the mintuhouse/fix-main-ci-startup branch October 23, 2026 18:29
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