feat: implement project code management for Windows compatibility and enhance debug logger tests

Author: qorzj

src/session.ts

@@ -57,6 +57,8 @@ export type {
 } from "./common/permissions";
 
 const MAX_SESSION_ENTRIES = 50;
+const MAX_PROJECT_CODE_LENGTH = 64;
+const PROJECT_CODE_HASH_LENGTH = 16;
 const DEFAULT_NEW_PROMPT_API_URL = "https://deepcode.vegamo.cn/api/plugin/new";
 const NEW_PROMPT_REPORT_TIMEOUT_MS = 3000;
 const DEFAULT_COMPACT_PROMPT_TOKEN_THRESHOLD = 128 * 1024;
@@ -75,6 +77,36 @@ export function getCompactPromptTokenThreshold(model: string): number {
     : DEFAULT_COMPACT_PROMPT_TOKEN_THRESHOLD;
 }
 
+// Keep project storage paths short enough for Git's internal files on Windows.
+export function getProjectCode(projectRoot: string): string {
+  const legacyCode = getLegacyProjectCode(projectRoot);
+  if (legacyCode.length <= MAX_PROJECT_CODE_LENGTH) {
+    return legacyCode;
+  }
+
+  const normalizedRoot = path.resolve(projectRoot);
+  const hashInput = process.platform === "win32" ? normalizedRoot.toLowerCase() : normalizedRoot;
+  const hash = crypto.createHash("sha256").update(hashInput).digest("hex").slice(0, PROJECT_CODE_HASH_LENGTH);
+  const prefixLimit = MAX_PROJECT_CODE_LENGTH - PROJECT_CODE_HASH_LENGTH - 1;
+  const basename = path.basename(normalizedRoot);
+  const prefix =
+    sanitizeProjectCodePart(basename)
+      .slice(0, prefixLimit)
+      .replace(/[-.]+$/g, "") || "project";
+  return `${prefix}-${hash}`;
+}
+
+function getLegacyProjectCode(projectRoot: string): string {
+  return projectRoot.replace(/[\\/]/g, "-").replace(/:/g, "");
+}
+
+function sanitizeProjectCodePart(value: string): string {
+  return value
+    .replace(/[^A-Za-z0-9._-]/g, "-")
+    .replace(/-+/g, "-")
+    .replace(/^[-.]+|[-.]+$/g, "");
+}
+
 function isUsageRecord(value: unknown): value is Record<string, unknown> {
   return value !== null && typeof value === "object" && !Array.isArray(value);
 }
@@ -1726,16 +1758,12 @@ ${skillMd}
     };
   }
 
-  private getProjectCode(projectRoot: string): string {
-    return projectRoot.replace(/[\\/]/g, "-").replace(/:/g, "");
-  }
-
   private getProjectStorage(): {
     projectCode: string;
     projectDir: string;
     sessionsIndexPath: string;
   } {
-    const projectCode = this.getProjectCode(this.projectRoot);
+    const projectCode = getProjectCode(this.projectRoot);
     const projectDir = path.join(os.homedir(), ".deepcode", "projects", projectCode);
     const sessionsIndexPath = path.join(projectDir, "sessions-index.json");
     return { projectCode, projectDir, sessionsIndexPath };

src/tests/debug-logger.test.ts

@@ -7,8 +7,12 @@ import { getDebugLogPath, logOpenAIChatCompletionDebug } from "../common/debug-l
 
 test("debug logger appends full entries without rotation", () => {
   const originalHome = process.env.HOME;
+  const originalUserProfile = process.env.USERPROFILE;
   const home = fs.mkdtempSync(path.join(os.tmpdir(), "deepcode-debug-log-home-"));
   process.env.HOME = home;
+  if (process.platform === "win32") {
+    process.env.USERPROFILE = home;
+  }
   try {
     for (let index = 0; index < 25; index += 1) {
       logOpenAIChatCompletionDebug({
@@ -42,5 +46,10 @@ test("debug logger appends full entries without rotation", () => {
     } else {
       process.env.HOME = originalHome;
     }
+    if (originalUserProfile === undefined) {
+      delete process.env.USERPROFILE;
+    } else {
+      process.env.USERPROFILE = originalUserProfile;
+    }
   }
 });

src/tests/session.test.ts

@@ -6,8 +6,7 @@ import * as os from "os";
 import * as path from "path";
 import { GitFileHistory } from "../common/file-history";
 import { clearSessionState } from "../common/state";
-import { type SessionMessage } from "../session";
-import { SessionManager } from "../session";
+import { getProjectCode, SessionManager, type SessionMessage } from "../session";
 
 const originalFetch = globalThis.fetch;
 const originalConsoleWarn = console.warn;
@@ -45,6 +44,21 @@ afterEach(() => {
   }
 });
 
+test("getProjectCode shortens long project roots for Windows-compatible storage paths", () => {
+  const shortRoot = "short-project";
+  assert.equal(getProjectCode(shortRoot), shortRoot.replace(/[\\/]/g, "-").replace(/:/g, ""));
+
+  const longRoot = path.join(
+    os.tmpdir(),
+    "deepcode-project-code-workspace-with-a-long-name-that-would-create-long-git-internal-paths"
+  );
+  const projectCode = getProjectCode(longRoot);
+
+  assert.ok(projectCode.length <= 64);
+  assert.match(projectCode, /^[A-Za-z0-9._-]+$/);
+  assert.notEqual(projectCode, longRoot.replace(/[\\/]/g, "-").replace(/:/g, ""));
+});
+
 test("SessionManager preserves structured system content when building OpenAI messages", () => {
   const manager = new SessionManager({
     projectRoot: process.cwd(),
@@ -271,7 +285,7 @@ test("SessionManager normalizes legacy sessions without activeTokens to zero", (
   const home = createTempDir("deepcode-legacy-active-tokens-home-");
   setHomeDir(home);
 
-  const projectCode = workspace.replace(/[\\/]/g, "-").replace(/:/g, "");
+  const projectCode = getProjectCode(workspace);
   const projectDir = path.join(home, ".deepcode", "projects", projectCode);
   fs.mkdirSync(projectDir, { recursive: true });
   fs.writeFileSync(
@@ -324,7 +338,7 @@ test("SessionManager marks skills loaded from existing session messages", async
     "utf8"
   );
 
-  const projectCode = workspace.replace(/[\\/]/g, "-").replace(/:/g, "");
+  const projectCode = getProjectCode(workspace);
   const projectDir = path.join(home, ".deepcode", "projects", projectCode);
   fs.mkdirSync(projectDir, { recursive: true });
   fs.writeFileSync(
@@ -982,14 +996,7 @@ test("createSession initializes file-history repo and session branch", async (t)
 
   const sessionId = await manager.createSession({ text: "first prompt" });
   const userMessage = manager.listSessionMessages(sessionId).find((message) => message.role === "user");
-  const gitDir = path.join(
-    home,
-    ".deepcode",
-    "projects",
-    workspace.replace(/[\\/]/g, "-").replace(/:/g, ""),
-    "file-history",
-    ".git"
-  );
+  const gitDir = path.join(home, ".deepcode", "projects", getProjectCode(workspace), "file-history", ".git");
 
   assert.ok(fs.existsSync(gitDir));
   assert.ok(userMessage?.checkpointHash);
@@ -2850,13 +2857,7 @@ test("SessionManager.deleteSession removes the messages file", () => {
   (manager as any).activateSession = async () => {};
 
   const sessionId = createSessionAndMessages(manager, "session-delete-msg", "Test session");
-  const messagePath = path.join(
-    home,
-    ".deepcode",
-    "projects",
-    workspace.replace(/[\\\\/]/g, "-").replace(/:/g, ""),
-    `${sessionId}.jsonl`
-  );
+  const messagePath = path.join(home, ".deepcode", "projects", getProjectCode(workspace), `${sessionId}.jsonl`);
 
   // Verify messages file exists
   assert.ok(fs.existsSync(messagePath));
@@ -2962,7 +2963,7 @@ function createFileHistoryCommit(
   sessionId: string,
   files: Record<string, string>
 ): string {
-  const projectCode = workspace.replace(/[\\/]/g, "-").replace(/:/g, "");
+  const projectCode = getProjectCode(workspace);
   const gitDir = path.join(home, ".deepcode", "projects", projectCode, "file-history", ".git");
   const fileHistory = new GitFileHistory(workspace, gitDir);
   fileHistory.ensureSession(sessionId);
@@ -2980,7 +2981,7 @@ function createFileHistoryCommit(
 }
 
 function getFileHistoryGitDir(home: string, workspace: string): string {
-  const projectCode = workspace.replace(/[\\/]/g, "-").replace(/:/g, "");
+  const projectCode = getProjectCode(workspace);
   return path.join(home, ".deepcode", "projects", projectCode, "file-history", ".git");
 }