fix(app): handle item prefix as string

hendrikheil · hendrikheil · commit e6f5f7620445 · 2025-11-12T11:35:16.000+01:00
diff --git a/src/app/src/components/shared/item/ItemCardForm.vue b/src/app/src/components/shared/item/ItemCardForm.vue
@@ -65,7 +65,21 @@ const schema = computed(() => z.object({
   extension: z.enum([...CONTENT_EXTENSIONS, ...MEDIA_EXTENSIONS] as [string, ...string[]]).nullish(),
   prefix: z.preprocess(
     val => val === '' ? null : val,
-    z.number().int().positive().nullish(),
+    z.string()
+      .regex(/^\d+$/, 'Prefix be a string containing only digits')
+      .refine(
+        (prefix: string | null | undefined) => {
+          if (prefix === null || prefix === undefined) {
+            return true
+          }
+
+          const num = Number(prefix)
+
+          return Number.isInteger(num) && num >= 0
+        },
+        'Prefix must be a positive integer',
+      )
+      .nullish(),
   ),
 }).refine(() => {
   const siblings = props.parentItem.children?.filter(child => !child.hide) || []
@@ -87,7 +101,7 @@ const schema = computed(() => z.object({
 type Schema = {
   name: string
   extension: string | null | undefined
-  prefix: number | null | undefined
+  prefix: string | null | undefined
 }
 const state = reactive<Schema>({
   name: originalName.value,
@@ -267,11 +281,11 @@ async function onSubmit() {
                       <span />
                     </template>
                     <UInput
-                      v-model.number="state.prefix"
-                      type="number"
+                      v-model="state.prefix"
+                      type="text"
+                      pattern="[0-9]*"
                       variant="soft"
                       placeholder="N°"
-                      min="1"
                       class="h-5"
                       size="xs"
                       :disabled="isLoading"
diff --git a/src/app/src/types/tree.ts b/src/app/src/types/tree.ts
@@ -10,7 +10,7 @@ export interface TreeItem {
   name: string
   fsPath: string // unique identifier
   type: 'file' | 'directory' | 'root'
-  prefix: number | null
+  prefix: string | null
   status?: TreeStatus
   routePath?: string
   children?: TreeItem[]
diff --git a/src/app/src/utils/file.ts b/src/app/src/utils/file.ts
@@ -50,11 +50,11 @@ export const FILE_ICONS = {
   ...AUDIO_EXTENSIONS.reduce((acc, ext) => ({ ...acc, [ext]: 'i-lucide-file-audio' }), {}),
 }
 
-export function parseName(name: string): { name: string, prefix: number | null, extension: string | null } {
+export function parseName(name: string): { name: string, prefix: string | null, extension: string | null } {
   const prefixMatch = name.match(/^(\d+)\./)
   const extensionMatch = name.match(/\.(\w+)$/)
   return {
-    prefix: prefixMatch ? Number.parseInt(prefixMatch[1], 10) : null,
+    prefix: prefixMatch ? prefixMatch[1] : null,
     extension: extensionMatch ? extensionMatch[1] : null,
     name: name.replace(/^\d+\./, ''),
   }
diff --git a/src/app/test/mocks/tree.ts b/src/app/test/mocks/tree.ts
@@ -12,34 +12,34 @@ export const tree: TreeItem[] = [
     name: 'getting-started',
     fsPath: '1.getting-started',
     type: 'directory',
-    prefix: 1,
+    prefix: '1',
     children: [
       {
         name: 'introduction',
         fsPath: '1.getting-started/2.introduction.md',
         type: 'file',
         routePath: '/getting-started/introduction',
-        prefix: 2,
+        prefix: '2',
       },
       {
         name: 'installation',
         fsPath: '1.getting-started/3.installation.md',
         type: 'file',
         routePath: '/getting-started/installation',
-        prefix: 3,
+        prefix: '3',
       },
       {
         name: 'advanced',
         fsPath: '1.getting-started/1.advanced',
         type: 'directory',
-        prefix: 1,
+        prefix: '1',
         children: [
           {
             name: 'studio',
             fsPath: '1.getting-started/1.advanced/1.studio.md',
             type: 'file',
             routePath: '/getting-started/installation/advanced/studio',
-            prefix: 1,
+            prefix: '1',
           },
         ],
       },
diff --git a/src/app/test/unit/utils/tree.test.ts b/src/app/test/unit/utils/tree.test.ts
@@ -25,21 +25,21 @@ describe('buildTree of documents with one level of depth', () => {
       name: 'getting-started',
       fsPath: '1.getting-started',
       type: 'directory',
-      prefix: 1,
+      prefix: '1',
       children: [
         {
           name: 'introduction',
           fsPath: '1.getting-started/2.introduction.md',
           type: 'file',
           routePath: '/getting-started/introduction',
-          prefix: 2,
+          prefix: '2',
         },
         {
           name: 'installation',
           fsPath: '1.getting-started/3.installation.md',
           type: 'file',
           routePath: '/getting-started/installation',
-          prefix: 3,
+          prefix: '3',
         },
       ],
     },
@@ -97,7 +97,7 @@ describe('buildTree of documents with one level of depth', () => {
             type: 'file',
             routePath: deletedDbItem.path,
             status: TreeStatus.Deleted,
-            prefix: 2,
+            prefix: '2',
           },
         ],
       },
@@ -131,7 +131,7 @@ describe('buildTree of documents with one level of depth', () => {
             type: 'file',
             routePath: deletedDbItem.path,
             status: TreeStatus.Deleted,
-            prefix: 3,
+            prefix: '3',
           },
         ],
       },
@@ -282,7 +282,7 @@ describe('buildTree of documents with one level of depth', () => {
             name: createdDbItem.path!.split('/').pop()!,
             type: 'file',
             status: TreeStatus.Renamed,
-            prefix: 2,
+            prefix: '2',
           },
         ],
       },
@@ -296,27 +296,27 @@ describe('buildTree of documents with two levels of depth', () => {
       name: 'essentials',
       fsPath: '1.essentials',
       type: 'directory',
-      prefix: 1,
+      prefix: '1',
       children: [
         {
           name: 'configuration',
           fsPath: '1.essentials/2.configuration.md',
           type: 'file',
           routePath: '/essentials/configuration',
-          prefix: 2,
+          prefix: '2',
         },
         {
           name: 'nested',
           fsPath: '1.essentials/1.nested',
           type: 'directory',
-          prefix: 1,
+          prefix: '1',
           children: [
             {
               name: 'advanced',
               fsPath: '1.essentials/1.nested/2.advanced.md',
               type: 'file',
               routePath: '/essentials/nested/advanced',
-              prefix: 2,
+              prefix: '2',
             },
           ],
         },
@@ -424,7 +424,7 @@ describe('buildTree of documents with two levels of depth', () => {
               routePath: deletedDbItem.path,
               type: 'file',
               status: TreeStatus.Deleted,
-              prefix: 2,
+              prefix: '2',
             },
           ],
         },
@@ -452,21 +452,21 @@ describe('buildTree of documents with language prefixed', () => {
           name: 'getting-started',
           fsPath: 'en/1.getting-started',
           type: 'directory',
-          prefix: 1,
+          prefix: '1',
           children: [
             {
               name: 'introduction',
               fsPath: 'en/1.getting-started/2.introduction.md',
               type: 'file',
               routePath: '/en/getting-started/introduction',
-              prefix: 2,
+              prefix: '2',
             },
             {
               name: 'installation',
               fsPath: 'en/1.getting-started/3.installation.md',
               type: 'file',
               routePath: '/en/getting-started/installation',
-              prefix: 3,
+              prefix: '3',
             },
           ],
         },

Original file line number	Diff line number	Diff line change
`@@ -50,11 +50,11 @@ export const FILE_ICONS = {`
`50`	`50`	`...AUDIO_EXTENSIONS.reduce((acc, ext) => ({ ...acc, [ext]: 'i-lucide-file-audio' }), {}),`
`51`	`51`	`}`
`52`	`52`
`53`		`-export function parseName(name: string): { name: string, prefix: number \| null, extension: string \| null } {`
	`53`	`+export function parseName(name: string): { name: string, prefix: string \| null, extension: string \| null } {`
`54`	`54`	`const prefixMatch = name.match(/^(\d+)\./)`
`55`	`55`	`const extensionMatch = name.match(/\.(\w+)$/)`
`56`	`56`	`return {`
`57`		`- prefix: prefixMatch ? Number.parseInt(prefixMatch[1], 10) : null,`
	`57`	`+ prefix: prefixMatch ? prefixMatch[1] : null,`
`58`	`58`	`extension: extensionMatch ? extensionMatch[1] : null,`
`59`	`59`	`name: name.replace(/^\d+\./, ''),`
`60`	`60`	`}`