remove unnecessary guarding code (#2160)

* refactor(report): require data-group-id on dump elements, remove backward compat

Drop `buildPlaywrightTaskFromElement` and the ungrouped fallback path.
All `<script type="midscene_web_dump">` elements must now have a
`data-group-id` attribute; missing it throws an error at parse time.

https://claude.ai/code/session_017ir8y83LbmEhoMgshK3zzH

* refactor(report): remove unreachable fallback and try-catch in grouped dump parsing

The `elements` array from groupMap is guaranteed non-empty, so `baseDump`
is always assigned after the loop — remove the dead else branch. Also
remove the try-catch that silently swallowed parse errors with an empty
GroupedActionDump; errors should propagate to the caller.

https://claude.ai/code/session_017ir8y83LbmEhoMgshK3zzH

---------

Co-authored-by: Claude <noreply@anthropic.com>

Author: ottomao

apps/report/src/App.tsx

@@ -371,53 +371,30 @@ export function App() {
       result.push({
         get: () => {
           if (!isParsed) {
-            try {
-              console.time('parse_grouped_dump');
-              const allExecutions: any[] = [];
-              let baseDump: GroupedActionDump | null = null;
-
-              for (const el of elements) {
-                const content = antiEscapeScriptTag(el.textContent || '');
-                const parsed = JSON.parse(content);
-                const restored = restoreImageReferences(
-                  parsed,
-                  resolveImageFromDom,
-                );
-                const dump = GroupedActionDump.fromJSON(restored);
-                if (!baseDump) {
-                  baseDump = dump;
-                }
-                allExecutions.push(...dump.executions);
+            console.time('parse_grouped_dump');
+            const allExecutions: any[] = [];
+            let baseDump: GroupedActionDump | null = null;
+
+            for (const el of elements) {
+              const content = antiEscapeScriptTag(el.textContent || '');
+              const parsed = JSON.parse(content);
+              const restored = restoreImageReferences(
+                parsed,
+                resolveImageFromDom,
+              );
+              const dump = GroupedActionDump.fromJSON(restored);
+              if (!baseDump) {
+                baseDump = dump;
               }
+              allExecutions.push(...dump.executions);
+            }
 
-              if (baseDump) {
-                baseDump.executions = allExecutions;
-                cachedJsonContent = baseDump;
-              } else {
-                cachedJsonContent = new GroupedActionDump({
-                  sdkVersion: '',
-                  groupName: '',
-                  modelBriefs: [],
-                  executions: [],
-                });
-              }
+            baseDump!.executions = allExecutions;
+            cachedJsonContent = baseDump!;
 
-              console.timeEnd('parse_grouped_dump');
-              (cachedJsonContent as any).attributes = attributes;
-              isParsed = true;
-            } catch (e) {
-              console.error('failed to parse grouped dump content', e);
-              cachedJsonContent = new GroupedActionDump({
-                sdkVersion: '',
-                groupName: '',
-                modelBriefs: [],
-                executions: [],
-              });
-              (cachedJsonContent as any).attributes = attributes;
-              (cachedJsonContent as any).error =
-                'Failed to parse grouped JSON content';
-              isParsed = true;
-            }
+            console.timeEnd('parse_grouped_dump');
+            (cachedJsonContent as any).attributes = attributes;
+            isParsed = true;
           }
           return cachedJsonContent!;
         },