fix minor issues in the nest.js v3 docs, switch to SQLite by lsmith77 · Pull Request #532 · zenstackhq/zenstack-docs

lsmith77 · 2025-12-19T14:57:56Z

If you prefer I can keep PostgreSQL but I think SQLite makes it easier for people to get this setup quickly. How to deal with PostgreSQL is then handled elsewhere in the docs.

Summary by CodeRabbit

Documentation
- Updated NestJS recipe to use SQLite-backed examples instead of PostgreSQL.
- Reorganized example file structure and adjusted import paths for clarity.
- Clarified module exports and updated code samples for consistent access-control and API-handler examples.

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

vercel · 2025-12-19T14:58:00Z

@lsmith77 is attempting to deploy a commit to the ZenStack Team on Vercel.

A member of the Team first needs to authorize it.

coderabbitai · 2025-12-19T15:00:11Z

Walkthrough

Reworks NestJS recipe docs: replaces PostgreSQL dialect with SqliteDialect and a SQLite DB in samples; updates file path annotations and import paths to use src/ layout; adds explicit export class AppModule and includes schema import in the API handler example.

Changes

Cohort / File(s)	Change Summary
NestJS Recipe Documentation `versioned_docs/version-3.x/recipe/nestjs.md`	Replaced Postgres setup with `SqliteDialect` and a SQLite-backed database in `DbService` sample; updated file path annotations from root-level names to `src/db/db.service.ts`, `src/app.module.ts`, `src/app.controller.ts`; adjusted import paths to the new locations; added `export class AppModule` and added schema import to API handler example.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Consistent, repetitive documentation changes across a single file.
Focused on examples/path adjustments and a dialect swap; no executable code outside docs to validate.

Possibly related PRs

docs: nestjs guide for v3 #526: Modifies the same NestJS recipe documentation with overlapping example and import/path updates.

Pre-merge checks and finishing touches

✅ 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 describes the main changes: fixing minor documentation issues and switching the example database from PostgreSQL to SQLite in the NestJS v3 docs.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing touches

🧪 Generate unit tests (beta)

Create PR with unit tests
Post copyable unit tests in a comment

📜 Recent review details

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between e99836f and a6b149c.

📒 Files selected for processing (1)

versioned_docs/version-3.x/recipe/nestjs.md (4 hunks)

🔇 Additional comments (6)

versioned_docs/version-3.x/recipe/nestjs.md (6)
13-29: Import paths and SQLite setup look correct.
All imports properly reference the new src/ directory structure. The path ../../zenstack/schema correctly resolves from src/db/db.service.ts to the root-level zenstack/schema. SQLite configuration with better-sqlite3 is straightforward and makes the quick-start simpler than PostgreSQL.
33-43: AppModule structure and imports are correct.
The explicit export class AppModule {} is good practice. Import path ./db/db.service correctly resolves from src/app.module.ts to src/db/db.service.ts.
47-60: AppController import paths verified.
Import path ./db/db.service correctly resolves from src/app.controller.ts.
68-105: Access-controlled ORM AppModule is consistent.
The explicit export class AppModule {} and import paths match the plain ORM pattern, maintaining consistency across examples.
109-124: Access-controlled ORM AppController is consistent.
Import paths align with the plain ORM example.
130-168: CRUD service example properly imports schema.
The schema import on Line 135 (../zenstack/schema) correctly resolves from src/app.controller.ts to the root-level schema file. This aligns with the API handler setup shown in the code.

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.}

coderabbitai

Actionable comments posted: 0

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (2)

versioned_docs/version-3.x/recipe/nestjs.md (2)
47-60: Fix incorrect import path on line 49.
The import path './db.service' is inconsistent with the file structure. Since DbService is located at src/db/db.service.ts, the correct import from src/app.controller.ts should be './db/db.service'. This matches the import path used in the AppModule (line 36) and the CRUD example (line 134).
🔎 Proposed fix
 import { Controller, Get } from '@nestjs/common';
- import { DbService } from './db.service';+ import { DbService } from './db/db.service';
109-124: Fix incorrect import path on line 111.
The import path './db.service' is inconsistent with the file structure. Since DbService is located at src/db/db.service.ts, the correct import from src/app.controller.ts should be './db/db.service'. This matches the import path used in the access-controlled AppModule (line 71) and the CRUD example (line 134).
🔎 Proposed fix
 import { Controller, Get, Inject } from '@nestjs/common';
- import { DbService } from './db.service';+ import { DbService } from './db/db.service';

📜 Review details

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 89846e0 and e99836f.

📒 Files selected for processing (1)

versioned_docs/version-3.x/recipe/nestjs.md (4 hunks)

🔇 Additional comments (3)

versioned_docs/version-3.x/recipe/nestjs.md (3)
13-43: SQLite migration and module setup look good.
The DbService correctly switches to SqliteDialect with better-sqlite3, and the AppModule properly registers it as a provider with correct import paths.
68-105: Access-controlled module setup is correct.
The AppModule properly registers both the standard and access-controlled DbService instances, with correct imports and the explicit export class AppModule {} declaration.
130-168: CRUD API handler example is well-structured.
The import paths are correct ('./db/db.service' and '../zenstack/schema'), and the example properly demonstrates injecting the access-controlled DbService and initializing the RestApiHandler with the schema.

ymc9

LGTM. Thanks @lsmith77 !

lsmith77 · 2025-12-19T14:57:56Z

If you prefer I can keep PostgreSQL but I think SQLite makes it easier for people to get this setup quickly. How to deal with PostgreSQL is then handled elsewhere in the docs.

Summary by CodeRabbit

Documentation
- Updated NestJS recipe to use SQLite-backed examples instead of PostgreSQL.
- Reorganized example file structure and adjusted import paths for clarity.
- Clarified module exports and updated code samples for consistent access-control and API-handler examples.

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

vercel · 2025-12-19T14:58:00Z

@lsmith77 is attempting to deploy a commit to the ZenStack Team on Vercel.

A member of the Team first needs to authorize it.

coderabbitai · 2025-12-19T15:00:11Z

Walkthrough

Reworks NestJS recipe docs: replaces PostgreSQL dialect with SqliteDialect and a SQLite DB in samples; updates file path annotations and import paths to use src/ layout; adds explicit export class AppModule and includes schema import in the API handler example.

Changes

Cohort / File(s)	Change Summary
NestJS Recipe Documentation `versioned_docs/version-3.x/recipe/nestjs.md`	Replaced Postgres setup with `SqliteDialect` and a SQLite-backed database in `DbService` sample; updated file path annotations from root-level names to `src/db/db.service.ts`, `src/app.module.ts`, `src/app.controller.ts`; adjusted import paths to the new locations; added `export class AppModule` and added schema import to API handler example.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Consistent, repetitive documentation changes across a single file.
Focused on examples/path adjustments and a dialect swap; no executable code outside docs to validate.

Possibly related PRs

docs: nestjs guide for v3 #526: Modifies the same NestJS recipe documentation with overlapping example and import/path updates.

Pre-merge checks and finishing touches

✅ 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 describes the main changes: fixing minor documentation issues and switching the example database from PostgreSQL to SQLite in the NestJS v3 docs.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing touches

🧪 Generate unit tests (beta)

Create PR with unit tests
Post copyable unit tests in a comment

📜 Recent review details

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between e99836f and a6b149c.

📒 Files selected for processing (1)

versioned_docs/version-3.x/recipe/nestjs.md (4 hunks)

🔇 Additional comments (6)

versioned_docs/version-3.x/recipe/nestjs.md (6)
13-29: Import paths and SQLite setup look correct.
All imports properly reference the new src/ directory structure. The path ../../zenstack/schema correctly resolves from src/db/db.service.ts to the root-level zenstack/schema. SQLite configuration with better-sqlite3 is straightforward and makes the quick-start simpler than PostgreSQL.
33-43: AppModule structure and imports are correct.
The explicit export class AppModule {} is good practice. Import path ./db/db.service correctly resolves from src/app.module.ts to src/db/db.service.ts.
47-60: AppController import paths verified.
Import path ./db/db.service correctly resolves from src/app.controller.ts.
68-105: Access-controlled ORM AppModule is consistent.
The explicit export class AppModule {} and import paths match the plain ORM pattern, maintaining consistency across examples.
109-124: Access-controlled ORM AppController is consistent.
Import paths align with the plain ORM example.
130-168: CRUD service example properly imports schema.
The schema import on Line 135 (../zenstack/schema) correctly resolves from src/app.controller.ts to the root-level schema file. This aligns with the API handler setup shown in the code.

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.}

coderabbitai

Actionable comments posted: 0

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (2)

versioned_docs/version-3.x/recipe/nestjs.md (2)
47-60: Fix incorrect import path on line 49.
The import path './db.service' is inconsistent with the file structure. Since DbService is located at src/db/db.service.ts, the correct import from src/app.controller.ts should be './db/db.service'. This matches the import path used in the AppModule (line 36) and the CRUD example (line 134).
🔎 Proposed fix
 import { Controller, Get } from '@nestjs/common';
- import { DbService } from './db.service';+ import { DbService } from './db/db.service';
109-124: Fix incorrect import path on line 111.
The import path './db.service' is inconsistent with the file structure. Since DbService is located at src/db/db.service.ts, the correct import from src/app.controller.ts should be './db/db.service'. This matches the import path used in the access-controlled AppModule (line 71) and the CRUD example (line 134).
🔎 Proposed fix
 import { Controller, Get, Inject } from '@nestjs/common';
- import { DbService } from './db.service';+ import { DbService } from './db/db.service';

📜 Review details

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 89846e0 and e99836f.

📒 Files selected for processing (1)

versioned_docs/version-3.x/recipe/nestjs.md (4 hunks)

🔇 Additional comments (3)

versioned_docs/version-3.x/recipe/nestjs.md (3)
13-43: SQLite migration and module setup look good.
The DbService correctly switches to SqliteDialect with better-sqlite3, and the AppModule properly registers it as a provider with correct import paths.
68-105: Access-controlled module setup is correct.
The AppModule properly registers both the standard and access-controlled DbService instances, with correct imports and the explicit export class AppModule {} declaration.
130-168: CRUD API handler example is well-structured.
The import paths are correct ('./db/db.service' and '../zenstack/schema'), and the example properly demonstrates injecting the access-controlled DbService and initializing the RestApiHandler with the schema.

ymc9

LGTM. Thanks @lsmith77 !

lsmith77 · 2025-12-19T14:57:56Z

If you prefer I can keep PostgreSQL but I think SQLite makes it easier for people to get this setup quickly. How to deal with PostgreSQL is then handled elsewhere in the docs.

Summary by CodeRabbit

Documentation
- Updated NestJS recipe to use SQLite-backed examples instead of PostgreSQL.
- Reorganized example file structure and adjusted import paths for clarity.
- Clarified module exports and updated code samples for consistent access-control and API-handler examples.

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

vercel · 2025-12-19T14:58:00Z

@lsmith77 is attempting to deploy a commit to the ZenStack Team on Vercel.

A member of the Team first needs to authorize it.

coderabbitai · 2025-12-19T15:00:11Z

Walkthrough

Reworks NestJS recipe docs: replaces PostgreSQL dialect with SqliteDialect and a SQLite DB in samples; updates file path annotations and import paths to use src/ layout; adds explicit export class AppModule and includes schema import in the API handler example.

Changes

Cohort / File(s)	Change Summary
NestJS Recipe Documentation `versioned_docs/version-3.x/recipe/nestjs.md`	Replaced Postgres setup with `SqliteDialect` and a SQLite-backed database in `DbService` sample; updated file path annotations from root-level names to `src/db/db.service.ts`, `src/app.module.ts`, `src/app.controller.ts`; adjusted import paths to the new locations; added `export class AppModule` and added schema import to API handler example.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Consistent, repetitive documentation changes across a single file.
Focused on examples/path adjustments and a dialect swap; no executable code outside docs to validate.

Possibly related PRs

docs: nestjs guide for v3 #526: Modifies the same NestJS recipe documentation with overlapping example and import/path updates.

Pre-merge checks and finishing touches

✅ 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 describes the main changes: fixing minor documentation issues and switching the example database from PostgreSQL to SQLite in the NestJS v3 docs.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing touches

🧪 Generate unit tests (beta)

Create PR with unit tests
Post copyable unit tests in a comment

📜 Recent review details

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between e99836f and a6b149c.

📒 Files selected for processing (1)

versioned_docs/version-3.x/recipe/nestjs.md (4 hunks)

🔇 Additional comments (6)

versioned_docs/version-3.x/recipe/nestjs.md (6)
13-29: Import paths and SQLite setup look correct.
All imports properly reference the new src/ directory structure. The path ../../zenstack/schema correctly resolves from src/db/db.service.ts to the root-level zenstack/schema. SQLite configuration with better-sqlite3 is straightforward and makes the quick-start simpler than PostgreSQL.
33-43: AppModule structure and imports are correct.
The explicit export class AppModule {} is good practice. Import path ./db/db.service correctly resolves from src/app.module.ts to src/db/db.service.ts.
47-60: AppController import paths verified.
Import path ./db/db.service correctly resolves from src/app.controller.ts.
68-105: Access-controlled ORM AppModule is consistent.
The explicit export class AppModule {} and import paths match the plain ORM pattern, maintaining consistency across examples.
109-124: Access-controlled ORM AppController is consistent.
Import paths align with the plain ORM example.
130-168: CRUD service example properly imports schema.
The schema import on Line 135 (../zenstack/schema) correctly resolves from src/app.controller.ts to the root-level schema file. This aligns with the API handler setup shown in the code.

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.}

coderabbitai

Actionable comments posted: 0

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (2)

versioned_docs/version-3.x/recipe/nestjs.md (2)
47-60: Fix incorrect import path on line 49.
The import path './db.service' is inconsistent with the file structure. Since DbService is located at src/db/db.service.ts, the correct import from src/app.controller.ts should be './db/db.service'. This matches the import path used in the AppModule (line 36) and the CRUD example (line 134).
🔎 Proposed fix
 import { Controller, Get } from '@nestjs/common';
- import { DbService } from './db.service';+ import { DbService } from './db/db.service';
109-124: Fix incorrect import path on line 111.
The import path './db.service' is inconsistent with the file structure. Since DbService is located at src/db/db.service.ts, the correct import from src/app.controller.ts should be './db/db.service'. This matches the import path used in the access-controlled AppModule (line 71) and the CRUD example (line 134).
🔎 Proposed fix
 import { Controller, Get, Inject } from '@nestjs/common';
- import { DbService } from './db.service';+ import { DbService } from './db/db.service';

📜 Review details

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 89846e0 and e99836f.

📒 Files selected for processing (1)

versioned_docs/version-3.x/recipe/nestjs.md (4 hunks)

🔇 Additional comments (3)

versioned_docs/version-3.x/recipe/nestjs.md (3)
13-43: SQLite migration and module setup look good.
The DbService correctly switches to SqliteDialect with better-sqlite3, and the AppModule properly registers it as a provider with correct import paths.
68-105: Access-controlled module setup is correct.
The AppModule properly registers both the standard and access-controlled DbService instances, with correct imports and the explicit export class AppModule {} declaration.
130-168: CRUD API handler example is well-structured.
The import paths are correct ('./db/db.service' and '../zenstack/schema'), and the example properly demonstrates injecting the access-controlled DbService and initializing the RestApiHandler with the schema.

ymc9

LGTM. Thanks @lsmith77 !

lsmith77 · 2025-12-19T14:57:56Z

If you prefer I can keep PostgreSQL but I think SQLite makes it easier for people to get this setup quickly. How to deal with PostgreSQL is then handled elsewhere in the docs.

Summary by CodeRabbit

Documentation
- Updated NestJS recipe to use SQLite-backed examples instead of PostgreSQL.
- Reorganized example file structure and adjusted import paths for clarity.
- Clarified module exports and updated code samples for consistent access-control and API-handler examples.

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

vercel · 2025-12-19T14:58:00Z

@lsmith77 is attempting to deploy a commit to the ZenStack Team on Vercel.

A member of the Team first needs to authorize it.

coderabbitai · 2025-12-19T15:00:11Z

Walkthrough

Reworks NestJS recipe docs: replaces PostgreSQL dialect with SqliteDialect and a SQLite DB in samples; updates file path annotations and import paths to use src/ layout; adds explicit export class AppModule and includes schema import in the API handler example.

Changes

Cohort / File(s)	Change Summary
NestJS Recipe Documentation `versioned_docs/version-3.x/recipe/nestjs.md`	Replaced Postgres setup with `SqliteDialect` and a SQLite-backed database in `DbService` sample; updated file path annotations from root-level names to `src/db/db.service.ts`, `src/app.module.ts`, `src/app.controller.ts`; adjusted import paths to the new locations; added `export class AppModule` and added schema import to API handler example.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Consistent, repetitive documentation changes across a single file.
Focused on examples/path adjustments and a dialect swap; no executable code outside docs to validate.

Possibly related PRs

docs: nestjs guide for v3 #526: Modifies the same NestJS recipe documentation with overlapping example and import/path updates.

Pre-merge checks and finishing touches

✅ 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 describes the main changes: fixing minor documentation issues and switching the example database from PostgreSQL to SQLite in the NestJS v3 docs.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing touches

🧪 Generate unit tests (beta)

Create PR with unit tests
Post copyable unit tests in a comment

📜 Recent review details

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between e99836f and a6b149c.

📒 Files selected for processing (1)

versioned_docs/version-3.x/recipe/nestjs.md (4 hunks)

🔇 Additional comments (6)

versioned_docs/version-3.x/recipe/nestjs.md (6)
13-29: Import paths and SQLite setup look correct.
All imports properly reference the new src/ directory structure. The path ../../zenstack/schema correctly resolves from src/db/db.service.ts to the root-level zenstack/schema. SQLite configuration with better-sqlite3 is straightforward and makes the quick-start simpler than PostgreSQL.
33-43: AppModule structure and imports are correct.
The explicit export class AppModule {} is good practice. Import path ./db/db.service correctly resolves from src/app.module.ts to src/db/db.service.ts.
47-60: AppController import paths verified.
Import path ./db/db.service correctly resolves from src/app.controller.ts.
68-105: Access-controlled ORM AppModule is consistent.
The explicit export class AppModule {} and import paths match the plain ORM pattern, maintaining consistency across examples.
109-124: Access-controlled ORM AppController is consistent.
Import paths align with the plain ORM example.
130-168: CRUD service example properly imports schema.
The schema import on Line 135 (../zenstack/schema) correctly resolves from src/app.controller.ts to the root-level schema file. This aligns with the API handler setup shown in the code.

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.}

coderabbitai

Actionable comments posted: 0

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (2)

versioned_docs/version-3.x/recipe/nestjs.md (2)
47-60: Fix incorrect import path on line 49.
The import path './db.service' is inconsistent with the file structure. Since DbService is located at src/db/db.service.ts, the correct import from src/app.controller.ts should be './db/db.service'. This matches the import path used in the AppModule (line 36) and the CRUD example (line 134).
🔎 Proposed fix
 import { Controller, Get } from '@nestjs/common';
- import { DbService } from './db.service';+ import { DbService } from './db/db.service';
109-124: Fix incorrect import path on line 111.
The import path './db.service' is inconsistent with the file structure. Since DbService is located at src/db/db.service.ts, the correct import from src/app.controller.ts should be './db/db.service'. This matches the import path used in the access-controlled AppModule (line 71) and the CRUD example (line 134).
🔎 Proposed fix
 import { Controller, Get, Inject } from '@nestjs/common';
- import { DbService } from './db.service';+ import { DbService } from './db/db.service';

📜 Review details

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 89846e0 and e99836f.

📒 Files selected for processing (1)

versioned_docs/version-3.x/recipe/nestjs.md (4 hunks)

🔇 Additional comments (3)

versioned_docs/version-3.x/recipe/nestjs.md (3)
13-43: SQLite migration and module setup look good.
The DbService correctly switches to SqliteDialect with better-sqlite3, and the AppModule properly registers it as a provider with correct import paths.
68-105: Access-controlled module setup is correct.
The AppModule properly registers both the standard and access-controlled DbService instances, with correct imports and the explicit export class AppModule {} declaration.
130-168: CRUD API handler example is well-structured.
The import paths are correct ('./db/db.service' and '../zenstack/schema'), and the example properly demonstrates injecting the access-controlled DbService and initializing the RestApiHandler with the schema.

ymc9

LGTM. Thanks @lsmith77 !

lsmith77 · 2025-12-19T14:57:56Z

If you prefer I can keep PostgreSQL but I think SQLite makes it easier for people to get this setup quickly. How to deal with PostgreSQL is then handled elsewhere in the docs.

Summary by CodeRabbit

Documentation
- Updated NestJS recipe to use SQLite-backed examples instead of PostgreSQL.
- Reorganized example file structure and adjusted import paths for clarity.
- Clarified module exports and updated code samples for consistent access-control and API-handler examples.

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

vercel · 2025-12-19T14:58:00Z

@lsmith77 is attempting to deploy a commit to the ZenStack Team on Vercel.

A member of the Team first needs to authorize it.

coderabbitai · 2025-12-19T15:00:11Z

Walkthrough

Reworks NestJS recipe docs: replaces PostgreSQL dialect with SqliteDialect and a SQLite DB in samples; updates file path annotations and import paths to use src/ layout; adds explicit export class AppModule and includes schema import in the API handler example.

Changes

Cohort / File(s)	Change Summary
NestJS Recipe Documentation `versioned_docs/version-3.x/recipe/nestjs.md`	Replaced Postgres setup with `SqliteDialect` and a SQLite-backed database in `DbService` sample; updated file path annotations from root-level names to `src/db/db.service.ts`, `src/app.module.ts`, `src/app.controller.ts`; adjusted import paths to the new locations; added `export class AppModule` and added schema import to API handler example.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Consistent, repetitive documentation changes across a single file.
Focused on examples/path adjustments and a dialect swap; no executable code outside docs to validate.

Possibly related PRs

docs: nestjs guide for v3 #526: Modifies the same NestJS recipe documentation with overlapping example and import/path updates.

Pre-merge checks and finishing touches

✅ 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 describes the main changes: fixing minor documentation issues and switching the example database from PostgreSQL to SQLite in the NestJS v3 docs.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing touches

🧪 Generate unit tests (beta)

Create PR with unit tests
Post copyable unit tests in a comment

📜 Recent review details

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between e99836f and a6b149c.

📒 Files selected for processing (1)

versioned_docs/version-3.x/recipe/nestjs.md (4 hunks)

🔇 Additional comments (6)

versioned_docs/version-3.x/recipe/nestjs.md (6)
13-29: Import paths and SQLite setup look correct.
All imports properly reference the new src/ directory structure. The path ../../zenstack/schema correctly resolves from src/db/db.service.ts to the root-level zenstack/schema. SQLite configuration with better-sqlite3 is straightforward and makes the quick-start simpler than PostgreSQL.
33-43: AppModule structure and imports are correct.
The explicit export class AppModule {} is good practice. Import path ./db/db.service correctly resolves from src/app.module.ts to src/db/db.service.ts.
47-60: AppController import paths verified.
Import path ./db/db.service correctly resolves from src/app.controller.ts.
68-105: Access-controlled ORM AppModule is consistent.
The explicit export class AppModule {} and import paths match the plain ORM pattern, maintaining consistency across examples.
109-124: Access-controlled ORM AppController is consistent.
Import paths align with the plain ORM example.
130-168: CRUD service example properly imports schema.
The schema import on Line 135 (../zenstack/schema) correctly resolves from src/app.controller.ts to the root-level schema file. This aligns with the API handler setup shown in the code.

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.}

coderabbitai

Actionable comments posted: 0

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (2)

versioned_docs/version-3.x/recipe/nestjs.md (2)
47-60: Fix incorrect import path on line 49.
The import path './db.service' is inconsistent with the file structure. Since DbService is located at src/db/db.service.ts, the correct import from src/app.controller.ts should be './db/db.service'. This matches the import path used in the AppModule (line 36) and the CRUD example (line 134).
🔎 Proposed fix
 import { Controller, Get } from '@nestjs/common';
- import { DbService } from './db.service';+ import { DbService } from './db/db.service';
109-124: Fix incorrect import path on line 111.
The import path './db.service' is inconsistent with the file structure. Since DbService is located at src/db/db.service.ts, the correct import from src/app.controller.ts should be './db/db.service'. This matches the import path used in the access-controlled AppModule (line 71) and the CRUD example (line 134).
🔎 Proposed fix
 import { Controller, Get, Inject } from '@nestjs/common';
- import { DbService } from './db.service';+ import { DbService } from './db/db.service';

📜 Review details

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 89846e0 and e99836f.

📒 Files selected for processing (1)

versioned_docs/version-3.x/recipe/nestjs.md (4 hunks)

🔇 Additional comments (3)

versioned_docs/version-3.x/recipe/nestjs.md (3)
13-43: SQLite migration and module setup look good.
The DbService correctly switches to SqliteDialect with better-sqlite3, and the AppModule properly registers it as a provider with correct import paths.
68-105: Access-controlled module setup is correct.
The AppModule properly registers both the standard and access-controlled DbService instances, with correct imports and the explicit export class AppModule {} declaration.
130-168: CRUD API handler example is well-structured.
The import paths are correct ('./db/db.service' and '../zenstack/schema'), and the example properly demonstrates injecting the access-controlled DbService and initializing the RestApiHandler with the schema.

ymc9

LGTM. Thanks @lsmith77 !

lsmith77 · 2025-12-19T14:57:56Z

If you prefer I can keep PostgreSQL but I think SQLite makes it easier for people to get this setup quickly. How to deal with PostgreSQL is then handled elsewhere in the docs.

Summary by CodeRabbit

Documentation
- Updated NestJS recipe to use SQLite-backed examples instead of PostgreSQL.
- Reorganized example file structure and adjusted import paths for clarity.
- Clarified module exports and updated code samples for consistent access-control and API-handler examples.

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

vercel · 2025-12-19T14:58:00Z

@lsmith77 is attempting to deploy a commit to the ZenStack Team on Vercel.

A member of the Team first needs to authorize it.

coderabbitai · 2025-12-19T15:00:11Z

Walkthrough

Reworks NestJS recipe docs: replaces PostgreSQL dialect with SqliteDialect and a SQLite DB in samples; updates file path annotations and import paths to use src/ layout; adds explicit export class AppModule and includes schema import in the API handler example.

Changes

Cohort / File(s)	Change Summary
NestJS Recipe Documentation `versioned_docs/version-3.x/recipe/nestjs.md`	Replaced Postgres setup with `SqliteDialect` and a SQLite-backed database in `DbService` sample; updated file path annotations from root-level names to `src/db/db.service.ts`, `src/app.module.ts`, `src/app.controller.ts`; adjusted import paths to the new locations; added `export class AppModule` and added schema import to API handler example.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Consistent, repetitive documentation changes across a single file.
Focused on examples/path adjustments and a dialect swap; no executable code outside docs to validate.

Possibly related PRs

docs: nestjs guide for v3 #526: Modifies the same NestJS recipe documentation with overlapping example and import/path updates.

Pre-merge checks and finishing touches

✅ 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 describes the main changes: fixing minor documentation issues and switching the example database from PostgreSQL to SQLite in the NestJS v3 docs.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing touches

🧪 Generate unit tests (beta)

Create PR with unit tests
Post copyable unit tests in a comment

📜 Recent review details

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between e99836f and a6b149c.

📒 Files selected for processing (1)

versioned_docs/version-3.x/recipe/nestjs.md (4 hunks)

🔇 Additional comments (6)

versioned_docs/version-3.x/recipe/nestjs.md (6)
13-29: Import paths and SQLite setup look correct.
All imports properly reference the new src/ directory structure. The path ../../zenstack/schema correctly resolves from src/db/db.service.ts to the root-level zenstack/schema. SQLite configuration with better-sqlite3 is straightforward and makes the quick-start simpler than PostgreSQL.
33-43: AppModule structure and imports are correct.
The explicit export class AppModule {} is good practice. Import path ./db/db.service correctly resolves from src/app.module.ts to src/db/db.service.ts.
47-60: AppController import paths verified.
Import path ./db/db.service correctly resolves from src/app.controller.ts.
68-105: Access-controlled ORM AppModule is consistent.
The explicit export class AppModule {} and import paths match the plain ORM pattern, maintaining consistency across examples.
109-124: Access-controlled ORM AppController is consistent.
Import paths align with the plain ORM example.
130-168: CRUD service example properly imports schema.
The schema import on Line 135 (../zenstack/schema) correctly resolves from src/app.controller.ts to the root-level schema file. This aligns with the API handler setup shown in the code.

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.}

coderabbitai

Actionable comments posted: 0

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (2)

versioned_docs/version-3.x/recipe/nestjs.md (2)
47-60: Fix incorrect import path on line 49.
The import path './db.service' is inconsistent with the file structure. Since DbService is located at src/db/db.service.ts, the correct import from src/app.controller.ts should be './db/db.service'. This matches the import path used in the AppModule (line 36) and the CRUD example (line 134).
🔎 Proposed fix
 import { Controller, Get } from '@nestjs/common';
- import { DbService } from './db.service';+ import { DbService } from './db/db.service';
109-124: Fix incorrect import path on line 111.
The import path './db.service' is inconsistent with the file structure. Since DbService is located at src/db/db.service.ts, the correct import from src/app.controller.ts should be './db/db.service'. This matches the import path used in the access-controlled AppModule (line 71) and the CRUD example (line 134).
🔎 Proposed fix
 import { Controller, Get, Inject } from '@nestjs/common';
- import { DbService } from './db.service';+ import { DbService } from './db/db.service';

📜 Review details

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 89846e0 and e99836f.

📒 Files selected for processing (1)

versioned_docs/version-3.x/recipe/nestjs.md (4 hunks)

🔇 Additional comments (3)

versioned_docs/version-3.x/recipe/nestjs.md (3)
13-43: SQLite migration and module setup look good.
The DbService correctly switches to SqliteDialect with better-sqlite3, and the AppModule properly registers it as a provider with correct import paths.
68-105: Access-controlled module setup is correct.
The AppModule properly registers both the standard and access-controlled DbService instances, with correct imports and the explicit export class AppModule {} declaration.
130-168: CRUD API handler example is well-structured.
The import paths are correct ('./db/db.service' and '../zenstack/schema'), and the example properly demonstrates injecting the access-controlled DbService and initializing the RestApiHandler with the schema.

ymc9

LGTM. Thanks @lsmith77 !

lsmith77 · 2025-12-19T14:57:56Z

If you prefer I can keep PostgreSQL but I think SQLite makes it easier for people to get this setup quickly. How to deal with PostgreSQL is then handled elsewhere in the docs.

Summary by CodeRabbit

Documentation
- Updated NestJS recipe to use SQLite-backed examples instead of PostgreSQL.
- Reorganized example file structure and adjusted import paths for clarity.
- Clarified module exports and updated code samples for consistent access-control and API-handler examples.

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

vercel · 2025-12-19T14:58:00Z

@lsmith77 is attempting to deploy a commit to the ZenStack Team on Vercel.

A member of the Team first needs to authorize it.

coderabbitai · 2025-12-19T15:00:11Z

Walkthrough

Reworks NestJS recipe docs: replaces PostgreSQL dialect with SqliteDialect and a SQLite DB in samples; updates file path annotations and import paths to use src/ layout; adds explicit export class AppModule and includes schema import in the API handler example.

Changes

Cohort / File(s)	Change Summary
NestJS Recipe Documentation `versioned_docs/version-3.x/recipe/nestjs.md`	Replaced Postgres setup with `SqliteDialect` and a SQLite-backed database in `DbService` sample; updated file path annotations from root-level names to `src/db/db.service.ts`, `src/app.module.ts`, `src/app.controller.ts`; adjusted import paths to the new locations; added `export class AppModule` and added schema import to API handler example.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Consistent, repetitive documentation changes across a single file.
Focused on examples/path adjustments and a dialect swap; no executable code outside docs to validate.

Possibly related PRs

docs: nestjs guide for v3 #526: Modifies the same NestJS recipe documentation with overlapping example and import/path updates.

Pre-merge checks and finishing touches

✅ 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 describes the main changes: fixing minor documentation issues and switching the example database from PostgreSQL to SQLite in the NestJS v3 docs.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing touches

🧪 Generate unit tests (beta)

Create PR with unit tests
Post copyable unit tests in a comment

📜 Recent review details

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between e99836f and a6b149c.

📒 Files selected for processing (1)

versioned_docs/version-3.x/recipe/nestjs.md (4 hunks)

🔇 Additional comments (6)

versioned_docs/version-3.x/recipe/nestjs.md (6)
13-29: Import paths and SQLite setup look correct.
All imports properly reference the new src/ directory structure. The path ../../zenstack/schema correctly resolves from src/db/db.service.ts to the root-level zenstack/schema. SQLite configuration with better-sqlite3 is straightforward and makes the quick-start simpler than PostgreSQL.
33-43: AppModule structure and imports are correct.
The explicit export class AppModule {} is good practice. Import path ./db/db.service correctly resolves from src/app.module.ts to src/db/db.service.ts.
47-60: AppController import paths verified.
Import path ./db/db.service correctly resolves from src/app.controller.ts.
68-105: Access-controlled ORM AppModule is consistent.
The explicit export class AppModule {} and import paths match the plain ORM pattern, maintaining consistency across examples.
109-124: Access-controlled ORM AppController is consistent.
Import paths align with the plain ORM example.
130-168: CRUD service example properly imports schema.
The schema import on Line 135 (../zenstack/schema) correctly resolves from src/app.controller.ts to the root-level schema file. This aligns with the API handler setup shown in the code.

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.}

coderabbitai

Actionable comments posted: 0

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (2)

versioned_docs/version-3.x/recipe/nestjs.md (2)
47-60: Fix incorrect import path on line 49.
The import path './db.service' is inconsistent with the file structure. Since DbService is located at src/db/db.service.ts, the correct import from src/app.controller.ts should be './db/db.service'. This matches the import path used in the AppModule (line 36) and the CRUD example (line 134).
🔎 Proposed fix
 import { Controller, Get } from '@nestjs/common';
- import { DbService } from './db.service';+ import { DbService } from './db/db.service';
109-124: Fix incorrect import path on line 111.
The import path './db.service' is inconsistent with the file structure. Since DbService is located at src/db/db.service.ts, the correct import from src/app.controller.ts should be './db/db.service'. This matches the import path used in the access-controlled AppModule (line 71) and the CRUD example (line 134).
🔎 Proposed fix
 import { Controller, Get, Inject } from '@nestjs/common';
- import { DbService } from './db.service';+ import { DbService } from './db/db.service';

📜 Review details

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 89846e0 and e99836f.

📒 Files selected for processing (1)

versioned_docs/version-3.x/recipe/nestjs.md (4 hunks)

🔇 Additional comments (3)

versioned_docs/version-3.x/recipe/nestjs.md (3)
13-43: SQLite migration and module setup look good.
The DbService correctly switches to SqliteDialect with better-sqlite3, and the AppModule properly registers it as a provider with correct import paths.
68-105: Access-controlled module setup is correct.
The AppModule properly registers both the standard and access-controlled DbService instances, with correct imports and the explicit export class AppModule {} declaration.
130-168: CRUD API handler example is well-structured.
The import paths are correct ('./db/db.service' and '../zenstack/schema'), and the example properly demonstrates injecting the access-controlled DbService and initializing the RestApiHandler with the schema.

ymc9

LGTM. Thanks @lsmith77 !

lsmith77 · 2025-12-19T14:57:56Z

If you prefer I can keep PostgreSQL but I think SQLite makes it easier for people to get this setup quickly. How to deal with PostgreSQL is then handled elsewhere in the docs.

Summary by CodeRabbit

Documentation
- Updated NestJS recipe to use SQLite-backed examples instead of PostgreSQL.
- Reorganized example file structure and adjusted import paths for clarity.
- Clarified module exports and updated code samples for consistent access-control and API-handler examples.

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

vercel · 2025-12-19T14:58:00Z

@lsmith77 is attempting to deploy a commit to the ZenStack Team on Vercel.

A member of the Team first needs to authorize it.

coderabbitai · 2025-12-19T15:00:11Z

Walkthrough

Reworks NestJS recipe docs: replaces PostgreSQL dialect with SqliteDialect and a SQLite DB in samples; updates file path annotations and import paths to use src/ layout; adds explicit export class AppModule and includes schema import in the API handler example.

Changes

Cohort / File(s)	Change Summary
NestJS Recipe Documentation `versioned_docs/version-3.x/recipe/nestjs.md`	Replaced Postgres setup with `SqliteDialect` and a SQLite-backed database in `DbService` sample; updated file path annotations from root-level names to `src/db/db.service.ts`, `src/app.module.ts`, `src/app.controller.ts`; adjusted import paths to the new locations; added `export class AppModule` and added schema import to API handler example.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Consistent, repetitive documentation changes across a single file.
Focused on examples/path adjustments and a dialect swap; no executable code outside docs to validate.

Possibly related PRs

docs: nestjs guide for v3 #526: Modifies the same NestJS recipe documentation with overlapping example and import/path updates.

Pre-merge checks and finishing touches

✅ 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 describes the main changes: fixing minor documentation issues and switching the example database from PostgreSQL to SQLite in the NestJS v3 docs.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing touches

🧪 Generate unit tests (beta)

Create PR with unit tests
Post copyable unit tests in a comment

📜 Recent review details

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between e99836f and a6b149c.

📒 Files selected for processing (1)

versioned_docs/version-3.x/recipe/nestjs.md (4 hunks)

🔇 Additional comments (6)

versioned_docs/version-3.x/recipe/nestjs.md (6)
13-29: Import paths and SQLite setup look correct.
All imports properly reference the new src/ directory structure. The path ../../zenstack/schema correctly resolves from src/db/db.service.ts to the root-level zenstack/schema. SQLite configuration with better-sqlite3 is straightforward and makes the quick-start simpler than PostgreSQL.
33-43: AppModule structure and imports are correct.
The explicit export class AppModule {} is good practice. Import path ./db/db.service correctly resolves from src/app.module.ts to src/db/db.service.ts.
47-60: AppController import paths verified.
Import path ./db/db.service correctly resolves from src/app.controller.ts.
68-105: Access-controlled ORM AppModule is consistent.
The explicit export class AppModule {} and import paths match the plain ORM pattern, maintaining consistency across examples.
109-124: Access-controlled ORM AppController is consistent.
Import paths align with the plain ORM example.
130-168: CRUD service example properly imports schema.
The schema import on Line 135 (../zenstack/schema) correctly resolves from src/app.controller.ts to the root-level schema file. This aligns with the API handler setup shown in the code.

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.}

coderabbitai

Actionable comments posted: 0

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (2)

versioned_docs/version-3.x/recipe/nestjs.md (2)
47-60: Fix incorrect import path on line 49.
The import path './db.service' is inconsistent with the file structure. Since DbService is located at src/db/db.service.ts, the correct import from src/app.controller.ts should be './db/db.service'. This matches the import path used in the AppModule (line 36) and the CRUD example (line 134).
🔎 Proposed fix
 import { Controller, Get } from '@nestjs/common';
- import { DbService } from './db.service';+ import { DbService } from './db/db.service';
109-124: Fix incorrect import path on line 111.
The import path './db.service' is inconsistent with the file structure. Since DbService is located at src/db/db.service.ts, the correct import from src/app.controller.ts should be './db/db.service'. This matches the import path used in the access-controlled AppModule (line 71) and the CRUD example (line 134).
🔎 Proposed fix
 import { Controller, Get, Inject } from '@nestjs/common';
- import { DbService } from './db.service';+ import { DbService } from './db/db.service';

📜 Review details

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 89846e0 and e99836f.

📒 Files selected for processing (1)

versioned_docs/version-3.x/recipe/nestjs.md (4 hunks)

🔇 Additional comments (3)

versioned_docs/version-3.x/recipe/nestjs.md (3)
13-43: SQLite migration and module setup look good.
The DbService correctly switches to SqliteDialect with better-sqlite3, and the AppModule properly registers it as a provider with correct import paths.
68-105: Access-controlled module setup is correct.
The AppModule properly registers both the standard and access-controlled DbService instances, with correct imports and the explicit export class AppModule {} declaration.
130-168: CRUD API handler example is well-structured.
The import paths are correct ('./db/db.service' and '../zenstack/schema'), and the example properly demonstrates injecting the access-controlled DbService and initializing the RestApiHandler with the schema.

ymc9

LGTM. Thanks @lsmith77 !

coderabbitaiBot reviewed Dec 19, 2025
View reviewed changes

lsmith77 commented Dec 19, 2025
View reviewed changes

Comment threadversioned_docs/version-3.x/recipe/nestjs.md

fix minor issues in the nest.js v3 docs, switch to SQLite
a6b149c

lsmith77 force-pushed the fix-v3-nestjs branch from e99836f to a6b149cCompare December 20, 2025 10:00

ymc9 approved these changes Dec 22, 2025
View reviewed changes

ymc9 merged commit 86bbb8c into zenstackhq:mainDec 22, 2025
2 of 3 checks passed

coderabbitaiBot reviewed Dec 19, 2025
View reviewed changes

lsmith77 commented Dec 19, 2025
View reviewed changes

Comment threadversioned_docs/version-3.x/recipe/nestjs.md

fix minor issues in the nest.js v3 docs, switch to SQLite
a6b149c

lsmith77 force-pushed the fix-v3-nestjs branch from e99836f to a6b149cCompare December 20, 2025 10:00

ymc9 approved these changes Dec 22, 2025
View reviewed changes

ymc9 merged commit 86bbb8c into zenstackhq:mainDec 22, 2025
2 of 3 checks passed

coderabbitaiBot reviewed Dec 19, 2025
View reviewed changes

lsmith77 commented Dec 19, 2025
View reviewed changes

Comment threadversioned_docs/version-3.x/recipe/nestjs.md

fix minor issues in the nest.js v3 docs, switch to SQLite
a6b149c

lsmith77 force-pushed the fix-v3-nestjs branch from e99836f to a6b149cCompare December 20, 2025 10:00

ymc9 approved these changes Dec 22, 2025
View reviewed changes

ymc9 merged commit 86bbb8c into zenstackhq:mainDec 22, 2025
2 of 3 checks passed

coderabbitaiBot reviewed Dec 19, 2025
View reviewed changes

lsmith77 commented Dec 19, 2025
View reviewed changes

Comment threadversioned_docs/version-3.x/recipe/nestjs.md

fix minor issues in the nest.js v3 docs, switch to SQLite
a6b149c

lsmith77 force-pushed the fix-v3-nestjs branch from e99836f to a6b149cCompare December 20, 2025 10:00

ymc9 approved these changes Dec 22, 2025
View reviewed changes

ymc9 merged commit 86bbb8c into zenstackhq:mainDec 22, 2025
2 of 3 checks passed

coderabbitaiBot reviewed Dec 19, 2025
View reviewed changes

lsmith77 commented Dec 19, 2025
View reviewed changes

Comment threadversioned_docs/version-3.x/recipe/nestjs.md

fix minor issues in the nest.js v3 docs, switch to SQLite
a6b149c

lsmith77 force-pushed the fix-v3-nestjs branch from e99836f to a6b149cCompare December 20, 2025 10:00

ymc9 approved these changes Dec 22, 2025
View reviewed changes

ymc9 merged commit 86bbb8c into zenstackhq:mainDec 22, 2025
2 of 3 checks passed

coderabbitaiBot reviewed Dec 19, 2025
View reviewed changes

lsmith77 commented Dec 19, 2025
View reviewed changes

Comment threadversioned_docs/version-3.x/recipe/nestjs.md

fix minor issues in the nest.js v3 docs, switch to SQLite
a6b149c

lsmith77 force-pushed the fix-v3-nestjs branch from e99836f to a6b149cCompare December 20, 2025 10:00

ymc9 approved these changes Dec 22, 2025
View reviewed changes

ymc9 merged commit 86bbb8c into zenstackhq:mainDec 22, 2025
2 of 3 checks passed

coderabbitaiBot reviewed Dec 19, 2025
View reviewed changes

lsmith77 commented Dec 19, 2025
View reviewed changes

Comment threadversioned_docs/version-3.x/recipe/nestjs.md

fix minor issues in the nest.js v3 docs, switch to SQLite
a6b149c

lsmith77 force-pushed the fix-v3-nestjs branch from e99836f to a6b149cCompare December 20, 2025 10:00

ymc9 approved these changes Dec 22, 2025
View reviewed changes

ymc9 merged commit 86bbb8c into zenstackhq:mainDec 22, 2025
2 of 3 checks passed

coderabbitaiBot reviewed Dec 19, 2025
View reviewed changes

lsmith77 commented Dec 19, 2025
View reviewed changes

Comment threadversioned_docs/version-3.x/recipe/nestjs.md

fix minor issues in the nest.js v3 docs, switch to SQLite
a6b149c

lsmith77 force-pushed the fix-v3-nestjs branch from e99836f to a6b149cCompare December 20, 2025 10:00

ymc9 approved these changes Dec 22, 2025
View reviewed changes

ymc9 merged commit 86bbb8c into zenstackhq:mainDec 22, 2025
2 of 3 checks passed

Uh oh!

Uh oh!

Conversation

lsmith77 commented Dec 19, 2025• edited by coderabbitai BotLoading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary by CodeRabbit

Uh oh!

vercelBot commented Dec 19, 2025

Uh oh!

coderabbitaiBot commented Dec 19, 2025• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Changes

Estimated code review effort

Possibly related PRs

Pre-merge checks and finishing touches

Uh oh!

coderabbitaiBot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

ymc9 left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Uh oh!

Uh oh!

Conversation

lsmith77 commented Dec 19, 2025• edited by coderabbitai BotLoading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary by CodeRabbit

Uh oh!

vercelBot commented Dec 19, 2025

Uh oh!

coderabbitaiBot commented Dec 19, 2025• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Changes

Estimated code review effort

Possibly related PRs

Pre-merge checks and finishing touches

Uh oh!

coderabbitaiBot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

ymc9 left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Uh oh!

Uh oh!

Conversation

lsmith77 commented Dec 19, 2025• edited by coderabbitai BotLoading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary by CodeRabbit

Uh oh!

vercelBot commented Dec 19, 2025

Uh oh!

coderabbitaiBot commented Dec 19, 2025• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Changes

Estimated code review effort

lsmith77 commented Dec 19, 2025•
edited by coderabbitai Bot
Loading

coderabbitaiBot commented Dec 19, 2025•
edited
Loading

lsmith77 commented Dec 19, 2025•
edited by coderabbitai Bot
Loading

coderabbitaiBot commented Dec 19, 2025•
edited
Loading

lsmith77 commented Dec 19, 2025•
edited by coderabbitai Bot
Loading

coderabbitaiBot commented Dec 19, 2025•
edited
Loading

lsmith77 commented Dec 19, 2025•
edited by coderabbitai Bot
Loading

coderabbitaiBot commented Dec 19, 2025•
edited
Loading