fix(webapp): the [[ menu finds maps, and the preview draws a map embed (#1277) #1278

Merged
claude-bot merged 3 commits from fix/map-embed-link-menu into main 2026-10-29 20:00:39 +00:00
Collaborator

Closes #1277.

  • [[ menu: EditorPane loads the campaign's maps with the existing mapEmbedLoader, once, the first time the menu opens.
    • The new editorText.acMaps(query, maps) decides what to offer. After Map: it offers only maps whose name contains the rest (up to 6), with no "Create" option. Otherwise it offers up to 3 maps whose name starts with what's typed, after the pages.
    • Options read "Map: " with the type "Map", and the screen-reader live line counts maps as well as pages.
  • Preview: the editor's preview passes campaignId to WikiText, so ![[Map: X]] draws the map instead of showing it as a page waiting to be written.
  • Changelog fragment 1277-map-embed-link-menu.fixed.md.

Checked: vitest over src/pages/wiki and src/wiki (512 passed), with new tests for acMaps, for the menu after Map: and among pages, and for the preview drawing the embed. eslint reports 0 errors.
Not checked: not seen in a browser.

🤖 Generated with Claude Code

Closes #1277. - **`[[` menu:** `EditorPane` loads the campaign's maps with the existing `mapEmbedLoader`, once, the first time the menu opens. - The new `editorText.acMaps(query, maps)` decides what to offer. After `Map:` it offers only maps whose name contains the rest (up to 6), with no "Create" option. Otherwise it offers up to 3 maps whose name starts with what's typed, after the pages. - Options read "Map: <name>" with the type "Map", and the screen-reader live line counts maps as well as pages. - **Preview:** the editor's preview passes `campaignId` to `WikiText`, so `![[Map: X]]` draws the map instead of showing it as a page waiting to be written. - Changelog fragment `1277-map-embed-link-menu.fixed.md`. **Checked:** vitest over `src/pages/wiki` and `src/wiki` (512 passed), with new tests for `acMaps`, for the menu after `Map:` and among pages, and for the preview drawing the embed. eslint reports 0 errors. **Not checked:** not seen in a browser. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
fix(webapp): the [[ menu finds maps, and the preview draws a map embed (#1277)
All checks were successful
CI / Bot/backend version sync (pull_request) Successful in 41s
CI / Docker image build (pull_request) Successful in 43s
CI / Backend lint (ruff) (pull_request) Successful in 48s
CI / Does this change touch the recording path? (pull_request) Successful in 17s
CI / Synthetic session harness (no GPU, no LLM) (pull_request) Has been skipped
CI / Which journeys does this change need? (pull_request) Successful in 18s
CI / Summarisation accuracy eval harness (stub provider) (pull_request) Successful in 1m13s
CI / Feedback site tests (pull_request) Successful in 2m16s
CI / Bot tests and audit (pull_request) Successful in 2m23s
CI / Frontend (shard 2/2) (pull_request) Successful in 7m55s
CI / End-to-end journeys (desktop) (pull_request) Successful in 13m29s
CI / Frontend (shard 1/2) (pull_request) Successful in 17m1s
CI / Backend migration, tests, and audit (pull_request) Successful in 34m19s
9de1014bda
The menu searched only pages, so ![[Map: … offered to create a page;
it now loads the campaign's maps once, on first opening, and after Map:
offers only maps. The editor's preview rendered WikiText without the
campaign id, which map embeds need, so it showed a wanted page.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Merge remote-tracking branch 'origin/main' into fix/map-embed-link-menu
Some checks failed
CI / Summarisation accuracy eval harness (stub provider) (pull_request) Successful in 1m6s
CI / Feedback site tests (pull_request) Successful in 44s
CI / Bot/backend version sync (pull_request) Successful in 29s
CI / Backend lint (ruff) (pull_request) Successful in 1m23s
CI / Does this change touch the recording path? (pull_request) Successful in 16s
CI / Synthetic session harness (no GPU, no LLM) (pull_request) Has been skipped
CI / Which journeys does this change need? (pull_request) Successful in 18s
CI / Bot tests and audit (pull_request) Successful in 2m17s
CI / Docker image build (pull_request) Successful in 2m34s
CI / Frontend (shard 2/2) (pull_request) Successful in 9m42s
CI / Frontend (shard 1/2) (pull_request) Successful in 11m59s
CI / End-to-end journeys (desktop) (pull_request) Failing after 4m6s
CI / Backend migration, tests, and audit (pull_request) Has been cancelled
a0653a1236
claude-bot scheduled this pull request to auto merge when all checks succeed 2026-10-29 17:56:00 +00:00
ci: re-run (the runner's disk was full building the e2e images)
All checks were successful
CI / Summarisation accuracy eval harness (stub provider) (pull_request) Successful in 1m0s
CI / Backend lint (ruff) (pull_request) Successful in 33s
CI / Feedback site tests (pull_request) Successful in 47s
CI / Bot/backend version sync (pull_request) Successful in 24s
CI / Does this change touch the recording path? (pull_request) Successful in 13s
CI / Synthetic session harness (no GPU, no LLM) (pull_request) Has been skipped
CI / Which journeys does this change need? (pull_request) Successful in 16s
CI / Docker image build (pull_request) Successful in 44s
CI / Bot tests and audit (pull_request) Successful in 2m10s
CI / Frontend (shard 2/2) (pull_request) Successful in 7m51s
CI / Frontend (shard 1/2) (pull_request) Successful in 9m1s
CI / End-to-end journeys (desktop) (pull_request) Successful in 12m7s
CI / Backend migration, tests, and audit (pull_request) Successful in 1h8m23s
0e8fe4312f
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
claude-bot deleted branch fix/map-embed-link-menu 2026-10-29 20:00:40 +00:00
Sign in to join this conversation.
No description provided.