Fix 10/20 failing Maestro web E2E tests

Fix 1 (test 08): Muscle search uses word-start matching instead of
substring to prevent "hamstrings".contains("ring") false positive.

Fix 2 (tests 09, 15): Remove sig.set(Vec::new()) from reload_exercises
to preserve ExerciseCard show_instructions state across DB reloads.

Fix 3 (tests 10, 11): Add id/for to exercise name label/input so that
tapping the label focuses the input.

Fix 4 (test 04): Add dl_db_url= to launch URL so that a prior test's
localStorage exercise_db_url is reset to the default GitHub URL.

Fix 5 (tests 17-20): Add assertVisible wait in setup_past_session.yml
before openLink so that SetDbUrl has written localStorage before the
page reload caused by openLink.

Co-authored-by: gfauredev <19304085+gfauredev@users.noreply.github.com>

Author: Copilot

maestro/web/04_clean_state_credits.yml

@@ -1,4 +1,4 @@
-url: http://localhost:8080/LogOut/ # Clean State Credits
+url: http://localhost:8080/LogOut/?dl_db_url=https%3A%2F%2Fgfauredev.github.io%2Ffree-exercise-db%2F # Clean State Credits
 ---
 # User Story: Clean State Credits
 # Precondition: No custom DB URL set (launchApp clears localStorage)

maestro/web/_flows/setup_past_session.yml

@@ -3,6 +3,10 @@ url: http://localhost:8080/LogOut/?dl_db_url=http%3A%2F%2Flocalhost%3A8080%2FLog
 # Sub-flow: set up a completed past session with two exercises (ring_dip and
 # romanian_deadlift). Works standalone and as a runFlow setup step in other tests.
 - launchApp
+# Wait for the WASM app to fully initialize so that the SetDbUrl deep-link
+# handler has run and persisted exercise_db_url to localStorage before
+# openLink triggers a page reload.
+- assertVisible: "💪 LogOut"
 # Create a completed past session via deep link; exercises load from mock DB
 - openLink: "http://localhost:8080/LogOut/?dl_session=ring_dip%3A80%3A10%2Cromanian_deadlift%3A60%3A6"
 # Wait for mock exercises to load and session to be saved and shown on home

src/components/exercise_form_fields.rs

@@ -113,8 +113,9 @@ pub fn ExerciseFormFields(
 
     rsx! {
         div {
-            label { "Exercise Name *" }
+            label { r#for: "exercise-name-input", "Exercise Name *" }
             input {
+                id: "exercise-name-input",
                 r#type: "text",
                 placeholder: "Pushups",
                 value: "{name_input}",

src/services/exercise_db.rs

@@ -252,6 +252,15 @@ fn normalize_for_search(s: &str) -> String {
         .collect()
 }
 
+/// Returns true if any whitespace-separated word in the (already-lowercased)
+/// `tag` starts with `query_lower`.  Used for muscle / category / etc. tag
+/// matching so that partial-word substrings elsewhere in a word are not hit
+/// (e.g. "ring" must not match "hamst**ring**s").
+fn tag_word_prefix_matches(tag: &str, query_lower: &str) -> bool {
+    tag.split_whitespace()
+        .any(|word| word.starts_with(query_lower))
+}
+
 /// Returns true if an already-lowercased `name_lc` matches the given
 /// pre-computed search components (all lowercase / normalised).
 fn name_lc_matches(name_lc: &str, query_lower: &str, query_norm: &str, tokens: &[String]) -> bool {
@@ -324,11 +333,11 @@ pub fn search_exercises<'a>(
                 || exercise
                     .primary_muscles
                     .iter()
-                    .any(|m| m.as_ref().contains(&query_lower))
+                    .any(|m| tag_word_prefix_matches(m.as_ref(), &query_lower))
                 || exercise
                     .secondary_muscles
                     .iter()
-                    .any(|m| m.as_ref().contains(&query_lower))
+                    .any(|m| tag_word_prefix_matches(m.as_ref(), &query_lower))
                 || exercise.category.as_ref().contains(&query_lower)
                 || exercise
                     .force
@@ -814,6 +823,24 @@ mod tests {
         assert_eq!(results[0].id, "pull_up");
     }
 
+    #[test]
+    fn search_muscle_word_start_no_false_positive() {
+        // "ring" is a substring of "hamstrings" but the word "hamstrings" does
+        // not *start* with "ring".  "running" (which has Hamstrings as a primary
+        // muscle) must therefore not be returned when searching for "ring".
+        let exercises = sample_exercises();
+        let results = search_exercises(&exercises, "ring", None);
+        assert!(!results.iter().any(|e| e.id == "running"));
+    }
+
+    #[test]
+    fn search_muscle_word_start_prefix_matches() {
+        // "ham" is a word-start prefix of "hamstrings", so "running" should match.
+        let exercises = sample_exercises();
+        let results = search_exercises(&exercises, "ham", None);
+        assert!(results.iter().any(|e| e.id == "running"));
+    }
+
     #[test]
     fn exercises_json_url_uses_fork() {
         // The default JSON endpoint references the gfauredev GitHub Pages

src/services/exercise_loader.rs

@@ -39,9 +39,6 @@ pub fn use_exercises() -> Signal<Vec<Exercise>> {
 /// empty response, JSON parse) it shows an appropriate error message so the
 /// user knows the URL change did not take effect.
 pub async fn reload_exercises(mut sig: Signal<Vec<Exercise>>, mut toast: Signal<Option<String>>) {
-    // Clear immediately so the UI does not show stale data from the old URL
-    sig.set(Vec::new());
-
     #[cfg(target_arch = "wasm32")]
     {
         use crate::services::storage::idb_exercises;