skills for db and storage - #204
Conversation
|
@codex review |
🦋 Changeset detectedLatest commit: a19818d The changes in this PR will be included in the next version bump. This PR includes changesets to release 6 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 |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
Greptile SummaryThe PR promotes the storage namespace into the visible CLI command list and ships new storage and database-client skill references.
Confidence Score: 3/5The PR is not yet safe to merge because the installed skill still creates unusable S3 setups and the documented Node range includes runtimes that fail on combined cancellation signals. The current skill workflow creates a storage zone without the immutable S3 enablement flag before requesting S3 credentials, while the database-client documentation and package metadata continue to support Node versions where the unguarded combined-signal path cannot execute. Files Needing Attention: skills/bunny-cli/SKILL.md; skills/bunny-cli/references/database-client.md; packages/database-client/src/protocol.ts; packages/database-client/package.json
|
| Filename | Overview |
|---|---|
| skills/bunny-cli/SKILL.md | Adds storage and database-client routing, but the quick storage workflow still requests S3 credentials after creating a zone without S3 enabled. |
| skills/bunny-cli/references/storage.md | Adds comprehensive storage guidance and now consistently enables S3 in creation examples that request S3 credentials. |
| skills/bunny-cli/references/database-client.md | Adds database-client guidance, but retains a Node support claim that is incompatible with the combined timeout-and-signal implementation. |
| packages/cli/src/commands/skills/content.ts | Embeds and exposes the two new reference files through the installed skill. |
| packages/cli/src/cli.ts | Promotes the storage namespace from the experimental command list to the normal visible command list. |
| packages/cli/src/commands/storage/index.ts | Gives the promoted storage namespace a visible help description. |
Reviews (3): Last reviewed commit: "docs(db): point connect() at db quicksta..." | Re-trigger Greptile
![greptile-apps[bot]](https://avatars.githubusercontent.com/in/867647?s=60&v=4){kind=small}
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 4182e9e0eb
ℹ️ 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".
3 participants
No description provided.