fix(bitbucket): overfetch small pipeline log tails

Author: Bill Leoutsakos

apps/sim/tools/bitbucket/pipelines.test.ts

@@ -398,6 +398,19 @@ describe('Bitbucket pipeline step logs', () => {
     })
   })
 
+  it('overfetches a provider-compatible minimum for small log tails', () => {
+    const params = {
+      ...REPOSITORY_PARAMS,
+      pipelineUuid: '{pipeline-1}',
+      stepUuid: '{step-1}',
+      maxCharacters: 100,
+    } satisfies BitbucketGetPipelineStepLogParams
+
+    expect(bitbucketGetPipelineStepLogTool.request.headers(params)).toMatchObject({
+      Range: 'bytes=-4096',
+    })
+  })
+
   it('trims the partial leading line of a ranged log and reports total bytes', async () => {
     const body = 'ise\nFAILED: expected 1 to be 2\n'
     const result = await bitbucketGetPipelineStepLogTool.transformResponse!(

apps/sim/tools/bitbucket/utils.ts

@@ -25,6 +25,7 @@ export const BITBUCKET_DEFAULT_MAX_CHARACTERS = 100_000
 export const BITBUCKET_MAX_CHARACTERS = 500_000
 export const BITBUCKET_DEFAULT_LOG_CHARACTERS = 20_000
 export const BITBUCKET_MAX_LOG_CHARACTERS = 200_000
+export const BITBUCKET_MIN_LOG_RANGE_BYTES = 4_096
 export const BITBUCKET_RAW_TRANSFER_MAX_BYTES = 10 * 1024 * 1024
 
 export const BITBUCKET_READ_RETRY: ToolRetryConfig = {
@@ -242,7 +243,10 @@ export function bitbucketHeadRange(maxCharacters: number | undefined): string {
 }
 
 export function bitbucketTailRange(maxCharacters: number | undefined): string {
-  const byteLimit = bitbucketMaxCharacters(maxCharacters, true) * 4
+  const byteLimit = Math.max(
+    bitbucketMaxCharacters(maxCharacters, true) * 4,
+    BITBUCKET_MIN_LOG_RANGE_BYTES
+  )
   return `bytes=-${byteLimit}`
 }