style: remove all section divider comments across src/server/

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: 16bit-ykiko

src/server/compiler.h

@@ -58,11 +58,9 @@ class Compiler {
              llvm::DenseMap<std::uint32_t, Session>& sessions);
     ~Compiler();
 
-    // ----- Module / compile graph (operates on Workspace) -----
 
     void init_compile_graph();
 
-    // ----- Compile argument resolution -----
 
     /// Fill compile arguments for a file (CDB lookup + header context fallback).
     /// @param session  If non-null, used for header context resolution on open files.
@@ -71,13 +69,11 @@ class Compiler {
                            std::vector<std::string>& arguments,
                            Session* session = nullptr);
 
-    // ----- Pull-based compilation -----
 
     /// Compile an open file's AST if dirty.  On success, updates session's
     /// file_index, pch_ref, ast_deps, and publishes diagnostics.
     et::task<bool> ensure_compiled(Session& session);
 
-    // ----- Feature request forwarding to workers -----
 
     using RawResult = et::task<et::serde::RawValue, et::ipc::Error>;
 

src/server/indexer.cpp

@@ -21,7 +21,6 @@ namespace clice {
 
 namespace lsp = eventide::ipc::lsp;
 
-// -- Indexer: data management -------------------------------------------------
 
 void Indexer::merge(const void* tu_index_data, std::size_t size) {
     auto tu_index = index::TUIndex::from(tu_index_data);
@@ -183,7 +182,6 @@ bool Indexer::need_update(llvm::StringRef file_path) {
     return merged_it->second.index.need_update(path_mapping);
 }
 
-// -- Indexer: symbol queries --------------------------------------------------
 
 bool Indexer::find_symbol_info(index::SymbolHash hash, std::string& name, SymbolKind& kind) const {
     for(auto& [_, session]: sessions) {
@@ -369,7 +367,6 @@ std::optional<SymbolInfo>
     return lookup_symbol(uri, path, range.start, session);
 }
 
-// -- Indexer: relation collection helpers --------------------------------------
 
 void Indexer::collect_grouped_relations(
     index::SymbolHash hash,
@@ -436,7 +433,6 @@ void Indexer::collect_unique_targets(index::SymbolHash hash,
     }
 }
 
-// -- Indexer: hierarchy queries ------------------------------------------------
 
 /// Resolve a symbol hash into a SymbolInfo with definition location.
 /// Returns nullopt if the symbol or its definition cannot be found.
@@ -581,7 +577,6 @@ std::vector<protocol::SymbolInformation> Indexer::search_symbols(llvm::StringRef
     return results;
 }
 
-// -- Indexer: static utilities ------------------------------------------------
 
 protocol::SymbolKind Indexer::to_lsp_symbol_kind(SymbolKind kind) {
     switch(kind) {

src/server/master_server.cpp

@@ -362,9 +362,7 @@ void MasterServer::register_handlers() {
         }());
     });
 
-    /// ---------------------------------------------------------------
     /// Document lifecycle — handled directly by MasterServer.
-    /// ---------------------------------------------------------------
 
     peer.on_notification([this](const protocol::DidOpenTextDocumentParams& params) {
         if(lifecycle != ServerLifecycle::Ready)
@@ -482,9 +480,7 @@ void MasterServer::register_handlers() {
         LOG_DEBUG("didSave: {}", path);
     });
 
-    /// ---------------------------------------------------------------
     /// Feature requests — stateful forwarding.
-    /// ---------------------------------------------------------------
 
     peer.on_request([this](RequestContext& ctx, const protocol::HoverParams& params) -> RawResult {
         auto path = uri_to_path(params.text_document_position_params.text_document.uri);
@@ -592,9 +588,7 @@ void MasterServer::register_handlers() {
         return indexer.resolve_hierarchy_item(uri, path, range, data, session);
     };
 
-    /// ---------------------------------------------------------------
     /// Feature requests — index-based with AST fallback.
-    /// ---------------------------------------------------------------
 
     peer.on_request([this, query_at](RequestContext& ctx,
                                      const protocol::DefinitionParams& params) -> RawResult {
@@ -650,9 +644,7 @@ void MasterServer::register_handlers() {
             co_return serde_raw{"null"};
         });
 
-    /// ---------------------------------------------------------------
     /// Feature requests — stateless forwarding.
-    /// ---------------------------------------------------------------
 
     peer.on_request([this](RequestContext& ctx,
                            const protocol::CompletionParams& params) -> RawResult {
@@ -677,9 +669,7 @@ void MasterServer::register_handlers() {
                                                       sit->second);
         });
 
-    /// ---------------------------------------------------------------
     /// Hierarchy queries — index-based.
-    /// ---------------------------------------------------------------
 
     peer.on_request(
         [this, lookup_at](RequestContext& ctx,
@@ -772,9 +762,7 @@ void MasterServer::register_handlers() {
             co_return to_raw(results);
         });
 
-    /// ---------------------------------------------------------------
     /// clice/ extension commands.
-    /// ---------------------------------------------------------------
 
     peer.on_request(
         "clice/queryContext",

src/server/session.h

@@ -26,12 +26,10 @@ namespace et = eventide;
 /// Sessions may READ from Workspace (e.g. to obtain PCH/PCM paths, module
 /// mappings, include graph) but all compilation results stay here.
 struct Session {
-    // ----- Identity -----
 
     /// Path ID of this file in PathPool.  Set on creation, never changes.
     std::uint32_t path_id = 0;
 
-    // ----- Document state (from LSP) -----
 
     /// LSP document version, incremented by the client on each edit.
     int version = 0;
@@ -56,7 +54,6 @@ struct Session {
 
     std::shared_ptr<PendingCompile> compiling;
 
-    // ----- Compilation artifacts (per-file, references Workspace caches) -----
 
     /// Reference to the PCH entry in Workspace.pch_cache, if any.
     /// The PCH itself is owned by Workspace (shared, content-addressed);
@@ -73,7 +70,6 @@ struct Session {
     /// Used for two-layer staleness detection (mtime + content hash).
     std::optional<DepsSnapshot> ast_deps;
 
-    // ----- Header context (only meaningful for header files) -----
 
     /// Compilation context for header files that lack their own CDB entry.
     /// Stores the host source file and synthesized preamble for this header.
@@ -83,7 +79,6 @@ struct Session {
     /// When set, overrides automatic header context resolution.
     std::optional<std::uint32_t> active_context;
 
-    // ----- Index (per-file, not merged into Workspace) -----
 
     /// Symbol index built from the latest compilation of this file's buffer.
     /// Used for queries (hover, goto, references) on this file.

src/server/workspace.cpp

@@ -35,7 +35,6 @@ const static index::Occurrence* lookup_occurrence(const std::vector<index::Occur
     return best;
 }
 
-// -- OpenFileIndex ------------------------------------------------------------
 
 std::optional<std::pair<index::SymbolHash, protocol::Range>>
     OpenFileIndex::find_occurrence(std::uint32_t offset) const {
@@ -54,7 +53,6 @@ std::optional<std::pair<index::SymbolHash, protocol::Range>>
     };
 }
 
-// -- MergedIndexShard ---------------------------------------------------------
 
 std::optional<std::pair<index::SymbolHash, protocol::Range>>
     MergedIndexShard::find_occurrence(std::uint32_t offset) const {
@@ -76,7 +74,6 @@ std::optional<std::pair<index::SymbolHash, protocol::Range>>
     return result;
 }
 
-// -- Workspace ----------------------------------------------------------------
 
 llvm::SmallVector<std::uint32_t> Workspace::on_file_saved(std::uint32_t path_id) {
     llvm::SmallVector<std::uint32_t> dirtied;
@@ -98,7 +95,6 @@ void Workspace::on_file_closed(std::uint32_t path_id) {
     pch_cache.erase(path_id);
 }
 
-// -- Free helper functions (used by Workspace and Compiler) -------------------
 
 std::uint64_t hash_file(llvm::StringRef path) {
     auto buf = llvm::MemoryBuffer::getFile(path);
@@ -145,7 +141,6 @@ bool deps_changed(const PathPool& pool, const DepsSnapshot& snap) {
     return false;
 }
 
-// -- Cache persistence structs ------------------------------------------------
 
 namespace {
 
@@ -179,7 +174,6 @@ struct CacheData {
 
 }  // namespace
 
-// -- Workspace cache / module methods -----------------------------------------
 
 void Workspace::load_cache() {
     if(config.cache_dir.empty())

src/server/workspace.h

@@ -171,16 +171,13 @@ struct PCMState {
 /// Sessions (open files) may READ from Workspace but must not WRITE to it
 /// except through the didSave path.
 struct Workspace {
-    // ----- Configuration -----
 
     CliceConfig config;
     CompilationDatabase cdb;
 
-    // ----- Path management -----
 
     PathPool path_pool;
 
-    // ----- Dependency graphs -----
 
     /// Include relationships between files on disk (#include edges).
     /// Built once at startup from CDB scan; updated incrementally on didSave.
@@ -195,7 +192,6 @@ struct Workspace {
     /// declarations change.
     llvm::DenseMap<std::uint32_t, std::string> path_to_module;
 
-    // ----- Compilation caches (shared, content-addressed) -----
 
     /// PCH cache, keyed by file path_id.  Content-addressed by preamble hash,
     /// so different files with identical preambles share the same PCH.
@@ -209,7 +205,6 @@ struct Workspace {
     /// Maps to the .pcm file on disk used as -fmodule-file argument.
     llvm::DenseMap<std::uint32_t, std::string> pcm_paths;
 
-    // ----- Symbol index (disk-derived) -----
 
     /// Global symbol table across all indexed translation units.
     index::ProjectIndex project_index;
@@ -219,7 +214,6 @@ struct Workspace {
     /// for position mapping.
     llvm::DenseMap<std::uint32_t, MergedIndexShard> merged_indices;
 
-    // ----- Lifecycle methods -----
 
     /// Called when a file is saved to disk.  Cascades invalidation through
     /// compile_graph and clears affected PCM caches.
@@ -230,7 +224,6 @@ struct Workspace {
     /// is a module unit so dependents can be re-evaluated on next compile.
     void on_file_closed(std::uint32_t path_id);
 
-    // ----- Cache / module methods (moved from Compiler) -----
 
     /// Load PCH/PCM cache from cache.json on disk.
     void load_cache();