docs: replace ASCII diagrams and clarify fallback config - #21594
Open
camaragon wants to merge 1 commit into
Open
docs: replace ASCII diagrams and clarify fallback config#21594camaragon wants to merge 1 commit into
camaragon wants to merge 1 commit into
Conversation
camaragon
force-pushed
the
auto/hermes-self-improvement
branch
11 times, most recently
from
May 15, 2026 17:16
5056668 to
5819a03
Compare
camaragon
force-pushed
the
auto/hermes-self-improvement
branch
12 times, most recently
from
May 22, 2026 09:02
f63dab5 to
319a506
Compare
camaragon
force-pushed
the
auto/hermes-self-improvement
branch
2 times, most recently
from
May 24, 2026 05:59
d009280 to
9f6b584
Compare
teknium1
reviewed
Jul 13, 2026
teknium1
left a comment
Contributor
There was a problem hiding this comment.
Thanks for the focused documentation cleanup. One correction is needed during salvage.
Problems
website/docs/user-guide/features/kanban.md:511changes the architecture toDB --> UI, but live updates are delivered by the FastAPI/eventsWebSocket, which tailstask_events(plugins/kanban/dashboard/plugin_api.py:10-11,:2379). The previous diagram named that WebSocket path; the Mermaid replacement drops it.
Suggested changes
- Route the live-update edge through the API/router to the UI and label it as the WebSocket tail of
task_events; keep the DB as the persistence/write target. - Preserve the legacy
fallback_modelcompatibility note when reconciling the fallback documentation (hermes_cli/fallback_config.py:51-64).
Automated hermes-sweeper review.
camaragon
force-pushed
the
auto/hermes-self-improvement
branch
from
August 7, 2026 22:42
9f6b584 to
c3a6921
Compare
Contributor
Author
|
Refreshed onto current Current scope:
Verification:
|
camaragon
force-pushed
the
auto/hermes-self-improvement
branch
from
August 18, 2026 23:36
c3a6921 to
6ac541a
Compare
Contributor
Author
|
Refreshed onto current |
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.
Why change
Replace remaining docs ASCII diagrams with supported Mermaid and clarify fallback configuration without losing legacy compatibility guidance.
Files changed
cron-script-only.md: replace the no-agent execution diagram with Mermaid.kanban.md: replace the GUI architecture diagram with Mermaid and show the actual DB persistence -> FastAPI/WebSocket tail -> React UI boundary.providers.md: clarify that fallback routing has no environment variables, preserve legacyfallback_modelcompatibility, and recommendfallback_providersfor new configuration.Verification run
git diff --check— passedRisk level
Low. Docs-only changes; no runtime code or dependency changes.