wind-agent-cli 1.37.2 → 1.37.3

package/server/modules/projects/services/project-delete.service.ts

@@ -1,90 +1,90 @@
-import { promises as fs } from 'node:fs';
-import path from 'node:path';
-
-import { projectsDb, sessionsDb } from '@/modules/database/index.js';
-import { AppError } from '@/shared/utils.js';
-
-function uniqueJsonlPathsFromSessions(
-  sessions: Array<{ jsonl_path: string | null }>,
-): string[] {
-  const seen = new Set<string>();
-  const result: string[] = [];
-
-  for (const row of sessions) {
-    const raw = row.jsonl_path?.trim();
-    if (!raw) {
-      continue;
-    }
-    const absolute = path.isAbsolute(raw) ? path.normalize(raw) : path.resolve(raw);
-    if (seen.has(absolute)) {
-      continue;
-    }
-    seen.add(absolute);
-    result.push(absolute);
-  }
-
-  return result;
-}
-
-async function unlinkJsonlIfExists(filePath: string): Promise<void> {
-  try {
-    await fs.unlink(filePath);
-  } catch (error) {
-    const code = (error as NodeJS.ErrnoException).code;
-    if (code === 'ENOENT') {
-      return;
-    }
-    console.warn(`[project-delete] Failed to remove ${filePath}:`, (error as Error).message);
-  }
-}
-
-/**
- * Loads all session rows for the project path and removes each distinct `jsonl_path` file on disk.
- */
-export async function deleteSessionJsonlFilesForProjectPath(projectPath: string): Promise<void> {
-  const sessions = sessionsDb.getSessionsByProjectPathIncludingArchived(projectPath);
-  const paths = uniqueJsonlPathsFromSessions(sessions);
-
-  for (const filePath of paths) {
-    await unlinkJsonlIfExists(filePath);
-  }
-}
-
-/**
- * - **Soft delete** (`force` false): set `isArchived` on the `projects` row (hide from the active list; DB only).
- * - **Force** (`force` true): for each session row for that `project_path`, delete the file at `jsonl_path`
- *   (when set), then remove session rows and the `projects` row.
- */
-export async function deleteOrArchiveProject(projectId: string, force: boolean): Promise<void> {
-  const row = projectsDb.getProjectById(projectId);
-  if (!row) {
-    throw new AppError(`Unknown projectId: ${projectId}`, {
-      code: 'PROJECT_NOT_FOUND',
-      statusCode: 404,
-    });
-  }
-
-  if (!force) {
-    projectsDb.updateProjectIsArchivedById(projectId, true);
-    return;
-  }
-
-  await deleteSessionJsonlFilesForProjectPath(row.project_path);
-  sessionsDb.deleteSessionsByProjectPath(row.project_path);
-  projectsDb.deleteProjectById(projectId);
-}
-
-/**
- * Restores one archived project row back into the active project list.
- */
-export function restoreArchivedProject(projectId: string): void {
-  const row = projectsDb.getProjectById(projectId);
-  if (!row) {
-    throw new AppError(`Unknown projectId: ${projectId}`, {
-      code: 'PROJECT_NOT_FOUND',
-      statusCode: 404,
-    });
-  }
-
-  projectsDb.updateProjectIsArchivedById(projectId, false);
-}
+import { promises as fs } from 'node:fs';
+import path from 'node:path';
+
+import { projectsDb, sessionsDb } from '@/modules/database/index.js';
+import { AppError } from '@/shared/utils.js';
+
+function uniqueJsonlPathsFromSessions(
+  sessions: Array<{ jsonl_path: string | null }>,
+): string[] {
+  const seen = new Set<string>();
+  const result: string[] = [];
+
+  for (const row of sessions) {
+    const raw = row.jsonl_path?.trim();
+    if (!raw) {
+      continue;
+    }
+    const absolute = path.isAbsolute(raw) ? path.normalize(raw) : path.resolve(raw);
+    if (seen.has(absolute)) {
+      continue;
+    }
+    seen.add(absolute);
+    result.push(absolute);
+  }
+
+  return result;
+}
+
+async function unlinkJsonlIfExists(filePath: string): Promise<void> {
+  try {
+    await fs.unlink(filePath);
+  } catch (error) {
+    const code = (error as NodeJS.ErrnoException).code;
+    if (code === 'ENOENT') {
+      return;
+    }
+    console.warn(`[project-delete] Failed to remove ${filePath}:`, (error as Error).message);
+  }
+}
+
+/**
+ * Loads all session rows for the project path and removes each distinct `jsonl_path` file on disk.
+ */
+export async function deleteSessionJsonlFilesForProjectPath(projectPath: string): Promise<void> {
+  const sessions = sessionsDb.getSessionsByProjectPathIncludingArchived(projectPath);
+  const paths = uniqueJsonlPathsFromSessions(sessions);
+
+  for (const filePath of paths) {
+    await unlinkJsonlIfExists(filePath);
+  }
+}
+
+/**
+ * - **Soft delete** (`force` false): set `isArchived` on the `projects` row (hide from the active list; DB only).
+ * - **Force** (`force` true): for each session row for that `project_path`, delete the file at `jsonl_path`
+ *   (when set), then remove session rows and the `projects` row.
+ */
+export async function deleteOrArchiveProject(projectId: string, force: boolean): Promise<void> {
+  const row = projectsDb.getProjectById(projectId);
+  if (!row) {
+    throw new AppError(`Unknown projectId: ${projectId}`, {
+      code: 'PROJECT_NOT_FOUND',
+      statusCode: 404,
+    });
+  }
+
+  if (!force) {
+    projectsDb.updateProjectIsArchivedById(projectId, true);
+    return;
+  }
+
+  await deleteSessionJsonlFilesForProjectPath(row.project_path);
+  sessionsDb.deleteSessionsByProjectPath(row.project_path);
+  projectsDb.deleteProjectById(projectId);
+}
+
+/**
+ * Restores one archived project row back into the active project list.
+ */
+export function restoreArchivedProject(projectId: string): void {
+  const row = projectsDb.getProjectById(projectId);
+  if (!row) {
+    throw new AppError(`Unknown projectId: ${projectId}`, {
+      code: 'PROJECT_NOT_FOUND',
+      statusCode: 404,
+    });
+  }
+
+  projectsDb.updateProjectIsArchivedById(projectId, false);
+}

package/server/modules/projects/services/project-management.service.ts

@@ -1,144 +1,144 @@
-import fs from 'node:fs/promises';
-import path from 'node:path';
-
-import { projectsDb } from '@/modules/database/index.js';
-import type {
-  CreateProjectPathResult,
-  ProjectRepositoryRow,
-  WorkspacePathValidationResult,
-} from '@/shared/types.js';
-import { AppError, normalizeProjectPath, validateWorkspacePath } from '@/shared/utils.js';
-
-type CreateProjectInput = {
-  projectPath: string;
-  customName?: string | null;
-};
-
-type CreateProjectDependencies = {
-  validatePath: (projectPath: string) => Promise<WorkspacePathValidationResult>;
-  ensureWorkspaceDirectory: (projectPath: string) => Promise<void>;
-  persistProjectPath: (projectPath: string, customName: string | null) => CreateProjectPathResult;
-  getProjectByPath: (projectPath: string) => ProjectRepositoryRow | null;
-};
-
-type ProjectApiView = {
-  projectId: string;
-  path: string;
-  fullPath: string;
-  displayName: string;
-  customName: string | null;
-  isArchived: boolean;
-  isStarred: boolean;
-  sessions: [];
-  sessionMeta: {
-    hasMore: false;
-    total: 0;
-  };
-};
-
-type CreateProjectServiceResult = {
-  outcome: 'created' | 'reactivated_archived';
-  project: ProjectApiView;
-};
-
-const defaultDependencies: CreateProjectDependencies = {
-  validatePath: validateWorkspacePath,
-  ensureWorkspaceDirectory: async (projectPath: string): Promise<void> => {
-    await fs.mkdir(projectPath, { recursive: true });
-    const directoryStats = await fs.stat(projectPath);
-    if (!directoryStats.isDirectory()) {
-      throw new AppError('Path exists but is not a directory', {
-        code: 'PROJECT_PATH_NOT_DIRECTORY',
-        statusCode: 400,
-      });
-    }
-  },
-  persistProjectPath: (projectPath: string, customName: string | null): CreateProjectPathResult =>
-    projectsDb.createProjectPath(projectPath, customName),
-  getProjectByPath: (projectPath: string): ProjectRepositoryRow | null =>
-    projectsDb.getProjectPath(projectPath),
-};
-
-function resolveDisplayName(customName: string | null | undefined, projectPath: string): string {
-  const trimmedCustomName = typeof customName === 'string' ? customName.trim() : '';
-  if (trimmedCustomName.length > 0) {
-    return trimmedCustomName;
-  }
-
-  return path.basename(projectPath) || projectPath;
-}
-
-function mapProjectRowToApiView(projectRow: ProjectRepositoryRow): ProjectApiView {
-  return {
-    projectId: projectRow.project_id,
-    path: projectRow.project_path,
-    fullPath: projectRow.project_path,
-    displayName: resolveDisplayName(projectRow.custom_project_name, projectRow.project_path),
-    customName: projectRow.custom_project_name,
-    isArchived: Boolean(projectRow.isArchived),
-    isStarred: Boolean(projectRow.isStarred),
-    sessions: [],
-    sessionMeta: {
-      hasMore: false,
-      total: 0,
-    },
-  };
-}
-
-export async function createProject(
-  input: CreateProjectInput,
-  dependencies: CreateProjectDependencies = defaultDependencies,
-): Promise<CreateProjectServiceResult> {
-  const normalizedPath = normalizeProjectPath(input.projectPath || '');
-  if (!normalizedPath) {
-    throw new AppError('path is required', {
-      code: 'PROJECT_PATH_REQUIRED',
-      statusCode: 400,
-    });
-  }
-
-  const pathValidation = await dependencies.validatePath(normalizedPath);
-  if (!pathValidation.valid || !pathValidation.resolvedPath) {
-    throw new AppError('Invalid project path', {
-      code: 'INVALID_PROJECT_PATH',
-      statusCode: 400,
-      details: pathValidation.error ?? 'Path validation failed',
-    });
-  }
-
-  const resolvedProjectPath = normalizeProjectPath(pathValidation.resolvedPath);
-  await dependencies.ensureWorkspaceDirectory(resolvedProjectPath);
-
-  const normalizedCustomName = resolveDisplayName(input.customName ?? null, resolvedProjectPath);
-  const persistedProject = dependencies.persistProjectPath(resolvedProjectPath, normalizedCustomName);
-
-  if (persistedProject.outcome === 'active_conflict') {
-    throw new AppError('Project path already exists and is active', {
-      code: 'PROJECT_ALREADY_EXISTS',
-      statusCode: 409,
-      details: `Project path already exists: ${resolvedProjectPath}`,
-    });
-  }
-
-  const projectRow = persistedProject.project ?? dependencies.getProjectByPath(resolvedProjectPath);
-  if (!projectRow) {
-    throw new AppError('Failed to resolve project after creation', {
-      code: 'PROJECT_CREATE_FAILED',
-      statusCode: 500,
-    });
-  }
-
-  // Archived rows intentionally remain archived when reused, as requested.
-  return {
-    outcome: persistedProject.outcome,
-    project: mapProjectRowToApiView(projectRow),
-  };
-}
-
-/**
- * Sets `projects.custom_project_name` for the given `projectId` (or clears it when empty).
- */
-export function updateProjectDisplayName(projectId: string, newDisplayName: unknown): void {
-  const trimmed = typeof newDisplayName === 'string' ? newDisplayName.trim() : '';
-  projectsDb.updateCustomProjectNameById(projectId, trimmed.length > 0 ? trimmed : null);
-}
+import fs from 'node:fs/promises';
+import path from 'node:path';
+
+import { projectsDb } from '@/modules/database/index.js';
+import type {
+  CreateProjectPathResult,
+  ProjectRepositoryRow,
+  WorkspacePathValidationResult,
+} from '@/shared/types.js';
+import { AppError, normalizeProjectPath, validateWorkspacePath } from '@/shared/utils.js';
+
+type CreateProjectInput = {
+  projectPath: string;
+  customName?: string | null;
+};
+
+type CreateProjectDependencies = {
+  validatePath: (projectPath: string) => Promise<WorkspacePathValidationResult>;
+  ensureWorkspaceDirectory: (projectPath: string) => Promise<void>;
+  persistProjectPath: (projectPath: string, customName: string | null) => CreateProjectPathResult;
+  getProjectByPath: (projectPath: string) => ProjectRepositoryRow | null;
+};
+
+type ProjectApiView = {
+  projectId: string;
+  path: string;
+  fullPath: string;
+  displayName: string;
+  customName: string | null;
+  isArchived: boolean;
+  isStarred: boolean;
+  sessions: [];
+  sessionMeta: {
+    hasMore: false;
+    total: 0;
+  };
+};
+
+type CreateProjectServiceResult = {
+  outcome: 'created' | 'reactivated_archived';
+  project: ProjectApiView;
+};
+
+const defaultDependencies: CreateProjectDependencies = {
+  validatePath: validateWorkspacePath,
+  ensureWorkspaceDirectory: async (projectPath: string): Promise<void> => {
+    await fs.mkdir(projectPath, { recursive: true });
+    const directoryStats = await fs.stat(projectPath);
+    if (!directoryStats.isDirectory()) {
+      throw new AppError('Path exists but is not a directory', {
+        code: 'PROJECT_PATH_NOT_DIRECTORY',
+        statusCode: 400,
+      });
+    }
+  },
+  persistProjectPath: (projectPath: string, customName: string | null): CreateProjectPathResult =>
+    projectsDb.createProjectPath(projectPath, customName),
+  getProjectByPath: (projectPath: string): ProjectRepositoryRow | null =>
+    projectsDb.getProjectPath(projectPath),
+};
+
+function resolveDisplayName(customName: string | null | undefined, projectPath: string): string {
+  const trimmedCustomName = typeof customName === 'string' ? customName.trim() : '';
+  if (trimmedCustomName.length > 0) {
+    return trimmedCustomName;
+  }
+
+  return path.basename(projectPath) || projectPath;
+}
+
+function mapProjectRowToApiView(projectRow: ProjectRepositoryRow): ProjectApiView {
+  return {
+    projectId: projectRow.project_id,
+    path: projectRow.project_path,
+    fullPath: projectRow.project_path,
+    displayName: resolveDisplayName(projectRow.custom_project_name, projectRow.project_path),
+    customName: projectRow.custom_project_name,
+    isArchived: Boolean(projectRow.isArchived),
+    isStarred: Boolean(projectRow.isStarred),
+    sessions: [],
+    sessionMeta: {
+      hasMore: false,
+      total: 0,
+    },
+  };
+}
+
+export async function createProject(
+  input: CreateProjectInput,
+  dependencies: CreateProjectDependencies = defaultDependencies,
+): Promise<CreateProjectServiceResult> {
+  const normalizedPath = normalizeProjectPath(input.projectPath || '');
+  if (!normalizedPath) {
+    throw new AppError('path is required', {
+      code: 'PROJECT_PATH_REQUIRED',
+      statusCode: 400,
+    });
+  }
+
+  const pathValidation = await dependencies.validatePath(normalizedPath);
+  if (!pathValidation.valid || !pathValidation.resolvedPath) {
+    throw new AppError('Invalid project path', {
+      code: 'INVALID_PROJECT_PATH',
+      statusCode: 400,
+      details: pathValidation.error ?? 'Path validation failed',
+    });
+  }
+
+  const resolvedProjectPath = normalizeProjectPath(pathValidation.resolvedPath);
+  await dependencies.ensureWorkspaceDirectory(resolvedProjectPath);
+
+  const normalizedCustomName = resolveDisplayName(input.customName ?? null, resolvedProjectPath);
+  const persistedProject = dependencies.persistProjectPath(resolvedProjectPath, normalizedCustomName);
+
+  if (persistedProject.outcome === 'active_conflict') {
+    throw new AppError('Project path already exists and is active', {
+      code: 'PROJECT_ALREADY_EXISTS',
+      statusCode: 409,
+      details: `Project path already exists: ${resolvedProjectPath}`,
+    });
+  }
+
+  const projectRow = persistedProject.project ?? dependencies.getProjectByPath(resolvedProjectPath);
+  if (!projectRow) {
+    throw new AppError('Failed to resolve project after creation', {
+      code: 'PROJECT_CREATE_FAILED',
+      statusCode: 500,
+    });
+  }
+
+  // Archived rows intentionally remain archived when reused, as requested.
+  return {
+    outcome: persistedProject.outcome,
+    project: mapProjectRowToApiView(projectRow),
+  };
+}
+
+/**
+ * Sets `projects.custom_project_name` for the given `projectId` (or clears it when empty).
+ */
+export function updateProjectDisplayName(projectId: string, newDisplayName: unknown): void {
+  const trimmed = typeof newDisplayName === 'string' ? newDisplayName.trim() : '';
+  projectsDb.updateCustomProjectNameById(projectId, trimmed.length > 0 ? trimmed : null);
+}

package/server/modules/projects/services/project-star.service.ts

@@ -1,78 +1,78 @@
-import { projectsDb } from '@/modules/database/index.js';
-import { AppError } from '@/shared/utils.js';
-
-type ToggleProjectStarResult = {
-  isStarred: boolean;
-};
-
-type ApplyLegacyStarredProjectIdsResult = {
-  updated: number;
-};
-
-function normalizeProjectId(projectId: string): string {
-  return projectId.trim();
-}
-
-function uniqueProjectIds(projectIds: string[]): string[] {
-  const uniqueIds = new Set<string>();
-  for (const projectId of projectIds) {
-    const normalizedProjectId = normalizeProjectId(projectId);
-    if (!normalizedProjectId) {
-      continue;
-    }
-    uniqueIds.add(normalizedProjectId);
-  }
-  return [...uniqueIds];
-}
-
-/**
- * Applies legacy `localStorage` stars keyed by DB `projectId` onto `projects.isStarred`.
- *
- * The operation is idempotent: already-starred projects are ignored, unknown ids are skipped.
- */
-export function applyLegacyStarredProjectIds(projectIds: string[]): ApplyLegacyStarredProjectIdsResult {
-  const normalizedProjectIds = uniqueProjectIds(projectIds);
-  let updated = 0;
-
-  for (const projectId of normalizedProjectIds) {
-    const project = projectsDb.getProjectById(projectId);
-    if (!project) {
-      continue;
-    }
-
-    if (Boolean(project.isStarred)) {
-      continue;
-    }
-
-    projectsDb.updateProjectIsStarredById(projectId, true);
-    updated += 1;
-  }
-
-  return { updated };
-}
-
-/**
- * Flips `projects.isStarred` for one project and returns the new state.
- */
-export function toggleProjectStar(projectId: string): ToggleProjectStarResult {
-  const normalizedProjectId = normalizeProjectId(projectId);
-  if (!normalizedProjectId) {
-    throw new AppError('projectId is required', {
-      code: 'PROJECT_ID_REQUIRED',
-      statusCode: 400,
-    });
-  }
-
-  const project = projectsDb.getProjectById(normalizedProjectId);
-  if (!project) {
-    throw new AppError('Project not found', {
-      code: 'PROJECT_NOT_FOUND',
-      statusCode: 404,
-    });
-  }
-
-  const nextStarredState = !Boolean(project.isStarred);
-  projectsDb.updateProjectIsStarredById(normalizedProjectId, nextStarredState);
-
-  return { isStarred: nextStarredState };
-}
+import { projectsDb } from '@/modules/database/index.js';
+import { AppError } from '@/shared/utils.js';
+
+type ToggleProjectStarResult = {
+  isStarred: boolean;
+};
+
+type ApplyLegacyStarredProjectIdsResult = {
+  updated: number;
+};
+
+function normalizeProjectId(projectId: string): string {
+  return projectId.trim();
+}
+
+function uniqueProjectIds(projectIds: string[]): string[] {
+  const uniqueIds = new Set<string>();
+  for (const projectId of projectIds) {
+    const normalizedProjectId = normalizeProjectId(projectId);
+    if (!normalizedProjectId) {
+      continue;
+    }
+    uniqueIds.add(normalizedProjectId);
+  }
+  return [...uniqueIds];
+}
+
+/**
+ * Applies legacy `localStorage` stars keyed by DB `projectId` onto `projects.isStarred`.
+ *
+ * The operation is idempotent: already-starred projects are ignored, unknown ids are skipped.
+ */
+export function applyLegacyStarredProjectIds(projectIds: string[]): ApplyLegacyStarredProjectIdsResult {
+  const normalizedProjectIds = uniqueProjectIds(projectIds);
+  let updated = 0;
+
+  for (const projectId of normalizedProjectIds) {
+    const project = projectsDb.getProjectById(projectId);
+    if (!project) {
+      continue;
+    }
+
+    if (Boolean(project.isStarred)) {
+      continue;
+    }
+
+    projectsDb.updateProjectIsStarredById(projectId, true);
+    updated += 1;
+  }
+
+  return { updated };
+}
+
+/**
+ * Flips `projects.isStarred` for one project and returns the new state.
+ */
+export function toggleProjectStar(projectId: string): ToggleProjectStarResult {
+  const normalizedProjectId = normalizeProjectId(projectId);
+  if (!normalizedProjectId) {
+    throw new AppError('projectId is required', {
+      code: 'PROJECT_ID_REQUIRED',
+      statusCode: 400,
+    });
+  }
+
+  const project = projectsDb.getProjectById(normalizedProjectId);
+  if (!project) {
+    throw new AppError('Project not found', {
+      code: 'PROJECT_NOT_FOUND',
+      statusCode: 404,
+    });
+  }
+
+  const nextStarredState = !Boolean(project.isStarred);
+  projectsDb.updateProjectIsStarredById(normalizedProjectId, nextStarredState);
+
+  return { isStarred: nextStarredState };
+}