test(validation): exercise one-shot resume fix - #2
Closed
SE87H wants to merge 8 commits into
Closed
Conversation
Owner
Author
|
Validation closed successfully. Validated code target: Validation branch delta beyond that target: two attribution-only files required by this fork's contributor-governance check. No runtime, test, dependency, workflow, or product code differed from the upstream PR head. Full CI run NousResearch#19 (
This disposable PR is now ready to be closed without merge. It provided evidence only and is not a deployment or promotion source. |
SE87H
pushed a commit
that referenced
this pull request
Aug 4, 2026
… a broken chat A completely unconfigured install previously booted into a working-looking chat (banner showed model 'unknown'), accepted a message, spun ~30s, then failed with 'Set OPENROUTER_API_KEY' — a provider the user never chose — and never offered setup. - HermesCLI.run() now probes provider readiness at startup (TTY only) and offers the shared provider picker (hermes model flow, which fronts Quick Setup / Nous Portal OAuth) when nothing is configured. Decline is respected; picker state re-syncs into the live CLI so the next turn works without a restart. - New silent probe _runtime_credentials_ready(): no printing, no state mutation; handles keyless local endpoints and callable bearer providers. - The empty-api-key error is provider-aware: names the actual resolved provider and points at 'hermes model' / 'hermes setup' instead of hardcoding OPENROUTER_API_KEY. - Banner: unconfigured installs render 'no model configured — run /model' in red instead of the silent 'unknown' model slug. Consumer-onboarding audit finding #2 (sev 5), Aug 2026.
SE87H
pushed a commit
that referenced
this pull request
Aug 4, 2026
A wedged adapter transport (network hang, dead websocket) previously blocked _check_session_stalls forever: sibling candidates in the same pass were never evaluated and the watcher stopped ticking. Wrap the send in asyncio.wait_for (15s); on timeout log a WARNING and do NOT latch, so the next tick retries. Regression uses a never-resolving fake adapter and proves the pass completes, a healthy sibling candidate is still notified in the same pass, and the watcher ticks again (sabotage-verified against the unbounded send).
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
1 participant
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.
Disposable internal validation PR for NousResearch#74397.
This branch contains the exact upstream fix head
b5123936cb13005ba446835ee46b5da5d24564f7plus one attribution-only commit required by this fork's CI governance. It exists solely to run the complete repository suite against the samemainbase used by the upstream PR.Do not merge. After CI evidence is recorded, this PR will be closed.
Validation target:
TERMINAL_CWDsynchronization after successfulchdir--no-restore-cwdpreservation