Skip to content

fix: harden sync and failure handling - #33

Merged
AksharP5 merged 3 commits into
mainfrom
agent/fix-code-audit
Aug 19, 2026
Merged

fix: harden sync and failure handling#33
AksharP5 merged 3 commits into
mainfrom
agent/fix-code-audit

Conversation

@AksharP5

Copy link
Copy Markdown
Owner

Repository and GitHub failures could leave Blippy with stale or misleading state. Scans could erase cached remotes, linked probes could retry continuously, cross-repo links could jump to the wrong item, and PR metadata failures appeared as valid empty data.

This keeps cached data intact, makes unavailable metadata explicit, and stops failed background probes from hammering the API. It also includes the earlier sync, cache, discovery, CLI, input, and CI hardening from this branch.

What changed

  • preserve cached remotes when Git inspection fails
  • stop automatic linked-item retries after API failures
  • reject cross-repository links that cannot be opened safely in the current TUI context
  • show unknown PR view state and surface review metadata failures
  • close issues before posting optional close comments to avoid duplicate comments on retry
  • keep sync snapshots, repository discovery, input handling, CLI behavior, and CI failure paths consistent

Checks

  • cargo test --all-targets --all-features --locked
  • cargo clippy --all-targets --all-features --locked -- -D warnings
  • cargo fmt --all -- --check
  • git diff --check

@AksharP5
AksharP5 merged commit 461bc9f into main Aug 19, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant