Remove doc comments that restate the code

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

Author: dak2

rust/ruby-rbs/src/environment/source.rs

@@ -2,8 +2,8 @@ use std::path::PathBuf;
 
 use crate::ast::{Declaration, Directive};
 
-/// Where a loaded signature file came from, corresponding to the `source`
-/// values yielded by `RBS::EnvironmentLoader#each_dir`.
+/// Corresponds to the `source` values yielded by
+/// `RBS::EnvironmentLoader#each_dir`.
 #[derive(Debug, Clone, PartialEq, Eq)]
 pub enum SourceKind {
     Core,
@@ -20,14 +20,11 @@ pub enum SourceKind {
 }
 
 impl SourceKind {
-    /// Whether `_`-prefixed subdirectories are skipped while scanning.
-    /// Only a user-specified [`SourceKind::Dir`] does not skip them.
     pub fn skips_hidden(&self) -> bool {
         !matches!(self, SourceKind::Dir { .. })
     }
 }
 
-/// `RBS::Source::RBS` equivalent.
 #[derive(Debug)]
 pub struct Source {
     pub path: PathBuf,

rust/ruby-rbs/src/file_finder.rs

@@ -1,8 +1,7 @@
 use std::io;
 use std::path::{Path, PathBuf};
 
-/// Enumerates `.rbs` entries under `path` in path-sorted order, mirroring
-/// `RBS::FileFinder.each_file` and its `Dir.glob` semantics.
+/// Mirrors `RBS::FileFinder.each_file` and its `Dir.glob` semantics.
 ///
 /// Divergences: the sort compares the platform separator rather than
 /// `/`-joined strings, and only `PermissionDenied`/`NotFound` are skipped

rust/ruby-rbs/src/interners.rs

@@ -1,7 +1,6 @@
 use crate::interner::StringInterner;
 use crate::type_name::TypeNameInterner;
 
-/// The pair of interners that an owned AST's ids refer to.
 #[derive(Default)]
 pub struct Interners {
     pub strings: StringInterner,

rust/ruby-rbs/src/loader/mod.rs

@@ -44,9 +44,7 @@ pub struct LoadedFile {
     pub kind: SourceKind,
 }
 
-/// Enumerates core, library, and explicit signature directories, parses
-/// every `.rbs` file exactly once, and feeds the results into an
-/// [`Environment`], mirroring `RBS::EnvironmentLoader`.
+/// Mirrors `RBS::EnvironmentLoader`.
 ///
 /// Unlike the Ruby implementation, this does not resolve gem names or
 /// versions to directories, and does not expand `manifest.yaml`
@@ -61,7 +59,6 @@ pub struct EnvironmentLoader {
 }
 
 impl EnvironmentLoader {
-    /// `core_root` is `None` to skip core.
     pub fn new(core_root: Option<PathBuf>) -> Self {
         EnvironmentLoader {
             core_root,
@@ -70,14 +67,11 @@ impl EnvironmentLoader {
         }
     }
 
-    /// Adds a library by name and its already-resolved signature directory.
     pub fn add_library(mut self, name: &str, path: PathBuf) -> Self {
         self.libs.push((name.to_string(), path));
         self
     }
 
-    /// Adds an explicit signature directory. Unlike libraries, `_`-prefixed
-    /// subdirectories are not skipped.
     pub fn add_dir(mut self, path: PathBuf) -> Self {
         self.dirs.push(path);
         self
@@ -113,8 +107,6 @@ impl EnvironmentLoader {
         Ok(loaded)
     }
 
-    /// Groups directories core → libs → dirs, matching
-    /// `RBS::EnvironmentLoader#each_dir`'s enumeration order.
     fn each_dir(&self) -> Vec<(SourceKind, PathBuf)> {
         let mut result = Vec::new();
 
@@ -144,10 +136,6 @@ impl EnvironmentLoader {
 /// can later be parallelised by handing each worker its own [`Interners`].
 /// The parser's `SignatureNode` holds raw pointers and is not `Send`, so it
 /// must not escape this function — only the owned `Source` does.
-///
-/// Crate-private: a caller outside the crate has no way to merge its
-/// worker-local [`Interners`] into the environment, so the `Source` it
-/// produced would carry ids that environment cannot resolve.
 pub(crate) fn parse_one(
     path: &Path,
     kind: &SourceKind,