Skip to content

fix: use CliExitError for invalid load path errors#481

Merged
theoephraim merged 2 commits intomainfrom
fix/load-path-errors
Mar 24, 2026
Merged

fix: use CliExitError for invalid load path errors#481
theoephraim merged 2 commits intomainfrom
fix/load-path-errors

Conversation

@theoephraim
Copy link
Member

@theoephraim theoephraim commented Mar 24, 2026

Summary

  • Invalid load path errors in loadVarlockEnvGraph now throw CliExitError instead of using console.error + gracefulExit, for consistent error handling across the CLI
  • Fix missing trailing comma in type-generation.ts

Test plan

  • Run varlock load --path /nonexistent and verify the error is formatted consistently with other CLI errors
  • Run varlock load with an invalid varlock.loadPath in package.json and verify the same

🤖 Generated with Claude Code

theoephraim and others added 2 commits March 24, 2026 00:15
…r + gracefulExit

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@changeset-bot
Copy link

changeset-bot bot commented Mar 24, 2026

🦋 Changeset detected

Latest commit: fc54f38

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

This PR includes changesets to release 12 packages
Name Type
varlock Patch
@varlock/astro-integration Patch
@varlock/nextjs-integration Patch
@varlock/vite-integration Patch
@varlock/1password-plugin Patch
@varlock/aws-secrets-plugin Patch
@varlock/azure-key-vault-plugin Patch
@varlock/bitwarden-plugin Patch
@varlock/google-secret-manager-plugin Patch
@varlock/hashicorp-vault-plugin Patch
@varlock/infisical-plugin Patch
@varlock/pass-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

@pkg-pr-new
Copy link

pkg-pr-new bot commented Mar 24, 2026

Open in StackBlitz

npm i https://pkg.pr.new/varlock@481

commit: fc54f38

@theoephraim theoephraim merged commit 80c0475 into main Mar 24, 2026
17 checks passed
theoephraim added a commit that referenced this pull request Mar 25, 2026
* fix: throw CliExitError for invalid load path instead of console.error + gracefulExit

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* fix: add missing trailing comma in type-generation.ts

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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.

1 participant