637 lines · 20.3 KB · TypeScript
RawDownload
import * as NodeServices from "@effect/platform-node/NodeServices";import { it } from "@effect/vitest";import { Effect, FileSystem, Layer, Path, Result } from "effect";import { expect } from "vitest"; import { ServerConfig } from "../../config.ts";import { CodexTextGenerationLive } from "./CodexTextGeneration.ts";import { TextGenerationError } from "@t3tools/contracts";import { TextGeneration } from "../Services/TextGeneration.ts";import { ServerSettingsService } from "../../serverSettings.ts"; const DEFAULT_TEST_MODEL_SELECTION = {  provider: "codex" as const,  model: "gpt-5.4-mini",}; const CodexTextGenerationTestLayer = CodexTextGenerationLive.pipe(  Layer.provideMerge(ServerSettingsService.layerTest()),  Layer.provideMerge(    ServerConfig.layerTest(process.cwd(), {      prefix: "t3code-codex-text-generation-test-",    }),  ),  Layer.provideMerge(NodeServices.layer),); function makeFakeCodexBinary(  dir: string,  input: {    output: string;    exitCode?: number;    stderr?: string;    requireImage?: boolean;    requireFastServiceTier?: boolean;    requireReasoningEffort?: string;    forbidReasoningEffort?: boolean;    stdinMustContain?: string;    stdinMustNotContain?: string;  },) {  return Effect.gen(function* () {    const fs = yield* FileSystem.FileSystem;    const path = yield* Path.Path;    const binDir = path.join(dir, "bin");    const codexPath = path.join(binDir, "codex");    yield* fs.makeDirectory(binDir, { recursive: true });     yield* fs.writeFileString(      codexPath,      [        "#!/bin/sh",        'output_path=""',        'seen_image="0"',        'seen_fast_service_tier="0"',        'seen_reasoning_effort=""',        "while [ $# -gt 0 ]; do",        '  if [ "$1" = "--image" ]; then',        "    shift",        '    if [ -n "$1" ]; then',        '      seen_image="1"',        "    fi",        "    shift",        "    continue",        "  fi",        '  if [ "$1" = "--config" ]; then',        "    shift",        '    if [ "$1" = "service_tier=\\"fast\\"" ]; then',        '      seen_fast_service_tier="1"',        "    fi",        '    case "$1" in',        "      model_reasoning_effort=*)",        '        seen_reasoning_effort="$1"',        "        ;;",        "    esac",        "    shift",        "    continue",        "  fi",        '  if [ "$1" = "--output-last-message" ]; then',        "    shift",        '    output_path="$1"',        "    shift",        "    continue",        "  fi",        "  shift",        "done",        'stdin_content="$(cat)"',        ...(input.requireImage          ? [              'if [ "$seen_image" != "1" ]; then',              '  printf "%s\\n" "missing --image input" >&2',              `  exit 2`,              "fi",            ]          : []),        ...(input.requireFastServiceTier          ? [              'if [ "$seen_fast_service_tier" != "1" ]; then',              '  printf "%s\\n" "missing fast service tier config" >&2',              `  exit 5`,              "fi",            ]          : []),        ...(input.requireReasoningEffort !== undefined          ? [              `if [ "$seen_reasoning_effort" != "model_reasoning_effort=\\"${input.requireReasoningEffort}\\"" ]; then`,              '  printf "%s\\n" "unexpected reasoning effort config: $seen_reasoning_effort" >&2',              `  exit 6`,              "fi",            ]          : []),        ...(input.forbidReasoningEffort          ? [              'if [ -n "$seen_reasoning_effort" ]; then',              '  printf "%s\\n" "reasoning effort config should be omitted: $seen_reasoning_effort" >&2',              `  exit 7`,              "fi",            ]          : []),        ...(input.stdinMustContain !== undefined          ? [              `if ! printf "%s" "$stdin_content" | grep -F -- ${JSON.stringify(input.stdinMustContain)} >/dev/null; then`,              '  printf "%s\\n" "stdin missing expected content" >&2',              `  exit 3`,              "fi",            ]          : []),        ...(input.stdinMustNotContain !== undefined          ? [              `if printf "%s" "$stdin_content" | grep -F -- ${JSON.stringify(input.stdinMustNotContain)} >/dev/null; then`,              '  printf "%s\\n" "stdin contained forbidden content" >&2',              `  exit 4`,              "fi",            ]          : []),        ...(input.stderr !== undefined          ? [`printf "%s\\n" ${JSON.stringify(input.stderr)} >&2`]          : []),        'if [ -n "$output_path" ]; then',        "  cat > \"$output_path\" <<'__T3CODE_FAKE_CODEX_OUTPUT__'",        input.output,        "__T3CODE_FAKE_CODEX_OUTPUT__",        "fi",        `exit ${input.exitCode ?? 0}`,        "",      ].join("\n"),    );    yield* fs.chmod(codexPath, 0o755);    return codexPath;  });} function withFakeCodexEnv<A, E, R>(  input: {    output: string;    exitCode?: number;    stderr?: string;    requireImage?: boolean;    requireFastServiceTier?: boolean;    requireReasoningEffort?: string;    forbidReasoningEffort?: boolean;    stdinMustContain?: string;    stdinMustNotContain?: string;  },  effect: Effect.Effect<A, E, R>,) {  return Effect.acquireUseRelease(    Effect.gen(function* () {      const fs = yield* FileSystem.FileSystem;      const tempDir = yield* fs.makeTempDirectoryScoped({ prefix: "t3code-codex-text-" });      const codexPath = yield* makeFakeCodexBinary(tempDir, input);      const serverSettings = yield* ServerSettingsService;      const previousSettings = yield* serverSettings.getSettings;      yield* serverSettings.updateSettings({        providers: {          codex: {            binaryPath: codexPath,          },        },      });      return { serverSettings, previousBinaryPath: previousSettings.providers.codex.binaryPath };    }),    () => effect,    ({ serverSettings, previousBinaryPath }) =>      serverSettings        .updateSettings({          providers: {            codex: {              binaryPath: previousBinaryPath,            },          },        })        .pipe(Effect.asVoid),  );} it.layer(CodexTextGenerationTestLayer)("CodexTextGenerationLive", (it) => {  it.effect("generates and sanitizes commit messages without branch by default", () =>    withFakeCodexEnv(      {        output: JSON.stringify({          subject:            "  Add important change to the system with too much detail and a trailing period.\nsecondary line",          body: "\n- added migration\n- updated tests\n",        }),        stdinMustNotContain: "branch must be a short semantic git branch fragment",      },      Effect.gen(function* () {        const textGeneration = yield* TextGeneration;         const generated = yield* textGeneration.generateCommitMessage({          cwd: process.cwd(),          branch: "feature/codex-effect",          stagedSummary: "M README.md",          stagedPatch: "diff --git a/README.md b/README.md",          modelSelection: DEFAULT_TEST_MODEL_SELECTION,        });         expect(generated.subject.length).toBeLessThanOrEqual(72);        expect(generated.subject.endsWith(".")).toBe(false);        expect(generated.body).toBe("- added migration\n- updated tests");        expect(generated.branch).toBeUndefined();      }),    ),  );   it.effect(    "forwards codex fast mode and non-default reasoning effort into codex exec config",    () =>      withFakeCodexEnv(        {          output: JSON.stringify({            subject: "Add important change",            body: "",          }),          requireFastServiceTier: true,          requireReasoningEffort: "xhigh",          stdinMustNotContain: "branch must be a short semantic git branch fragment",        },        Effect.gen(function* () {          const textGeneration = yield* TextGeneration;           yield* textGeneration.generateCommitMessage({            cwd: process.cwd(),            branch: "feature/codex-effect",            stagedSummary: "M README.md",            stagedPatch: "diff --git a/README.md b/README.md",            modelSelection: {              provider: "codex",              model: "gpt-5.4",              options: {                reasoningEffort: "xhigh",                fastMode: true,              },            },          });        }),      ),  );   it.effect("defaults git text generation codex effort to low", () =>    withFakeCodexEnv(      {        output: JSON.stringify({          subject: "Add important change",          body: "",        }),        requireReasoningEffort: "low",      },      Effect.gen(function* () {        const textGeneration = yield* TextGeneration;         yield* textGeneration.generateCommitMessage({          cwd: process.cwd(),          branch: "feature/codex-effect",          stagedSummary: "M README.md",          stagedPatch: "diff --git a/README.md b/README.md",          modelSelection: DEFAULT_TEST_MODEL_SELECTION,        });      }),    ),  );   it.effect("generates commit message with branch when includeBranch is true", () =>    withFakeCodexEnv(      {        output: JSON.stringify({          subject: "Add important change",          body: "",          branch: "fix/important-system-change",        }),        stdinMustContain: "branch must be a short semantic git branch fragment",      },      Effect.gen(function* () {        const textGeneration = yield* TextGeneration;         const generated = yield* textGeneration.generateCommitMessage({          cwd: process.cwd(),          branch: "feature/codex-effect",          stagedSummary: "M README.md",          stagedPatch: "diff --git a/README.md b/README.md",          includeBranch: true,          modelSelection: DEFAULT_TEST_MODEL_SELECTION,        });         expect(generated.subject).toBe("Add important change");        expect(generated.branch).toBe("feature/fix/important-system-change");      }),    ),  );   it.effect("generates PR content and trims markdown body", () =>    withFakeCodexEnv(      {        output: JSON.stringify({          title: "  Improve orchestration flow\nwith ignored suffix",          body: "\n## Summary\n- improve flow\n\n## Testing\n- bun test\n\n",        }),      },      Effect.gen(function* () {        const textGeneration = yield* TextGeneration;         const generated = yield* textGeneration.generatePrContent({          cwd: process.cwd(),          baseBranch: "main",          headBranch: "feature/codex-effect",          commitSummary: "feat: improve orchestration flow",          diffSummary: "2 files changed",          diffPatch: "diff --git a/a.ts b/a.ts",          modelSelection: DEFAULT_TEST_MODEL_SELECTION,        });         expect(generated.title).toBe("Improve orchestration flow");        expect(generated.body.startsWith("## Summary")).toBe(true);        expect(generated.body.endsWith("\n\n")).toBe(false);      }),    ),  );   it.effect("generates branch names and normalizes branch fragments", () =>    withFakeCodexEnv(      {        output: JSON.stringify({          branch: "  Feat/Session  ",        }),        stdinMustNotContain: "Image attachments supplied to the model",      },      Effect.gen(function* () {        const textGeneration = yield* TextGeneration;         const generated = yield* textGeneration.generateBranchName({          cwd: process.cwd(),          message: "Please update session handling.",          modelSelection: DEFAULT_TEST_MODEL_SELECTION,        });         expect(generated.branch).toBe("feat/session");      }),    ),  );   it.effect("generates thread titles and trims them for sidebar use", () =>    withFakeCodexEnv(      {        output: JSON.stringify({          title:            '  "Investigate websocket reconnect regressions after worktree restore"  \nignored line',        }),      },      Effect.gen(function* () {        const textGeneration = yield* TextGeneration;         const generated = yield* textGeneration.generateThreadTitle({          cwd: process.cwd(),          message: "Please investigate websocket reconnect regressions after a worktree restore.",          modelSelection: DEFAULT_TEST_MODEL_SELECTION,        });         expect(generated.title).toBe("Investigate websocket reconnect regressions aft...");      }),    ),  );   it.effect("falls back when thread title normalization becomes whitespace-only", () =>    withFakeCodexEnv(      {        output: JSON.stringify({          title: '  """   """  ',        }),      },      Effect.gen(function* () {        const textGeneration = yield* TextGeneration;         const generated = yield* textGeneration.generateThreadTitle({          cwd: process.cwd(),          message: "Name this thread.",          modelSelection: DEFAULT_TEST_MODEL_SELECTION,        });         expect(generated.title).toBe("New thread");      }),    ),  );   it.effect("trims whitespace exposed after quote removal in thread titles", () =>    withFakeCodexEnv(      {        output: JSON.stringify({          title: `  "' hello world '"  `,        }),      },      Effect.gen(function* () {        const textGeneration = yield* TextGeneration;         const generated = yield* textGeneration.generateThreadTitle({          cwd: process.cwd(),          message: "Name this thread.",          modelSelection: DEFAULT_TEST_MODEL_SELECTION,        });         expect(generated.title).toBe("hello world");      }),    ),  );   it.effect("omits attachment metadata section when no attachments are provided", () =>    withFakeCodexEnv(      {        output: JSON.stringify({          branch: "fix/session-timeout",        }),        stdinMustNotContain: "Attachment metadata:",      },      Effect.gen(function* () {        const textGeneration = yield* TextGeneration;         const generated = yield* textGeneration.generateBranchName({          cwd: process.cwd(),          message: "Fix timeout behavior.",          modelSelection: DEFAULT_TEST_MODEL_SELECTION,        });         expect(generated.branch).toBe("fix/session-timeout");      }),    ),  );   it.effect("passes image attachments through as codex image inputs", () =>    withFakeCodexEnv(      {        output: JSON.stringify({          branch: "fix/ui-regression",        }),        requireImage: true,        stdinMustContain: "Attachment metadata:",      },      Effect.gen(function* () {        const fs = yield* FileSystem.FileSystem;        const path = yield* Path.Path;        const { attachmentsDir } = yield* ServerConfig;        const attachmentId = `thread-branch-image-${Date.now()}-${Math.random().toString(36).slice(2, 10)}`;        const attachmentPath = path.join(attachmentsDir, `${attachmentId}.png`);        yield* fs.makeDirectory(attachmentsDir, { recursive: true });        yield* fs.writeFile(attachmentPath, Buffer.from("hello"));         const textGeneration = yield* TextGeneration;        const generated = yield* textGeneration.generateBranchName({          modelSelection: DEFAULT_TEST_MODEL_SELECTION,          cwd: process.cwd(),          message: "Fix layout bug from screenshot.",          attachments: [            {              type: "image",              id: attachmentId,              name: "bug.png",              mimeType: "image/png",              sizeBytes: 5,            },          ],        });         expect(generated.branch).toBe("fix/ui-regression");      }),    ),  );   it.effect("resolves persisted attachment ids to files for codex image inputs", () =>    withFakeCodexEnv(      {        output: JSON.stringify({          branch: "fix/ui-regression",        }),        requireImage: true,      },      Effect.gen(function* () {        const fs = yield* FileSystem.FileSystem;        const path = yield* Path.Path;        const { attachmentsDir } = yield* ServerConfig;        const attachmentId = `thread-1-${Date.now()}-${Math.random().toString(36).slice(2, 10)}`;        const imagePath = path.join(attachmentsDir, `${attachmentId}.png`);        yield* fs.makeDirectory(attachmentsDir, { recursive: true });

Lines 1–500 of 637.