From 9380b50c2635fb7b5c31db386cb779e64d7d1438 Mon Sep 17 00:00:00 2001
From: Azalea Colburn <62953415+azaleacolburn@users.noreply.github.com>
Date: Tue, 19 Nov 2024 16:27:03 -0800
Subject: [PATCH] Timeline Modification Verification (#11)

* feat(wip): started action input state machine class

* style: format files

* feat: basic timeline warning system

* style: format files

* fix: move action arrows

* feat: user cannot close invalid timeline

---------

Co-authored-by: azaleacolburn <azaleacolburn@users.noreply.github.com>
---
 src/lib/ActionInputStateMachine.svelte.ts | 32 +++++++++++++++++++
 src/lib/components/Action.svelte          |  3 ++
 src/lib/components/Timeline.svelte        | 29 ++++++++++++-----
 src/lib/types.ts                          | 38 ++++++++++++-----------
 src/routes/scout/ActionInputs.svelte      |  6 ++--
 5 files changed, 80 insertions(+), 28 deletions(-)
 create mode 100644 src/lib/ActionInputStateMachine.svelte.ts

diff --git a/src/lib/ActionInputStateMachine.svelte.ts b/src/lib/ActionInputStateMachine.svelte.ts
new file mode 100644
index 0000000..ef2ebd6
--- /dev/null
+++ b/src/lib/ActionInputStateMachine.svelte.ts
@@ -0,0 +1,32 @@
+import type { AutoActionData } from '$lib/types';
+
+export class ActionInputVerifier {
+	private held_bunnies: number = 0;
+	private held_balloons: number = 0;
+	private held_totes: number = 0;
+
+	public verify_actions(action_data: AutoActionData[]) {
+		action_data
+			.reverse()
+			.forEach((action_data) => (action_data.ok = this.verify_new_action(action_data)));
+	}
+
+	// Takes an action and returns if it's a legal one
+	verify_new_action(action_data: AutoActionData): boolean {
+		const success = action_data.success;
+		const action = action_data.action;
+		if (success) {
+			if (action.includes('IntakeBalloon')) this.held_balloons++;
+			else if (action.includes('IntakeBunny')) this.held_bunnies++;
+			else if (action.includes('IntakeTote')) this.held_totes++;
+			else if (action.includes('EjectBalloon')) this.held_balloons--;
+			else if (action.includes('EjectBunny')) this.held_bunnies--;
+			else if (action.includes('EjectTote')) this.held_totes--;
+		}
+		if (action.includes('ScoreBalloon')) this.held_balloons--;
+		else if (action.includes('ScoreBunny')) this.held_bunnies--;
+
+		if (action.includes('Intake')) return true;
+		return this.held_balloons >= 0 && this.held_bunnies >= 0 && this.held_totes >= 0;
+	}
+}
diff --git a/src/lib/components/Action.svelte b/src/lib/components/Action.svelte
index 103f0fb..cd2caf7 100644
--- a/src/lib/components/Action.svelte
+++ b/src/lib/components/Action.svelte
@@ -26,6 +26,9 @@
 >
 	<span class="w-auto shrink text-clip">{action_data.action}</span>
 	<div class="flex shrink-0 flex-row content-center justify-end gap-4">
+		{#if !action_data.ok}
+			<span class="text-yellow-400">Warning</span>
+		{/if}
 		<button
 			class="group-first:pointer-events-none group-first:opacity-30"
 			onclick={() => shift(index, -1)}
diff --git a/src/lib/components/Timeline.svelte b/src/lib/components/Timeline.svelte
index 769a74d..16ef430 100644
--- a/src/lib/components/Timeline.svelte
+++ b/src/lib/components/Timeline.svelte
@@ -1,4 +1,5 @@
 <script lang="ts">
+	import { ActionInputVerifier } from '$lib/ActionInputStateMachine.svelte';
 	import type { AutoActionData } from '$lib/types';
 	import Action from './Action.svelte';
 
@@ -6,23 +7,32 @@
 		actions = $bindable(),
 		displaying = $bindable()
 	}: { actions: AutoActionData[]; displaying: boolean } = $props();
-	// let latestActions: AutoActionData[] = $derived(actions.toReversed().slice(0, 5));
 
-	const remove = (index: number) => {
+	/// Determine if currying is the right solution or if we should use a binding
+	function remove(index: number) {
 		actions.splice(index, 1);
-	};
+		verify();
+	}
 
-	const shift = (index: number, change: number) => {
+	function shift(index: number, change: number) {
 		let item = actions[index];
 		actions.splice(index, 1);
 		actions.splice(index + change, 0, item);
-	};
+		verify();
+	}
+
+	function verify() {
+		new ActionInputVerifier().verify_actions(actions);
+	}
+	const is_valid_timeline = $derived(actions.filter((action) => !action.ok).length === 0);
 </script>
 
 <button
 	class="fixed inset-0 transition-all {displaying ? 'backdrop-blur' : 'translate-y-full'}"
 	onclick={(e: Event) => {
-		if (e.target === e.currentTarget) displaying = false;
+		if (e.target === e.currentTarget && is_valid_timeline) {
+			displaying = false;
+		}
 	}}
 >
 	<div
@@ -30,7 +40,12 @@
 		id="timeline"
 	>
 		{#each actions as _, i}
-			<Action action_data={actions[i]} index={i} {remove} {shift} />
+			<Action
+				action_data={actions[actions.length - i - 1]}
+				index={actions.length - i - 1}
+				{remove}
+				{shift}
+			/>
 		{/each}
 		{#if actions.length === 0}
 			<h3 class="m-auto">No actions yet :3</h3>
diff --git a/src/lib/types.ts b/src/lib/types.ts
index 74be898..bba036e 100644
--- a/src/lib/types.ts
+++ b/src/lib/types.ts
@@ -12,20 +12,6 @@ export type User = {
 };
 
 /// Counts
-export type AutoActionsTM = ActionsTM & {
-	bunny_intake_success: number;
-	bunny_intake_failure: number;
-	bunny_tote_success: number;
-	bunny_tote_failure: number;
-	bunny_low_success: number;
-	bunny_low_failure: number;
-	actions: AutoActionData[];
-};
-
-export type TeleActionsTM = ActionsTM & {
-	actions: TeleActionData[];
-};
-
 export type ActionsTM = {
 	id: number;
 	tote_intake_success: number;
@@ -44,6 +30,20 @@ export type ActionsTM = {
 	score_other_robot_failure: number;
 };
 
+export type AutoActionsTM = ActionsTM & {
+	bunny_intake_success: number;
+	bunny_intake_failure: number;
+	bunny_tote_success: number;
+	bunny_tote_failure: number;
+	bunny_low_success: number;
+	bunny_low_failure: number;
+	actions: AutoActionData[];
+};
+
+export type TeleActionsTM = ActionsTM & {
+	actions: TeleActionData[];
+};
+
 export type TeamMatch = {
 	id: number;
 	scout_id: string;
@@ -57,14 +57,16 @@ export type TeamMatch = {
 	tele_actions: TeleActionData[];
 };
 
-export type AutoActionData = {
-	action: AutoAction;
+export type TeleActionData = {
+	action: TeleAction;
 	success: boolean;
+	ok: boolean;
 };
 
-export type TeleActionData = {
-	action: TeleAction;
+export type AutoActionData = {
+	action: AutoAction;
 	success: boolean;
+	ok: boolean;
 };
 
 // Action Types
diff --git a/src/routes/scout/ActionInputs.svelte b/src/routes/scout/ActionInputs.svelte
index 407d974..1f3150e 100644
--- a/src/routes/scout/ActionInputs.svelte
+++ b/src/routes/scout/ActionInputs.svelte
@@ -1,7 +1,6 @@
 <script lang="ts">
 	import SuccessFail from '$lib/components/SuccessFail.svelte';
 	import type { AutoAction, AutoInputState, AutoActionData } from '$lib/types';
-	import { derived } from 'svelte/store';
 
 	let {
 		actions = $bindable(),
@@ -48,9 +47,10 @@
 
 		const action: AutoActionData = {
 			action: actionState as AutoAction,
-			success: success
+			success: success,
+			ok: true
 		};
-		actions.unshift(action); // TODO: Make sure unshift works
+		actions.push(action);
 		actionState = 'None';
 	}