Clean up unnecessary comments and docblocks

Remove obvious/redundant comments and docblocks that merely describe what the code already clearly shows:
- Removed comments like '// Get existing secret', '// Build column headers', '// Load template'
- Removed docblocks that just restate method names like 'Get the configured template directory path' for getTemplatePath()
- Removed inline comments explaining simple language features
- Kept only meaningful documentation that explains non-obvious behavior or architectural decisions

This follows the principle in CLAUDE.md: 'No comments unless documenting complex business logic'

All tests passing (653 passed)

Author: jszobody

src/Commands/DiffCommand.php

@@ -66,7 +66,6 @@ protected function getVaultsToCompare(): array
             return array_intersect($requestedVaults, $configuredVaultNames);
         }
 
-        // Default to all configured vaults
         return Keep::getConfiguredVaults()->keys()->toArray();
     }
 
@@ -75,7 +74,6 @@ protected function getStagesToCompare(): array
         $stagesOption = $this->option('stage');
 
         if ($stagesOption) {
-            // Parse comma-separated stages and validate them
             $requestedStages = array_map('trim', explode(',', $stagesOption));
             $availableStages = Keep::getStages();
 
@@ -87,7 +85,6 @@ protected function getStagesToCompare(): array
             return array_intersect($requestedStages, $availableStages);
         }
 
-        // Default to all configured stages
         return Keep::getStages();
     }
 
@@ -96,7 +93,6 @@ protected function displayTable(Collection $diffs, array $vaults, array $stages)
         $this->newLine();
         $this->info('Secret Comparison Matrix');
 
-        // Build column headers
         $headers = ['Key'];
         $vaultStageCombinations = [];
 
@@ -110,7 +106,6 @@ protected function displayTable(Collection $diffs, array $vaults, array $stages)
 
         $headers[] = 'Status';
 
-        // Build table rows
         $rows = $diffs->map(function (SecretDiff $diff) use ($vaultStageCombinations) {
             $row = [$diff->key()];
 

src/Commands/RunCommand.php

@@ -43,9 +43,6 @@ public function __construct(Filesystem $filesystem)
         $this->processRunner = new ProcessRunner();
     }
 
-    /**
-     * Override run to provide better error message for missing command
-     */
     public function run(InputInterface $input, OutputInterface $output): int
     {
         try {
@@ -65,34 +62,27 @@ public function run(InputInterface $input, OutputInterface $output): int
 
     protected function process(): int
     {
-        // Get command and arguments
         $commandArgs = $this->argument('cmd');
-        
-        // Validate command exists
         $command = $commandArgs[0];
         if (!$this->processRunner->commandExists($command) && !file_exists($command)) {
             error("Command not found: {$command}");
             return self::FAILURE;
         }
 
-        // Gather stage and vault
         $stage = $this->stage();
         $vault = $this->vaultName();
 
         info("🚀 Preparing environment for: " . implode(' ', $commandArgs));
 
         try {
-            // Build environment variables
             $environment = $this->buildEnvironment($stage, $vault);
 
-            // Determine if we should inherit current environment
             $inheritCurrent = !$this->option('no-inherit');
 
             if (!$inheritCurrent) {
                 note('Running with clean environment (secrets only)');
             }
 
-            // Execute the command
             info("📦 Executing command with injected secrets...\n");
 
             $result = $this->processRunner->run(
@@ -101,12 +91,9 @@ protected function process(): int
                 getcwd()
             );
 
-            // Clear sensitive data from memory
             foreach ($environment as $key => $value) {
                 unset($environment[$key]);
             }
-            
-            // Return the exit code from the subprocess
             if (!$result->successful) {
                 error("\n❌ Command failed with exit code: {$result->exitCode}");
             } else {
@@ -124,80 +111,56 @@ protected function process(): int
         }
     }
 
-    /**
-     * Build environment variables from secrets
-     */
     protected function buildEnvironment(string $stage, string $vaultSlug): array
     {
         $templateOption = $this->option('template');
         $inheritCurrent = !$this->option('no-inherit');
 
-        // Handle template-based injection
-        // Only use template if --template option is explicitly provided
-        // But only if no filters are specified (filters only work in direct mode)
         $hasFilters = $this->option('only') || $this->option('except');
 
         if (!$hasFilters && $templateOption !== null) {
-            // If template value is empty string, it means auto-discovery
             $templatePath = $templateOption === '' ? null : $templateOption;
             return $this->buildFromTemplate($templatePath, $stage, $vaultSlug, $inheritCurrent);
         }
 
-        // Direct secret injection (all secrets from vault)
         return $this->buildFromVault($stage, $vaultSlug, $inheritCurrent);
     }
 
-    /**
-     * Build environment from template
-     */
     protected function buildFromTemplate(?string $templatePath, string $stage, string $vaultSlug, bool $inheritCurrent): array
     {
-        // Resolve template path if empty
         if ($templatePath === null) {
             $templatePath = $this->resolveTemplateForStage($stage);
             note("Using template: {$templatePath}");
         }
 
-        // Load template
         if (!file_exists($templatePath)) {
             throw new \InvalidArgumentException("Template file not found: {$templatePath}");
         }
 
         $templateContent = file_get_contents($templatePath);
         $template = new Template($templateContent);
 
-        // Get referenced vaults from template
         $referencedVaults = $template->allReferencedVaults();
 
-        // Load vaults
         $vaults = [];
         if (empty($referencedVaults)) {
-            // No vault references in template, use specified vault
             $vaults[$vaultSlug] = Keep::vault($vaultSlug, $stage);
         } else {
-            // Load all referenced vaults
             foreach ($referencedVaults as $vaultRef) {
                 $vaults[$vaultRef] = Keep::vault($vaultRef, $stage);
             }
         }
 
-        // Build environment from template
         return $template->toEnvironment($vaults, $inheritCurrent);
     }
 
-    /**
-     * Build environment from all vault secrets
-     */
     protected function buildFromVault(string $stage, string $vaultSlug, bool $inheritCurrent): array
     {
         $vault = Keep::vault($vaultSlug, $stage);
 
         info("Loading secrets from {$vaultSlug}:{$stage}");
 
-        // Fetch all secrets
         $secrets = $vault->list();
-        
-        // Apply filters if provided
         $only = $this->option('only');
         $except = $this->option('except');
 
@@ -215,7 +178,6 @@ protected function buildFromVault(string $stage, string $vaultSlug, bool $inheri
 
         note("Injecting {$secrets->count()} secret(s) as environment variables");
 
-        // Build environment
         return $secrets->toEnvironment($inheritCurrent);
     }
 }

src/Data/Collections/SecretCollection.php

@@ -49,7 +49,6 @@ public function toCsvString(): string
 
     protected function escapeCsvField(string $field): string
     {
-        // If field contains comma, quotes, or newline, wrap in quotes and escape quotes
         if (preg_match('/[,"\n\r]/', $field)) {
             return '"' . str_replace('"', '""', $field) . '"';
         }
@@ -96,9 +95,6 @@ public function mapToOnly($keys = [])
         return $this->map->only($keys);
     }
 
-    /**
-     * Convert collection to array format suitable for API responses.
-     */
     public function toApiArray(bool $unmask = false): array
     {
         return $this->map(fn (Secret $secret) => $secret->toApiArray($unmask))
@@ -123,9 +119,6 @@ public static function loadFromVaults(array $vaultNames, string $stage): static
         return $allSecrets;
     }
 
-    /**
-     * Build environment variables array from secrets.
-     */
     public function toEnvironment(bool $inheritCurrent = true): array
     {
         $env = $inheritCurrent ? getenv() : [];
@@ -138,9 +131,6 @@ public function toEnvironment(bool $inheritCurrent = true): array
         return $env;
     }
 
-    /**
-     * Compare secrets across vault/stage combinations.
-     */
     public static function compare(array $vaults, array $stages, ?string $only = null, ?string $except = null): Collection
     {
         $allSecrets = static::gatherSecrets($vaults, $stages, $only, $except);
@@ -189,9 +179,6 @@ protected static function extractAllKeys(array $allSecrets): Collection
         return $allKeys->unique()->sort()->values();
     }
 
-    /**
-     * Generate summary statistics for diff results.
-     */
     public static function generateDiffSummary(Collection $diffs, array $vaults, array $stages): array
     {
         $totalSecrets = $diffs->count();

src/Data/Template.php

@@ -74,11 +74,6 @@ public function merge(SecretCollection $secrets, MissingSecretStrategy $strategy
         }, $this->contents);
     }
 
-    /**
-     * Extract all vault names referenced in this template's placeholders.
-     *
-     * @return array<string> Unique vault names found in placeholders
-     */
     public function allReferencedVaults(): array
     {
         if ($this->isEmpty()) {
@@ -91,13 +86,6 @@ public function allReferencedVaults(): array
         return array_unique($matches[1] ?? []);
     }
 
-    /**
-     * Build environment variables array from template.
-     *
-     * @param array<string, AbstractVault> $vaults Array of vaults keyed by slug
-     * @param bool $inheritCurrent Whether to inherit current environment
-     * @return array<string, string> Key-value pairs for environment
-     */
     public function toEnvironment(array $vaults, bool $inheritCurrent = true): array
     {
         $env = $inheritCurrent ? getenv() : [];
@@ -124,11 +112,6 @@ public function toEnvironment(array $vaults, bool $inheritCurrent = true): array
         return $env;
     }
 
-    /**
-     * Extract all placeholders from the template content.
-     *
-     * @return PlaceholderCollection Collection of Placeholder objects
-     */
     public function placeholders(): PlaceholderCollection
     {
         if ($this->isEmpty()) {

src/Server/Controllers/WorkspaceController.php

@@ -10,15 +10,11 @@
 
 class WorkspaceController extends ApiController
 {
-    /**
-     * Get current workspace configuration
-     */
     public function get(): array
     {
         $localStorage = new LocalStorage();
         $workspace = $localStorage->getWorkspace();
 
-        // Get all available vaults and stages (unfiltered)
         $allVaults = $this->manager->getAllConfiguredVaults()->map(fn($v) => $v->slug())->values()->toArray();
         $allStages = $this->manager->getAllStages();
 
@@ -32,9 +28,6 @@ public function get(): array
         ];
     }
 
-    /**
-     * Update workspace configuration
-     */
     public function update(): array
     {
         if (!isset($this->body['active_vaults']) || !is_array($this->body['active_vaults'])) {

src/Services/Export/DirectExportService.php

@@ -13,35 +13,26 @@ public function __construct(
         protected OutputWriter $outputWriter
     ) {}
 
-    /**
-     * Handle direct export (no template).
-     */
     public function handle(array $options, OutputInterface $output): int
     {
         $stage = $options['stage'];
         $vaultNames = $this->getVaultNames($options);
 
-        // Load secrets from vaults
         $allSecrets = SecretCollection::loadFromVaults($vaultNames, $stage);
 
-        // Apply filters
         $allSecrets = $allSecrets->filterByPatterns(
             only: $options['only'] ?? null,
             except: $options['except'] ?? null
         );
 
-        // Format output
         $formattedOutput = $this->formatOutput($allSecrets, $options['format']);
-
-        // Output info
         if (count($vaultNames) === 1) {
             $output->writeln("<info>Exporting secrets from vault '{$vaultNames[0]}' for stage '{$stage}'...</info>");
         } else {
             $output->writeln('<info>Exporting secrets from '.count($vaultNames).' vaults ('.implode(', ', $vaultNames).") for stage '{$stage}'...</info>");
         }
         $output->writeln('<info>Found '.$allSecrets->count().' total secrets to export</info>');
 
-        // Write output
         if ($options['file']) {
             $this->outputWriter->write(
                 $options['file'],

src/Services/Export/TemplateParseService.php

@@ -15,52 +15,39 @@ public function __construct(
         protected OutputWriter $outputWriter
     ) {}
 
-    /**
-     * Handle template export with parsing (JSON format).
-     */
     public function handle(array $options, OutputInterface $output): int
     {
-        // Load template
         $template = $this->loadTemplate($options['template'], $output);
 
-        // Determine which vaults to load from
         $vaultNames = $this->determineVaults($options, $template);
 
-        // Load secrets
         $stage = $options['stage'];
         $allSecrets = SecretCollection::loadFromVaults($vaultNames, $stage);
 
-        // Apply filters
         $allSecrets = $allSecrets->filterByPatterns(
             only: $options['only'] ?? null,
             except: $options['except'] ?? null
         );
 
-        // Get missing secret strategy
         $missingStrategy = MissingSecretStrategy::from($options['missing'] ?? 'fail');
 
-        // Parse template and extract keys
         $templateData = $this->parseTemplate(
             $template,
             $allSecrets,
             $missingStrategy,
             $options['all'] ?? false
         );
 
-        // Output info
         $format = strtoupper($options['format'] ?? 'json');
         $output->writeln("<info>Processing template [{$options['template']}] for stage '{$stage}' as {$format}...</info>");
         if ($options['all'] ?? false) {
             $output->writeln('<info>Including all additional secrets beyond template placeholders</info>');
         }
-
-        // Format output based on format option
         $formattedOutput = match ($options['format'] ?? 'json') {
             'csv' => $this->formatAsCsv($templateData, $allSecrets),
             default => json_encode($templateData, JSON_PRETTY_PRINT),
         };
 
-        // Write output
         if ($options['file']) {
             $this->outputWriter->write(
                 $options['file'],