Simplify opening training: train on entire opening family instead of single variation
- Add family training mode where users select an opening family (e.g. "Italian Game") and can play any variation within that family - Create variation tree data structure to efficiently track which variations match the current move sequence - FamilySelector now navigates directly to /learning/openings/family/[familyName] instead of showing a second selector for individual variations - OpeningTrainer displays matching variations, possible moves, and current line - LLM tutor explains which variation is being played and mentions alternatives - Remove the OpeningSelector step from the flow for a simpler UX The new flow: 1. User selects an opening family (e.g. "Sicilian Defense") 2. Training starts immediately with all variations loaded 3. User can play any move that exists in any variation 4. The UI shows which variations are still possible 5. The AI tutor guides the user through the repertoire
This commit is contained in:
Claude
@@ -6,7 +6,15 @@ import { Chessboard } from 'react-chessboard';
|
||||
import { OpeningMetadata } from '@/lib/openings';
|
||||
import { useOpeningTraining, useChessInstance } from '@/contexts/OpeningTrainingContext';
|
||||
import { loadSession } from '@/lib/openingTrainer/sessionManager';
|
||||
import { parseMoveSequence, getUserColor } from '@/lib/openingTrainer/gameLogic';
|
||||
import {
|
||||
parseMoveSequence,
|
||||
getUserColor,
|
||||
VariationTree,
|
||||
getAllPossibleNextMoves,
|
||||
identifyCurrentVariation,
|
||||
isMoveInVariationTree,
|
||||
describeCurrentPosition,
|
||||
} from '@/lib/openingTrainer/gameLogic';
|
||||
import { getWikipediaSummary } from '@/lib/openingTrainer/wikipediaService';
|
||||
import { WikipediaSummary as WikipediaSummaryType } from '@/types/openingTraining';
|
||||
import { extractFamilyName } from '@/lib/openingTrainer/openingFamilies';
|
||||
@@ -22,10 +30,21 @@ interface OpeningTrainerProps {
|
||||
personality: Personality;
|
||||
apiKey: string;
|
||||
language: SupportedLanguage;
|
||||
// Family training mode - allows multiple variations
|
||||
variationTree?: VariationTree;
|
||||
allVariations?: OpeningMetadata[];
|
||||
}
|
||||
|
||||
export default function OpeningTrainer({ opening, personality, apiKey, language }: OpeningTrainerProps) {
|
||||
export default function OpeningTrainer({
|
||||
opening,
|
||||
personality,
|
||||
apiKey,
|
||||
language,
|
||||
variationTree,
|
||||
allVariations,
|
||||
}: OpeningTrainerProps) {
|
||||
const router = useRouter();
|
||||
const isFamilyMode = !!variationTree && !!allVariations;
|
||||
|
||||
const {
|
||||
session,
|
||||
@@ -365,12 +384,42 @@ export default function OpeningTrainer({ opening, personality, apiKey, language
|
||||
const lastUserMove = userMoves.length > 0 ? userMoves[userMoves.length - 1] : null;
|
||||
const lastTutorMove = tutorMoves.length > 0 ? tutorMoves[tutorMoves.length - 1] : null;
|
||||
|
||||
// Family mode: compute current position info from variation tree
|
||||
const variationPositionInfo = useMemo(() => {
|
||||
if (!isFamilyMode || !variationTree) {
|
||||
return null;
|
||||
}
|
||||
const positionDesc = describeCurrentPosition(variationTree, session.moveHistory);
|
||||
const currentVariations = identifyCurrentVariation(variationTree, session.moveHistory);
|
||||
const possibleMoves = getAllPossibleNextMoves(variationTree, session.moveHistory);
|
||||
|
||||
return {
|
||||
...positionDesc,
|
||||
currentVariations,
|
||||
possibleMoves,
|
||||
isInAnyVariation: positionDesc.matchingCount > 0,
|
||||
};
|
||||
}, [isFamilyMode, variationTree, session.moveHistory]);
|
||||
|
||||
// Get all possible next moves (for display and tutor context)
|
||||
const theoreticalMoves = useMemo(() => {
|
||||
if (isFamilyMode && variationPositionInfo) {
|
||||
return variationPositionInfo.nextMoves;
|
||||
}
|
||||
// Single variation mode
|
||||
const moves = parseMoveSequence(opening.moves);
|
||||
const nextMove = moves[session.moveHistory.length];
|
||||
return nextMove ? [nextMove] : [];
|
||||
}, [isFamilyMode, variationPositionInfo, opening.moves, session.moveHistory.length]);
|
||||
|
||||
const openingPracticeMode = {
|
||||
openingName: opening.name,
|
||||
openingEco: opening.eco,
|
||||
repertoireMoves,
|
||||
currentMoveIndex: session.moveHistory.length,
|
||||
isInTheory: session.deviationMoveIndex === null,
|
||||
isInTheory: isFamilyMode
|
||||
? (variationPositionInfo?.isInAnyVariation ?? false)
|
||||
: session.deviationMoveIndex === null,
|
||||
deviationMoveIndex: session.deviationMoveIndex,
|
||||
lastUserMove: lastUserMove ? {
|
||||
from: lastUserMove.uci.substring(0, 2),
|
||||
@@ -395,11 +444,19 @@ export default function OpeningTrainer({ opening, personality, apiKey, language
|
||||
currentFeedback: currentFeedback ? {
|
||||
category: currentFeedback.classification.category,
|
||||
evaluationChange: currentFeedback.classification.evaluationChange,
|
||||
theoreticalAlternatives: currentFeedback.classification.theoreticalAlternatives
|
||||
theoreticalAlternatives: isFamilyMode ? theoreticalMoves : currentFeedback.classification.theoreticalAlternatives
|
||||
} : null,
|
||||
wikipediaSummary: wikipediaSummary?.extract || undefined,
|
||||
shouldTutorSpeak,
|
||||
onTutorMessageSent: handleTutorMessageSent,
|
||||
// Family mode specific info
|
||||
isFamilyMode,
|
||||
variationInfo: isFamilyMode && variationPositionInfo ? {
|
||||
matchingVariations: variationPositionInfo.matchingCount,
|
||||
currentVariationNames: variationPositionInfo.currentVariationNames,
|
||||
possibleMoves: variationPositionInfo.nextMoves,
|
||||
isEndOfLine: variationPositionInfo.isEndOfLine,
|
||||
} : undefined,
|
||||
};
|
||||
|
||||
return (
|
||||
@@ -572,13 +629,81 @@ export default function OpeningTrainer({ opening, personality, apiKey, language
|
||||
<span className="text-gray-600 dark:text-gray-400">Moves played:</span>
|
||||
<span className="font-medium text-gray-900 dark:text-white">{moveCount}</span>
|
||||
</div>
|
||||
{session.deviationMoveIndex !== null && (
|
||||
|
||||
{/* Family mode: show variation info */}
|
||||
{isFamilyMode && variationPositionInfo && (
|
||||
<>
|
||||
<div className="pt-2 border-t border-gray-300 dark:border-gray-600">
|
||||
<div className="flex justify-between mb-1">
|
||||
<span className="text-gray-600 dark:text-gray-400">Matching variations:</span>
|
||||
<span className="font-medium text-blue-600 dark:text-blue-400">
|
||||
{variationPositionInfo.matchingCount}
|
||||
</span>
|
||||
</div>
|
||||
|
||||
{/* Show possible moves */}
|
||||
{variationPositionInfo.nextMoves.length > 0 && (
|
||||
<div className="mt-2">
|
||||
<span className="text-gray-600 dark:text-gray-400 block mb-1">Possible moves:</span>
|
||||
<div className="flex flex-wrap gap-1">
|
||||
{variationPositionInfo.nextMoves.map((move) => (
|
||||
<span
|
||||
key={move}
|
||||
className="inline-block px-2 py-1 bg-green-100 dark:bg-green-900/30 text-green-800 dark:text-green-400 rounded text-xs font-mono"
|
||||
>
|
||||
{move}
|
||||
</span>
|
||||
))}
|
||||
</div>
|
||||
</div>
|
||||
)}
|
||||
|
||||
{/* Show current variation names (if narrowed down) */}
|
||||
{variationPositionInfo.matchingCount > 0 && variationPositionInfo.matchingCount <= 3 && (
|
||||
<div className="mt-2">
|
||||
<span className="text-gray-600 dark:text-gray-400 block mb-1">Current line:</span>
|
||||
<div className="space-y-1">
|
||||
{variationPositionInfo.currentVariationNames.slice(0, 3).map((name) => (
|
||||
<span
|
||||
key={name}
|
||||
className="block text-xs text-gray-700 dark:text-gray-300 truncate"
|
||||
title={name}
|
||||
>
|
||||
{name}
|
||||
</span>
|
||||
))}
|
||||
</div>
|
||||
</div>
|
||||
)}
|
||||
|
||||
{variationPositionInfo.isEndOfLine && (
|
||||
<div className="mt-2">
|
||||
<span className="inline-block px-2 py-1 bg-purple-100 dark:bg-purple-900/30 text-purple-800 dark:text-purple-400 rounded text-xs">
|
||||
End of repertoire line
|
||||
</span>
|
||||
</div>
|
||||
)}
|
||||
</div>
|
||||
</>
|
||||
)}
|
||||
|
||||
{/* Show off-book indicator (non-family mode or when truly off-book) */}
|
||||
{!isFamilyMode && session.deviationMoveIndex !== null && (
|
||||
<div className="pt-2 border-t border-gray-300 dark:border-gray-600">
|
||||
<span className="inline-block px-2 py-1 bg-orange-100 dark:bg-orange-900/30 text-orange-800 dark:text-orange-400 rounded text-xs">
|
||||
Off-book since move {session.deviationMoveIndex + 1}
|
||||
</span>
|
||||
</div>
|
||||
)}
|
||||
|
||||
{/* Family mode: show off-book when not in any variation */}
|
||||
{isFamilyMode && variationPositionInfo && !variationPositionInfo.isInAnyVariation && moveCount > 0 && (
|
||||
<div className="pt-2 border-t border-gray-300 dark:border-gray-600">
|
||||
<span className="inline-block px-2 py-1 bg-orange-100 dark:bg-orange-900/30 text-orange-800 dark:text-orange-400 rounded text-xs">
|
||||
Off-book - move not in any variation
|
||||
</span>
|
||||
</div>
|
||||
)}
|
||||
</div>
|
||||
</div>
|
||||
|
||||
|
||||
Reference in New Issue
Block a user