From e7a4ad91560327cd4fd525b335d872284b9cf242 Mon Sep 17 00:00:00 2001
From: vzhang03 <vzhang@vassar.edu>
Date: Thu, 27 Jun 2024 14:58:35 -0400
Subject: [PATCH] Addressed minor pr comments

---
 examples/timeline-variables.html        | 28 +++----------------------
 packages/jspsych/src/modules/plugins.ts |  2 --
 packages/metadata/package.json          |  2 +-
 packages/metadata/src/PluginCache.ts    |  3 +--
 4 files changed, 5 insertions(+), 30 deletions(-)

diff --git a/examples/timeline-variables.html b/examples/timeline-variables.html
index d3d87dd4..15152ad1 100644
--- a/examples/timeline-variables.html
+++ b/examples/timeline-variables.html
@@ -14,19 +14,9 @@
     const metadata_options = {
       randomField: "this is a field",
       author: {
-        "John Cena": {
-          name: "John Cena",
-          givenName: "Sammy Salami",
-        },
-        "Darnell": {
-          givenName: "bob",
-        },
-        "Bob": "Bob",
-        "Joe": {
-          name: "Joe",
-        },
-        "Ostrich": {
-          bbqblaster: 10,
+        "John": {
+          name: "John",
+          givenName: "Johnathan",
         },
       },
       variables: {
@@ -35,21 +25,9 @@
             "chat-plugin": "this chat plugin allows you to talk to gpt!",
           }
         },
-        "time_elapsed": { 
-          bbq: "indicates the number of barbeques",
-          description: {
-            "chat-plugin": "this chat plugin allows you to talk to gpt!",
-            "donut-plugin": "this plugin lets you eat donuts!",
-            "bbq-plugin": "this plugin lets you spell barbeque",
-          }
-        },
         "trial_index": {
-          value: "NEW VALUE",
-          bbq: "number of barbeques per sq ft",
           name: "index",
-          donuts: "Number of donuts",
         },
-
       },
     }
 
diff --git a/packages/jspsych/src/modules/plugins.ts b/packages/jspsych/src/modules/plugins.ts
index f0a32284..9340da92 100644
--- a/packages/jspsych/src/modules/plugins.ts
+++ b/packages/jspsych/src/modules/plugins.ts
@@ -144,9 +144,7 @@ export type UniversalPluginParameters = InferredParameters<typeof universalPlugi
 
 export interface PluginInfo {
   name: string;
-  version?: string;
   parameters: ParameterInfos;
-  data?: ParameterInfos;
 }
 
 export interface JsPsychPlugin<I extends PluginInfo> {
diff --git a/packages/metadata/package.json b/packages/metadata/package.json
index bfadcc53..ab673b16 100644
--- a/packages/metadata/package.json
+++ b/packages/metadata/package.json
@@ -1,7 +1,7 @@
 {
   "name": "@jspsych/metadata",
   "version": "0.0.1",
-  "description": "jsPsych plugin for creating and customizing metadata according to Psych-DS standards.",
+  "description": "jsPsych package for creating and customizing metadata according to Psych-DS standards.",
   "type": "module",
   "main": "dist/index.cjs",
   "exports": {
diff --git a/packages/metadata/src/PluginCache.ts b/packages/metadata/src/PluginCache.ts
index 49452b41..80d47903 100644
--- a/packages/metadata/src/PluginCache.ts
+++ b/packages/metadata/src/PluginCache.ts
@@ -18,7 +18,6 @@ export class PluginCache {
   async getPluginInfo(pluginType: string, variableName: string) {
     // fetches if it doesn't exist
     if (!(pluginType in this.pluginFields)) {
-      console.log("doesn't exist -- plugintype:", pluginType, "variableName:", variableName); // checking search
       const fields = await this.generatePluginFields(pluginType);
       this.pluginFields[pluginType] = fields;
     }
@@ -57,7 +56,7 @@ export class PluginCache {
         pluginType,
         "with error",
         error,
-        "If you are using a plugin not in the main JsPsych branch it will not automatically fetch."
+        "Note: if you are using a plugin not in the main JsPsych branch this will always fail."
       );
       return undefined;
     }