| 123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146 |
- /** The `readBytes` endpoint: the byte window it cuts, its defaults and cap, and the gates it shares with `read`. */
- import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest'
- import { mkdir, writeFile } from 'node:fs/promises'
- import { join } from 'node:path'
- import { FsVersion } from '@deepseek-ai/dsh-fs'
- import { agent, failureOf, openWorkspace, signal, type Harness } from './harness.ts'
- let harness: Harness
- let workspace: string
- beforeEach(async () => {
- harness = await openWorkspace('dsh-workspace-files-read-bytes-')
- workspace = harness.workspace
- })
- afterEach(async () => {
- await harness.dispose()
- })
- const endpoint = (caps?: { maxBytes?: number }): ReturnType<Harness['endpoint']> => harness.endpoint(caps)
- /** 256 bytes, each equal to its offset: the window's content names its position. */
- const RAMP = Buffer.from(Array.from({ length: 256 }, (_, i) => i))
- const decode = (data: string): Buffer => Buffer.from(data, 'base64')
- describe('workspaceFiles.readBytes — the window', () => {
- it('returns the whole file as one window by default, with its absolute path, version, and size', async () => {
- await writeFile(join(workspace, 'ramp.bin'), RAMP)
- const result = await endpoint().readBytes(agent, 'ramp.bin', {}, signal())
- expect(decode(result.data).equals(RAMP)).toBe(true)
- expect(result).toMatchObject({ offset: 0, eof: true, bytes: 256 })
- expect(result.absolutePath.endsWith('ramp.bin')).toBe(true)
- expect(result.version.length).toBeGreaterThan(0)
- })
- it('cuts the requested window and reports that more follows', async () => {
- await writeFile(join(workspace, 'ramp.bin'), RAMP)
- const result = await endpoint().readBytes(agent, 'ramp.bin', { offset: 16, length: 8 }, signal())
- expect([...decode(result.data)]).toEqual([16, 17, 18, 19, 20, 21, 22, 23])
- expect(result).toMatchObject({ offset: 16, eof: false, bytes: 256 })
- })
- it('reads a window of a file far above the byte cap', async () => {
- await writeFile(join(workspace, 'huge.bin'), Buffer.alloc(200_000, 7))
- const result = await endpoint({ maxBytes: 1024 }).readBytes(agent, 'huge.bin', { offset: 199_000, length: 1024 }, signal())
- expect(decode(result.data)).toHaveLength(1000)
- expect(result).toMatchObject({ eof: true, bytes: 200_000 })
- })
- it('infers eof from a short window when the backend reports no size', async () => {
- await writeFile(join(workspace, 'ramp.bin'), RAMP)
- vi.spyOn(harness.ctx.fs, 'stat').mockResolvedValue({ version: FsVersion('v-sizeless'), type: 'file' })
- const full = await endpoint().readBytes(agent, 'ramp.bin', { offset: 0, length: 256 }, signal())
- expect(full.eof).toBe(false)
- const short = await endpoint().readBytes(agent, 'ramp.bin', { offset: 250, length: 10 }, signal())
- expect(short).toMatchObject({ eof: true })
- expect(short.bytes).toBeUndefined()
- })
- it('reports eof on the window that holds the last byte, whether or not the length is reached', async () => {
- await writeFile(join(workspace, 'ramp.bin'), RAMP)
- const exact = await endpoint().readBytes(agent, 'ramp.bin', { offset: 248, length: 8 }, signal())
- expect(exact.eof).toBe(true)
- expect(decode(exact.data)).toHaveLength(8)
- const short = await endpoint().readBytes(agent, 'ramp.bin', { offset: 250, length: 100 }, signal())
- expect(short.eof).toBe(true)
- expect([...decode(short.data)]).toEqual([250, 251, 252, 253, 254, 255])
- })
- it('returns an empty eof window for an offset at or past the end', async () => {
- await writeFile(join(workspace, 'ramp.bin'), RAMP)
- const result = await endpoint().readBytes(agent, 'ramp.bin', { offset: 300, length: 8 }, signal())
- expect(result).toMatchObject({ data: '', offset: 300, eof: true, bytes: 256 })
- })
- it('returns an empty eof window for an empty file', async () => {
- await writeFile(join(workspace, 'empty.bin'), Buffer.alloc(0))
- const result = await endpoint().readBytes(agent, 'empty.bin', {}, signal())
- expect(result).toMatchObject({ data: '', offset: 0, eof: true, bytes: 0 })
- })
- it('carries bytes a text read would refuse: NUL and invalid UTF-8 round-trip through base64', async () => {
- const raw = Buffer.from([0, 0xff, 0xfe, 0x80, 0x41, 0])
- await writeFile(join(workspace, 'blob.bin'), raw)
- const result = await endpoint().readBytes(agent, 'blob.bin', {}, signal())
- expect(decode(result.data).equals(raw)).toBe(true)
- })
- it('names the version a stat of the same file reports', async () => {
- await writeFile(join(workspace, 'ramp.bin'), RAMP)
- const stat = await endpoint().stat(agent, 'ramp.bin', signal())
- const result = await endpoint().readBytes(agent, 'ramp.bin', {}, signal())
- expect(result.version).toBe(stat.version)
- })
- })
- describe('workspaceFiles.readBytes — defaults and cap', () => {
- it('defaults the length to the configured byte cap', async () => {
- await writeFile(join(workspace, 'ramp.bin'), RAMP.subarray(0, 64))
- const result = await endpoint({ maxBytes: 64 }).readBytes(agent, 'ramp.bin', {}, signal())
- expect(decode(result.data)).toHaveLength(64)
- expect(result.eof).toBe(true)
- })
- it('refuses a window longer than the cap as too-large rather than shortening it', async () => {
- await writeFile(join(workspace, 'ramp.bin'), RAMP)
- const failure = await failureOf(endpoint({ maxBytes: 64 }).readBytes(agent, 'ramp.bin', { length: 65 }, signal()))
- expect(failure.code).toBe('workspace-file/too-large')
- expect(failure.details).toMatchObject({ limit: 64 })
- })
- it('accepts a window exactly at the cap', async () => {
- await writeFile(join(workspace, 'ramp.bin'), RAMP.subarray(0, 64))
- const result = await endpoint({ maxBytes: 64 }).readBytes(agent, 'ramp.bin', { length: 64 }, signal())
- expect(decode(result.data)).toHaveLength(64)
- })
- it('refuses a negative or fractional offset and a non-positive length as bad requests', async () => {
- await writeFile(join(workspace, 'ramp.bin'), RAMP)
- // Beyond-safe integers and a window whose end overflows are refused too: they cannot index a file.
- const ranges = [
- { offset: -1 }, { offset: 1.5 }, { length: 0 }, { length: 2.5 },
- { offset: 2 ** 53 }, { offset: Number.MAX_SAFE_INTEGER, length: 2 },
- ]
- for (const range of ranges) {
- const failure = await failureOf(endpoint().readBytes(agent, 'ramp.bin', range, signal()))
- expect(failure.code).toBe('gateway/bad-request')
- }
- })
- })
- describe('workspaceFiles.readBytes — the gates it shares with read', () => {
- it('rejects a directory, a missing path, and an empty path', async () => {
- await mkdir(join(workspace, 'dir'))
- expect((await failureOf(endpoint().readBytes(agent, 'dir', {}, signal()))).code).toBe('workspace-file/not-regular-file')
- expect((await failureOf(endpoint().readBytes(agent, 'missing.bin', {}, signal()))).code).toBe('workspace-file/not-found')
- expect((await failureOf(endpoint().readBytes(agent, '', {}, signal()))).code).toBe('gateway/bad-request')
- })
- it('rejects an absolute path outside the workspace', async () => {
- await writeFile(join(harness.outside, 'secret.bin'), RAMP)
- const failure = await failureOf(endpoint().readBytes(agent, join(harness.outside, 'secret.bin'), {}, signal()))
- expect(failure.code).toBe('workspace-file/outside-workspace')
- })
- })
|