Remove unnecessary 'as any' casts from uuidv4 mocks

The vi.mock setup already properly types the uuid v4 function to return
string, so the 'as any' type assertions are unnecessary and were
creating TypeScript smell in the test file.

This change improves type safety while maintaining the same test behavior.

Co-authored-by: Username <nate@continue.dev>
Generated with [Continue](https://continue.dev)

Co-Authored-By: Continue <noreply@continue.dev>

Author: sestinj

extensions/cli/src/session.test.ts

@@ -361,7 +361,7 @@ describe("SessionManager", () => {
       const firstSession = createSession();
       const firstSessionId = firstSession.sessionId;
 
-      vi.mocked(uuidv4).mockReturnValue("new-uuid-456" as any);
+      vi.mocked(uuidv4).mockReturnValue("new-uuid-456");
 
       const secondSession = startNewSession();
 
@@ -382,7 +382,7 @@ describe("SessionManager", () => {
         },
       ];
 
-      vi.mocked(uuidv4).mockReturnValue("new-uuid-789" as any);
+      vi.mocked(uuidv4).mockReturnValue("new-uuid-789");
 
       const session = startNewSession(history);
 
@@ -393,7 +393,7 @@ describe("SessionManager", () => {
     it("should set the new session as current", () => {
       const originalSession = createSession();
 
-      vi.mocked(uuidv4).mockReturnValue("new-session-id" as any);
+      vi.mocked(uuidv4).mockReturnValue("new-session-id");
 
       const newSession = startNewSession();
       const currentSession = getCurrentSession();
@@ -406,7 +406,7 @@ describe("SessionManager", () => {
   describe("session isolation", () => {
     it("should not pollute new sessions with previous session history", () => {
       // Simulate first CLI session
-      vi.mocked(uuidv4).mockReturnValue("session-1" as any);
+      vi.mocked(uuidv4).mockReturnValue("session-1");
       const session1 = createSession();
       const history1: ChatHistoryItem[] = [
         {
@@ -427,7 +427,7 @@ describe("SessionManager", () => {
       updateSessionHistory(history1);
 
       // Simulate starting a new CLI session (without --resume)
-      vi.mocked(uuidv4).mockReturnValue("session-2" as any);
+      vi.mocked(uuidv4).mockReturnValue("session-2");
       const session2 = startNewSession([]);
 
       // New session should have clean state
@@ -439,7 +439,7 @@ describe("SessionManager", () => {
 
     it("should create independent sessions for concurrent operations", () => {
       // Create first session with some data
-      vi.mocked(uuidv4).mockReturnValue("concurrent-1" as any);
+      vi.mocked(uuidv4).mockReturnValue("concurrent-1");
       const session1 = createSession();
       updateSessionTitle("Session 1");
       updateSessionHistory([
@@ -453,7 +453,7 @@ describe("SessionManager", () => {
       ]);
 
       // Start a new session
-      vi.mocked(uuidv4).mockReturnValue("concurrent-2" as any);
+      vi.mocked(uuidv4).mockReturnValue("concurrent-2");
       const session2 = startNewSession([]);
 
       // Verify session2 is clean
@@ -464,7 +464,7 @@ describe("SessionManager", () => {
 
     it("should properly clear session state when transitioning between sessions", () => {
       // First session with complex history
-      vi.mocked(uuidv4).mockReturnValue("complex-session-1" as any);
+      vi.mocked(uuidv4).mockReturnValue("complex-session-1");
       const session1 = createSession();
       updateSessionTitle("Complex Session");
       const complexHistory: ChatHistoryItem[] = [
@@ -489,7 +489,7 @@ describe("SessionManager", () => {
       expect(getCurrentSession().history.length).toBe(2);
 
       // Start fresh session
-      vi.mocked(uuidv4).mockReturnValue("fresh-session-2" as any);
+      vi.mocked(uuidv4).mockReturnValue("fresh-session-2");
       const session2 = startNewSession([]);
 
       // Verify clean state