Skip to content

docs(vite-plus): give vp dev the leased DATABASE_URL via cedarpg run --attach - #47

Merged
simoncrypta merged 2 commits into
mainfrom
docs/vite-plus-env
Oct 8, 2026
Merged

simoncrypta merged 2 commits into
mainfrom
docs/vite-plus-env

Conversation

@simoncrypta

Copy link
Copy Markdown
Collaborator

Summary

  • cedar-pg-i7s.4 — Vite+ README / @cedarjs/pg/vite-plus example: dev is now cedarpg run --attach --mode=dev -- vp dev (with cache: false).
  • Vite+ dependsOn does not forward env, so the old command: "vp dev" + env: ["DATABASE_URL"] never saw the leased URL. test unchanged.

Testing

  • vp check, vp test, vp run smoke (docs/example only)

…--attach

Vite+ dependsOn does not forward env, and task env only fingerprints and
passes through the vp process env, so the old dev example never saw the
lease URL. Wrap vp dev with attach-only cedarpg run; db:acquire stays the
dependency. test is unchanged: the stock Vitest setup acquires itself.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Oct 8, 2026 •

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: cbca8337-cfc5-488f-9b83-10c371cb7fbe
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@pkg-pr-new

pkg-pr-new Bot commented Oct 8, 2026

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/@cedarjs/pg@47

commit: d3b4660

@simoncrypta
simoncrypta merged commit 0bc5308 into main Oct 8, 2026
2 checks passed
@simoncrypta
simoncrypta deleted the docs/vite-plus-env branch October 8, 2026 16:27
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