Skip to content

Add @public item decorator as alias for @sensitive=false#274

Merged
philmillman merged 4 commits intomainfrom
copilot/add-public-item-decorator
Feb 6, 2026
Merged

Add @public item decorator as alias for @sensitive=false#274
philmillman merged 4 commits intomainfrom
copilot/add-public-item-decorator

Conversation

Copy link
Contributor

Copilot AI commented Feb 6, 2026

Implementation Plan for @public Decorator

  • Add @public to the decorator registry in decorators.ts with incompatibleWith: ['sensitive']
  • Update processSensitive() method in config-item.ts to handle @public decorator (similar to how processRequired() handles @optional)
  • Add comprehensive tests in sensitive-decorator.test.ts
  • Run tests to verify implementation
  • Request code review
  • Address review feedback (confirmed that duplicate handling is already in place via existing validation logic)
  • Add documentation for @public decorator in item-decorators.mdx
  • Add changeset for version tracking
  • Fix linting errors (trailing spaces)
Original prompt

This section details on the original issue you should resolve

<issue_title>[Feature Request] Add @public item decorator - alias for @sensitive=false</issue_title>
<issue_description>### Description

While it is not totally necessary, we have @required / @optional, and soon will have @dynamic / @static, so if feels weird that @sensitive does not have it's counterpart.</issue_description>

<agent_instructions>This is the opposite of @sensitive - it should be implemented in a similar way to @required/@optional</agent_instructions>

Comments on the Issue (you are @copilot in this section)


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

@changeset-bot
Copy link

changeset-bot bot commented Feb 6, 2026

🦋 Changeset detected

Latest commit: 060fafb

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

This PR includes changesets to release 6 packages
Name Type
varlock Patch
@varlock/astro-integration Patch
@varlock/nextjs-integration Patch
@varlock/vite-integration Patch
@varlock/1password-plugin Patch
@varlock/google-secret-manager-plugin Patch

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

Co-authored-by: theoephraim <1158956+theoephraim@users.noreply.github.com>
Copilot AI changed the title [WIP] Add @public item decorator as alias for @sensitive=false Add @public item decorator as alias for @sensitive=false Feb 6, 2026
Copilot AI requested a review from theoephraim February 6, 2026 06:06
Copy link
Member

@theoephraim theoephraim left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@copilot - code looks good. Need to add to docs, and add a changeset

Co-authored-by: theoephraim <1158956+theoephraim@users.noreply.github.com>
@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Feb 6, 2026

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 6a1e9cd Commit Preview URL

Branch Preview URL
Feb 06 2026, 07:09 AM

Copy link
Contributor Author

Copilot AI commented Feb 6, 2026

@copilot - code looks good. Need to add to docs, and add a changeset

Done! Added documentation to item-decorators.mdx and created a changeset. (6a1e9cd)

@pkg-pr-new
Copy link

pkg-pr-new bot commented Feb 6, 2026

Open in StackBlitz

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

commit: 060fafb

@pkg-pr-new
Copy link

pkg-pr-new bot commented Feb 6, 2026

Open in StackBlitz

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

commit: 6a1e9cd

@philmillman
Copy link
Member

@copilot fix linting errors

Co-authored-by: philmillman <3722211+philmillman@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Feb 6, 2026

@copilot fix linting errors

Fixed! Removed trailing spaces in config-item.ts and sensitive-decorator.test.ts. (060fafb)

Copilot AI requested a review from philmillman February 6, 2026 15:46
@philmillman philmillman marked this pull request as ready for review February 6, 2026 15:51
@philmillman philmillman merged commit c872e71 into main Feb 6, 2026
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature Request] Add @public item decorator - alias for @sensitive=false

3 participants