Skip to content

docs: the approval flow as a demo, now that it reliably completes - #103

Merged
HackTuah merged 1 commit into
mainfrom
docs/readme-gif
Sep 26, 2026
Merged

HackTuah merged 1 commit into
mainfrom
docs/readme-gif

Conversation

@HackTuah

@HackTuah HackTuah commented Sep 26, 2026 •

Copy link
Copy Markdown
Member

A 48-frame capture of the whole turn, replacing the approval card still at the top of the README: the request typed, the model asking to run a write-risk tool, Trinity stopping with the exact arguments it will pass, the WRITE badge and four choices, and the call running only after "Allow once" is clicked. It ends on the tool result and the answer.

122KB, so it costs the page nothing.

This is the asset that was held back twice

The first attempt filmed a turn that never completed. The cause was scripts/dev_demo_approval.sh leaving Trinity.Repo.Receipts on the test sandbox pool: outside the suite that pool blocks, and because a decision is receipted before it is dispatched, the tool task stopped after the approval row existed and the card was painted but before it could answer the Session. Slice 126 fixed the script. Slice 125 fixed the redaction filter that was dying in the same runs.

The fix is visible in the capture itself. The status pill reads waiting for approval, where before it sat on running tools until the call timed out and reported an error for something the owner had allowed.

Why the still is removed rather than kept

The GIF shows that same screen and more, and two images was already the budget. What remains is the GIF and the receipts page for the same call, which is the whole argument: it asks before it acts, and the answer is evidence afterwards.

scripts/dev_demo_approval.sh regenerates both, so they are reproducible rather than pictures nobody can make again.

mix gate ; echo "EXIT=$?"
prod_check: PASS
trinity.names: OK over 649 tracked files
trinity.reuse: OK. Every commentable tracked file carries an SPDX header
Result: 846 passed (17 properties, 829 tests), 18 excluded
trinity.coverage: 042 80.31% vs 005 80.31%: OK
EXIT=0

Replaces the approval card still with a 48-frame capture of the whole turn: the
request typed, the model asking to run a write-risk tool, Trinity stopping with
the exact arguments, the WRITE badge and four choices, and the call running only
after Allow once is clicked. It ends on the tool result and the answer.

122KB, so it costs the page nothing.

This is the asset that was held back twice. The first attempt filmed a turn that
never completed, because the demo script left Trinity.Repo.Receipts on the test
sandbox pool and every receipt write queued behind a long-lived connection
holder; since a decision is receipted before it is dispatched, the tool task
stopped after the approval existed and the card was painted but before it could
answer the session. Slice 126 fixed the script and slice 125 fixed the redaction
filter that was dying in the same runs.

The fix is visible in the capture: the status pill reads "waiting for approval",
where before it sat on "running tools" until the call timed out.

The approval card still is removed rather than kept beside it. The GIF shows the
same screen and more, and two images was already the budget.

Signed-off-by: Ayla Croft <aylacroft@proton.me>
@HackTuah
HackTuah merged commit 333ae19 into main Sep 26, 2026
8 checks passed
@HackTuah
HackTuah deleted the docs/readme-gif branch September 26, 2026 16:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant