@pasko70/pibo 3.0.1 → 3.1.0

package/dist/agent-runtimes/codex-native/adapter.js

@@ -290,6 +290,7 @@ export class CodexNativeThreadSession {
     recycleProcessAfterInterruptedTurn = false;
     selectedToolNames = new Set();
     observedToolNames = new Set();
+    emittedResourceWarnings = new Set();
     toolInventoryWarning;
     constructor(runtimeInstanceId, process, threads, settings, resourceDelivery, binding, structuredUserInput, reloadProcess, productContext, bindingPersistence, firstUseTestFailpoints) {
         this.runtimeInstanceId = runtimeInstanceId;
@@ -390,6 +391,7 @@ export class CodexNativeThreadSession {
         this.assertActive();
         await this.ensureFreshResourcesForTurn();
         this.assertIdle();
+        this.emitPendingResourceWarnings();
         this.operationInFlight = true;
         let ownedFirstUseAttempt;
         try {
@@ -473,6 +475,7 @@ export class CodexNativeThreadSession {
             contextUsage: this.settings.currentContextUsage,
             warnings: [
                 ...diagnostics.filter((entry) => entry.level === "warning").map((entry) => entry.message),
+                ...this.resourceDelivery.warnings.map((entry) => entry.message),
                 ...(this.resourceWarning ? [this.resourceWarning] : []),
                 ...(this.toolInventoryWarning ? [this.toolInventoryWarning] : []),
             ],
@@ -659,6 +662,15 @@ export class CodexNativeThreadSession {
             throw new Error(`Native Codex portable-resource credential refresh failed while ${phase}; Pibo will retry while the session remains idle.`);
         }
     }
+    emitPendingResourceWarnings() {
+        for (const warning of this.resourceDelivery.warnings) {
+            const key = `${warning.code}\0${warning.message}`;
+            if (this.emittedResourceWarnings.has(key))
+                continue;
+            this.emittedResourceWarnings.add(key);
+            this.emit({ type: "warning", message: warning.message, details: { code: warning.code } });
+        }
+    }
     updateSelectedToolNames(resourceDelivery) {
         const names = [...new Set(resourceDelivery.enabledToolNames)].sort();
         this.selectedToolNames = new Set(names.slice(0, MAX_INSPECTED_SELECTED_TOOL_NAMES));

package/dist/agent-runtimes/codex-native/resource-delivery.js

@@ -256,21 +256,25 @@ export class CodexNativeResourceDelivery {
     skillRoots;
     selectedSkills;
     expectedMcpServers;
+    resources;
     portableTools;
     mcpAccess;
     credentialIssuedAtMs;
     credentialExpiresAtMs;
-    constructor(environment, threadConfig, developerInstructions, skillRoots, selectedSkills, expectedMcpServers, portableTools, mcpAccess, credentialIssuedAtMs, credentialExpiresAtMs) {
+    resourceWarnings;
+    constructor(environment, threadConfig, developerInstructions, skillRoots, selectedSkills, expectedMcpServers, resources, portableTools, mcpAccess, credentialIssuedAtMs, credentialExpiresAtMs, resourceWarnings = []) {
         this.environment = environment;
         this.threadConfig = threadConfig;
         this.developerInstructions = developerInstructions;
         this.skillRoots = skillRoots;
         this.selectedSkills = selectedSkills;
         this.expectedMcpServers = expectedMcpServers;
+        this.resources = resources;
         this.portableTools = portableTools;
         this.mcpAccess = mcpAccess;
         this.credentialIssuedAtMs = credentialIssuedAtMs;
         this.credentialExpiresAtMs = credentialExpiresAtMs;
+        this.resourceWarnings = resourceWarnings;
     }
     static async prepare(input) {
         const resourceEnvironment = { ...(input.resources?.getAdapterEnvironment() ?? {}) };
@@ -323,7 +327,7 @@ export class CodexNativeResourceDelivery {
             const selectedSkills = input.resources
                 ? await Promise.all(input.resources.getInspection().skills.flatMap((skill) => {
                     const path = skill.materializedPath;
-                    return path ? [{ name: skill.name, path }] : [];
+                    return path ? [{ contributionId: skill.contributionId, name: skill.name, path }] : [];
                 }).map(async (skill) => ({ ...skill, path: await realpath(skill.path) })))
                 : [];
             const materializedSkillsRoot = input.resources?.getInspection().paths?.skills;
@@ -339,7 +343,7 @@ export class CodexNativeResourceDelivery {
                     : { multi_agent: false, multi_agent_v2: false };
                 threadConfig.agents = { enabled: input.nativeSubagentsEnabled };
             }
-            return new CodexNativeResourceDelivery(resourceEnvironment, Object.keys(threadConfig).length > 0 ? threadConfig : undefined, buildDeveloperInstructions(input.resources), skillRoots, selectedSkills, expectedMcpServers, input.portableTools, mcpAccess, credentialIssuedAtMs, credentialExpiresAtMs);
+            return new CodexNativeResourceDelivery(resourceEnvironment, Object.keys(threadConfig).length > 0 ? threadConfig : undefined, buildDeveloperInstructions(input.resources), skillRoots, selectedSkills, expectedMcpServers, input.resources, input.portableTools, mcpAccess, credentialIssuedAtMs, credentialExpiresAtMs);
         }
         catch (error) {
             if (mcpAccess && input.portableTools)
@@ -350,6 +354,9 @@ export class CodexNativeResourceDelivery {
     get hasMcpServers() {
         return this.expectedMcpServers.length > 0;
     }
+    get warnings() {
+        return this.resourceWarnings;
+    }
     get enabledToolNames() {
         return this.expectedMcpServers
             .flatMap((server) => server.toolNames.map((toolName) => `${server.name}/${toolName}`))
@@ -379,6 +386,7 @@ export class CodexNativeResourceDelivery {
         this.credentialExpiresAtMs = expiresAt;
     }
     async configureProcess(client, workspace) {
+        this.resourceWarnings = [];
         if (this.skillRoots.length === 0)
             return;
         let response;
@@ -411,15 +419,35 @@ export class CodexNativeResourceDelivery {
                 }
             }
         }
+        const collisions = new Set();
         for (const skill of this.selectedSkills) {
             const sameNamePaths = loadedSkills.get(skill.name) ?? new Set();
             const conflictingPath = [...sameNamePaths].find((path) => path !== skill.path);
-            if (conflictingPath) {
-                throw new Error(`Selected Pibo skill "${skill.name}" conflicts with a native Codex skill; explicit Pibo-skill precedence cannot be guaranteed.`);
-            }
-            if (!loadedPaths.has(skill.path))
+            if (conflictingPath)
+                collisions.add(skill.name);
+            if (!loadedPaths.has(skill.path) && !conflictingPath)
                 throw new Error(`Codex did not load selected Pibo skill "${skill.name}".`);
         }
+        let collisionMessage;
+        if (collisions.size > 0) {
+            const names = [...collisions].sort().map((name) => `"${name}"`).join(", ");
+            collisionMessage = `Codex native skill name collision for ${names}: the selected Pibo skill remains materialized, but precedence is ambiguous. Rename or disable one source to remove the collision.`;
+            this.resourceWarnings = [{
+                    code: "codex_native_skill_name_collision",
+                    message: collisionMessage,
+                }];
+        }
+        this.resources?.recordAdapterDelivery?.(this.selectedSkills.map((skill) => {
+            const collided = collisions.has(skill.name);
+            return {
+                contributionId: skill.contributionId,
+                status: collided ? "degraded" : "delivered",
+                mode: "codex-extra-roots",
+                fidelity: collided ? "lossy" : "exact",
+                target: skill.path,
+                ...(collided && collisionMessage ? { diagnostic: collisionMessage } : {}),
+            };
+        }));
     }
     async verifyThread(client, threadId) {
         if (this.expectedMcpServers.length === 0)

package/dist/agent-runtimes/omp/adapter.js

@@ -631,13 +631,9 @@ class OmpAgentRuntimeAdapter {
         }
         return emptyOmpHistoryPage(this.instanceId);
     }
-    async resolveBinding() {
-        return {
-            piboSessionId: "",
-            runtimeInstanceId: this.instanceId,
-            adapterId: OMP_ADAPTER_ID,
-            state: "unbound",
-        };
+    async resolveBinding(input) {
+        // OMP can verify and resume the transcript only after startup via switch_session.
+        return structuredClone(input.binding);
     }
 }
 export const OMP_AGENT_RUNTIME_DRIVER = {

package/dist/agent-runtimes/pi/adapter.js

@@ -135,6 +135,7 @@ function cloneProfileForPiSession(input) {
         parentSessionId: profile.parentSessionId,
         model: profile.model,
         mainModel: profile.mainModel,
+        mainModelFallbacks: profile.mainModelFallbacks,
         subagentModel: profile.subagentModel,
         thinkingLevel: profile.thinkingLevel,
         mainThinkingLevel: profile.mainThinkingLevel,
@@ -148,6 +149,7 @@ function cloneProfileForPiSession(input) {
         mcpServers: profile.mcpServers,
         piPackages: profile.piPackages,
         contextFiles: profile.contextFiles,
+        diagnostics: profile.diagnostics,
         builtinTools: profile.builtinTools,
         builtinToolNames: profile.builtinToolNames,
         autoContextFiles: profile.autoContextFiles,
@@ -276,7 +278,7 @@ class PiAgentRuntimeSession {
     bindingNativeSessionId;
     nativePresenceExpected;
     disposed = false;
-    constructor(runtimeInstanceId, piboSessionId, runtime, binding, initialFastMode, providerWebSearchEnabled) {
+    constructor(runtimeInstanceId, piboSessionId, runtime, binding, initialFastMode, providerWebSearchEnabled, providerFallbacksEnabled) {
         this.runtimeInstanceId = runtimeInstanceId;
         this.piboSessionId = piboSessionId;
         this.runtime = runtime;
@@ -284,7 +286,7 @@ class PiAgentRuntimeSession {
         this.cwd = runtime.cwd;
         this.bindingNativeSessionId = runtime.session.sessionId;
         this.nativePresenceExpected = runtime.session.sessionManager.buildSessionContext().messages.length > 0;
-        this.routed = new PiRoutedSession(piboSessionId, runtime, (event) => this.handlePiboEvent(event), PiboPluginRegistry.create(), true, undefined, initialFastMode, undefined, undefined, (state) => this.handleEngineState(state));
+        this.routed = new PiRoutedSession(piboSessionId, runtime, (event) => this.handlePiboEvent(event), PiboPluginRegistry.create(), true, undefined, initialFastMode, undefined, undefined, (state) => this.handleEngineState(state), undefined, undefined, providerFallbacksEnabled);
         if (providerWebSearchEnabled)
             this.routed.enableProviderWebSearchObservation();
         this.controls = this.createControls();
@@ -685,7 +687,7 @@ class PiAgentRuntimeAdapter {
                     && runtime.session.sessionManager.buildSessionContext().messages.length > 0,
             },
         };
-        return new PiAgentRuntimeSession(this.instanceId, input.piboSession.id, runtime, binding, compatibility?.initialFastMode ?? false, profile.tools.some((tool) => tool.enabled !== false && isWebSearchProviderTool(tool)));
+        return new PiAgentRuntimeSession(this.instanceId, input.piboSession.id, runtime, binding, compatibility?.initialFastMode ?? false, profile.tools.some((tool) => tool.enabled !== false && isWebSearchProviderTool(tool)), compatibility?.providerFallbacksEnabled ?? false);
     }
     loadModelCatalog() {
         const now = Date.now();

package/dist/agent-runtimes/pi/routed-session.js

@@ -1,3 +1,5 @@
+import { existsSync, mkdirSync, writeFileSync } from "node:fs";
+import { dirname } from "node:path";
 import { SessionManager, shouldCompact } from "@earendil-works/pi-coding-agent";
 import { PiboSteeringUnavailableError } from "../../core/events.js";
 import { getOpenAiCodexProviderUsageForActiveModel } from "../../auth/openai-codex-usage.js";
@@ -18,6 +20,16 @@ const RUN_REMINDER_CAPABILITY_TOOLS = new Set([
     "pibo_run_cancel",
     "pibo_run_ack",
 ]);
+function materializeForkedSession(sessionManager) {
+    const sessionFile = sessionManager.getSessionFile();
+    const header = sessionManager.getHeader();
+    if (!sessionManager.isPersisted() || !sessionFile || !header || existsSync(sessionFile))
+        return;
+    mkdirSync(dirname(sessionFile), { recursive: true });
+    writeFileSync(sessionFile, `${[header, ...sessionManager.getEntries()].map((entry) => JSON.stringify(entry)).join("\n")}\n`, { encoding: "utf8", flag: "wx" });
+    // Reload through the public API so Pi also records that the file is flushed.
+    sessionManager.setSessionFile(sessionFile);
+}
 function modelSupportsFastServiceTier(model) {
     if (!model)
         return false;
@@ -587,6 +599,7 @@ export class RoutedSession {
     onStateChange;
     onMessagesInterrupted;
     messagePreflight;
+    providerFallbacksEnabled;
     queue = [];
     processing = false;
     disposed = false;
@@ -605,6 +618,9 @@ export class RoutedSession {
     nextAssistantIndex = 0;
     activeThinkingIndex;
     nextThinkingIndex = 0;
+    nextUsageIndex = 0;
+    activeCompactionIndex;
+    nextCompactionIndex = 0;
     sessionIdentityOperationInFlight = false;
     pendingAssistantError;
     pendingAssistantErrorRetryable = false;
@@ -616,7 +632,7 @@ export class RoutedSession {
     recoverySession;
     isContinuePatched = false;
     compatibilityAuthController;
-    constructor(piboSessionId, runtime, emit, pluginRegistry, forwardPiEvents, onPiEventTelemetry, initialFastMode, onSessionOperation, onKillChildren, onStateChange, onMessagesInterrupted, messagePreflight) {
+    constructor(piboSessionId, runtime, emit, pluginRegistry, forwardPiEvents, onPiEventTelemetry, initialFastMode, onSessionOperation, onKillChildren, onStateChange, onMessagesInterrupted, messagePreflight, providerFallbacksEnabled = false) {
         this.piboSessionId = piboSessionId;
         this.runtime = runtime;
         this.emit = emit;
@@ -628,6 +644,7 @@ export class RoutedSession {
         this.onStateChange = onStateChange;
         this.onMessagesInterrupted = onMessagesInterrupted;
         this.messagePreflight = messagePreflight;
+        this.providerFallbacksEnabled = providerFallbacksEnabled;
         this.fastMode = initialFastMode && this.fastModeSupported();
         this.bindRuntimeSession();
         this.patchProviderRequest();
@@ -1005,26 +1022,34 @@ export class RoutedSession {
         const candidate = event;
         const eventId = this.activeMessage?.id ?? this.activeExecutionEvent?.id;
         if (candidate.type === "compaction_start") {
+            const compactionIndex = this.activeCompactionIndex ?? this.nextCompactionIndex;
+            if (this.activeCompactionIndex === undefined)
+                this.nextCompactionIndex += 1;
+            this.activeCompactionIndex = compactionIndex;
             this.emit(this.withActiveMessage({
                 type: "compaction_start",
                 piboSessionId: this.piboSessionId,
                 eventId,
+                compactionIndex,
                 reason: typeof candidate.reason === "string" ? candidate.reason : "unknown",
             }));
         }
         if (candidate.type === "compaction_end") {
+            const compactionIndex = this.activeCompactionIndex ?? this.nextCompactionIndex;
+            if (this.activeCompactionIndex === undefined)
+                this.nextCompactionIndex += 1;
+            this.activeCompactionIndex = undefined;
             if (candidate.result && candidate.aborted !== true) {
-                // Reset assistant message indices so the next assistant response starts
-                // fresh after compaction, matching the reduced agent context.
+                // End any open output block, but keep durable part indices monotonic for
+                // the lifetime of the routed event across context-guard continuation.
                 this.activeAssistantIndex = undefined;
-                this.nextAssistantIndex = 0;
                 this.activeThinkingIndex = undefined;
-                this.nextThinkingIndex = 0;
             }
             this.emit(this.withActiveMessage({
                 type: "compaction_end",
                 piboSessionId: this.piboSessionId,
                 eventId,
+                compactionIndex,
                 reason: typeof candidate.reason === "string" ? candidate.reason : "unknown",
                 result: candidate.result,
                 aborted: candidate.aborted === true,
@@ -1082,9 +1107,15 @@ export class RoutedSession {
         if (event.action === "compact") {
             return this.enqueueCompactAction(event);
         }
-        const changesSessionIdentity = event.action === "session.fork" || event.action === "session.clone";
-        if (changesSessionIdentity) {
-            this.assertSessionIdentityOperationIdle(event.action === "session.fork" ? "fork" : "clone");
+        const sessionIdentityOperation = event.action === "session.fork"
+            ? "fork"
+            : event.action === "session.clone"
+                ? "clone"
+                : event.action === "session.switch"
+                    ? "switch"
+                    : undefined;
+        if (sessionIdentityOperation) {
+            this.assertSessionIdentityOperationIdle(sessionIdentityOperation);
             this.sessionIdentityOperationInFlight = true;
         }
         try {
@@ -1102,7 +1133,7 @@ export class RoutedSession {
             return output;
         }
         finally {
-            if (changesSessionIdentity)
+            if (sessionIdentityOperation)
                 this.sessionIdentityOperationInFlight = false;
         }
     }
@@ -1213,6 +1244,11 @@ export class RoutedSession {
         this.assertSessionWorkIdle("fork");
         const previous = this.createSessionSnapshot();
         const result = await this.runtime.fork(entryId);
+        if (!result.cancelled) {
+            // Pi defers persistence until an assistant message. Materialize an empty
+            // first-message branch so history readers can use its advertised path.
+            materializeForkedSession(this.runtime.session.sessionManager);
+        }
         return {
             piboSessionId: this.piboSessionId,
             previous,
@@ -1518,6 +1554,9 @@ export class RoutedSession {
             this.nextAssistantIndex = 0;
             this.activeThinkingIndex = undefined;
             this.nextThinkingIndex = 0;
+            this.nextUsageIndex = 0;
+            this.activeCompactionIndex = undefined;
+            this.nextCompactionIndex = 0;
             this.emit({
                 type: "message_started",
                 piboSessionId: this.piboSessionId,
@@ -1539,7 +1578,8 @@ export class RoutedSession {
             await this.resumeContextGuardRecovery(session);
             if (this.disposed || inFlight.cancelled)
                 return;
-            await this.recoverTransientProviderErrors(session);
+            if (!this.providerFallbacksEnabled)
+                await this.recoverTransientProviderErrors(session);
             if (this.disposed || inFlight.cancelled)
                 return;
             this.flushPendingAssistantError();
@@ -1578,6 +1618,9 @@ export class RoutedSession {
             this.nextAssistantIndex = 0;
             this.activeThinkingIndex = undefined;
             this.nextThinkingIndex = 0;
+            this.nextUsageIndex = 0;
+            this.activeCompactionIndex = undefined;
+            this.nextCompactionIndex = 0;
             this.providerWebSearchLifecycle.clear();
             if (this.inFlightMessage === inFlight)
                 this.inFlightMessage = undefined;
@@ -1817,8 +1860,12 @@ export class RoutedSession {
                 this.activeThinkingIndex = undefined;
             return output;
         }
-        if (event.type === "assistant_usage" ||
-            event.type === "compaction_start" ||
+        if (event.type === "assistant_usage") {
+            const usageIndex = this.nextUsageIndex;
+            this.nextUsageIndex += 1;
+            return { ...event, ...correlation, usageIndex };
+        }
+        if (event.type === "compaction_start" ||
             event.type === "compaction_end" ||
             event.type === "tool_call" ||
             event.type === "tool_execution_started" ||

package/dist/agent-runtimes/pi/runtime.js

@@ -332,6 +332,10 @@ export async function createPiboRuntime(options = {}) {
         }
         const resourceLoader = services.resourceLoader;
         const diagnostics = [
+            ...profile.diagnostics.map((diagnostic) => ({
+                type: diagnostic.severity,
+                message: `[${diagnostic.code}] ${diagnostic.message}`,
+            })),
             ...piPackageOptions.diagnostics,
             ...services.diagnostics,
             ...collectResourceDiagnostics(resourceLoader.getSkills().diagnostics),
@@ -416,6 +420,7 @@ export async function inspectPiboProfile(options = {}) {
         mcpServers: profile.mcpServers,
         piPackages: profile.piPackages,
         contextFiles: profile.contextFiles,
+        diagnostics: profile.diagnostics,
         builtinTools: profile.builtinTools,
         builtinToolNames: profile.builtinToolNames,
         autoContextFiles: profile.autoContextFiles,
@@ -456,6 +461,7 @@ export async function inspectPiboProfile(options = {}) {
             runtimeOptions: structuredClone(profile.runtimeOptions),
             ...(profile.model ? { model: { ...profile.model } } : {}),
             ...(profile.mainModel ? { mainModel: { ...profile.mainModel } } : {}),
+            mainModelFallbacks: profile.mainModelFallbacks.map((model) => ({ ...model })),
             ...(profile.subagentModel ? { subagentModel: { ...profile.subagentModel } } : {}),
             ...(profile.thinkingLevel ? { thinkingLevel: profile.thinkingLevel } : {}),
             ...(profile.mainThinkingLevel ? { mainThinkingLevel: profile.mainThinkingLevel } : {}),

package/dist/apps/chat/agent-profiles.js

@@ -14,6 +14,14 @@ export function createCustomAgentProfileDefinition(agent, options = {}) {
                 catch (error) {
                     if (!isUnknownSkillError(error, skillName))
                         throw error;
+                    const message = `Unknown skill "${skillName}" referenced by custom agent "${agent.profileName}"`;
+                    builder.addDiagnostic({
+                        severity: "warning",
+                        code: "custom_agent_unknown_skill",
+                        message,
+                        resourceKind: "skill",
+                        resourceName: skillName,
+                    });
                     if (shouldWarnMissingReferences)
                         console.warn(`Skipping unknown skill "${skillName}" for custom agent "${agent.profileName}"`);
                 }
@@ -25,12 +33,29 @@ export function createCustomAgentProfileDefinition(agent, options = {}) {
                 catch (error) {
                     if (!isUnknownContextFileError(error, contextFileKey))
                         throw error;
+                    const message = `Unknown context file "${contextFileKey}" referenced by custom agent "${agent.profileName}"`;
+                    builder.addDiagnostic({
+                        severity: "warning",
+                        code: "custom_agent_unknown_context_file",
+                        message,
+                        resourceKind: "context-file",
+                        resourceName: contextFileKey,
+                    });
                     if (shouldWarnMissingReferences)
                         console.warn(`Skipping unknown context file "${contextFileKey}" for custom agent "${agent.profileName}"`);
                 }
             }
-            for (const toolName of agent.nativeTools)
-                builder.addTool(context.getTool(toolName));
+            for (const toolName of agent.nativeTools) {
+                try {
+                    builder.addTool(context.getTool(toolName));
+                }
+                catch (error) {
+                    if (!isUnknownToolError(error, toolName))
+                        throw error;
+                    if (shouldWarnMissingReferences)
+                        console.warn(`Skipping unknown tool "${toolName}" for custom agent "${agent.profileName}"`);
+                }
+            }
             for (const subagent of agent.subagents)
                 builder.addSubagent(subagent);
             return builder.createSession();
@@ -61,6 +86,7 @@ function createCustomAgentBuilder(agent) {
         .withToolPackages({ runControl: agent.runControl, goalControl: agent.goalControl ?? true });
     if (agent.mainModel)
         builder.withMainModel(agent.mainModel);
+    builder.withMainModelFallbacks(agent.mainModelFallbacks ?? []);
     if (agent.subagentModel)
         builder.withSubagentModel(agent.subagentModel);
     if (agent.thinkingLevel)
@@ -80,6 +106,9 @@ function createCustomAgentBuilder(agent) {
 function uniqueAliases(aliases, profileName) {
     return [...new Set(aliases.filter((alias) => alias && alias !== profileName))];
 }
+function isUnknownToolError(error, toolName) {
+    return error instanceof Error && error.message === `Unknown tool "${toolName}"`;
+}
 function isUnknownContextFileError(error, contextFileKey) {
     return error instanceof Error && error.message === `Unknown context file "${contextFileKey}"`;
 }