feat(server): Update getDraftHead to resolve modified time from commit history

Author: nellh

packages/openneuro-server/src/datalad/dataset.ts

@@ -69,15 +69,26 @@ export const createDataset = async (
   }
 }
 
+interface WorkerDraftFields {
+  // Commit id hash
+  ref: string
+  // Commit tree ref
+  tree: string
+  // Commit message
+  message: string
+  // Commit author time
+  modified: string
+}
+
 /**
  * Return the latest commit
  * @param {string} id Dataset accession number
  */
-export const getDraftHead = async (id) => {
+export const getDraftHead = async (id): Promise<WorkerDraftFields> => {
   const draftRes = await request
     .get(`${getDatasetWorker(id)}/datasets/${id}/draft`)
     .set("Accept", "application/json")
-  return draftRes.body.hexsha
+  return draftRes.body
 }
 
 /**
@@ -87,7 +98,7 @@ export const getDataset = async (id) => {
   const dataset = await Dataset.findOne({ id }).lean()
   return {
     ...dataset,
-    revision: await getDraftHead(id),
+    revision: (await getDraftHead(id)).ref,
   }
 }
 

packages/openneuro-server/src/graphql/resolvers/dataset.ts

@@ -276,11 +276,14 @@ const worker = (obj) => getDatasetWorker(obj.id)
  */
 const Dataset = {
   uploader: (ds) => user(ds, { id: ds.uploader }),
-  draft: async (obj) => ({
-    id: obj.id,
-    revision: await datalad.getDraftHead(obj.id),
-    modified: obj.modified,
-  }),
+  draft: async (obj) => {
+    const draftHead = await datalad.getDraftHead(obj.id)
+    return {
+      id: obj.id,
+      revision: draftHead.ref,
+      modified: draftHead.modified,
+    }
+  },
   snapshots,
   latestSnapshot,
   analytics,

packages/openneuro-server/src/graphql/resolvers/snapshots.ts

@@ -227,7 +227,7 @@ export const latestSnapshot = async (obj, _, context) => {
     // In the case where there are no real snapshots, return most recent commit as snapshot
     return await snapshot(
       obj,
-      { datasetId: obj.id, tag: await getDraftHead(obj.id) },
+      { datasetId: obj.id, tag: (await getDraftHead(obj.id)).ref },
       context,
     )
   }