ref: Reuse parseAsInteger logic, add positive check

Author: franky47

packages/nuqs/src/parsers.test.ts

@@ -3,12 +3,12 @@ import {
   parseAsArrayOf,
   parseAsFloat,
   parseAsHex,
+  parseAsIndex,
   parseAsInteger,
-  parseAsIsoDateTime,
   parseAsIsoDate,
+  parseAsIsoDateTime,
   parseAsString,
-  parseAsTimestamp,
-  parseAsIndex
+  parseAsTimestamp
 } from './parsers'
 
 describe('parsers', () => {
@@ -33,6 +33,8 @@ describe('parsers', () => {
     expect(parseAsIndex.parse('1')).toBe(0)
     expect(parseAsIndex.parse('3.14')).toBe(2)
     expect(parseAsIndex.parse('3,14')).toBe(2)
+    expect(parseAsIndex.parse('0')).toBeNull()
+    expect(parseAsIndex.parse('-1')).toBeNull()
     expect(parseAsIndex.serialize(0)).toBe('1')
     expect(parseAsIndex.serialize(3.14)).toBe('4')
   })

packages/nuqs/src/parsers.ts

@@ -157,13 +157,13 @@ export const parseAsInteger = createParser({
 
 export const parseAsIndex = createParser({
   parse: v => {
-    const int = parseInt(v)
-    if (Number.isNaN(int)) {
+    const int = parseAsInteger.parse(v)
+    if (int === null || int <= 0) {
       return null
     }
     return int - 1
   },
-  serialize: v => Math.round(v + 1).toFixed()
+  serialize: v => parseAsInteger.serialize(v + 1)
 })
 
 export const parseAsHex = createParser({