chore(database-client): add npm metadata and LICENSE - #187
Conversation
|
@codex review |
🦋 Changeset detectedLatest commit: abdc8c5 The changes in this PR will be included in the next version bump. This PR includes changesets to release 9 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
Greptile SummaryAdds publication metadata and an MIT license for
Confidence Score: 5/5The PR appears safe to merge with no actionable defects identified. The added metadata matches the package’s runtime requirements and publishing layout, while the side-effects declaration is consistent with the current implementation.
|
| Filename | Overview |
|---|---|
| packages/database-client/package.json | Adds consistent npm metadata, an accurate minimum Node version, a valid side-effects declaration, and LICENSE publication. |
| packages/database-client/LICENSE | Adds the standard MIT license referenced by the package metadata. |
| .changeset/database-client-npm-metadata.md | Correctly records the package metadata update as a patch release. |
Reviews (1): Last reviewed commit: "chore(database-client): add npm metadata..." | Re-trigger Greptile
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: abdc8c5ff5
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| { | ||
| "name": "@bunny.net/database-client", | ||
| "version": "0.0.1", | ||
| "description": "Dependency-free SQL client for Bunny Database. Runs anywhere fetch does: Bunny Edge Scripting, Bun, Node, and Deno.", |
There was a problem hiding this comment.
Remove the browser-compatible implication from the description
When this description appears in npm search or package-manager output without the README warning, “Runs anywhere fetch does” includes browsers and advertises the client as browser-compatible. Following that claim can expose the bearer token—which grants whole-database access—and raw SQL capabilities to untrusted clients; restrict the description to named server runtimes and explicitly retain the server-side-only constraint.
AGENTS.md reference: AGENTS.md:L75-L75
Useful? React with 👍 / 👎.
No description provided.