Remove dead code and unnecessary global declarations

[Copilot]

Static analysis (pyflakes) surfaced several dead-code and style issues across main.py. This PR cleans them up.

Dead / unreachable code

• Removed first print_summary_table definition (the 148-line version at line 316): it was immediately shadowed by a second definition at line 2358 and never called by either name.
• Removed unreachable return True in validate_hostname: placed after the inner except block that already unconditionally returns.
• Removed unreachable return True in validate_folder_url: placed after a try-except where every branch returns explicitly.

Unnecessary global declarations

global is only required in Python when reassigning a module-level name. All flagged sites only mutated dict entries, making the declarations pure noise. Removed from:

• load_disk_cache — global _cache_stats (kept global _disk_cache, which is reassigned)
• _parse_rate_limit_headers — global _rate_limit_info
• _api_get, _api_delete, _api_post, _api_post_form — global _api_stats
• _gh_get — global _cache_stats, _api_stats
• main — global _disk_cache (only .clear() is called, no reassignment)

Unused exception variable

# Before
except httpx.HTTPStatusError as e:
    raise  # e never referenced

# After
except httpx.HTTPStatusError:
    raise

Original prompt

---

This section details on the original issue you should resolve

<issue_title>[Code Quality] Discussion Task Miner - Code Quality Improvement Agent - Issue Group</issue_title>
<issue_description># Discussion Task Miner - Code Quality Improvement Agent

Parent issue for grouping related issues from Discussion Task Miner - Code Quality Improvement Agent.

Sub-issues are automatically linked below (max 64 per parent).

Workflow: Discussion Task Miner - Code Quality Improvement Agent

• expires on Feb 21, 2026, 5:44 AM UTC</issue_description>

Comments on the Issue (you are @copilot in this section)

Custom agent used: Development Partner
The Development Partner Protocol

• Fixes [Code Quality] Discussion Task Miner - Code Quality Improvement Agent - Issue Group #364

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

[trunk-io]

Merging to main in this repository is managed by Trunk.

• To merge this pull request, check the box to the left or comment /trunk merge below.

[github-actions]

👋 Development Partner is reviewing this PR. Will provide feedback shortly.

[Copilot]

Pull request overview

This PR removes dead code and unnecessary global declarations identified by static analysis (pyflakes) in main.py. The changes eliminate unreachable code, clean up unnecessary global variable declarations, and remove an unused exception variable, improving code quality without altering functionality.

Changes:

• Removed dead print_summary_table function definition (148 lines) that was shadowed by a second definition
• Removed unreachable return True statements in validate_hostname and validate_folder_url
• Removed unnecessary global declarations from 7 functions where only dictionary mutations occur (not reassignments)
• Removed unused exception variable from bare re-raise pattern

[github-actions]

👋 Development Partner is reviewing this PR. Will provide feedback shortly.