docs: working install URLs, a book sweep, and an English sidebar label - #331
Merged
Merged
Conversation
The one-line installers pointed at /releases/latest/, which GitHub resolves
to nothing while every release is a pre-release, and at
termherd-installer.{sh,ps1}, while dist names them after the package
(termherd-app-installer.*). Both halves 404'd. The commands now name the tag
and the real file. The attestation example had the same wrong tarball name,
and the SHA256SUMS it promised only exists from the next release onward.
A sweep of the rest of the book against the source fixed:
- README: prompt_in_session missing from the live-bridge table, and still
listed as a pending follow-up although it shipped (#196).
- The mcp.allow_claude_nesting setting was in none of the settings pages,
the template, the README list, or the file-only enumerations.
- mcp/keyboard.md: the press step field is `result`, not `outcome`, and the
close prompt reports as `tab-close-confirm`.
- termherd://keys/schema lists neither activate-tab-N nor a default for
copy/paste; three pages claimed the whole catalogue.
- snapshot's focused_terminal argument was undocumented.
- quick-start claimed splits resize by keyboard; nothing resizes them.
- tabs-and-splits gave Shift+Right for a Cmd/Ctrl+Shift+Right chord.
- AGENTS.md counted sixteen live-bridge tools; there are seventeen.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The sidebar's collapse button was the one user-facing label left outside
strings.rs, hard-coded in French ("◀ Masquer le panneau") while the book
shows "◀ Hide". It now reads strings::SIDEBAR_HIDE.
Doc wording the audit flagged as doubtful, now settled against the code:
- mcp/keyboard.md: the second rename that ignores `enter` is tab-rename;
there is no rename in the doc pane.
- contributing.md / book.toml: create-missing = false checks SUMMARY.md
entries only, not links inside a page.
- reference/keyboard.md: list the modifier aliases and the `plus` key.
- README / terminal.md: the copy chord asks the terminal for its selection,
including one scrolled out of view, rather than reading the screen.
.serena/ (local state of the Serena MCP tool) is now ignored.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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
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.
Why
Installing on Linux with the documented one-liner failed:
Two independent causes, either one enough:
/releases/latest/skipspre-releases, so it resolves to nothing;
termherd-app-installer.sh/.ps1, nottermherd-installer.*.What
Install (
README.md,docs/src/guide/installation.md): the one-linersname the tag and the real file. All three documented URLs checked at 200. The
attestation example used the wrong tarball name too, and the
SHA256SUMSitpromised comes from a signing workflow added after
v0.1.0-prerelease.4, sono published release has it yet — the page now says so and points at the
.sha256/sha256.sumfiles that do exist.Book sweep against the source:
prompt_in_session, and still listed it asa pending follow-up (F-mcp-agent-loop: composed prompt->wait->read over any session (shell or Claude) #196 shipped it).
mcp.allow_claude_nestingwas missing from the settings reference, thetemplate, the README list, the file-only enumerations and the complete
example.
mcp/keyboard.md: the step field isresult, notoutcome; the closeprompt reports as
tab-close-confirm; the second rename ignoringenteristab-rename.termherd://keys/schemaomitsactivate-tab-Nand shows no default forcopy/paste; three pages claimed the whole catalogue.snapshot'sfocused_terminalargument was undocumented.Shift+→forCmd/Ctrl+Shift+→.plus.create-missing = falsechecksSUMMARY.mdonly, not in-page links.view) rather than reading the screen.
Code: the sidebar's collapse button was hard-coded in French
(
"◀ Masquer le panneau") outsidestrings.rs; it now readsstrings::SIDEBAR_HIDE="◀ Hide", as the book shows.Chore:
.serena/(Serena MCP tool state) is gitignored.Checks
markdownlint-cli2: 0 issuescargo fmt --all --check,cargo clippy -p termherd-app --all-targets -- -D warnings: cleancargo test -p termherd-app: greenmdbook buildnot run locally (not installed) — left to thebookjob🤖 Generated with Claude Code