- Responded to reviewer feedback

Author: heath-freenome

packages/core/src/components/fields/LayoutGridField.tsx

@@ -134,11 +134,11 @@ function getNonNullishValue<T = unknown>(value?: T, fallback?: T): T | undefined
 
 /** Detects if a `str` is made up entirely of numeric characters
  *
- * @param str - The string to check to see if it as number
+ * @param str - The string to check to see if it is a numeric index
  * @return - True if the string consists entirely of numeric characters
  */
-function isNumeric(str: string) {
-  return /^-?\d+(\.\d+)?$/.test(str); // Matches integers or decimals, positive or negative
+function isNumericIndex(str: string) {
+  return /^\d+?$/.test(str); // Matches positive integers
 }
 
 /** The `LayoutGridField` will render a schema, uiSchema and formData combination out into a GridTemplate in the shape
@@ -531,7 +531,7 @@ export default class LayoutGridField<
     idSchema: IdSchema<T>;
   } {
     let rawSchema: S | undefined;
-    if (isNumeric(potentialIndex) && schema && schema?.type === 'array' && has(schema, ITEMS_KEY)) {
+    if (isNumericIndex(potentialIndex) && schema && schema?.type === 'array' && has(schema, ITEMS_KEY)) {
       const index = Number(potentialIndex);
       const items = schema[ITEMS_KEY];
       if (Array.isArray(items)) {

packages/core/test/LayoutGridField.test.tsx

@@ -1246,7 +1246,7 @@ describe('LayoutGridField', () => {
       retrieveSchemaSpy.mockRestore();
       toIdSchemaSpy.mockRestore();
     });
-    test('returns schema, isRequired: false, isReadonly: undefined, options when oneOf schema is requested', () => {
+    test('returns schema, isRequired: false, isReadonly: undefined, options undefined when 1d array schema is requested', () => {
       const path = 'example.0';
       const schema = innerArraySchema;
       expect(
@@ -1267,7 +1267,7 @@ describe('LayoutGridField', () => {
       expect(retrieveSchemaSpy).toHaveBeenCalledTimes(2);
       expect(toIdSchemaSpy).not.toHaveBeenCalled();
     });
-    test('returns schema, isRequired: true, isReadonly: true, options: undefined when selecting readonly field', () => {
+    test('returns schema, isRequired: false, isReadonly: undefined, options: undefined when 2d array schema is requested', () => {
       const path = 'example.0.1';
       const schema = get(innerArraySchema.items, '1');
       expect(