Problem Statement

Basically, the SDK was not returning when the key was not found and the connect service looks like can have an empty item in it which is weird, but it is what it is.

Related Issue

Fixes #5753

Proposed Changes

How do you like to solve the issue and why?

Format

Please ensure that your PR follows the following format for the title:

feat(scope): add new feature
fix(scope): fix bug
docs(scope): update documentation
chore(scope): update build tool or dependencies
ref(scope): refactor code
clean(scope): provider cleanup
test(scope): add tests
perf(scope): improve performance
desig(scope): improve design

Where scope is optionally one of:

• charts
• release
• testing
• security
• templating

Checklist

• I have read the contribution guidelines
• All commits are signed with git commit --signoff
• My changes have reasonable test coverage
• All tests pass with make test
• I ensured my PR is ready for review with make reviewable

Fix deletion of empty 1Password secrets

Addresses hanging deletion operations when using PushSecret with multiple keys against 1Password Connect Server. The fix implements three key changes:

1. Empty section handling: When a 1Password item contains only an empty placeholder section (with empty ID and Label), the section is now cleared to allow the item to be deleted.

2. Non-existent key handling: Treats deletion of non-existent keys as a no-op by returning success instead of an error, preventing deletion loops from failing.

3. Test coverage: Adds tests for deleting items with empty sections and for deleting multiple fields from the same item in sequence.

These changes ensure that 1Password items are properly removed when they become empty after field deletions, resolving indefinite hangs in the deletion process.