Repository navigation
fix: restore repository-wide validation and runtime compatibility - #24
Open
nicholas-ruest wants to merge 2 commits into
Open
nicholas-ruest wants to merge 2 commits into
nicholas-ruest wants to merge 2 commits into
Conversation
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.
Summary
This resumes the existing CLI-discovery candidate and restores repository-wide functional validation without weakening production behavior or gates.
claude-flowproduction dependencyfleet-benchmarkstrict typing and optional-config normalizationwebcryptoimport for Node 18/20/22 compatibilityNo tests, coverage thresholds, security policy, or production persistence behavior were weakened.
Frozen evaluation
Base:
4d58c46426fe085d25fef3fcb35fa5e2cce83b30Initial CLI candidate:
959d889b940de3e6ce6cc9e935d38f820dde149aExact candidate:
1c4f445288f360120ebae6fb00c6fbf312c16931Clean
npm cisucceeded and installed 594 packages instead of 924.npm run build:allproduced the SDK and functions bundles successfully.MetaHarness / Darwin / Flywheel
0.4.17: score 82; compile confidence 100; task coverage 100; tool safety 95; memory usefulness 53; 6/6 hard constraints0.10.3, upstreame0dfd44da72b7adfb7c57fdd42d124be58ed7086: real sandbox, seed101108, one generation/one child; baseline 0.985, child 0.985, delta 0.000; no mutation promoted0.1.12: frozen functional holdout (lint/typecheck/test/build/format) improved from 0/5 to 5/5; signed replay passed; lineageroot__verification_gen1 → root; gate fingerprintc5942e7c9bf8f28fe7af47a6ec3887d11eeb3f8c8e8601cc6a7a54969ea0b981Flywheel's bounded functional candidate passed its frozen gate, but the repository's independent 80% coverage gate and remaining production advisories are explicit merge blockers.
Exact GitHub Actions result
CI run #38139330512 evaluated exact head
1c4f445288f360120ebae6fb00c6fbf312c16931.Passed: Node 18, Node 20, Node 22, integration tests, ESLint, TypeScript, Prettier, and package build.
Failed: code coverage; aggregate CI consequently failed.
Merge status
BLOCKED — do not merge. The exact candidate is conflict-free, two commits ahead and zero behind
main, but the repository's unchanged 80% coverage requirement remains red.npm audit --omit=devalso still reports 14 advisories including 1 critical and 6 high.Rollback
Revert
1c4f445288f360120ebae6fb00c6fbf312c16931to remove the repository-wide recovery while retaining the initial CLI-discovery commit, or revert both PR commits to return tomainat4d58c46426fe085d25fef3fcb35fa5e2cce83b30.