Check pull requests for undefined names and run unit tests

[raf-li]

Why

The Check job only byte-compiled the code, so the deleted chan_name variable (bb518fc) passed Check, was deployed, and broke every @ai answer.

What

• Check job (.github/workflows/deploy.yml) now also:
  • runs flake8 --select=E9,F63,F7,F82 (syntax errors and undefined names; style is not checked);
  • runs python -m unittest discover -s tests -t ..
    Deploy still needs: check, so a failing test also blocks a deploy from main.
• tests/: 50 stdlib unittest tests for the modules that run without the TeamTalk SDK:
  • test_ai_chat.py: channel/PM/no-channel-name context lines, history order, retry without tools, error response, tool call → request_tool → follow-up answer, _extract_tool_calls variants, background ask/drain, tool definitions well-formed
  • test_ai_review.py: parse_verdict, format_message (incl. @ai note), request layout, background submit/drain, errors
  • test_badwords.py: words, stretched spellings, wildcards, phrases, broad-wildcard refusal, saving, default list seeding, every shipped entry usable
  • test_abuse_tracker.py: stage escalation, per-person counting, window expiry, reset
  • test_help_texts.py: every /help topic renders
• tt_client.py: get_bot_info used an undefined tt5 (the only other flake8 finding); it now uses version.collect_version_info() like /version.
• docs/deploy.md and CHANGELOG describe the checks and how to run them locally.

Verified

• flake8 and the tests pass on this branch.
• Against bb518fc's ai_chat.py, flake8 reports F821 undefined name 'chan_name' and 4 AI chat tests fail with NameError.

The tt_client.py integration tests still need the TeamTalk SDK and are not part of CI yet.

🤖 Generated with Claude Code