fix: break 3-file import cycle in cli/project-files

project-files.ts -> auth.ts -> logger.ts -> project-files.ts

Import getConfigDir directly from config-dir instead of auth to
break the circular dependency. config-dir.ts exports the identical
function (auth.ts just wraps it). anonymous-id.ts already imports
from config-dir directly.

Add getProjectDataDir tests to exercise the fixed import path.

Author: Dataflow Dev

cli/src/__tests__/project-files-chat-id.test.ts

@@ -8,7 +8,10 @@ import {
   getCurrentChatId,
   setCurrentChatId,
   startNewChat,
+  setProjectRoot,
+  getProjectDataDir,
 } from '../project-files'
+import { getConfigDir } from '../utils/config-dir'
 
 describe('chat id lifecycle', () => {
   test('getCurrentChatId is stable across calls', () => {
@@ -32,3 +35,20 @@ describe('chat id lifecycle', () => {
     expect(rotated).toMatch(/^\d{4}-\d{2}-\d{2}T\d{2}-\d{2}-\d{2}\.\d{3}Z$/)
   })
 })
+
+describe('getProjectDataDir', () => {
+  test('returns a path containing the project basename', () => {
+    setProjectRoot('/tmp/my-project')
+    const dataDir = getProjectDataDir()
+    expect(dataDir).toContain('my-project')
+    expect(dataDir).toContain('projects')
+  })
+
+  test('uses config-dir getConfigDir (not auth re-export)', () => {
+    setProjectRoot('/tmp/test-repo')
+    const dataDir = getProjectDataDir()
+    // Should resolve via config-dir's getConfigDir without pulling in auth.ts
+    const configDir = getConfigDir()
+    expect(dataDir).toContain(configDir)
+  })
+})

cli/src/project-files.ts

@@ -1,7 +1,7 @@
 import { mkdirSync, readdirSync, statSync } from 'fs'
 import path from 'path'
 
-import { getConfigDir } from './utils/auth'
+import { getConfigDir } from './utils/config-dir'
 
 let projectRoot: string | undefined
 let currentChatId: string | undefined