Skip to content

Commit d4a5a09

Browse files
committed
fix: lint and type errors
• Replace 'any' types with proper TypeScript types in test and implementation files • Remove non-null assertions and use type casts where appropriate • Ensure type safety in embedding provider and benchmark code
1 parent 1c3630e commit d4a5a09

File tree

5 files changed

+12
-6
lines changed

5 files changed

+12
-6
lines changed

packages/ai-tool-retriever/src/core/retriever.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -102,7 +102,7 @@ describe('ToolRetriever', () => {
102102
})
103103

104104
describe('strict mode', () => {
105-
let consoleWarnSpy: any
105+
let consoleWarnSpy: ReturnType<typeof vi.spyOn>
106106

107107
beforeEach(() => {
108108
consoleWarnSpy = vi.spyOn(console, 'warn').mockImplementation(() => {})

packages/ai-tool-retriever/src/providers/embedding/transformers.test.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ vi.mock('@xenova/transformers', () => ({
1717
const PIPELINE_PROMISE_SYMBOL = Symbol.for('ai-tool-retriever.pipeline_promise')
1818

1919
describe('TransformersEmbeddingProvider', () => {
20-
let TransformersEmbeddingProvider: any
20+
let TransformersEmbeddingProvider: typeof import('./transformers').TransformersEmbeddingProvider
2121

2222
beforeEach(async () => {
2323
vi.resetModules()
@@ -27,7 +27,7 @@ describe('TransformersEmbeddingProvider', () => {
2727

2828
afterEach(() => {
2929
vi.clearAllMocks()
30-
delete (globalThis as any)[PIPELINE_PROMISE_SYMBOL]
30+
delete (globalThis as Record<string | symbol, unknown>)[PIPELINE_PROMISE_SYMBOL]
3131
})
3232

3333
it('should initialize the pipeline only once on multiple create calls', async () => {

packages/ai-tool-retriever/src/providers/embedding/transformers.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -85,7 +85,7 @@ export class TransformersEmbeddingProvider implements EmbeddingProvider {
8585
})()
8686
}
8787

88-
return _global[PIPELINE_PROMISE_SYMBOL]!
88+
return _global[PIPELINE_PROMISE_SYMBOL] as Promise<FeatureExtractionPipeline>
8989
}
9090

9191
/**

packages/benchmark/src/search-scalability.ts

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,12 @@ import type { ToolDefinition } from 'ai-tool-retriever'
44
import { InMemoryStore } from 'ai-tool-retriever/providers/store/in-memory'
55
import { z } from 'zod'
66

7+
type EmbeddingProvider = {
8+
dimensions: number
9+
getFloatEmbedding: (text: string) => Promise<number[]>
10+
getFloatEmbeddingsBatch: (texts: string[]) => Promise<number[][]>
11+
}
12+
713
const mockEmbeddingProvider = {
814
dimensions: 4,
915
getFloatEmbedding: async () => [0, 0, 0, 0],
@@ -31,7 +37,7 @@ async function runBenchmark() {
3137

3238
for (const count of toolCounts) {
3339
const tools = createMockTools(count)
34-
await store.sync(tools, mockEmbeddingProvider as any)
40+
await store.sync(tools, mockEmbeddingProvider as EmbeddingProvider)
3541

3642
const searchIterations = 100
3743
const startTime = performance.now()

packages/e2e/src/cli.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ async function main() {
2323
}
2424

2525
process.exit(0)
26-
} catch (e) {
26+
} catch (e: unknown) {
2727
if (e instanceof ToolNotFoundError) {
2828
console.error(`E2E_ERROR: ${e.message}`)
2929
process.exit(10) // custom exit code for specific error

0 commit comments

Comments
 (0)