Fix: Fix navigation to sections with numbers in the name (styleguidist#1595)

Fixes styleguidist#1594

Author: mitsuruog

src/client/utils/__tests__/getInfoFromHash.spec.ts

@@ -35,4 +35,24 @@ describe('getInfoFromHash', () => {
 			targetIndex: undefined,
 		});
 	});
+
+	it('should extract target index when the URL ends with a number', () => {
+		const result = getInfoFromHash('#/Documentation/Files/Buttons/5');
+		expect(result).toEqual({
+			isolate: false,
+			hashArray: ['Documentation', 'Files', 'Buttons'],
+			targetName: 'Documentation',
+			targetIndex: 5,
+		});
+	});
+
+	it('should return a proper parsed result even though the hash starts with a number', () => {
+		const result = getInfoFromHash('#/1.Documentation');
+		expect(result).toEqual({
+			isolate: false,
+			hashArray: ['1.Documentation'],
+			targetName: '1.Documentation',
+			targetIndex: undefined,
+		});
+	});
 });

src/client/utils/getInfoFromHash.ts

@@ -1,8 +1,7 @@
-import isNaN from 'lodash/isNaN';
 import { hasInHash, getHashAsArray } from './handleHash';
 
-function filterNumbers(item: string): boolean {
-	return isNaN(parseInt(item, 10)) && item !== '';
+function hasDigitsOnly(item: string): boolean {
+	return item.match(/^\d+$/) !== null;
 }
 
 /**
@@ -25,12 +24,12 @@ export default function getInfoFromHash(
 	const shouldIsolate = hasInHash(hash, '#!/');
 	if (shouldIsolate || hasInHash(hash, '#/')) {
 		const hashArray = getHashAsArray(hash, shouldIsolate ? '#!/' : '#/');
-		const index = parseInt(hashArray[hashArray.length - 1], 10);
+		const targetHash = hashArray[hashArray.length - 1];
 		return {
 			isolate: shouldIsolate,
-			hashArray: hashArray.filter(filterNumbers),
+			hashArray: hashArray.filter(item => !hasDigitsOnly(item)),
 			targetName: hashArray[0],
-			targetIndex: isNaN(index) ? undefined : index,
+			targetIndex: hasDigitsOnly(targetHash) ? parseInt(targetHash, 10) : undefined,
 		};
 	}
 	return {};