Repository navigation
docs: retitle the showcase entries and add two simulation runs - #592
Merged
Merged
Conversation
The showcase titles named the artifact type, not the run. Four now say what the run made: - the game entry names the engine and the genre it built, replacing a title that only said "FPS game" and listed the graph's three stages; - the framework comparison says the frameworks are agent-orchestration ones; - the light-pollution and GPS entries lead with their subject instead of their format, matching the interrogative titles two other entries already use. Only the bold title text inside each cell changed. Cell attributes, asset URLs, alt text and table structure are untouched, and both files keep nine entries in the same order. Co-authored-by: Claude (claude-opus-5[1m]) <noreply@anthropic.com>
Two engineering runs join the showcase after the game entry, each with its task graph above and the run's own footage below: - a CalculiX cantilever driven to its plastic limit, where eight rounds of bisection narrow the bracket from 1800-2000 kN down to 3.125 kN; - a dam-break solve tuned over seven rounds until the water fraction stays inside [0,1], the out-of-bounds excursion falling from 1e0 to 1.36e-10. Both graphs are the same shape: Raven-Research into Raven-Code into Raven-Oncall, run in sequence. They sit right after the game entry so that they follow the AI4S benchmark figure the section opens against. The four assets are hosted on issue 505 and referenced by URL, because the repository bans image files by extension at any size. Co-authored-by: Claude (claude-opus-5[1m]) <noreply@anthropic.com>
The two entries now close the section rather than open it, and their recordings render at 90 percent width while their task graphs stay at 100. Only the recordings take the reduction because only they are portrait: 720x1260 against the 2270x400 of the task-graph bars beside them. Narrowing a tall asset buys height back, and narrowing a wide thin one buys nothing. Co-authored-by: Claude (claude-opus-5[1m]) <noreply@anthropic.com>
LivXue
requested review from
0xKT,
Handsome-wzw,
ZuyiZhou and
gloryfromca
and removed request for
0xKT and
gloryfromca
September 21, 2026 09:54
gloryfromca
reviewed
Sep 21, 2026
gloryfromca
left a comment
Member
There was a problem hiding this comment.
No blockers; this can merge as far as I am concerned.
I reviewed the full diff and the surrounding showcase/history. I also covered AGENTS.md/CLAUDE.md, canonical terminology via CONTEXT-MAP.md, README consumers, backward compatibility, test weakening (none), and architecture constraints (not implicated by this documentation-only change). The English and Chinese entries stay structurally aligned, the new assets are external as required, and the retired titles have no remaining consumers.
Independent verification:
uv run --frozen --all-extras pytest tests/test_cli_onboard_commands.py tests/test_docker_runtime.py tests/test_living_docs.py -q: 346 passed.uv run python scripts/check_large_files.py github/main..HEAD: passed.uv run python scripts/check_source_language.py github/main..HEAD: passed.git diff --check github/main...HEAD: passed.- GitHub Markdown rendering produced matching image and cell counts for both READMEs; all four new attachment URLs returned partial PNG/GIF content successfully.
Handsome-wzw
approved these changes
Sep 21, 2026
6 of 12 tasks
gloryfromca
added a commit
that referenced
this pull request
Sep 21, 2026
## Summary Second catch-up of `refactor/ui_web_architecture` with `main`, this time to `6ebc2baa` (#592): the 31 commits `main` gained since #529 took it to `dbe259f6`. Synced main to `6ebc2baa`; the next round starts from that SHA. #529 landed as a squash, so git still saw the two branches diverge at `5a29ea22` and re-raised every conflict #529 had already settled (68 paths instead of 33). The first commit here merges `dbe259f6` with `-s ours` -- tree unchanged, ancestry recorded -- so the second commit merges only what is new; a third takes #528 and a fifth takes #592 (READMEs only, no conflict), both of which landed on `main` while the earlier commits were being verified. The same steps work for every later round. Thirty-three paths conflicted in the second commit, one in the third; the fourth commit is what an independent read of the merge turned up (the sheet's Connect, `capabilities_wanted`'s fourth case, the dead styles); the sixth merges this branch's own tip (#584, #591, #586), where only the fixture-shape gate's UNSENT list met (both sides kept). How each conflict was settled: - 17 legacy ui-web files (`live/`, `demo/`, `features/xa`, `features/knowledge`, `scripts/boot-order.test.mjs`, `composer/open-conversation.test.ts`), edited on `main` for #542, #549 and #554: deleted on this branch, and the deletions stand. `main`'s `shell/workdir.ts` and its test, which git relocated into `lib/`, are dropped for the same reason -- see the follow-up below. - `main.tsx`, `page.html`, `state/session/registry.test.ts`: this branch's version. `main`'s edits there were the legacy chip wiring; the registry conflict was a rename git guessed from `scripts/page-switch-live.test.mjs`. - `raven/agent/subagent/probe.py`: both imports; Test records a snapshot for a preset too (`main`, #554) through `record_capabilities` (this branch, #559); a kept record carries `needs_auth`; the boot backfill re-measures an outdated model menu (this branch) and a recorded credential refusal (`main`). - `raven/rpc/methods/subagents.py`: `needs_auth` reads the snapshot this branch already loads per row. - `raven/agent/subagent/manager.py` (#528, third commit): both sides kept at both hunks -- `main`'s two shutdown refusals beside `SPAWN_REFUSED_PREFIX` with this branch's `_row_pin` after them, and the spawn announce that types its mark and carries `node_id` (this branch) then returns early when `_inject` was refused (`main`). - `tests/test_subagent_probe.py`, `test_subagent_acp.py`, `test_rpc_subagents.py`, `test_rpc_console.py`, `test_config_update.py`: both sides' new tests, in order. This branch's two backfill tests pin `_unconfigured_acp_preset_rows` the way `main`'s do, and the `_agent_home` helper both sides defined is one function passing a `Path`, as the config property does. - `README.md`, `README.zh-CN.md`: `main`'s figure and showcase section. - `ui-web/src/rpc/generated.ts`, `ui-tui/src/rpc/generated.ts`, `ui-tui/src/i18n/messages.generated.ts`: regenerated from the merged schema and catalogue. Carried onto the rebuilt tree, since the backend halves merged cleanly: - #554: the agent hub's rows, its sheet and the wizard step name a refused agent `Unauthorized` (disabled) instead of offering Connect; the sheet offers Test beside it, the press that re-measures the handshake and lets the row return to Connect after a sign-in. `capabilities_wanted` treats a recorded refusal as wanting a re-measure, the same fourth case the boot backfill gained from `main`. Four tests. - #542: rail rows carry the session's `workdir`, so the folder group `RailPage.tsx` gained in the merge can fill. The tag's class moves under the rail namespace (`rail-wdt`) with its rule in the new `features/rail/styles.css`, as check-class-namespace requires. - #549: the page-side upload ceiling follows `raven/rpc/files.py` to 100 MB. - The offline fixtures send `needs_auth`; `session.list.sessions[].workdir` is pinned unsent in fixture-shape until the chip lands. The folder chip's thirty lines of popover rules that `main` added to `styles/page.css` are dropped with the chip; the follow-up brings its own. Follow-up, not in this PR: the folder chip itself (#542's `shell/workdir.ts`: the composer chip, the `fs.dirs` browser, `workdir` on `session.create`) has to be rebuilt on this tree's chip pattern (`state/perm.ts` + `chrome/PermChip.tsx`). The backend, the i18n keys and the rail half are already here. ## Type - [ ] Fix - [ ] Feature - [ ] Docs - [ ] CI / tooling - [ ] Refactor - [x] Other (catch-up merge) ## Verification Run in the worktree on the merged tree: - `uv run pytest tests/test_subagent_probe.py tests/test_subagent_acp.py tests/test_rpc_subagents.py tests/test_rpc_console.py tests/test_config_update.py tests/test_rpc_session.py tests/test_rpc_contract_shapes.py tests/test_rpc_schema_match.py tests/test_rpc_registration.py tests/test_i18n_boundary.py tests/test_subagent_manager.py tests/test_subagent_dag_runner.py tests/test_rpc_bootstrap.py tests/test_cli_tui_commands.py tests/test_cli_gateway_commands.py tests/test_rpc_transport.py tests/test_cli_serve_commands.py tests/test_importer_orchestrator.py tests/test_cli_gateway_page.py -q` on the final head -- 1893 passed (the same files, run per commit as the branch grew: 1141, 590, 501). - `make lint-python` -- ruff check clean; ruff format clean after formatting the three spliced test files. - `npm test` in `ui-web` on the final head -- 189 files, 2468 tests passed (before the sync merge: 193 files, 2609; the first run had failed two gates, check-class-namespace and fixture-shape, on the merged `.wdt` class and the three new optional wire fields, both fixed as described above). - `npm run type-check`, `npm run lint` (0 errors, 5 pre-existing warnings), `npm run gen:check` (193 methods) in `ui-web`. - `npm run lint:i18n`, `npm run lint:rpc`, `npm run type-check`, `npm test` in `ui-tui` -- all clean. - `make build-ui` on the final head -- page built, `boot-snapshot: OK` on both snapshots (235 nodes, the golden #591 set). - `scripts/check_source_language.py origin/refactor/ui_web_architecture..HEAD` and `scripts/check_large_files.py origin/refactor/ui_web_architecture..HEAD` -- both exit 0; `git merge-tree --write-tree origin/main HEAD` -- clean. The full Python suite is left to CI. - [x] Relevant tests pass locally - [x] Relevant lint / type checks pass locally - [ ] User-facing docs or screenshots are updated when needed ## Risk No source change of its own beyond the resolutions and the three carries above. On the page: a refused external agent now shows a disabled `Unauthorized` button where it showed Connect; conversations pinned to a folder group under their own rail heading with the folder's name on the row; uploads up to 100 MB are sent instead of refused at 25 MB. All three match what `main` ships today. This branch is squash-only, so the merge base stays at `5a29ea22` after this lands; the next round repeats the `-s ours 6ebc2ba` step first. Rollback is reverting the squash commit; the branch is back at `99c37e74`. - [x] Security impact considered - [x] Backward compatibility considered - [x] Rollback path is clear for risky changes ## Related Issues N/A. Unblocks #475. --------- Co-authored-by: xfng-sd <xufang@shanda.com> Co-authored-by: Claude (claude-opus-5) <noreply@anthropic.com> Co-authored-by: Handsome-wzw <68996445+Handsome-wzw@users.noreply.github.com> Co-authored-by: Dizhan Xue <dizhan.xue@evermind.ai> Co-authored-by: Kevin Hu <kevinhu.sh@gmail.com> Co-authored-by: admin <admin@SH-HuKai.local> Co-authored-by: 江国庆/Forrest <mr.jianggq@163.com> Co-authored-by: 江国庆 <guoqingjiang@deepglint.com> Co-authored-by: Codex <noreply@openai.com> Co-authored-by: ypflll <ypflll@163.com> Co-authored-by: yao pengfei <yaopengfei@shanda.com> Co-authored-by: zhanghui <huizhang1995@gmail.com> Co-authored-by: Zuyi Zhou <144661423+ZuyiZhou@users.noreply.github.com> Co-authored-by: Tong Li <litong02@shanda.com> Co-authored-by: litong <238663200+TongLi31@users.noreply.github.com> Co-authored-by: Tchen-data <176354753+Tchen-data@users.noreply.github.com> Co-authored-by: gloryfromca <23442919+gloryfromca@users.noreply.github.com> Co-authored-by: Dizhan Xue <xuedizhan17@mails.ucas.ac.cn> Co-authored-by: silverLXT <25195409+silverLXT@users.noreply.github.com> Co-authored-by: xiaotian.luo <xiaotian.luo@thetahealth.ai> Co-authored-by: userName20260323 <zhao.wang@evermind.ai> Co-authored-by: zhao.wang <270284818+userName20260323@users.noreply.github.com> Co-authored-by: arelchan <1239372199@qq.com> Co-authored-by: arelchan <204152633+arelchan@users.noreply.github.com>
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
The showcase named the artifact type rather than the run, so a reader scanning the
section learned the format and not the work. This retitles four entries and adds two.
Retitled:
Added as the last two entries, closing the section:
plastic limit, and eight rounds of bisection narrow the bracket from 1800-2000 kN
down to 3.125 kN, or 0.16 percent.
until the water fraction stays inside [0,1]; the out-of-bounds excursion falls from
1e0 to 1.36e-10 once the MULES correction is switched off.
Both new entries carry the same task graph shape: Raven-Research into Raven-Code into
Raven-Oncall, run in sequence, 3 of 3 done.
Sign-offs taken before publishing:
staying inside [0,1], which is what the footage's own metric panel tracks.
both READMEs can share it; its frames were re-read round by round to confirm.
Only the recordings are portrait, 720x1260 against the 2270x400 of the bars
beside them, so only they gain height back from a narrower box.
its title now reads "agent-orchestration". That is deliberate: the alt describes
what the image shows, and the board's own wording did not change.
Assets: the four new files, two run recordings and two task graphs, are hosted on the
asset issue and referenced by URL, because the repository bans image files by
extension at any size. Each was checked byte for byte against its local source after
upload.
Type
Verification
Scoped with three dots throughout.
git diff origin/main...HEADis two files, 38insertions and 8 deletions, and
git rev-list --left-right --count origin/main...HEADis
0 3after a rebase onto the current base.uv run --frozen --all-extras pytest tests/test_cli_onboard_commands.py tests/test_docker_runtime.py tests/test_living_docs.py -qgave 346 passed. Those are the tests that read the root READMEs.make lint check-commits check-large-files check-source-languageall exit 0 at the rebased head.Rendered both files through the GitHub markdown API. Each returns one video player, 41 images, 69 cells, zero empty cells and eleven bold titles, and the eleven come back in the same order in both languages, with the two new entries last.
Both READMEs carry the same four new asset URLs. The only per-language assets are the three that were already language-specific before this branch.
Grepped the whole tracked tree, with no path filter and no extension filter, for the four retired titles in both languages: zero hits outside these two files.
Checked the added lines against every retired term in the repository's
_Avoid_entries. One term matched,run, which that entry retires as a TUI transcript construct; the same phrase,run in sequence, is already onorigin/mainin this exact alt-text position.Relevant tests pass locally
Relevant lint / type checks pass locally
User-facing docs or screenshots are updated when needed
Risk
Text and image references on two README files. No code path, no schema, and nothing
links to a title. The section grows by one table, so the page is longer and pulls two
more assets, the larger of which is 8.5 MB.
Rollback is a revert of the three commits.
Related Issues
505 hosts the four assets this adds. Nothing is closed by this PR.