@linhey/rhine-design 0.9.1 → 0.11.0

package/tests/ReviewBuild.test.tsx

@@ -0,0 +1,59 @@
+// @vitest-environment node
+import { afterEach, describe, expect, it } from 'vitest';
+import { mkdtemp, mkdir, readFile, rm, writeFile } from 'node:fs/promises';
+import { tmpdir } from 'node:os';
+import { join, resolve } from 'node:path';
+import { build, preview } from 'vite';
+import react from '@vitejs/plugin-react';
+import { reviewPlugin, reviewProductionGuard } from '../review/vite.mjs';
+
+const directories: string[] = [];
+async function fixture() { const path = await mkdtemp(join(tmpdir(), 'rhine-review-test-')); directories.push(path); return path; }
+afterEach(async () => { await Promise.all(directories.splice(0).map(path => rm(path, { recursive: true, force: true }))); });
+describe('review build integration', () => {
+  it.each(['/nested/review/', './'])('emits final CSS and usable preview for base %s', async base => {
+    const root = await fixture();
+    await build({ configFile: false, root, base, logLevel: 'silent', plugins: [react(), reviewPlugin({ catalog: resolve('examples/review/http/catalog.tsx') })] });
+    for (const name of ['index.html', 'canvas.html']) {
+      const html = await readFile(join(root, 'dist', name), 'utf8');
+      const stylesheet = html.match(/rel="stylesheet" href="([^\"]+\.css)"/)?.[1];
+      expect(stylesheet?.startsWith(base)).toBe(true);
+      const css = stylesheet?.slice(base.length);
+      expect(css).toBeTruthy();
+      expect(await readFile(join(root, 'dist', css!), 'utf8')).toContain('.rhine-review-header');
+      expect(html).toContain("connect-src 'none'");
+      expect(html).toContain(`src="${base}assets/`);
+    }
+    const server = await preview({ configFile: false, root, base, logLevel: 'silent', plugins: [reviewPlugin({ catalog: resolve('examples/review/http/catalog.tsx') })], preview: { host: '127.0.0.1', port: 0 } });
+    try {
+      const address = server.httpServer.address();
+      if (!address || typeof address === 'string') throw new Error('Missing preview port');
+      const url = `http://127.0.0.1:${address.port}${base === './' ? '/' : base}`;
+      const response = await fetch(url);
+      expect(response.status).toBe(200);
+      expect(await response.text()).toContain('Rhine Review');
+      expect((await fetch(`${url}api/missing`)).status).toBe(404);
+    } finally { await new Promise<void>((resolve, reject) => server.httpServer.close(error => error ? reject(error) : resolve())); }
+  }, 20000);
+  it('allows production theme directories but rejects explicit fixture paths', async () => {
+    const root = await fixture();
+    await mkdir(join(root, 'design')); await mkdir(join(root, 'fixtures'));
+    await writeFile(join(root, 'design/theme.js'), 'export const theme = "synthetic-theme";');
+    await writeFile(join(root, 'fixtures/data.js'), 'export const data = "synthetic-fixture";');
+    const entry = join(root, 'entry.js');
+    const options = { configFile: false as const, root, logLevel: 'silent' as const, plugins: [reviewProductionGuard({ forbidden: ['./fixtures'] })], build: { lib: { entry, formats: ['es' as const] } } };
+    await writeFile(entry, 'export {theme} from "./design/theme.js";');
+    await expect(build(options)).resolves.toBeTruthy();
+    await writeFile(entry, 'export {data} from "./fixtures/data.js";');
+    await expect(build(options)).rejects.toThrow('forbidden fixtures: fixtures/data.js');
+  }, 20000);
+  it('rejects accidental official runtime inclusion without a product denylist', async () => {
+    const root = await fixture(); const entry = join(root, 'entry.js');
+    await writeFile(entry, `export { defineReview } from ${JSON.stringify(resolve('review/index.ts'))};`);
+    await expect(build({ configFile: false, root, logLevel: 'silent', plugins: [reviewProductionGuard()], build: { lib: { entry, formats: ['es'] } } })).rejects.toThrow('Production bundle imports review runtime');
+  }, 20000);
+  it('refuses backend proxy and project env configuration', async () => {
+    const root = await fixture();
+    await expect(build({ configFile: false, root, logLevel: 'silent', plugins: [reviewPlugin({ catalog: './catalog.tsx' })], server: { proxy: { '/api': 'http://synthetic.invalid' } } })).rejects.toThrow('no server/preview proxies');
+  });
+});

package/tests/ReviewShell.test.tsx

@@ -0,0 +1,63 @@
+import { act, cleanup, fireEvent, render, screen } from '@testing-library/react';
+import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';
+import { ReviewShell, reviewWaitMs } from '../review/shell';
+import { appendFeedback, emptyFeedback } from '../review/feedback';
+import { protocol } from '../review/protocol';
+
+const catalog = { title: '故障恢复', scenes: [{ id: 'a', title: '场景 A' }, { id: 'b', title: '场景 B' }] };
+function iframe() { return screen.getByTitle('产品场景画布') as HTMLIFrameElement; }
+function send(data: Record<string, unknown>, source = iframe().contentWindow!) {
+  fireEvent(window, new MessageEvent('message', { origin: location.origin, source, data: { protocol, ...data } }));
+}
+function connect() {
+  const post = vi.spyOn(iframe().contentWindow!, 'postMessage');
+  send({ type: 'ready', catalog });
+  return { post, request: () => post.mock.calls.filter(call => call[0].type === 'select').at(-1)![0].requestId as number };
+}
+beforeEach(() => { vi.useFakeTimers(); history.replaceState(null, '', '#scene=a'); });
+afterEach(() => { cleanup(); vi.useRealTimers(); });
+describe('review shell recovery', () => {
+  it('offers recovery when the initial iframe never handshakes', () => {
+    render(<ReviewShell />); const oldFrame = iframe();
+    act(() => vi.advanceTimersByTime(reviewWaitMs));
+    expect(screen.getByRole('alert')).toHaveTextContent('连接画布超时');
+    fireEvent.click(screen.getByRole('button', { name: '重新载入画布' }));
+    expect(iframe()).not.toBe(oldFrame);
+    const { request } = connect(); send({ type: 'rendered', requestId: request() });
+    expect(screen.getByRole('status')).toHaveTextContent('已就绪');
+    expect(screen.getByRole('alert')).toHaveTextContent('连接画布超时');
+  });
+  it('replaces a stuck browsing context, keeps selection, ignores old acknowledgements', () => {
+    render(<ReviewShell />); const { request } = connect();
+    send({ type: 'rendered', requestId: request() });
+    fireEvent.click(screen.getByRole('link', { name: '场景 B' }));
+    const pending = request();
+    send({ type: 'rendered', requestId: pending - 1 });
+    act(() => vi.advanceTimersByTime(reviewWaitMs));
+    expect(screen.getByRole('alert')).toHaveTextContent('初始化或清理超过');
+    const oldFrame = iframe(), oldWindow = oldFrame.contentWindow!;
+    fireEvent.click(screen.getByRole('button', { name: '重新载入画布' }));
+    expect(oldFrame.isConnected).toBe(false); expect(location.hash).toContain('scene=b');
+    send({ type: 'feedback', kind: 'error', message: 'stale error' }, oldWindow);
+    expect(screen.queryByText(/stale error/)).not.toBeInTheDocument();
+    const next = connect(); send({ type: 'rendered', requestId: next.request() });
+    expect(screen.getByRole('status')).toHaveTextContent('已就绪');
+  });
+  it('retains scoped errors across ready and selection until explicit acknowledgement', () => {
+    render(<ReviewShell />); const { request } = connect();
+    send({ type: 'feedback', scene: 'a', kind: 'error', message: '清理失败 A' });
+    send({ type: 'rendered', requestId: request() });
+    fireEvent.click(screen.getByRole('link', { name: '场景 B' }));
+    send({ type: 'rendered', requestId: request() });
+    expect(screen.getByRole('alert')).toHaveTextContent('清理失败 A');
+    fireEvent.click(screen.getByText(/评审反馈 ·/));
+    expect(screen.getByText('场景 A', { selector: 'strong' })).toBeInTheDocument();
+    fireEvent.click(screen.getByRole('button', { name: '确认并清除反馈' }));
+    expect(screen.queryByRole('alert')).not.toBeInTheDocument();
+  });
+  it('does not silently lose unacknowledged errors when bounded history overflows', () => {
+    let log = emptyFeedback();
+    for (let id = 0; id < 30; id++) log = appendFeedback(log, { id, kind: id < 4 ? 'error' : 'info', message: String(id), scene: 'a' });
+    expect(log.entries).toHaveLength(20); expect(log.errors).toBe(4); expect(log.omittedErrors).toBe(4);
+  });
+});

package/tsconfig.json

@@ -20,13 +20,16 @@
     "types": [
       "vite/client",
       "node"
-    ]
+    ],
+    "allowJs": true
   },
   "include": [
     "components",
     "preview",
     "tests",
     "vite.config.ts",
-    "vitest.config.ts"
+    "vitest.config.ts",
+    "review",
+    "examples/review"
   ]
 }