@loucompanion/forge-bridge 0.1.3-dev.0b2f81d660c0 → 0.1.3-dev.34ddcb672775

package/dist/bridge/bridge.service.js

@@ -2,7 +2,6 @@ import { HubConnectionBuilder, HttpTransportType, LogLevel } from "@microsoft/si
 import fs from "node:fs";
 import path from "node:path";
 import { defaultForgeBridgeConfig } from "../config.js";
-import { supportsDraftImageAttachmentsFor } from "../shared/provider-capabilities.js";
 import { redactSecrets, redactText } from "../shared/redaction/redaction.js";
 import { BridgeSessionService } from "../session/session.service.js";
 import { failureResult, parseRunOnceDispatchPacket } from "../session/session.types.js";
@@ -352,7 +351,7 @@ export function defaultBridgeServiceConfig(overrides = {}) {
         supportsRunnerAuthoredPhaseArtifacts: true,
         supportsDraftRunOnce: true,
         supportsDraftAttachments: true,
-        supportsDraftImageAttachments: supportsDraftImageAttachmentsFor(cliProviders),
+        supportsDraftImageAttachments: false,
         version: process.env.FORGE_BRIDGE_VERSION ?? forgeBridgeVersion,
         protocolVersion: 1,
         minServerVersion: null,
@@ -362,9 +361,7 @@ export function defaultBridgeServiceConfig(overrides = {}) {
     };
     return {
         ...config,
-        providerStatuses: overrides.providerStatuses ?? providerStatusesFor(config.cliProviders),
         sandboxProfiles: config.sandboxProfiles ?? defaultSandboxProfiles(config.cliProviders),
-        supportsDraftImageAttachments: (overrides.supportsDraftImageAttachments ?? true) && supportsDraftImageAttachmentsFor(config.cliProviders),
         heartbeatIntervalMs: positiveFiniteNumber(config.heartbeatIntervalMs, 30_000)
     };
 }

package/dist/cli/cli.service.js

@@ -6,7 +6,6 @@ import { createInterface } from "node:readline/promises";
 import { promisify } from "node:util";
 import { resolveCursorAgentDirectLaunch } from "../shared/cursor-agent/cursor-agent.launch.js";
 import { buildWindowsCmdLaunch } from "../shared/process/windows-launch.js";
-import { supportsDraftImageAttachmentsFor } from "../shared/provider-capabilities.js";
 import { redactSecrets, redactText } from "../shared/redaction/redaction.js";
 import { defaultBridgeServiceConfig, ForgeBridgeService } from "../bridge/bridge.service.js";
 import { forgeBridgeVersion } from "../version.js";
@@ -501,7 +500,7 @@ function pairedConfig(current, args, claim) {
         supportsRunnerAuthoredPhaseArtifacts: current.supportsRunnerAuthoredPhaseArtifacts ?? true,
         supportsDraftRunOnce: current.supportsDraftRunOnce ?? true,
         supportsDraftAttachments: current.supportsDraftAttachments ?? true,
-        supportsDraftImageAttachments: supportsDraftImageAttachmentsFor(args.cliProviders),
+        supportsDraftImageAttachments: current.supportsDraftImageAttachments ?? false,
         repoMappings: current.repoMappings ?? []
     };
 }

package/dist/session/phase-completion.artifacts.js

@@ -424,7 +424,7 @@ async function synthesizeQaCycleVerdictPayload(config, payload, status) {
     ]);
     merged.browser_checks = dedupeUnknownArray([
         ...qaArray(payload, "browser_checks", payloadEvidence),
-        ...evidenceGates.flatMap((gate) => qaArray(gate, "browser_checks").map(normalizeInheritedQaBrowserCheck))
+        ...evidenceGates.flatMap((gate) => qaArray(gate, "browser_checks"))
     ]);
     merged.sidecars = dedupeUnknownArray([
         ...qaArray(payload, "sidecars", payloadEvidence),
@@ -516,27 +516,6 @@ function dedupeStrings(values) {
     }
     return deduped;
 }
-function normalizeInheritedQaBrowserCheck(browserCheck) {
-    if (!browserCheck || typeof browserCheck !== "object" || Array.isArray(browserCheck)) {
-        return browserCheck;
-    }
-    const record = browserCheck;
-    if (optionalString(record, "summary")) {
-        return record;
-    }
-    const target = optionalString(record, "url") ?? optionalString(record, "route") ?? "the configured browser target";
-    const status = optionalString(record, "status") ?? "recorded";
-    return {
-        ...record,
-        summary: `Inherited browser check ${status} for ${target}.`
-    };
-}
-function normalizeSha256ContentHash(contentHash) {
-    const normalized = contentHash.trim().toLowerCase();
-    return normalized.startsWith("sha256:")
-        ? normalized.slice("sha256:".length)
-        : normalized;
-}
 async function validateQaSidecarReferences(worktreePath, workItemId, sidecars, ownerType, ownerId) {
     if (sidecars.length === 0) {
         return [];
@@ -578,7 +557,7 @@ async function validateQaSidecarReferences(worktreePath, workItemId, sidecars, o
             throw new Error(`sidecars[${index}].bytes does not match the referenced file.`);
         }
         const actualHash = await sha256File(fullPath);
-        if (actualHash !== normalizeSha256ContentHash(contentHash)) {
+        if (actualHash !== contentHash.trim().toLowerCase()) {
             throw new Error(`sidecars[${index}].content_hash does not match the referenced file.`);
         }
         validated.push({

package/dist/session/provider-cli/provider-cli.prompts.d.ts

@@ -20,7 +20,6 @@ export declare function buildProviderCliCommand(dispatch: RunOnceDispatchPacket,
     readonly worktreePath?: string;
     readonly completionTools?: CompletionToolRuntime;
     readonly skipGitRepoCheck?: boolean;
-    readonly imagePaths?: readonly string[];
 }): ProviderCliCommand;
 export declare function codexSandboxModeForPlatform(platform?: NodeJS.Platform): "workspace-write" | "danger-full-access";
 export declare function buildArchitectPrompt(dispatch: RunOnceDispatchPacket): string;

package/dist/session/provider-cli/provider-cli.prompts.js

@@ -164,9 +164,6 @@ export function buildProviderCliCommand(dispatch, run) {
             if (run.skipGitRepoCheck) {
                 args.push("--skip-git-repo-check");
             }
-            for (const imagePath of run.imagePaths ?? []) {
-                args.push("--image", imagePath);
-            }
             args.push(run.prompt ?? "");
         }
         return { command: "codex", args };

package/dist/session/provider-cli/provider-cli.service.js

@@ -64,17 +64,16 @@ export class ProcessBackedProviderCliService {
         await fs.mkdir(metadataRoot, { recursive: true });
         await fs.writeFile(path.join(metadataRoot, "draft-input.json"), JSON.stringify(durableDraftInput(dispatch), null, 2) + "\n", "utf8");
         await fs.writeFile(path.join(metadataRoot, "spec.md"), buildDraftSpec(dispatch), "utf8");
-        const stagedAttachments = await stageDraftAttachments(dispatch, metadataRoot);
+        const attachmentReferences = await stageDraftTextAttachments(dispatch, metadataRoot);
         const decomposeDraft = isDecomposeDraftDispatch(dispatch);
         const completionTools = await prepareCompletionToolRuntime({ dispatch, worktree, phase: decomposeDraft ? "decompose" : "draft" });
-        const draftPrompt = buildDraftPrompt(dispatch, { attachmentReferences: stagedAttachments.textReferences });
+        const draftPrompt = buildDraftPrompt(dispatch, { attachmentReferences });
         const pipePromptToStdin = shouldPipePromptToStdin(dispatch.execution.provider, draftPrompt);
         const command = buildProviderCliCommand(dispatch, {
             worktreePath: worktree.worktreePath,
             prompt: pipePromptToStdin ? "-" : draftPrompt,
             completionTools: completionTools ?? undefined,
-            skipGitRepoCheck: false,
-            imagePaths: stagedAttachments.imagePaths
+            skipGitRepoCheck: false
         });
         const draftArtifactSlugs = dispatch.artifacts
             .filter((artifact) => artifact.scope === "operational" && ["command", "skill", "hook"].includes(artifact.kind))
@@ -577,42 +576,23 @@ export class ProcessBackedProviderCliService {
         };
     }
 }
-async function stageDraftAttachments(dispatch, metadataRoot) {
+async function stageDraftTextAttachments(dispatch, metadataRoot) {
     const attachments = dispatch.draft?.attachments ?? [];
-    const stageable = attachments.filter((attachment) => (attachment.kind === "text" && typeof attachment.textContent === "string")
-        || (attachment.kind === "image" && typeof attachment.contentBase64 === "string"));
-    if (stageable.length === 0) {
-        return { textReferences: [], imagePaths: [] };
+    const textAttachments = attachments.filter((attachment) => attachment.kind === "text" && typeof attachment.textContent === "string");
+    if (textAttachments.length === 0) {
+        return [];
     }
     const attachmentRoot = path.join(metadataRoot, "draft-attachments");
     const worktreeRoot = path.resolve(metadataRoot, "../../..");
     await fs.mkdir(attachmentRoot, { recursive: true });
-    const textReferences = [];
-    const imagePaths = [];
-    for (const attachment of stageable) {
+    const references = [];
+    for (const attachment of textAttachments) {
         const fileName = `${String(attachment.index + 1).padStart(2, "0")}-${safeAttachmentFileName(attachment.fileName)}`;
         const absolutePath = path.join(attachmentRoot, fileName);
-        if (attachment.kind === "text") {
-            await fs.writeFile(absolutePath, attachment.textContent ?? "", { encoding: "utf8", mode: 0o600 });
-            textReferences.push({ index: attachment.index, path: path.relative(worktreeRoot, absolutePath).replaceAll(path.sep, "/") });
-            continue;
-        }
-        const imageContent = decodeDraftImageAttachment(attachment.contentBase64 ?? "", attachment.fileName, attachment.index);
-        await fs.writeFile(absolutePath, imageContent, { mode: 0o600 });
-        imagePaths.push(absolutePath);
-    }
-    return { textReferences, imagePaths };
-}
-function decodeDraftImageAttachment(contentBase64, fileName, index) {
-    const normalized = contentBase64.replace(/\s+/gu, "");
-    if (normalized.length === 0 || normalized.length % 4 !== 0 || !/^[A-Za-z0-9+/]*={0,2}$/u.test(normalized)) {
-        throw new Error(`Draft image attachment ${index + 1} (${fileName}) is not valid base64.`);
-    }
-    const decoded = Buffer.from(normalized, "base64");
-    if (decoded.length === 0 || decoded.toString("base64") !== normalized) {
-        throw new Error(`Draft image attachment ${index + 1} (${fileName}) is not valid base64.`);
+        await fs.writeFile(absolutePath, attachment.textContent ?? "", { encoding: "utf8", mode: 0o600 });
+        references.push({ index: attachment.index, path: path.relative(worktreeRoot, absolutePath).replaceAll(path.sep, "/") });
     }
-    return decoded;
+    return references;
 }
 function safeAttachmentFileName(fileName) {
     const sanitized = path.basename(fileName || "attachment.txt")

package/dist/version.d.ts

@@ -1 +1 @@
-export declare const forgeBridgeVersion = "0.1.3-dev.0b2f81d660c0";
+export declare const forgeBridgeVersion = "0.1.3-dev.34ddcb672775";

package/dist/version.js

@@ -1 +1 @@
-export const forgeBridgeVersion = "0.1.3-dev.0b2f81d660c0";
+export const forgeBridgeVersion = "0.1.3-dev.34ddcb672775";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@loucompanion/forge-bridge",
-  "version": "0.1.3-dev.0b2f81d660c0",
+  "version": "0.1.3-dev.34ddcb672775",
   "description": "Forge runner bridge CLI for connecting local developer machines to Forge dispatch.",
   "license": "UNLICENSED",
   "type": "module",

package/dist/shared/provider-capabilities.d.ts

@@ -1 +0,0 @@
-export declare function supportsDraftImageAttachmentsFor(cliProviders: readonly string[]): boolean;

package/dist/shared/provider-capabilities.js

@@ -1,3 +0,0 @@
-export function supportsDraftImageAttachmentsFor(cliProviders) {
-    return cliProviders.some((provider) => provider.trim().toLowerCase() === "codex");
-}