soconnor/hristudio
1
0
Fork 0
mirror of https://github.com/soconnor0919/hristudio.git synced 2026-03-23 19:27:51 -04:00
Code Issues Actions Packages Projects Releases Wiki Activity

Add debug logging for branching flow

Browse Source
This commit is contained in:
Sean O'Connor
2026-03-21 20:26:55 -04:00
parent cc58593891
commit 3491bf4463
2 changed files with 15 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
src/components/trials/wizard/WizardInterface.tsx
View File

@@ -579,11 +579,14 @@ export const WizardInterface = React.memo(function WizardInterface({
}; };
const handleNextStep = (targetIndex?: number) => { const handleNextStep = (targetIndex?: number) => {
console.log(`[DEBUG] handleNextStep called: targetIndex=${targetIndex}, currentStepIndex=${currentStepIndex}`);
console.log(`[DEBUG] Steps: ${steps.map((s, i) => `${i}:${s.name}`).join(' | ')}`);
// If explicit target provided (from branching choice), use it // If explicit target provided (from branching choice), use it
if (typeof targetIndex === "number") { if (typeof targetIndex === "number") {
// Find step by index to ensure safety // Find step by index to ensure safety
if (targetIndex >= 0 && targetIndex < steps.length) { if (targetIndex >= 0 && targetIndex < steps.length) {
console.log(`[WizardInterface] Manual jump to step ${targetIndex}`); console.log(`[WizardInterface] Manual jump to step ${targetIndex} (${steps[targetIndex]?.name})`);
// Log manual jump // Log manual jump
logEventMutation.mutate({ logEventMutation.mutate({
@@ -609,6 +612,8 @@ export const WizardInterface = React.memo(function WizardInterface({
return next; return next;
}); });
return; return;
} else {
console.warn(`[DEBUG] Invalid targetIndex: ${targetIndex}, steps.length=${steps.length}`);
} }
} }
@@ -863,13 +868,21 @@ export const WizardInterface = React.memo(function WizardInterface({
if (parameters.nextStepId) { if (parameters.nextStepId) {
const nextId = String(parameters.nextStepId); const nextId = String(parameters.nextStepId);
const targetIndex = steps.findIndex((s) => s.id === nextId); const targetIndex = steps.findIndex((s) => s.id === nextId);
console.log(`[DEBUG] Branch choice: value=${parameters.value}, label=${parameters.label}`);
console.log(`[DEBUG] Target step ID: ${nextId}`);
console.log(`[DEBUG] Target index in steps array: ${targetIndex}`);
console.log(`[DEBUG] Available step IDs: ${steps.map(s => s.id).join(', ')}`);
if (targetIndex !== -1) { if (targetIndex !== -1) {
console.log( console.log(
`[WizardInterface] Choice-based jump to step ${targetIndex} (${nextId})`, `[WizardInterface] Choice-based jump to step ${targetIndex} (${nextId})`,
); );
handleNextStep(targetIndex); handleNextStep(targetIndex);
return; // Exit after jump return; // Exit after jump
} else {
console.warn(`[DEBUG] Target step not found! nextStepId=${nextId}`);
} }
} else {
console.warn(`[DEBUG] No nextStepId in parameters!`, parameters);
} }
} }

1
src/components/trials/wizard/panels/WizardActionItem.tsx
View File

@@ -535,6 +535,7 @@ export function WizardActionItem({
className="hover:border-primary hover:bg-primary/5 h-auto justify-start px-4 py-3 text-left" className="hover:border-primary hover:bg-primary/5 h-auto justify-start px-4 py-3 text-left"
onClick={(e) => { onClick={(e) => {
e.preventDefault(); e.preventDefault();
console.log(`[DEBUG WizardActionItem] Choice clicked: actionId=${action.id}, value=${value}, label=${label}, nextStepId=${nextStepId}`);
onExecute(action.id, { value, label, nextStepId }); onExecute(action.id, { value, label, nextStepId });
// Don't call onCompleted() here - the branching logic in handleWizardResponse // Don't call onCompleted() here - the branching logic in handleWizardResponse
// will handle the jump and reset completedActionsCount // will handle the jump and reset completedActionsCount