Title: [Bug] FEC data not populating for most members — remove is_active_candidate filter

[OpenSourcePatents]

Here you go — just copy everything below the line:

Problem
fetch_fec_candidate() in fetch_finance.py passes is_active_candidate: True as a query parameter when hitting the FEC API. This filters out any member who hasn’t run a campaign recently, silently returning no FEC data for a large portion of currently serving Congress members.
Root cause
FEC’s definition of “active candidate” is narrower than “currently serving.” Members between election cycles, in safe seats, or who haven’t recently filed get excluded by this filter even though they’re in office right now.
Fix
Remove the is_active_candidate parameter from the FEC API call in fetch_fec_candidate(). Allow FEC to return all matching candidate records, then rely on name and state matching to identify the correct record.
File: fetch_finance.py → function fetch_fec_candidate()
Acceptance criteria
∙ is_active_candidate param removed from FEC API call
∙ Members previously returning empty FEC data now populate fec_candidate_id, fec_committee_id, total_raised, pac_contributions, individual_contributions, and cash_on_hand
∙ No regression on members who were already populating correctly
∙ No new duplicate candidate matches introduced by loosening the filter​​​​​​​​​​​​​​​​

[OpenSourcePatents]

Update — Implementation Complete, Pending Test Run
Both files have been created and are ready for testing.
Changes made to fetch_finance.py
∙ Removed is_active_candidate: True from fetch_fec_candidate() — this was silently filtering out serving members between election cycles
∙ Updated cycle from 2024 → 2026 in fetch_fec_totals()
∙ Updated two_year_transaction_period from 2024 → 2026 in fetch_fec_top_donors()
∙ Fixed committee_id key lookup in principal_committees (was using id, FEC API returns committee_id)
∙ Cleaned up fetch_edgar_trades() — removed dead hardcoded request that was never returning valid data
∙ Added if name == "main" guard
∙ Consistent {} return on all FEC exceptions
Test infrastructure added
∙ Created fetch_finance_test.py at repo root — identical logic with TEST_MODE = True, processes 5 members only (Mike Rounds prioritized first), saves to data/members_test.json without touching members.json
∙ Created .github/workflows/test-finance.yml — manual trigger only (workflow_dispatch), runs fetch_finance_test.py, prints results to Action logs
Next step
Trigger Test Finance Pipeline manually from the Actions tab and verify Mike Rounds returns a populated fec_candidate_id, total_raised, and top_donors in the logs. If clean, merge fetch_finance.py to main and close issue.​​​​​​​​​​​​​​​​