Merge pull request #3 from livesession/feat/git-native

feat(git-native): 'native' git impl

Author: zdunecki

.github/workflows/tests.yml

@@ -0,0 +1,47 @@
+name: Tests
+
+on:
+  push:
+    branches: [main]
+  pull_request:
+    branches: [main]
+
+jobs:
+  unit:
+    name: Unit Tests
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+
+      - uses: pnpm/action-setup@v4
+        with:
+          version: 9
+
+      - uses: actions/setup-node@v4
+        with:
+          node-version: 22
+          cache: pnpm
+
+      - run: pnpm install
+      - run: pnpm run build:lib
+      - run: pnpm test
+
+  e2e:
+    name: E2E Tests
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+
+      - uses: pnpm/action-setup@v4
+        with:
+          version: 9
+
+      - uses: actions/setup-node@v4
+        with:
+          node-version: 22
+          cache: pnpm
+
+      - run: pnpm install
+      - run: pnpm run build:lib
+      - run: npx playwright install --with-deps chromium
+      - run: pnpm test:e2e

.gitignore

@@ -35,5 +35,8 @@ pnpm-debug.log*
 # Test coverage
 coverage/
 
+# Test results
+test-results/
+
 # Local research / planning notes (Claude Code scratch)
 memory/

package.json

@@ -84,6 +84,7 @@
     "build:publish": "npm run build:lib && npm run build:types",
     "type-check": "tsc --noEmit",
     "test": "vitest run",
+    "test:e2e": "npx playwright test",
     "test:watch": "vitest",
     "bench": "vitest bench",
     "bench:run": "vitest bench --run",
@@ -95,17 +96,19 @@
     "brotli": "^1.3.3",
     "brotli-wasm": "^3.0.1",
     "comlink": "^4.4.2",
+    "isomorphic-git": "^1.37.5",
     "pako": "^2.1.0",
     "resolve.exports": "^2.0.3",
+    "sha.js": "^2.4.12",
     "zod": "^4.3.6"
   },
   "peerDependencies": {
-    "@xterm/xterm": "^6.0.0",
     "@xterm/addon-fit": "^0.11.0",
     "@xterm/addon-webgl": "^0.19.0",
     "@xterm/addon-serialize": "^0.14.0",
-    "vite": "^5.0.0 || ^6.0.0 || ^7.0.0 || ^8.0.0",
-    "next": "^13.0.0 || ^14.0.0 || ^15.0.0 || ^16.0.0"
+    "@xterm/xterm": "^6.0.0",
+    "next": "^13.0.0 || ^14.0.0 || ^15.0.0 || ^16.0.0",
+    "vite": "^5.0.0 || ^6.0.0 || ^7.0.0 || ^8.0.0"
   },
   "peerDependenciesMeta": {
     "@xterm/xterm": {
@@ -131,10 +134,14 @@
     "@types/node": "^25.0.10",
     "@types/pako": "^2.0.4",
     "esbuild": "^0.27.2",
+    "tsx": "^4.21.0",
     "typescript": "^5.9.3",
-    "vite": "^5.4.0",
+    "@playwright/test": "^1.53.1",
+    "@vitest/browser": "^4.1.5",
+    "playwright": "^1.53.1",
+    "vite": "^6.0.0",
     "vite-plugin-top-level-await": "^1.6.0",
     "vite-plugin-wasm": "^3.5.0",
-    "vitest": "^4.0.18"
+    "vitest": "^4.1.5"
   }
 }

playwright.config.ts

@@ -0,0 +1,34 @@
+import { defineConfig } from '@playwright/test';
+
+export default defineConfig({
+  testDir: './src/__tests__/e2e',
+  timeout: 60000,
+  projects: [
+    {
+      name: 'git-clone',
+      testDir: './src/__tests__/e2e/git-clone',
+      testMatch: '*.test.ts',
+      use: { baseURL: 'http://localhost:4567' },
+    },
+    {
+      name: 'git-clone-api',
+      testDir: './src/__tests__/e2e/git-clone-api',
+      testMatch: '*.test.ts',
+      use: { baseURL: 'http://localhost:4568' },
+    },
+  ],
+  webServer: [
+    {
+      command: 'npx vite --config src/__tests__/e2e/git-clone/vite.config.ts',
+      port: 4567,
+      reuseExistingServer: true,
+      timeout: 30000,
+    },
+    {
+      command: 'npx vite --config src/__tests__/e2e/git-clone-api/vite.config.ts',
+      port: 4568,
+      reuseExistingServer: true,
+      timeout: 30000,
+    },
+  ],
+});

src/__tests__/e2e/git-clone-api/git-clone-api.test.ts

@@ -0,0 +1,55 @@
+import { test, expect } from '@playwright/test';
+import { startMockGithub } from './mock-github';
+import { boot, gitClone, listFiles, readFile } from '../utils/page-helpers';
+import type { Server } from 'node:http';
+
+let mockServer: Server;
+
+test.beforeAll(async () => {
+  mockServer = await startMockGithub(4569);
+});
+
+test.afterAll(async () => {
+  mockServer?.close();
+});
+
+test.describe('git clone API mode', () => {
+  test.beforeEach(async ({ page }) => {
+    await page.goto('http://localhost:4568');
+    expect(await boot(page)).toBe(true);
+  });
+
+  test('clones mock repo and checks out files', async ({ page }) => {
+    const result = await gitClone(page, 'https://github.com/test-org/test-repo');
+    expect(result.exitCode).toBe(0);
+
+    const files = await listFiles(page, '/workspace/test-repo');
+    expect(files).toContain('package.json');
+    expect(files).toContain('README.md');
+
+    const pkgContent = await readFile(page, '/workspace/test-repo/package.json');
+    expect(pkgContent).toBeTruthy();
+    const pkg = JSON.parse(pkgContent);
+    expect(pkg.name).toBe('test-repo');
+  });
+
+  test('clones mock repo with subdirectories', async ({ page }) => {
+    const result = await gitClone(page, 'https://github.com/test-org/test-repo');
+    expect(result.exitCode).toBe(0);
+
+    const content = await readFile(page, '/workspace/test-repo/src/index.ts');
+    expect(content).toBeTruthy();
+    expect(content).toContain('hello');
+  });
+
+  test('README.md has expected content', async ({ page }) => {
+    await gitClone(page, 'https://github.com/test-org/test-repo');
+    const readme = await readFile(page, '/workspace/test-repo/README.md');
+    expect(readme).toContain('mock repository');
+  });
+
+  test('clone non-existent repo returns error', async ({ page }) => {
+    const result = await gitClone(page, 'https://github.com/test-org/nonexistent-repo');
+    expect(result.exitCode).not.toBe(0);
+  });
+});

src/__tests__/e2e/git-clone-api/index.html

@@ -0,0 +1,8 @@
+<!DOCTYPE html>
+<html>
+<head><title>nodepod git-clone-api e2e</title></head>
+<body>
+<pre id="output"></pre>
+<script type="module" src="./main.ts"></script>
+</body>
+</html>

src/__tests__/e2e/git-clone-api/main.ts

@@ -0,0 +1,19 @@
+import { Nodepod } from '@scelar/nodepod';
+import { createTestApi } from '../utils/test-api';
+
+const output = document.getElementById('output')!;
+const log = (msg: string) => { output.textContent += msg + '\n'; console.log(msg); };
+
+(window as any).__nodepodTest = createTestApi(
+  () => Nodepod.boot({
+    workdir: '/workspace',
+    git: 'api',
+    gitApiBase: 'http://localhost:4569/api',
+    gitRawBase: 'http://localhost:4569/raw',
+    serviceWorker: false,
+    watermark: false,
+  }),
+  log,
+);
+
+log('[test] fixture loaded (API mode)');

src/__tests__/e2e/git-clone-api/mock-github.ts

@@ -0,0 +1,72 @@
+/**
+ * Mock GitHub API server for testing git clone in API mode.
+ * Serves fake repo metadata, tree, and file contents.
+ */
+import http from 'node:http';
+
+const MOCK_OWNER = 'test-org';
+const MOCK_REPO = 'test-repo';
+
+const files: Record<string, string> = {
+  'package.json': JSON.stringify({ name: 'test-repo', version: '1.0.0', private: true }, null, 2),
+  'README.md': '# Test Repo\n\nThis is a mock repository for testing git clone API mode.',
+  'src/index.ts': 'export const hello = "world";\n',
+};
+
+function handleRequest(req: http.IncomingMessage, res: http.ServerResponse) {
+  res.setHeader('Access-Control-Allow-Origin', '*');
+  res.setHeader('Access-Control-Allow-Headers', '*');
+
+  if (req.method === 'OPTIONS') {
+    res.writeHead(204);
+    res.end();
+    return;
+  }
+
+  const url = req.url || '';
+
+  // GET /api/repos/:owner/:repo
+  if (url === `/api/repos/${MOCK_OWNER}/${MOCK_REPO}`) {
+    res.writeHead(200, { 'Content-Type': 'application/json' });
+    res.end(JSON.stringify({
+      default_branch: 'main',
+      name: MOCK_REPO,
+      full_name: `${MOCK_OWNER}/${MOCK_REPO}`,
+    }));
+    return;
+  }
+
+  // GET /api/repos/:owner/:repo/git/trees/main?recursive=1
+  if (url.startsWith(`/api/repos/${MOCK_OWNER}/${MOCK_REPO}/git/trees/`)) {
+    const tree = Object.keys(files).map(path => ({
+      path,
+      type: 'blob',
+      sha: 'abc123',
+    }));
+    res.writeHead(200, { 'Content-Type': 'application/json' });
+    res.end(JSON.stringify({ tree }));
+    return;
+  }
+
+  // GET /raw/:owner/:repo/:branch/:path — raw file content
+  const rawMatch = url.match(/^\/raw\/[^/]+\/[^/]+\/[^/]+\/(.+)$/);
+  if (rawMatch) {
+    const filePath = rawMatch[1];
+    if (files[filePath]) {
+      res.writeHead(200, { 'Content-Type': 'text/plain' });
+      res.end(files[filePath]);
+      return;
+    }
+  }
+
+  // 404
+  res.writeHead(404, { 'Content-Type': 'application/json' });
+  res.end(JSON.stringify({ message: 'Not Found' }));
+}
+
+export function startMockGithub(port: number): Promise<http.Server> {
+  return new Promise((resolve) => {
+    const server = http.createServer(handleRequest);
+    server.listen(port, () => resolve(server));
+  });
+}

src/__tests__/e2e/git-clone-api/vite.config.ts

@@ -0,0 +1,11 @@
+import { defineConfig } from 'vite';
+import { resolve } from 'path';
+import nodepod from '@scelar/nodepod/vite';
+
+export default defineConfig({
+  root: __dirname,
+  plugins: [nodepod()], // API mode — no git: 'native'
+  server: {
+    port: 4568,
+  },
+});

src/__tests__/e2e/git-clone/git-clone.test.ts

@@ -0,0 +1,41 @@
+import { test, expect } from '@playwright/test';
+import { boot, gitClone, listFiles, readFile, fileExists } from '../utils/page-helpers';
+
+test.describe('git clone e2e', () => {
+  test.beforeEach(async ({ page }) => {
+    await page.goto('http://localhost:4567');
+    expect(await boot(page)).toBe(true);
+  });
+
+  test('clones ScelarOrg/Nodepod and checks out files', async ({ page }) => {
+    const result = await gitClone(page, 'https://github.com/ScelarOrg/Nodepod');
+    expect(result.exitCode).toBe(0);
+
+    const files = await listFiles(page, '/workspace/Nodepod');
+    expect(files).toContain('package.json');
+    expect(files).toContain('README.md');
+
+    const pkgContent = await readFile(page, '/workspace/Nodepod/package.json');
+    expect(pkgContent).toBeTruthy();
+    const pkg = JSON.parse(pkgContent);
+    expect(pkg.name).toBe('@scelar/nodepod');
+  });
+
+  test('README.md contains nodepod description', async ({ page }) => {
+    await gitClone(page, 'https://github.com/ScelarOrg/Nodepod');
+    const readme = await readFile(page, '/workspace/Nodepod/README.md');
+    expect(readme).toBeTruthy();
+    expect(readme.toLowerCase()).toContain('nodepod');
+  });
+
+  test('.git directory is created', async ({ page }) => {
+    await gitClone(page, 'https://github.com/ScelarOrg/Nodepod');
+    expect(await fileExists(page, '/workspace/Nodepod/.git')).toBe(true);
+    expect(await fileExists(page, '/workspace/Nodepod/.git/HEAD')).toBe(true);
+  });
+
+  test('clone non-existent repo returns error', async ({ page }) => {
+    const result = await gitClone(page, 'https://github.com/xyd-js/this-repo-does-not-exist-12345');
+    expect(result.exitCode).not.toBe(0);
+  });
+});