feat(providers): gemeinsame ProviderError-Basis + RateLimitError für einheitliche Fehler-/Rate-Limit-Erkennung

[Seretos]

#ai-generated

Problem

Die Provider-Abstraktion leakt die Fehler-Semantik. Jeder Provider wirft eine eigene, voneinander unabhängige Exception:

• GitHubError(RuntimeError) — providers/github.py, .status: int, .message
• GitLabError(RuntimeError) — providers/gitlab.py, .status, .message
• AzureDevOpsError(RuntimeError) — providers/azuredevops.py, .status

Es gibt keinen gemeinsamen Basistyp in providers/base.py (dort nur Relation-Exceptions) und keinen dedizierten Rate-Limit-Typ. Folgen für Consumer (workboard, agent-project-issues):

• except <ProviderError> ist nicht möglich — man muss alle drei konkreten Typen importieren oder breit except Exception fangen.
• „War das ein Rate-Limit und wann darf ich wieder?" lässt sich nur per .status == 403/429 plus String-Schnüffeln in .message (GitHub) beantworten — pro Provider unterschiedlich. Genau das Detail, das die Abstraktion kapseln sollte, liegt beim Consumer.

Konkret hat das im Workboard dazu geführt, dass die Rate-Limit-Behandlung toter Code war (sie fing httpx.HTTPStatusError, das real kein Provider wirft) und das Board bei Rate-Limit still „Keine offenen Tickets" zeigte. Siehe workboard-Ticket (Relation).

Vorschlag

In providers/base.py:

class ProviderError(RuntimeError):
    def __init__(self, status: int, message: str):
        super().__init__(message)
        self.status = status
        self.message = message

class RateLimitError(ProviderError):
    def __init__(self, status: int, message: str, retry_after: int | None = None):
        super().__init__(status, message)
        self.retry_after = retry_after  # Sekunden bis Reset, falls bekannt

• GitHubError / GitLabError / AzureDevOpsError erben von ProviderError (Signatur/.status/.message bleiben → rückwärtskompatibel, bestehende except GitHubError funktionieren weiter).
• Jeder _check() wirft RateLimitError statt des generischen Provider-Errors, wenn er ein Rate-Limit erkennt, und füllt retry_after:
  • GitHub: 403 + Header x-ratelimit-remaining == "0" → retry_after = x-ratelimit-reset - now (Sekunden). (Logik existiert bereits halb in github.py::_check, nur als Message-Präfix.)
  • GitLab: 429 → Retry-After bzw. RateLimit-Reset.
  • Azure: 429 → Retry-After.

Akzeptanzkriterien

• ProviderError + RateLimitError in base.py, die drei Provider-Errors erben davon.
• Pro Provider ein Test: Rate-Limit-Response → RateLimitError mit korrektem retry_after; gewöhnlicher Fehler → konkreter ProviderError-Subtyp mit .status.
• RateLimitError ist auch ein ProviderError und (für Kompat) weiterhin ein RuntimeError.
• Export über providers/__init__.py bzw. Package-Top-Level, sodass Consumer from lib_python_projects.providers.base import ProviderError, RateLimitError nutzen können.

Downstream (eigene Tickets)

• workboard konsumiert RateLimitError/ProviderError statt httpx.HTTPStatusError + .status-Sniffing.
• agent-project-issues — vermutlich No-op (surfacet Fehler schon via _safe), Verify-Ticket.

Optional als Folge-Ticket: ETag/If-None-Match-Conditional-Requests im Provider-Layer (GitHub 304 Not Modified zählt nicht gegen das Primär-Rate-Limit) — größter Hebel für pollende Consumer auf einem geteilten Token.

[Seretos]

#ai-generated

Implementation plan

Goal: Give providers a shared error base type and a dedicated rate-limit exception with a machine-readable retry_after.

Approach:

• Add ProviderError(RuntimeError) (carries .status, .message) and RateLimitError(ProviderError) (adds .retry_after: int | None) to providers/base.py.
• Re-parent GitHubError, GitLabError, AzureDevOpsError from RuntimeError → ProviderError. Concrete __init__s keep building their prefixed str() and setting .status/.message exactly as today, so existing except <Provider>Error call sites and str(err) stay backwards-compatible.
• Each provider's _check raises RateLimitError on a rate limit:
  • GitHub: 403 + x-ratelimit-remaining == "0" → retry_after = x-ratelimit-reset - now.
  • GitLab: 429 → parse Retry-After / RateLimit-Reset.
  • Azure DevOps: 429 → parse Retry-After.
  • Missing/unparseable header → retry_after = None.
• Export ProviderError/RateLimitError from providers/__init__.py.

Affected files: providers/base.py, providers/github.py, providers/gitlab.py, providers/azuredevops.py, providers/__init__.py, plus tests (tests/test_github_errors.py extended; new tests/test_gitlab_errors.py, tests/test_azuredevops_errors.py).

Tests: per provider — rate-limit response → RateLimitError with correct retry_after; ordinary error → concrete subtype with .status; isinstance(err, ProviderError). Run with python -m pytest.

[Seretos]

#ai-generated

Draft PR opened: #97. Implements ProviderError/RateLimitError, re-parents the three provider error classes (backwards-compatible), and raises RateLimitError on rate limits in GitHub/GitLab/Azure DevOps _check. Reviewer verdict: APPROVE. Tests: 932 passed, 5 pre-existing skips, 0 failures.