| 123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150 |
- import { describe, expect, test } from "bun:test"
- import path from "path"
- import { ReadTool } from "../../src/tool/read"
- import { Instance } from "../../src/project/instance"
- import { tmpdir } from "../fixture/fixture"
- import type { PermissionNext } from "../../src/permission/next"
- const ctx = {
- sessionID: "test",
- messageID: "",
- callID: "",
- agent: "build",
- abort: AbortSignal.any([]),
- metadata: () => {},
- ask: async () => {},
- }
- describe("tool.read external_directory permission", () => {
- test("allows reading absolute path inside project directory", async () => {
- await using tmp = await tmpdir({
- init: async (dir) => {
- await Bun.write(path.join(dir, "test.txt"), "hello world")
- },
- })
- await Instance.provide({
- directory: tmp.path,
- fn: async () => {
- const read = await ReadTool.init()
- const result = await read.execute({ filePath: path.join(tmp.path, "test.txt") }, ctx)
- expect(result.output).toContain("hello world")
- },
- })
- })
- test("allows reading file in subdirectory inside project directory", async () => {
- await using tmp = await tmpdir({
- init: async (dir) => {
- await Bun.write(path.join(dir, "subdir", "test.txt"), "nested content")
- },
- })
- await Instance.provide({
- directory: tmp.path,
- fn: async () => {
- const read = await ReadTool.init()
- const result = await read.execute({ filePath: path.join(tmp.path, "subdir", "test.txt") }, ctx)
- expect(result.output).toContain("nested content")
- },
- })
- })
- test("asks for external_directory permission when reading absolute path outside project", async () => {
- await using outerTmp = await tmpdir({
- init: async (dir) => {
- await Bun.write(path.join(dir, "secret.txt"), "secret data")
- },
- })
- await using tmp = await tmpdir({ git: true })
- await Instance.provide({
- directory: tmp.path,
- fn: async () => {
- const read = await ReadTool.init()
- const requests: Array<Omit<PermissionNext.Request, "id" | "sessionID" | "tool">> = []
- const testCtx = {
- ...ctx,
- ask: async (req: Omit<PermissionNext.Request, "id" | "sessionID" | "tool">) => {
- requests.push(req)
- },
- }
- await read.execute({ filePath: path.join(outerTmp.path, "secret.txt") }, testCtx)
- const extDirReq = requests.find((r) => r.permission === "external_directory")
- expect(extDirReq).toBeDefined()
- expect(extDirReq!.patterns.some((p) => p.includes(outerTmp.path))).toBe(true)
- },
- })
- })
- test("asks for external_directory permission when reading relative path outside project", async () => {
- await using tmp = await tmpdir({ git: true })
- await Instance.provide({
- directory: tmp.path,
- fn: async () => {
- const read = await ReadTool.init()
- const requests: Array<Omit<PermissionNext.Request, "id" | "sessionID" | "tool">> = []
- const testCtx = {
- ...ctx,
- ask: async (req: Omit<PermissionNext.Request, "id" | "sessionID" | "tool">) => {
- requests.push(req)
- },
- }
- // This will fail because file doesn't exist, but we can check if permission was asked
- await read.execute({ filePath: "../outside.txt" }, testCtx).catch(() => {})
- const extDirReq = requests.find((r) => r.permission === "external_directory")
- expect(extDirReq).toBeDefined()
- },
- })
- })
- test("does not ask for external_directory permission when reading inside project", async () => {
- await using tmp = await tmpdir({
- git: true,
- init: async (dir) => {
- await Bun.write(path.join(dir, "internal.txt"), "internal content")
- },
- })
- await Instance.provide({
- directory: tmp.path,
- fn: async () => {
- const read = await ReadTool.init()
- const requests: Array<Omit<PermissionNext.Request, "id" | "sessionID" | "tool">> = []
- const testCtx = {
- ...ctx,
- ask: async (req: Omit<PermissionNext.Request, "id" | "sessionID" | "tool">) => {
- requests.push(req)
- },
- }
- await read.execute({ filePath: path.join(tmp.path, "internal.txt") }, testCtx)
- const extDirReq = requests.find((r) => r.permission === "external_directory")
- expect(extDirReq).toBeUndefined()
- },
- })
- })
- })
- describe("tool.read env file blocking", () => {
- test.each([
- [".env", true],
- [".env.local", true],
- [".env.production", true],
- [".env.sample", false],
- [".env.example", false],
- [".envrc", false],
- ["environment.ts", false],
- ])("%s blocked=%s", async (filename, blocked) => {
- await using tmp = await tmpdir({
- init: (dir) => Bun.write(path.join(dir, filename), "content"),
- })
- await Instance.provide({
- directory: tmp.path,
- fn: async () => {
- const read = await ReadTool.init()
- const promise = read.execute({ filePath: path.join(tmp.path, filename) }, ctx)
- if (blocked) {
- await expect(promise).rejects.toThrow("blocked")
- } else {
- expect((await promise).output).toContain("content")
- }
- },
- })
- })
- })
|