DEVX-102: Fix wiki integrity check on transient API timeout #159

Closed
emil wants to merge 0 commits from DEVX-102-wiki-integrity-retry into master
Owner

Summary

The wiki integrity check failed when the Gitea API was temporarily unavailable after a batch of wiki page updates. list_wiki_pages caught APIError and returned an empty dict, causing all pages to be reported as missing and empty — even though the sync itself succeeded.

Changes

  • list_wiki_pages now raises APIError instead of swallowing it
  • verify_wiki_integrity uses tenacity (same pattern as api_clients) for exponential backoff retry on the page list
  • If retries are exhausted, returns no failures with a warning — sync already succeeded, transient API outage should not fail the job
  • The --verify path in main() also uses the retry wrapper and skips gracefully on failure
  • Added missing translation keys for new warning messages

Closes DEVX-102

## Summary The wiki integrity check failed when the Gitea API was temporarily unavailable after a batch of wiki page updates. list_wiki_pages caught APIError and returned an empty dict, causing all pages to be reported as missing and empty — even though the sync itself succeeded. ## Changes - list_wiki_pages now raises APIError instead of swallowing it - verify_wiki_integrity uses tenacity (same pattern as api_clients) for exponential backoff retry on the page list - If retries are exhausted, returns no failures with a warning — sync already succeeded, transient API outage should not fail the job - The --verify path in main() also uses the retry wrapper and skips gracefully on failure - Added missing translation keys for new warning messages Closes DEVX-102
emil reviewed 2026-06-30 08:29:07 +00:00
emil left a comment
Author
Owner

Automated PR Review

  • Architecture compliance: OK
  • Best practices: OK
  • Security: OK
  • i18n: OK
  • Resource management: OK
  • Documentation: WARNING — source files changed but no docs updated
  • Tests: OK
  • Commit conventions: OK

No issues found by automated checks.


Auto-merge: If all CI checks pass, this PR will be merged automatically.

## Automated PR Review - Architecture compliance: OK - Best practices: OK - Security: OK - i18n: OK - Resource management: OK - Documentation: WARNING — source files changed but no docs updated - Tests: OK - Commit conventions: OK No issues found by automated checks. --- **Auto-merge:** If all CI checks pass, this PR will be merged automatically.
kireto approved these changes 2026-06-30 08:29:53 +00:00
Dismissed
kireto left a comment
Member

Auto-approved: all CI checks passed (quality, pr-review, release-dry-run).

Auto-approved: all CI checks passed (quality, pr-review, release-dry-run).
emil changed title from DEVX-102: fix: retry wiki integrity check on transient API timeout to DEVX-102: Fix wiki integrity check on transient API timeout 2026-06-30 08:34:29 +00:00
emil reviewed 2026-06-30 08:35:03 +00:00
emil left a comment
Author
Owner

Automated PR Review

  • Architecture compliance: OK
  • Best practices: OK
  • Security: OK
  • i18n: OK
  • Resource management: OK
  • Documentation: WARNING — source files changed but no docs updated
  • Tests: OK
  • Commit conventions: OK

No issues found by automated checks.


Auto-merge: If all CI checks pass, this PR will be merged automatically.

## Automated PR Review - Architecture compliance: OK - Best practices: OK - Security: OK - i18n: OK - Resource management: OK - Documentation: WARNING — source files changed but no docs updated - Tests: OK - Commit conventions: OK No issues found by automated checks. --- **Auto-merge:** If all CI checks pass, this PR will be merged automatically.
kireto approved these changes 2026-06-30 08:35:39 +00:00
kireto left a comment
Member

Auto-approved: all CI checks passed (quality, pr-review, release-dry-run).

Auto-approved: all CI checks passed (quality, pr-review, release-dry-run).
emil closed this pull request 2026-06-30 08:41:18 +00:00

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
No Reviewers
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: oblachno-oss/devx#159