@playcademy/sandbox 0.10.1-beta.1 → 0.10.1-beta.2

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.1",
+    version: "0.10.1-beta.2",
     description: "Local development server for Playcademy game development",
     type: "module",
     exports: {
@@ -11002,6 +11002,32 @@ function scorePlatformAssessmentItem(item, responses) {
     isCorrect: verdicts.length > 0 && verdicts.every(Boolean)
   };
 }
+function reportReviewRequiredPairUnavailable(review, administeredItemIdentifiers2, report) {
+  if (!review.requireBothQuestions) {
+    return;
+  }
+  const administered = new Set(administeredItemIdentifiers2);
+  const failures = [];
+  for (const standard of review.standards) {
+    const key = assessmentStandardRefKey(standard);
+    const selected = new Set(review.selections.filter((selection) => assessmentStandardRefKey(selection.standard) === key).map((selection) => selection.itemIdentifier));
+    const administeredCount = [...selected].filter((identifier) => administered.has(identifier)).length;
+    if (selected.size < 2 || administeredCount === 1) {
+      failures.push({
+        standard,
+        reason: selected.size < 2 ? "selection_shortage" : "not_administered",
+        requiredCount: 2,
+        selectedCount: selected.size,
+        administeredCount
+      });
+    }
+  }
+  if (failures.length > 0) {
+    try {
+      report({ code: "REVIEW_REQUIRED_PAIR_UNAVAILABLE", failures });
+    } catch {}
+  }
+}
 function reviewStandardFieldsWithinLimits(input) {
   return input.framework.trim().length <= TIMEBACK_ASSESSMENT_REVIEW_REQUEST_LIMITS.standardFieldLength && input.identifier.trim().length <= TIMEBACK_ASSESSMENT_REVIEW_REQUEST_LIMITS.standardFieldLength;
 }
@@ -11054,7 +11080,11 @@ function normalizeReviewRequest(request) {
   if (standards.length === 0) {
     throw new Error("At least one review standard is required");
   }
-  return { standards, candidateItemsPerStandard };
+  return {
+    standards,
+    candidateItemsPerStandard,
+    ...request.requireBothQuestions ? { requireBothQuestions: true } : {}
+  };
 }
 function reviewRequestFingerprint(request, policy) {
   const normalized = normalizeReviewRequest(request);
@@ -11064,6 +11094,7 @@ function reviewRequestFingerprint(request, policy) {
       version: policy.version
     },
     candidateItemsPerStandard: normalized.candidateItemsPerStandard,
+    ...normalized.requireBothQuestions ? { requireBothQuestions: true } : {},
     standardKeys: normalized.standards.map(assessmentStandardRefKey)
   });
 }
@@ -11646,7 +11677,7 @@ function isRoutedDiagnosticAttemptMetadata(value, itemSubmissions, responseVersi
   });
 }
 function isReviewAttemptMetadata(value) {
-  return isRecord(value) && typeof value.requestFingerprint === "string" && typeof value.bankRevision === "string" && Array.isArray(value.standards) && value.standards.every(isAssessmentStandardRef) && Number.isInteger(value.candidateItemsPerStandard) && Array.isArray(value.selections) && value.selections.every((selection) => isRecord(selection) && isAssessmentStandardRef(selection.standard) && typeof selection.itemIdentifier === "string") && isReviewFulfillment(value.fulfillment);
+  return isRecord(value) && typeof value.requestFingerprint === "string" && typeof value.bankRevision === "string" && Array.isArray(value.standards) && value.standards.every(isAssessmentStandardRef) && Number.isInteger(value.candidateItemsPerStandard) && (value.requireBothQuestions === undefined || typeof value.requireBothQuestions === "boolean") && Array.isArray(value.selections) && value.selections.every((selection) => isRecord(selection) && isAssessmentStandardRef(selection.standard) && typeof selection.itemIdentifier === "string") && isReviewFulfillment(value.fulfillment);
 }
 function isOptionalInteger(value) {
   return value === undefined || Number.isInteger(value);
@@ -12344,6 +12375,7 @@ var init_assessment_runtime2 = __esm(() => {
     }),
     StartAssessmentBaseSchema.extend({
       purpose: exports_external.literal("review"),
+      requireBothQuestions: exports_external.boolean().optional(),
       standards: exports_external.array(AssessmentStandardRefSchema2).min(1).max(TIMEBACK_ASSESSMENT_REVIEW_REQUEST_LIMITS.standards),
       candidateItemsPerStandard: exports_external.number().int().positive().max(TIMEBACK_ASSESSMENT_REVIEW_REQUEST_LIMITS.candidateItemsPerStandard).optional()
     }),
@@ -100314,6 +100346,7 @@ function buildConventionalAssessmentSnapshot(input) {
       purpose: metadata2.purpose,
       standards: [...metadata2.review.standards],
       candidateItemsPerStandard: metadata2.review.candidateItemsPerStandard,
+      ...metadata2.review.requireBothQuestions ? { requireBothQuestions: true } : {},
       selections: [...metadata2.review.selections],
       fulfillment: metadata2.review.fulfillment
     };
@@ -102785,11 +102818,13 @@ function buildPreparedAssessmentMetadata(plan, timestamp6, testName) {
       review: {
         requestFingerprint: reviewRequestFingerprint({
           standards: plan.review.standards,
-          candidateItemsPerStandard: plan.review.candidateItemsPerStandard
+          candidateItemsPerStandard: plan.review.candidateItemsPerStandard,
+          requireBothQuestions: plan.review.requireBothQuestions
         }, DEFAULT_REVIEW_SELECTION_POLICY),
         bankRevision: plan.review.bankRevision,
         standards: plan.review.standards,
         candidateItemsPerStandard: plan.review.candidateItemsPerStandard,
+        ...plan.review.requireBothQuestions ? { requireBothQuestions: true } : {},
         selections: plan.review.selections,
         fulfillment: plan.review.fulfillment
       }
@@ -103972,6 +104007,7 @@ var init_timeback_assessment_runtime_service = __esm(async () => {
           bankRevision: catalog.bank.bankRevision,
           standards: [...normalized.standards],
           candidateItemsPerStandard: normalized.candidateItemsPerStandard,
+          ...normalized.requireBothQuestions ? { requireBothQuestions: true } : {},
           selections: [...selected.selections],
           fulfillment: selected.fulfillment
         }
@@ -104243,6 +104279,7 @@ var init_timeback_assessment_runtime_service = __esm(async () => {
             bankRevision: source.bank.bankRevision,
             standards: [...normalizedRequest.standards],
             candidateItemsPerStandard: normalizedRequest.candidateItemsPerStandard,
+            ...normalizedRequest.requireBothQuestions ? { requireBothQuestions: true } : {},
             selections: [...selected.selections],
             fulfillment: selected.fulfillment
           }
@@ -104518,6 +104555,16 @@ var init_timeback_assessment_runtime_service = __esm(async () => {
         };
       });
       if (accepted.repair) {
+        const { metadata: metadata2 } = accepted.repair.graded;
+        if (metadata2.purpose === "review") {
+          reportReviewRequiredPairUnavailable(metadata2.review, metadata2.itemSubmissions.map((item) => item.itemIdentifier), (error88) => addEvent("assessment.review_required_pair_unavailable", {
+            "app.assessment.attempt_id": params.attemptId,
+            "app.assessment.submission_id": input.submissionId,
+            "app.error.code": error88.code,
+            "app.assessment.review.required_pair_failure_count": error88.failures.length,
+            "app.assessment.review.required_pair_failures": error88.failures.map((failure) => JSON.stringify(failure))
+          }));
+        }
         this.reconcileTranscriptChildren(accepted.repair.context, accepted.repair.graded).catch((error88) => this.recordTranscriptFailure("completion_repair", params.attemptId, error88));
       }
       if (accepted.emission) {

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.1",
+    version: "0.10.1-beta.2",
     description: "Local development server for Playcademy game development",
     type: "module",
     exports: {
@@ -11001,6 +11001,32 @@ function scorePlatformAssessmentItem(item, responses) {
     isCorrect: verdicts.length > 0 && verdicts.every(Boolean)
   };
 }
+function reportReviewRequiredPairUnavailable(review, administeredItemIdentifiers2, report) {
+  if (!review.requireBothQuestions) {
+    return;
+  }
+  const administered = new Set(administeredItemIdentifiers2);
+  const failures = [];
+  for (const standard of review.standards) {
+    const key = assessmentStandardRefKey(standard);
+    const selected = new Set(review.selections.filter((selection) => assessmentStandardRefKey(selection.standard) === key).map((selection) => selection.itemIdentifier));
+    const administeredCount = [...selected].filter((identifier) => administered.has(identifier)).length;
+    if (selected.size < 2 || administeredCount === 1) {
+      failures.push({
+        standard,
+        reason: selected.size < 2 ? "selection_shortage" : "not_administered",
+        requiredCount: 2,
+        selectedCount: selected.size,
+        administeredCount
+      });
+    }
+  }
+  if (failures.length > 0) {
+    try {
+      report({ code: "REVIEW_REQUIRED_PAIR_UNAVAILABLE", failures });
+    } catch {}
+  }
+}
 function reviewStandardFieldsWithinLimits(input) {
   return input.framework.trim().length <= TIMEBACK_ASSESSMENT_REVIEW_REQUEST_LIMITS.standardFieldLength && input.identifier.trim().length <= TIMEBACK_ASSESSMENT_REVIEW_REQUEST_LIMITS.standardFieldLength;
 }
@@ -11053,7 +11079,11 @@ function normalizeReviewRequest(request) {
   if (standards.length === 0) {
     throw new Error("At least one review standard is required");
   }
-  return { standards, candidateItemsPerStandard };
+  return {
+    standards,
+    candidateItemsPerStandard,
+    ...request.requireBothQuestions ? { requireBothQuestions: true } : {}
+  };
 }
 function reviewRequestFingerprint(request, policy) {
   const normalized = normalizeReviewRequest(request);
@@ -11063,6 +11093,7 @@ function reviewRequestFingerprint(request, policy) {
       version: policy.version
     },
     candidateItemsPerStandard: normalized.candidateItemsPerStandard,
+    ...normalized.requireBothQuestions ? { requireBothQuestions: true } : {},
     standardKeys: normalized.standards.map(assessmentStandardRefKey)
   });
 }
@@ -11645,7 +11676,7 @@ function isRoutedDiagnosticAttemptMetadata(value, itemSubmissions, responseVersi
   });
 }
 function isReviewAttemptMetadata(value) {
-  return isRecord(value) && typeof value.requestFingerprint === "string" && typeof value.bankRevision === "string" && Array.isArray(value.standards) && value.standards.every(isAssessmentStandardRef) && Number.isInteger(value.candidateItemsPerStandard) && Array.isArray(value.selections) && value.selections.every((selection) => isRecord(selection) && isAssessmentStandardRef(selection.standard) && typeof selection.itemIdentifier === "string") && isReviewFulfillment(value.fulfillment);
+  return isRecord(value) && typeof value.requestFingerprint === "string" && typeof value.bankRevision === "string" && Array.isArray(value.standards) && value.standards.every(isAssessmentStandardRef) && Number.isInteger(value.candidateItemsPerStandard) && (value.requireBothQuestions === undefined || typeof value.requireBothQuestions === "boolean") && Array.isArray(value.selections) && value.selections.every((selection) => isRecord(selection) && isAssessmentStandardRef(selection.standard) && typeof selection.itemIdentifier === "string") && isReviewFulfillment(value.fulfillment);
 }
 function isOptionalInteger(value) {
   return value === undefined || Number.isInteger(value);
@@ -12343,6 +12374,7 @@ var init_assessment_runtime2 = __esm(() => {
     }),
     StartAssessmentBaseSchema.extend({
       purpose: exports_external.literal("review"),
+      requireBothQuestions: exports_external.boolean().optional(),
       standards: exports_external.array(AssessmentStandardRefSchema2).min(1).max(TIMEBACK_ASSESSMENT_REVIEW_REQUEST_LIMITS.standards),
       candidateItemsPerStandard: exports_external.number().int().positive().max(TIMEBACK_ASSESSMENT_REVIEW_REQUEST_LIMITS.candidateItemsPerStandard).optional()
     }),
@@ -100313,6 +100345,7 @@ function buildConventionalAssessmentSnapshot(input) {
       purpose: metadata2.purpose,
       standards: [...metadata2.review.standards],
       candidateItemsPerStandard: metadata2.review.candidateItemsPerStandard,
+      ...metadata2.review.requireBothQuestions ? { requireBothQuestions: true } : {},
       selections: [...metadata2.review.selections],
       fulfillment: metadata2.review.fulfillment
     };
@@ -102784,11 +102817,13 @@ function buildPreparedAssessmentMetadata(plan, timestamp6, testName) {
       review: {
         requestFingerprint: reviewRequestFingerprint({
           standards: plan.review.standards,
-          candidateItemsPerStandard: plan.review.candidateItemsPerStandard
+          candidateItemsPerStandard: plan.review.candidateItemsPerStandard,
+          requireBothQuestions: plan.review.requireBothQuestions
         }, DEFAULT_REVIEW_SELECTION_POLICY),
         bankRevision: plan.review.bankRevision,
         standards: plan.review.standards,
         candidateItemsPerStandard: plan.review.candidateItemsPerStandard,
+        ...plan.review.requireBothQuestions ? { requireBothQuestions: true } : {},
         selections: plan.review.selections,
         fulfillment: plan.review.fulfillment
       }
@@ -103971,6 +104006,7 @@ var init_timeback_assessment_runtime_service = __esm(async () => {
           bankRevision: catalog.bank.bankRevision,
           standards: [...normalized.standards],
           candidateItemsPerStandard: normalized.candidateItemsPerStandard,
+          ...normalized.requireBothQuestions ? { requireBothQuestions: true } : {},
           selections: [...selected.selections],
           fulfillment: selected.fulfillment
         }
@@ -104242,6 +104278,7 @@ var init_timeback_assessment_runtime_service = __esm(async () => {
             bankRevision: source.bank.bankRevision,
             standards: [...normalizedRequest.standards],
             candidateItemsPerStandard: normalizedRequest.candidateItemsPerStandard,
+            ...normalizedRequest.requireBothQuestions ? { requireBothQuestions: true } : {},
             selections: [...selected.selections],
             fulfillment: selected.fulfillment
           }
@@ -104517,6 +104554,16 @@ var init_timeback_assessment_runtime_service = __esm(async () => {
         };
       });
       if (accepted.repair) {
+        const { metadata: metadata2 } = accepted.repair.graded;
+        if (metadata2.purpose === "review") {
+          reportReviewRequiredPairUnavailable(metadata2.review, metadata2.itemSubmissions.map((item) => item.itemIdentifier), (error88) => addEvent("assessment.review_required_pair_unavailable", {
+            "app.assessment.attempt_id": params.attemptId,
+            "app.assessment.submission_id": input.submissionId,
+            "app.error.code": error88.code,
+            "app.assessment.review.required_pair_failure_count": error88.failures.length,
+            "app.assessment.review.required_pair_failures": error88.failures.map((failure) => JSON.stringify(failure))
+          }));
+        }
         this.reconcileTranscriptChildren(accepted.repair.context, accepted.repair.graded).catch((error88) => this.recordTranscriptFailure("completion_repair", params.attemptId, error88));
       }
       if (accepted.emission) {

package/package.json

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