Feat: cleanup

src/views/render.rs

@@ -1,3 +1,5 @@
+//! Top-level render dispatch — composes header, page views, modals, and effects each frame.
+
 use std::collections::HashSet;
 use std::time::Duration;
 
@@ -9,7 +11,7 @@ use ratatui::Frame;
 
 use crate::app::App;
 use crate::engine::{LinkState, SequencerSnapshot};
-use crate::model::SourceSpan;
+use crate::model::{ExecutionTrace, SourceSpan};
 use crate::page::Page;
 use crate::state::{
     EditorTarget, EuclideanField, FlashKind, Modal, PanelFocus, PatternField, RenameTarget,
@@ -62,6 +64,44 @@ pub fn adjust_resolved_for_line(
         .collect()
 }
 
+pub fn highlight_script_lines(
+    script: &str,
+    trace: Option<&ExecutionTrace>,
+    user_words: &HashSet<String>,
+    max_lines: usize,
+) -> Vec<Line<'static>> {
+    let resolved_display: Vec<(SourceSpan, String)> = trace
+        .map(|t| {
+            t.resolved
+                .iter()
+                .map(|(s, v)| (*s, v.display()))
+                .collect()
+        })
+        .unwrap_or_default();
+
+    let mut line_start = 0usize;
+    script
+        .lines()
+        .take(max_lines)
+        .map(|line_str| {
+            let tokens = if let Some(t) = trace {
+                let exec = adjust_spans_for_line(&t.executed_spans, line_start, line_str.len());
+                let sel = adjust_spans_for_line(&t.selected_spans, line_start, line_str.len());
+                let res = adjust_resolved_for_line(&resolved_display, line_start, line_str.len());
+                highlight_line_with_runtime(line_str, &exec, &sel, &res, user_words)
+            } else {
+                highlight_line_with_runtime(line_str, &[], &[], &[], user_words)
+            };
+            line_start += line_str.len() + 1;
+            let spans: Vec<Span> = tokens
+                .into_iter()
+                .map(|(style, text, _)| Span::styled(text, style))
+                .collect();
+            Line::from(spans)
+        })
+        .collect()
+}
+
 pub fn horizontal_padding(width: u16) -> u16 {
     if width >= 120 {
         4
@@ -522,7 +562,6 @@ fn render_modal(
     term: Rect,
 ) -> Option<Rect> {
     let theme = theme::get();
-    let user_words: HashSet<String> = app.dict.lock().keys().cloned().collect();
     let inner = match &app.ui.modal {
         Modal::None => return None,
         Modal::Confirm { action, selected } => {
@@ -598,8 +637,14 @@ fn render_modal(
                 .height(18)
                 .render_centered(frame, term)
         }
-        Modal::Preview => render_modal_preview(frame, app, snapshot, &user_words, term),
-        Modal::Editor => render_modal_editor(frame, app, snapshot, &user_words, term),
+        Modal::Preview => {
+            let user_words: HashSet<String> = app.dict.lock().keys().cloned().collect();
+            render_modal_preview(frame, app, snapshot, &user_words, term)
+        }
+        Modal::Editor => {
+            let user_words: HashSet<String> = app.dict.lock().keys().cloned().collect();
+            render_modal_editor(frame, app, snapshot, &user_words, term)
+        }
         Modal::PatternProps {
             bank,
             pattern,
@@ -859,34 +904,8 @@ fn render_modal_preview(
             None
         };
 
-        let resolved_display: Vec<(SourceSpan, String)> = trace
-            .map(|t| t.resolved.iter().map(|(s, v)| (*s, v.display())).collect())
-            .unwrap_or_default();
-
-        let mut line_start = 0usize;
-        let lines: Vec<Line> = script
-            .lines()
-            .map(|line_str| {
-                let tokens = if let Some(t) = trace {
-                    let exec = adjust_spans_for_line(&t.executed_spans, line_start, line_str.len());
-                    let sel = adjust_spans_for_line(&t.selected_spans, line_start, line_str.len());
-                    let res =
-                        adjust_resolved_for_line(&resolved_display, line_start, line_str.len());
-                    highlight_line_with_runtime(line_str, &exec, &sel, &res, user_words)
-                } else {
-                    highlight_line_with_runtime(line_str, &[], &[], &[], user_words)
-                };
-                line_start += line_str.len() + 1;
-                let spans: Vec<Span> = tokens
-                    .into_iter()
-                    .map(|(style, text, _)| Span::styled(text, style))
-                    .collect();
-                Line::from(spans)
-            })
-            .collect();
-
-        let paragraph = Paragraph::new(lines);
-        frame.render_widget(paragraph, inner);
+        let lines = highlight_script_lines(script, trace, user_words, usize::MAX);
+        frame.render_widget(Paragraph::new(lines), inner);
     }
 
     inner