Conversation
Add instruction force using build in Bun.S3Client
|
📝 WalkthroughWalkthroughA single line is added to CLAUDE.md recommending Bun.S3Client for S3 operations while explicitly advising against aws-sdk usage. No existing content is modified. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Tip Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs). 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. Comment |
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@CLAUDE.md`:
- Line 22: Update the incorrect reference to Bun's S3 API: replace the
usage/mention of Bun.S3Client with the official import form so examples and docs
show importing S3Client directly (e.g., reference the symbol S3Client and the
import pattern import { S3Client } from "bun" instead of Bun.S3Client) so
readers use the correct API.
| - `bun:sqlite` for SQLite. Don't use `better-sqlite3`. | ||
| - `Bun.redis` for Redis. Don't use `ioredis`. | ||
| - `Bun.sql` for Postgres. Don't use `pg` or `postgres.js`. | ||
| - `Bun.S3Client` for S3. Don't use `aws-sdk`. |
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
# Check for S3Client references in the codebase
echo "=== Searching for S3Client in TypeScript definitions ==="
fd -e d.ts -x rg -l "S3Client" {} 2>/dev/null || echo "No .d.ts files with S3Client found"
echo ""
echo "=== Searching for S3Client in source files (ts/js/tsx/jsx) ==="
rg "S3Client" --type ts --type js --type tsx --type jsx 2>/dev/null | head -20 || echo "No S3Client references found in source files"
echo ""
echo "=== Checking for any aws-sdk references ==="
rg "aws-sdk" --type ts --type js --type tsx --type jsx 2>/dev/null | head -20 || echo "No aws-sdk references found"
echo ""
echo "=== Listing all files in repo to understand structure ==="
git ls-files | head -30Repository: ImBIOS/bun-node
Length of output: 1438
🌐 Web query:
Bun S3Client API official documentation
💡 Result:
Official Bun documentation for S3Client (Bun’s built-in S3 API):
https://bun.com/docs/runtime/s3
This page documents import { S3Client } from "bun" and related S3 helpers (e.g., s3, write, client.file(), etc.). [1]
Correct the S3Client API reference to match official Bun syntax.
The API does exist, but the guideline references it incorrectly. Bun's official documentation shows the S3 API is imported as import { S3Client } from "bun", not accessed via Bun.S3Client. Update the line to reflect the correct import pattern so developers follow the proper syntax.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@CLAUDE.md` at line 22, Update the incorrect reference to Bun's S3 API:
replace the usage/mention of Bun.S3Client with the official import form so
examples and docs show importing S3Client directly (e.g., reference the symbol
S3Client and the import pattern import { S3Client } from "bun" instead of
Bun.S3Client) so readers use the correct API.
1 participant
Add instruction force using build in Bun.S3Client
Closes #
📑 Description
✅ Checks
ℹ Additional Information
Summary by CodeRabbit