Repository navigation
feat(nonstop_cli): harden the melos starter and enforce full coverage - #44
Merged
Merged
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
Followed the generate → improve → verify → port → regenerate workflow for the
default NonStop CLI Melos starter. Changes are limited to the mono-repo brick,
its bundled generator, template verification, and related documentation/CI.
Remote Config defaults, route guards, notification registration and startup failures.
remove payload/identity logging from default observers and network logs.
Firebase platform doubles and error/lifecycle regression cases.
architecture guidance, updated READMEs and changelog.
than silently downloading a different registry template.
Verification
Tested using Flutter 3.44.0 / Dart 3.12.0:
and parsed for valid Dart and disabled-module imports, plus bundle/hook parity.
clean formatting/analysis and successful web build.
formatting/analysis and successful web build.
formatting/analysis and successful web build.
analysis, tests, 100% coverage gate and web builds during verification.
gate fail with its missing line; removing it restores 100%. Gate unit tests also
cover cross-package/path-alias merging, exclusions and missing tests/reports.
The new CI workflow repeats fresh generation for six representative profiles;
the 512-combination check is structural, not 512 full Flutter builds.
Coverage scope and integration boundaries
100% means executable-line coverage of handwritten runtime Dart under workspace
lib/directories. Only generated.g.dart,.freezed.dartand.i69n.dartfiles are excluded; tooling, tests and native platform code are outside the metric.
This is not a claim of complete branch coverage or absence of all bugs.
Firebase credentials, live push delivery, native provider sign-in, backend
authorization/security rules, and product privacy policy remain app-specific
integration work. The starter boots before Firebase is configured; demo-only
unguarded routes are explicitly documented and must not protect real data.