fix: handle unexpected API responses with clean errors #167

Merged

fuzzy merged 1 commit from fix/134-forgejo-keyerror into main

2026-07-06 08:50:43 +00:00

automation commented

2026-07-06 08:37:32 +00:00

Owner

Wraps data["id"] access in get_user_id() and get_repo_id() with try/except for KeyError, TypeError, ValueError to raise ForgejoError with a descriptive message instead of a raw traceback.

Closes #134

Wraps `data["id"]` access in `get_user_id()` and `get_repo_id()` with try/except for `KeyError`, `TypeError`, `ValueError` to raise `ForgejoError` with a descriptive message instead of a raw traceback. Closes #134

automation added 1 commit

2026-07-06 08:37:32 +00:00

fix(forgejo): handle unexpected API responses with clean errors

CI / lint-and-test (pull_request) Successful in 1m32s

Details

9a5f746f0b

Wrap data['id'] access with try/except to raise ForgejoError
instead of raw KeyError/TypeError/ValueError from unexpected
API responses.

Closes #134

fuzzy scheduled this pull request to auto merge when all checks succeed

2026-07-06 08:37:46 +00:00

the.auditor reviewed

2026-07-06 08:38:00 +00:00

the.auditor left a comment

Audit Review — PR #167

Clean fix for #134. data["id"] wrapped in try/except for KeyError, TypeError, ValueError, raising ForgejoError with descriptive message per endpoint (user '{username}' / repo '{owner}/{repo}').

No suggestions.

Summary

Result: COMMENT — no issues found.

## Audit Review — PR #167 Clean fix for #134. `data["id"]` wrapped in try/except for `KeyError`, `TypeError`, `ValueError`, raising `ForgejoError` with descriptive message per endpoint (`user '{username}'` / `repo '{owner}/{repo}'`). No suggestions. ## Summary **Result:** COMMENT — no issues found.

the.auditor approved these changes

2026-07-06 08:38:04 +00:00

the.auditor left a comment

APPROVED. API response handling improved, closes #134. Merge when ready.