progress

Author: psteinroe

crates/pgt_cli/src/commands/check.rs

@@ -11,16 +11,13 @@ use pgt_fs::FileSystem;
 use pgt_workspace::DynRef;
 use std::ffi::OsString;
 
-#[allow(dead_code)]
 pub struct CheckArgs {
     pub configuration: Option<PartialConfiguration>,
     pub paths: Vec<OsString>,
     pub stdin_file_path: Option<String>,
     pub staged: bool,
     pub changed: bool,
     pub since: Option<String>,
-    pub apply: bool,
-    pub apply_unsafe: bool,
 }
 
 pub fn check(

crates/pgt_cli/src/execute/process_file/workspace_file.rs

@@ -9,8 +9,11 @@ use std::path::{Path, PathBuf};
 /// Small wrapper that holds information and operations around the current processed file
 pub(crate) struct WorkspaceFile<'ctx, 'app> {
     guard: FileGuard<'app, dyn Workspace + 'ctx>,
+    /// File handle for the underlying filesystem entry, if backed by a real file.
+    /// Not present for stdin execution where content is provided as a temporary buffer.
+    /// Currently unused but will be needed when autofix/write operations are implemented.
     #[allow(dead_code)]
-    file: Option<Box<dyn File>>, // only present when backed by a real fs entry
+    file: Option<Box<dyn File>>,
     pub(crate) path: PathBuf,
 }
 

crates/pgt_cli/src/execute/walk.rs

@@ -161,8 +161,10 @@ fn traverse_inputs(
 
 struct DiagnosticsPrinter<'ctx> {
     _config: &'ctx ExecutionConfig,
+    /// The maximum number of diagnostics the console thread is allowed to print
     max_diagnostics: u32,
     remaining_diagnostics: AtomicU32,
+    /// Count of diagnostics that exceeded max_diagnostics and weren't printed
     not_printed_diagnostics: AtomicU32,
     printed_diagnostics: AtomicU32,
     total_skipped_suggested_fixes: AtomicU32,

crates/pgt_cli/src/lib.rs

@@ -87,8 +87,6 @@ impl<'app> CliSession<'app> {
                     staged,
                     changed,
                     since,
-                    apply: false,
-                    apply_unsafe: false,
                 },
             ),
             PgtCommand::Clean => commands::clean::clean(self),

crates/pgt_cli/src/reporter/gitlab.rs

@@ -139,12 +139,19 @@ impl Display for GitLabDiagnostics<'_> {
     }
 }
 
+/// An entry in the GitLab Code Quality report.
+/// See https://docs.gitlab.com/ee/ci/testing/code_quality.html#implement-a-custom-tool
 #[derive(Serialize)]
 pub struct GitLabDiagnostic<'a> {
+    /// A description of the code quality violation.
     description: String,
+    /// A unique name representing the static analysis check that emitted this issue.
     check_name: &'a str,
+    /// A unique fingerprint to identify the code quality violation. For example, an MD5 hash.
     fingerprint: String,
+    /// A severity string (can be info, minor, major, critical, or blocker).
     severity: &'a str,
+    /// The location where the code quality violation occurred.
     location: Location,
 }