@citolab/tspci-tao 2.3.0-alpha.4 → 2.3.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@citolab/tspci-tao",
-  "version": "2.3.0-alpha.4",
+  "version": "2.3.0",
   "description": "create a zip file that can be imported into TAO",
   "author": "patrick <patrick.deklein@cito.nl>, Marcel Hoekstra <marcel.hoekstra@cito.nl>",
   "repository": "",
@@ -12,5 +12,5 @@
     "inquirer": "9.2.0"
   },
   "scripts": {},
-  "gitHead": "a73cb286cca0f12d474e155915c251f85abf4b16"
+  "gitHead": "c7100a7ee3cd62cb7beb2bd6379c830cdec04a59"
 }

package/readme.md

@@ -17,15 +17,49 @@ Configuration of a PCI can be done by the author because this packager reads the
 If scoring can be done with a MATCH_CORRECT (single string comparison) the item author can provide the correct answer in TAO by interaction with the PCI.
 This way one PCI can be re-used in different configurations supporting multiple items.
 
+## Cli
+To add TAO support to an existing tspci PCI you can run:
+
+``` sh
+ tspci add --target tao
+```
+
+This command and all the TAO specific changes that are needed to build importable TAO zip file. This will work for a new project created by ``` tspci init ```. For PCI's that are modified, it will do best effort to fully implement TAO specific changes. Please read the rest of this file in case ``` tspci init ``` fails on your PCI.
+
+<p align="center">
+  <img src="https://github.com/Citolab/tspci/blob/main/lib/tspci-tao/readme-images/tspci-tao-console.png" alt="cli in console">
+</p>
+
+The cli will ask a few things before the script is executed:
+
+- short: name on the interaction icon in TAO.
+- label: label of the interaction
+- scoringMethod: the MATCH CORRECT scoring can be applied on the PCI. The user can interacted with the PCI to provide the correct response. To be able to show the correct response in the author system after the author provided the correct response; the setResponse function of the TAO interface should be implemented. Scoring can (for now) only be done using a string comparison. But sinces the PCI developer implement the getResponse method, as string comparison can be done to compare JSON of formatted numbers.
+
+## Changes
+
 To be able to add an IMS-PCI to the TAO authoring system properties are added to the `package.json`
 and implemented lifecycle events that should be implemented in the PCI.
 
-To create this zip file run:
+
+
+or
+
+``` sh
+ npm run tspci -- add --target tao
+```
+
+To create the zip file that can be added in TAO run:
 
 ``` sh 
  tspci --target tao
 ```
 
+or
+
+``` sh
+ npm run tspci -- --target tao
+```
 ## life cycle methods
 
 To be TAO compliant the TAOpci interface should be implemented:
@@ -68,6 +102,20 @@ class App implements IMSpci<PropTypes>, TAOpci {
 
 
 ### package.json
+
+The version and description fields of the package.json are used while creating the zip file and are mandatory.
+
+Other config properties should be placed under: config -> tspci:
+
+``` json
+ "config": {
+  "tspci": {
+    "typeIdentifier": "myPci",
+    ...
+  }
+ }
+```
+
 - typeIdentifier: a field 'typeIdentifier' should be added. This should match the 'typeIdentifier' value in the PCI entry file (The one that implements IMSPci). Only (alpha)numeric values are allowed. This because a hyphen in the identifier causes an error in TAO. To be sure, only alphanumeric are allowed.
 - label: used as label in TAO
 - short: short name used in TAO to label the PCI where you can drag it in an item.

package/release-notes.md

@@ -1,3 +1,6 @@
+# 2.3.0
+- added a script to add TAO support in a PCI that only support the IMS interface.
+
 # 2.0.0
 
 - synchronized versions across tspci libraries

package/src/init.mjs

@@ -3,6 +3,7 @@ import path from "path";
 import readline from "readline";
 import chalk from "chalk";
 import inquirer from "inquirer";
+
 function askOption(prompt) {
   const rl = readline.createInterface({
     input: process.stdin,
@@ -59,11 +60,23 @@ export const init = async () => {
   packageData.config.tspci.typeIdentifier = typeIdentifer;
   packageData.config.tspci.score = scoreMethodArray;
 
+  packageData.devDependencies = packageData.devDependencies || {};
+  
+  // get package.json from PACKAGER_PATH
+  const PACKAGER_PATH = path.join(PACKAGE_ROOT_PATH + "/node_modules/@citolab/tspci-tao");
+  const packagerPackageData = JSON.parse(fs.readFileSync(path.join(PACKAGER_PATH, "package.json"), "utf8"));
+  // add devDependencies from packagerPackageData to packageData
+  Object.keys(packagerPackageData.devDependencies).forEach((key) => {
+    packageData.devDependencies[key] = packagerPackageData.devDependencies[key];
+  });
+
+  packageData.dependencies["@citolab/tspci"] = "^1.0.0";
   packageData.scripts = packageData.scripts || {};
   packageData.scripts["package:tao"] = "tspci --target tao";
   // store packageData
   fs.writeFileSync(path.join(PACKAGE_ROOT_PATH, "package.json"), JSON.stringify(packageData, null, 2), "utf8");
-
+  console.log(chalk.green("📦 Installing dependencies"));
+  execSync("npm install", { cwd: destinationFolder, stdio: [0, 1, 2] });
   console.log("Adding TAO interface to PCI");
   if (!packageData.source) {
     console.error("Failed implementing TAO interface; No source property found in package.json.");
@@ -85,19 +98,19 @@ export const init = async () => {
         newSourceFile = `import { QtiVariableJSON } from "@citolab/tspci";\r\n${newSourceFile}`;
       }
       const defaultImplementation = `
-    off = () => {}; // called when setting correct response in tao
-    on = (val) => {};
-    trigger = (event: string, value: any) => {
-      this.config.properties[event] = value;
-      this.render();
-    };
-    resetResponse = () => {
-      // used in tao to reset the response. clean-up and re-render the PCI would be best. 
-      // this.render();
-    };
-    setResponse = (response: QtiVariableJSON) => {
-      // used to set the response in tao. E.g. when the item author has set a correct response. And you refresh TAO it will call this method to set the response in the PCI in the score-editor.
-    };
+off = () => {}; // called when setting correct response in tao
+on = (val) => {};
+trigger = (event: string, value: any) => {
+  this.config.properties[event] = value;
+  this.render();
+};
+resetResponse = () => {
+  // used in tao to reset the response. clean-up and re-render the PCI would be best. 
+  // this.render();
+};
+setResponse = (response: QtiVariableJSON) => {
+  // used to set the response in tao. E.g. when the item author has set a correct response. And you refresh TAO it will call this method to set the response in the PCI in the score-editor.
+};
       `
       const lastBracket = newSourceFile.lastIndexOf("}");
       newSourceFile = newSourceFile.substring(0, lastBracket) + defaultImplementation + newSourceFile.substring(lastBracket);