fix(web-shell): keep pasted text visible

packages/web-shell/client/e2e/web-shell.smoke.spec.ts

@@ -69,23 +69,23 @@ test('submits a prompt and renders a streamed assistant response @smoke', async
   );
 });
 
-test('pastes long plain text as a placeholder and expands it on submit @smoke', async ({
+test('pastes long plain text as editable composer content @smoke', async ({
   page,
 }, testInfo) => {
   const scenario = createWebShellDaemonScenario();
   const daemon = await installScenario(page, scenario, testInfo);
   const pasted = `${'original '.repeat(151)}end`;
-  const placeholder = `[Pasted Content ${pasted.length} chars]`;
   const edited = `${pasted} edited`;
 
   await gotoSession(page, scenario, daemon);
   await pasteComposerText(page, pasted);
 
   const editor = page.locator('[data-web-shell-composer-editor] .cm-content');
-  await expect(editor).toHaveText(placeholder);
+  await expect(editor).toHaveText(pasted);
+  await expect(editor).not.toContainText('Pasted Content');
 
   await page.keyboard.type(' edited');
-  await expect(editor).toHaveText(`${placeholder} edited`);
+  await expect(editor).toHaveText(edited);
   await page.locator('[data-web-shell-composer-submit]').click();
 
   await expect.poll(() => daemon.promptRequests().length).toBe(1);

packages/web-shell/client/hooks/useComposerCore.dom.test.tsx

@@ -743,6 +743,26 @@ describe('useComposerCore history and drafts', () => {
   });
 });
 
+describe('useComposerCore paste', () => {
+  it('lets long plain text paste directly into the editor', async () => {
+    await mount();
+    const event = new Event('paste', { bubbles: true, cancelable: true });
+    Object.defineProperty(event, 'clipboardData', {
+      value: {
+        items: [{ type: 'text/plain', getAsFile: () => null }],
+        getData: () => 'line\n'.repeat(200),
+      },
+    });
+
+    act(() => {
+      container!.querySelector('.cm-content')!.dispatchEvent(event);
+    });
+
+    expect(latest!.getText()).toBe('line\n'.repeat(200));
+    expect(latest!.getText()).not.toContain('Pasted Content');
+  });
+});
+
 describe('useComposerCore tags', () => {
   it('keeps the composer API stable across tag updates', async () => {
     await mount();

packages/web-shell/client/hooks/useComposerCore.test.ts

@@ -2,15 +2,10 @@ import { describe, expect, it } from 'vitest';
 import {
   buildComposerPrompt,
   buildComposerPromptWithInlineTagPlacements,
-  createLargePastePlaceholder,
-  expandLargePastePlaceholders,
   getComposerTagDisplay,
   getComposerTagLabel,
   getComposerTagValue,
   getFollowupCompletion,
-  isLargePaste,
-  normalizePastedText,
-  prunePendingPastes,
   replaceInlineTagPlacements,
   serializeComposerTag,
 } from './useComposerCore';
@@ -122,104 +117,3 @@ describe('composer tag serialization', () => {
     ).toBe('a <one /> and <two />');
   });
 });
-
-describe('large paste helpers', () => {
-  it('normalizes CRLF and CR line endings to LF', () => {
-    expect(normalizePastedText('a\r\nb\rc\n')).toBe('a\nb\nc\n');
-  });
-
-  it('treats pastes over 1000 chars or 10 lines as large', () => {
-    expect(isLargePaste('a'.repeat(1000))).toBe(false);
-    expect(isLargePaste('a'.repeat(1001))).toBe(true);
-    // 10 lines => 9 newlines => split length 10, not large.
-    expect(isLargePaste('a\n'.repeat(9) + 'a')).toBe(false);
-    // 11 lines => split length 11, large.
-    expect(isLargePaste('a\n'.repeat(10) + 'a')).toBe(true);
-  });
-
-  it('counts code points, not UTF-16 units, for the char threshold', () => {
-    // 1001 emoji are 1001 code points but 2002 UTF-16 units.
-    expect(isLargePaste('😀'.repeat(1001))).toBe(true);
-    expect(isLargePaste('😀'.repeat(1000))).toBe(false);
-  });
-
-  it('creates incrementing placeholders keyed by code-point count', () => {
-    const pending = new Map<string, string>();
-    const first = createLargePastePlaceholder(pending, 1, 'hello');
-    expect(first.placeholderText).toBe('[Pasted Content 5 chars]');
-    expect(first.nextPasteId).toBe(2);
-    const second = createLargePastePlaceholder(
-      pending,
-      first.nextPasteId,
-      'hi',
-    );
-    expect(second.placeholderText).toBe('[Pasted Content 2 chars] #2');
-    expect(second.nextPasteId).toBe(3);
-    expect(pending.get('[Pasted Content 5 chars]')).toBe('hello');
-    expect(pending.get('[Pasted Content 2 chars] #2')).toBe('hi');
-  });
-
-  it('prunes placeholders absent from the doc and resets the id when empty', () => {
-    const pending = new Map<string, string>([
-      ['[Pasted Content 5 chars]', 'hello'],
-      ['[Pasted Content 2 chars] #2', 'hi'],
-    ]);
-    // Only the first placeholder remains in the doc.
-    expect(
-      prunePendingPastes(pending, 'x [Pasted Content 5 chars] y'),
-    ).toBeNull();
-    expect(pending.size).toBe(1);
-    expect(pending.has('[Pasted Content 2 chars] #2')).toBe(false);
-    // Removing the last placeholder resets the next id to 1.
-    expect(prunePendingPastes(pending, 'no placeholders here')).toBe(1);
-    expect(pending.size).toBe(0);
-  });
-
-  it('prunes by exact placeholder match, not substring', () => {
-    const pending = new Map<string, string>([
-      ['[Pasted Content 5 chars]', 'aaaaa'],
-      ['[Pasted Content 5 chars] #2', 'bbbbb'],
-    ]);
-    // Only the longer placeholder is in the doc; the shorter one must be
-    // pruned even though it is a substring of the longer one.
-    expect(
-      prunePendingPastes(pending, '[Pasted Content 5 chars] #2'),
-    ).toBeNull();
-    expect(pending.size).toBe(1);
-    expect(pending.has('[Pasted Content 5 chars]')).toBe(false);
-    expect(pending.has('[Pasted Content 5 chars] #2')).toBe(true);
-  });
-
-  it('expands placeholders back to their pasted content', () => {
-    const pending = new Map<string, string>([
-      ['[Pasted Content 5 chars]', 'hello'],
-    ]);
-    expect(
-      expandLargePastePlaceholders(pending, 'a [Pasted Content 5 chars] b'),
-    ).toBe('a hello b');
-    expect(expandLargePastePlaceholders(pending, 'no placeholder')).toBe(
-      'no placeholder',
-    );
-    expect(
-      expandLargePastePlaceholders(new Map(), '[Pasted Content 5 chars]'),
-    ).toBe('[Pasted Content 5 chars]');
-  });
-
-  it('replaces a placeholder that is a substring of another', () => {
-    // "[Pasted Content 5 chars]" is a prefix of "[Pasted Content 5 chars] #2";
-    // the longer placeholder must win regardless of map insertion order.
-    const pending = new Map<string, string>([
-      ['[Pasted Content 5 chars]', 'aaaaa'],
-      ['[Pasted Content 5 chars] #2', 'bbbbb'],
-    ]);
-    expect(
-      expandLargePastePlaceholders(pending, '[Pasted Content 5 chars] #2'),
-    ).toBe('bbbbb');
-    expect(
-      expandLargePastePlaceholders(
-        pending,
-        '[Pasted Content 5 chars] and [Pasted Content 5 chars] #2',
-      ),
-    ).toBe('aaaaa and bbbbb');
-  });
-});