docs(tooling): a page or an internal row for every command of torb #757

Merged
argus merged 4 commits from loop/752-command-pages into main 2026-11-13 18:09:29 +00:00
Collaborator

Closes #752

What changes

  • docs/tooling/torb-upgrade.md: new page for torb upgrade (version, --channel, --list), described from compiler/src/cli/upgrade.trb and its declaration.
  • docs/tooling/torb-docs-check.md: new page for torb docs check, index, declarations, skill and bundle (one page; source and site already have theirs).
  • docs/tooling/the-torb-command.md: upgrade in the synopsis and its own entry, the docs entry names all seven subcommands, and ### Internal commands is one table marked internal and unstable (tokens, ast, parse, ir, natives, manifest, highlight).
  • docs/tooling/index.md: the two new entries of the generated block, written by hand in the order torb docs index produces.
  • compiler/tests/command-pages.test.trb: reads the command table from main.trb (declaredCommands, packageCommands, the registry list and every dispatch arm) and fails for a command that has neither docs/tooling/torb-<command>.md nor a row in the internal table; for docs it checks each subcommand.

Negative check

The last test of the group feeds the same check an invented command frobnicate, an empty internal table and an empty page list, and asserts that exactly those are reported. So a command without a page or a line fails the check, and the test that runs on the real table would fail the same way.

Test

No local torb binary, so nothing of this ran locally; CI is the verifier (torb test, torb docs check, torb docs index --check). pdisc.py check is ok.

Follow-ups

None.

Closes #752 ## What changes - `docs/tooling/torb-upgrade.md`: new page for `torb upgrade` (version, `--channel`, `--list`), described from `compiler/src/cli/upgrade.trb` and its declaration. - `docs/tooling/torb-docs-check.md`: new page for `torb docs check`, `index`, `declarations`, `skill` and `bundle` (one page; `source` and `site` already have theirs). - `docs/tooling/the-torb-command.md`: `upgrade` in the synopsis and its own entry, the `docs` entry names all seven subcommands, and `### Internal commands` is one table marked internal and unstable (tokens, ast, parse, ir, natives, manifest, highlight). - `docs/tooling/index.md`: the two new entries of the generated block, written by hand in the order `torb docs index` produces. - `compiler/tests/command-pages.test.trb`: reads the command table from `main.trb` (`declaredCommands`, `packageCommands`, the registry list and every dispatch arm) and fails for a command that has neither `docs/tooling/torb-<command>.md` nor a row in the internal table; for `docs` it checks each subcommand. ## Negative check The last test of the group feeds the same check an invented command `frobnicate`, an empty internal table and an empty page list, and asserts that exactly those are reported. So a command without a page or a line fails the check, and the test that runs on the real table would fail the same way. ## Test No local `torb` binary, so nothing of this ran locally; CI is the verifier (`torb test`, `torb docs check`, `torb docs index --check`). `pdisc.py check` is ok. ## Follow-ups None.
Refs #752
test(docs): every command of torb has a tooling page or a row in the internal table
Some checks failed
ci / commit messages (pull_request) Successful in 7s
ci / what the change touches (pull_request) Successful in 6s
ci / the VS Code extension (editors/vscode) (pull_request) Has been skipped
pdisc / pdisc (pull_request) Successful in 11s
ci / which runners exist (pull_request) Successful in 1s
ci / bootstrap and conformance (windows-x64) (pull_request) Has been skipped
ci / bootstrap and conformance (linux-arm64) (pull_request) Has been skipped
ci / bootstrap and conformance (macos-arm64) (pull_request) Has been skipped
ci / bootstrap (linux-x64) (pull_request) Failing after 32s
ci / tier B (linux-x64) (pull_request) Has been skipped
ci / cross-compiled on linux-x64 (windows-x64) (pull_request) Has been skipped
ci / cross-compiled on linux-x64 (linux-arm64) (pull_request) Has been skipped
ci / cross-compiled on linux-x64 (macos-arm64) (pull_request) Has been skipped
ci / every host emits the same C (pull_request) Has been skipped
ci / gates (pull_request) Has been cancelled
ci / tier A (linux-x64) (pull_request) Has been cancelled
ci / tools/release-sync and tools/registry (linux-x64) (pull_request) Has been cancelled
ci / the playground (torb as WebAssembly) (pull_request) Has been cancelled
ci / conformance (JavaScript under Node.js) (pull_request) Has been cancelled
ci / conformance (PHP 5.6, 7.4 and 8.4) (pull_request) Has been cancelled
ci / pack the seed (pull_request) Has been cancelled
ci / archives (pull_request) Has been cancelled
1aa47b0fc5
Refs #752
test(docs): parenthesize the method calls of the command-pages test
Some checks failed
ci / commit messages (pull_request) Successful in 7s
ci / what the change touches (pull_request) Successful in 7s
ci / the VS Code extension (editors/vscode) (pull_request) Has been skipped
pdisc / pdisc (pull_request) Successful in 12s
ci / which runners exist (pull_request) Successful in 0s
ci / bootstrap and conformance (windows-x64) (pull_request) Has been skipped
ci / bootstrap and conformance (linux-arm64) (pull_request) Has been skipped
ci / bootstrap and conformance (macos-arm64) (pull_request) Has been skipped
ci / bootstrap (linux-x64) (pull_request) Successful in 6m48s
ci / tier B (linux-x64) (pull_request) Has been skipped
ci / cross-compiled on linux-x64 (windows-x64) (pull_request) Has been skipped
ci / cross-compiled on linux-x64 (linux-arm64) (pull_request) Has been skipped
ci / cross-compiled on linux-x64 (macos-arm64) (pull_request) Has been skipped
ci / tools/release-sync and tools/registry (linux-x64) (pull_request) Successful in 5m17s
ci / the playground (torb as WebAssembly) (pull_request) Successful in 3m55s
ci / conformance (JavaScript under Node.js) (pull_request) Successful in 1m43s
ci / every host emits the same C (pull_request) Successful in 10m9s
ci / conformance (PHP 5.6, 7.4 and 8.4) (pull_request) Successful in 3m36s
ci / tier A (linux-x64) (pull_request) Failing after 40m17s
ci / pack the seed (pull_request) Successful in 11s
ci / archives (pull_request) Has been skipped
ci / gates (pull_request) Failing after 0s
3fad3e4c7c
Collaborator

CI red on 3fad3e4: ci / tier A (linux-x64) (gates) - two red gates: format --check (./compiler/tests/command-pages.test.trb would change, 1 of 1718 files) and docs skill --check (the derived Agent Skill is stale: skills/torbscript/references/index.md, tooling/index.md, tooling/the-torb-command.md differ; tooling/torb-docs-check.md and tooling/torb-upgrade.md are missing - run torb docs skill). Everything else is green, including pdisc.

CI red on 3fad3e4: ci / tier A (linux-x64) (gates) - two red gates: `format --check` (`./compiler/tests/command-pages.test.trb` would change, 1 of 1718 files) and `docs skill --check` (the derived Agent Skill is stale: `skills/torbscript/references/index.md`, `tooling/index.md`, `tooling/the-torb-command.md` differ; `tooling/torb-docs-check.md` and `tooling/torb-upgrade.md` are missing - run `torb docs skill`). Everything else is green, including pdisc.
fix(docs): regenerate the derived Agent Skill and format the command-pages test
All checks were successful
pdisc / pdisc (pull_request) Successful in 45s
ci / commit messages (pull_request) Successful in 7s
ci / what the change touches (pull_request) Successful in 8s
ci / which runners exist (pull_request) Successful in 1s
ci / bootstrap and conformance (linux-arm64) (pull_request) Has been skipped
ci / bootstrap and conformance (macos-arm64) (pull_request) Has been skipped
ci / bootstrap and conformance (windows-x64) (pull_request) Has been skipped
ci / the VS Code extension (editors/vscode) (pull_request) Successful in 21s
ci / bootstrap (linux-x64) (pull_request) Successful in 9m6s
ci / cross-compiled on linux-x64 (windows-x64) (pull_request) Has been skipped
ci / cross-compiled on linux-x64 (linux-arm64) (pull_request) Has been skipped
ci / cross-compiled on linux-x64 (macos-arm64) (pull_request) Has been skipped
ci / tools/release-sync and tools/registry (linux-x64) (pull_request) Successful in 4m49s
ci / every host emits the same C (pull_request) Successful in 10m2s
ci / the playground (torb as WebAssembly) (pull_request) Successful in 4m28s
ci / conformance (JavaScript under Node.js) (pull_request) Successful in 2m14s
ci / conformance (PHP 5.6, 7.4 and 8.4) (pull_request) Successful in 5m39s
ci / tier B (linux-x64) (pull_request) Successful in 29m4s
ci / tier A (linux-x64) (pull_request) Successful in 43m5s
ci / pack the seed (pull_request) Has been skipped
ci / archives (pull_request) Has been skipped
ci / gates (pull_request) Successful in 0s
9438e9f17e
orion requested review from argus 2026-11-13 15:20:02 +00:00
argus approved these changes 2026-11-13 18:09:27 +00:00
argus left a comment

Review of 9438e9f (closes #752)

  • Tests: ok. compiler/tests/command-pages.test.trb reads the command table from main.trb and fails for a command with neither a page nor an internal-table row, with a negative check (frobnicate, empty table). CI green on this head, including the two gates that were red on 3fad3e4 (format --check, docs skill --check) and pdisc.
  • Correctness: ok. The new pages match compiler/src/cli/upgrade.trb (install.trb requirement, toolchains/, active marker, channels, --list) and the docs subcommand arms in main.trb; the derived skill copies are in the diff.
  • Scope: ok. Docs for every command plus the test that holds it, as in #752. No unrelated changes.
  • Security/data/workflows: nothing touched.
  • Docs: ok. Frontmatter present, index blocks written in generated order, no INDEX.md in the diff.

Approved.

Review of 9438e9f (closes #752) - Tests: ok. `compiler/tests/command-pages.test.trb` reads the command table from `main.trb` and fails for a command with neither a page nor an internal-table row, with a negative check (`frobnicate`, empty table). CI green on this head, including the two gates that were red on 3fad3e4 (`format --check`, `docs skill --check`) and pdisc. - Correctness: ok. The new pages match `compiler/src/cli/upgrade.trb` (install.trb requirement, toolchains/<version>, active marker, channels, --list) and the docs subcommand arms in `main.trb`; the derived skill copies are in the diff. - Scope: ok. Docs for every command plus the test that holds it, as in #752. No unrelated changes. - Security/data/workflows: nothing touched. - Docs: ok. Frontmatter present, index blocks written in generated order, no INDEX.md in the diff. Approved.
argus merged commit 6a1782cf42 into main 2026-11-13 18:09:29 +00:00
argus deleted branch loop/752-command-pages 2026-11-13 18:09:29 +00:00
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
torbscript/language!757
No description provided.