Skip to content

Commit e684c8a

Browse files
committed
fix: lint and formatting issues
1 parent 4d85cd5 commit e684c8a

File tree

2 files changed

+18
-6
lines changed

2 files changed

+18
-6
lines changed

src/browser/hooks/useWorkspaceName.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -76,7 +76,7 @@ export function useWorkspaceName(options: UseWorkspaceNameOptions): UseWorkspace
7676
const oldRequestId = requestIdRef.current;
7777
requestIdRef.current++;
7878
// Resolve any waiters so they don't hang forever
79-
if (generationPromiseRef.current && generationPromiseRef.current.requestId === oldRequestId) {
79+
if (generationPromiseRef.current?.requestId === oldRequestId) {
8080
generationPromiseRef.current.resolve("");
8181
generationPromiseRef.current = null;
8282
setIsGenerating(false);

src/node/services/workspaceTitleGenerator.test.ts

Lines changed: 17 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -2,17 +2,29 @@ import { describe, it, expect } from "bun:test";
22
import { getPreferredNameModel } from "./workspaceTitleGenerator";
33
import type { AIService } from "./aiService";
44
import { getKnownModel } from "@/common/constants/knownModels";
5+
import type { LanguageModel } from "ai";
6+
import type { Result } from "@/common/types/result";
7+
import type { SendMessageError } from "@/common/types/errors";
8+
9+
type CreateModelResult = Result<LanguageModel, SendMessageError>;
510

611
// Helper to create a mock AIService that succeeds for specific models
712
function createMockAIService(availableModels: string[]): AIService {
8-
return {
9-
createModel: async (modelString: string) => {
13+
const service: Partial<AIService> = {
14+
createModel: (modelString: string): Promise<CreateModelResult> => {
1015
if (availableModels.includes(modelString)) {
11-
return { success: true, data: {} as never };
16+
// Return a minimal success result - data is not used by getPreferredNameModel
17+
const result: CreateModelResult = { success: true, data: null as never };
18+
return Promise.resolve(result);
1219
}
13-
return { success: false, error: { type: "api_key_not_found", provider: "test" } };
20+
const err: CreateModelResult = {
21+
success: false,
22+
error: { type: "api_key_not_found", provider: "test" },
23+
};
24+
return Promise.resolve(err);
1425
},
15-
} as unknown as AIService;
26+
};
27+
return service as AIService;
1628
}
1729

1830
describe("workspaceTitleGenerator", () => {

0 commit comments

Comments
 (0)