Sprint 46: Polish coaching tree visuals

[KevinBigham]

Summary

• Adds six hand-crafted coaching archetype SVG glyphs and maps existing coach archetype strings into visual families.
• Adds glyphs inside the existing CoachingTree coach cards while preserving the current card text and lineage rendering.
• Adds a passive SVG connection overlay for mentor and disciple relationships with gold direct-mentor, dashed indirect, and cyan disciple lines.
• Adds renderToStaticMarkup coverage for glyph variants, connection lines, relationship styling, and existing tree surface behavior.

Save schema impact

None. SAVE_VERSION remains 35; no engine/save/migration/fixture files changed.

How to test

• NODE_OPTIONS=--no-experimental-webstorage npx --yes pnpm@9.15.9 -r typecheck
• NODE_OPTIONS=--no-experimental-webstorage npx --yes pnpm@9.15.9 test
• NODE_OPTIONS=--no-experimental-webstorage npx --yes pnpm@9.15.9 --filter @mfd/web build
• bash scripts/check-bundle-size.sh
• bash scripts/check-math-random.sh

Look for all commands to pass, index gzip below 181 KB, engine bundle gate unchanged at 257 KB, and no Math.random violations.

Risks / handoff for Claude Code

• First full pnpm test attempt timed out in long engine integration files under local worker load; the timed-out files passed in isolation, then the full pnpm test gate passed on rerun.
• CoachingTree logic is preserved; the diff wraps the existing three-column layout only to add a passive SVG overlay.
• No packages/engine changes.

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 61e585ee6f

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[chatgpt-codex-connector]

Clamp disciple connector coordinates to the SVG viewport

The disciple path formula pushes Y coordinates beyond the viewBox height once there are many disciples (31 + idx * 10 exceeds 100 at idx >= 7), so later disciples lose their connector lines even though cards still render. This affects long-running saves where a head coach has 8+ disciples and makes the new relationship overlay incomplete for valid data.

Useful? React with 👍 / 👎.