@playcademy/vite-plugin 1.3.1-beta.2 → 1.3.1-beta.3

package/dist/index.js

@@ -24317,7 +24317,7 @@ import path2 from "node:path";
 // package.json
 var package_default = {
   name: "@playcademy/vite-plugin",
-  version: "1.3.1-beta.2",
+  version: "1.3.1-beta.3",
   type: "module",
   exports: {
     ".": {
@@ -25934,7 +25934,7 @@ var package_default2;
 var init_package = __esm(() => {
   package_default2 = {
     name: "@playcademy/sandbox",
-    version: "0.8.1-beta.2",
+    version: "0.8.1-beta.3",
     description: "Local development server for Playcademy game development",
     type: "module",
     exports: {
@@ -35697,39 +35697,21 @@ function compareNewestCompletedAssessmentAttempt(left, right) {
   return newestAttempt(left, right, "scoreDate");
 }
 function compareAssessmentCatalogOrder(left, right) {
-  if (left.sortOrder !== null || right.sortOrder !== null) {
-    if (left.sortOrder === null) {
-      return 1;
-    }
-    if (right.sortOrder === null) {
-      return -1;
-    }
-    const orderDifference = left.sortOrder - right.sortOrder;
-    if (orderDifference) {
-      return orderDifference;
-    }
-  }
-  const leftUpdatedAt = Date.parse(left.updatedAt);
-  const rightUpdatedAt = Date.parse(right.updatedAt);
-  const leftHasValidDate = Number.isFinite(leftUpdatedAt);
-  const rightHasValidDate = Number.isFinite(rightUpdatedAt);
-  if (leftHasValidDate && rightHasValidDate) {
-    const updatedDifference = leftUpdatedAt - rightUpdatedAt;
-    if (updatedDifference) {
-      return updatedDifference;
-    }
-  } else if (leftHasValidDate !== rightHasValidDate) {
-    return leftHasValidDate ? -1 : 1;
-  }
   if (left.assessmentKey !== null && right.assessmentKey !== null) {
-    const keyDifference = left.assessmentKey.localeCompare(right.assessmentKey);
+    const keyDifference = compareCodePoints(left.assessmentKey, right.assessmentKey);
     if (keyDifference) {
       return keyDifference;
     }
   } else if (left.assessmentKey !== null || right.assessmentKey !== null) {
     return left.assessmentKey === null ? 1 : -1;
   }
-  return left.qtiTestIdentifier.localeCompare(right.qtiTestIdentifier);
+  return compareCodePoints(left.qtiTestIdentifier, right.qtiTestIdentifier);
+}
+function compareCodePoints(left, right) {
+  if (left === right) {
+    return 0;
+  }
+  return left < right ? -1 : 1;
 }
 function orderRuntimeAssessmentTests(tests) {
   return [...tests].toSorted(compareAssessmentCatalogOrder);
@@ -36574,7 +36556,7 @@ var REVIEW_SELECTION_POLICY_VERSION = "unseen-first-least-recently-delivered-v1"
 var PLAYCADEMY_REVIEW_BANK_MANIFEST_METADATA_KEY = "playcademyReviewBank";
 var PLAYCADEMY_REVIEW_BANK_MANIFEST_VERSION = 1;
 var DEFAULT_REVIEW_SELECTION_POLICY;
-var ASSESSMENT_RUNTIME_FIXTURE_BUNDLE_VERSION = 7;
+var ASSESSMENT_RUNTIME_FIXTURE_BUNDLE_VERSION = 8;
 var RuntimeSubjectSchema;
 var RuntimeGradeSchema;
 var OptionalQueryGradeSchema;
@@ -71716,7 +71698,6 @@ var UpdateAssessmentRequestSchema;
 var CopyAssessmentRequestSchema;
 var AssessmentAssociationImportEntrySchema;
 var AttachExistingAssessmentsRequestSchema;
-var ReorderAssessmentsRequestSchema;
 var ReorderQuestionsRequestSchema;
 var init_schemas4 = __esm(() => {
   init_esm();
@@ -72004,8 +71985,7 @@ var init_schemas4 = __esm(() => {
     assessmentKey: AssessmentKeySchema,
     qtiTestIdentifier: exports_external.string().trim().min(1, "Assessment identifier is required"),
     purpose: AssessmentPurposeSchema,
-    standard: AssessmentStandardRefSchema2.optional(),
-    sortOrder: exports_external.number().int().nonnegative().nullable()
+    standard: AssessmentStandardRefSchema2.optional()
   }).superRefine((input, context2) => {
     requireMasteryStandard(input, context2);
     rejectSystemManagedReview(input, context2);
@@ -72055,10 +72035,6 @@ var init_schemas4 = __esm(() => {
       seenKeys.add(assessment.assessmentKey);
     });
   });
-  ReorderAssessmentsRequestSchema = exports_external.object({
-    purpose: AssessmentPurposeSchema,
-    testIdentifiers: exports_external.array(exports_external.string().trim().min(1, "Assessment identifier is required")).min(1, "At least one assessment is required")
-  }).superRefine(rejectSystemManagedReview);
   ReorderQuestionsRequestSchema = exports_external.object({
     items: exports_external.array(exports_external.object({
       identifier: exports_external.string().min(1),
@@ -126748,9 +126724,6 @@ function buildAssessmentAssociationUpdates(row, input) {
       updates.standardFramework = null;
       updates.standardIdentifier = null;
     }
-    if (row.status === "live" && input.purpose !== row.purpose) {
-      updates.sortOrder = null;
-    }
   }
   if (input.standard !== undefined) {
     updates.standardFramework = input.standard.framework;
@@ -126758,9 +126731,6 @@ function buildAssessmentAssociationUpdates(row, input) {
   }
   if (input.status !== undefined) {
     updates.status = input.status;
-    if (input.status === "archived") {
-      updates.sortOrder = null;
-    }
   }
   return updates;
 }
@@ -126769,24 +126739,8 @@ function assessmentStandardForRow(row) {
 }
 function validateUniqueAssessmentIdentifiers(testIdentifiers) {
   if (new Set(testIdentifiers).size !== testIdentifiers.length) {
-    throw new ValidationError("Assessment order must contain unique identifiers");
-  }
-}
-function assertAssessmentOrderUpdateSucceeded(updatedRow) {
-  if (!updatedRow) {
-    throw new ValidationError("Assessment order changed while it was being saved. Refresh and try again.");
+    throw new ValidationError("Assessment manifests must contain unique QTI test identifiers");
   }
-  return updatedRow;
-}
-function lockOrderAssessmentRows(rows) {
-  return rows.toSorted((left, right) => left.id.localeCompare(right.id));
-}
-function orderLiveAssessmentRows(liveRows, purpose, testIdentifiers) {
-  const rowsByIdentifier = new Map(liveRows.map((row) => [row.qtiTestIdentifier, row]));
-  if (liveRows.length !== testIdentifiers.length || testIdentifiers.some((identifier) => !rowsByIdentifier.has(identifier))) {
-    throw new ValidationError(`Assessment order must include every live ${purpose} assessment exactly once`);
-  }
-  return testIdentifiers.map((identifier) => rowsByIdentifier.get(identifier));
 }
 var init_timeback_assessment_rules_util = __esm(() => {
   init_errors2();
@@ -128291,8 +128245,6 @@ var init_timeback_assessment_runtime_service = __esm(async () => {
         id: row.id,
         assessmentKey: row.assessmentKey,
         qtiTestIdentifier: row.qtiTestIdentifier,
-        sortOrder: row.sortOrder,
-        updatedAt: row.updatedAt.toISOString(),
         standard: assessmentStandardForRow(row)
       })).filter((test) => !matchesMasteryStandard || matchesMasteryStandard(test.standard));
     }
@@ -129903,7 +129855,6 @@ class TimebackAssessmentsService {
       ...test ? { title: test.title } : {},
       purpose: entry2.purpose,
       ...entry2.standard ? { standard: entry2.standard } : {},
-      sourceSortOrder: entry2.sortOrder,
       ...test && gameSlug ? { editable: isQtiTestOwnedByGame(test, gameSlug) } : {}
     };
   }
@@ -129963,7 +129914,6 @@ class TimebackAssessmentsService {
           qtiTestIdentifier: entry2.qtiTestIdentifier,
           purpose: entry2.purpose,
           status: input.targetStatus,
-          sortOrder: entry2.sortOrder,
           standardFramework: entry2.standard?.framework,
           standardIdentifier: entry2.standard?.identifier
         }).onConflictDoNothing().returning();
@@ -130096,7 +130046,7 @@ class TimebackAssessmentsService {
       if (plan.kind === "delete") {
         await tx.delete(gameTimebackAssessmentTests).where(eq(gameTimebackAssessmentTests.id, row.id));
       } else if (plan.kind === "archive") {
-        await tx.update(gameTimebackAssessmentTests).set({ status: "archived", sortOrder: null, updatedAt: new Date }).where(eq(gameTimebackAssessmentTests.id, row.id));
+        await tx.update(gameTimebackAssessmentTests).set({ status: "archived", updatedAt: new Date }).where(eq(gameTimebackAssessmentTests.id, row.id));
       }
       if (plan.kind !== "none") {
         setAttribute("app.assessment.operation", plan.operation);
@@ -130104,27 +130054,6 @@ class TimebackAssessmentsService {
       return { action: plan.action };
     });
   }
-  async reorderAssessments(integrationId, purpose, testIdentifiers) {
-    if (purpose === "review") {
-      throw new ValidationError("The review bank is system-managed and cannot be reordered.");
-    }
-    await this.requireIntegration(integrationId);
-    validateUniqueAssessmentIdentifiers(testIdentifiers);
-    const updatedAt = new Date;
-    await this.deps.db.transaction(async (tx) => {
-      const liveRows = await tx.query.gameTimebackAssessmentTests.findMany({
-        where: and(eq(gameTimebackAssessmentTests.integrationId, integrationId), eq(gameTimebackAssessmentTests.purpose, purpose), eq(gameTimebackAssessmentTests.status, "live"))
-      });
-      const orderedRows = orderLiveAssessmentRows(liveRows, purpose, testIdentifiers);
-      const positionByRowId = new Map(orderedRows.map((row, index2) => [row.id, index2 + 1]));
-      for (const row of lockOrderAssessmentRows(orderedRows)) {
-        const testIdentifier = row.qtiTestIdentifier;
-        const [updatedRow] = await tx.update(gameTimebackAssessmentTests).set({ sortOrder: positionByRowId.get(row.id), updatedAt }).where(and(eq(gameTimebackAssessmentTests.id, row.id), eq(gameTimebackAssessmentTests.integrationId, integrationId), eq(gameTimebackAssessmentTests.qtiTestIdentifier, testIdentifier), eq(gameTimebackAssessmentTests.purpose, purpose), eq(gameTimebackAssessmentTests.status, "live"))).returning({ id: gameTimebackAssessmentTests.id });
-        assertAssessmentOrderUpdateSucceeded(updatedRow);
-      }
-    });
-    setAttribute("app.assessment.operation", "reorder_live_assessments");
-  }
   async listQuestions(integrationId, qtiTestIdentifier) {
     const client2 = this.requireClient();
     await this.requireAssessmentRow(integrationId, qtiTestIdentifier);
@@ -130587,15 +130516,14 @@ class TimebackAssessmentsService {
     if (reservedKeyOwner && reservedKeyOwner.purpose !== "review") {
       throw new ValidationError("The reserved review-bank assessment key is already in use.");
     }
-    const singletonCurrent = Boolean(singleton && singleton.assessmentKey === input.sinkIdentifier && singleton.status === "live" && singleton.sortOrder === null && singleton.standardFramework === null && singleton.standardIdentifier === null && singleton.diagnosticKey === null && singleton.diagnosticRoutingManifest === null);
-    const legacyCurrent = legacyReviews.every((association) => association.status === "archived" && association.sortOrder === null && association.assessmentKey !== input.sinkIdentifier);
+    const singletonCurrent = Boolean(singleton && singleton.assessmentKey === input.sinkIdentifier && singleton.status === "live" && singleton.standardFramework === null && singleton.standardIdentifier === null && singleton.diagnosticKey === null && singleton.diagnosticRoutingManifest === null);
+    const legacyCurrent = legacyReviews.every((association) => association.status === "archived" && association.assessmentKey !== input.sinkIdentifier);
     const changed = !singletonCurrent || !legacyCurrent;
     await input.beforeInstall();
     for (const legacy of legacyReviews) {
-      if (legacy.status !== "archived" || legacy.sortOrder !== null || legacy.assessmentKey === input.sinkIdentifier) {
+      if (legacy.status !== "archived" || legacy.assessmentKey === input.sinkIdentifier) {
         await input.tx.update(gameTimebackAssessmentTests).set({
           status: "archived",
-          sortOrder: null,
           ...legacy.assessmentKey === input.sinkIdentifier ? { assessmentKey: null } : {},
           updatedAt: new Date
         }).where(eq(gameTimebackAssessmentTests.id, legacy.id));
@@ -130609,7 +130537,6 @@ class TimebackAssessmentsService {
         assessmentKey: input.sinkIdentifier,
         purpose: "review",
         status: "live",
-        sortOrder: null,
         standardFramework: null,
         standardIdentifier: null,
         diagnosticKey: null,
@@ -130624,7 +130551,6 @@ class TimebackAssessmentsService {
       qtiTestIdentifier: input.sinkIdentifier,
       purpose: "review",
       status: "live",
-      sortOrder: null,
       standardFramework: null,
       standardIdentifier: null,
       diagnosticKey: null,
@@ -130827,7 +130753,6 @@ class TimebackAssessmentsService {
       qtiTestIdentifier: row.qtiTestIdentifier,
       purpose: row.purpose,
       status: row.status,
-      sortOrder: row.sortOrder,
       standard: assessmentStandardForRow(row),
       diagnostic: diagnosticDefinitionForRow(row),
       createdAt: row.createdAt,
@@ -193561,7 +193486,6 @@ var createAssessment;
 var attachExistingAssessments;
 var updateAssessment;
 var synchronizeReviewBank;
-var reorderAssessments;
 var reorderQuestions;
 var removeAssessment;
 var listQuestions;
@@ -194277,16 +194201,6 @@ var init_timeback_controller = __esm(() => {
     const integrationId = await ctx.services.timebackAssessments.resolveIntegrationId(gameId, courseId, ctx.user);
     return ctx.services.timebackAssessments.synchronizeReviewBank(integrationId);
   });
-  reorderAssessments = requireDeveloper(async (ctx) => {
-    const { gameId, courseId } = ctx.params;
-    if (!gameId || !courseId) {
-      throw ApiError.badRequest("Missing gameId or courseId parameter");
-    }
-    const body2 = await parseRequestBody(ctx.request, ReorderAssessmentsRequestSchema);
-    const integrationId = await ctx.services.timebackAssessments.resolveIntegrationId(gameId, courseId, ctx.user);
-    await ctx.services.timebackAssessments.reorderAssessments(integrationId, body2.purpose, body2.testIdentifiers);
-    return { success: true };
-  });
   reorderQuestions = requireDeveloper(async (ctx) => {
     const { gameId, courseId, testIdentifier } = ctx.params;
     if (!gameId || !courseId || !testIdentifier) {
@@ -194427,7 +194341,6 @@ var init_timeback_controller = __esm(() => {
     attachExistingAssessments,
     updateAssessment,
     synchronizeReviewBank,
-    reorderAssessments,
     removeAssessment,
     reorderQuestions,
     listQuestions,

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@playcademy/vite-plugin",
-    "version": "1.3.1-beta.2",
+    "version": "1.3.1-beta.3",
     "type": "module",
     "exports": {
         ".": {
@@ -19,14 +19,14 @@
     "dependencies": {
         "archiver": "^7.0.1",
         "picocolors": "^1.1.1",
-        "playcademy": "0.29.1-beta.2"
+        "playcademy": "0.29.1-beta.3"
     },
     "devDependencies": {
         "@electric-sql/pglite": "^0.3.16",
         "@inquirer/prompts": "^7.8.6",
         "@playcademy/constants": "0.0.1",
-        "@playcademy/sandbox": "0.8.1-beta.2",
-        "@playcademy/sdk": "0.17.1-beta.2",
+        "@playcademy/sandbox": "0.8.1-beta.3",
+        "@playcademy/sdk": "0.17.1-beta.3",
         "@playcademy/types": "0.0.1",
         "@playcademy/utils": "0.0.1",
         "@types/archiver": "^6.0.3",