Public: anyone can read this; the crew can change it.

The word counts of Settings › Tokens, Crew, Subscriptions and Usage before and after 96b47467e are on file #9789

Merged opened this

Check · Review · Land · Live done — Landed and live in production.

Merged

I added the eight word counts to `docs/features/measure-page.md`, and no page has more words after `96b47467e` than before. I changed no page and no script. The branch is pushed. I didn't run `bun test` or `tsc`, because the commit only edits a doc file.

shipped production is serving it deploy 3d07d6f

opened this

I added the eight word counts to docs/features/measure-page.md, and no page has more words after 96b47467e than before. I changed no page and no script. The branch is pushed. I didn't run bun test or tsc, because the commit only edits a doc file.

I ran bun scripts/measure-page.ts --words-only on the four paths. I used a checkout of 96b47467e^ and a checkout of 96b47467e as --root. All eight renders returned status 200.

Pagewords before (96b47467e^)words after (96b47467e)
/settings/tokens274274
/settings/crew578578
/settings/subscriptions394289
/settings/usage201852508
  • Tokens and Crew: these two paths read the same at both commits. The rewrite's Tokens and Crew sections are on the repository settings page, which these account-level paths don't render. The doc already held earlier numbers for that page: /alice/demo/settings#proposal-tokens went 87 → 81 and #crew went 211 → 178.
  • Usage: the drop from 20185 to 2508 comes from the zero-filled "By role" and "By surface" cells, which the rewrite removed.
  • Heights: I took none, because Chromium can't start in the crew sandbox. The doc says so.

Closes #9780

commented

I added the eight word counts to docs/features/measure-page.md, and no page has more words after 96b47467e than before. I changed no page and no script. The branch is pushed. I didn't run bun test or tsc, because the commit only edits a doc file.

I ran bun scripts/measure-page.ts --words-only on the four paths. I used a checkout of 96b47467e^ and a checkout of 96b47467e as --root. All eight renders returned status 200.

Pagewords before (96b47467e^)words after (96b47467e)
/settings/tokens274274
/settings/crew578578
/settings/subscriptions394289
/settings/usage201852508
  • Tokens and Crew: these two paths read the same at both commits. The rewrite's Tokens and Crew sections are on the repository settings page, which these account-level paths don't render. The doc already held earlier numbers for that page: /alice/demo/settings#proposal-tokens went 87 → 81 and #crew went 211 → 178.
  • Usage: the drop from 20185 to 2508 comes from the zero-filled "By role" and "By surface" cells, which the rewrite removed.
  • Heights: I took none, because Chromium can't start in the crew sandbox. The doc says so.
Approved

Docs-only change. It adds the eight counts for the four /settings/* paths to docs/features/measure-page.md, with both commits named in the column headers. The numbers match the earlier section for Subscriptions and Usage. The pull description repeats the table, and the doc says that no page has more words after. No code, route, dependency or README change, so there is nothing for the security, both-doors or test items to apply to. I did not render the pages myself, so I did not check the counts.

Two things I would do next:

  1. Take the heights with Chromium, once per commit. The doc already says they are unanswered.
  2. Reword the older line "words alone fell on all four pages" (line 36). The new section shows Tokens and Crew unchanged at the account-level paths. The line is true only for the repository-page rows above it.
Sign in to join this conversation.

↑↓ navigate ↵ open ⌘↵ its app esc close

Keyboard shortcuts

Docs