test(desktop): move the composer slash, paste and Skill-draft journeys off Electron

apps/desktop/e2e/fixture-thread-search.spec.ts

@@ -25,7 +25,7 @@ import {
 import { expect, test } from './fixtures.js';
 
 test('fixture-seeded transcripts return content hits with turn ids', async ({
-promptRailWindow: page,
+threadSearchWindow: page,
 }) => {
  const outcome = await page.evaluate(async () =>
  window.maka.search.thread({

apps/desktop/e2e/fixtures.ts

@@ -571,6 +571,7 @@ type E2eTestFixtures = {
  parentRemovalWindow: Page;
  railRenderWindow: Page;
  promptRailWindow: Page;
+ threadSearchWindow: Page;
  partialHistoryWindow: Page;
  requestHeaderRowWindow: Page;
  permissionCenterWindow: Page;
@@ -753,6 +754,18 @@ export const test = base.extend<E2eTestFixtures, E2eWorkerFixtures>({
  await setPromptRailWindowVisible(promptRailWorker, false);
  }
  },
+ // The same seeded transcript, on a window of its own. Search reads the Host
+ // through the bridge and renders nothing, so it needs neither the warm
+ // window's compositor nor its between-test reset — and taking it off the
+ // reused window is what retires the readiness gate's cross-test bleed (#4707).
+ threadSearchWindow: async ({}, use) => {
+ await withE2eWindow({
+ seed: false,
+ readinessSelector: '[data-turn-id]',
+ e2eFixtureScenario: 'chat-prompt-rail',
+ locale: 'zh-CN',
+ }, use);
+ },
  // A transcript larger than the bounded Desktop range. Clicking an unloaded
  // prompt exercises the real load-around path and its partial-history UI.
  partialHistoryWindow: async ({}, use) => {

apps/desktop/e2e/new-task-reload.spec.ts

@@ -17,15 +17,18 @@
  * under the License.
  */
 
-import { COMPOSER_INPUT, ensureSidebarExpanded, expect, test } from './fixtures';
+import { COMPOSER_INPUT, awaitSendReady, ensureSidebarExpanded, expect, test } from './fixtures';
 
 test('an explicit new task survives a renderer reload without reopening history', async ({
  window: page,
 }) => {
  const composer = page.locator(COMPOSER_INPUT);
  await composer.fill('create history');
+ await awaitSendReady(page);
  await composer.press('Enter');
- await expect(page.getByText(/Fake backend received: create history/)).toBeVisible();
+ await expect(page.getByText(/Fake backend received: create history/)).toBeVisible({
+ timeout: 20_000,
+ });
 
  await ensureSidebarExpanded(page);
  await page.getByRole('button', { name: '新任务', exact: true }).click();

apps/desktop/e2e/slash-command-compact.spec.ts

@@ -0,0 +1,108 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements. See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership. The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License. You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied. See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+
+/**
+ * What the slash menu's move to Storybook could not take with it.
+ *
+ * The stories cover the menu: which rows a state offers, what picking one
+ * writes, which slashes are triggers at all. All of that is the composer, and
+ * none of it needs Electron. Two claims underneath it do, and this is them.
+ *
+ * The first is the binding between a shell that has a Session and a menu that
+ * offers the commands needing one. The stories are handed `hasSession`, so
+ * `Boolean(activeId)` in app-shell is theirs to assume, not to check — and an
+ * active Session silently losing `/compact` and `/side` would pass every one
+ * of them.
+ *
+ * The second is that `/compact` compacts. Selecting it only writes an
+ * invocation into the draft; what happens when that draft is submitted is a
+ * Host round trip — `sessions.compact()`, a status change, a cleared composer
+ * — and the thing it must not do is reach the model as an ordinary message.
+ * That routing lives inline in `app-shell.tsx`, with no seam under it to hang
+ * a renderer test on, and opening one there is what the architecture ratchet
+ * exists to refuse.
+ */
+
+import { awaitSendReady, COMPOSER_INPUT, expect, test } from './fixtures';
+
+test('a session gets the commands that need one, and /compact compacts it', async ({
+ invocableSkillsWindow: page,
+}) => {
+ const composer = page.locator(COMPOSER_INPUT);
+ await composer.fill('seed session');
+ await awaitSendReady(page);
+ await composer.press('Enter');
+ await expect(page.getByText('Fake backend received: seed session')).toBeVisible({
+ timeout: 20_000,
+ });
+
+ const sessionId = await page.evaluate(async () => (await window.maka.sessions.list())[0]?.id);
+ expect(sessionId).toBeTruthy();
+ await page.evaluate((activeSessionId) => {
+ const testWindow = window as typeof window & {
+ __makaObservedCompactCompletion?: boolean;
+ };
+ testWindow.__makaObservedCompactCompletion = false;
+ window.maka.sessions.subscribeChanges((event) => {
+ if (event.reason === 'status-change' && event.sessionId === activeSessionId) {
+ testWindow.__makaObservedCompactCompletion = true;
+ }
+ });
+ }, sessionId!);
+
+ await composer.click();
+ await composer.pressSequentially('/');
+
+ const menu = page.getByRole('listbox', { name: '命令和技能' });
+ const commands = menu.getByRole('group', { name: '命令' });
+ // Four, not the two a shell without a Session offers: this is the binding a
+ // story cannot see, because a story is handed the answer.
+ await expect(commands.getByRole('option')).toHaveCount(4);
+
+ await commands.getByRole('option', { name: /压缩上下文.*\/compact/ }).click();
+ await expect.poll(() => composer.textContent()).toBe('/compact ');
+ await composer.press('Enter');
+
+ await expect
+ .poll(() =>
+ page.evaluate(
+ () =>
+ (window as typeof window & { __makaObservedCompactCompletion?: boolean })
+ .__makaObservedCompactCompletion,
+ ),
+ )
+ .toBe(true);
+ await expect.poll(() => composer.textContent()).toBe('');
+ await expect(page.getByText('压缩失败')).toHaveCount(0);
+
+ // After the compact completes the composer clears and can remount. `fill()`
+ // can land before the contentEditable is focused again, so the draft never
+ // populates and Enter submits nothing — the flake in issue #3289. Type
+ // through the focused element and require the draft to have settled before
+ // dispatching.
+ await composer.click();
+ await composer.pressSequentially('after compact');
+ await expect.poll(() => composer.textContent()).toBe('after compact');
+ await awaitSendReady(page);
+ await composer.press('Enter');
+ await expect(page.getByText('Fake backend received: after compact')).toBeVisible({
+ timeout: 20_000,
+ });
+ await expect(page.getByText('Fake backend received: /compact')).toHaveCount(0);
+});