Allow hiding user profile avatars

Author: ciembor

lib/polish_open_source_rank/contexts/publication/infrastructure/sqlite/sqlite_profile_read_model.rb

@@ -116,7 +116,8 @@ def organization_repository_profile(platform, full_name, period_start:)
             def fetch_user_profile(platform, login, period_start)
               database.fetch_all(<<~SQL, [period_start, platform, login]).first
                 SELECT users.platform, users.github_id AS source_id, users.github_id, users.login, users.name, users.location_raw, users.city,
-                       users.country, users.email, users.homepage, users.html_url, users.avatar_url,
+                       users.country, users.email, users.homepage, users.html_url,
+                       CASE users.avatar_hidden WHEN 1 THEN NULL ELSE users.avatar_url END AS avatar_url,
                        stats.period_start, stats.public_repo_count, stats.total_stars, stats.monthly_stars_delta,
                        stats.merged_pull_requests_count
                 FROM users

lib/polish_open_source_rank/infrastructure/platform_schema_migration.rb

@@ -160,6 +160,7 @@ def ensure_current_columns
         if table_columns('user_monthly_stats').include?('public_activity_count')
           rebuild_user_monthly_stats_without_public_activity_count
         end
+        add_column_unless_exists('users', 'avatar_hidden INTEGER NOT NULL DEFAULT 0')
         add_column_unless_exists('user_monthly_stats', 'merged_pull_requests_count INTEGER NOT NULL DEFAULT 0')
         add_column_unless_exists('organization_monthly_stats', 'merged_pull_requests_count INTEGER NOT NULL DEFAULT 0')
         add_column_unless_exists('organization_monthly_stats', 'members_count INTEGER NOT NULL DEFAULT 0')

lib/polish_open_source_rank/infrastructure/sqlite_schema.sql

@@ -67,6 +67,7 @@ CREATE TABLE IF NOT EXISTS users (
   homepage TEXT,
   html_url TEXT NOT NULL,
   avatar_url TEXT,
+  avatar_hidden INTEGER NOT NULL DEFAULT 0,
   updated_at TEXT NOT NULL,
   PRIMARY KEY(platform, github_id),
   UNIQUE(platform, login)

spec/polish_open_source_rank/contexts/publication/infrastructure/sqlite/sqlite_profile_read_model_spec.rb

@@ -112,6 +112,14 @@
     expect(read_model.repository_profile('github', 'alice/missing', period_start: period)).to be_nil
   end
 
+  it 'hides user avatars flagged in the database' do
+    seed_user_record(id: 1, login: 'alice', avatar_url: 'https://avatars.example/alice.png', avatar_hidden: true)
+
+    profile = read_model.user_profile('github', 'alice', period_start: nil)
+
+    expect(profile).to include(login: 'alice', avatar_url: nil)
+  end
+
   it 'lists every public user identity for sitemap rendering' do
     seed_user_record(id: 1, login: 'alice')
     seed_user_record(id: 2, login: 'bob')
@@ -184,14 +192,24 @@ def seed_user(id:, login:, total_stars:, period_start: period, city: 'Kraków',
                                       '2026-05-01T00:10:00Z'])
   end
 
-  def seed_user_record(id:, login:, city: nil, country: nil)
+  def seed_user_record(id:, login:, city: nil, country: nil, avatar_url: nil, avatar_hidden: false)
     database.execute(
       <<~SQL.strip,
         INSERT OR IGNORE INTO users(
-          platform, github_id, login, city, country, html_url, updated_at
-        ) VALUES (?, ?, ?, ?, ?, ?, ?)
+          platform, github_id, login, city, country, html_url, avatar_url, avatar_hidden, updated_at
+        ) VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?)
       SQL
-      ['github', id, login, city, country, "https://github.com/#{login}", '2026-05-01T00:01:00Z']
+      [
+        'github',
+        id,
+        login,
+        city,
+        country,
+        "https://github.com/#{login}",
+        avatar_url,
+        avatar_hidden ? 1 : 0,
+        '2026-05-01T00:01:00Z'
+      ]
     )
   end
 

spec/polish_open_source_rank/infrastructure/platform_schema_migration_spec.rb

@@ -14,7 +14,7 @@
     expect(database.fetch_value(package_table_sql('package_crawl_runs'))).to eq(1)
     expect(database.fetch_value(package_table_sql('registry_package_snapshots'))).to eq(1)
     expect(database.fetch_value(package_table_sql('public_snapshot_publications'))).to eq(1)
-    expect(database.table_info('users').map { |column| column.fetch('name') }).to include('platform')
+    expect(database.table_info('users').map { |column| column.fetch('name') }).to include('platform', 'avatar_hidden')
     expect(database.table_info('user_monthly_stats').map { |column| column.fetch('name') })
       .to include('merged_pull_requests_count')
     expect(database.table_info('organization_monthly_stats').map { |column| column.fetch('name') })
@@ -119,6 +119,7 @@
 
     expect(database.table_info('user_monthly_stats').map { |column| column.fetch('name') })
       .to include('merged_pull_requests_count')
+    expect(database.table_info('users').map { |column| column.fetch('name') }).to include('avatar_hidden')
     expect(database.table_info('organization_monthly_stats').map { |column| column.fetch('name') })
       .to include('merged_pull_requests_count', 'members_count')
   end