Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -52,9 +52,9 @@
"@opencode-ai/sdk": "1.14.41"
},
"devDependencies": {
"@bfra.me/eslint-config": "0.51.0",
"@bfra.me/prettier-config": "0.16.8",
"@bfra.me/tsconfig": "0.13.0",
"@bfra.me/eslint-config": "0.51.1",
"@bfra.me/prettier-config": "0.16.9",
"@bfra.me/tsconfig": "0.13.1",
"@octokit/webhooks-types": "7.6.1",
"@semantic-release/exec": "7.1.0",
"@semantic-release/git": "10.0.1",
Expand Down
2 changes: 1 addition & 1 deletion packages/runtime/src/agent/prompt.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -1428,7 +1428,7 @@ function createMockTriggerContext(overrides: Partial<TriggerContext> = {}): Trig
hasMention: true,
command: null,
isBotReviewRequested: false,
raw: {payload: {action: 'created'}} as unknown,
raw: {payload: {action: 'created'}},
...overrides,
}
}
Expand Down
6 changes: 3 additions & 3 deletions packages/runtime/src/session/storage-message-mappers.ts
Original file line number Diff line number Diff line change
Expand Up @@ -100,7 +100,7 @@ export function mapSdkPartToPart(p: SdkPart): Part {

export function mapSdkMessageToMessage(m: SdkMessageExtended): Message {
if (m.role === 'user') {
const user = m as SdkUserMessage & {variant?: string}
const user = m
return {
id: user.id,
sessionID: user.sessionID,
Expand All @@ -124,7 +124,7 @@ export function mapSdkMessageToMessage(m: SdkMessageExtended): Message {
variant: user.variant,
}
}
const assistant = m as SdkAssistantMessage & {agent?: string}
const assistant = m
return {
id: assistant.id,
sessionID: assistant.sessionID,
Expand Down Expand Up @@ -155,7 +155,7 @@ export function mapSdkMessages(messages: readonly (SdkMessage | SdkMessageWithPa
return [
...messages.map(item => {
const sdkMessage = 'info' in item ? item.info : item
const message = mapSdkMessageToMessage(sdkMessage as SdkMessageExtended)
const message = mapSdkMessageToMessage(sdkMessage)
const parts = 'parts' in item ? item.parts.map(mapSdkPartToPart) : undefined
return parts == null || parts.length === 0 ? message : ({...message, parts} as unknown as Message)
}),
Expand Down
2 changes: 1 addition & 1 deletion packages/runtime/src/session/title-reassert.ts
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ export async function reassertSessionTitle(
try {
const response = await client.session.update({
path: {id: sessionId},
body: {title} as Record<string, unknown>,
body: {title},
})

if (response.error != null) {
Expand Down
4 changes: 2 additions & 2 deletions packages/runtime/src/shared/logger.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -160,7 +160,7 @@ describe('redactSensitiveFields', () => {
name: 'test',
},
}
const result = redactSensitiveFields(input) as {config: {apiKey: string; name: string}}
const result = redactSensitiveFields(input)
expect(result.config.apiKey).toBe('[REDACTED]')
expect(result.config.name).toBe('test')
})
Expand All @@ -169,7 +169,7 @@ describe('redactSensitiveFields', () => {
const input = {
items: [{secret: 'hidden', value: 'visible'}],
}
const result = redactSensitiveFields(input) as {items: {secret: string; value: string}[]}
const result = redactSensitiveFields(input)
expect(result.items[0]?.secret).toBe('[REDACTED]')
expect(result.items[0]?.value).toBe('visible')
})
Expand Down
329 changes: 207 additions & 122 deletions pnpm-lock.yaml

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion src/features/agent/context.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@ function createMockTriggerContext(overrides: Partial<TriggerContext> = {}): Trig
hasMention: false,
command: null,
isBotReviewRequested: false,
raw: {} as TriggerContext['raw'],
raw: {},
...overrides,
}
}
Expand Down
2 changes: 1 addition & 1 deletion src/features/agent/diff-context.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ function createTriggerContext(overrides: Partial<TriggerContext> = {}): TriggerC
hasMention: false,
command: null,
isBotReviewRequested: false,
raw: {} as TriggerContext['raw'],
raw: {},
...overrides,
}
}
Expand Down
2 changes: 1 addition & 1 deletion src/features/agent/opencode.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -261,7 +261,7 @@ function createMockServerHandle(options: {
handle: {
client: options.client as unknown as OpenCodeServerHandle['client'],
server: mockServer as unknown as OpenCodeServerHandle['server'],
shutdown: vi.fn() as unknown as () => void,
shutdown: vi.fn(),
},
mockServer,
}
Expand Down
2 changes: 1 addition & 1 deletion src/harness/phases/acquire-lock.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ import {createMockLogger} from '../../shared/test-helpers.js'
import {runAcquireLock} from './acquire-lock.js'

const acquireLockMock = vi.hoisted(() => vi.fn())
const createS3AdapterMock = vi.hoisted(() => vi.fn(() => ({}) as unknown))
const createS3AdapterMock = vi.hoisted(() => vi.fn(() => ({})))

vi.mock('@fro-bot/runtime', async () => {
const actual = await vi.importActual<typeof import('@fro-bot/runtime')>('@fro-bot/runtime')
Expand Down
2 changes: 1 addition & 1 deletion src/services/artifact/upload.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ describe('uploadLogArtifact', () => {
// #given the log directory exists but contains no files
const fs = await import('node:fs/promises')
vi.mocked(fs.access).mockResolvedValue(undefined)
vi.mocked(fs.readdir).mockResolvedValue([] as unknown as Awaited<ReturnType<typeof fs.readdir>>)
vi.mocked(fs.readdir).mockResolvedValue([])

const {uploadLogArtifact} = await import('./upload.js')
const logger = createMockLogger()
Expand Down
2 changes: 1 addition & 1 deletion src/services/setup/bun.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@ describe('bun', () => {

beforeEach(async () => {
mockLogger = createMockLogger()
mockAddPath = vi.fn() as unknown as (inputPath: string) => void
mockAddPath = vi.fn()
originalPlatform = process.platform
originalArch = process.arch

Expand Down
6 changes: 3 additions & 3 deletions src/shared/validation.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -86,11 +86,11 @@ describe('validateNonEmptyString', () => {
})

it('rejects non-string types - number', () => {
expect(() => validateNonEmptyString(123 as unknown, 'name')).toThrow('name must be a string, received number')
expect(() => validateNonEmptyString(123, 'name')).toThrow('name must be a string, received number')
})

it('rejects non-string types - null', () => {
expect(() => validateNonEmptyString(null as unknown, 'name')).toThrow('name must be a string, received object')
expect(() => validateNonEmptyString(null, 'name')).toThrow('name must be a string, received object')
})

it('rejects non-string types - undefined', () => {
Expand All @@ -100,6 +100,6 @@ describe('validateNonEmptyString', () => {
})

it('rejects non-string types - object', () => {
expect(() => validateNonEmptyString({} as unknown, 'name')).toThrow('name must be a string, received object')
expect(() => validateNonEmptyString({}, 'name')).toThrow('name must be a string, received object')
})
})
Loading