Remove unusable documentation infrastructure

Rd files are not installed!!

Author: lionel-

crates/ark/src/lsp/inputs/documentation.rs

@@ -5,8 +5,6 @@
 //
 //
 
-pub mod documentation;
-pub mod documentation_rd_file;
 pub mod library;
 pub mod package;
 pub mod package_description;

crates/ark/src/lsp/inputs/documentation_rd_file.rs

@@ -8,7 +8,6 @@
 use std::fs;
 use std::path::PathBuf;
 
-use crate::lsp::inputs::documentation::Documentation;
 use crate::lsp::inputs::package_description::Description;
 use crate::lsp::inputs::package_namespace::Namespace;
 
@@ -21,7 +20,6 @@ pub struct Package {
 
     pub description: Description,
     pub namespace: Namespace,
-    pub documentation: Documentation,
 
     // List of symbols exported via NAMESPACE `export()` directives and via
     // `DocType{data}`. Note the latter should only apply to packages with
@@ -32,38 +30,21 @@ pub struct Package {
 }
 
 impl Package {
-    pub fn new(
-        path: PathBuf,
-        description: Description,
-        namespace: Namespace,
-        documentation: Documentation,
-    ) -> Self {
+    pub fn new(path: PathBuf, description: Description, namespace: Namespace) -> Self {
         // Compute exported symbols. Start from explicit NAMESPACE exports.
-        let mut exported_symbols = namespace.exports.clone();
-
-        // Add exported datasets. Ideally we'd only do that for packages
-        // specifying `LazyData: true`.
-        let exported_datasets = documentation.rd_files.iter().filter_map(|rd| {
-            if rd.doc_type == Some(crate::lsp::inputs::documentation_rd_file::RdDocType::Data) {
-                rd.name.clone()
-            } else {
-                None
-            }
-        });
-        exported_symbols.extend(exported_datasets);
+        let exported_symbols = namespace.exports.clone();
 
         Self {
             path,
             description,
             namespace,
-            documentation,
             exported_symbols,
         }
     }
 
     #[cfg(test)]
     pub fn from_parts(path: PathBuf, description: Description, namespace: Namespace) -> Self {
-        Self::new(path, description, namespace, Default::default())
+        Self::new(path, description, namespace)
     }
 
     /// Load a package from a given path.
@@ -92,20 +73,10 @@ impl Package {
             Namespace::default()
         };
 
-        let documentation_path = package_path.join("man");
-        let documentation = match Documentation::load_from_folder(&documentation_path) {
-            Ok(documentation) => documentation,
-            Err(err) => {
-                tracing::warn!("Can't load package documentation: {err:?}");
-                Documentation::default()
-            },
-        };
-
         Ok(Some(Self::new(
             package_path.to_path_buf(),
             description,
             namespace,
-            documentation,
         )))
     }
 
@@ -130,57 +101,3 @@ impl Package {
         }
     }
 }
-
-#[cfg(test)]
-mod tests {
-    use super::*;
-    use crate::lsp::inputs::documentation_rd_file::RdDocType;
-    use crate::lsp::inputs::documentation_rd_file::RdFile;
-    use crate::lsp::inputs::package_description::Description;
-    use crate::lsp::inputs::package_namespace::Namespace;
-
-    #[test]
-    fn test_exported_symbols_combining_namespace_and_rd_files() {
-        let namespace = Namespace {
-            exports: vec!["foo".to_string(), "bar".to_string()],
-            ..Default::default()
-        };
-
-        let rd_files = vec![
-            RdFile {
-                name: Some("data1".to_string()),
-                doc_type: Some(RdDocType::Data),
-            },
-            RdFile {
-                name: Some("pkgdoc".to_string()),
-                doc_type: Some(RdDocType::Package),
-            },
-            RdFile {
-                name: Some("other".to_string()),
-                doc_type: None,
-            },
-        ];
-        let documentation = Documentation { rd_files };
-
-        let description = Description {
-            name: "mypkg".to_string(),
-            version: "1.0.0".to_string(),
-            depends: vec![],
-            fields: Default::default(),
-        };
-
-        let package = Package::new(
-            PathBuf::from("/mock/path"),
-            description,
-            namespace,
-            documentation,
-        );
-
-        assert!(package.exported_symbols.contains(&"foo".to_string()));
-        assert!(package.exported_symbols.contains(&"bar".to_string()));
-        assert!(package.exported_symbols.contains(&"data1".to_string()));
-
-        assert!(!package.exported_symbols.contains(&"pkgdoc".to_string()));
-        assert!(!package.exported_symbols.contains(&"other".to_string()));
-    }
-}