Integrate and fix contributor PRs #865, #868, #869, #870

[SawyerHood]

Supersedes #865, #868, #869, and #870 — all four reviewed, fixed, and integrated into one branch.

Each original PR was reviewed against a local checkout (not from the diff), and each had defects. This branch keeps every contributor commit with its original authorship and adds one follow-up commit per PR.

What's here

Original	Author	What the review found	Follow-up
#868 Docs YAML frontmatter	@amadad	Any text between two --- lines was treated as frontmatter, so a document opening with a thematic break had its first section hidden from the editor, nav titles, and previews. The blank line after frontmatter was dropped on every save. renameToTitle bailed on any frontmatter. Previews dropped the first heading of any level.	Frontmatter is only recognised when the fenced block parses to a YAML mapping; the body's leading newline is replayed on save; rename gates on the parsed title; only a title-matching heading is excluded from the preview. 47 → 57 tests.
#869 remark-directive colons	@bhuvansingla	Real bug, real fix — but two tests were left red, and the change also rewrote container directives, which stopped nested plugin directives from mounting and collapsed ::: blocks onto one line.	Rewrite narrowed to textDirective; containers left on the previous path; the literal is merged into an adjacent text node; stale assertions updated; regression test added for the reported symptom.
#865 Project machine picker	@amadad	The single-machine gate counted every non-destroyed enrollment, so one machine offline for a month silently replaced the native OS folder picker. Two smaller states could submit into a silent no-op.	Gate counts connected machines; a still-loading host list is treated as not-single-machine; the all-offline state renders an empty state and blocks submit; the target host is now a required parameter.
#870 Quiet skill dedup logs	@amadad	Correct change, three broken tests — the capturing logger stubbed debug out entirely.	Test logger captures debug; the three assertions were retargeted, not deleted.

#867 (Pi SDK upgrade) is deliberately not included. It still has a blocking defect: Pi advertises max reasoning while PI_SERVER_CAPABILITIES caps at xhigh, so setting the override persists and then 400s on the next turn.

Verification

Automated, via Turbo:

pnpm exec turbo run test typecheck --filter=@bb/app --filter=@bb/server --filter=bb-plugin-simple-notes --force

@bb/app                  typecheck clean · tests pass
bb-plugin-simple-notes   3 files · 57 tests passed   (was 47)
@bb/server               146/148 files · 1253/1255 tests

Both server failures are environmental and pass individually: internal-skill-trees.test.ts expects mode: 420 (0644) and gets 436 under umask 0002 — it also fails on main, and passes under umask 022; public-host-management.test.ts hit a 5s per-test timeout under concurrent load and passes alone in 883ms.

Live QA against a running dev instance on this exact tree:

• Fix Docs YAML frontmatter parsing #868 — thematic-break document visible and editable again (2 <hr>, both sections); blank line after frontmatter survives a real autosave, verified with cat -A on the saved bytes; a title-less-frontmatter doc renames to follow its H1 while a frontmatter-titled one stays put; a non-title H2 survives in the preview.
• fix(app): stop remark-directive from eating colons in assistant messages #869 — a real Haiku turn emitting the fixture renders Meeting at 9:30 and 10:45 today. as a single #text node with zero injected empty <div>s, and :::note still wraps a real <strong> in a DIV.
• Let New project select an execution machine #865 — with a second machine seeded 30 days offline, the picker auto-selects the connected machine and renders the offline one disabled behind an Offline badge. The native-picker path itself needs the Electron shell and is covered by hook tests only.
• Demote skill-tier override dedup logs to debug #870 — with a planted bb-cli skill collision, five catalog resolutions produced five override lines, all at level 20, zero at level 30.

Also checked under a dark Nord palette and at a 760px width.

Known, not fixed

Three cosmetic issues that all reproduce on main and are not regressions from this branch:

1. Documents opening with a bare --- still title as --- in the Docs nav — cleanLine only strips list markers when whitespace follows the dash. This is main behaviour and Fix Docs YAML frontmatter parsing #868's fix restores it exactly, but it's the same symptom Fix Docs YAML frontmatter parsing #868 set out to remove.
2. The trailing newline is still dropped when the Docs editor saves — pre-existing Tiptap round-trip behaviour.
3. useQuickCreateProject now gates on connected machines while ProjectPathDialog still computes showMachinePicker from the total. Harmless today; worth unifying.

Separately: none of the five PRs had any CI (gh pr checks reported no checks on every branch), and three were red against main's suite.

Co-authored-by: Ali Madad ali@scty.org
Co-authored-by: Bhuvan Singla bhuvansingla2000@gmail.com

🤖 Generated with Claude Code