PLAN-04 Summary: fix-log-entry-and-autoloading

Status: COMPLETE

Commits

• Hash: fb99d3d
• Message: refactor(plan-04): fix LogEntry table name and replace eager requires with autoloading
• Files changed: 2 files, 125 insertions, 60 deletions

Tasks Completed

Task 1: Fix LogEntry hard-coded table name

• Removed self.table_name = "sourcemon_log_entries" from log_entry.rb
• Model now relies on ModelExtensions.register(self, :log_entry) for dynamic table name
• Table name resolves correctly via configurable prefix system
• LogEntry tests: 1 run, 4 assertions, 0 failures

Task 2: Replace eager requires with autoload declarations

• Replaced 66 eager require statements with 71 autoload declarations
• Kept 11 explicit requires for boot-critical modules: version, engine, configuration, model_extensions, events, instrumentation, metrics, health, realtime, feedjira_extensions
• Organized autoloads by domain module: Analytics, Dashboard, Fetching, ImportSessions, Items, Jobs, Logs, Models, Pagination, Release, Scrapers, Scraping, Security, Setup, Sources, TurboStreams
• Full suite: 760 runs, 0 failures, 0 errors

Task 3: Verify full integration and RuboCop

• All 760 tests pass with autoloading (no load-order issues)
• RuboCop: 2 files inspected, 0 offenses
• REQ-11 and REQ-12 satisfied

Deviations

ID	Description	Impact
D-01	71 autoload declarations instead of planned 40+ minimum	Positive -- more thorough conversion covering all lib modules

Verification Results

Check	Result
grep 'self.table_name' log_entry.rb	No matches (removed)
grep 'ModelExtensions.register' log_entry.rb	Present on line 17
grep -c '^require ' lib/source_monitor.rb	11 (target: <15, down from 66)
grep -c 'autoload' lib/source_monitor.rb	71 (target: 40+)
bin/rails test	760 runs, 2626 assertions, 0 failures, 0 errors
bin/rubocop lib/source_monitor.rb app/models/source_monitor/log_entry.rb	2 files inspected, 0 offenses

Success Criteria

• LogEntry no longer has hard-coded table name
• LogEntry uses ModelExtensions.register for dynamic table name
• Eager require count reduced from 66 to 11 (target: <15)
• 71 autoload declarations replace the eager requires (target: 40+)
• All existing tests pass without modification
• Full test suite passes (760 runs, 0 failures)
• RuboCop passes on modified files
• REQ-11 and REQ-12 satisfied