Skip to content

Tighten exception handling flagged by Ruff - #183

Merged
ZFordDev merged 6 commits into
mainfrom
codex/179-exception-boundaries
Aug 27, 2026
Merged

Tighten exception handling flagged by Ruff#183
ZFordDev merged 6 commits into
mainfrom
codex/179-exception-boundaries

Conversation

@ZFordDev

Copy link
Copy Markdown
Owner

Closes #179

This PR narrows optional UI imports to ImportError, logs resilient reminder failures, restores failed undo actions instead of silently discarding them, adds undo coverage, and documents the exception-boundary policy.

This PR is stacked on #182 to keep its review isolated.

Validation:

  • BLE001, S110, and stale suppression checks pass across src and tests.
  • Ruff formatting passes.
  • 20 relevant tests pass locally; the real Windows message-box test is excluded because it opens a modal OS dialog.
  • Only the three PyQt model findings assigned to Resolve Ruff safety findings in the PyQt task model #180 remain in the full Ruff result.

Base automatically changed from codex/178-datetime-semantics to main August 27, 2026 18:54
@ZFordDev
ZFordDev merged commit 9765547 into main Aug 27, 2026
2 of 3 checks passed
@ZFordDev ZFordDev linked an issue Aug 27, 2026 that may be closed by this pull request
@ZFordDev
ZFordDev deleted the codex/179-exception-boundaries branch August 27, 2026 18:54
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.

Tighten exception handling flagged by Ruff

1 participant