Add contributing guidelines, issue templates, and security policy; en… - #5
Merged
Conversation
…hance README and changelog
There was a problem hiding this comment.
Pull request overview
This PR improves project contribution/security hygiene (templates and policies) and documents recent feature work, alongside updates to Prisma instrumentation to better tag raw SQL under concurrent operations and keep bootstrap reconnects fail-open.
Changes:
- Add SECURITY policy, Code of Conduct, Contributing guide, PR template, and issue templates.
- Update README with contributor instructions and add a Keep-a-Changelog style CHANGELOG.
- Improve Prisma plugin instrumentation: async-local operation context for concurrency-safe adapter SQL tagging, bounded reconnects, and stronger fail-open wrapping; expand tests accordingly.
Reviewed changes
Copilot reviewed 11 out of 11 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| test/plugins.spec.ts | Adds coverage for Prisma adapter concurrency tagging and fail-open reconnect behavior. |
| src/plugins/prisma/prisma.plugin.ts | Introduces AsyncLocalStorage-based per-operation context and bounded reconnect logic; hardens fail-open wrapping. |
| SECURITY.md | Adds vulnerability reporting and support policy. |
| README.md | Adds a Contributing section with local dev commands and links to policies. |
| CONTRIBUTING.md | Adds contribution workflow, style expectations, and CI-aligned commands. |
| CODE_OF_CONDUCT.md | Adds Contributor Covenant Code of Conduct. |
| CHANGELOG.md | Adds an Unreleased section documenting recent additions/changes. |
| .github/PULL_REQUEST_TEMPLATE.md | Adds a PR checklist aligned with CI and project conventions. |
| .github/ISSUE_TEMPLATE/feature_request.yml | Adds a structured feature request template. |
| .github/ISSUE_TEMPLATE/config.yml | Disables blank issues and adds contact links for questions/security. |
| .github/ISSUE_TEMPLATE/bug_report.yml | Adds a structured bug report template capturing environment details. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
+79
to
+95
| /** Resolve when `promise` settles or after `ms`, whichever comes first (never rejects on timeout). */ | ||
| function withTimeout(promise: Promise<unknown>, ms: number): Promise<unknown> { | ||
| return new Promise((resolve) => { | ||
| const timer = setTimeout(resolve, ms); | ||
| if (typeof timer.unref === 'function') timer.unref(); // don't keep the process alive | ||
| promise.then( | ||
| () => { | ||
| clearTimeout(timer); | ||
| resolve(undefined); | ||
| }, | ||
| () => { | ||
| clearTimeout(timer); | ||
| resolve(undefined); | ||
| }, | ||
| ); | ||
| }); | ||
| } |
Comment on lines
+138
to
+143
| /** | ||
| * Per-operation context. Unlike the correlator's stack, an async-local store | ||
| * follows each operation's own async chain, so concurrent operations | ||
| * (`Promise.all([findMany, count])`) tag their raw SQL correctly instead of | ||
| * both resolving to whichever token happens to be on top of the stack. | ||
| */ |
rakibulislam8226
added a commit
that referenced
this pull request
Jul 21, 2026
Add contributing guidelines, issue templates, and security policy; en…
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
…hance README and changelog