revert: undo unintended master changes

Author: chillenious

.github/workflows/ci.yml

@@ -26,7 +26,7 @@ jobs:
       - name: Setup Bun
         uses: oven-sh/setup-bun@v2
         with:
-          bun-version: 1.3.6
+          bun-version: 1.3.5
 
       - name: Cache Bun downloads
         uses: actions/cache@v4
@@ -59,7 +59,7 @@ jobs:
       - name: Setup Bun
         uses: oven-sh/setup-bun@v2
         with:
-          bun-version: 1.3.6
+          bun-version: 1.3.5
 
       - name: Check translation coverage
         run: |
@@ -77,7 +77,7 @@ jobs:
       - name: Setup Bun
         uses: oven-sh/setup-bun@v2
         with:
-          bun-version: 1.3.6
+          bun-version: 1.3.5
 
       - name: Scan for hardcoded strings
         run: |
@@ -99,7 +99,7 @@ jobs:
       - name: Setup Bun
         uses: oven-sh/setup-bun@v2
         with:
-          bun-version: 1.3.6
+          bun-version: 1.3.5
 
       - name: Cache Bun downloads
         uses: actions/cache@v4
@@ -131,7 +131,7 @@ jobs:
       - name: Setup Bun
         uses: oven-sh/setup-bun@v2
         with:
-          bun-version: 1.3.6
+          bun-version: 1.3.5
 
       - name: Cache Bun downloads
         uses: actions/cache@v4

.github/workflows/deploy-pages.yml

@@ -32,7 +32,7 @@ jobs:
       - name: Setup Bun
         uses: oven-sh/setup-bun@v2
         with:
-          bun-version: 1.3.6
+          bun-version: 1.3.5
 
       - name: Cache Bun downloads
         uses: actions/cache@v4

.github/workflows/release.yml

@@ -53,7 +53,7 @@ jobs:
       - name: Setup Bun
         uses: oven-sh/setup-bun@v2
         with:
-          bun-version: 1.3.6
+          bun-version: 1.3.5
 
       - name: Cache Bun downloads
         uses: actions/cache@v4

bun.lock

@@ -19,7 +19,6 @@
 	let uploadError = $state<string | null>(null);
 	let results = $state<AssessmentResults | null>(null);
 	let runKey = $state(0);
-	let showCorrectAnswers = $state(false);
 
 	type DisplayItemScore = {
 		id: string;
@@ -123,7 +122,6 @@
 
 	function handleRetake() {
 		results = null;
-		showCorrectAnswers = false;
 		runKey += 1;
 	}
 
@@ -150,13 +148,7 @@
 	{#if selectedAssessment}
 		<div class="player-section">
 			{#if results}
-				<AssessmentEndScreen
-					{results}
-					items={displayItemScores}
-					assessment={selectedAssessment}
-					bind:showCorrectAnswers
-					onRetake={handleRetake}
-				/>
+				<AssessmentEndScreen {results} items={displayItemScores} onRetake={handleRetake} />
 			{:else}
 				{#key `${selectedAssessment.id}:${runKey}`}
 					<AssessmentShell

packages/qti2-example/src/routes/assessment-demo/+page.svelte

@@ -1,13 +1,5 @@
 <script lang="ts">
 	import type { AssessmentResults } from '@pie-qti/qti2-assessment-player';
-	import { registerDefaultComponents } from '@pie-qti/qti2-default-components';
-	import { typesetMathInElement } from '@pie-qti/qti2-typeset-katex';
-	// @ts-expect-error - Svelte-check can't resolve workspace subpath exports, but runtime works correctly
-	import { ItemBody } from '@pie-qti/qti2-item-player/components';
-	import { Player } from '@pie-qti/qti2-item-player';
-	import type { SampleAssessment } from '$lib/sample-assessments';
-	import { typesetAction } from '@pie-qti/qti2-default-components/shared';
-	import { getSecurityConfig } from '$lib/player-config';
 
 	type DisplayItemScore = {
 		id: string;
@@ -20,36 +12,10 @@
 	interface Props {
 		results: AssessmentResults;
 		items: DisplayItemScore[];
-		assessment: SampleAssessment;
-		showCorrectAnswers: boolean;
 		onRetake?: () => void;
 	}
 
-	let { results, items, assessment, showCorrectAnswers = $bindable(), onRetake }: Props = $props();
-
-	const itemXmlById = $derived.by(() => {
-		const map = new Map<string, string>();
-		for (const tp of assessment.assessment.testParts || []) {
-			for (const section of tp.sections || []) {
-				for (const q of section.questionRefs || []) {
-					map.set(q.identifier, q.itemXml);
-				}
-			}
-		}
-		return map;
-	});
-
-	const itemResultsById = $derived.by(() => new Map(results.itemResults.map((r) => [r.itemIdentifier, r])));
-
-	function createScorerPlayer(itemXml: string) {
-		const p = new Player({
-			itemXml,
-			role: 'scorer',
-			security: getSecurityConfig(),
-		});
-		registerDefaultComponents(p.getComponentRegistry());
-		return p;
-	}
+	const { results, items, onRetake }: Props = $props();
 </script>
 
 <div class="card bg-base-100">
@@ -95,53 +61,6 @@
 			</div>
 		</div>
 
-		<div class="divider"></div>
-
-		<div class="form-control">
-			<label class="label cursor-pointer justify-start gap-4">
-				<input type="checkbox" class="checkbox checkbox-primary" bind:checked={showCorrectAnswers} />
-				<div>
-					<span class="label-text font-semibold">Show correct answers</span>
-					<p class="label-text-alt text-xs">
-						Displays the scorer view for each item (read-only), with correct responses and your submitted responses.
-					</p>
-				</div>
-			</label>
-		</div>
-
-		{#if showCorrectAnswers}
-			<div class="mt-4 space-y-6">
-				{#each results.itemResults as r (r.itemIdentifier)}
-					{@const itemXml = itemXmlById.get(r.itemIdentifier)}
-					{#if itemXml}
-						{@const player = createScorerPlayer(itemXml)}
-						<div class="card bg-base-200" use:typesetAction={{ typeset: (el) => typesetMathInElement(el) }}>
-							<div class="card-body">
-								<div class="flex items-baseline justify-between gap-4">
-									<h3 class="font-semibold">
-										{items.find((i) => i.id === r.itemIdentifier)?.title ?? r.itemIdentifier}
-									</h3>
-									<div class="text-sm opacity-70 font-mono">{r.score} / {r.maxScore}</div>
-								</div>
-
-								<ItemBody
-									player={player}
-									responses={r.responses}
-									disabled={true}
-									role="scorer"
-									typeset={typesetMathInElement}
-								/>
-							</div>
-						</div>
-					{:else}
-						<div class="alert alert-warning">
-							Missing item XML for <code>{r.itemIdentifier}</code>.
-						</div>
-					{/if}
-				{/each}
-			</div>
-		{/if}
-
 		<div class="card-actions justify-end mt-6">
 			<button class="btn btn-primary" type="button" onclick={() => onRetake?.()}>Retake assessment</button>
 		</div>

packages/qti2-example/src/routes/assessment-demo/components/AssessmentEndScreen.svelte

@@ -84,7 +84,6 @@
 				{player}
 				{responses}
 				{disabled}
-				{role}
 				i18n={i18n ?? undefined}
 				typeset={typesetMathInElement}
 				{onResponseChange}