pi-oracle 0.7.11 → 0.7.13

package/extensions/oracle/lib/config.ts

@@ -22,6 +22,9 @@ import { getProjectId } from "./runtime.js";
 export const ORACLE_PROVIDERS = ["chatgpt", "grok"] as const;
 export type OracleProvider = (typeof ORACLE_PROVIDERS)[number];
 
+export { resolveOracleArchiveFormat, resolveOracleProviderArchivePlan } from "./provider-capabilities.js";
+export type { OracleArchiveFormat, OracleProviderArchivePlan } from "./provider-capabilities.js";
+
 export const MODEL_FAMILIES = ["instant", "thinking", "pro", "grok"] as const;
 export type OracleModelFamily = (typeof MODEL_FAMILIES)[number];
 
@@ -209,13 +212,19 @@ export function getProviderAuthSeedProfileDir(config: OracleConfig, provider: Or
 }
 
 export function resolveOracleConfigForProvider(config: OracleConfig, provider: OracleProvider): OracleConfig {
-  if (provider === "chatgpt") return config;
+  const defaults = {
+    ...config.defaults,
+    provider,
+  };
+  if (provider === "chatgpt") {
+    return {
+      ...config,
+      defaults,
+    };
+  }
   return {
     ...config,
-    defaults: {
-      ...config.defaults,
-      provider,
-    },
+    defaults,
     browser: {
       ...config.browser,
       authSeedProfileDir: getProviderAuthSeedProfileDir(config, provider),

package/extensions/oracle/lib/jobs.ts

@@ -4,9 +4,11 @@
 // Usage: Imported by oracle commands, tools, queue logic, poller flows, and runtime cleanup/reconciliation paths.
 // Invariants/Assumptions: Job mutations happen under per-job locks, worker identity checks defend against PID reuse, and persisted jobs remain the source of truth.
 import { createHash, randomUUID } from "node:crypto";
+import { execFileSync } from "node:child_process";
 import { existsSync, readdirSync, readFileSync, realpathSync } from "node:fs";
 import { chmod, mkdir, readFile, rename, rm, writeFile } from "node:fs/promises";
 import { isAbsolute, join, relative as relativePath, resolve, sep } from "node:path";
+import { fileURLToPath } from "node:url";
 import type { ExtensionContext } from "@earendil-works/pi-coding-agent";
 import {
   ACTIVE_ORACLE_JOB_STATUSES,
@@ -28,6 +30,7 @@ import type { OracleJobLifecycleEvent as SharedOracleJobLifecycleEvent, OracleJo
 import { hasDurableWorkerHandoff as sharedHasDurableWorkerHandoff } from "../shared/job-coordination-helpers.mjs";
 import { isTrackedProcessAlive, readProcessStartedAt, spawnDetachedNodeProcess, terminateTrackedProcess } from "../shared/process-helpers.mjs";
 import type { OracleConfig, OracleResolvedSelection } from "./config.js";
+import { resolveOracleProviderArchivePlan } from "./provider-capabilities.js";
 import { withJobLock, withLock } from "./locks.js";
 import { cleanupRuntimeArtifacts, getProjectId, getSessionId, parseConversationId, requirePersistedSessionFile, type OracleCleanupReport } from "./runtime.js";
 
@@ -116,6 +119,14 @@ export interface OracleArtifactRecord {
   matchesUploadedArchive?: boolean;
 }
 
+export interface OracleExtensionProvenance {
+  schemaVersion: 1;
+  packageName: string;
+  packageVersion: string;
+  sourcePath: string;
+  gitHead?: string;
+}
+
 export interface OracleJob {
   id: string;
   status: OracleJobStatus;
@@ -134,6 +145,7 @@ export interface OracleJob {
   originSessionFile?: string;
   requestSource: "command" | "tool";
   selection: OracleResolvedSelection;
+  extensionProvenance?: OracleExtensionProvenance;
   followUpToJobId?: string;
   chatUrl?: string;
   conversationId?: string;
@@ -451,8 +463,8 @@ export async function cleanupJobResources(
 
 function getCleanupRetentionMs(job: OracleJob): { complete: number; failed: number } {
   return {
-    complete: job.config.cleanup?.completeJobRetentionMs ?? ORACLE_COMPLETE_JOB_RETENTION_MS,
-    failed: job.config.cleanup?.failedJobRetentionMs ?? ORACLE_FAILED_JOB_RETENTION_MS,
+    complete: job.config?.cleanup?.completeJobRetentionMs ?? ORACLE_COMPLETE_JOB_RETENTION_MS,
+    failed: job.config?.cleanup?.failedJobRetentionMs ?? ORACLE_FAILED_JOB_RETENTION_MS,
   };
 }
 
@@ -898,6 +910,39 @@ export async function cancelOracleJob(id: string, reason = "Cancelled by user"):
   });
 }
 
+function readExtensionProvenance(cwd: string): OracleExtensionProvenance {
+  const sourcePath = resolve(fileURLToPath(new URL("../../../", import.meta.url)));
+  let packageName = "pi-oracle";
+  let packageVersion = "unknown";
+  try {
+    const packageJson = JSON.parse(readFileSync(join(sourcePath, "package.json"), "utf8")) as { name?: string; version?: string };
+    packageName = packageJson.name || packageName;
+    packageVersion = packageJson.version || packageVersion;
+  } catch {
+    // Keep provenance present even when package metadata is unavailable in an
+    // unusual loader; release proof rejects unknown versions.
+  }
+
+  let gitHead: string | undefined;
+  try {
+    gitHead = execFileSync("git", ["rev-parse", "HEAD"], { cwd: sourcePath, encoding: "utf8" }).trim();
+  } catch {
+    try {
+      gitHead = execFileSync("git", ["rev-parse", "HEAD"], { cwd, encoding: "utf8" }).trim();
+    } catch {
+      gitHead = undefined;
+    }
+  }
+
+  return {
+    schemaVersion: 1,
+    packageName,
+    packageVersion,
+    sourcePath,
+    gitHead,
+  };
+}
+
 export async function createJob(
   id: string,
   input: OracleSubmitInput,
@@ -911,7 +956,7 @@ export async function createJob(
   const logsDir = join(jobDir, "logs");
   const workerLogPath = join(logsDir, "worker.log");
   const promptPath = join(jobDir, "prompt.md");
-  const archivePath = join(jobDir, `context-${id}.tar.zst`);
+  const archivePath = join(jobDir, `context-${id}.${resolveOracleProviderArchivePlan(input.selection.provider).archiveExtension}`);
   const responsePath = join(jobDir, "response.md");
   const reasoningPath = join(jobDir, "reasoning.md");
   const artifactsManifestPath = join(jobDir, "artifacts.json");
@@ -945,6 +990,7 @@ export async function createJob(
     originSessionFile: sessionFile,
     requestSource: input.requestSource,
     selection: input.selection,
+    extensionProvenance: readExtensionProvenance(cwd),
     followUpToJobId: input.followUpToJobId,
     chatUrl: input.chatUrl,
     conversationId,

package/extensions/oracle/lib/provider-capabilities.ts

@@ -0,0 +1,41 @@
+// Purpose: Centralize provider-specific archive capabilities for oracle submissions.
+// Responsibilities: Map each supported provider to archive format, file extension, upload ceiling, and local compression dependencies.
+// Scope: Static provider capability data only; config loading, archive writing, and runtime preflight live in sibling modules.
+// Usage: Imported by config, jobs, tools, runtime, and archive code so provider policy stays in one canonical place.
+// Invariants/Assumptions: Provider ids are defined by config.ts, while this module must not import config values to avoid runtime/config cycles.
+import type { OracleProvider } from "./config.js";
+
+export type OracleArchiveFormat = "tar.zst" | "tar.gz";
+
+export interface OracleProviderArchivePlan {
+  provider: OracleProvider;
+  archiveFormat: OracleArchiveFormat;
+  archiveExtension: OracleArchiveFormat;
+  maxArchiveBytes: number;
+  requiresZstd: boolean;
+}
+
+const ORACLE_PROVIDER_ARCHIVE_PLANS: Record<OracleProvider, OracleProviderArchivePlan> = {
+  chatgpt: {
+    provider: "chatgpt",
+    archiveFormat: "tar.zst",
+    archiveExtension: "tar.zst",
+    maxArchiveBytes: 250 * 1024 * 1024,
+    requiresZstd: true,
+  },
+  grok: {
+    provider: "grok",
+    archiveFormat: "tar.gz",
+    archiveExtension: "tar.gz",
+    maxArchiveBytes: 200 * 1024 * 1024,
+    requiresZstd: false,
+  },
+};
+
+export function resolveOracleProviderArchivePlan(provider: OracleProvider): OracleProviderArchivePlan {
+  return ORACLE_PROVIDER_ARCHIVE_PLANS[provider];
+}
+
+export function resolveOracleArchiveFormat(provider: OracleProvider): OracleArchiveFormat {
+  return resolveOracleProviderArchivePlan(provider).archiveFormat;
+}

package/extensions/oracle/lib/runtime.ts

@@ -11,7 +11,8 @@ import { delimiter, dirname, join } from "node:path";
 import { assertNotKnownBrowserUserDataPath, sweetCookieSafeStoragePasswordScrubbedEnv } from "../shared/browser-profile-helpers.mjs";
 import { jobBlocksAdmission } from "../shared/job-coordination-helpers.mjs";
 import { isTrackedProcessAlive } from "../shared/process-helpers.mjs";
-import type { OracleConfig } from "./config.js";
+import type { OracleConfig, OracleProvider } from "./config.js";
+import { resolveOracleProviderArchivePlan } from "./provider-capabilities.js";
 import { createLease, listLeaseMetadata, readLeaseMetadata, releaseLease, withAuthLock } from "./locks.js";
 
 const SEED_GENERATION_FILE = ".oracle-seed-generation";
@@ -49,12 +50,15 @@ function cpCommand(): string {
   return process.env.PI_ORACLE_CP_PATH?.trim() || "cp";
 }
 
-function requiredOracleDependencies(config: OracleConfig): Array<{ name: string; command: string }> {
+function requiredOracleDependencies(config: OracleConfig, provider = config.defaults.provider): Array<{ name: string; command: string }> {
+  const archivePlan = resolveOracleProviderArchivePlan(provider);
   const dependencies = [
     { name: "agent-browser", command: AGENT_BROWSER_BIN },
     { name: "tar", command: "tar" },
-    { name: "zstd", command: "zstd" },
   ];
+  if (archivePlan.requiresZstd) {
+    dependencies.push({ name: "zstd", command: "zstd" });
+  }
   if (config.browser.cloneStrategy === "apfs-clone" && process.platform === "darwin") {
     dependencies.push({ name: "cp", command: cpCommand() });
   }
@@ -325,11 +329,11 @@ export async function assertOracleAuthSeedProfileReady(config: OracleConfig): Pr
   }
 }
 
-export async function assertOracleSubmitPrerequisites(config: OracleConfig): Promise<void> {
+export async function assertOracleSubmitPrerequisites(config: OracleConfig, provider: OracleProvider = config.defaults.provider): Promise<void> {
   assertSafeOracleProfilePath(config.browser.runtimeProfilesDir, "runtime profiles", config);
   await assertOracleAuthSeedProfileReady(config);
   await assertConfiguredBrowserExecutableReady(config.browser.executablePath);
-  for (const dependency of requiredOracleDependencies(config)) {
+  for (const dependency of requiredOracleDependencies(config, provider)) {
     await assertRequiredLocalDependencyReady(dependency.name, dependency.command);
   }
   await assertWritableDirectory(config.browser.runtimeProfilesDir, "runtime profiles");