source_monitor 0.7.0 → 0.8.0

data/.vbw-planning/codebase/TESTING.md

@@ -1,154 +0,0 @@
-# Testing
-
-## Framework & Tools
-
-- **Test Framework**: Minitest (Rails default)
-- **System Tests**: Capybara + Selenium WebDriver (Chrome)
-- **HTTP Mocking**: WebMock (disables all external connections) + VCR (recorded cassettes)
-- **Coverage**: SimpleCov with branch coverage enabled
-- **Profiling**: test-prof (TagProf, EventProf) + StackProf
-- **Parallelization**: Built-in Rails parallel testing (configurable workers via `SOURCE_MONITOR_TEST_WORKERS`)
-
-## Test Infrastructure
-
-### Test Helper (`test/test_helper.rb`)
-- Loads SimpleCov in CI or when `COVERAGE` env is set
-- Configures Rails test environment pointing to dummy app
-- Sets migration paths to include both engine and dummy migrations
-- Uses `:test` ActiveJob queue adapter by default
-- Fixtures loaded from `test/fixtures/`
-- VCR cassettes stored in `test/vcr_cassettes/`
-- WebMock allows localhost only
-- Random test ordering enabled
-- Configuration reset in every test via `SourceMonitor.reset_configuration!`
-
-### test-prof Integration (`test/test_prof.rb`)
-- `TestProf::BeforeAll::Minitest` for `before_all` blocks (shared expensive setup)
-- `SourceMonitor::TestProfSupport::SetupOnce` -- `setup_once` alias for `before_all`
-- `SourceMonitor::TestProfSupport::InlineJobs` -- `with_inline_jobs` helper
-- `TestProf::MinitestSample` -- SAMPLE/SAMPLE_GROUPS env var support for focused runs
-
-### Shared Test Helpers
-- `create_source!(attributes = {})` -- factory method for creating test sources
-- `with_queue_adapter(adapter)` -- temporarily switch ActiveJob adapter
-
-## Test Categories
-
-| Category | File Count | Path | Purpose |
-|----------|-----------|------|---------|
-| Unit (lib) | ~75 | `test/lib/source_monitor/` | Lib module tests |
-| Model | ~10 | `test/models/source_monitor/` | Model validation, scopes, behavior |
-| System | 6 | `test/system/` | Browser-driven end-to-end tests |
-| Integration | 4 | `test/integration/` | Engine mounting, navigation, packaging |
-| Example | 4 | `test/examples/` | Template and adapter examples |
-| Task | 2 | `test/tasks/` | Rake task tests |
-| Mailer | 1 | `test/mailers/` | Application mailer test |
-| Module | 1 | `test/source_monitor_test.rb` | Top-level module tests |
-
-**Total: ~124 test files**
-
-## Test Structure
-
-Tests mirror the source directory structure:
-
-```
-test/lib/source_monitor/
-  configuration_test.rb
-  feedjira_configuration_test.rb
-  instrumentation_test.rb
-  health/
-    health_module_test.rb
-    source_health_check_test.rb
-    source_health_monitor_test.rb
-    source_health_reset_test.rb
-  pagination/
-    paginator_test.rb
-  scraping/
-    bulk_result_presenter_test.rb
-    bulk_source_scraper_test.rb
-    enqueuer_test.rb
-    item_scraper_test.rb
-    item_scraper/
-      adapter_resolver_test.rb
-      persistence_test.rb
-    scheduler_test.rb
-    state_test.rb
-  setup/
-    bundle_installer_test.rb
-    cli_test.rb
-    dependency_checker_test.rb
-    detectors_test.rb
-    gemfile_editor_test.rb
-    initializer_patcher_test.rb
-    install_generator_test.rb
-    migration_installer_test.rb
-    node_installer_test.rb
-    prompter_test.rb
-    requirements_test.rb
-    workflow_test.rb
-    verification/
-      action_cable_verifier_test.rb
-      printer_test.rb
-      runner_test.rb
-      solid_queue_verifier_test.rb
-      telemetry_logger_test.rb
-  turbo_streams/
-    stream_responder_test.rb
-  release/
-    runner_test.rb
-```
-
-## Dummy Application (`test/dummy/`)
-
-Full Rails application used as the host app for testing:
-- PostgreSQL database (`config/database.yml`)
-- Solid Queue configuration (`config/solid_queue.yml`, `config/queue.yml`)
-- Solid Cable configuration (`config/cable.yml`)
-- Mission Control integration
-- Custom STI model: `SourceMonitor::SponsoredSource` (tests model extensions)
-- Extension concern: `DummySourceMonitor::SourceExtensions`
-- `User` model for testing authentication
-- Source Monitor initializer exercising the full configuration API
-
-## CI Pipeline
-
-### Test Job
-1. Sets up PostgreSQL 15 service container
-2. Installs Ruby 3.4.4, Node 20
-3. Builds frontend assets (`npm run build`)
-4. Creates and migrates test database
-5. Runs full test suite with coverage (`bin/test-coverage`)
-6. Enforces diff coverage (`bin/check-diff-coverage`)
-7. Uploads coverage artifact
-8. Captures system test screenshots on failure
-
-### Release Verification Job
-- Depends on lint, security, test jobs
-- Runs `test/integration/release_packaging_test.rb` specifically
-- Enforces diff coverage again
-
-### Profiling Job (Nightly)
-- Runs on schedule (`cron: "30 6 * * *"`)
-- TagProf by type
-- EventProf on `sql.active_record`
-- StackProf on integration tests
-- Enforces profiling guardrails (`bin/check-test-prof-metrics`)
-- Uploads profiling artifacts
-
-## Coverage
-
-- Branch coverage enabled
-- `refuse_coverage_drop :line` prevents regressions
-- Coverage baseline tracked in `config/coverage_baseline.json` (lists uncovered lines per file)
-- Diff coverage enforcement ensures new code is tested
-- `SOURCE_MONITOR_SKIP_COVERAGE` env var to disable coverage collection
-- `# :nocov:` annotations used for defensive/fallback code paths
-
-## Notable Testing Patterns
-
-- **Configuration Reset**: Every test resets `SourceMonitor.reset_configuration!` in `setup`
-- **WebMock**: All external HTTP disabled; tests use stubs or VCR cassettes
-- **Job Testing**: Tests use `:test` adapter; `with_inline_jobs` for synchronous execution
-- **Parallel Safety**: Tests designed for parallel execution; use `SecureRandom.hex` for unique fixtures
-- **Factory Method**: `create_source!` with `save!(validate: false)` for flexible test data
-- **Setup Verification Tests**: `bin/check-setup-tests` ensures all setup files have corresponding tests

data/.vbw-planning/config.json

@@ -1,53 +0,0 @@
-{
-  "effort": "thorough",
-  "autonomy": "standard",
-  "auto_commit": true,
-  "planning_tracking": "manual",
-  "auto_push": "never",
-  "verification_tier": "standard",
-  "skill_suggestions": true,
-  "auto_install_skills": false,
-  "discovery_questions": true,
-  "context_compiler": true,
-  "visual_format": "unicode",
-  "max_tasks_per_plan": 5,
-  "prefer_teams": "always",
-  "branch_per_milestone": false,
-  "plain_summary": true,
-  "active_profile": "default",
-  "custom_profiles": {},
-  "model_profile": "quality",
-  "model_overrides": {},
-  "agent_max_turns": {
-    "scout": 15,
-    "qa": 25,
-    "architect": 30,
-    "debugger": 80,
-    "lead": 50,
-    "dev": 75
-  },
-  "qa_skip_agents": [
-    "docs"
-  ],
-  "v3_delta_context": false,
-  "v3_context_cache": false,
-  "v3_plan_research_persist": false,
-  "v3_metrics": false,
-  "v3_contract_lite": false,
-  "v3_lock_lite": false,
-  "v3_validation_gates": false,
-  "v3_smart_routing": false,
-  "v3_event_log": false,
-  "v3_schema_validation": false,
-  "v3_snapshot_resume": false,
-  "v3_lease_locks": false,
-  "v3_event_recovery": false,
-  "v3_monorepo_routing": false,
-  "v2_hard_contracts": false,
-  "v2_hard_gates": false,
-  "v2_typed_protocol": false,
-  "v2_role_isolation": false,
-  "v2_two_phase_completion": false,
-  "v2_token_budgets": false,
-  "compaction_trigger": 130000
-}

data/.vbw-planning/discovery.json

@@ -1,26 +0,0 @@
-{
-  "answered": [
-    {
-      "question": "What matters most in the conventions cleanup?",
-      "answer": "All of the above: Model conventions, Controller patterns, Dead code removal",
-      "category": "scope",
-      "phase": "4",
-      "date": "2026-02-10"
-    },
-    {
-      "question": "How should we handle convention violations that would change public API behavior?",
-      "answer": "Fix everything -- rename/restructure even if it changes method signatures or route patterns",
-      "category": "api-policy",
-      "phase": "4",
-      "date": "2026-02-10"
-    }
-  ],
-  "inferred": [
-    "User wants comprehensive cleanup, not surface-level",
-    "Public API changes are acceptable for convention alignment",
-    "Tests should be updated to match any changes, not removed",
-    "All layers: models, controllers, services, dead code",
-    "Generator should be maximally helpful -- create files rather than just warn",
-    "Target Rails 8 defaults (queue.yml) rather than supporting legacy naming"
-  ]
-}

data/.vbw-planning/milestones/default/ROADMAP.md

@@ -1,115 +0,0 @@
-<!-- VBW ROADMAP TEMPLATE (ARTF-07) -- Phase-based project roadmap -->
-<!-- Created by Architect agent during /vbw scope -->
-
-# SourceMonitor Roadmap
-
-## Overview
-
-This roadmap focuses on stabilizing and improving the existing SourceMonitor codebase through test coverage improvements and refactoring for maintainability. 4 phases progressing from analysis to coverage to refactoring to cleanup.
-
-## Phases
-
-- [x] Phase 1: Coverage Analysis & Quick Wins
-- [x] Phase 2: Critical Path Test Coverage
-- [x] Phase 3: Large File Refactoring
-- [x] Phase 4: Code Quality & Conventions Cleanup
-
-## Phase Details
-
-### Phase 1: Coverage Analysis & Quick Wins
-
-**Goal:** Analyze the coverage baseline, identify the highest-impact gaps, and close easy coverage wins (frozen_string_literal, small utility classes).
-**Depends on:** None
-
-**Requirements:**
-- REQ-13: Frozen string literal consistency
-- REQ-14: RuboCop audit
-
-**Success Criteria:**
-1. All Ruby files have frozen_string_literal: true
-2. Zero RuboCop violations against omakase ruleset
-3. Coverage baseline shrinks by at least 10%
-
-**Plans:**
-- [x] Plan 01: frozen-string-literal-audit (5f02db8)
-- [x] Plan 02: rubocop-audit-and-fix (no changes needed)
-
-### Phase 2: Critical Path Test Coverage
-
-**Goal:** Close the major test coverage gaps in the most critical business logic -- feed fetching, item creation, configuration, dashboard queries, and broadcasting.
-**Depends on:** Phase 1
-
-**Requirements:**
-- REQ-01: FeedFetcher coverage
-- REQ-02: ItemCreator coverage
-- REQ-03: Configuration coverage
-- REQ-04: Dashboard::Queries coverage
-- REQ-05: Broadcaster coverage
-- REQ-06: BulkSourceScraper coverage
-- REQ-07: SourcesIndexMetrics coverage
-
-**Success Criteria:**
-1. Coverage baseline shrinks by at least 50% from original
-2. All critical path files have branch coverage above 80%
-3. CI pipeline passes with no regressions
-
-**Plans:**
-- [x] Plan 01: feed-fetcher-tests (8d4e8d3)
-- [x] Plan 02: item-creator-tests (ce8ede4)
-- [x] Plan 03: configuration-tests (66b8df2)
-- [x] Plan 04: dashboard-and-analytics-tests (a8f2611, 2e50580)
-- [x] Plan 05: scraping-and-broadcasting-tests (e497891, 66b8df2)
-
-### Phase 3: Large File Refactoring
-
-**Goal:** Break down the three largest files (FeedFetcher, Configuration, ImportSessionsController) into focused, single-responsibility modules while maintaining all existing tests.
-**Depends on:** Phase 2
-
-**Requirements:**
-- REQ-08: Extract FeedFetcher
-- REQ-09: Extract Configuration
-- REQ-10: Extract ImportSessionsController
-- REQ-11: Fix LogEntry table name
-- REQ-12: Replace eager requires with autoloading
-
-**Success Criteria:**
-1. No single file exceeds 300 lines
-2. All existing tests pass without modification (or with minimal adapter changes)
-3. Public API remains unchanged
-
-**Plans:**
-- [x] Plan 01: extract-feed-fetcher (2f00274)
-- [x] Plan 02: extract-configuration-settings (ab823a3)
-- [x] Plan 03: extract-import-sessions-controller (9dce996)
-- [x] Plan 04: fix-log-entry-and-autoloading (fb99d3d)
-
-### Phase 4: Code Quality & Conventions Cleanup
-
-**Goal:** Final pass to ensure all code follows Rails best practices and conventions, clean up any remaining debt.
-**Depends on:** Phase 3
-
-**Requirements:**
-- REQ-15: Rails conventions audit
-
-**Success Criteria:**
-1. All models, controllers, and service objects follow established conventions
-2. No RuboCop violations
-3. Coverage baseline is at least 60% smaller than original
-4. CI pipeline fully green
-
-**Plans:**
-- [x] Plan 01: conventions-audit (44fe6b6, c30a503, f070ea6, 78600b5, ec67c65)
-- [x] Plan 02: item-creator-extraction (3d33a71)
-- [x] Plan 03: final-verification (cc2e538)
-
-## Progress
-
-| Phase | Plans Complete | Status | Completed |
-|-------|---------------|--------|-----------|
-| 1 - Coverage Analysis & Quick Wins | 2/2 | complete | 2026-02-09 |
-| 2 - Critical Path Test Coverage | 5/5 | complete | 2026-02-09 |
-| 3 - Large File Refactoring | 4/4 | complete | 2026-02-10 |
-| 4 - Code Quality & Conventions Cleanup | 3/3 | complete | 2026-02-10 |
-
----
-*Last updated: 2026-02-10 after Phase 4 completion*

data/.vbw-planning/milestones/default/STATE.md

@@ -1,82 +0,0 @@
-<!-- VBW STATE TEMPLATE (ARTF-05) -- Session dashboard, auto-updated -->
-<!-- Updated after each plan completion and at checkpoints -->
-
-# Project State
-
-## Project Reference
-
-See: .vbw-planning/PROJECT.md (updated 2026-02-09)
-
-**Core value:** Drop-in Rails engine for feed monitoring, content scraping, and operational dashboards.
-**Current focus:** All phases complete
-
-## Current Position
-
-Phase: 4 of 4 (Code Quality & Conventions Cleanup)
-Plan: 3 of 3 in current phase
-Status: Built
-Last activity: 2026-02-10 -- Phase 4 complete (3/3 plans done)
-
-Progress: [##########] 100%
-
-## Codebase Profile
-
-- **Total source files:** 535
-- **Primary language:** Ruby (330 files)
-- **Templates:** ERB (48 files)
-- **Tests:** 137 test files detected
-- **Test suite:** 841 runs, 2776 assertions, 0 failures (up from 473 runs in Phase 1)
-- **Coverage:** 86.97% line, 58.84% branch (510 uncovered lines, down from 2117)
-- **CI/CD:** GitHub Actions (1 workflow)
-- **Docker:** Yes (2 files)
-- **Monorepo:** No
-- **Stack:** Ruby on Rails 8 engine, Solid Queue, Hotwire, Tailwind CSS, PostgreSQL, Minitest, RuboCop, Brakeman
-
-## Accumulated Context
-
-### Decisions
-
-- [bootstrap]: Focus on coverage + refactoring before new features
-- [bootstrap]: Keep PostgreSQL-only for now
-- [bootstrap]: Keep host-app auth model
-- [phase-1]: Omakase RuboCop config only enables 45/775 cops; all Metrics cops disabled
-- [phase-1]: No .rubocop.yml exclusions needed for large files (Metrics cops off)
-- [phase-2]: PG parallel fork segfault when running single test files; use PARALLEL_WORKERS=1 or full suite
-- [phase-2]: Configuration tests (Plan 03) were committed under mislabeled "dev-plan05" commit; corrected in summaries
-- [phase-3]: FeedFetcher extraction (Plan 01) was committed under mislabeled "plan-04" commit (2f00274); corrected in summaries
-- [phase-3]: Ruby autoload used instead of Zeitwerk for lib/ modules -- safest drop-in replacement for eager requires
-- [phase-3]: 11 boot-critical requires kept explicit; 71 autoload declarations replace 66 eager requires
-- [phase-4]: ItemCreator extracted from 601 to 174 lines (EntryParser + ContentExtractor sub-modules)
-- [phase-4]: Fix-everything approach for public API convention violations
-- [phase-4]: 3 files slightly exceed 300 lines (entry_parser 390, queries 356, application_helper 346) -- all single-responsibility, cannot be split further
-
-
-None
-
-### Blockers/Concerns
-
-None
-
-### Skills
-
-**Installed:**
-- agent-browser (global)
-- flowdeck (global)
-- ralph-tui-create-json (global)
-- ralph-tui-prd (global)
-- vastai (global)
-- find-skills (global)
-
-**Suggested (not installed):**
-- dhh-rails-style (registry) -- DHH-style Rails conventions
-- ruby-rails (registry) -- Ruby on Rails development
-- github-actions (registry) -- GitHub Actions workflow
-
-**Stack detected:** Ruby on Rails 8 engine, Solid Queue, Hotwire (Turbo + Stimulus), Tailwind CSS, PostgreSQL, ESLint, GitHub Actions, Minitest, RuboCop, Brakeman
-**Registry available:** yes
-
-## Session Continuity
-
-Last session: 2026-02-10
-Stopped at: All 4 phases complete
-Resume file: none

data/.vbw-planning/milestones/default/phases/01-coverage-analysis-quick-wins/PLAN-01-SUMMARY.md

@@ -1,56 +0,0 @@
-# PLAN-01 Summary: frozen-string-literal-audit
-
-## Status: COMPLETE
-
-## Commit
-
-- **Hash:** `5f02db8`
-- **Message:** `style(frozen-string-literal): add pragma to all Ruby files`
-- **Files changed:** 113 files, 223 insertions
-
-## Tasks Completed
-
-### Task 1: Add frozen_string_literal to app/ and lib/ source files
-- Added pragma to all 29 app/ and lib/ `.rb` files
-- No shebang lines encountered in any files
-
-### Task 2: Add frozen_string_literal to config and migration files
-- Added pragma to `config/routes.rb` and 4 migration files
-
-### Task 3: Add frozen_string_literal to test files (excluding test/dummy and test/tmp)
-- Added pragma to all 43 test `.rb` files
-- `test/test_helper.rb` had no shebang, straightforward prepend
-
-### Task 4: Add frozen_string_literal to test/dummy files
-- Added pragma to all 22 dummy app `.rb` files
-
-### Task 5: Validate full codebase compliance and run tests
-- `git ls-files -- '*.rb' | ... | grep -cv 'frozen_string_literal: true'` returns **0**
-- `bin/rubocop --only Style/FrozenStringLiteralComment` exits **0** (341 files, no offenses)
-- `bin/rails test` passes: **473 runs, 1927 assertions, 0 failures, 0 errors**
-
-## Deviations
-
-### DEVN-02: Non-.rb Ruby files also needed the pragma (14 additional files)
-
-The plan scoped only `.rb` files, but the success criterion requires `bin/rubocop --only Style/FrozenStringLiteralComment` to pass. RuboCop also inspects non-`.rb` Ruby files: `Gemfile`, `Rakefile`, `source_monitor.gemspec`, 3 `.rake` files, `test/dummy/Gemfile`, `test/dummy/Rakefile`, 5 `test/dummy/bin/*` scripts, and `test/dummy/config.ru`. All 14 were given the pragma.
-
-### DEVN-02: test/tmp/ excluded from RuboCop
-
-RuboCop was scanning 179 files under `test/tmp/` (untracked generated host app templates) which all lacked the pragma. Since these are not git-tracked and are generated artifacts, added `test/tmp/**/*` to `.rubocop.yml` AllCops Exclude list rather than modifying generated files.
-
-### DEVN-01: Shebang handling for bin scripts
-
-Five `test/dummy/bin/*` files had shebang lines. The pragma was correctly inserted after the shebang on line 2. Two files (`bin/dev`, `bin/jobs`) had a blank line between the shebang and the first require, which would have created a double blank line; these were cleaned up to have a single blank line separator.
-
-## Decisions
-
-1. **Included non-.rb Ruby files** -- To satisfy the RuboCop success criterion, all Ruby-like files RuboCop inspects needed the pragma, not just `.rb` files.
-2. **Excluded test/tmp/ from RuboCop** -- These are generated Rails app templates that are not git-tracked and should not be linted. This is a minimal `.rubocop.yml` change.
-3. **Did not modify test/lib/tmp/install_generator/config/routes.rb** -- This file is not git-tracked (despite the path existing on disk from a previous test run), so it was left as-is.
-
-## Success Criteria
-
-- [x] Every git-tracked `.rb` file has `# frozen_string_literal: true` (REQ-13)
-- [x] RuboCop `Style/FrozenStringLiteralComment` cop passes with zero offenses
-- [x] No test regressions introduced (473 tests, 0 failures)

data/.vbw-planning/milestones/default/phases/01-coverage-analysis-quick-wins/PLAN-01.md

@@ -1,187 +0,0 @@
----
-phase: 1
-plan: 1
-title: frozen-string-literal-audit
-wave: 1
-depends_on: []
-skills_used: []
-must_haves:
-  truths:
-    - "Every git-tracked `.rb` file begins with `# frozen_string_literal: true` as its first line (verified by `git ls-files -- '*.rb' | xargs head -1 | grep -cv 'frozen_string_literal: true'` returning 0)"
-    - "`bin/rubocop --only Style/FrozenStringLiteralComment` exits 0 with zero offenses"
-    - "All existing tests pass (`bin/rails test` exits 0)"
-  artifacts:
-    - "98 Ruby files modified to add the `# frozen_string_literal: true` pragma"
-  key_links:
-    - "REQ-13 fully satisfied by this plan"
----
-
-# Plan 01: frozen-string-literal-audit
-
-## Objective
-
-Add the `# frozen_string_literal: true` magic comment to every git-tracked Ruby file that is currently missing it. This is a mechanical, low-risk change that brings the codebase into full compliance with REQ-13 and prepares for the RuboCop audit in Plan 02.
-
-## Context
-
-<context>
-@.vbw-planning/REQUIREMENTS.md -- REQ-13: frozen_string_literal consistency
-@.vbw-planning/codebase/CONVENTIONS.md -- documents existing frozen_string_literal convention
-@.rubocop.yml -- RuboCop configuration (omakase base)
-
-**Decomposition rationale:** This plan is separated from the RuboCop audit because (a) frozen_string_literal changes touch nearly 100 files and are purely mechanical, making them ideal for a single focused commit, and (b) completing this first removes a large category of RuboCop violations, simplifying Plan 02's scope.
-
-**Current state:**
-- 325 git-tracked Ruby files total
-- 227 already have `# frozen_string_literal: true`
-- 98 are missing it
-- Missing files span: app/ (5), lib/ (24 including setup/, engine, version, assets), test/ (43 non-dummy test files), test/dummy/ (22), config/ (1), db/migrate/ (4)
-- The codebase convention (per CONVENTIONS.md) is to use frozen_string_literal consistently, but it has drifted in setup/, engine infrastructure, test files, and dummy app files
-
-**Constraints:**
-- Do NOT modify files under `test/tmp/` (generated artifacts, not tracked in git)
-- The `test/lib/tmp/install_generator/config/initializers/source_monitor.rb` already has the pragma (verified)
-- Migration files and config/routes.rb need the pragma too
-- Some files may have a shebang line (`#!/usr/bin/env ruby`) as line 1 -- the pragma must go AFTER the shebang
-- `test/dummy/db/schema.rb` is excluded from RuboCop in `.rubocop.yml` but should still get the pragma for consistency
-</context>
-
-## Tasks
-
-### Task 1: Add frozen_string_literal to app/ and lib/ source files
-
-- **name:** add-frozen-pragma-to-source-files
-- **files:**
-  - `app/controllers/source_monitor/application_controller.rb`
-  - `app/controllers/source_monitor/health_controller.rb`
-  - `app/helpers/source_monitor/application_helper.rb`
-  - `app/jobs/source_monitor/application_job.rb`
-  - `app/models/source_monitor/application_record.rb`
-  - `lib/source_monitor.rb`
-  - `lib/source_monitor/assets.rb`
-  - `lib/source_monitor/assets/bundler.rb`
-  - `lib/source_monitor/engine.rb`
-  - `lib/source_monitor/version.rb`
-  - `lib/source_monitor/setup/initializer_patcher.rb`
-  - `lib/source_monitor/setup/bundle_installer.rb`
-  - `lib/source_monitor/setup/workflow.rb`
-  - `lib/source_monitor/setup/gemfile_editor.rb`
-  - `lib/source_monitor/setup/requirements.rb`
-  - `lib/source_monitor/setup/detectors.rb`
-  - `lib/source_monitor/setup/shell_runner.rb`
-  - `lib/source_monitor/setup/cli.rb`
-  - `lib/source_monitor/setup/verification/printer.rb`
-  - `lib/source_monitor/setup/verification/telemetry_logger.rb`
-  - `lib/source_monitor/setup/verification/solid_queue_verifier.rb`
-  - `lib/source_monitor/setup/verification/result.rb`
-  - `lib/source_monitor/setup/verification/action_cable_verifier.rb`
-  - `lib/source_monitor/setup/verification/runner.rb`
-  - `lib/source_monitor/setup/install_generator.rb`
-  - `lib/source_monitor/setup/dependency_checker.rb`
-  - `lib/source_monitor/setup/prompter.rb`
-  - `lib/source_monitor/setup/migration_installer.rb`
-  - `lib/source_monitor/setup/node_installer.rb`
-- **action:** Prepend `# frozen_string_literal: true\n\n` to each file listed above. If the file already begins with a shebang (`#!`), insert the pragma on line 2 (after the shebang) with a blank line separating them. For `lib/source_monitor.rb` specifically, note it starts with `require` statements -- the pragma goes before all requires.
-- **verify:** Run `grep -cL 'frozen_string_literal: true' app/**/*.rb lib/**/*.rb` returns no results (all files have the pragma). Additionally run `ruby -c lib/source_monitor.rb` to confirm syntax validity.
-- **done:** All 29 app/ and lib/ Ruby files have `# frozen_string_literal: true` as their first non-shebang line.
-
-### Task 2: Add frozen_string_literal to config and migration files
-
-- **name:** add-frozen-pragma-to-config-and-migrations
-- **files:**
-  - `config/routes.rb`
-  - `db/migrate/20251009103000_add_feed_content_readability_to_sources.rb`
-  - `db/migrate/20251014171659_add_performance_indexes.rb`
-  - `db/migrate/20251014172525_add_fetch_status_check_constraint.rb`
-  - `db/migrate/20251108120116_refresh_fetch_status_constraint.rb`
-- **action:** Prepend `# frozen_string_literal: true\n\n` to each file. These are small files (migration class definitions and route definitions).
-- **verify:** Run `head -1` on each file to confirm the pragma is present. Run `ruby -c config/routes.rb` to confirm syntax validity (it will fail since it references SourceMonitor::Engine, so instead just visually confirm the pragma is on line 1).
-- **done:** All 5 config/ and db/migrate/ files have the pragma.
-
-### Task 3: Add frozen_string_literal to test files (excluding test/dummy and test/tmp)
-
-- **name:** add-frozen-pragma-to-test-files
-- **files:**
-  - `test/test_helper.rb`
-  - `test/source_monitor_test.rb`
-  - `test/gemspec_test.rb`
-  - `test/application_system_test_case.rb`
-  - `test/system/dropdown_fallback_test.rb`
-  - `test/integration/engine_mounting_test.rb`
-  - `test/integration/navigation_test.rb`
-  - `test/controllers/source_monitor/health_controller_test.rb`
-  - `test/jobs/source_monitor/fetch_feed_job_test.rb`
-  - `test/jobs/source_monitor/item_cleanup_job_test.rb`
-  - `test/jobs/source_monitor/log_cleanup_job_test.rb`
-  - All files under `test/lib/source_monitor/setup/` (12 files)
-  - All files under `test/lib/source_monitor/setup/verification/` (5 files)
-  - `test/lib/source_monitor/instrumentation_test.rb`
-  - `test/lib/source_monitor/feedjira_configuration_test.rb`
-  - `test/lib/source_monitor/metrics_test.rb`
-  - `test/lib/source_monitor/scheduler_test.rb`
-  - `test/lib/source_monitor/release/runner_test.rb`
-  - `test/lib/source_monitor/release/changelog_test.rb`
-  - `test/lib/source_monitor/items/item_creator_test.rb`
-  - `test/lib/source_monitor/items/retention_pruner_test.rb`
-  - `test/lib/source_monitor/events/event_system_test.rb`
-  - `test/lib/source_monitor/assets/bundler_test.rb`
-  - `test/lib/source_monitor/engine_assets_configuration_test.rb`
-  - `test/lib/source_monitor/http_test.rb`
-  - `test/lib/source_monitor/fetching/feed_fetcher_test.rb`
-  - `test/lib/source_monitor/fetching/fetch_runner_test.rb`
-- **action:** Prepend `# frozen_string_literal: true\n\n` to each file. The `test/test_helper.rb` file may have a shebang or special first lines -- check and handle accordingly.
-- **verify:** Run `find test -name '*.rb' -not -path 'test/tmp/*' -not -path 'test/dummy/*' -exec head -1 {} \; | grep -cv 'frozen_string_literal'` returns 0.
-- **done:** All 43 test files (excluding dummy and tmp) have the pragma.
-
-### Task 4: Add frozen_string_literal to test/dummy files
-
-- **name:** add-frozen-pragma-to-dummy-app
-- **files:**
-  - `test/dummy/app/controllers/application_controller.rb`
-  - `test/dummy/app/controllers/test_support_controller.rb`
-  - `test/dummy/app/helpers/application_helper.rb`
-  - `test/dummy/app/jobs/application_job.rb`
-  - `test/dummy/app/mailers/application_mailer.rb`
-  - `test/dummy/app/models/application_record.rb`
-  - `test/dummy/app/models/user.rb`
-  - `test/dummy/config/application.rb`
-  - `test/dummy/config/boot.rb`
-  - `test/dummy/config/environment.rb`
-  - `test/dummy/config/environments/development.rb`
-  - `test/dummy/config/environments/production.rb`
-  - `test/dummy/config/environments/test.rb`
-  - `test/dummy/config/importmap.rb`
-  - `test/dummy/config/puma.rb`
-  - `test/dummy/config/routes.rb`
-  - `test/dummy/config/initializers/assets.rb`
-  - `test/dummy/config/initializers/content_security_policy.rb`
-  - `test/dummy/config/initializers/filter_parameter_logging.rb`
-  - `test/dummy/config/initializers/inflections.rb`
-  - `test/dummy/db/migrate/20251124080000_create_users.rb`
-  - `test/dummy/db/schema.rb`
-- **action:** Prepend `# frozen_string_literal: true\n\n` to each file. The `test/dummy/db/schema.rb` may be auto-generated by Rails -- still add the pragma since it is git-tracked and part of the project.
-- **verify:** Run `find test/dummy -name '*.rb' -exec head -1 {} \; | grep -cv 'frozen_string_literal'` returns 0.
-- **done:** All 22 dummy app Ruby files have the pragma.
-
-### Task 5: Validate full codebase compliance and run tests
-
-- **name:** validate-frozen-pragma-compliance
-- **files:** (no files modified -- validation only)
-- **action:** Run three verification commands: (1) Count git-tracked Ruby files without the pragma -- must be 0. (2) Run `bin/rubocop --only Style/FrozenStringLiteralComment` to confirm zero RuboCop offenses for this cop. (3) Run the test suite to confirm no regressions from adding the pragma.
-- **verify:**
-  - `git ls-files -- '*.rb' | xargs head -1 | grep -cv 'frozen_string_literal: true'` outputs `0`
-  - `bin/rubocop --only Style/FrozenStringLiteralComment` exits 0
-  - `bin/rails test` exits 0 (or at minimum no new failures)
-- **done:** Full codebase compliance confirmed. REQ-13 is satisfied.
-
-## Verification
-
-1. `git ls-files -- '*.rb' | xargs head -1 | grep -cv 'frozen_string_literal: true'` returns `0`
-2. `bin/rubocop --only Style/FrozenStringLiteralComment` exits 0 with zero offenses
-3. Test suite passes with no regressions
-
-## Success Criteria
-
-- [x] Every git-tracked Ruby file has `# frozen_string_literal: true` (REQ-13)
-- [x] RuboCop `Style/FrozenStringLiteralComment` cop passes with zero offenses
-- [x] No test regressions introduced