@playcademy/sandbox 0.10.1-beta.4 → 0.10.1-beta.5

package/dist/cli.js

@@ -1128,7 +1128,7 @@ var package_default;
 var init_package = __esm(() => {
   package_default = {
     name: "@playcademy/sandbox",
-    version: "0.10.1-beta.4",
+    version: "0.10.1-beta.5",
     description: "Local development server for Playcademy game development",
     type: "module",
     exports: {
@@ -47989,7 +47989,7 @@ function rejectSystemManagedReview(input, context2) {
     });
   }
 }
-var TimebackGradeSchema, TimebackSubjectSchema, CourseGoalsSchema, UpdateGameTimebackIntegrationRequestSchema, CreateGameTimebackIntegrationRequestSchema, TimebackActivityDataSchema, EndActivityRequestSchema, GameRunMetricsSchema, GameCourseMetricsSchema, GameMetricsResponseSchema, AdvanceCourseRequestSchema, UnenrollCourseRequestSchema, HeartbeatRequestSchema, PopulateStudentRequestSchema, DerivedPlatformCourseConfigSchema, TimebackBaseConfigSchema, PlatformTimebackSetupRequestSchema, AdminTimebackMutationBaseSchema, AdminAttributionDateSchema, ADMIN_GRANT_XP_MIN = -1e5, ADMIN_GRANT_XP_MAX = 1e5, ADMIN_GRANT_XP_AMOUNT_RANGE_MESSAGE, GrantTimebackXpRequestSchema, AdjustTimebackTimeRequestSchema, AdjustTimebackMasteryRequestSchema, ReconcileMasteryForConfigChangeSchema, EnrollStudentRequestSchema, UnenrollStudentRequestSchema, ReactivateEnrollmentRequestSchema, VerifyTimebackMetricDiscrepancyRequestSchema, AssessmentPurposeSchema, AssessmentStatusSchema, AssessmentKeySchema, DiagnosticKeySchema, DiagnosticRoutingManifestSchema, DiagnosticAssessmentDefinitionInputSchema, AssessmentStandardRefSchema3, CreateAssessmentRequestSchema, UpdateAssessmentRequestSchema, CopyAssessmentRequestSchema, AssessmentAssociationImportEntrySchema, AttachExistingAssessmentsRequestSchema, ReorderQuestionsRequestSchema;
+var TimebackGradeSchema, TimebackSubjectSchema, CourseGoalsSchema, UpdateGameTimebackIntegrationRequestSchema, CreateGameTimebackIntegrationRequestSchema, TimebackActivityDataSchema, EndActivityRequestSchema, GameRunMetricsSchema, GameCourseMetricsSchema, GameMetricsResponseSchema, AdvanceCourseRequestSchema, UnenrollCourseRequestSchema, HeartbeatRequestSchema, PopulateStudentRequestSchema, DerivedPlatformCourseConfigSchema, TimebackBaseConfigSchema, PlatformTimebackSetupRequestSchema, AdminTimebackMutationBaseSchema, AdminAttributionDateSchema, ADMIN_GRANT_XP_MIN = -1e5, ADMIN_GRANT_XP_MAX = 1e5, ADMIN_GRANT_XP_AMOUNT_RANGE_MESSAGE, GrantTimebackXpRequestSchema, AdjustTimebackTimeRequestSchema, AdjustTimebackMasteryRequestSchema, ReconcileMasteryForConfigChangeSchema, EnrollStudentRequestSchema, UnenrollStudentRequestSchema, ReactivateEnrollmentRequestSchema, VerifyTimebackMetricDiscrepancyRequestSchema, AssessmentPurposeSchema, AssessmentStatusSchema, AssessmentKeySchema, DiagnosticKeySchema, DiagnosticRoutingManifestSchema, DiagnosticAssessmentDefinitionInputSchema, AssessmentStandardRefSchema3, CreateAssessmentRequestSchema, UpdateAssessmentRequestSchema, CopyAssessmentRequestSchema, ReviseAssessmentRequestSchema, AssessmentAssociationImportEntrySchema, AttachExistingAssessmentsRequestSchema, ReorderQuestionsRequestSchema;
 var init_schemas4 = __esm(() => {
   init_esm();
   init_src();
@@ -48272,6 +48272,11 @@ var init_schemas4 = __esm(() => {
     requireMasteryStandard(input, context2);
     rejectSystemManagedReview(input, context2);
   });
+  ReviseAssessmentRequestSchema = exports_external.object({
+    currentQtiTestIdentifier: exports_external.string().trim().min(1, "The current assessment identifier is required"),
+    title: exports_external.string().trim().min(1, "Assessment title is required"),
+    questions: exports_external.array(exports_external.record(exports_external.string(), exports_external.unknown())).min(1, "A revision must contain at least one question").max(500, "A revision cannot contain more than 500 questions")
+  });
   AssessmentAssociationImportEntrySchema = exports_external.object({
     assessmentKey: AssessmentKeySchema,
     qtiTestIdentifier: exports_external.string().trim().min(1, "Assessment identifier is required"),
@@ -106222,6 +106227,69 @@ var init_timeback_assessment_import_util = __esm(() => {
   init_timeback_qti_authoring_util();
 });
 
+// ../api-core/src/utils/timeback-assessment-revision.util.ts
+function assertRevisableAssessment(row, reviewBankIdentifier) {
+  if (row.purpose === "review" || row.qtiTestIdentifier === reviewBankIdentifier) {
+    throw new ValidationError("The review bank is system-managed. Revise its mastery sources instead.");
+  }
+  if (row.purpose === "diagnostic") {
+    throw new ValidationError("Diagnostics are revised by importing a new revision and archiving the current one.");
+  }
+  if (row.status !== "live") {
+    throw new ValidationError(row.status === "draft" ? "Draft assessments are edited directly. Only live assessments can be revised in place." : "Archived assessments cannot be revised. Restore it to live first.");
+  }
+  if (!row.assessmentKey) {
+    throw new ValidationError("This unmanaged assessment cannot be revised. Republish it with an explicit assessment key.");
+  }
+}
+function assertRevisableTestStructure(current) {
+  const parts2 = qtiTestParts(current);
+  const sectionCount = parts2.reduce((count, part) => count + part["qti-assessment-section"].length, 0);
+  if (parts2.length !== 1 || sectionCount !== 1) {
+    throw new ValidationError(`Assessment ${current.identifier} has ${parts2.length} test part(s) and ${sectionCount} section(s). Revisions can only replace content in single-section assessments.`);
+  }
+}
+function buildRevisionSourceTestInput(input) {
+  assertRevisableTestStructure(input.current);
+  const { "qti-outcome-declaration": outcomeDeclarations, ...structure } = buildQtiTestStructureInput(input.current, input.sourceIdentifier);
+  return {
+    identifier: input.sourceIdentifier,
+    title: input.title,
+    ...qtiTestOptionalAttributes(input.current),
+    metadata: {
+      ...sourceMetadata(input.current.metadata),
+      ownerSystem: PLAYCADEMY_QTI_OWNER_SYSTEM,
+      ownerGameSlug: input.gameSlug,
+      assessmentKey: input.assessmentKey,
+      integrationId: input.integrationId,
+      subject: input.subject,
+      grade: String(input.grade),
+      [PLAYCADEMY_REVIEW_BANK_MANIFEST_METADATA_KEY]: emptyReviewBankManifest(input.sourceIdentifier)
+    },
+    ...outcomeDeclarations && outcomeDeclarations.length > 0 ? { "qti-outcome-declaration": outcomeDeclarations } : {},
+    "qti-test-part": structure["qti-test-part"].map((part) => ({
+      ...part,
+      "qti-assessment-section": part["qti-assessment-section"].map((section) => ({
+        ...section,
+        "qti-assessment-item-ref": []
+      }))
+    }))
+  };
+}
+function revisionConflict(requestedIdentifier, currentIdentifier) {
+  return new ConflictError("This assessment was revised by someone else. Reload to see the current version before revising again.", {
+    requestedQtiTestIdentifier: requestedIdentifier,
+    currentQtiTestIdentifier: currentIdentifier
+  });
+}
+var init_timeback_assessment_revision_util = __esm(() => {
+  init_assessment_runtime2();
+  init_timeback3();
+  init_errors2();
+  init_timeback_assessment_publication_util();
+  init_timeback_qti_authoring_util();
+});
+
 // ../api-core/src/utils/timeback-qti-library.util.ts
 function newPlaycademyTestIdentifier() {
   return `${PLAYCADEMY_QTI_SOURCE_PREFIX}${crypto.randomUUID()}`;
@@ -106647,6 +106715,109 @@ class TimebackAssessmentsService {
       throw error88;
     }
   }
+  async reviseAssessment(integrationId, input) {
+    const client2 = this.requireClient();
+    const reviewBankIdentifier = reviewBankSinkIdentifier(integrationId);
+    const current = await this.requireCurrentRevisionRow(integrationId, input.currentQtiTestIdentifier);
+    assertRevisableAssessment(current, reviewBankIdentifier);
+    const ownership = await this.requireQtiTestOwnershipContext(integrationId);
+    const { integration } = ownership;
+    const creations = input.questions.map((question) => {
+      const creation = parseQtiQuestionCreationInput(question);
+      if (creation.kind === "authoring") {
+        assertSupportedQtiQuestionInteraction(creation.input);
+      }
+      return creation;
+    });
+    assertAssessmentHasQuestions(creations);
+    const currentTest = await client2.qtiApi.assessmentTests.get(current.qtiTestIdentifier);
+    const sourceIdentifier = input.sourceTestIdentifier;
+    const sourceInput = buildRevisionSourceTestInput({
+      sourceIdentifier,
+      title: input.title,
+      current: currentTest,
+      assessmentKey: current.assessmentKey,
+      integrationId,
+      gameSlug: ownership.gameSlug,
+      subject: integration.subject,
+      grade: integration.grade
+    });
+    const createdItemIdentifiers = [];
+    let sourceCreateAttempted = false;
+    try {
+      sourceCreateAttempted = true;
+      await client2.qtiApi.assessmentTests.create(sourceInput);
+      const sourceTest = await client2.qtiApi.assessmentTests.get(sourceIdentifier);
+      for (const creation of creations) {
+        const created = await this.createQuestionInOwnedTest({
+          client: client2,
+          integrationId,
+          qtiTestIdentifier: sourceIdentifier,
+          test: sourceTest,
+          gameSlug: ownership.gameSlug,
+          creation
+        });
+        createdItemIdentifiers.push(created.reference.identifier);
+      }
+      const loaded = await loadHydratedQtiTest(client2, sourceIdentifier);
+      await this.validateAssessmentHasQuestions(loaded);
+      const publication = await this.publishManagedAssessment(current, current.purpose, null, loaded);
+      if (!publication.artifactsAvailable) {
+        throw new ValidationError("The revised assessment could not be compiled for the runtime. The current version stays live; try again.");
+      }
+      const result = await this.deps.db.transaction(async (tx) => {
+        const rows = await tx.select().from(gameTimebackAssessmentTests).where(and(eq(gameTimebackAssessmentTests.integrationId, integrationId), eq(gameTimebackAssessmentTests.assessmentKey, current.assessmentKey))).for("update");
+        const row = rows[0];
+        if (!row) {
+          throw new NotFoundError(`Assessment not found: ${input.currentQtiTestIdentifier}`);
+        }
+        if (row.qtiTestIdentifier !== input.currentQtiTestIdentifier) {
+          throw revisionConflict(input.currentQtiTestIdentifier, row.qtiTestIdentifier);
+        }
+        assertRevisableAssessment(row, reviewBankIdentifier);
+        const previousQtiTestIdentifier = row.qtiTestIdentifier;
+        const unchanged = publication.qtiTestIdentifier === previousQtiTestIdentifier;
+        if (unchanged) {
+          setAttribute("app.assessment.operation", "revise_unchanged");
+          return { row, previousQtiTestIdentifier, unchanged };
+        }
+        const [updatedRow] = await tx.update(gameTimebackAssessmentTests).set({
+          qtiTestIdentifier: publication.qtiTestIdentifier,
+          updatedAt: new Date
+        }).where(eq(gameTimebackAssessmentTests.id, row.id)).returning();
+        setAttribute("app.assessment.operation", "revise");
+        addEvent("assessment.revised", {
+          "app.assessment.assessment_key": row.assessmentKey,
+          "app.assessment.previous_qti_test_identifier": previousQtiTestIdentifier,
+          "app.assessment.qti_test_identifier": publication.qtiTestIdentifier
+        });
+        return { row: updatedRow ?? row, previousQtiTestIdentifier, unchanged };
+      });
+      let warning;
+      if (!result.unchanged && result.row.purpose === "mastery") {
+        try {
+          const sync = await this.synchronizeReviewBank(integrationId);
+          if (sync.warnings.length > 0) {
+            warning = `Review bank: ${sync.warnings.map((entry) => entry.message).join(" ")}`;
+          }
+        } catch (error88) {
+          addEvent("assessment.revision_review_bank_sync_failed", {
+            "app.assessment.qti_test_identifier": result.row.qtiTestIdentifier,
+            "exception.type": errorType(error88),
+            "app.error.message": errorMessage(error88)
+          });
+          warning = `The review bank could not be synchronized: ${errorMessage(error88)}. Use Sync Review Bank to retry.`;
+        }
+      }
+      return {
+        ...this.associationSummary(result.row, warning),
+        previousQtiTestIdentifier: result.previousQtiTestIdentifier,
+        unchanged: result.unchanged
+      };
+    } finally {
+      await this.cleanupQtiAssessmentCopy(client2, sourceCreateAttempted ? sourceIdentifier : undefined, createdItemIdentifiers);
+    }
+  }
   async removeAssessment(integrationId, qtiTestIdentifier) {
     return this.withLockedAssessmentAssociations(integrationId, qtiTestIdentifier, async (row, tx) => {
       if (qtiTestIdentifier === reviewBankSinkIdentifier(integrationId)) {
@@ -106919,106 +107090,118 @@ class TimebackAssessmentsService {
       assertAllAssessmentAssociationsDraft(associations);
       const ownership = await this.requireOwnedDraftAssessment(integrationId, qtiTestIdentifier, row, tx);
       const creation = parseQtiQuestionCreationInput(input);
-      const itemIdentifier = newPlaycademyQuestionIdentifier(qtiTestIdentifier);
-      let title;
-      let metadata2;
-      let xml;
-      let structuredInput;
-      let fallbackItem;
-      if (creation.kind === "copy") {
-        const source = await client2.qtiApi.assessmentItems.get(creation.sourceItemIdentifier);
-        const copy = buildQtiQuestionCopyInput({
-          source,
-          targetIdentifier: itemIdentifier,
-          targetTestIdentifier: qtiTestIdentifier,
-          gameSlug: ownership.gameSlug,
-          integrationId
-        });
-        title = copy.title;
-        metadata2 = copy.metadata;
-        xml = copy.xml;
-        fallbackItem = {
-          ...source,
-          identifier: itemIdentifier,
-          title,
-          rawXml: xml,
-          metadata: metadata2
-        };
-      } else {
-        structuredInput = creation.input;
-        assertSupportedQtiQuestionInteraction(structuredInput);
-        title = typeof structuredInput.title === "string" ? structuredInput.title.trim() : "";
-        metadata2 = buildOwnedQtiQuestionMetadata(structuredInput, {
-          gameSlug: ownership.gameSlug,
-          testIdentifier: qtiTestIdentifier
-        });
-        xml = buildQuestionXml(structuredInput, itemIdentifier, title);
-        fallbackItem = {
-          ...structuredInput,
-          identifier: itemIdentifier,
-          title,
-          ...xml ? { rawXml: xml } : {}
-        };
-      }
-      const section = await this.qtiSectionItems(client2, qtiTestIdentifier, undefined, undefined, ownership.test);
-      const href = qtiItemHref(client2, itemIdentifier);
-      let item;
-      let itemCreationAttempted = false;
-      let referenceCreationAttempted = false;
-      try {
-        itemCreationAttempted = true;
-        item = xml ? await client2.course.createAssessmentItemXml({ xml, metadata: metadata2 }) : await client2.qtiApi.assessmentItems.create({
-          ...structuredInput,
-          identifier: itemIdentifier,
-          metadata: metadata2
-        });
-        referenceCreationAttempted = true;
-        await section.items.add({
-          identifier: itemIdentifier,
-          href
-        });
-      } catch (error88) {
-        let referenceRemoved = !referenceCreationAttempted;
-        const creationStatus = isApiError(error88) ? error88.statusCode : undefined;
-        const itemCreationDefinitelyRejected = !referenceCreationAttempted && typeof creationStatus === "number" && creationStatus >= 400 && creationStatus < 500 && creationStatus !== 408;
-        if (referenceCreationAttempted) {
-          try {
-            await section.items.remove(itemIdentifier);
-            referenceRemoved = true;
-          } catch (cleanupError) {
-            referenceRemoved = isApiError(cleanupError) && cleanupError.statusCode === 404;
-            addEvent("assessment.qti_question_reference_cleanup_failed", {
-              "app.assessment.qti_test_identifier": qtiTestIdentifier,
-              "app.assessment.qti_item_identifier": itemIdentifier,
-              "exception.type": errorType(cleanupError),
-              "app.error.message": errorMessage(cleanupError)
-            });
-          }
-        }
-        if (itemCreationAttempted && !itemCreationDefinitelyRejected && referenceRemoved) {
-          try {
-            await client2.qtiApi.assessmentItems.delete(itemIdentifier);
-          } catch (cleanupError) {
-            addEvent("assessment.qti_question_cleanup_failed", {
-              "app.assessment.qti_test_identifier": qtiTestIdentifier,
-              "app.assessment.qti_item_identifier": itemIdentifier,
-              "exception.type": errorType(cleanupError),
-              "app.error.message": errorMessage(cleanupError)
-            });
-          }
-        }
-        throw error88;
-      }
+      const created = await this.createQuestionInOwnedTest({
+        client: client2,
+        integrationId,
+        qtiTestIdentifier,
+        test: ownership.test,
+        gameSlug: ownership.gameSlug,
+        creation
+      });
       setAttribute("app.assessment.operation", creation.kind === "copy" ? "copy_question" : "create_question");
-      return buildCreatedQtiQuestionReference({
-        item,
-        fallbackItem,
-        itemIdentifier,
-        href,
-        partIdentifier: section.partIdentifier,
-        sectionIdentifier: section.sectionIdentifier,
+      return created;
+    });
+  }
+  async createQuestionInOwnedTest(input) {
+    const { client: client2, integrationId, qtiTestIdentifier, creation } = input;
+    const itemIdentifier = newPlaycademyQuestionIdentifier(qtiTestIdentifier);
+    let title;
+    let metadata2;
+    let xml;
+    let structuredInput;
+    let fallbackItem;
+    if (creation.kind === "copy") {
+      const source = await client2.qtiApi.assessmentItems.get(creation.sourceItemIdentifier);
+      const copy = buildQtiQuestionCopyInput({
+        source,
+        targetIdentifier: itemIdentifier,
+        targetTestIdentifier: qtiTestIdentifier,
+        gameSlug: input.gameSlug,
+        integrationId
+      });
+      title = copy.title;
+      metadata2 = copy.metadata;
+      xml = copy.xml;
+      fallbackItem = {
+        ...source,
+        identifier: itemIdentifier,
+        title,
+        rawXml: xml,
+        metadata: metadata2
+      };
+    } else {
+      structuredInput = creation.input;
+      assertSupportedQtiQuestionInteraction(structuredInput);
+      title = typeof structuredInput.title === "string" ? structuredInput.title.trim() : "";
+      metadata2 = buildOwnedQtiQuestionMetadata(structuredInput, {
+        gameSlug: input.gameSlug,
+        testIdentifier: qtiTestIdentifier
+      });
+      xml = buildQuestionXml(structuredInput, itemIdentifier, title);
+      fallbackItem = {
+        ...structuredInput,
+        identifier: itemIdentifier,
+        title,
+        ...xml ? { rawXml: xml } : {}
+      };
+    }
+    const section = await this.qtiSectionItems(client2, qtiTestIdentifier, undefined, undefined, input.test);
+    const href = qtiItemHref(client2, itemIdentifier);
+    let item;
+    let itemCreationAttempted = false;
+    let referenceCreationAttempted = false;
+    try {
+      itemCreationAttempted = true;
+      item = xml ? await client2.course.createAssessmentItemXml({ xml, metadata: metadata2 }) : await client2.qtiApi.assessmentItems.create({
+        ...structuredInput,
+        identifier: itemIdentifier,
         metadata: metadata2
       });
+      referenceCreationAttempted = true;
+      await section.items.add({
+        identifier: itemIdentifier,
+        href
+      });
+    } catch (error88) {
+      let referenceRemoved = !referenceCreationAttempted;
+      const creationStatus = isApiError(error88) ? error88.statusCode : undefined;
+      const itemCreationDefinitelyRejected = !referenceCreationAttempted && typeof creationStatus === "number" && creationStatus >= 400 && creationStatus < 500 && creationStatus !== 408;
+      if (referenceCreationAttempted) {
+        try {
+          await section.items.remove(itemIdentifier);
+          referenceRemoved = true;
+        } catch (cleanupError) {
+          referenceRemoved = isApiError(cleanupError) && cleanupError.statusCode === 404;
+          addEvent("assessment.qti_question_reference_cleanup_failed", {
+            "app.assessment.qti_test_identifier": qtiTestIdentifier,
+            "app.assessment.qti_item_identifier": itemIdentifier,
+            "exception.type": errorType(cleanupError),
+            "app.error.message": errorMessage(cleanupError)
+          });
+        }
+      }
+      if (itemCreationAttempted && !itemCreationDefinitelyRejected && referenceRemoved) {
+        try {
+          await client2.qtiApi.assessmentItems.delete(itemIdentifier);
+        } catch (cleanupError) {
+          addEvent("assessment.qti_question_cleanup_failed", {
+            "app.assessment.qti_test_identifier": qtiTestIdentifier,
+            "app.assessment.qti_item_identifier": itemIdentifier,
+            "exception.type": errorType(cleanupError),
+            "app.error.message": errorMessage(cleanupError)
+          });
+        }
+      }
+      throw error88;
+    }
+    return buildCreatedQtiQuestionReference({
+      item,
+      fallbackItem,
+      itemIdentifier,
+      href,
+      partIdentifier: section.partIdentifier,
+      sectionIdentifier: section.sectionIdentifier,
+      metadata: metadata2
     });
   }
   async updateQuestion(integrationId, qtiTestIdentifier, itemIdentifier, input) {
@@ -107266,6 +107449,22 @@ class TimebackAssessmentsService {
     }
     return row;
   }
+  async requireCurrentRevisionRow(integrationId, qtiTestIdentifier) {
+    const row = await this.deps.db.query.gameTimebackAssessmentTests.findFirst({
+      where: and(eq(gameTimebackAssessmentTests.integrationId, integrationId), eq(gameTimebackAssessmentTests.qtiTestIdentifier, qtiTestIdentifier))
+    });
+    if (row) {
+      return row;
+    }
+    const assessmentKey = assessmentKeyFromManagedQtiIdentifier(qtiTestIdentifier);
+    const successor = assessmentKey ? await this.deps.db.query.gameTimebackAssessmentTests.findFirst({
+      where: and(eq(gameTimebackAssessmentTests.integrationId, integrationId), eq(gameTimebackAssessmentTests.assessmentKey, assessmentKey))
+    }) : undefined;
+    if (successor) {
+      throw revisionConflict(qtiTestIdentifier, successor.qtiTestIdentifier);
+    }
+    throw new NotFoundError(`Assessment not found: ${qtiTestIdentifier}`);
+  }
   async requireDraftAssessmentRow(integrationId, qtiTestIdentifier, lockedRow) {
     const row = lockedRow ?? await this.requireAssessmentRow(integrationId, qtiTestIdentifier);
     assertDraftAssessment(row);
@@ -107413,9 +107612,21 @@ class TimebackAssessmentsService {
       itemHref: (identifier) => qtiItemHref(client2, identifier)
     });
     const artifactSource = await this.ensureManagedQtiPublication(client2, plan);
-    const artifactsAvailable = await this.storeManagedArtifacts(artifactSource, managedAssessmentDiagnosticRoutingManifest(artifactSource.test, plan.assessmentKey));
+    const artifactsAvailable = await this.managedArtifactsStored(plan.qtiTestIdentifier) || await this.storeManagedArtifacts(artifactSource, managedAssessmentDiagnosticRoutingManifest(artifactSource.test, plan.assessmentKey));
     const warning = assessmentArtifactPublicationWarning(artifactsAvailable);
-    return { ...plan, ...warning ? { warning } : {} };
+    return { ...plan, artifactsAvailable, ...warning ? { warning } : {} };
+  }
+  async managedArtifactsStored(qtiTestIdentifier) {
+    try {
+      return await this.deps.assessmentArtifacts.loadManagedPlayable(qtiTestIdentifier) !== null;
+    } catch (error88) {
+      addEvent("assessment.artifact_lookup_failed", {
+        "app.assessment.qti_test_identifier": qtiTestIdentifier,
+        "exception.type": errorType(error88),
+        "app.error.message": errorMessage(error88)
+      });
+      return false;
+    }
   }
   async cleanupQtiAssessmentCopy(client2, testIdentifier, itemIdentifiers) {
     let testDeleted = !testIdentifier;
@@ -107542,6 +107753,7 @@ var init_timeback_assessments_service = __esm(async () => {
   init_timeback_assessment_diagnostic_util();
   init_timeback_assessment_import_util();
   init_timeback_assessment_publication_util();
+  init_timeback_assessment_revision_util();
   init_timeback_assessment_rules_util();
   init_timeback_qti_authoring_util();
   init_timeback_qti_hydration_util();
@@ -167551,7 +167763,7 @@ function parseQtiLibraryParams(searchParams) {
     limit
   };
 }
-var populateStudent, getUser, getUserEnrollments, getUserById, setupIntegration, getIntegrations, getRemovedIntegrations, createIntegration, updateIntegration, deactivateCourse, reactivateCourse, getIntegrationConfig, verifyIntegration, getConfig, deleteIntegrations, endActivity, heartbeat, advanceCourse, unenrollCourse, prepareRuntimeAssessment, startRuntimeAssessment, getRuntimeAssessment, getLatestRuntimeAssessment, resumeRuntimeAssessment, saveRuntimeAssessment, recordRuntimeAssessmentProgress, submitRuntimeAssessmentItem, submitRuntimeAssessment, finalizeRuntimeAssessment, exportRuntimeAssessmentFixture, getStudentXp, getStudentMastery, getStudentHighestGradeMastered, getRoster, getStudentOverview, getGameMetrics, getStudentActivity, getGradeLevelTestResults, getGradeLevelTestReview, getActivityDetail, listMetricDiscrepancies, verifyMetricDiscrepancy, grantXp, adjustTime, adjustMastery, reconcileMasteryForConfigChange, searchStudents, enrollStudent, unenrollStudent, reactivateEnrollment, listAssessments, createAssessment, attachExistingAssessments, updateAssessment, synchronizeReviewBank, reorderQuestions, removeAssessment, listQuestions, listQuestionLibrary, listTestLibrary, copyAssessment, createQuestion, updateQuestion, removeQuestion, timeback2;
+var populateStudent, getUser, getUserEnrollments, getUserById, setupIntegration, getIntegrations, getRemovedIntegrations, createIntegration, updateIntegration, deactivateCourse, reactivateCourse, getIntegrationConfig, verifyIntegration, getConfig, deleteIntegrations, endActivity, heartbeat, advanceCourse, unenrollCourse, prepareRuntimeAssessment, startRuntimeAssessment, getRuntimeAssessment, getLatestRuntimeAssessment, resumeRuntimeAssessment, saveRuntimeAssessment, recordRuntimeAssessmentProgress, submitRuntimeAssessmentItem, submitRuntimeAssessment, finalizeRuntimeAssessment, exportRuntimeAssessmentFixture, getStudentXp, getStudentMastery, getStudentHighestGradeMastered, getRoster, getStudentOverview, getGameMetrics, getStudentActivity, getGradeLevelTestResults, getGradeLevelTestReview, getActivityDetail, listMetricDiscrepancies, verifyMetricDiscrepancy, grantXp, adjustTime, adjustMastery, reconcileMasteryForConfigChange, searchStudents, enrollStudent, unenrollStudent, reactivateEnrollment, listAssessments, createAssessment, attachExistingAssessments, updateAssessment, reviseAssessment, synchronizeReviewBank, reorderQuestions, removeAssessment, listQuestions, listQuestionLibrary, listTestLibrary, copyAssessment, createQuestion, updateQuestion, removeQuestion, timeback2;
 var init_timeback_controller = __esm(() => {
   init_esm();
   init_src();
@@ -168296,6 +168508,20 @@ var init_timeback_controller = __esm(() => {
     const body2 = await parseRequestBody(ctx.request, UpdateAssessmentRequestSchema);
     return ctx.services.timebackAssessments.updateAssessment(integrationId, testIdentifier, body2);
   });
+  reviseAssessment = 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, ReviseAssessmentRequestSchema);
+    const integrationId = await ctx.services.timebackAssessments.resolveIntegrationId(gameId, courseId, ctx.user);
+    return ctx.services.timebackAssessments.reviseAssessment(integrationId, {
+      currentQtiTestIdentifier: body2.currentQtiTestIdentifier,
+      sourceTestIdentifier: newPlaycademyTestIdentifier(),
+      title: body2.title,
+      questions: body2.questions
+    });
+  });
   synchronizeReviewBank = requireDeveloper(async (ctx) => {
     const { gameId, courseId } = ctx.params;
     if (!gameId || !courseId) {
@@ -168446,6 +168672,7 @@ var init_timeback_controller = __esm(() => {
     createAssessment,
     attachExistingAssessments,
     updateAssessment,
+    reviseAssessment,
     synchronizeReviewBank,
     removeAssessment,
     reorderQuestions,

package/dist/server.js

@@ -1127,7 +1127,7 @@ var package_default;
 var init_package = __esm(() => {
   package_default = {
     name: "@playcademy/sandbox",
-    version: "0.10.1-beta.4",
+    version: "0.10.1-beta.5",
     description: "Local development server for Playcademy game development",
     type: "module",
     exports: {
@@ -47988,7 +47988,7 @@ function rejectSystemManagedReview(input, context2) {
     });
   }
 }
-var TimebackGradeSchema, TimebackSubjectSchema, CourseGoalsSchema, UpdateGameTimebackIntegrationRequestSchema, CreateGameTimebackIntegrationRequestSchema, TimebackActivityDataSchema, EndActivityRequestSchema, GameRunMetricsSchema, GameCourseMetricsSchema, GameMetricsResponseSchema, AdvanceCourseRequestSchema, UnenrollCourseRequestSchema, HeartbeatRequestSchema, PopulateStudentRequestSchema, DerivedPlatformCourseConfigSchema, TimebackBaseConfigSchema, PlatformTimebackSetupRequestSchema, AdminTimebackMutationBaseSchema, AdminAttributionDateSchema, ADMIN_GRANT_XP_MIN = -1e5, ADMIN_GRANT_XP_MAX = 1e5, ADMIN_GRANT_XP_AMOUNT_RANGE_MESSAGE, GrantTimebackXpRequestSchema, AdjustTimebackTimeRequestSchema, AdjustTimebackMasteryRequestSchema, ReconcileMasteryForConfigChangeSchema, EnrollStudentRequestSchema, UnenrollStudentRequestSchema, ReactivateEnrollmentRequestSchema, VerifyTimebackMetricDiscrepancyRequestSchema, AssessmentPurposeSchema, AssessmentStatusSchema, AssessmentKeySchema, DiagnosticKeySchema, DiagnosticRoutingManifestSchema, DiagnosticAssessmentDefinitionInputSchema, AssessmentStandardRefSchema3, CreateAssessmentRequestSchema, UpdateAssessmentRequestSchema, CopyAssessmentRequestSchema, AssessmentAssociationImportEntrySchema, AttachExistingAssessmentsRequestSchema, ReorderQuestionsRequestSchema;
+var TimebackGradeSchema, TimebackSubjectSchema, CourseGoalsSchema, UpdateGameTimebackIntegrationRequestSchema, CreateGameTimebackIntegrationRequestSchema, TimebackActivityDataSchema, EndActivityRequestSchema, GameRunMetricsSchema, GameCourseMetricsSchema, GameMetricsResponseSchema, AdvanceCourseRequestSchema, UnenrollCourseRequestSchema, HeartbeatRequestSchema, PopulateStudentRequestSchema, DerivedPlatformCourseConfigSchema, TimebackBaseConfigSchema, PlatformTimebackSetupRequestSchema, AdminTimebackMutationBaseSchema, AdminAttributionDateSchema, ADMIN_GRANT_XP_MIN = -1e5, ADMIN_GRANT_XP_MAX = 1e5, ADMIN_GRANT_XP_AMOUNT_RANGE_MESSAGE, GrantTimebackXpRequestSchema, AdjustTimebackTimeRequestSchema, AdjustTimebackMasteryRequestSchema, ReconcileMasteryForConfigChangeSchema, EnrollStudentRequestSchema, UnenrollStudentRequestSchema, ReactivateEnrollmentRequestSchema, VerifyTimebackMetricDiscrepancyRequestSchema, AssessmentPurposeSchema, AssessmentStatusSchema, AssessmentKeySchema, DiagnosticKeySchema, DiagnosticRoutingManifestSchema, DiagnosticAssessmentDefinitionInputSchema, AssessmentStandardRefSchema3, CreateAssessmentRequestSchema, UpdateAssessmentRequestSchema, CopyAssessmentRequestSchema, ReviseAssessmentRequestSchema, AssessmentAssociationImportEntrySchema, AttachExistingAssessmentsRequestSchema, ReorderQuestionsRequestSchema;
 var init_schemas4 = __esm(() => {
   init_esm();
   init_src();
@@ -48271,6 +48271,11 @@ var init_schemas4 = __esm(() => {
     requireMasteryStandard(input, context2);
     rejectSystemManagedReview(input, context2);
   });
+  ReviseAssessmentRequestSchema = exports_external.object({
+    currentQtiTestIdentifier: exports_external.string().trim().min(1, "The current assessment identifier is required"),
+    title: exports_external.string().trim().min(1, "Assessment title is required"),
+    questions: exports_external.array(exports_external.record(exports_external.string(), exports_external.unknown())).min(1, "A revision must contain at least one question").max(500, "A revision cannot contain more than 500 questions")
+  });
   AssessmentAssociationImportEntrySchema = exports_external.object({
     assessmentKey: AssessmentKeySchema,
     qtiTestIdentifier: exports_external.string().trim().min(1, "Assessment identifier is required"),
@@ -106221,6 +106226,69 @@ var init_timeback_assessment_import_util = __esm(() => {
   init_timeback_qti_authoring_util();
 });
 
+// ../api-core/src/utils/timeback-assessment-revision.util.ts
+function assertRevisableAssessment(row, reviewBankIdentifier) {
+  if (row.purpose === "review" || row.qtiTestIdentifier === reviewBankIdentifier) {
+    throw new ValidationError("The review bank is system-managed. Revise its mastery sources instead.");
+  }
+  if (row.purpose === "diagnostic") {
+    throw new ValidationError("Diagnostics are revised by importing a new revision and archiving the current one.");
+  }
+  if (row.status !== "live") {
+    throw new ValidationError(row.status === "draft" ? "Draft assessments are edited directly. Only live assessments can be revised in place." : "Archived assessments cannot be revised. Restore it to live first.");
+  }
+  if (!row.assessmentKey) {
+    throw new ValidationError("This unmanaged assessment cannot be revised. Republish it with an explicit assessment key.");
+  }
+}
+function assertRevisableTestStructure(current) {
+  const parts2 = qtiTestParts(current);
+  const sectionCount = parts2.reduce((count, part) => count + part["qti-assessment-section"].length, 0);
+  if (parts2.length !== 1 || sectionCount !== 1) {
+    throw new ValidationError(`Assessment ${current.identifier} has ${parts2.length} test part(s) and ${sectionCount} section(s). Revisions can only replace content in single-section assessments.`);
+  }
+}
+function buildRevisionSourceTestInput(input) {
+  assertRevisableTestStructure(input.current);
+  const { "qti-outcome-declaration": outcomeDeclarations, ...structure } = buildQtiTestStructureInput(input.current, input.sourceIdentifier);
+  return {
+    identifier: input.sourceIdentifier,
+    title: input.title,
+    ...qtiTestOptionalAttributes(input.current),
+    metadata: {
+      ...sourceMetadata(input.current.metadata),
+      ownerSystem: PLAYCADEMY_QTI_OWNER_SYSTEM,
+      ownerGameSlug: input.gameSlug,
+      assessmentKey: input.assessmentKey,
+      integrationId: input.integrationId,
+      subject: input.subject,
+      grade: String(input.grade),
+      [PLAYCADEMY_REVIEW_BANK_MANIFEST_METADATA_KEY]: emptyReviewBankManifest(input.sourceIdentifier)
+    },
+    ...outcomeDeclarations && outcomeDeclarations.length > 0 ? { "qti-outcome-declaration": outcomeDeclarations } : {},
+    "qti-test-part": structure["qti-test-part"].map((part) => ({
+      ...part,
+      "qti-assessment-section": part["qti-assessment-section"].map((section) => ({
+        ...section,
+        "qti-assessment-item-ref": []
+      }))
+    }))
+  };
+}
+function revisionConflict(requestedIdentifier, currentIdentifier) {
+  return new ConflictError("This assessment was revised by someone else. Reload to see the current version before revising again.", {
+    requestedQtiTestIdentifier: requestedIdentifier,
+    currentQtiTestIdentifier: currentIdentifier
+  });
+}
+var init_timeback_assessment_revision_util = __esm(() => {
+  init_assessment_runtime2();
+  init_timeback3();
+  init_errors2();
+  init_timeback_assessment_publication_util();
+  init_timeback_qti_authoring_util();
+});
+
 // ../api-core/src/utils/timeback-qti-library.util.ts
 function newPlaycademyTestIdentifier() {
   return `${PLAYCADEMY_QTI_SOURCE_PREFIX}${crypto.randomUUID()}`;
@@ -106646,6 +106714,109 @@ class TimebackAssessmentsService {
       throw error88;
     }
   }
+  async reviseAssessment(integrationId, input) {
+    const client2 = this.requireClient();
+    const reviewBankIdentifier = reviewBankSinkIdentifier(integrationId);
+    const current = await this.requireCurrentRevisionRow(integrationId, input.currentQtiTestIdentifier);
+    assertRevisableAssessment(current, reviewBankIdentifier);
+    const ownership = await this.requireQtiTestOwnershipContext(integrationId);
+    const { integration } = ownership;
+    const creations = input.questions.map((question) => {
+      const creation = parseQtiQuestionCreationInput(question);
+      if (creation.kind === "authoring") {
+        assertSupportedQtiQuestionInteraction(creation.input);
+      }
+      return creation;
+    });
+    assertAssessmentHasQuestions(creations);
+    const currentTest = await client2.qtiApi.assessmentTests.get(current.qtiTestIdentifier);
+    const sourceIdentifier = input.sourceTestIdentifier;
+    const sourceInput = buildRevisionSourceTestInput({
+      sourceIdentifier,
+      title: input.title,
+      current: currentTest,
+      assessmentKey: current.assessmentKey,
+      integrationId,
+      gameSlug: ownership.gameSlug,
+      subject: integration.subject,
+      grade: integration.grade
+    });
+    const createdItemIdentifiers = [];
+    let sourceCreateAttempted = false;
+    try {
+      sourceCreateAttempted = true;
+      await client2.qtiApi.assessmentTests.create(sourceInput);
+      const sourceTest = await client2.qtiApi.assessmentTests.get(sourceIdentifier);
+      for (const creation of creations) {
+        const created = await this.createQuestionInOwnedTest({
+          client: client2,
+          integrationId,
+          qtiTestIdentifier: sourceIdentifier,
+          test: sourceTest,
+          gameSlug: ownership.gameSlug,
+          creation
+        });
+        createdItemIdentifiers.push(created.reference.identifier);
+      }
+      const loaded = await loadHydratedQtiTest(client2, sourceIdentifier);
+      await this.validateAssessmentHasQuestions(loaded);
+      const publication = await this.publishManagedAssessment(current, current.purpose, null, loaded);
+      if (!publication.artifactsAvailable) {
+        throw new ValidationError("The revised assessment could not be compiled for the runtime. The current version stays live; try again.");
+      }
+      const result = await this.deps.db.transaction(async (tx) => {
+        const rows = await tx.select().from(gameTimebackAssessmentTests).where(and(eq(gameTimebackAssessmentTests.integrationId, integrationId), eq(gameTimebackAssessmentTests.assessmentKey, current.assessmentKey))).for("update");
+        const row = rows[0];
+        if (!row) {
+          throw new NotFoundError(`Assessment not found: ${input.currentQtiTestIdentifier}`);
+        }
+        if (row.qtiTestIdentifier !== input.currentQtiTestIdentifier) {
+          throw revisionConflict(input.currentQtiTestIdentifier, row.qtiTestIdentifier);
+        }
+        assertRevisableAssessment(row, reviewBankIdentifier);
+        const previousQtiTestIdentifier = row.qtiTestIdentifier;
+        const unchanged = publication.qtiTestIdentifier === previousQtiTestIdentifier;
+        if (unchanged) {
+          setAttribute("app.assessment.operation", "revise_unchanged");
+          return { row, previousQtiTestIdentifier, unchanged };
+        }
+        const [updatedRow] = await tx.update(gameTimebackAssessmentTests).set({
+          qtiTestIdentifier: publication.qtiTestIdentifier,
+          updatedAt: new Date
+        }).where(eq(gameTimebackAssessmentTests.id, row.id)).returning();
+        setAttribute("app.assessment.operation", "revise");
+        addEvent("assessment.revised", {
+          "app.assessment.assessment_key": row.assessmentKey,
+          "app.assessment.previous_qti_test_identifier": previousQtiTestIdentifier,
+          "app.assessment.qti_test_identifier": publication.qtiTestIdentifier
+        });
+        return { row: updatedRow ?? row, previousQtiTestIdentifier, unchanged };
+      });
+      let warning;
+      if (!result.unchanged && result.row.purpose === "mastery") {
+        try {
+          const sync = await this.synchronizeReviewBank(integrationId);
+          if (sync.warnings.length > 0) {
+            warning = `Review bank: ${sync.warnings.map((entry) => entry.message).join(" ")}`;
+          }
+        } catch (error88) {
+          addEvent("assessment.revision_review_bank_sync_failed", {
+            "app.assessment.qti_test_identifier": result.row.qtiTestIdentifier,
+            "exception.type": errorType(error88),
+            "app.error.message": errorMessage(error88)
+          });
+          warning = `The review bank could not be synchronized: ${errorMessage(error88)}. Use Sync Review Bank to retry.`;
+        }
+      }
+      return {
+        ...this.associationSummary(result.row, warning),
+        previousQtiTestIdentifier: result.previousQtiTestIdentifier,
+        unchanged: result.unchanged
+      };
+    } finally {
+      await this.cleanupQtiAssessmentCopy(client2, sourceCreateAttempted ? sourceIdentifier : undefined, createdItemIdentifiers);
+    }
+  }
   async removeAssessment(integrationId, qtiTestIdentifier) {
     return this.withLockedAssessmentAssociations(integrationId, qtiTestIdentifier, async (row, tx) => {
       if (qtiTestIdentifier === reviewBankSinkIdentifier(integrationId)) {
@@ -106918,106 +107089,118 @@ class TimebackAssessmentsService {
       assertAllAssessmentAssociationsDraft(associations);
       const ownership = await this.requireOwnedDraftAssessment(integrationId, qtiTestIdentifier, row, tx);
       const creation = parseQtiQuestionCreationInput(input);
-      const itemIdentifier = newPlaycademyQuestionIdentifier(qtiTestIdentifier);
-      let title;
-      let metadata2;
-      let xml;
-      let structuredInput;
-      let fallbackItem;
-      if (creation.kind === "copy") {
-        const source = await client2.qtiApi.assessmentItems.get(creation.sourceItemIdentifier);
-        const copy = buildQtiQuestionCopyInput({
-          source,
-          targetIdentifier: itemIdentifier,
-          targetTestIdentifier: qtiTestIdentifier,
-          gameSlug: ownership.gameSlug,
-          integrationId
-        });
-        title = copy.title;
-        metadata2 = copy.metadata;
-        xml = copy.xml;
-        fallbackItem = {
-          ...source,
-          identifier: itemIdentifier,
-          title,
-          rawXml: xml,
-          metadata: metadata2
-        };
-      } else {
-        structuredInput = creation.input;
-        assertSupportedQtiQuestionInteraction(structuredInput);
-        title = typeof structuredInput.title === "string" ? structuredInput.title.trim() : "";
-        metadata2 = buildOwnedQtiQuestionMetadata(structuredInput, {
-          gameSlug: ownership.gameSlug,
-          testIdentifier: qtiTestIdentifier
-        });
-        xml = buildQuestionXml(structuredInput, itemIdentifier, title);
-        fallbackItem = {
-          ...structuredInput,
-          identifier: itemIdentifier,
-          title,
-          ...xml ? { rawXml: xml } : {}
-        };
-      }
-      const section = await this.qtiSectionItems(client2, qtiTestIdentifier, undefined, undefined, ownership.test);
-      const href = qtiItemHref(client2, itemIdentifier);
-      let item;
-      let itemCreationAttempted = false;
-      let referenceCreationAttempted = false;
-      try {
-        itemCreationAttempted = true;
-        item = xml ? await client2.course.createAssessmentItemXml({ xml, metadata: metadata2 }) : await client2.qtiApi.assessmentItems.create({
-          ...structuredInput,
-          identifier: itemIdentifier,
-          metadata: metadata2
-        });
-        referenceCreationAttempted = true;
-        await section.items.add({
-          identifier: itemIdentifier,
-          href
-        });
-      } catch (error88) {
-        let referenceRemoved = !referenceCreationAttempted;
-        const creationStatus = isApiError(error88) ? error88.statusCode : undefined;
-        const itemCreationDefinitelyRejected = !referenceCreationAttempted && typeof creationStatus === "number" && creationStatus >= 400 && creationStatus < 500 && creationStatus !== 408;
-        if (referenceCreationAttempted) {
-          try {
-            await section.items.remove(itemIdentifier);
-            referenceRemoved = true;
-          } catch (cleanupError) {
-            referenceRemoved = isApiError(cleanupError) && cleanupError.statusCode === 404;
-            addEvent("assessment.qti_question_reference_cleanup_failed", {
-              "app.assessment.qti_test_identifier": qtiTestIdentifier,
-              "app.assessment.qti_item_identifier": itemIdentifier,
-              "exception.type": errorType(cleanupError),
-              "app.error.message": errorMessage(cleanupError)
-            });
-          }
-        }
-        if (itemCreationAttempted && !itemCreationDefinitelyRejected && referenceRemoved) {
-          try {
-            await client2.qtiApi.assessmentItems.delete(itemIdentifier);
-          } catch (cleanupError) {
-            addEvent("assessment.qti_question_cleanup_failed", {
-              "app.assessment.qti_test_identifier": qtiTestIdentifier,
-              "app.assessment.qti_item_identifier": itemIdentifier,
-              "exception.type": errorType(cleanupError),
-              "app.error.message": errorMessage(cleanupError)
-            });
-          }
-        }
-        throw error88;
-      }
+      const created = await this.createQuestionInOwnedTest({
+        client: client2,
+        integrationId,
+        qtiTestIdentifier,
+        test: ownership.test,
+        gameSlug: ownership.gameSlug,
+        creation
+      });
       setAttribute("app.assessment.operation", creation.kind === "copy" ? "copy_question" : "create_question");
-      return buildCreatedQtiQuestionReference({
-        item,
-        fallbackItem,
-        itemIdentifier,
-        href,
-        partIdentifier: section.partIdentifier,
-        sectionIdentifier: section.sectionIdentifier,
+      return created;
+    });
+  }
+  async createQuestionInOwnedTest(input) {
+    const { client: client2, integrationId, qtiTestIdentifier, creation } = input;
+    const itemIdentifier = newPlaycademyQuestionIdentifier(qtiTestIdentifier);
+    let title;
+    let metadata2;
+    let xml;
+    let structuredInput;
+    let fallbackItem;
+    if (creation.kind === "copy") {
+      const source = await client2.qtiApi.assessmentItems.get(creation.sourceItemIdentifier);
+      const copy = buildQtiQuestionCopyInput({
+        source,
+        targetIdentifier: itemIdentifier,
+        targetTestIdentifier: qtiTestIdentifier,
+        gameSlug: input.gameSlug,
+        integrationId
+      });
+      title = copy.title;
+      metadata2 = copy.metadata;
+      xml = copy.xml;
+      fallbackItem = {
+        ...source,
+        identifier: itemIdentifier,
+        title,
+        rawXml: xml,
+        metadata: metadata2
+      };
+    } else {
+      structuredInput = creation.input;
+      assertSupportedQtiQuestionInteraction(structuredInput);
+      title = typeof structuredInput.title === "string" ? structuredInput.title.trim() : "";
+      metadata2 = buildOwnedQtiQuestionMetadata(structuredInput, {
+        gameSlug: input.gameSlug,
+        testIdentifier: qtiTestIdentifier
+      });
+      xml = buildQuestionXml(structuredInput, itemIdentifier, title);
+      fallbackItem = {
+        ...structuredInput,
+        identifier: itemIdentifier,
+        title,
+        ...xml ? { rawXml: xml } : {}
+      };
+    }
+    const section = await this.qtiSectionItems(client2, qtiTestIdentifier, undefined, undefined, input.test);
+    const href = qtiItemHref(client2, itemIdentifier);
+    let item;
+    let itemCreationAttempted = false;
+    let referenceCreationAttempted = false;
+    try {
+      itemCreationAttempted = true;
+      item = xml ? await client2.course.createAssessmentItemXml({ xml, metadata: metadata2 }) : await client2.qtiApi.assessmentItems.create({
+        ...structuredInput,
+        identifier: itemIdentifier,
         metadata: metadata2
       });
+      referenceCreationAttempted = true;
+      await section.items.add({
+        identifier: itemIdentifier,
+        href
+      });
+    } catch (error88) {
+      let referenceRemoved = !referenceCreationAttempted;
+      const creationStatus = isApiError(error88) ? error88.statusCode : undefined;
+      const itemCreationDefinitelyRejected = !referenceCreationAttempted && typeof creationStatus === "number" && creationStatus >= 400 && creationStatus < 500 && creationStatus !== 408;
+      if (referenceCreationAttempted) {
+        try {
+          await section.items.remove(itemIdentifier);
+          referenceRemoved = true;
+        } catch (cleanupError) {
+          referenceRemoved = isApiError(cleanupError) && cleanupError.statusCode === 404;
+          addEvent("assessment.qti_question_reference_cleanup_failed", {
+            "app.assessment.qti_test_identifier": qtiTestIdentifier,
+            "app.assessment.qti_item_identifier": itemIdentifier,
+            "exception.type": errorType(cleanupError),
+            "app.error.message": errorMessage(cleanupError)
+          });
+        }
+      }
+      if (itemCreationAttempted && !itemCreationDefinitelyRejected && referenceRemoved) {
+        try {
+          await client2.qtiApi.assessmentItems.delete(itemIdentifier);
+        } catch (cleanupError) {
+          addEvent("assessment.qti_question_cleanup_failed", {
+            "app.assessment.qti_test_identifier": qtiTestIdentifier,
+            "app.assessment.qti_item_identifier": itemIdentifier,
+            "exception.type": errorType(cleanupError),
+            "app.error.message": errorMessage(cleanupError)
+          });
+        }
+      }
+      throw error88;
+    }
+    return buildCreatedQtiQuestionReference({
+      item,
+      fallbackItem,
+      itemIdentifier,
+      href,
+      partIdentifier: section.partIdentifier,
+      sectionIdentifier: section.sectionIdentifier,
+      metadata: metadata2
     });
   }
   async updateQuestion(integrationId, qtiTestIdentifier, itemIdentifier, input) {
@@ -107265,6 +107448,22 @@ class TimebackAssessmentsService {
     }
     return row;
   }
+  async requireCurrentRevisionRow(integrationId, qtiTestIdentifier) {
+    const row = await this.deps.db.query.gameTimebackAssessmentTests.findFirst({
+      where: and(eq(gameTimebackAssessmentTests.integrationId, integrationId), eq(gameTimebackAssessmentTests.qtiTestIdentifier, qtiTestIdentifier))
+    });
+    if (row) {
+      return row;
+    }
+    const assessmentKey = assessmentKeyFromManagedQtiIdentifier(qtiTestIdentifier);
+    const successor = assessmentKey ? await this.deps.db.query.gameTimebackAssessmentTests.findFirst({
+      where: and(eq(gameTimebackAssessmentTests.integrationId, integrationId), eq(gameTimebackAssessmentTests.assessmentKey, assessmentKey))
+    }) : undefined;
+    if (successor) {
+      throw revisionConflict(qtiTestIdentifier, successor.qtiTestIdentifier);
+    }
+    throw new NotFoundError(`Assessment not found: ${qtiTestIdentifier}`);
+  }
   async requireDraftAssessmentRow(integrationId, qtiTestIdentifier, lockedRow) {
     const row = lockedRow ?? await this.requireAssessmentRow(integrationId, qtiTestIdentifier);
     assertDraftAssessment(row);
@@ -107412,9 +107611,21 @@ class TimebackAssessmentsService {
       itemHref: (identifier) => qtiItemHref(client2, identifier)
     });
     const artifactSource = await this.ensureManagedQtiPublication(client2, plan);
-    const artifactsAvailable = await this.storeManagedArtifacts(artifactSource, managedAssessmentDiagnosticRoutingManifest(artifactSource.test, plan.assessmentKey));
+    const artifactsAvailable = await this.managedArtifactsStored(plan.qtiTestIdentifier) || await this.storeManagedArtifacts(artifactSource, managedAssessmentDiagnosticRoutingManifest(artifactSource.test, plan.assessmentKey));
     const warning = assessmentArtifactPublicationWarning(artifactsAvailable);
-    return { ...plan, ...warning ? { warning } : {} };
+    return { ...plan, artifactsAvailable, ...warning ? { warning } : {} };
+  }
+  async managedArtifactsStored(qtiTestIdentifier) {
+    try {
+      return await this.deps.assessmentArtifacts.loadManagedPlayable(qtiTestIdentifier) !== null;
+    } catch (error88) {
+      addEvent("assessment.artifact_lookup_failed", {
+        "app.assessment.qti_test_identifier": qtiTestIdentifier,
+        "exception.type": errorType(error88),
+        "app.error.message": errorMessage(error88)
+      });
+      return false;
+    }
   }
   async cleanupQtiAssessmentCopy(client2, testIdentifier, itemIdentifiers) {
     let testDeleted = !testIdentifier;
@@ -107541,6 +107752,7 @@ var init_timeback_assessments_service = __esm(async () => {
   init_timeback_assessment_diagnostic_util();
   init_timeback_assessment_import_util();
   init_timeback_assessment_publication_util();
+  init_timeback_assessment_revision_util();
   init_timeback_assessment_rules_util();
   init_timeback_qti_authoring_util();
   init_timeback_qti_hydration_util();
@@ -167550,7 +167762,7 @@ function parseQtiLibraryParams(searchParams) {
     limit
   };
 }
-var populateStudent, getUser, getUserEnrollments, getUserById, setupIntegration, getIntegrations, getRemovedIntegrations, createIntegration, updateIntegration, deactivateCourse, reactivateCourse, getIntegrationConfig, verifyIntegration, getConfig, deleteIntegrations, endActivity, heartbeat, advanceCourse, unenrollCourse, prepareRuntimeAssessment, startRuntimeAssessment, getRuntimeAssessment, getLatestRuntimeAssessment, resumeRuntimeAssessment, saveRuntimeAssessment, recordRuntimeAssessmentProgress, submitRuntimeAssessmentItem, submitRuntimeAssessment, finalizeRuntimeAssessment, exportRuntimeAssessmentFixture, getStudentXp, getStudentMastery, getStudentHighestGradeMastered, getRoster, getStudentOverview, getGameMetrics, getStudentActivity, getGradeLevelTestResults, getGradeLevelTestReview, getActivityDetail, listMetricDiscrepancies, verifyMetricDiscrepancy, grantXp, adjustTime, adjustMastery, reconcileMasteryForConfigChange, searchStudents, enrollStudent, unenrollStudent, reactivateEnrollment, listAssessments, createAssessment, attachExistingAssessments, updateAssessment, synchronizeReviewBank, reorderQuestions, removeAssessment, listQuestions, listQuestionLibrary, listTestLibrary, copyAssessment, createQuestion, updateQuestion, removeQuestion, timeback2;
+var populateStudent, getUser, getUserEnrollments, getUserById, setupIntegration, getIntegrations, getRemovedIntegrations, createIntegration, updateIntegration, deactivateCourse, reactivateCourse, getIntegrationConfig, verifyIntegration, getConfig, deleteIntegrations, endActivity, heartbeat, advanceCourse, unenrollCourse, prepareRuntimeAssessment, startRuntimeAssessment, getRuntimeAssessment, getLatestRuntimeAssessment, resumeRuntimeAssessment, saveRuntimeAssessment, recordRuntimeAssessmentProgress, submitRuntimeAssessmentItem, submitRuntimeAssessment, finalizeRuntimeAssessment, exportRuntimeAssessmentFixture, getStudentXp, getStudentMastery, getStudentHighestGradeMastered, getRoster, getStudentOverview, getGameMetrics, getStudentActivity, getGradeLevelTestResults, getGradeLevelTestReview, getActivityDetail, listMetricDiscrepancies, verifyMetricDiscrepancy, grantXp, adjustTime, adjustMastery, reconcileMasteryForConfigChange, searchStudents, enrollStudent, unenrollStudent, reactivateEnrollment, listAssessments, createAssessment, attachExistingAssessments, updateAssessment, reviseAssessment, synchronizeReviewBank, reorderQuestions, removeAssessment, listQuestions, listQuestionLibrary, listTestLibrary, copyAssessment, createQuestion, updateQuestion, removeQuestion, timeback2;
 var init_timeback_controller = __esm(() => {
   init_esm();
   init_src();
@@ -168295,6 +168507,20 @@ var init_timeback_controller = __esm(() => {
     const body2 = await parseRequestBody(ctx.request, UpdateAssessmentRequestSchema);
     return ctx.services.timebackAssessments.updateAssessment(integrationId, testIdentifier, body2);
   });
+  reviseAssessment = 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, ReviseAssessmentRequestSchema);
+    const integrationId = await ctx.services.timebackAssessments.resolveIntegrationId(gameId, courseId, ctx.user);
+    return ctx.services.timebackAssessments.reviseAssessment(integrationId, {
+      currentQtiTestIdentifier: body2.currentQtiTestIdentifier,
+      sourceTestIdentifier: newPlaycademyTestIdentifier(),
+      title: body2.title,
+      questions: body2.questions
+    });
+  });
   synchronizeReviewBank = requireDeveloper(async (ctx) => {
     const { gameId, courseId } = ctx.params;
     if (!gameId || !courseId) {
@@ -168445,6 +168671,7 @@ var init_timeback_controller = __esm(() => {
     createAssessment,
     attachExistingAssessments,
     updateAssessment,
+    reviseAssessment,
     synchronizeReviewBank,
     removeAssessment,
     reorderQuestions,

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@playcademy/sandbox",
-    "version": "0.10.1-beta.4",
+    "version": "0.10.1-beta.5",
     "description": "Local development server for Playcademy game development",
     "type": "module",
     "exports": {