this so the app runs on router reboot

.github/copilot-instructions.md

@@ -1,5 +1,8 @@
-you can use block code as so:
+you should use block code to refence code, like so:
 ```rust
 hello_world!(println);
+    // indented lines
 ```
-in the chat.
+in the chat. Else it fucks up formatting.
+
+We work towards clean, maintainable design. im not the best at this thing.

Cargo.lock

@@ -1179,7 +1179,7 @@ checksum = "ba73ea9cf16a25df0c8caa16c51acb937d5712a8429db78a3ee29d5dcacd3a65"
 
 [[package]]
 name = "wakey"
-version = "0.1.3"
+version = "0.1.4"
 dependencies = [
  "axum",
  "axum-extra",

Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "wakey"
-version = "0.1.3"
+version = "0.1.4"
 edition = "2024"
 publish = ["gitea"]
 

scripts/init/openwrt/update_wakey

@@ -2,12 +2,12 @@
 START=66
 
 start() {
-  LOGFILE=/var/log/update_wakey.log
+    LOGFILE=/var/log/update_wakey.log
-  MAX_RETRIES=10
+    MAX_RETRIES=10
-  RETRY_COUNT=0
+    RETRY_COUNT=0
 
-  # shellcheck disable=SC2016
+    # shellcheck disable=SC2016
-  nohup sh -c '
+    sh -c '
   while [ "$RETRY_COUNT" -lt "$MAX_RETRIES" ]; do
     if /etc/ldlda_help/update_wakey.sh; then
       echo "[update_wakey] Success at $(date)" >>"$LOGFILE"

src/route/status.rs

@@ -1,6 +1,6 @@
 use crate::{
     arpparse::NUDState,
-    utils::parse::{de_many, serialize_macs},
+    utils::{parse::{de_many, serialize_macs}, query::get_macs},
 };
 use axum::{Json, http::StatusCode, response::IntoResponse};
 use axum_extra::extract::Query;
@@ -93,9 +93,10 @@ pub async fn get_status_json(
         mac: mac.clone(),
     };
     let mut tasks = Vec::new();
+    // can we jus collect the whole table instead of doing this.
     for d in &dev_opts {
         for n in &nud_opts {
-            tasks.push(crate::utils::query::get_macs(
+            tasks.push(get_macs( // this i hate sm
                 name.as_deref(),
                 ips_opt.as_deref(),
                 d.as_deref(),
@@ -109,7 +110,7 @@ pub async fn get_status_json(
         .map(|v| v.into_iter().flatten().collect::<Vec<_>>())
     {
         Ok(mut table) => {
-            if !mac.is_empty() {
+            if !mac.is_empty() { // mac here
                 let wanted: HashSet<MacAddr> = mac.into_iter().collect();
                 table.retain(|row| row.mac.map(|m| wanted.contains(&m)).unwrap_or(false));
             }

src/route/wake.rs

@@ -89,7 +89,7 @@ pub async fn wake_multi_split(
             } else {
                 wake_one(
                     &sock,
-                    c.try_into().expect("complete struct failed to try_into"),
+                    c.try_into().expect("complete struct failed to try_into"), // type state pattern?
                 )
                 .await
                 .into()

src/utils/query/dev.rs

@@ -1,10 +1,20 @@
-use std::collections::HashSet;
+use std::{collections::HashSet, fs};
 
 pub fn get_dev() -> HashSet<String> {
     let mut devs: HashSet<String> = HashSet::new();
     if let Ok(rd) = std::fs::read_dir("/sys/class/net") {
         for e in rd.flatten() {
-            if let Ok(name) = e.file_name().into_string()
+            if e.file_type()
+                .map(|ft| {
+                    if ft.is_symlink() {
+                        // true
+                        fs::metadata(e.path()).map(|m| m.is_dir()).unwrap_or(false)
+                    } else {
+                        ft.is_dir()
+                    }
+                })
+                .unwrap_or(false)
+                && let Ok(name) = e.file_name().into_string()
                 && name != "lo"
                 && !name.is_empty()
             {

src/utils/query/macs.rs

@@ -81,7 +81,7 @@ pub async fn get_macs(
     state: Option<NUDState>,
 ) -> Result<Vec<IpNeighLine>> {
     let ip_list: Option<Vec<IpAddr>> =
-        ips.map(|slice| slice.iter().copied().map(|ip| ip.to_canonical()).collect());
+        ips.map(|slice| slice.iter().map(|ip| ip.to_canonical()).collect());
     let ip_list = match (ip_list, machine_name) {
         (Some(list), _) => list,
         (None, Some(name)) => get_ips(name).await?.collect(),
@@ -136,7 +136,9 @@ pub async fn get_mac(
     Ok(rows)
 }
 
-/* pub async fn _get_macs_good(
+/* 
+/// get macs where you just run ip neigh then rust handles the filtering (faster than get mac)
+pub async fn get_macs_rust(
     machine_names: Option<&str>,
     ips: Option<&[IpAddr]>,
     devs: Option<&[&str]>,

src/utils/route.rs

@@ -58,12 +58,12 @@ pub async fn status_smart_redirect(q: String) -> DeviceQuery {
         };
     }
     // 5) Try DNS: if it resolves, treat as name
-    if tokio::net::lookup_host((s, 0)).await.is_ok() {
+    /* if tokio::net::lookup_host((s, 0)).await.is_ok() {
         return DeviceQuery {
             name: Some(s.to_string()),
             ..Default::default()
         };
-    }
+    } */
     // Default: name last // it will fail also
     DeviceQuery {
         name: Some(s.to_string()),