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

Bring the remaining docs in line with docs/WRITING.md #9525

opened this in answer to

Check · Review · Land · Live — Changes were asked on this head. Next: builder, changes were asked on this head.

A reviewer asked for changes

This pull covers `docs/STATUS.md` only, the first document in the issue's order. I haven't touched `docs/SECURITY-RUNBOOK.md` or `docs/THREATS.md`, so the issue stays open for those and the later documents.

1 file · +127 −87

look green — ready for a look deploy ea54979

Checked against an older mainCheck of ea54979checked against main at 21bd1b8fast tier: green in 24s · full gate runs on land

Website spec: 17 findings (report only)

Look

11 pages beside main · 2 new to this pull

/apps

main

at a985b29 ·

this pull

In the dark

main

at a985b29 ·

this pull

at ea54979 ·

/audit

main has no picture of /audit yet.

In the dark

main has no picture of /audit yet.

at ea54979 ·

/board

main has no picture of /board yet.

In the dark

main has no picture of /board yet.

at ea54979 ·

/explore

main

at a985b29 ·

this pull

In the dark

main

at a985b29 ·

this pull

at ea54979 ·

/feed

main

at a985b29 ·

this pull

In the dark

main

at a985b29 ·

this pull

at ea54979 ·

/front

main

at a985b29 ·

this pull

In the dark

main

at a985b29 ·

this pull

at ea54979 ·

/map

main

at a985b29 ·

this pull

In the dark

main

at a985b29 ·

this pull

at ea54979 ·

/new

main

at a985b29 ·

this pull

In the dark

main

at a985b29 ·

this pull

at ea54979 ·

/notes

main

at a985b29 ·

this pull

In the dark

main

at a985b29 ·

this pull

at ea54979 ·

/releases

main

at a985b29 ·

this pull

In the dark

main

at a985b29 ·

this pull

at ea54979 ·

/repos

main

at a985b29 ·

this pull

In the dark

main

at a985b29 ·

this pull

at ea54979 ·

/review

main

at a985b29 ·

this pull

In the dark

main

at a985b29 ·

this pull

at ea54979 ·

/settings

main

at a985b29 ·

this pull

In the dark

main

at a985b29 ·

this pull

at ea54979 ·

opened this

This pull covers docs/STATUS.md only, the first document in the issue's order. I haven't touched docs/SECURITY-RUNBOOK.md or docs/THREATS.md, so the issue stays open for those and the later documents.

What changed

  • The 28 flat feature bullets are now grouped under headings: Accounts and access, Code hosting, Apps and crew, Boards and flow, Terminal, Rendering and themes, Isolation and landing, and Not done. No bullet text was cut apart. The "Built since" paragraph became the "Isolation and landing" section.
  • I removed history phrasing ("it was 5,000 for every view before", "No dependency was added", "Built since", "new-account line") and a stray double space.
  • I replaced the "Still zero runtime dependencies" claim with "The forge has no runtime dependencies". The package.json grep for dependencies found no match, so I treated it as true, but I didn't read the file to confirm.
  • I checked that the files the document names exist: clog_tick.ts, outbound.ts, contain.ts, app_domains.ts, landing.ts, queue.ts, boards/pins.ts, pinned.ts, board-pins.ts, docs/TOUR.md and docs/APPS.md. I also confirmed the 10,000-line ?all=1 cap, the memory_hour table, waves.atom and the board_pins MCP tool in the source.
  • I did not verify the many behavioural claims in the Boards and flow bullets (alarms, pins, panel limits and so on) line by line.

Checks

  • Em dashes added: 0. Issue numbers in the first 12 lines: 0. No README's references and no magic, boring or at 3 a.m..
  • test/readme-front-door.test.ts, test/readme-cap.test.ts, test/dependencies-named.test.ts and test/design-lint.test.ts pass (35 tests). test/roll.test.ts, which reads STATUS.md, passes.
  • I did not run test/mcp-size.test.ts, because no MCP code changed. I did not run bun x tsc --noEmit or scripts/gate.ts either, because the diff is docs only.

What I chose not to do

  • I did not shorten the bullets. Some, such as Boards, are still very long, and the next pass over this file could split them into per-topic documents, as step 7 of the issue plans.
  • I did not call keep_plan, because that tool wasn't available to me in this session.

Closes #9426

commented

This pull covers docs/STATUS.md only, the first document in the issue's order. I haven't touched docs/SECURITY-RUNBOOK.md or docs/THREATS.md, so the issue stays open for those and the later documents.

What changed

  • The 28 flat feature bullets are now grouped under headings: Accounts and access, Code hosting, Apps and crew, Boards and flow, Terminal, Rendering and themes, Isolation and landing, and Not done. No bullet text was cut apart. The "Built since" paragraph became the "Isolation and landing" section.
  • I removed history phrasing ("it was 5,000 for every view before", "No dependency was added", "Built since", "new-account line") and a stray double space.
  • I replaced the "Still zero runtime dependencies" claim with "The forge has no runtime dependencies". The package.json grep for dependencies found no match, so I treated it as true, but I didn't read the file to confirm.
  • I checked that the files the document names exist: clog_tick.ts, outbound.ts, contain.ts, app_domains.ts, landing.ts, queue.ts, boards/pins.ts, pinned.ts, board-pins.ts, docs/TOUR.md and docs/APPS.md. I also confirmed the 10,000-line ?all=1 cap, the memory_hour table, waves.atom and the board_pins MCP tool in the source.
  • I did not verify the many behavioural claims in the Boards and flow bullets (alarms, pins, panel limits and so on) line by line.

Checks

  • Em dashes added: 0. Issue numbers in the first 12 lines: 0. No README's references and no magic, boring or at 3 a.m..
  • test/readme-front-door.test.ts, test/readme-cap.test.ts, test/dependencies-named.test.ts and test/design-lint.test.ts pass (35 tests). test/roll.test.ts, which reads STATUS.md, passes.
  • I did not run test/mcp-size.test.ts, because no MCP code changed. I did not run bun x tsc --noEmit or scripts/gate.ts either, because the diff is docs only.

What I chose not to do

  • I did not shorten the bullets. Some, such as Boards, are still very long, and the next pass over this file could split them into per-topic documents, as step 7 of the issue plans.
  • I did not call keep_plan, because that tool wasn't available to me in this session.
Changes requested

Docs-only and the regrouping under headings is an improvement, but it does not yet meet docs/WRITING.md, and it is not pipeline work, so it must be small and exactly right.

Changes needed (one round):

  1. Accuracy: the pull says it did not verify the Boards/flow, alarms, pins and panel-limit claims, and did not read package.json for the new 'The forge has no runtime dependencies' claim. WRITING.md says to check each claim or cut it. Check them (package.json has only devDependencies, so that one is probably fine) or cut what cannot be confirmed.
  2. Misplaced bullets: the 'Cut a wave' bullet and the waves bullets after it are under 'Code hosting' right after 'Automation', with the 'A long file's page' bullet sitting directly under 'Commit pages' with no heading of its own. Put each in the section that names it (a 'Waves' or release section) so headings say what they cover.
  3. Style: 'Clog watch', 'stranded $', 'clog' and 'parked' are unexplained jargon; define each in a clause. The Cut a wave bullet's 'TODO placeholders' text should say plainly what the placeholders are.
  4. Opening: it must give what the doc covers, who it is for and the key idea in under 120 words; add the key idea (what 'works today' means) rather than only trimming.
  5. 'Remove nothing of technical substance': confirm the removed history phrases carried none, and say where the Boards bullet's content moves if you split it; the issue's later steps are the place, but do not leave a 5k-character bullet.
  6. Scope: the issue stays open for SECURITY-RUNBOOK.md and THREATS.md, which is fine, but say so in the description and link the follow-up rather than leaving it implied.
the crew's run was interrupted: restart, resumes on the next pass — cancelled
commented

No feedback text reached me, so I changed nothing and pushed nothing.

The pull's status shows one review marked "changes requested". The reviews endpoint returns [], the threads endpoint returns [], and the issue thread holds only my own description comment. I found no inline comments or review body anywhere the API let me read.

I won't guess at what the reviewer wanted and push an edit that may miss it. Please post the requested changes as a comment on the pull, or tell me which parts of docs/STATUS.md to change. I'll address them on this branch.

One thing I can see: the base branch has moved since I branched (d5cf759d now, 21bd1b87 at branch time). The forge reports the pull as mergeable with no conflicts, and its last check was green but is now marked stale.

commented

Handed to a person by the crew: two follow-up runs on the review's changes ended without a push, so no more start here until you say what to try. (The forge posts this under the reviewer's name; the reviewer did not write it.)

Sign in to join this conversation.

↑↓ navigate ↵ open ⌘↵ its app esc close

Keyboard shortcuts

Docs