Warn on loading Geo* archetypes without the map_view feature (#8061)

### What
* Closes #8052

### Checklist
* [x] I have read and agree to [Contributor
Guide](https://github.com/rerun-io/rerun/blob/main/CONTRIBUTING.md) and
the [Code of
Conduct](https://github.com/rerun-io/rerun/blob/main/CODE_OF_CONDUCT.md)
* [x] I've included a screenshot or gif (if applicable)
* [x] I have tested the web demo (if applicable):
* Using examples from latest `main` build:
[rerun.io/viewer](https://rerun.io/viewer/pr/8061?manifest_url=https://app.rerun.io/version/main/examples_manifest.json)
* Using full set of examples from `nightly` build:
[rerun.io/viewer](https://rerun.io/viewer/pr/8061?manifest_url=https://app.rerun.io/version/nightly/examples_manifest.json)
* [x] The PR title and labels are set such as to maximize their
usefulness for the next release's CHANGELOG
* [x] If applicable, add a new check to the [release
checklist](https://github.com/rerun-io/rerun/blob/main/tests/python/release_checklist)!
* [x] If have noted any breaking changes to the log API in
`CHANGELOG.md` and the migration guide

- [PR Build Summary](https://build.rerun.io/pr/8061)
- [Recent benchmark results](https://build.rerun.io/graphs/crates.html)
- [Wasm size tracking](https://build.rerun.io/graphs/sizes.html)

To run all checks from `main`, comment on the PR with `@rerun-bot
full-check`.

Author: emilk

crates/build/re_types_builder/src/codegen/docs/mod.rs

@@ -9,9 +9,8 @@ use itertools::Itertools;
 
 use crate::{
     codegen::{autogen_warning, common::ExampleInfo, Target},
-    objects::FieldKind,
+    objects::{FieldKind, ViewReference},
     CodeGenerator, GeneratedFiles, Object, ObjectField, ObjectKind, Objects, Reporter, Type,
-    ATTR_DOCS_VIEW_TYPES,
 };
 
 pub const DATAFRAME_VIEW_FQNAME: &str = "rerun.blueprint.views.DataframeView";
@@ -34,12 +33,6 @@ impl DocsCodeGenerator {
     }
 }
 
-struct ViewReference {
-    /// Typename of the view. Not a fully qualified name, just the name as specified on the attribute.
-    view_name: String,
-    explanation: Option<String>,
-}
-
 type ViewsPerArchetype = BTreeMap<String, Vec<ViewReference>>;
 
 impl CodeGenerator for DocsCodeGenerator {
@@ -144,23 +137,9 @@ on [Entities and Components](../../concepts/entity-component.md).",
 fn collect_view_types_per_archetype(objects: &Objects) -> ViewsPerArchetype {
     let mut view_types_per_object = ViewsPerArchetype::new();
     for object in objects.objects.values() {
-        let Some(view_types) = object.try_get_attr::<String>(ATTR_DOCS_VIEW_TYPES) else {
-            continue;
-        };
-
-        let view_types = view_types
-            .split(',')
-            .map(|view_type| {
-                let mut parts = view_type.splitn(2, ':');
-                let view_name = parts.next().unwrap().trim().to_owned();
-                let explanation = parts.next().map(|s| s.trim().to_owned());
-                ViewReference {
-                    view_name,
-                    explanation,
-                }
-            })
-            .collect();
-        view_types_per_object.insert(object.fqname.clone(), view_types);
+        if let Some(view_types) = object.archetype_view_types() {
+            view_types_per_object.insert(object.fqname.clone(), view_types);
+        }
     }
 
     view_types_per_object

crates/build/re_types_builder/src/codegen/rust/reflection.rs

@@ -1,6 +1,7 @@
 use std::collections::{BTreeMap, BTreeSet, HashMap};
 
 use camino::Utf8PathBuf;
+use itertools::Itertools as _;
 use proc_macro2::TokenStream;
 use quote::{format_ident, quote};
 
@@ -217,10 +218,24 @@ fn generate_archetype_reflection(reporter: &Reporter, objects: &Objects) -> Toke
             .join("\n");
         }
 
+        let quoted_view_types = obj
+            .archetype_view_types()
+            .unwrap_or_default()
+            .iter()
+            .map(|view_type| {
+                let view_name = &view_type.view_name;
+                quote! { #view_name }
+            })
+            .collect_vec();
+
         let quoted_archetype_reflection = quote! {
             ArchetypeReflection {
                 display_name: #display_name,
 
+                view_types: &[
+                    #(#quoted_view_types,)*
+                ],
+
                 fields: vec![
                     #(#quoted_field_reflections,)*
                 ],

crates/build/re_types_builder/src/objects.rs

@@ -280,6 +280,13 @@ impl ObjectKind {
     }
 }
 
+pub struct ViewReference {
+    /// Typename of the view. Not a fully qualified name, just the name as specified on the attribute.
+    pub view_name: String,
+
+    pub explanation: Option<String>,
+}
+
 /// A high-level representation of a flatbuffers object, which can be either a struct, a union or
 /// an enum.
 #[derive(Debug, Clone)]
@@ -531,6 +538,25 @@ impl Object {
         self.attrs.has(name)
     }
 
+    pub fn archetype_view_types(&self) -> Option<Vec<ViewReference>> {
+        let view_types = self.try_get_attr::<String>(crate::ATTR_DOCS_VIEW_TYPES)?;
+
+        Some(
+            view_types
+                .split(',')
+                .map(|view_type| {
+                    let mut parts = view_type.splitn(2, ':');
+                    let view_name = parts.next().unwrap().trim().to_owned();
+                    let explanation = parts.next().map(|s| s.trim().to_owned());
+                    ViewReference {
+                        view_name,
+                        explanation,
+                    }
+                })
+                .collect(),
+        )
+    }
+
     pub fn is_struct(&self) -> bool {
         self.class == ObjectClass::Struct
     }

crates/store/re_types_core/src/reflection.rs

@@ -233,6 +233,11 @@ pub struct ArchetypeReflection {
     /// The name of the field in human case.
     pub display_name: &'static str,
 
+    /// The views that this archetype can be added to.
+    ///
+    /// e.g. `Spatial3DView`.
+    pub view_types: &'static [&'static str],
+
     /// All the component fields of the archetype, in the order they appear in the archetype.
     pub fields: Vec<ArchetypeFieldReflection>,
 }

crates/viewer/re_viewer/src/app.rs

@@ -1189,6 +1189,8 @@ impl App {
                     if let Some(caches) = store_hub.active_caches() {
                         caches.on_store_events(&store_events);
                     }
+
+                    self.validate_loaded_events(&store_events);
                 }
 
                 Err(err) => {
@@ -1292,6 +1294,32 @@ impl App {
         }
     }
 
+    /// After loading some data; check if the loaded data makes sense.
+    fn validate_loaded_events(&self, store_events: &[re_chunk_store::ChunkStoreEvent]) {
+        re_tracing::profile_function!();
+
+        for event in store_events {
+            let chunk = &event.diff.chunk;
+            for component in chunk.component_names() {
+                if let Some(archetype_name) = component.indicator_component_archetype() {
+                    if let Some(archetype) = self
+                        .reflection
+                        .archetype_reflection_from_short_name(&archetype_name)
+                    {
+                        for &view_type in archetype.view_types {
+                            // TODO(#7876): remove once `map_view` feature is gone
+                            if !cfg!(feature = "map_view") && view_type == "MapView" {
+                                re_log::warn_once!("Found map-related archetype, but viewer was not compiled with the `map_view` feature.");
+                            }
+                        }
+                    } else {
+                        re_log::debug_once!("Unknown archetype: {archetype_name}");
+                    }
+                }
+            }
+        }
+    }
+
     fn purge_memory_if_needed(&mut self, store_hub: &mut StoreHub) {
         re_tracing::profile_function!();