Skip to content

Conversation

@AdeboyeDN
Copy link

@AdeboyeDN AdeboyeDN commented Jan 20, 2026

Summary by CodeRabbit

  • Documentation

    • Added a comprehensive deployment guide for running Node.js apps with Prisma + Postgres on Northflank. Covers prerequisites, step‑by‑step deployment workflow (including generating the Prisma Client during build), service creation and build configuration, setting DATABASE_URL and environment variables, options for running migrations, scaling and health checks, troubleshooting, and practical configuration examples.
  • Chores

    • Added "Northflank" to the spellchecker vocabulary.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 20, 2026

Walkthrough

Adds documentation and a spelling entry: a new MDX guide describing how to deploy a Node.js app using Prisma/Postgres to Northflank (prereqs, build/runtime config, env vars, migration/run options, health checks, scaling, troubleshooting) and a cSpell addition for "Northflank".

Changes

Cohort / File(s) Summary
Northflank Deployment Guide
content/250-postgres/350-integrations/700-northflank.mdx
New MDX file (+~175 lines) providing step‑by‑step deployment instructions: generate Prisma Client in postinstall, Northflank service and build settings, DATABASE_URL setup, migration/run options (CLI, command override, Job), example package.json and prisma.config.ts, Dockerfile snippets, health checks, scaling, and troubleshooting.
Spellcheck Vocabulary
cSpell.json
Added "Northflank" to the words array (+1 line) to prevent false positive spellcheck flags.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately and concisely summarizes the main change: adding documentation for deploying Prisma Postgres to Northflank.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


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 and usage tips.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 2

🤖 Fix all issues with AI agents
In `@content/250-postgres/350-integrations/700-northflank.mdx`:
- Line 142: The subheading string "Prisma schema not found or Client not
generated correctly" should be updated to sentence case and explicitly reference
the product name "Prisma Client"; replace that heading (the line containing "###
Prisma schema not found or Client not generated correctly") with "### Prisma
schema not found or Prisma Client not generated correctly" so the casing is
sentence case and the product name is preserved.
- Around line 95-115: Update the three option headings to sentence case: change
"Option 1: Use the Northflank CLI", "Option 2: Use command override", and
"Option 3: Use a Job" so only the first word and proper nouns remain capitalized
(e.g., keep "Northflank" and "CLI" capitalized but change "Job" to "job"); edit
the heading strings in the file where those exact headings appear (look for the
lines containing "Option 1:", "Option 2:", and "Option 3:") and apply
sentence-case formatting consistently.

@AdeboyeDN AdeboyeDN force-pushed the doc/northflank-integration-guide branch from dc27391 to 3787e6b Compare January 20, 2026 12:47
coderabbitai[bot]
coderabbitai bot previously approved these changes Jan 20, 2026
@AdeboyeDN
Copy link
Author

cc @ankur-arch

Kindly check this out

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Fix all issues with AI agents
In `@content/250-postgres/350-integrations/700-northflank.mdx`:
- Around line 95-99: The northflank CLI example for running Prisma migrations is
missing required flags and will fail; update the `northflank exec service`
invocation to include the project and service identifiers (use `--projectId` and
`--serviceId` or the short forms `--project` and `--service`) so the command
becomes e.g. `northflank exec service --project <project-id> --service
<service-id> --cmd "npx prisma migrate deploy"`; ensure this replacement is
applied to the block containing the `northflank exec service` example.
🧹 Nitpick comments (1)
content/250-postgres/350-integrations/700-northflank.mdx (1)

75-89: Clarify when prisma.config.ts applies (vs schema.prisma).
Some Prisma ORM projects won’t have prisma.config.ts; a short note helps readers map the env var to their actual config file.

📌 Suggested clarification
-Ensure that the data source in your `prisma.config.ts` file is configured to use the `DATABASE_URL` environment variable:
+Ensure that the data source in your `prisma.config.ts` file is configured to use the `DATABASE_URL` environment variable. If your project doesn't use `prisma.config.ts`, make sure the datasource in `schema.prisma` uses `env("DATABASE_URL")` instead.

@ankur-arch
Copy link
Contributor

ankur-arch commented Jan 21, 2026

Thanks for putting up the PR, @AdeboyeDN! You’ll need to add Northflank to the repo’s cspell.json, since it’s not currently recognized as a valid spelling. Once that’s done, I can merge it.

@AdeboyeDN
Copy link
Author

Thanks for putting up the PR, @AdeboyeDN! You’ll need to add Northflank to the repo’s cspell.json, since it’s not currently recognized as a valid spelling. Once that’s done, I can merge it.

Hi, this should be good to merge now

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