Skip to content

Add Keeper Security plugin for Secrets Manager#545

Open
Copilot wants to merge 5 commits intomainfrom
copilot/add-keeper-plugin-support
Open

Add Keeper Security plugin for Secrets Manager#545
Copilot wants to merge 5 commits intomainfrom
copilot/add-keeper-plugin-support

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 2, 2026 •
edited
Loading

  • Replace process-phase UID collection with resolve-phase batching (like the 1password plugin):
    • Remove old secretsCache / fetchSecrets(recordsFilter?) / _fetchSecrets approach
    • Add pendingBatch + _executeBatch() using createDeferredPromise + setImmediate
    • fetchSecrets(uid?) accumulates UIDs into the batch; setImmediate fires a single filtered SDK call
    • Add KEEPER_UID_RE + extractUidFromRef() helper; used in getSecretByNotation / getSecretField to pass UID to fetchSecrets
    • No changes needed to process() — all batching happens at resolve time

@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Apr 2, 2026 •
edited
Loading

🦋 Changeset detected

Latest commit: 70781a6

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@varlock/keeper-plugin Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copilot AI linked an issue Apr 2, 2026 that may be closed by this pull request
[Feature Request] An official plugin for Keeper password manager #366
Open
@theoephraim
feat: add Keeper Security plugin for Secrets Manager
d0e4b85 
Implements @varlock/keeper-plugin with:
- @initKeeper() root decorator for initialization
- keeper() resolver for fetching secrets by UID, title, or notation
- keeperSmToken data type for config token validation
- Multiple instance support
- Comprehensive tests and documentation

Agent-Logs-Url: https://github.com/dmno-dev/varlock/sessions/228e9664-29fc-48dc-8c95-7905e2089951

Co-authored-by: theoephraim <1158956+theoephraim@users.noreply.github.com>
Copilot AI changed the title [WIP] Add official plugin for Keeper password manager Add Keeper Security plugin for Secrets Manager Apr 2, 2026
Copilot AI requested a review from theoephraim April 2, 2026 21:07
@socket-security
Copy link
Copy Markdown

socket-security bot commented Apr 3, 2026 •
edited
Loading

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security		 Vulnerability Quality Maintenance License
Added@​keeper-security/​secrets-manager-core@​17.4.0901009291100

View full report

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new bot commented Apr 3, 2026 •
edited
Loading

Open in StackBlitz

npm i https://pkg.pr.new/dmno-dev/varlock@545

npm i https://pkg.pr.new/dmno-dev/varlock/@varlock/astro-integration@545

npm i https://pkg.pr.new/dmno-dev/varlock/@varlock/vite-integration@545

npm i https://pkg.pr.new/dmno-dev/varlock/@varlock/keeper-plugin@545

commit: 08b1d75

@philmillman
Copy link
Copy Markdown
Member

philmillman commented Apr 3, 2026

@copilot add a docs page

@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages bot commented Apr 3, 2026 •
edited
Loading

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs		 varlock-website 3c0482e Commit Preview URL

Branch Preview URL		 Apr 03 2026, 02:41 PM

Copy link
Copy Markdown
Contributor Author

Copilot AI commented Apr 3, 2026

@copilot add a docs page

Added in 3c0482e — created packages/varlock-website/src/content/docs/plugins/keeper.mdx with full documentation (installation, setup guide, secret access patterns, reference for @initKeeper() / keeperSmToken / keeper(), troubleshooting) and added the "Keeper" sidebar entry in astro.config.ts.

Copilot AI requested a review from philmillman April 3, 2026 14:40
@theoephraim theoephraim marked this pull request as ready for review April 7, 2026 06:13
Copilot stopped work on behalf of theoephraim due to an error April 7, 2026 06:25
The session was cancelled by the user.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@theoephraim theoephraim Awaiting requested review from theoephraim

@philmillman philmillman Awaiting requested review from philmillman

At least 1 approving review is required to merge this pull request.

Assignees

@theoephraim theoephraim

Copilot code review Copilot

Labels

maintainer

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[Feature Request] An official plugin for Keeper password manager

3 participants

@philmillman @theoephraim