chore: add clippy lints and package exclusions for crates.io

Author: AnttiJalomaki

Cargo.lock

@@ -1,12 +1,31 @@
 [package]
 name = "dioxus-maplibre"
-version = "0.0.2"
+version = "0.0.3"
 edition = "2024"
 description = "MapLibre GL JS wrapper for Dioxus 0.7"
 license = "MIT OR Apache-2.0"
 repository = "https://github.com/Nevaberry/dioxus-maplibre"
 keywords = ["dioxus", "maplibre", "maps", "gis", "wasm"]
 categories = ["gui", "wasm", "web-programming"]
+exclude = ["e2e/", "examples/", "temp/", "test-results/", ".github/"]
+
+[lints.rust]
+unsafe_code = "forbid"
+
+[lints.clippy]
+all = { level = "warn", priority = -1 }
+pedantic = { level = "warn", priority = -1 }
+nursery = { level = "warn", priority = -1 }
+module_name_repetitions = "allow"
+must_use_candidate = "allow"
+needless_raw_string_hashes = "allow"
+doc_markdown = "allow"
+missing_const_for_fn = "allow"
+suboptimal_flops = "allow"
+too_many_lines = "allow"
+option_if_let_else = "allow"
+pub_with_shorthand = "allow"
+redundant_pub_crate = "allow"
 
 [dependencies]
 dioxus = "0.7"

Cargo.toml

@@ -5,7 +5,7 @@ use dioxus::prelude::*;
 use crate::context::MapContext;
 
 /// Props for the CircleLayer component
-#[derive(Props, Clone, PartialEq)]
+#[derive(Props, Clone, PartialEq, Eq)]
 pub struct CircleLayerProps {
     /// Unique layer ID
     pub id: String,
@@ -39,11 +39,14 @@ pub struct CircleLayerProps {
 /// }
 /// ```
 #[component]
+#[allow(unused_variables)] // props only used on wasm32
 pub fn CircleLayer(props: CircleLayerProps) -> Element {
     // Get map context from parent
+    #[allow(unused_variables)] // only used on wasm32
     let ctx = use_context::<MapContext>();
 
     // Track whether layer has been added
+    #[allow(unused_variables, unused_mut)] // only used on wasm32
     let mut layer_added = use_signal(|| false);
 
     // Only do JS interop on wasm targets

src/components/circle_layer.rs

@@ -33,13 +33,16 @@ pub struct GeoJsonSourceProps {
 #[component]
 pub fn GeoJsonSource(props: GeoJsonSourceProps) -> Element {
     // Get map context from parent
+    #[allow(unused_variables)] // only used on wasm32
     let ctx = use_context::<MapContext>();
 
     // Track whether source has been added
+    #[allow(unused_variables, unused_mut)] // only used on wasm32
     let mut source_added = use_signal(|| false);
 
     // Track previous data for change detection
-    let mut prev_data = use_signal(|| String::new());
+    #[allow(unused_variables, unused_mut)] // only used on wasm32
+    let mut prev_data = use_signal(String::new);
 
     // Only do JS interop on wasm targets
     #[cfg(target_arch = "wasm32")]

src/components/geojson_source.rs

@@ -78,14 +78,16 @@ pub fn Map(props: MapProps) -> Element {
     let container_id = format!("{map_id}_container");
 
     // Track if map is ready
+    #[allow(unused_mut)] // mut needed only on wasm32
     let mut is_ready = use_signal(|| false);
 
     // Track if initialization has been started (to prevent multiple inits)
+    #[allow(unused_variables, unused_mut)] // only used on wasm32
     let mut init_started = use_signal(|| false);
 
     // Create context for child components
     let ctx = MapContext {
-        map_id: map_id.clone(),
+        map_id,
         is_ready,
     };
     use_context_provider(|| ctx);

src/components/map.rs

@@ -31,9 +31,11 @@ pub struct MarkerProps {
 #[component]
 pub fn Marker(props: MarkerProps) -> Element {
     // Get map context from parent
+    #[allow(unused_variables)] // only used on wasm32
     let ctx = use_context::<MapContext>();
 
     // Generate marker ID if not provided
+    #[allow(unused_variables)] // only used on wasm32
     let marker_id = use_hook(|| {
         props.id.clone().unwrap_or_else(|| {
             format!("marker_{}", uuid::Uuid::new_v4().to_string().replace('-', ""))

src/components/marker.rs

@@ -63,7 +63,8 @@ pub struct LayerHoverEvent {
 /// Internal event enum for communication from JS
 #[derive(Debug, Clone, Serialize, Deserialize)]
 #[serde(tag = "type")]
-pub(crate) enum MapEvent {
+#[allow(dead_code)] // Reserved for future typed event parsing
+pub enum MapEvent {
     #[serde(rename = "click")]
     Click(MapClickEvent),
     #[serde(rename = "marker_click")]

src/events.rs

@@ -3,6 +3,10 @@
 //! This module generates the JavaScript code needed to interact with MapLibre GL JS.
 //! Maps are stored in `window.__dioxus_maplibre_maps[map_id]` for lifecycle management.
 
+// These functions generate JS strings and are only called on wasm32 targets.
+// On other targets they appear unused but we keep them available for testing.
+#![allow(dead_code)]
+
 use uuid::Uuid;
 
 /// Generate a unique map ID
@@ -218,12 +222,11 @@ pub fn add_marker_js(
             r#"
             const el = document.createElement('div');
             el.className = 'maplibre-marker-emoji';
-            el.innerHTML = '{}';
+            el.innerHTML = '{e}';
             el.style.fontSize = '28px';
             el.style.cursor = 'pointer';
             el.style.filter = 'drop-shadow(0 2px 4px rgba(0,0,0,0.5))';
-            "#,
-            e
+            "#
         ),
         None => String::new(),
     };

src/interop/bridge.rs

@@ -2,4 +2,4 @@
 
 mod bridge;
 
-pub(crate) use bridge::*;
+pub use bridge::*;

src/interop/mod.rs

@@ -82,8 +82,8 @@ impl Bounds {
     /// Get the center of the bounds
     pub fn center(&self) -> LatLng {
         LatLng {
-            lat: (self.sw.lat + self.ne.lat) / 2.0,
-            lng: (self.sw.lng + self.ne.lng) / 2.0,
+            lat: f64::midpoint(self.sw.lat, self.ne.lat),
+            lng: f64::midpoint(self.sw.lng, self.ne.lng),
         }
     }
 }