Skip to content

docs(ui): drop the pasted duplicate sentence from the data notes - #326

Merged
argszero merged 1 commit into
mainfrom
docs/ui-readme-drop-duplicated-clause
Sep 29, 2026
Merged

argszero merged 1 commit into
mainfrom
docs/ui-readme-drop-duplicated-clause

Conversation

@argszero

Copy link
Copy Markdown
Owner

Summary

ui/README.md §数据说明 states the chat-mock billing rule twice, verbatim: once as its own bullet (- 消费模拟(聊天 Mock)按 输出参考价 × 0.19M tokens 计费…), and once pasted onto the tail of the multi bullet, where it is joined to an unrelated fact by a semicolon. This PR removes the pasted copy; the dedicated bullet keeps the rule.

Related Issue

None (self-found during a documentation audit; no tracking issue exists).

Changes

  • Delete the pasted duplicate sentence from the end of the 市场模型带 multi 标记… bullet in ui/README.md; that bullet now ends at …已随文档精简合并到此节) where it belongs.
  • The dedicated bullet for the billing rule is left untouched — it is where the sentence belongs.
  • No configuration/data-structure changes, so no example-file sync is required.

Origin of the duplication. 9a3ff3e (#22) first wrote the sentence at the tail of the multi bullet. 500e17d (#27) added the dedicated bullet but did not remove the original, so the sentence has appeared twice since 2026-08-17. A sweep of every tracked *.md (clauses of ≥25 chars occurring more than once) finds this as the only duplicated long clause in the repository, and it now returns zero.

Tests

Checklist

  • Branch name follows the convention (docs/<description>)
  • Commit message uses Conventional Commits (docs(ui): …)
  • Single responsibility, minimal change (1 file, +1/−1)

The 数据说明 section states the chat-mock billing rule twice: once as its
own bullet and once pasted onto the tail of the multi-key bullet. The
second copy entered when 500e17d (#27) added the dedicated bullet but
left the original sentence on the multi bullet, so the two unrelated
facts were joined by a semicolon. Remove the pasted copy; the dedicated
bullet keeps the rule.

Documentation only; no code or gate changes.
@argszero

Copy link
Copy Markdown
Owner Author

Self-review (committer, allow_self_merge: true).

Verified against the tree this branch is based on (9df6d00):

  1. The sentence 消费模拟(聊天 Mock)按 输出参考价 × 0.19M tokens 计费…(US-6) appeared verbatim twice in ui/README.md — as its own bullet and pasted onto the tail of the multi bullet. After this change it appears once, on its own bullet.
  2. A sweep of every tracked *.md (clauses of ≥25 chars occurring more than once) returned this as the only duplicated long clause before the change, and returns nothing after it.
  3. The removed text is byte-identical to the retained bullet, so no information is lost. The rule itself is still accurate (ui/js/app.js sets the mock to 0.19 tokens), and US-6 remains a live tag used across ui/js/app.js, ui/index.html, ui/css/style.css — so neither the number nor the tag was ever the problem; only the duplication was.

ui/README.md is include_str! corpus, so the full suite was re-run: cargo test 426/0, cargo fmt --check rc 0, cargo clippy --all-targets -- -D warnings rc 0.

Documentation-only change; no gate added, consistent with the repo's doc-comment-claims convention.

@argszero
argszero merged commit 9a0c853 into main Sep 29, 2026
2 checks passed
@argszero
argszero deleted the docs/ui-readme-drop-duplicated-clause branch September 29, 2026 14:46
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