• Mohamed Aziz Mejri's avatar
    Neon integration (#3178) · 661e1438
    Mohamed Aziz Mejri 提交于
    ### Context for review bots :
    
    1.The agent can execute sql commands directly on production , this is
    fine because they user may choose to start with a simple setup first and
    use the production database directly (in a follow up PR we should give
    the users the ability to sync dev branch to match production branch).The
    user also has the possiblity to revert database changes .
    2.Right now there’s no warning for destructive changes before migration
    , this is fine for now as we're gonna add a follow up PR for this
    3.When linking an existing neon project that doesnt have a developement
    database , we're setting the default branch as the active branch . This
    is fine because the user may prefer to start a simple setup at the
    beginning and work directly on the production database , in addition
    since the project was created outside dyad it may have a developement
    branch named differently so it wouldnt be wise to auto create one here .
    However, in a follow up PR we should allow the user to manually create
    branches and sync dev branch to match production .
    4.neon is only available for next.js now
    5.We dont have enough tests for now, i am gonna add a follow-up PR to
    cover this
    <!-- devin-review-badge-begin -->
    
    ---
    
    <a href="https://app.devin.ai/review/dyad-sh/dyad/pull/3178"
    target="_blank">
      <picture>
    <source media="(prefers-color-scheme: dark)"
    srcset="https://static.devin.ai/assets/gh-open-in-devin-review-dark.svg?v=1">
    <img
    src="https://static.devin.ai/assets/gh-open-in-devin-review-light.svg?v=1"
    alt="Open with Devin">
      </picture>
    </a>
    <!-- devin-review-badge-end -->
    
    ---------
    Co-authored-by: 's avatarClaude Opus 4.6 (1M context) <noreply@anthropic.com>
    661e1438
supabase_stale_ui.spec.ts_supabase---stale-ui-1.aria.yml 643 Bytes