Skip to content

feat(skills-next): add nestjs SDK references#244

Open
evanpurkhiser wants to merge 1 commit into
mainfrom
evanpurkhiser/feat-skills-next-add-nestjs-sdk-references
Open

feat(skills-next): add nestjs SDK references#244
evanpurkhiser wants to merge 1 commit into
mainfrom
evanpurkhiser/feat-skills-next-add-nestjs-sdk-references

Conversation

@evanpurkhiser

Copy link
Copy Markdown
Member

Direct LLM port of the existing nestjs SDK skill into the skills-next per-SDK reference layout under skills-next/references/sdks/nestjs/. This content has NOT been reviewed at all — it is a machine-generated port of the existing SDK skill, and every file should be treated as unverified.

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit bde6e02. Configure here.

Sentry.captureException(err);
}
});
}

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Missing await on isolation scope

Medium Severity

The cron example calls Sentry.withIsolationScope with an async callback but never awaits the returned promise, so the scheduled handler can finish before isolated work and Sentry context teardown complete. That undermines the documented goal of keeping job breadcrumbs and tags out of HTTP request scopes.

Additional Locations (1)
Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit bde6e02. Configure here.

Comment thread skills-next/references/sdks/nestjs/error-monitoring.md
| MySQL2 | ✅ | `mysql2Integration` |
| MongoDB | ✅ | `mongoIntegration` |
| Mongoose | ✅ | `mongooseIntegration` |
| Prisma | ⚠️ Manual | `prismaIntegration` — add explicitly: `integrations: [Sentry.prismaIntegration()]` |

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Bug: The documentation is contradictory, stating prismaIntegration is both manual and auto-enabled. This can lead to developers incorrectly assuming it's automatic and missing tracing data.
Severity: MEDIUM

Suggested Fix

To resolve the contradiction, remove prismaIntegration from the list of auto-enabled database integrations at line 669. The documentation should consistently state that it requires manual configuration.

Prompt for AI Agent
Review the code at the location below. A potential bug has been identified by an AI
agent. Verify if this is a real issue. If it is, propose a fix; if not, explain why it's
not valid.

Location: skills-next/references/sdks/nestjs/tracing.md#L553

Potential issue: The documentation for the NestJS SDK contains conflicting information
regarding the Sentry Prisma integration. One section states that `prismaIntegration`
requires manual setup (`⚠️ Manual`), while another section incorrectly lists it under
"Databases (all auto-enabled)". A developer following the latter section would assume
Prisma tracing is enabled by default, leading to a silent failure where no
Prisma-related spans are captured. This results in missing observability data in
production without any warning or error.

Also affects:

  • skills-next/references/sdks/nestjs/tracing.md:669~669

Did we get this right? 👍 / 👎 to inform future reviews.

@evanpurkhiser evanpurkhiser force-pushed the evanpurkhiser/feat-skills-next-add-nestjs-sdk-references branch from bde6e02 to 5010262 Compare July 2, 2026 21:14
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