docs(web): record the stats strip as a divergence, not as an adaptation - #926
Merged
Conversation
`ui-web/README.md` listed "the one-line session stats strip under the composer" among the things adapted from the DeepSeek Harness client. It is not one. The reference ran an in-page A/B between exactly that line and a two-pill variant with click-open dialogs, kept the pills, and deleted the line — its note on the change calls the single-line variant the loser by name. The line stays; that is a decision made here with the tradeoff understood. What could not stay is the README implying it came from upstream, because the whole point of that section is to tell the next person re-syncing this column what is a port and what is ours. A divergence filed under "adapted" is the one kind of error that section cannot afford: it reads as drift to be reconciled rather than as a choice to be re-made. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01WAB22BuCSRhjbpR3a8kc5v
Test Results 5 files 1 009 suites 29m 51s ⏱️ For more details on these failures, see this check. Results for commit 2fbf8a0. |
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.
ui-web/README.mdlisted "the one-line session stats strip under the composer"among the pieces adapted from the DeepSeek Harness client. It is not one.
The reference ran an in-page A/B between exactly that one-line strip and a
two-pill variant with click-open dialogs. The pills won on scannability and on
giving each figure family a home; the line was deleted. Its note on the change
(
.agents/notes/implemented/feature/2026-09-07-composer-session-stats-pills.md)names the single-line variant as the A/B loser, and the reference tree today
contains
StatsPills.tsxand noStatsLine.This does not change the component. Keeping the line is a decision, made
with the tradeoff visible: it reads the same figures from the same ledger and
drops a group with nothing measured, but exact token counts are not reachable
from it the way the dialogs made them.
What had to change is the README implying the line came from upstream. That
section exists to tell the next person re-syncing this column which parts are a
port and which are ours — and a deliberate divergence filed under "adapted" is
the one error it cannot absorb, because it reads as drift to be reconciled
rather than as a choice to be re-made. It now has its own "Diverged
deliberately" entry saying what upstream settled on and what this app has
instead.
Docs only.
🤖 Generated with Claude Code
https://claude.ai/code/session_01WAB22BuCSRhjbpR3a8kc5v