refactor: remove outdated comments and improve code clarity across multiple files

Author: leandromqrs

crates/aetheris-app/src/app.rs

@@ -198,10 +198,6 @@ flowboxchild.filter-chip-child:selected:hover {
   background-color: alpha(@accent_color, 0.24);
 }
 
-/* Object tables: rounded card look, matching the old boxed-list style —
- * card background for rows, a distinct shaded header, hairline row
- * separators. The frame class goes on the enclosing ScrolledWindow so the
- * rounded corners wrap header and rows together. */
 scrolledwindow.aetheris-table-frame {
   background-color: @card_bg_color;
   border: 1px solid alpha(currentColor, 0.1);
@@ -213,10 +209,6 @@ columnview.aetheris-table > listview {
   background: transparent;
 }
 
-/* GNOME-style header: dimmed label at rest (like .dim-label), and on
- * hover a genuinely darker surface (@shade_color is Adwaita's translucent
- * black shading color in both light and dark) with the label brightening
- * to full strength. */
 columnview.aetheris-table > header > button {
   background-color: alpha(currentColor, 0.05);
   border-bottom: 1px solid alpha(currentColor, 0.1);
@@ -315,9 +307,6 @@ pub struct App {
     add_cluster_button: gtk::Button,
     import_cluster_button: gtk::Button,
     clusters_content_stack: gtk::Stack,
-    /// State/Provider/Version/CPU/Memory/Pods snapshot per context name,
-    /// fetched lazily the first time the Clusters page shows a given
-    /// context and cached until the app restarts.
     cluster_summaries: std::collections::HashMap<String, ClusterSummaryState>,
     namespace_menu_button: gtk::MenuButton,
     namespace_selector_label: gtk::Label,
@@ -342,7 +331,6 @@ pub struct App {
     /// activation positions index into this model.
     object_sorted: gtk::SortListModel,
     object_columns: Vec<(ObjectTableColumn, gtk::ColumnViewColumn)>,
-    /// Switches between the object table and the "No objects" placeholder.
     object_list_stack: gtk::Stack,
     detail_stack: gtk::Stack,
     detail_name_label: gtk::Label,
@@ -368,8 +356,6 @@ pub struct App {
     detail_conditions_list: gtk::ListBox,
     detail_related_pods_store: gtk::gio::ListStore,
     detail_related_pods_sorted: gtk::SortListModel,
-    /// Switches the detail "Pods" tab between the pods table and an
-    /// informational message (non-Deployment kinds, empty selector match).
     detail_related_pods_stack: gtk::Stack,
     detail_related_pods_message: adw::StatusPage,
     detail_log_container_dropdown: gtk::DropDown,
@@ -414,8 +400,6 @@ pub struct App {
     project_dialog_description: gtk::Label,
     project_name_entry: adw::EntryRow,
     project_create_button: gtk::Button,
-    /// The project's name before this edit, when the project dialog is in
-    /// rename mode. `None` means the dialog is creating a new project.
     editing_project_name: Option<String>,
     create_yaml_dialog: adw::Dialog,
     create_yaml_buffer: sourceview5::Buffer,
@@ -432,8 +416,6 @@ pub struct App {
     setup_button: gtk::Button,
     editing_cluster: bool,
     editing_context_name: Option<String>,
-    /// The namespace being renamed while `rename_namespace_dialog` is open.
-    /// `None` means the dialog is closed.
     renaming_namespace: Option<String>,
 }
 

crates/aetheris-app/src/app/dialogs.rs

@@ -273,10 +273,6 @@ pub(super) fn option_row(title: &str, subtitle: &str, icon_name: &str) -> gtk::L
     row
 }
 
-/// Groups `AdwEntryRow`/`AdwPasswordEntryRow`/`AdwSwitchRow` fields into the
-/// standard GNOME HIG "boxed list" form container — each row carries its own
-/// label embedded in the top-left corner, so no separate label widget is
-/// needed above it.
 pub(super) fn entry_list(rows: &[&gtk::Widget]) -> gtk::ListBox {
     let list = gtk::ListBox::new();
     list.add_css_class("boxed-list");

crates/aetheris-app/src/app/methods.rs

@@ -17,11 +17,6 @@ impl App {
         self.root_stack.set_visible_child_name("browser");
     }
 
-    /// Shows the Clusters page for the currently selected project: rebuilds
-    /// the list, kicks off a background summary fetch for any context that
-    /// doesn't have one cached yet, and switches `root_stack`. Shared by
-    /// every path that lands on this page (picking a project, coming back
-    /// from Browser, and after adding/editing a cluster).
     pub(super) fn enter_clusters_page(&mut self, sender: ComponentSender<Self>) {
         self.rebuild_cluster_list();
         self.ensure_cluster_summaries_loading(sender);
@@ -60,9 +55,6 @@ impl App {
         self.rebuild_cluster_list();
     }
 
-    /// Loads the clusters view for whichever project is now selected in
-    /// `self.projects`. Shared by switching projects, deleting the current
-    /// one (falling back to another), and duplicating one.
     pub(super) fn switch_to_project(&mut self, sender: ComponentSender<Self>) {
         if !self
             .visible_contexts()
@@ -110,10 +102,6 @@ impl App {
         }
     }
 
-    /// Columns that make sense to offer/render for the currently selected
-    /// resource kind (e.g. "Status" is only meaningful for resources that
-    /// expose a ready/desired ratio, like Deployments; "CPU"/"Memory" only
-    /// for Pods and Nodes, the only kinds metrics.k8s.io covers).
     pub(super) fn offerable_object_columns(&self) -> Vec<ObjectColumn> {
         offerable_columns_for(self.selected_resource_kind())
     }

crates/aetheris-app/src/app/streams.rs

@@ -269,7 +269,6 @@ impl App {
         }
         session.input_tx = None;
         session.view.reset(true, true);
-        session.view.feed(b"Connecting to pod terminal...\r\n");
 
         let container = selected_log_container(&session.container_dropdown, &session.target)
             .or_else(|| default_terminal_container(&session.target))?;

crates/aetheris-app/src/app/widgets.rs

@@ -36,7 +36,6 @@ pub(super) fn selector_popover(list: &gtk::ListBox) -> gtk::Popover {
     popover
 }
 
-/// A left-aligned icon + label row styled as a flat popover menu item.
 pub(super) fn menu_action_button(icon_name: &str, label: &str) -> gtk::Button {
     let content = gtk::Box::new(gtk::Orientation::Horizontal, 8);
     content.set_margin_top(6);
@@ -448,8 +447,6 @@ fn cluster_state_chip(summary: Option<&ClusterSummaryState>) -> gtk::Label {
     chip
 }
 
-/// "{provider} · {version}" when loaded, dropping either half that's
-/// unavailable; a short status line otherwise.
 fn cluster_subtitle_text(summary: Option<&ClusterSummaryState>) -> String {
     match summary {
         None | Some(ClusterSummaryState::Loading) => String::from("Checking cluster…"),
@@ -579,12 +576,6 @@ pub(super) fn is_cluster_resource(resource: &ResourceKind) -> bool {
     )
 }
 
-/// Columns for the Deployment detail page's "Pods" tab, which always lists
-/// Pods as a compact preview. Pods never have a "Status" ready/desired
-/// ratio, and CPU/Memory depend on metrics-server being installed (often
-/// not, and not worth a column that's blank for everyone when it isn't) —
-/// both are dropped here regardless of which resource kind the main object
-/// list has selected.
 const RELATED_POD_COLUMNS: [ObjectColumn; 3] = [
     ObjectColumn::Namespace,
     ObjectColumn::Api,
@@ -741,8 +732,6 @@ fn summary_sorter(
     })
 }
 
-/// Persists user column resizes: clamps the dragged width back into range
-/// and forwards in-range values to the app for (debounced) saving.
 pub(super) fn connect_object_column_persistence(
     view_column: &gtk::ColumnViewColumn,
     table_column: ObjectTableColumn,
@@ -768,8 +757,6 @@ fn clamp_table_column_width(column: ObjectTableColumn, width: i32) -> i32 {
     }
 }
 
-/// The model item type behind the main object table: an `ObjectSummary`
-/// boxed into a `glib::Object` so it can live in a `gio::ListStore`.
 pub(super) fn boxed_object(object: &ObjectSummary) -> gtk::glib::BoxedAnyObject {
     gtk::glib::BoxedAnyObject::new(object.clone())
 }
@@ -782,10 +769,6 @@ fn list_item_object(
     Some((item, boxed))
 }
 
-/// Cell factory for the object table's "Name" column: optional status chip
-/// plus the object name. The chip's tone and presence change per object, so
-/// the cell content is rebuilt on each bind (only the handful of on-screen
-/// rows ever bind, so this stays cheap regardless of list size).
 pub(super) fn object_name_column_factory() -> gtk::SignalListItemFactory {
     let factory = gtk::SignalListItemFactory::new();
     factory.connect_setup(|_, item| {
@@ -822,9 +805,6 @@ pub(super) fn object_name_column_factory() -> gtk::SignalListItemFactory {
     factory
 }
 
-/// Cell factory for one data column of the object table. Text columns keep
-/// a single label alive and only swap its text on bind; the CPU/Memory
-/// LevelBar cells are rebuilt per bind like the name cell.
 pub(super) fn object_data_column_factory(column: ObjectColumn) -> gtk::SignalListItemFactory {
     let factory = gtk::SignalListItemFactory::new();
     match column {

crates/aetheris-app/src/app/yaml.rs

@@ -30,9 +30,6 @@ pub(super) fn ensure_text_tag(
     }
 }
 
-/// A `sourceview5::View` configured for editing a Kubernetes manifest: line
-/// numbers, current-line highlight, monospace, no wrapping (long lines
-/// scroll horizontally instead of reflowing).
 pub(super) fn build_yaml_view(buffer: &sourceview5::Buffer) -> sourceview5::View {
     let view = sourceview5::View::with_buffer(buffer);
     view.set_show_line_numbers(true);
@@ -235,9 +232,6 @@ fn update_yaml_error_state(buffer: &sourceview5::Buffer, error_label: &gtk::Labe
     error_label.set_label(&format!("Line {line}: {message}"));
 }
 
-/// Returns the (1-indexed) line and message of the first YAML syntax error
-/// in `text`, or `None` if it parses (or is blank — an empty editor isn't
-/// an error yet).
 pub(super) fn yaml_parse_error(text: &str) -> Option<(usize, String)> {
     if text.trim().is_empty() {
         return None;

crates/aetheris-kube/src/cluster.rs

@@ -3,11 +3,6 @@ use anyhow::{Context as AnyhowContext, Result};
 use crate::{ClusterSummary, KubeSession};
 
 impl KubeSession {
-    /// A best-effort snapshot for the Clusters list page. Only the initial
-    /// connectivity check (the version endpoint) is fatal; node listing,
-    /// pod counting and metrics-server queries each degrade to `None`
-    /// independently so a cluster without metrics-server (or with a slow
-    /// node list) still reports as reachable.
     /// Only the version endpoint is queried: it's a discovery call granted
     /// to `system:authenticated` in every stock RBAC setup, unlike listing
     /// Nodes or cluster-wide Pods, which regularly aren't (the same reason

crates/aetheris-kube/src/types.rs

@@ -121,10 +121,7 @@ pub struct ContainerUsage {
     pub memory: String,
 }
 
-/// A best-effort snapshot of a cluster used by the Clusters list page.
-/// Every field beyond connectivity itself is optional: metrics-server may
-/// not be installed, and provider detection is a heuristic guess, not an
-/// authoritative source.
+/// Version/provider data shown on the Clusters list page.
 #[derive(Debug, Clone, PartialEq, Serialize, Deserialize)]
 pub struct ClusterSummary {
     pub version: Option<String>,