oxe-cc 1.0.0 → 1.3.0

package/packages/runtime/src/workspace/strategies/git-worktree.ts

@@ -1,77 +1,79 @@
-import { execFileSync } from 'child_process';
-import path from 'path';
-import fs from 'fs';
-import crypto from 'crypto';
-import type { WorkspaceManager, WorkspaceRequest } from '../workspace-manager';
-import type { WorkspaceLease, SnapshotRef } from '../../models/workspace';
-
-export class GitWorktreeManager implements WorkspaceManager {
-  private leases = new Map<string, WorkspaceLease>();
-
-  constructor(private readonly projectRoot: string) {}
-
-  async allocate(req: WorkspaceRequest): Promise<WorkspaceLease> {
-    const wsId = `ws-${req.work_item_id}-a${req.attempt_number}`;
-    const branch = `oxe/${req.work_item_id}-attempt${req.attempt_number}`;
-    const worktreePath = path.join(this.projectRoot, '.oxe', 'workspaces', wsId);
-
-    const baseCommit = this.git(['rev-parse', 'HEAD']).trim();
-
-    fs.mkdirSync(path.dirname(worktreePath), { recursive: true });
-
-    // Create worktree on a new branch starting from HEAD
-    this.git(['worktree', 'add', worktreePath, '-b', branch]);
-
-    const lease: WorkspaceLease = {
-      workspace_id: wsId,
-      strategy: 'git_worktree',
-      branch,
-      base_commit: baseCommit,
-      root_path: worktreePath,
-      ttl_minutes: 45,
-    };
-    this.leases.set(wsId, lease);
-    return lease;
-  }
-
-  async snapshot(id: string): Promise<SnapshotRef> {
-    const lease = this.leases.get(id);
-    if (!lease || !lease.root_path) throw new Error(`Workspace ${id} not found`);
-    const commit = this.git(['rev-parse', 'HEAD'], lease.root_path).trim();
-    return {
-      snapshot_id: `snap-${crypto.randomBytes(4).toString('hex')}`,
-      workspace_id: id,
-      commit,
-      created_at: new Date().toISOString(),
-    };
-  }
-
-  async reset(id: string, snapRef: SnapshotRef): Promise<void> {
-    const lease = this.leases.get(id);
-    if (!lease) return;
-    this.git(['reset', '--hard', snapRef.commit], lease.root_path);
-  }
-
-  async dispose(id: string): Promise<void> {
-    const lease = this.leases.get(id);
-    if (!lease) return;
-    try {
-      this.git(['worktree', 'remove', lease.root_path, '--force']);
-    } catch {
-      // worktree may already be gone
-    }
-    try {
-      if (lease.branch) this.git(['branch', '-D', lease.branch]);
-    } catch {
-      // branch may already be deleted
-    }
-    this.leases.delete(id);
-  }
-
-  private git(args: string[], cwd?: string): string {
-    return execFileSync('git', args, {
-      cwd: cwd ?? this.projectRoot,
-      encoding: 'utf8',
-    });
-  }
-}
+import { execFileSync } from 'child_process';
+import path from 'path';
+import fs from 'fs';
+import crypto from 'crypto';
+import type { WorkspaceManager, WorkspaceRequest } from '../workspace-manager';
+import type { WorkspaceLease, SnapshotRef } from '../../models/workspace';
+
+export class GitWorktreeManager implements WorkspaceManager {
+  readonly isolation_level = 'isolated' as const;
+  private leases = new Map<string, WorkspaceLease>();
+
+  constructor(private readonly projectRoot: string) {}
+
+  async allocate(req: WorkspaceRequest): Promise<WorkspaceLease> {
+    const wsId = `ws-${req.work_item_id}-a${req.attempt_number}`;
+    const branch = `oxe/${req.work_item_id}-attempt${req.attempt_number}`;
+    const worktreePath = path.join(this.projectRoot, '.oxe', 'workspaces', wsId);
+
+    const baseCommit = this.git(['rev-parse', 'HEAD']).trim();
+
+    fs.mkdirSync(path.dirname(worktreePath), { recursive: true });
+
+    // Create worktree on a new branch starting from HEAD
+    this.git(['worktree', 'add', worktreePath, '-b', branch]);
+
+    const lease: WorkspaceLease = {
+      workspace_id: wsId,
+      strategy: 'git_worktree',
+      isolation_level: this.isolation_level,
+      branch,
+      base_commit: baseCommit,
+      root_path: worktreePath,
+      ttl_minutes: 45,
+    };
+    this.leases.set(wsId, lease);
+    return lease;
+  }
+
+  async snapshot(id: string): Promise<SnapshotRef> {
+    const lease = this.leases.get(id);
+    if (!lease || !lease.root_path) throw new Error(`Workspace ${id} not found`);
+    const commit = this.git(['rev-parse', 'HEAD'], lease.root_path).trim();
+    return {
+      snapshot_id: `snap-${crypto.randomBytes(4).toString('hex')}`,
+      workspace_id: id,
+      commit,
+      created_at: new Date().toISOString(),
+    };
+  }
+
+  async reset(id: string, snapRef: SnapshotRef): Promise<void> {
+    const lease = this.leases.get(id);
+    if (!lease) return;
+    this.git(['reset', '--hard', snapRef.commit], lease.root_path);
+  }
+
+  async dispose(id: string): Promise<void> {
+    const lease = this.leases.get(id);
+    if (!lease) return;
+    try {
+      this.git(['worktree', 'remove', lease.root_path, '--force']);
+    } catch {
+      // worktree may already be gone
+    }
+    try {
+      if (lease.branch) this.git(['branch', '-D', lease.branch]);
+    } catch {
+      // branch may already be deleted
+    }
+    this.leases.delete(id);
+  }
+
+  private git(args: string[], cwd?: string): string {
+    return execFileSync('git', args, {
+      cwd: cwd ?? this.projectRoot,
+      encoding: 'utf8',
+    });
+  }
+}

package/packages/runtime/src/workspace/strategies/inplace.ts

@@ -1,35 +1,38 @@
-import crypto from 'crypto';
-import type { WorkspaceManager, WorkspaceRequest } from '../workspace-manager';
-import type { WorkspaceLease, SnapshotRef } from '../../models/workspace';
-
-export class InplaceWorkspaceManager implements WorkspaceManager {
-  constructor(private readonly projectRoot: string) {}
-
-  async allocate(req: WorkspaceRequest): Promise<WorkspaceLease> {
-    return {
-      workspace_id: `ws-inplace-${req.work_item_id}-a${req.attempt_number}`,
-      strategy: 'inplace',
-      branch: null,
-      base_commit: null,
-      root_path: this.projectRoot,
-      ttl_minutes: 60,
-    };
-  }
-
-  async snapshot(id: string): Promise<SnapshotRef> {
-    return {
-      snapshot_id: `snap-${crypto.randomBytes(4).toString('hex')}`,
-      workspace_id: id,
-      commit: 'HEAD',
-      created_at: new Date().toISOString(),
-    };
-  }
-
-  async reset(_id: string, _snapRef: SnapshotRef): Promise<void> {
-    // inplace: no filesystem isolation — reset is a no-op
-  }
-
-  async dispose(_id: string): Promise<void> {
-    // inplace: nothing to tear down
-  }
-}
+import crypto from 'crypto';
+import type { WorkspaceManager, WorkspaceRequest } from '../workspace-manager';
+import type { WorkspaceLease, SnapshotRef } from '../../models/workspace';
+
+export class InplaceWorkspaceManager implements WorkspaceManager {
+  readonly isolation_level = 'shared' as const;
+
+  constructor(private readonly projectRoot: string) {}
+
+  async allocate(req: WorkspaceRequest): Promise<WorkspaceLease> {
+    return {
+      workspace_id: `ws-inplace-${req.work_item_id}-a${req.attempt_number}`,
+      strategy: 'inplace',
+      isolation_level: this.isolation_level,
+      branch: null,
+      base_commit: null,
+      root_path: this.projectRoot,
+      ttl_minutes: 60,
+    };
+  }
+
+  async snapshot(id: string): Promise<SnapshotRef> {
+    return {
+      snapshot_id: `snap-${crypto.randomBytes(4).toString('hex')}`,
+      workspace_id: id,
+      commit: 'HEAD',
+      created_at: new Date().toISOString(),
+    };
+  }
+
+  async reset(_id: string, _snapRef: SnapshotRef): Promise<void> {
+    // inplace: no filesystem isolation — reset is a no-op
+  }
+
+  async dispose(_id: string): Promise<void> {
+    // inplace: nothing to tear down
+  }
+}

package/packages/runtime/src/workspace/workspace-manager.ts

@@ -1,15 +1,16 @@
-import type { WorkspaceLease, SnapshotRef, WorkspaceStrategy } from '../models/workspace';
-
-export interface WorkspaceRequest {
-  work_item_id: string;
-  attempt_number: number;
-  strategy: WorkspaceStrategy;
-  mutation_scope: string[];
-}
-
-export interface WorkspaceManager {
-  allocate(req: WorkspaceRequest): Promise<WorkspaceLease>;
-  snapshot(id: string): Promise<SnapshotRef>;
-  reset(id: string, snapRef: SnapshotRef): Promise<void>;
-  dispose(id: string): Promise<void>;
-}
+import type { WorkspaceLease, SnapshotRef, WorkspaceStrategy, WorkspaceIsolationLevel } from '../models/workspace';
+
+export interface WorkspaceRequest {
+  work_item_id: string;
+  attempt_number: number;
+  strategy: WorkspaceStrategy;
+  mutation_scope: string[];
+}
+
+export interface WorkspaceManager {
+  readonly isolation_level: WorkspaceIsolationLevel;
+  allocate(req: WorkspaceRequest): Promise<WorkspaceLease>;
+  snapshot(id: string): Promise<SnapshotRef>;
+  reset(id: string, snapRef: SnapshotRef): Promise<void>;
+  dispose(id: string): Promise<void>;
+}

package/packages/runtime/tsconfig.json

@@ -1,17 +1,17 @@
-{
-  "compilerOptions": {
-    "target": "ES2020",
-    "module": "commonjs",
-    "moduleResolution": "node",
-    "strict": true,
-    "esModuleInterop": true,
-    "declaration": true,
-    "declarationMap": false,
-    "sourceMap": false,
-    "outDir": "../../lib/runtime",
-    "rootDir": "src",
-    "skipLibCheck": true
-  },
-  "include": ["src/**/*.ts"],
-  "exclude": ["tests/**", "node_modules/**", "dist-tests/**"]
-}
+{
+  "compilerOptions": {
+    "target": "ES2020",
+    "module": "commonjs",
+    "moduleResolution": "node",
+    "strict": true,
+    "esModuleInterop": true,
+    "declaration": true,
+    "declarationMap": false,
+    "sourceMap": false,
+    "outDir": "../../lib/runtime",
+    "rootDir": "src",
+    "skipLibCheck": true
+  },
+  "include": ["src/**/*.ts"],
+  "exclude": ["tests/**", "node_modules/**", "dist-tests/**"]
+}

package/vscode-extension/.vscodeignore

@@ -1,7 +1,7 @@
-.vscode/**
-.git/**
-.gitignore
-node_modules/**
-**/*.map
-**/*.test.js
-tests/**
+.vscode/**
+.git/**
+.gitignore
+node_modules/**
+**/*.map
+**/*.test.js
+tests/**