Fix project renumber (#1597)

* fix: generateUniqueFileName

* loadProject should not generate new names

* update comments

* extract and test getProjectNumber

* spell

* finish jsdoc

* use getProjectNumber

* cleanup loadProject

* create a `incrementProjectNumber` function

* spelling

Co-authored-by: Carlos Valente <34649812+cpvalente@users.noreply.github.com>

---------

Co-authored-by: Carlos Valente <34649812+cpvalente@users.noreply.github.com>

apps/server/src/utils/__tests__/fileManagement.test.ts

@@ -1,7 +1,12 @@
 import { describe, it, expect, Mock } from 'vitest';
 import * as fs from 'fs';
 
-import { appendToName, ensureJsonExtension, generateUniqueFileName } from '../fileManagement.js';
+import {
+  appendToName,
+  ensureJsonExtension,
+  generateUniqueFileName,
+  incrementProjectNumber,
+} from '../fileManagement.js';
 
 // Mock fs.existsSync to control the test environment
 vi.mock('fs', () => ({
@@ -90,3 +95,25 @@ describe('generateUniqueFileName', () => {
     expect(uniqueFilename).toBe(expectedFilename);
   });
 });
+
+describe('file index', () => {
+  it('sets index to 1 when there is no index', () => {
+    expect(incrementProjectNumber('test file.json')).toBe('test file (1).json');
+  });
+
+  it('increments to 2 when index is 1', () => {
+    expect(incrementProjectNumber('test file (1).json')).toBe('test file (2).json');
+  });
+
+  it('does not count number not wrapped in parenthesis', () => {
+    expect(incrementProjectNumber('test file 1.json')).toBe('test file 1 (1).json');
+  });
+
+  it('does not count number if there is not a space', () => {
+    expect(incrementProjectNumber('test file(1).json')).toBe('test file(1) (1).json');
+  });
+
+  it('counts multi digit numbers', () => {
+    expect(incrementProjectNumber('test file (890).json')).toBe('test file (891).json');
+  });
+});

apps/server/src/utils/fileManagement.ts

@@ -1,6 +1,6 @@
 import { existsSync, mkdirSync, PathLike } from 'fs';
 import { readdir, copyFile, unlink } from 'fs/promises';
-import { basename, extname, join, parse } from 'path';
+import { basename, join, parse } from 'path';
 
 /**
  * @description Creates a directory if it doesn't exist
@@ -53,16 +53,11 @@ export function appendToName(filePath: string, append: string): string {
  * If a file with the same name already exists, appends a counter to the filename.
  */
 export function generateUniqueFileName(directory: string, filename: string): string {
-  const extension = extname(filename);
-  const baseName = basename(filename, extension);
-
-  let counter = 0;
   let uniqueFilename = filename;
 
   while (fileExists(uniqueFilename)) {
-    counter++;
     // Append counter to filename if the file exists.
-    uniqueFilename = `${baseName} (${counter})${extension}`;
+    uniqueFilename = incrementProjectNumber(uniqueFilename);
   }
 
   return uniqueFilename;
@@ -114,3 +109,23 @@ export async function dockerSafeRename(oldPath: PathLike, newPath: PathLike) {
   await copyFile(oldPath, newPath);
   await unlink(oldPath);
 }
+
+/**
+ * finds potential file index number in our (*) format and increments
+ * the number section (*) must be separated from the name by a space
+ * @example incrementProjectNumber('test(1).json') -> 'test(1).json'
+  * @example incrementProjectNumber('test (1).json') -> 'test(2).json'
+ */
+export function incrementProjectNumber(path: string): string {
+  const { dir, name, ext } = parse(path);
+
+  if (!name.endsWith(')')) return join(dir, `${name} (1)${ext}`);
+
+  const openingParenIndex = name.lastIndexOf(' (');
+  if (openingParenIndex === -1) return join(dir, `${name} (1)${ext}`);
+
+  const maybeNumber = Number(name.slice(openingParenIndex + 2, -1));
+  if (isNaN(maybeNumber)) return join(dir, `${name} (1)${ext}`);
+
+  return join(dir, `${name.slice(0, openingParenIndex)} (${maybeNumber + 1})${ext}`);
+}