Skip to content

feat: one help-first assistant with map chat and note editing - #69

Merged
barrulus merged 31 commits into
mainfrom
feat/assistant-help-first
Sep 5, 2026
Merged

feat: one help-first assistant with map chat and note editing#69
barrulus merged 31 commits into
mainfrom
feat/assistant-help-first

Conversation

@barrulus

@barrulus barrulus commented Sep 5, 2026

Copy link
Copy Markdown
Owner

Merges three separate assistant lineages into one surface: the upstream help widget, the BYOK map agent, and the Quill notes editor. The dialog opens on Help and keeps the model AI one click away.

What changes

One assistant dialog. #helpAssistant gains a segmented mode control. Help is the upstream gateway widget, unchanged in behaviour. This map is the BYOK agent, moved out of the old AI Chat dialog. Switching modes never rebuilds the other panel, so a half-typed question or an in-flight request survives.

Help first. The ? bubble always opens Help. Only contextual entry points land on This map: the Tools button (relabelled Assistant) and the notes editor's robot button, where the user has already asked for the map assistant.

Model choice behind a click. Model, API key and local-server fields live in a drawer behind a gear. A status line names the current model and whether a key is set. The drawer opens itself only when sending is impossible without it. Storage keys are unchanged, so existing keys and conversations carry over.

Notes editing. The agent gets one mutation tool, write_note, alongside the read-only run. It writes through a new notes-editor bridge (current / write / remove / getSelectionHtml) so an open editor stays in sync, and refuses HTML the editor cannot hold rather than silently forcing raw mode. Every edit appears in the transcript with its own Undo, which restores the previous legend (or removes a created note); whole-note writes clear Quill's undo stack by design, so the assistant does not rely on Ctrl+Z.

Session plumbing. createSession(getConfig, tools) now dispatches any registered tool by name and reports unknown names back to the model as errors. run is the built-in first tool. Map editing later adds tools, not architecture.

Retired. The separate AI Chat dialog and its registry entry. The AI Text Generator stays registered but is no longer reachable from the notes editor.

Also included

  • Merge of feat/notes-editor-quill (TinyMCE replaced by a bundled Quill 2, TinyMCE replacement Azgaar/Fantasy-Map-Generator#1803). That work is a separate upstream PR; it arrives here as the early fork port because the notes bridge sits on it.
  • Version follows versioning.ts to 1.151.3. The previous -fork.1 suffix named a fork desktop release on the older upstream base and does not carry over.
  • Tour prompt moved from bottom: 100px; right: 36px to bottom: 120px; right: 20px: it overlapped the new help bubble by about 8px. Upstream has the same collision.

Verification

  • 102 unit test files, 1068 tests; tsc --noEmit silent; Biome clean; a real npm ci against the merged lockfile.
  • 7 Playwright scenarios in a headless browser against a local build, with both remote services stubbed: help round-trip, mode switching without losing typed text, notes-editor robot button to This map with the note context chip, a write_note edit landing in the note, the editor and the hover box with Undo restoring all three, the missing-key drawer, plus the three notes-editor tests.
  • Not run: the Electron app, and any live provider or gateway call.

Design: docs/superpowers/specs/2026-09-05-help-assistant-ai-merger-design.md. Plan: docs/superpowers/plans/2026-09-05-help-assistant-ai-merger.md.

barrulus and others added 30 commits September 5, 2026 14:08
… ordinary spaces, no th in the tool's tag list
@netlify

netlify Bot commented Sep 5, 2026

Copy link
Copy Markdown

Deploy Preview for bazgaars-fmg ready!

Name Link
🔨 Latest commit d572800
🔍 Latest deploy log https://app.netlify.com/projects/bazgaars-fmg/deploys/6a9c68c7420c5b0008b29808
😎 Deploy Preview https://deploy-preview-69--bazgaars-fmg.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@barrulus
barrulus merged commit 5fba752 into main Sep 5, 2026
9 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants