Walkthrough

The PR adds an is_closed() method to the Connection type and hardens Blob::read with a precondition check to raise a ProgrammingError when operating on a blob with a closed connection, aligning with CPython sqlite3 behavior.

Changes

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

• Single file with localized, straightforward changes to two related methods
• Simple conditional logic (checking Option state) with standard error handling
• Direct correspondence to issue requirements with minimal surface area

Possibly related PRs

• stdlib(sqlite): Raise ProgrammingError in closed Blob context manager #6041: Adds closed-state checks to Blob operations in sqlite.rs, addressing related connection validity concerns in blob context managers.
• Fix sqlite Connection initialization check #6199: Modifies Connection internal db (Option) state handling and error paths, affecting the same underlying state-management mechanism.
• feat(stdlib/sqlite): Implement slice assignment for Blob #6039: Implements additional Blob functionality in the same file, sharing the closed-connection validation context.

Suggested reviewers

• youknowone

Poem

🐰 A rabbit's cheer for checks so keen,
When blobs on closed databases convene!
No silent fails, just errors bright,
Connection states now held so tight. ✨

Pre-merge checks and finishing touches

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

_{Comment @coderabbitai help to get the list of available commands and usage tips.}