Skip to content

fix: remove double quotes from pgpm env output#723

Merged
pyramation merged 1 commit intomainfrom
devin/1771138222-fix-pgpm-env-quotes
Feb 15, 2026
Merged

fix: remove double quotes from pgpm env output#723
pyramation merged 1 commit intomainfrom
devin/1771138222-fix-pgpm-env-quotes

Conversation

@pyramation
Copy link
Contributor

Summary

Removes the double-quote wrapping from pgpm env export statements. Previously the output was:

export PGHOST="localhost"
export PGPORT="5432"

Now it outputs:

export PGHOST=localhost
export PGPORT=5432

The escaping logic for backslashes and embedded quotes within values was also removed since the quotes are no longer present.

Review & Testing Checklist for Human

  • Edge case: passwords with spaces or shell metacharacters. The old code quoted and escaped values defensively. The new code outputs bare values. If a profile ever has a password like my password or p@ss$word, eval "$(pgpm env)" will break. Verify this is acceptable for all current and expected profiles (default + supabase).
  • Run pgpm env and pgpm env --supabase locally and confirm clean output

Notes

@devin-ai-integration
Copy link
Contributor

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@pyramation pyramation merged commit 368f35c into main Feb 15, 2026
43 checks passed
@pyramation pyramation deleted the devin/1771138222-fix-pgpm-env-quotes branch February 15, 2026 07:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant