docs(readme): rewrite the tagline and surface the live instance - #239
Merged
Merged
Conversation
Host rant 2026-09-14T20:32:17: the opening tagline read poorly and had no
appeal, and the README never mentioned any live URL.
- Tagline (both README.md and README.en.md): restore the dropped subject,
name the object explicitly ("models shared by others"), and state the
benefit readers actually get (redeem points for others' models). The
vague comparison is replaced with the exchange value proposition.
- Use-case table line 25 (both files): same sentence-pattern problem,
rewritten in the same voice so the two occurrences stay consistent.
- Add the live instance https://aitokenpool.args.fun/ in two places: a
prominent line under the language switcher, and a no-install path at
the top of Quick Start.
Docs only; no ATP_MASTER_KEY= literal form added (deploy_gate stays
green); zh/en line counts remain equal (80/80).
argszero
added a commit
that referenced
this pull request
Sep 14, 2026
Ships two changes since v0.7.23 (#239, #240): - #240: the transactions summary/COUNT/trend statements joined keys/users/ api_keys unconditionally, which made SQLite drop the covering index and fall back to a row-by-row lookup (0.21s -> 22.6s on the dev db, 186k rows over NFS). That query held the shared DB lock long enough to queue /api/me and /api/models behind it, and the boot session restore waits for all four requests, so a refresh could sit on the login page with the token intact. The joins are now conditional on a column filter actually needing them. - #239: README tagline rewritten and the live instance surfaced. No schema change (12 -> 14 happened in v0.7.23). - Cargo.toml / Cargo.lock: 0.7.23 -> 0.7.24. - ui/index.html: asset cache-bust -> 20260914-11 (all five refs). - CHANGELOG.md: v0.7.24 entry. Gates: `cargo test` 249 passed / 0 failed, `cargo fmt --check` clean.
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.
Summary
Addresses host rant 2026-09-14T20:32:17 (
README 标语读起来不通顺、也没有吸引力，且没有放线上实例地址).① Tagline (both
README.mdandREADME.en.md) — three defects fixed together: the dropped subject (「需要时」需要什么？), the vague object (「用别人的」= 别人的什么？), and the missing "exchange" benefit (the reader's gain was never stated). The new line names the object explicitly and states the value proposition.② Live instance — the READMEs previously mentioned no online address at all. Added
https://aitokenpool.args.fun/in two places: a prominent line under the language switcher, and a "no install needed" path at the top of Quick Start.③ Line 25 restated sentence (both files) — same sentence-pattern problem as ①, rewritten in the same voice so the two occurrences stay consistent (otherwise fixing one makes the other more jarring).
Related Issue
No linked issue. Driven by rant timestamp
2026-09-14T20:32:17(projectaitokenpool).Changes
ATP_MASTER_KEY=literal form added —src/deploy_gate.rsinclude_str!s both READMEs, and this change adds none (the only remainingATP_MASTER_KEY=occurrences are the pre-existing$(openssl rand -hex 32)examples, classifiedNotALiteral)Tests
cargo testall pass — 249 passed; 0 failedcargo fmt --checkpassesdeploy_gate(which reads both READMEs) passes — 3 passedChecklist
docs/)