chore(cleanup): remove unused imports from session extractions (v0.5.343)

cargo fix --bin perry --lib -p perry-hir -p perry-codegen --allow-dirty
--release narrows `use` statements across 25 files. No behavior changes.

Mostly imports added during the v0.5.329-v0.5.342 split work that
turned out unused once the moved bodies inlined. Example:

- expr_object.rs: anyhow!, FuncId, ArrayElement, BinaryOp,
  is_destructuring_pattern, infer_type_from_expr
- lower_call/native.rs: Function, is_abort_controller_expr,
  lower_abort_controller_call, nanbox_string_inline
- compile/resolve.rs: find_file_dep_in_package_json, find_node_modules,
  has_perry_native_module, is_ts_file, resolve_exports,
  resolve_package_entry, resolve_package_source_entry,
  resolve_with_extensions (still callable via super:: re-exports —
  just not used inside the resolve module itself).

Workspace warning count: 321 → 255 (-66, -21%).

Remaining 255 warnings are pre-existing categorical issues (63
unnecessary-unsafe-block in perry-runtime, 25 unreachable-pattern in
the HIR/codegen lowerers, 23 unnecessary-transmute, etc.) — separate
bugs not introduced by this session.

Verified:
- cargo build --release clean
- cargo test --workspace 434/0/5 = baseline
- gap tests 25/28; doc-tests 80/82 = baseline.

Final session tally: 15 commits (v0.5.329→v0.5.343), all baselines
green throughout, ~14,000 LOC reorganized into 19 focused sub-modules,
the most-cited monolith (lower::lower_expr) shrunk by 91%, compile.rs
by 60%, lower.rs by 44%, lower_call.rs by 33%.

Author: proggeramlug

CLAUDE.md

@@ -111,7 +111,7 @@ opt-level = "s"       # Optimize for size in stdlib
 opt-level = 3
 
 [workspace.package]
-version = "0.5.342"
+version = "0.5.343"
 edition = "2021"
 license = "MIT"
 repository = "https://github.com/PerryTS/perry"

Cargo.lock

@@ -130,7 +130,7 @@ fn collect_nested_closure_boxed_vars_in_stmts(
     stmts: &[perry_hir::Stmt],
     out: &mut HashSet<u32>,
 ) {
-    use perry_hir::Stmt;
+    
     for s in stmts {
         collect_nested_closure_boxed_vars_in_stmt(s, out);
     }

Cargo.toml

@@ -308,7 +308,7 @@ fn collect_closures_in_expr(
     // Helper closure that recurses into a sub-expression. We use a
     // local closure rather than a method so we can keep the same
     // recursion entry point.
-    let mut walk = |sub: &Expr,
+    let walk = |sub: &Expr,
                     seen: &mut HashSet<perry_types::FuncId>,
                     out: &mut Vec<(perry_types::FuncId, Expr)>| {
         collect_closures_in_expr(sub, seen, out);
@@ -831,7 +831,7 @@ pub(crate) fn collect_ref_ids_in_stmts(stmts: &[perry_hir::Stmt], out: &mut Hash
 
 fn collect_ref_ids_in_expr(e: &perry_hir::Expr, out: &mut HashSet<u32>) {
     use perry_hir::{ArrayElement, CallArg, Expr};
-    let mut walk = |sub: &Expr, out: &mut HashSet<u32>| {
+    let walk = |sub: &Expr, out: &mut HashSet<u32>| {
         collect_ref_ids_in_expr(sub, out);
     };
     match e {
@@ -2036,7 +2036,7 @@ fn collect_localset_ids_in_expr_filtered(
     clamp_fn_ids: &HashSet<u32>,
 ) {
     use perry_hir::{ArrayElement, CallArg, Expr};
-    let mut walk = |sub: &Expr, out: &mut HashSet<u32>| {
+    let walk = |sub: &Expr, out: &mut HashSet<u32>| {
         collect_localset_ids_in_expr_filtered(sub, out, filter, flat_const_ids, flat_row_alias_ids, clamp_fn_ids);
     };
     match e {
@@ -2632,7 +2632,7 @@ pub(crate) fn collect_non_escaping_news(
     module_globals: &std::collections::HashMap<u32, String>,
     classes: &std::collections::HashMap<String, &perry_hir::Class>,
 ) -> std::collections::HashMap<u32, String> {
-    use perry_hir::{Expr, Stmt};
+    
 
     // Pass 1: find candidates — Let bindings of New that aren't boxed/global.
     let mut candidates: std::collections::HashMap<u32, String> = std::collections::HashMap::new();
@@ -3396,7 +3396,7 @@ fn check_escapes_in_expr(
 
 /// Helper: does this expression contain `LocalGet(target_id)` anywhere?
 fn expr_contains_local_get(e: &perry_hir::Expr, target_id: u32) -> bool {
-    use perry_hir::{ArrayElement, CallArg, Expr};
+    use perry_hir::Expr;
     match e {
         Expr::LocalGet(id) => *id == target_id,
         Expr::Binary { left, right, .. }

crates/perry-codegen/src/boxed_vars.rs

@@ -7975,7 +7975,7 @@ pub(crate) fn lower_expr(ctx: &mut FnCtx<'_>, expr: &Expr) -> Result<String> {
 
         Expr::ChildProcessSpawnBackground { command, args, log_file, env_json } => {
             let cmd_box = lower_expr(ctx, command)?;
-            let args_box = if let Some(a) = args {
+            let _args_box = if let Some(a) = args {
                 lower_expr(ctx, a)?
             } else {
                 double_literal(0.0)

crates/perry-codegen/src/collectors.rs

@@ -16,7 +16,7 @@
 use anyhow::Result;
 use perry_hir::Expr;
 
-use crate::expr::{lower_expr, nanbox_pointer_inline, nanbox_string_inline, unbox_to_i64, FnCtx};
+use crate::expr::{lower_expr, nanbox_pointer_inline, unbox_to_i64, FnCtx};
 use crate::nanbox::double_literal;
 use crate::types::{DOUBLE, I32, I64};
 

crates/perry-codegen/src/expr.rs

@@ -17,16 +17,14 @@
 use anyhow::{bail, Result};
 use perry_dispatch::{ArgKind as UiArgKind, ReturnKind as UiReturnKind};
 use perry_hir::Expr;
-use perry_types::Type as HirType;
 
-use crate::expr::{lower_expr, nanbox_pointer_inline, nanbox_string_inline, unbox_to_i64, FnCtx};
+use crate::expr::{lower_expr, nanbox_pointer_inline, unbox_to_i64, FnCtx};
 use crate::nanbox::{double_literal, POINTER_MASK_I64};
-use crate::types::{DOUBLE, I32, I64};
+use crate::types::{DOUBLE, I64};
 
 use super::{
     apply_inline_style, collect_closure_introduced_ids, extract_options_fields,
-    find_outer_writes_stmt, get_raw_string_ptr, is_abort_controller_expr,
-    lower_abort_controller_call, lower_fetch_native_method, lower_native_module_dispatch,
+    find_outer_writes_stmt, get_raw_string_ptr, lower_fetch_native_method, lower_native_module_dispatch,
     lower_notification_schedule, lower_perry_ui_table_call, native_module_lookup,
     perry_i18n_table_lookup, perry_plugin_instance_method_lookup, perry_plugin_table_lookup,
     perry_system_table_lookup, perry_ui_instance_method_lookup, perry_ui_table_lookup,

crates/perry-codegen/src/lower_call/builtin.rs

@@ -385,7 +385,7 @@ pub(crate) fn collect_assigned_locals_expr(expr: &Expr, assigned: &mut Vec<Local
             assigned.push(*array_id); // These may reallocate the array
             collect_assigned_locals_expr(value, assigned);
         }
-        Expr::ArrayPop(array_id) | Expr::ArrayShift(array_id) => {
+        Expr::ArrayPop(_array_id) | Expr::ArrayShift(_array_id) => {
             // These modify the array but don't reallocate
         }
         Expr::ArrayIndexOf { array, value } | Expr::ArrayIncludes { array, value } => {

crates/perry-codegen/src/lower_call/native.rs

@@ -4055,7 +4055,7 @@ fn lower_stmt(
                         ctxt: Default::default(),
                     };
                     let mutable = false;
-                    let is_var = false;
+                    let _is_var = false;
                     for decl in &fake_var.decls {
                         if let Some(init) = &decl.init {
                             match &decl.name {
@@ -6112,7 +6112,7 @@ fn try_lower_widget_decl(
     let mut placeholder: Option<Vec<(String, WidgetPlaceholderValue)>> = None;
     let mut family_param_name: Option<String> = None;
     let mut app_group: Option<String> = None;
-    let mut reload_after_seconds: Option<u32> = None;
+    let reload_after_seconds: Option<u32> = None;
 
     for prop in &config_obj.props {
         let kv = match prop {