Skip to content

update to Prisma v7 - #253

Open
samualtnorman wants to merge 1 commit into
solidjs:mainfrom
samualtnorman:update-prisma
Open

update to Prisma v7#253
samualtnorman wants to merge 1 commit into
solidjs:mainfrom
samualtnorman:update-prisma

Conversation

@samualtnorman

Copy link
Copy Markdown

No description provided.

@samualtnorman

samualtnorman commented Mar 13, 2026

Copy link
Copy Markdown
Author

prisma can cause problems with SolidStart when you use it in a server function that's declared in the same file as some client code. I wrote a custom babel plugin for this that we're using in production that I can add to this template in a follow up PR

}
}

export default defineConfig({

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

is this an out of the box prisma.config.ts or a custom one?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

this is a custom one, this was a while ago though so I don't remember what my thinking was. sorry about that, will take a look

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

so the default prisma config is apparently this

import "dotenv/config";
import { defineConfig, env } from "prisma/config";

export default defineConfig({
  schema: "prisma/schema.prisma",
  migrations: {
    path: "prisma/migrations",
  },
  datasource: {
    url: env("DATABASE_URL"),
  },
});

but I think this gave me problems, I can try and rediscover what these problems were

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

i think its fine. Looks like there is conflicts now

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

I've rebased and applied the changes to both solid-start-v{1,2}/with-prisma

@socket-security

Copy link
Copy Markdown

Dependency limit exceeded — report not shown.

This pull request scan exceeded the 10,000-dependency limit applied to this scan, so the results are incomplete and may be inaccurate. To avoid reporting false positives, Socket has not posted a report.

Upgrade your plan to raise the dependency limit and get complete reports, or view the partial scan in the dashboard.

Socket is always free for open source. If this is a non-commercial open source project, contact us to request a free Team account.

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.

2 participants