refactor(types): simplify pass on P1 batch

Follow-up to 15ac253b per /simplify review:

- gateway/platforms/discord.py:3638 - move self.resolved = True *after*
  the `if interaction.data is None: return` guard. Previously the view
  was marked resolved before the None-guard, so a None data payload
  silently rejected the user's next click.
- agent/display.py:732 - replace `if self.start_time is None: continue`
  with `assert self.start_time is not None`. start() sets start_time
  before the animate thread starts, so the None branch was dead; the
  `continue` form would have busy-looped (skipping the 0.12s sleep).
- tests/hermes_cli/test_config_shapes.py - drop __total__ dunder
  restatement test (it just echoes the class declaration); trim commit
  narration from module docstring.
- tests/agent/test_credential_pool.py, tests/tools/test_rl_training_tool.py -
  drop "added in commit ..." banners (narrates the change per CLAUDE.md).

tests/agent/test_credential_pool.py

@@ -1164,10 +1164,6 @@ def test_load_pool_does_not_seed_qwen_oauth_when_no_token(tmp_path, monkeypatch)
     assert pool.entries() == []
 
 
-# ---------------------------------------------------------------------------
-# Tests for CredentialPool.remove_entry (added in commit fc00f699)
-# ---------------------------------------------------------------------------
-
 def _build_pool_with_entries(tmp_path, monkeypatch, provider="openrouter", entries=None):
     """Helper: build a CredentialPool directly without seeding side-effects."""
     monkeypatch.setenv("HERMES_HOME", str(tmp_path / "hermes"))