Revert "fix: added add metadata row feature in study annotations (#1197)"

This reverts commit d9bc608.

Author: nicoalee

compose/neurosynth-frontend/cypress/fixtures/ImportSleuth/neurosynthResponses/annotationsSingleSleuthStudyResponse.json

@@ -25,16 +25,10 @@
     "source_id": null,
     "source_updated_at": null,
     "note_keys": {
-        "included": {
-            "type": "boolean",
-            "order": 0
-        },
-        "test1_new_txt": {
-            "type": "boolean",
-            "order": 1
-        }
+        "included": "boolean",
+        "test1_new_txt": "boolean"
     },
     "metadata": null,
     "name": "Annotation for Untitled sleuth project",
     "description": ""
-}
+}

compose/neurosynth-frontend/cypress/fixtures/IngestionFixtures/annotationsFixture.json

@@ -10,10 +10,7 @@
     "source_id": null,
     "source_updated_at": null,
     "note_keys": {
-        "included": {
-            "type": "boolean",
-            "order": 0
-        }
+        "included": "boolean"
     },
     "metadata": null,
     "name": "Annotation for studyset hQFjozWL9v8Q",

compose/neurosynth-frontend/cypress/fixtures/IngestionFixtures/annotationsPutFixture.json

@@ -10,10 +10,7 @@
     "source_id": null,
     "source_updated_at": null,
     "note_keys": {
-        "included": {
-            "type": "boolean",
-            "order": 0
-        }
+        "included": "boolean"
     },
     "metadata": null,
     "name": "Annotation for studyset hQFjozWL9v8Q",

compose/neurosynth-frontend/cypress/fixtures/annotation.json

@@ -5,14 +5,8 @@
     "metadata": null,
     "name": "Annotation for studyset 73HRs8HaJbR8",
     "note_keys": {
-      "included": {
-        "type": "boolean",
-        "order": 0
-      },
-      "string_key": {
-        "type": "string",
-        "order": 1
-      }
+      "included": "boolean",
+      "string_key": "string"
     },
     "notes": [
       {
@@ -50,4 +44,4 @@
     "user": "github|26612023",
     "username": "Nicholas Lee"
   }
-  
+  

compose/neurosynth-frontend/package-lock.json

@@ -102,7 +102,7 @@
         "prettier": "^3.3.3",
         "react-error-overlay": "^6.0.9",
         "ts-node": "^10.9.2",
-        "typescript": "^5.9.3",
+        "typescript": "^4.9.5",
         "typescript-eslint": "^8.13.0",
         "vite-tsconfig-paths": "^5.1.4",
         "vitest": "^3.0.7"

compose/neurosynth-frontend/package.json

@@ -3,7 +3,6 @@ import { EPropertyType } from 'components/EditMetadata/EditMetadata.types';
 export interface NoteKeyType {
     key: string;
     type: EPropertyType;
-    order: number;
 }
 
 export type AnnotationNoteValue = string | number | boolean | null;

compose/neurosynth-frontend/src/components/HotTables/HotTables.types.ts

@@ -4,32 +4,19 @@ import { CellValue } from 'handsontable/common';
 
 export const noteKeyObjToArr = (noteKeys?: object | null): NoteKeyType[] => {
     if (!noteKeys) return [];
-    const noteKeyTypes = noteKeys as { [key: string]: { type: EPropertyType; order?: number } };
-    const arr = Object.entries(noteKeyTypes)
-        .map(([key, descriptor]) => {
-            if (!descriptor?.type) throw new Error('Invalid note_keys descriptor: missing type');
-            return {
-                // rely on new descriptor shape (type + order)
-                type: descriptor.type,
-                key,
-                order: descriptor.order ?? 0,
-            };
-        })
-        .sort((a, b) => a.order - b.order || a.key.localeCompare(b.key))
-        .map((noteKey, index) => ({ ...noteKey, order: index }));
+    const noteKeyTypes = noteKeys as { [key: string]: EPropertyType };
+    const arr = Object.entries(noteKeyTypes).map(([key, type]) => ({
+        key,
+        type,
+    }));
     return arr;
 };
 
-export const noteKeyArrToObj = (
-    noteKeyArr: NoteKeyType[]
-): { [key: string]: { type: EPropertyType; order: number } } => {
-    const noteKeyObj = noteKeyArr.reduce((acc, curr, index) => {
-        acc[curr.key] = {
-            type: curr.type,
-            order: curr.order ?? index,
-        };
+export const noteKeyArrToObj = (noteKeyArr: NoteKeyType[]): { [key: string]: EPropertyType } => {
+    const noteKeyObj: { [key: string]: EPropertyType } = noteKeyArr.reduce((acc, curr) => {
+        acc[curr.key] = curr.type;
         return acc;
-    }, {} as { [key: string]: { type: EPropertyType; order: number } });
+    }, {} as { [key: string]: EPropertyType });
 
     return noteKeyObj;
 };

compose/neurosynth-frontend/src/components/HotTables/HotTables.utils.spec.ts

@@ -119,7 +119,11 @@ export const annotationNotesToHotData = (
     };
 };
 
-export const createColumnHeader = (colKey: string, colType: EPropertyType, allowRemoveColumn: boolean) => {
+export const createColumnHeader = (
+    colKey: string,
+    colType: EPropertyType,
+    allowRemoveColumn: boolean
+) => {
     const allowRemove = allowRemoveColumn
         ? `<div style="width: 50px; display: flex; align-items: center; justify-content: center">
         ${renderToString(
@@ -137,7 +141,7 @@ export const createColumnHeader = (colKey: string, colType: EPropertyType, allow
 
     return (
         `<div title="${colKey}" style="display: flex; align-items: center; justify-content: center;">` +
-        `<div class="${styles[colType]} ${styles.truncate}" style="width: 100px">${colKey}</div>` +
+        `<div class="${styles[colType]} ${styles.truncate}" style="width: 150px">${colKey}</div>` +
         allowRemove +
         `</div>`
     );
@@ -164,15 +168,17 @@ export const createColumns = (noteKeys: NoteKeyType[], disable?: boolean) =>
                     x.type === EPropertyType.NUMBER
                         ? numericValidator
                         : x.type === EPropertyType.BOOLEAN
-                          ? booleanValidator
-                          : undefined,
+                        ? booleanValidator
+                        : undefined,
             } as ColumnSettings;
         }),
     ] as ColumnSettings[];
 
 // we can assume that the hashmap maintains order and is sorted by key
 // this function gets all merge cells and only merge cells. If a cell does not need to be merged, a mergeCellObj is not creatd
-export const getMergeCells = (hotDataToStudyMapping: Map<number, { studyId: string; analysisId: string }>) => {
+export const getMergeCells = (
+    hotDataToStudyMapping: Map<number, { studyId: string; analysisId: string }>
+) => {
     const mergeCells: MergeCellsSettings[] = [];
 
     let studyId: string;