extensions icon indicating copy to clipboard operation
extensions copied to clipboard

[1Password] Extension showing items in search for empty vault

Open ShiftyMcCool opened this issue 2 years ago • 7 comments

Extension – 1Password

Author:

Description

Steps To Reproduce

  1. Add multiple vaults to 1Password
  2. Remove items from that vault
  3. Items are still visible when searching via the extension

The current behavior

Deleted items are still visible in search (sorry for all the blurring, this is a work account and client names are sometimes secret: image image

The expected behavior

Deleted items should not appear in search

Raycast version

v1.36.0

Let me know if you need more info. The repro steps may not be exactly accurate as someone else on my team moved all items from the indicated vault to another so I'm not sure exactly how they did that. All I know is that search is showing results for an empty vault (see screenshots)

EDIT: CLI is showing no results as well: image

ShiftyMcCool avatar Jun 08 '22 13:06 ShiftyMcCool

@pernielsentikaer any suggestions? I'm still seeing old/removed entries and nothing I have tried will cause an update in the extension. In fact, the entire vault has been renamed but still shows up as the old name.

ShiftyMcCool avatar Jul 19 '22 19:07 ShiftyMcCool

Same problem here. The deleted entries show up in the 'Recently deleted items' that can be found in the menu for the desktop app. Raycast should probably not be querying these items.

maarten00 avatar Aug 10 '22 07:08 maarten00

That sounds like a bug to me, I can take a look within a few days if @dteare don't beat me 🙂

pernielsentikaer avatar Aug 10 '22 09:08 pernielsentikaer

This issue/pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs in the next 7 days to keep our backlog clean. Thanks for your contributions.

stale[bot] avatar Oct 01 '22 02:10 stale[bot]

I couldn't find a way to determine if the item was in the trash, is that possible @dteare

pernielsentikaer avatar Oct 15 '22 07:10 pernielsentikaer

Hello @pernielsentikaer, @ShiftyMcCool. I am currently rewriting the extension utilizing the 1Password CLI 2 and this issue will be addressed in the new version. Thanks.

khasbilegt avatar Oct 17 '22 04:10 khasbilegt

Awesome 🎉

pernielsentikaer avatar Oct 17 '22 08:10 pernielsentikaer

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs in the next 7 days to keep our backlog clean 😊

stale[bot] avatar Nov 19 '22 12:11 stale[bot]

Just got a notification that this went stale. This is just a comment to bump and ask about progress. Thanks!

ShiftyMcCool avatar Nov 21 '22 13:11 ShiftyMcCool

Sorry for the late reply. I get too many notifications.

The screenshot in the OP is from 1Password 7. With 1Password 8 the CLI is used to provided the needed information to Raycast, and by default archived (a.k.a. trashed) items are not included. Upgrading to 1Password 8 and following the setup instructions for opbookmarks will get you up-to-date and resolve this issue. As @khasbilegt mentioned there will be an update to this extension in the future that will integrate with the 1Password CLI directly and avoid the setup & refresh steps.

As for 1Password 7, I believe there's a trashed flag in the metadata files that could be used to filter them out. To do this one would need to add an optional trashed attribute here and then filter out trashed items when constructing the item list cache, probably here.

I hope that helps. Cheers!

dteare avatar Nov 21 '22 18:11 dteare

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs in the next 7 days to keep our backlog clean 😊

stale[bot] avatar Dec 10 '22 12:12 stale[bot]

I'm closing this due to inactivity; feel free to comment in the thread when you're ready to continue working on it 🙂 You can also catch us in Slack if you want to discuss.

stale[bot] avatar Dec 18 '22 07:12 stale[bot]