@khanacademy/perseus-score 2.1.0 → 2.2.1

package/dist/es/index.js

@@ -12847,7 +12847,8 @@ function scoreIframe(userInput) {
 const {
   collinear,
   canonicalSineCoefficients,
-  similar
+  similar,
+  clockwise
 } = geometry;
 const {
   getClockwiseAngle
@@ -13007,9 +13008,27 @@ function scoreInteractiveGraph(userInput, rubric) {
         };
       }
     } else if (userInput.type === "angle" && rubric.correct.type === "angle") {
-      const guess = userInput.coords;
+      const coords = userInput.coords;
       const correct = rubric.correct.coords;
       const allowReflexAngles = rubric.correct.allowReflexAngles;
+
+      // While the angle graph should always have 3 points, our types
+      // technically allow for null values. We'll check for that here.
+      // TODO: (LEMS-2857) We would like to update the type of coords
+      // to be non-nullable, as the graph should always have 3 points.
+      if (!coords) {
+        return {
+          type: "invalid",
+          message: null
+        };
+      }
+
+      // We need to check both the direction of the angle and the
+      // whether the graph allows for reflexive angles in order to
+      // to determine if we need to reverse the coords for scoring.
+      const areClockwise = clockwise([coords[0], coords[2], coords[1]]);
+      const shouldReverseCoords = areClockwise && !allowReflexAngles;
+      const guess = shouldReverseCoords ? coords.slice().reverse() : coords;
       let match;
       if (rubric.correct.match === "congruent") {
         const angles = _.map([guess, correct], function (coords) {
@@ -13023,13 +13042,7 @@ function scoreInteractiveGraph(userInput, rubric) {
         match = approximateEqual(...angles);
       } else {
         /* exact */
-        match =
-        // @ts-expect-error - TS2532 - Object is possibly 'undefined'. | TS2532 - Object is possibly 'undefined'.
-        approximateDeepEqual(guess[1], correct[1]) &&
-        // @ts-expect-error - TS2532 - Object is possibly 'undefined'. | TS2532 - Object is possibly 'undefined'. | TS2532 - Object is possibly 'undefined'.
-        collinear(correct[1], correct[0], guess[0]) &&
-        // @ts-expect-error - TS2532 - Object is possibly 'undefined'. | TS2532 - Object is possibly 'undefined'. | TS2532 - Object is possibly 'undefined'.
-        collinear(correct[1], correct[2], guess[2]);
+        match = approximateDeepEqual(guess[1], correct[1]) && collinear(correct[1], correct[0], guess[0]) && collinear(correct[1], correct[2], guess[2]);
       }
       if (match) {
         return {