ci: improve test gates, browser auth and assembled coverage - #110
Merged
Merged
Conversation
morajabi
marked this pull request as ready for review
September 30, 2026 10:54
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
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.
CI repeated expensive Apple/Rust/protocol work while omitting browser package suites and meaningful assembled message checks. The new browser tests also exposed login credentials disappearing on reload. This PR reduces redundant validation, adds stable aggregate results and exercises built consumers against real local transport.
Validation: all 38 hosted PR checks passed for
f2194e404(tested merge793fbc6dc). Both iOS/macOS Debug and Release builds, all 12 Swift packages, CodeQL, the three new aggregates, server container/migration/HTTP/WebSocket/drain qualification, both CLI architectures, installed OpenClaw/Hermes matrices and Node 20/26 packed consumers are green. Server reports account for 441 files, 3,141 executed tests and zero skips. Retained artifacts confirm all ten assembled scenarios and four Chromium cases, with zero browser skips or retries. Local validation also passed 51 CI helper tests, 231 browser-client tests, 143 MCP tests, 27 additional browser-package tests, frozen install, builds/typechecks/lints, source contracts and actionlint 1.7.10. The scheduled published OpenClaw path was smoke-tested locally: published plugin 0.0.70 installed and registered through actual host 2026.9.7 in an isolated home.Bounds: browser APIs/OAuth introspection are controlled fixtures; Chat SDK webhook delivery is fixture-driven. Production auth/webhook behavior, real OpenClaw message handling, physical-device/native UI acceptance and deeper fault injection remain separate qualification. The browser shell has no chat composer. MCP coverage currently measures 78.46% lines and 85.42% functions; strict 95% thresholds remain unmet and unchanged, and are explicitly not enforced by CI yet. Configure the three new aggregate checks as required only after successful main runs following merge. No deployment or release is included.