feat(cli): add clock-context diagnostic arm for issue 898 - #1901
Merged
Conversation
wordingone
marked this pull request as ready for review
August 24, 2026 15:22
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.
Ember authority binding
goal_id: EMBER-02
workstream_id: EMBER-02A
next_executed_outcome: EMBER-02 first sufficiently pretrained clean-genesis 3B Ember
PR kind
kind:engineeringLinked issue or governing contract
Part of #898 and leaves #898 open.
Exact base SHA
4ffd491fcd5b944aa7f65be76bd57be4daf1db83Exact reviewed head SHA
19d1a771a0605f4dfbf5007cb7508c82ae26f23bIntended outcome
Add an opt-in diagnostic factor that pauses only the root
ClockContextone-second repaint so governed matched arms can test whether idle render cadence
drives the issue #898 memory slope.
Why this is one coherent PR
The exact environment gate and its mounted-App regression test form one bounded
diagnostic carrier. The test proves both the unchanged default cadence and the
flagged pause against the real reconciler rather than a substituted scheduler.
Affected areas
area:cockpitAffected milestones
EMBER-02 — Three-billion-parameter foundation birthThis diagnostic affects the live milestone but grants no runtime-cure,
performance, milestone, or issue-closure credit.
Implementation summary
EMBER_DIAGNOSTIC_DISABLE_CLOCK_CONTEXT_TICK=1.ClockContextvalue while suppressing only its rootone-second interval.
fireball behavior, telemetry/activity pollers, and flag-absent behavior
unchanged.
Appobserver.Acceptance-clause mapping
advances on its existing cadence.
1suppresses the rootclock interval and preserves its initial value.
REPL liveness/fireball, and idle-spinner cadence.
not itself identify or cure retained memory.
Local reproduction commands
From
tools/ember-cli/src, run the new mounted-App test and the focused resize,REPL liveness/fireball, home-screen fireball, and idle-spinner suites. Run the
project typecheck against both this head and the exact base, then run the normal
pre-commit repository guard.
Automated tests
expected second clock value.
error set, with zero diagnostics in either changed path.
Executed evidence
No production diagnostic arm is claimed by this source PR. The P0/P1/P2
7,500-second matched-arm contract and adjudication thresholds are separately
preregistered under Leo authority; execution begins only from the unchanged
merged and installed binary.
Generated receipts and artifacts
No runtime receipt is checked into this source carrier. The later arms retain
their census, renderer, stdout/stderr, allocator, process-custody, and
self-hashed consumer evidence outside the repository under no-overwrite roots.
Known failures
The repository-wide typecheck currently reports the same 49 pre-existing
diagnostics at exact base and head. The carrier adds no changed-path diagnostic
and is diff-neutral against that baseline.
Unverified areas
slope remains unadjudicated.
Claim boundary
This is an opt-in diagnostic carrier. Default production behavior is unchanged.
It grants no memory-cause finding, cure, benchmark, throughput, runtime PASS,
milestone, or issue-closure credit. Issue #898 remains open.
Review provenance
The two-path change was developed RED-first under Leo-approved bounded design.
Leo independently executed the RED, GREEN, focused neighboring regressions,
controlled base/head typecheck comparison, exact hash review, and canonical
hook chain. Fresh exact-head review and public required checks remain binding.
Rollback or revert procedure
Revert the eventual squash merge commit. This removes the diagnostic gate and
its test; the flag-absent default path is already identical to the predecessor.
Follow-up obligations that remain
with positive effective-
--smolevidence required before any P2 exclusion.