| 123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156 |
- import { PermissionV1 } from "@kirincode-ai/core/v1/permission"
- import { LayerNode } from "@kirincode-ai/core/effect/layer-node"
- import { describe, expect } from "bun:test"
- import path from "path"
- import { Effect } from "effect"
- import { CrossSpawnSpawner } from "@kirincode-ai/core/cross-spawn-spawner"
- import type { Tool } from "@/tool/tool"
- import { assertExternalDirectoryEffect } from "../../src/tool/external-directory"
- import { Filesystem } from "@/util/filesystem"
- import { TestInstance, tmpdirScoped } from "../fixture/fixture"
- import type { Permission } from "../../src/permission"
- import { SessionID, MessageID } from "../../src/session/schema"
- import { testEffect } from "../lib/effect"
- const it = testEffect(LayerNode.compile(CrossSpawnSpawner.node))
- const baseCtx: Omit<Tool.Context, "ask"> = {
- sessionID: SessionID.make("ses_test"),
- messageID: MessageID.make("msg_test"),
- callID: "",
- agent: "build",
- abort: AbortSignal.any([]),
- messages: [],
- metadata: () => Effect.void,
- }
- const glob = (p: string) =>
- process.platform === "win32" ? Filesystem.normalizePathPattern(p) : p.replaceAll("\\", "/")
- function makeCtx() {
- const requests: Array<Omit<PermissionV1.Request, "id" | "sessionID" | "tool">> = []
- const ctx: Tool.Context = {
- ...baseCtx,
- ask: (req) =>
- Effect.sync(() => {
- requests.push(req)
- }),
- }
- return { requests, ctx }
- }
- describe("tool.assertExternalDirectory", () => {
- it.live("no-ops for empty target", () =>
- Effect.gen(function* () {
- const { requests, ctx } = makeCtx()
- yield* assertExternalDirectoryEffect(ctx)
- expect(requests.length).toBe(0)
- }),
- )
- it.instance("no-ops for paths inside the instance directory", () =>
- Effect.gen(function* () {
- const test = yield* TestInstance
- const { requests, ctx } = makeCtx()
- yield* assertExternalDirectoryEffect(ctx, path.join(test.directory, "file.txt"))
- expect(requests.length).toBe(0)
- }),
- )
- it.instance("asks with a single canonical glob", () =>
- Effect.gen(function* () {
- const test = yield* TestInstance
- const { requests, ctx } = makeCtx()
- const target = path.join(path.dirname(test.directory), "outside", "file.txt")
- const expected = glob(path.join(path.dirname(target), "*"))
- yield* assertExternalDirectoryEffect(ctx, target)
- const req = requests.find((r) => r.permission === "external_directory")
- expect(req).toBeDefined()
- expect(req!.patterns).toEqual([expected])
- expect(req!.always).toEqual([expected])
- }),
- )
- it.instance("uses target directory when kind=directory", () =>
- Effect.gen(function* () {
- const test = yield* TestInstance
- const { requests, ctx } = makeCtx()
- const target = path.join(path.dirname(test.directory), "outside")
- const expected = glob(path.join(target, "*"))
- yield* assertExternalDirectoryEffect(ctx, target, { kind: "directory" })
- const req = requests.find((r) => r.permission === "external_directory")
- expect(req).toBeDefined()
- expect(req!.patterns).toEqual([expected])
- expect(req!.always).toEqual([expected])
- }),
- )
- it.live("skips prompting when bypass=true", () =>
- Effect.gen(function* () {
- const { requests, ctx } = makeCtx()
- yield* assertExternalDirectoryEffect(ctx, "/tmp/outside/file.txt", { bypass: true })
- expect(requests.length).toBe(0)
- }),
- )
- if (process.platform === "win32") {
- it.instance(
- "normalizes Windows path variants to one glob",
- () =>
- Effect.gen(function* () {
- const { requests, ctx } = makeCtx()
- const outerTmp = yield* tmpdirScoped()
- yield* Effect.promise(() => Bun.write(path.join(outerTmp, "outside.txt"), "x"))
- const target = path.join(outerTmp, "outside.txt")
- const alt = target
- .replace(/^[A-Za-z]:/, "")
- .replaceAll("\\", "/")
- .toLowerCase()
- yield* assertExternalDirectoryEffect(ctx, alt)
- const req = requests.find((r) => r.permission === "external_directory")
- const expected = glob(path.join(outerTmp, "*"))
- expect(req).toBeDefined()
- expect(req!.patterns).toEqual([expected])
- expect(req!.always).toEqual([expected])
- }),
- { git: true },
- )
- it.instance(
- "uses drive root glob for root files",
- () =>
- Effect.gen(function* () {
- const { requests, ctx } = makeCtx()
- const tmp = yield* TestInstance
- const root = path.parse(tmp.directory).root
- const target = path.join(root, "boot.ini")
- yield* assertExternalDirectoryEffect(ctx, target)
- const req = requests.find((r) => r.permission === "external_directory")
- const expected = path.join(root, "*")
- expect(req).toBeDefined()
- expect(req!.patterns).toEqual([expected])
- expect(req!.always).toEqual([expected])
- }),
- { git: true },
- )
- }
- })
|