@openneuro/server 4.2.3 → 4.3.0-alpha.0

package/Dockerfile

@@ -11,7 +11,8 @@ WORKDIR /srv
 COPY --from=build /srv/packages/openneuro-server/package.json /srv/package.json
 COPY --from=build /srv/.yarnrc.yml /srv/.yarnrc.yml
 COPY --from=build /srv/.yarn /srv/.yarn
-COPY --from=build /srv/.pnp.js /srv/.pnp.js
+COPY --from=build /srv/.pnp.cjs /srv/.pnp.cjs
+COPY --from=build /srv/.pnp.loader.mjs /srv/.pnp.loader.mjs
 COPY --from=build /srv/packages/openneuro-server/dist /srv/packages/openneuro-server/dist
 
 HEALTHCHECK --interval=10s --retries=10 CMD curl -f 'http://localhost:8111' || exit 1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@openneuro/server",
-  "version": "4.2.3",
+  "version": "4.3.0-alpha.0",
   "description": "Core service for the OpenNeuro platform.",
   "license": "MIT",
   "main": "src/server.js",
@@ -66,7 +66,7 @@
     "subscriptions-transport-ws": "0.9.18",
     "superagent": "^3.8.2",
     "ts-node": "9.1.1",
-    "typescript": "4.0.6",
+    "typescript": "4.5.4",
     "underscore": "^1.8.3",
     "uuid": "^3.0.1",
     "xmldoc": "^1.1.0"
@@ -104,5 +104,5 @@
   "publishConfig": {
     "access": "public"
   },
-  "gitHead": "423d7a2ba339e178f009e894de5e2a0f18b7d46d"
+  "gitHead": "966931efa8d06bbf0b9540038ab3decab086d745"
 }

package/src/graphql/resolvers/mutation.js

@@ -15,6 +15,7 @@ import {
   createSnapshot,
   deleteSnapshot,
   deprecateSnapshot,
+  undoDeprecateSnapshot,
 } from './snapshots.js'
 import { removeUser, setAdmin, setBlocked } from './user.js'
 import { updateSummary } from './summary.js'
@@ -44,6 +45,7 @@ const Mutation = {
   flagAnnexObject,
   createSnapshot,
   deprecateSnapshot,
+  undoDeprecateSnapshot,
   deleteSnapshot,
   updateSummary,
   updateValidation,

package/src/graphql/resolvers/snapshots.js

@@ -9,10 +9,6 @@ import { getFiles, filterFiles } from '../../datalad/files.js'
 import DatasetModel from '../../models/dataset'
 import { filterRemovedAnnexObjects } from '../utils/file.js'
 import DeprecatedSnapshot from '../../models/deprecatedSnapshot'
-import { checkDatasetAdmin } from '../permissions.js'
-import SnapshotModel from '../../models/snapshot'
-import User from '../../models/user'
-import * as Sentry from '@sentry/node'
 import { redis } from '../../libs/redis'
 import CacheItem, { CacheType } from '../../cache/item'
 import { normalizeDOI } from '../../libs/doi/normalize'
@@ -34,15 +30,16 @@ export const snapshot = (obj, { datasetId, tag }, context) => {
           getFiles(datasetId, snapshot.hexsha)
             .then(filterFiles(prefix))
             .then(filterRemovedAnnexObjects(datasetId, context.userInfo)),
-        deprecated: () => deprecated(snapshot),
+        deprecated: () => deprecated({ datasetId, tag }),
         related: () => related(datasetId),
       }))
     },
   )
 }
 
-export const deprecated = snapshot => {
-  return DeprecatedSnapshot.findOne({ id: snapshot.hexsha }).populate('user')
+export const deprecated = ({ datasetId, tag }) => {
+  const id = `${datasetId}:${tag}`
+  return DeprecatedSnapshot.findOne({ id }).lean().exec()
 }
 
 /**
@@ -89,22 +86,41 @@ export const deprecateSnapshot = async (
   { datasetId, tag, reason },
   { user, userInfo },
 ) => {
-  try {
-    await checkDatasetAdmin(datasetId, user, userInfo)
-    const [snapshot, userDoc] = await Promise.all([
-      SnapshotModel.findOne({ datasetId, tag }),
-      User.findOne({ id: user }),
-    ])
-    await DeprecatedSnapshot.create({
-      id: snapshot.hexsha,
-      user: userDoc._id,
-      cause: reason,
-      timestamp: new Date(),
-    })
-    return true
-  } catch (err) {
-    Sentry.captureException(err)
-    throw err
+  const id = `${datasetId}:${tag}`
+  await checkDatasetWrite(datasetId, user, userInfo)
+  const timestamp = new Date()
+  await DeprecatedSnapshot.updateOne(
+    { id },
+    {
+      id,
+      user,
+      reason,
+      timestamp,
+    },
+    { upsert: true },
+  )
+  return {
+    id,
+    deprecated: {
+      id,
+      user,
+      reason,
+      timestamp,
+    },
+  }
+}
+
+export const undoDeprecateSnapshot = async (
+  obj,
+  { datasetId, tag },
+  { user, userInfo },
+) => {
+  const id = `${datasetId}:${tag}`
+  await checkDatasetWrite(datasetId, user, userInfo)
+  await DeprecatedSnapshot.findOneAndDelete({ id })
+  return {
+    id,
+    deprecated: null,
   }
 }
 

package/src/graphql/schema.js

@@ -176,7 +176,9 @@ export const typeDefs = `
     # Reset draft commit
     resetDraft(datasetId: ID!, ref: String!): Boolean
     # Flag snapshot as deprecated
-    deprecateSnapshot(datasetId: ID!, tag: String!, reason: String!): Boolean
+    deprecateSnapshot(datasetId: ID!, tag: String!, reason: String!): Snapshot
+    # Unflag snapshot as deprecated
+    undoDeprecateSnapshot(datasetId: ID!, tag: String!): Snapshot
     # Create anonymous read only reviewer
     createReviewer(datasetId: ID!): DatasetReviewer
     # Remove reviewer
@@ -485,9 +487,9 @@ export const typeDefs = `
     # hexsha of deprecated snapshots
     id: ID!
     # ID of user who flagged snapshot as deprecated
-    user: User
+    user: String
     # Reason for deprecating snaphot
-    cause: String
+    reason: String
     # Timestamp of snapshot deprecation
     timestamp: Date
   }

package/src/models/deprecatedSnapshot.ts

@@ -4,7 +4,7 @@ const { Schema, model } = mongoose
 export interface DeprecatedSnapshotDocument extends Document {
   id: string // snapshot hexsha
   user: string
-  cause: string
+  reason: string
   timestamp: Date
 }
 
@@ -13,9 +13,8 @@ const deprecatedSnapshotSchema = new Schema({
   user: {
     type: String,
     required: true,
-    ref: 'User',
   },
-  cause: {
+  reason: {
     type: String,
     required: true,
   },

package/src/server.js

@@ -1,7 +1,7 @@
 /** Needs to run before the other imports in Node */
-import { start } from 'elastic-apm-node'
-start({
-  serviceName: '@openneuro/server',
+import apm from 'elastic-apm-node'
+apm.start({
+  serviceName: 'openneuro-server',
 })
 
 import * as Sentry from '@sentry/node'