notedeck

commit 1a3112d8efdb5ca826239a38aa30b5ba4bbb63a3
parent c1d0ea1901c17ffd04d0ba655b56c8938b38ccd9
Author: William Casarin <jb55@jb55.com>
Date:   Thu, 24 Jul 2025 12:29:11 -0700

Merge remote-tracking branch 'github/pr/1027'

Diffstat:
M
crates/notedeck_chrome/src/android.rs
 | 
11
++++++-----
M
crates/notedeck_chrome/src/notedeck.rs
 | 
21
++++++---------------

2 files changed, 12 insertions(+), 20 deletions(-)
diff --git a/crates/notedeck_chrome/src/android.rs b/crates/notedeck_chrome/src/android.rs
@@ -2,11 +2,13 @@
 //use egui_android::run_android;
 
 use egui_winit::winit::platform::android::activity::AndroidApp;
+use notedeck::enostr::Error;
 use notedeck_columns::Damus;
 use notedeck_dave::Dave;
 
 use crate::{app::NotedeckApp, chrome::Chrome, setup::setup_chrome};
 use notedeck::Notedeck;
+use tracing::error;
 
 #[no_mangle]
 #[tokio::main]
@@ -80,11 +82,10 @@ pub async fn android_main(app: AndroidApp) {
                 .intersection(columns.unrecognized_args())
                 .cloned()
                 .collect();
-            assert!(
-                completely_unrecognized.is_empty(),
-                "unrecognized args: {:?}",
-                completely_unrecognized
-            );
+            if !completely_unrecognized.is_empty() {
+                error!("Unrecognized arguments: {:?}", completely_unrecognized);
+                return Err(Error::Empty.into());
+            }
 
             chrome.add_app(NotedeckApp::Columns(columns));
             chrome.add_app(NotedeckApp::Dave(dave));
diff --git a/crates/notedeck_chrome/src/notedeck.rs b/crates/notedeck_chrome/src/notedeck.rs
@@ -9,6 +9,7 @@ use re_memory::AccountingAllocator;
 static GLOBAL: AccountingAllocator<std::alloc::System> =
     AccountingAllocator::new(std::alloc::System);
 
+use notedeck::enostr::Error;
 use notedeck::{DataPath, DataPathType, Notedeck};
 use notedeck_chrome::{
     setup::{generate_native_options, setup_chrome},
@@ -16,6 +17,7 @@ use notedeck_chrome::{
 };
 use notedeck_columns::Damus;
 use notedeck_dave::Dave;
+use tracing::error;
 use tracing_appender::non_blocking::WorkerGuard;
 use tracing_subscriber::EnvFilter;
 
@@ -104,10 +106,10 @@ async fn main() {
                 .intersection(columns.unrecognized_args())
                 .cloned()
                 .collect();
-            assert!(
-                completely_unrecognized.is_empty(),
-                "unrecognized args: {completely_unrecognized:?}"
-            );
+            if !completely_unrecognized.is_empty() {
+                error!("Unrecognized arguments: {:?}", completely_unrecognized);
+                return Err(Error::Empty.into());
+            }
 
             chrome.add_app(NotedeckApp::Columns(columns));
             chrome.add_app(NotedeckApp::Dave(dave));
@@ -212,17 +214,6 @@ mod tests {
         let mut app_ctx = notedeck.app_context();
         let app = Damus::new(&mut app_ctx, &args);
 
-        // ensure we recognized all the arguments
-        let completely_unrecognized: Vec<String> = unrecognized_args
-            .intersection(app.unrecognized_args())
-            .cloned()
-            .collect();
-        assert!(
-            completely_unrecognized.is_empty(),
-            "unrecognized args: {:?}",
-            completely_unrecognized
-        );
-
         assert_eq!(app.columns(app_ctx.accounts).columns().len(), 2);
 
         let tl1 = app