dev(compiler): add appropriate Debug and Copy impl

compiler/src/browser.rs

@@ -8,6 +8,7 @@ use crate::{package::browser::ProxyRegistry, vfs::browser::ProxyAccessModel};
 /// It is under development.
 pub type TypstBrowserWorld = crate::world::CompilerWorld<BrowserCompilerFeat>;
 
+#[derive(Debug, Clone, Copy)]
 pub struct BrowserCompilerFeat;
 
 impl crate::world::CompilerFeat for BrowserCompilerFeat {

compiler/src/font/system.rs

@@ -23,7 +23,7 @@ use typst_ts_core::{
 
 use crate::vfs::system::LazyFile;
 
-#[derive(Default)]
+#[derive(Debug, Default)]
 struct FontProfileRebuilder {
     path_items: HashMap<PathBuf, FontProfileItem>,
     pub profile: FontProfile,
@@ -77,6 +77,7 @@ impl FontProfileRebuilder {
 }
 
 /// Searches for fonts.
+#[derive(Debug)]
 pub struct SystemFontSearcher {
     db: Database,
 

compiler/src/font/web/mod.rs

@@ -347,6 +347,7 @@ impl WebFont {
 
 unsafe impl Send for WebFont {}
 
+#[derive(Debug)]
 pub struct WebFontLoader {
     font: WebFont,
     index: u32,

compiler/src/lib.rs

@@ -23,6 +23,9 @@
 //!   - [`service::CompileDriver`]: A driver for the compiler. Examples:
 //!     - Single thread (Sync): <https://github.com/Myriad-Dreamin/typst.ts/blob/main/cli/src/main.rs>
 //!     - Multiple thread (Async): <https://github.com/Enter-tainer/typst-preview-vscode/blob/main/src/main.rs>
+// #![warn(missing_docs)]
+// #![warn(missing_debug_implementations)]
+// #![warn(missing_copy_implementations)]
 
 pub(crate) mod macros;
 

compiler/src/package/browser.rs

@@ -7,7 +7,7 @@ use wasm_bindgen::{prelude::*, JsValue};
 use super::{PackageError, PackageSpec, Registry};
 
 #[wasm_bindgen]
-#[derive(Clone)]
+#[derive(Debug, Clone)]
 pub struct ProxyContext {
     context: JsValue,
 }
@@ -70,6 +70,7 @@ impl ProxyContext {
     }
 }
 
+#[derive(Debug)]
 pub struct ProxyRegistry {
     pub context: ProxyContext,
     pub real_resolve_fn: js_sys::Function,

compiler/src/parser/semantic_tokens.rs

@@ -6,7 +6,7 @@ use typst::syntax::{ast, LinkedNode, Source, SyntaxKind};
 use super::modifier_set::ModifierSet;
 use super::typst_tokens::{Modifier, TokenType};
 
-#[derive(serde::Deserialize, serde::Serialize)]
+#[derive(serde::Deserialize, serde::Serialize, Debug, Clone)]
 pub struct SemanticTokensLegend {
     #[serde(rename = "tokenTypes")]
     pub token_types: Vec<String>,
@@ -33,6 +33,7 @@ pub fn get_semantic_tokens_legend() -> SemanticTokensLegend {
     }
 }
 
+#[derive(Debug, Clone, Copy)]
 pub enum OffsetEncoding {
     Utf8,
     Utf16,
@@ -83,6 +84,7 @@ fn tokenize_tree(root: &LinkedNode<'_>, parent_modifiers: ModifierSet) -> Vec<Se
     token.chain(children).collect()
 }
 
+#[derive(Debug, Clone, Copy)]
 pub struct SemanticToken {
     pub delta_line: u32,
     pub delta_start_character: u32,

compiler/src/parser/typst_tokens.rs

@@ -4,7 +4,7 @@ use strum::EnumIter;
 
 /// Very similar to [`typst_ide::Tag`], but with convenience traits, and
 /// extensible because we want to further customize highlighting
-#[derive(Clone, Copy, EnumIter)]
+#[derive(Debug, Clone, Copy, EnumIter)]
 #[repr(u32)]
 pub enum TokenType {
     // Standard LSP types
@@ -68,7 +68,7 @@ impl From<TokenType> for &'static str {
     }
 }
 
-#[derive(Clone, Copy, EnumIter)]
+#[derive(Debug, Clone, Copy, EnumIter)]
 #[repr(u8)]
 pub enum Modifier {
     Strong,

compiler/src/service/compile.rs

@@ -421,6 +421,7 @@ impl<C: Compiler> CompileActor<C> {
         self.latest_doc.clone()
     }
 }
+#[derive(Debug, Clone)]
 pub struct CompileClient<Ctx> {
     steal_send: mpsc::UnboundedSender<BorrowTask<Ctx>>,
     memory_send: mpsc::UnboundedSender<MemoryEvent>,

compiler/src/service/diag.rs

@@ -118,6 +118,7 @@ pub fn status(entry_file: TypstFileId, status: DiagStatus) -> io::Result<()> {
     Ok(())
 }
 
+#[derive(Debug, Clone, Copy)]
 pub struct ConsoleDiagReporter<W>(PhantomParamData<W>);
 
 impl<W> Default for ConsoleDiagReporter<W>

compiler/src/service/export.rs

@@ -115,13 +115,11 @@ impl CompileReport {
 pub type ReportExporter = DynExporter<CompileReport>;
 pub type FeaturedReportExporter = DynExporter<(Arc<FeatureSet>, CompileReport)>;
 
-#[allow(dead_code)]
 pub struct CompileReporter<C: Compiler> {
     pub compiler: C,
     pub reporter: DynGenericExporter<C::World, (Arc<FeatureSet>, CompileReport)>,
 }
 
-#[allow(dead_code)]
 impl<C: Compiler> CompileReporter<C>
 where
     C::World: 'static,

compiler/src/service/features.rs

@@ -11,7 +11,7 @@ pub struct FeatureSlot(u16);
 static ALLOCATOR: Lazy<std::sync::atomic::AtomicU16> =
     Lazy::new(|| std::sync::atomic::AtomicU16::new(1));
 
-#[derive(Default)]
+#[derive(Debug, Default)]
 pub struct LazyFeatureSlot(once_cell::sync::OnceCell<FeatureSlot>);
 
 impl From<&LazyFeatureSlot> for FeatureSlot {
@@ -22,7 +22,7 @@ impl From<&LazyFeatureSlot> for FeatureSlot {
     }
 }
 
-#[derive(Default, Clone)]
+#[derive(Debug, Default, Clone)]
 pub struct FeatureSet {
     features: Vec<EcoString>,
 }
@@ -54,6 +54,7 @@ pub trait CompileFeature<T> {
     fn retrieve(&self, features: &FeatureSet) -> T;
 }
 
+#[derive(Debug, Clone, Copy)]
 pub struct DiagFmtFeature;
 const DIAG_FEATURE: FeatureSlot = FeatureSlot(0);
 pub static DIAG_FMT_FEATURE: DiagFmtFeature = DiagFmtFeature;
@@ -78,7 +79,7 @@ impl CompileFeature<DiagnosticFormat> for DiagFmtFeature {
     }
 }
 
-#[derive(Default)]
+#[derive(Debug, Default)]
 pub struct BuiltinFeature<T>(LazyFeatureSlot, std::marker::PhantomData<T>);
 
 impl<T> BuiltinFeature<T> {

compiler/src/service/mod.rs

@@ -315,6 +315,7 @@ where
 }
 
 /// The status in which the watcher can be.
+#[derive(Debug, Clone, Copy)]
 pub enum DiagStatus {
     Stage(&'static str),
     Success(std::time::Duration),

compiler/src/service/watch.rs

@@ -55,6 +55,7 @@ impl Default for WatchState {
 }
 
 /// The data entry of a watched file.
+#[derive(Debug)]
 struct WatchedEntry {
     /// The lifetime of the entry.
     ///
@@ -85,6 +86,7 @@ struct UndeterminedNotifyEvent {
 // Drop order is significant.
 /// The actor that watches files.
 /// It is used to watch files and send events to the consumers
+#[derive(Debug)]
 pub struct NotifyActor {
     /// The access model of the actor.
     /// We concrete the access model to `SystemAccessModel` for now.

compiler/src/system.rs

@@ -9,6 +9,7 @@ use crate::{
 };
 
 /// type trait of [`TypstSystemWorld`].
+#[derive(Debug, Clone, Copy)]
 pub struct SystemCompilerFeat;
 
 impl crate::world::CompilerFeat for SystemCompilerFeat {

compiler/src/vfs/browser.rs

@@ -9,6 +9,7 @@ use crate::time::SystemTime;
 
 use super::AccessModel;
 
+#[derive(Debug)]
 pub struct ProxyAccessModel {
     pub context: JsValue,
     pub mtime_fn: js_sys::Function,

compiler/src/vfs/cached.rs

@@ -12,6 +12,7 @@ use super::AccessModel;
 /// incrementally query a value from a self holding state
 type IncrQueryRef<S, E> = QueryRef<S, E, Option<S>>;
 
+#[derive(Debug)]
 pub struct FileCache<S> {
     lifetime_cnt: usize,
     mtime: SystemTime,
@@ -20,6 +21,7 @@ pub struct FileCache<S> {
     source_state: IncrQueryRef<S, FileError>,
 }
 
+#[derive(Debug)]
 pub struct CachedAccessModel<Inner: AccessModel, C> {
     inner: Inner,
     lifetime_cnt: usize,

compiler/src/vfs/dummy.rs

@@ -8,7 +8,7 @@ use crate::time::SystemTime;
 
 use super::AccessModel;
 
-#[derive(Default, Debug)]
+#[derive(Default, Debug, Clone, Copy)]
 pub struct DummyAccessModel;
 
 impl AccessModel for DummyAccessModel {

compiler/src/vfs/mod.rs

@@ -20,6 +20,7 @@ mod path_interner;
 
 pub(crate) use path_interner::PathInterner;
 
+use core::fmt;
 use std::{collections::HashMap, ffi::OsStr, hash::Hash, path::Path, sync::Arc};
 
 use append_only_vec::AppendOnlyVec;
@@ -65,6 +66,7 @@ pub trait AccessModel {
 type FileQuery<T> = QueryRef<T, FileError>;
 
 /// Holds canonical data for all paths pointing to the same entity.
+#[derive(Debug)]
 pub struct PathSlot {
     idx: FileId,
     sampled_path: once_cell::sync::OnceCell<ImmutPath>,
@@ -100,6 +102,18 @@ pub struct Vfs<M: AccessModel + Sized> {
     pub do_reparse: bool,
 }
 
+impl<M: AccessModel + Sized> fmt::Debug for Vfs<M> {
+    fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result {
+        f.debug_struct("Vfs")
+            .field("lifetime_cnt", &self.lifetime_cnt)
+            .field("path2slot", &self.path2slot)
+            .field("src2file_id", &self.src2file_id)
+            .field("slots", &self.slots)
+            .field("do_reparse", &self.do_reparse)
+            .finish()
+    }
+}
+
 impl<M: AccessModel + Sized> Vfs<M> {
     pub fn new(access_model: M) -> Self {
         let access_model = NotifyAccessModel::new(access_model);

compiler/src/vfs/notify.rs

@@ -198,6 +198,7 @@ pub enum NotifyMessage {
 
 /// Notify shadowing access model, which the typical underlying access model is
 /// [`crate::vfs::system::SystemAccessModel`]
+#[derive(Debug)]
 pub struct NotifyAccessModel<M: AccessModel> {
     files: HashMap<ImmutPath, FileSnapshot>,
     pub inner: M,

compiler/src/vfs/path_interner.rs

@@ -10,6 +10,7 @@ use rustc_hash::FxHasher;
 use super::FileId;
 
 /// Structure to map between [`VfsPath`] and [`FileId`].
+#[derive(Debug)]
 pub(crate) struct PathInterner<P, Ext = ()> {
     map: IndexMap<P, Ext, BuildHasherDefault<FxHasher>>,
 }

compiler/src/vfs/system.rs

@@ -10,6 +10,7 @@ use typst_ts_core::{Bytes, ReadAllOnce};
 
 use super::AccessModel;
 
+#[derive(Debug)]
 pub struct LazyFile {
     path: std::path::PathBuf,
     file: Option<std::io::Result<File>>,
@@ -34,11 +35,13 @@ impl ReadAllOnce for LazyFile {
     }
 }
 
+#[derive(Debug, Clone, Copy)]
 pub struct SystemFileMeta {
     mt: std::time::SystemTime,
     is_file: bool,
 }
 
+#[derive(Debug, Clone, Copy)]
 pub struct SystemAccessModel;
 
 impl SystemAccessModel {

compiler/src/vfs/trace.rs

@@ -8,6 +8,7 @@ use crate::time::SystemTime;
 
 use super::{cached::CachedAccessModel, AccessModel};
 
+#[derive(Debug)]
 pub struct TraceAccessModel<M: AccessModel + Sized> {
     inner: M,
     trace: [AtomicU64; 6],

compiler/src/world.rs

@@ -45,6 +45,7 @@ pub trait CompilerFeat {
 }
 
 /// A world that provides access to the operating system.
+#[derive(Debug)]
 pub struct CompilerWorld<F: CompilerFeat> {
     /// Path to the root directory of compilation.
     /// The world forbids direct access to files outside this directory.

core/src/concepts/marker.rs

@@ -10,6 +10,7 @@ use serde_with::{DeserializeAs, SerializeAs};
 /// Generic marker for type that only occurs in parameter position.
 ///
 /// Safety: The signature is type checked by the compiler.
+#[derive(Debug, Clone, Copy)]
 pub struct PhantomParamData<X>(std::marker::PhantomData<X>);
 unsafe impl<X> Send for PhantomParamData<X> {}
 

core/src/concepts/query.rs

@@ -127,3 +127,19 @@ impl<T, E> Default for QueryRef<T, E> {
         }
     }
 }
+
+impl<T, E, QC> fmt::Debug for QueryRef<T, E, QC>
+where
+    T: fmt::Debug,
+    E: fmt::Debug,
+    QC: fmt::Debug,
+{
+    fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result {
+        let borrowed = self.cell.borrow();
+        let (ref ctx, ref res) = *borrowed;
+        f.debug_struct("QueryRef")
+            .field("context", &ctx)
+            .field("result", &res)
+            .finish()
+    }
+}

core/src/font/partial_book.rs

@@ -21,7 +21,7 @@ impl From<&FontInfo> for FontInfoKey {
     }
 }
 
-#[derive(Default)]
+#[derive(Debug, Default)]
 pub struct PartialFontBook {
     pub revision: usize,
     pub partial_hit: bool,

core/src/font/resolver.rs

@@ -36,6 +36,7 @@ pub trait FontResolver {
     }
 }
 
+#[derive(Debug)]
 /// The default FontResolver implementation.
 pub struct FontResolverImpl {
     book: Prehashed<FontBook>,

core/src/font/slot.rs

@@ -1,3 +1,5 @@
+use core::fmt;
+
 use typst::font::Font;
 
 use crate::{FontLoader, QueryRef};
@@ -35,3 +37,11 @@ impl FontSlot {
         { unsafe { res.unwrap_unchecked() } }.clone()
     }
 }
+
+impl fmt::Debug for FontSlot {
+    fn fmt(&self, f: &mut fmt::Formatter) -> fmt::Result {
+        f.debug_tuple("FontSlot")
+            .field(&self.get_uninitialized())
+            .finish()
+    }
+}