polish(desktop): finish Projects navigation and context chrome

desktop/src-tauri/src/lib.rs

@@ -914,7 +914,6 @@ pub fn run() {
         RunEvent::Exit => {
             shut_down_app(app_handle, &run_shutdown_done);
             app_handle.state::<ClipboardState>().release();
-
             #[cfg(all(feature = "mesh-llm", target_os = "macos"))]
             if restart_requested.load(Ordering::SeqCst) {
                 relaunch_after_mesh_shutdown(app_handle);

desktop/src/features/messages/lib/mentionHighlightExtension.test.mjs

@@ -1,9 +1,19 @@
 import assert from "node:assert/strict";
 import test from "node:test";
 
+import { getSchema } from "@tiptap/core";
+import StarterKit from "@tiptap/starter-kit";
+
 import {
+  assignMentionHighlightNames,
   buildHighlightPatterns,
+  createMentionCaretSettlement,
   findHighlightMatches,
+  insertPosForMentionTextInput,
+  mentionTextInputInsertPos,
+  positionAfterArrowLeftThroughMentionSpace,
+  selectionAfterMentionTrailingSpace,
+  shouldAdvanceMentionCaret,
 } from "./mentionHighlightExtension.ts";
 
 // ── buildHighlightPatterns ────────────────────────────────────────────
@@ -164,3 +174,141 @@ test("#general should NOT match inside #generally (trailing word boundary)", ()
   const matches = findHighlightMatches("#generally", patterns);
   assert.equal(matches.length, 0);
 });
+
+const schema = getSchema([
+  StarterKit.configure({
+    heading: false,
+    trailingNode: false,
+    link: false,
+  }),
+]);
+const paragraph = (...content) => schema.nodes.paragraph.create(null, content);
+const text = (value) => schema.text(value);
+const document = (...content) => schema.nodes.doc.create(null, content);
+
+test("selectionAfterMentionTrailingSpace steps past the space after @Name", () => {
+  const doc = document(paragraph(text("@quinn ")));
+  const spacePos = 1 + "@quinn".length;
+  assert.equal(selectionAfterMentionTrailingSpace(doc, spacePos), spacePos + 1);
+  assert.equal(
+    selectionAfterMentionTrailingSpace(doc, spacePos + 1),
+    spacePos + 1,
+  );
+});
+
+test("selectionAfterMentionTrailingSpace leaves a caret inside the mention name", () => {
+  const doc = document(paragraph(text("@quinn ")));
+  assert.equal(selectionAfterMentionTrailingSpace(doc, 4), 4);
+});
+
+test("selectionAfterMentionTrailingSpace does not move without a trailing space", () => {
+  const doc = document(paragraph(text("@quinn")));
+  const end = 1 + "@quinn".length;
+  assert.equal(selectionAfterMentionTrailingSpace(doc, end), end);
+});
+
+test("shouldAdvanceMentionCaret restores a remap while this editor is settling", () => {
+  assert.equal(
+    shouldAdvanceMentionCaret({
+      from: 7,
+      next: 8,
+      settling: true,
+      docChanged: false,
+    }),
+    true,
+  );
+});
+
+test("shouldAdvanceMentionCaret does not steal ArrowLeft after settlement is cancelled", () => {
+  assert.equal(
+    shouldAdvanceMentionCaret({
+      from: 7,
+      next: 8,
+      settling: false,
+      docChanged: false,
+    }),
+    false,
+  );
+});
+
+test("shouldAdvanceMentionCaret still advances after a document change", () => {
+  assert.equal(
+    shouldAdvanceMentionCaret({
+      from: 7,
+      next: 8,
+      settling: false,
+      docChanged: true,
+    }),
+    true,
+  );
+});
+
+test("createMentionCaretSettlement keeps two editors independent", () => {
+  const composerA = createMentionCaretSettlement();
+  const composerB = createMentionCaretSettlement();
+  composerA.arm(8);
+  assert.equal(composerB.peek(), null);
+  composerB.arm(12);
+  composerA.cancel();
+  assert.equal(composerA.peek(), null);
+  assert.equal(composerB.peek(), 12);
+});
+
+test("insertPosForMentionTextInput redirects a caret at the chip edge", () => {
+  const doc = document(paragraph(text("@quinn ")));
+  const spacePos = 1 + "@quinn".length;
+  assert.equal(
+    insertPosForMentionTextInput(doc, spacePos, spacePos),
+    spacePos + 1,
+  );
+  assert.equal(
+    insertPosForMentionTextInput(doc, spacePos + 1, spacePos + 1),
+    null,
+  );
+});
+
+test("insertPosForMentionTextInput keeps a selected trailing space", () => {
+  const doc = document(paragraph(text("@quinn ")));
+  const spacePos = 1 + "@quinn".length;
+  assert.equal(
+    insertPosForMentionTextInput(doc, spacePos, spacePos + 1),
+    spacePos + 1,
+  );
+});
+
+test("mentionTextInputInsertPos honors a deliberate caret after settlement", () => {
+  const doc = document(paragraph(text("@bob ")));
+  const spacePos = 1 + "@bob".length;
+  assert.equal(mentionTextInputInsertPos(doc, spacePos, spacePos, false), null);
+  assert.equal(
+    mentionTextInputInsertPos(doc, spacePos, spacePos, true),
+    spacePos + 1,
+  );
+});
+
+test("positionAfterArrowLeftThroughMentionSpace steps onto the token end", () => {
+  const doc = document(paragraph(text("@bob ")));
+  const afterSpace = 1 + "@bob".length + 1;
+  assert.equal(
+    positionAfterArrowLeftThroughMentionSpace(doc, afterSpace),
+    afterSpace - 1,
+  );
+  assert.equal(
+    positionAfterArrowLeftThroughMentionSpace(doc, afterSpace - 1),
+    null,
+  );
+});
+
+test("assignMentionHighlightNames skips an unchanged list", () => {
+  const storage = { names: ["bob"], agentNames: [], channelNames: [] };
+  assert.equal(assignMentionHighlightNames(storage, ["bob"], [], []), false);
+});
+
+test("assignMentionHighlightNames updates when a new mention is added", () => {
+  const storage = { names: ["bob"], agentNames: [], channelNames: [] };
+  assert.equal(
+    assignMentionHighlightNames(storage, ["bob", "quinn"], [], []),
+    true,
+  );
+  assert.deepEqual(storage.names, ["bob", "quinn"]);
+});