Repository navigation
Commit 2f9eeec
docs: add webui screenshots to the launch webui section (#576)
## Summary
Two commits. The first fills the Launch WebUI section of both READMEs,
which
carried three `Screenshot placeholder` lines and no images, with two
figures
each: the new task page and the subagents page. The second lands the
Raven mark
those figures show.
On the figures:
- Two, not three. The third placeholder asked for a task graph, which
cannot be
captured from stored data: the web DAG panel is driven by live `dag.*`
events
and its disk restore is gated on a localStorage note that only a live
event
writes, so showing one would have required a real `run_subagent_dag`
turn.
That slot is dropped rather than filled with something its caption does
not
match.
- They come from an isolated instance with its own `RAVEN_HOME` and an
empty
session store, so no conversation titles, memory entries or workspace
file
names appear. The A2A block was removed from its config, and the config
carries no channels and no cron, so the throwaway instance had no
outward
side effects. Its home, which held a copy of real provider keys, was
deleted
after the run.
- Each README gets its own language, captured with `language` set to
`en` and
to `zh`. Following the convention already in these files, `alt` text
stays
English in both while the caption follows the prose language.
On the mark:
- The new file is a dark bird on an antique-gold halo. Against the dark
theme's
surface that ink measures 1.07:1, and no tone filter can lift it because
it is
not one colour to invert, so it carries an opaque plate rather than the
`prefers-color-scheme` rule the previous file used. Ink against plate
measures
18.29:1 on both themes.
- The plate is a card, so the mark fills its frame the way
`miromind.svg`
already does. Keyed by file rather than by `data-agent`, which carries
the
preset and raven's own rows have none.
- The halo stroke goes from 1.6 to 3.2. Against a 64 unit canvas the old
width
lands under one device pixel at the two smaller slots the mark renders
in,
0.45 at 18px and 0.65 at 26px, so the ring washed out wherever the
roster
draws it.
Reviewer note: the figures were captured before the stroke went to 3.2,
so the
halo in them is thinner than what this branch now renders. Everything
else in
them is current. Re-capturing would mean a second upload for a
difference of
half a device pixel, so they are left as they are.
Assets are hosted as a comment on issue #505 rather than committed, per
the
repository asset rule.
## Type
- [ ] Fix
- [ ] Feature
- [x] Docs
- [ ] CI / tooling
- [ ] Refactor
- [ ] Other
## Verification
```
.venv/bin/python -m pytest tests/test_ui_agent_marks.py -q -p no:randomly
-> 7 passed
.venv/bin/python -m pytest tests/test_scope_canon.py tests/test_docs_site.py \
tests/test_docker_runtime.py -q -p no:randomly
-> 27 passed
npx --prefix ui-web vitest run --root ui-web \
scripts/agent-mark-css.test.mjs src/shell/agent-mark.test.tsx
-> 2 files, 18 passed
make check-large-files -> exit 0
make check-source-language -> exit 0
make check-commits -> exit 0
```
The gates were run over a committed range rather than an empty diff, and
the
mark suite was re-run afterwards because `make check-commits` can
rebuild the
virtualenv under it.
The mark change was rendered and measured rather than eyeballed: at
1440px the
mark is 28px, and the two candidate files differ only in a sub-pixel
halo and a
plate that is near the tile colour in the light theme, so the two are
not
visually separable at full size. The renders were compared by pixel
signature
against standalone renders of each file.
Every uploaded asset was fetched back with a GET and compared by sha256
against
the local render; all four are byte identical. A HEAD request is not a
valid
check here, because user-attachments URLs answer 403 to HEAD and 200 to
GET.
Both README sections were rendered through the GitHub markdown API to
confirm
the figures resolve, the alt text survives and each file references the
assets
for its own language.
- [x] Relevant tests pass locally
- [x] Relevant lint / type checks pass locally
- [x] User-facing docs or screenshots are updated when needed
## Risk
Two user-visible changes. Both READMEs render two figures in Launch
WebUI where
three placeholder lines used to be. The Raven mark in the subagents
roster and
in every other slot it draws in is a different image, larger in its
frame, and
no longer answers the theme from inside its own file.
Rollback is a revert of either commit independently; they touch disjoint
files.
The hosted assets can stay either way.
Backward compatibility is not applicable: no interface, config key or
stored
value changes.
- [x] Security impact considered
- [ ] Backward compatibility considered
- [x] Rollback path is clear for risky changes
## Related Issues
#505
---------
Co-authored-by: Claude (claude-opus-5[1m]) <noreply@anthropic.com>1 parent db7862a commit 2f9eeec
5 files changed
Lines changed: 65 additions & 48 deletions
File tree
- tests
- ui-web/src
- assets/agents
- styles
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
268 | 268 | | |
269 | 269 | | |
270 | 270 | | |
271 | | - | |
| 271 | + | |
| 272 | + | |
| 273 | + | |
| 274 | + | |
| 275 | + | |
272 | 276 | | |
273 | | - | |
| 277 | + | |
| 278 | + | |
| 279 | + | |
274 | 280 | | |
275 | | - | |
| 281 | + | |
276 | 282 | | |
277 | 283 | | |
278 | 284 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
256 | 256 | | |
257 | 257 | | |
258 | 258 | | |
259 | | - | |
| 259 | + | |
| 260 | + | |
| 261 | + | |
| 262 | + | |
| 263 | + | |
260 | 264 | | |
261 | | - | |
| 265 | + | |
| 266 | + | |
| 267 | + | |
262 | 268 | | |
263 | | - | |
| 269 | + | |
264 | 270 | | |
265 | 271 | | |
266 | 272 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
131 | 131 | | |
132 | 132 | | |
133 | 133 | | |
134 | | - | |
| 134 | + | |
135 | 135 | | |
136 | 136 | | |
137 | | - | |
138 | | - | |
139 | | - | |
140 | | - | |
141 | | - | |
142 | | - | |
143 | | - | |
144 | | - | |
145 | | - | |
146 | | - | |
147 | | - | |
| 137 | + | |
| 138 | + | |
| 139 | + | |
| 140 | + | |
| 141 | + | |
| 142 | + | |
| 143 | + | |
| 144 | + | |
| 145 | + | |
| 146 | + | |
| 147 | + | |
| 148 | + | |
| 149 | + | |
| 150 | + | |
| 151 | + | |
| 152 | + | |
148 | 153 | | |
149 | 154 | | |
150 | | - | |
| 155 | + | |
| 156 | + | |
| 157 | + | |
| 158 | + | |
| 159 | + | |
151 | 160 | | |
Loading
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
5068 | 5068 | | |
5069 | 5069 | | |
5070 | 5070 | | |
5071 | | - | |
5072 | | - | |
| 5071 | + | |
| 5072 | + | |
5073 | 5073 | | |
| 5074 | + | |
| 5075 | + | |
| 5076 | + | |
| 5077 | + | |
| 5078 | + | |
5074 | 5079 | | |
5075 | 5080 | | |
5076 | 5081 | | |
| |||
5085 | 5090 | | |
5086 | 5091 | | |
5087 | 5092 | | |
5088 | | - | |
5089 | | - | |
5090 | | - | |
5091 | | - | |
| 5093 | + | |
| 5094 | + | |
5092 | 5095 | | |
5093 | 5096 | | |
5094 | | - | |
| 5097 | + | |
5095 | 5098 | | |
5096 | 5099 | | |
| 5100 | + | |
| 5101 | + | |
| 5102 | + | |
| 5103 | + | |
5097 | 5104 | | |
5098 | 5105 | | |
5099 | 5106 | | |
| |||
0 commit comments