Increase test coverage, bug fixes in various commands

Author: andrew

CHANGELOG.md

@@ -3,6 +3,9 @@
 ## [0.6.1] - 2026-01-05
 
 - Fix `stats` command crash on most changed dependencies query
+- Fix `search` command SQL alias error when displaying results
+- Fix `blame` and `stale` commands eager loading error
+- Fix `list` command returning empty output when ecosystem filter matches nothing
 
 ## [0.6.0] - 2026-01-05
 

lib/git/pkgs/commands/blame.rb

@@ -24,12 +24,17 @@ def run
           error "No analysis found for branch '#{branch_name}'. Run 'git pkgs init' first." unless branch&.last_analyzed_sha
 
           current_commit = Models::Commit.first(sha: branch.last_analyzed_sha)
-          snapshots = current_commit&.dependency_snapshots&.eager(:manifest) || []
+
+          return empty_result("No dependencies found") unless current_commit
+
+          snapshots = current_commit.dependency_snapshots_dataset.eager(:manifest)
 
           if @options[:ecosystem]
             snapshots = snapshots.where(ecosystem: @options[:ecosystem])
           end
 
+          snapshots = snapshots.all
+
           if snapshots.empty?
             empty_result "No dependencies found"
             return

lib/git/pkgs/commands/list.rb

@@ -24,11 +24,6 @@ def run
 
           deps = compute_dependencies_at_commit(target_commit, repo)
 
-          if deps.empty?
-            empty_result "No dependencies found"
-            return
-          end
-
           # Apply filters
           if @options[:ecosystem]
             deps = deps.select { |d| d[:ecosystem] == @options[:ecosystem] }
@@ -38,6 +33,11 @@ def run
             deps = deps.select { |d| d[:dependency_type] == @options[:type] }
           end
 
+          if deps.empty?
+            empty_result "No dependencies found"
+            return
+          end
+
           if @options[:format] == "json"
             require "json"
             puts JSON.pretty_generate(deps)

lib/git/pkgs/commands/search.rb

@@ -72,7 +72,7 @@ def output_json(matches, pattern)
               .where(name: name, ecosystem: platform)
               .eager(:commit)
               .association_join(:commit)
-              .order(Sequel[:commits][:committed_at])
+              .order(Sequel[:commit][:committed_at])
               .all
 
             first = changes.first
@@ -98,7 +98,7 @@ def dependency_summary(name, platform)
             .where(name: name, ecosystem: platform)
             .eager(:commit)
             .association_join(:commit)
-            .order(Sequel[:commits][:committed_at])
+            .order(Sequel[:commit][:committed_at])
             .all
 
           first = changes.first

lib/git/pkgs/commands/stale.rb

@@ -23,12 +23,17 @@ def run
           error "No analysis found for branch '#{branch_name}'. Run 'git pkgs init' first." unless branch&.last_analyzed_sha
 
           current_commit = Models::Commit.first(sha: branch.last_analyzed_sha)
-          snapshots = current_commit&.dependency_snapshots&.eager(:manifest) || []
+
+          return empty_result("No dependencies found") unless current_commit
+
+          snapshots = current_commit.dependency_snapshots_dataset.eager(:manifest)
 
           if @options[:ecosystem]
             snapshots = snapshots.where(ecosystem: @options[:ecosystem])
           end
 
+          snapshots = snapshots.all
+
           if snapshots.empty?
             empty_result "No dependencies found"
             return

lib/git/pkgs/database.rb

@@ -85,14 +85,18 @@ def self.refresh_models
           Git::Pkgs::Models::DependencySnapshot
         ].each do |model|
           model.dataset = @db[model.table_name]
-          # Clear cached association datasets and loaders that may reference old db
+          # Clear all cached association data that may reference old db
           model.association_reflections.each_value do |reflection|
+            reflection.delete(:_dataset)
+            reflection.delete(:associated_eager_dataset)
+            reflection.delete(:placeholder_eager_loader)
+            reflection.delete(:placeholder_eager_graph_loader)
             if reflection[:cache]
-              reflection[:cache].delete(:_dataset)
-              reflection[:cache].delete(:associated_eager_dataset)
-              reflection[:cache].delete(:placeholder_eager_loader)
+              reflection[:cache].clear
             end
           end
+          # Clear model instance caches
+          model.instance_variable_set(:@columns, nil) if model.instance_variable_defined?(:@columns)
         rescue Sequel::Error
           # Table may not exist yet
         end