docs: lead with react-dropzone versus framing, surface skills + registry install - #25
Conversation
|
|
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
📝 WalkthroughWalkthroughThe documentation positions mediadrop as a headless, react-dropzone-compatible, hooks-first uploader and adds Agent Skill, Context7, and shadcn registry guidance. ChangesDocumentation updates
Estimated code review effort: 1 (Trivial) | ~5 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
Deploying react-mediadrop with
|
| Latest commit: |
72595c6
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://5bd4ad6d.react-mediadrop.pages.dev |
| Branch Preview URL: | https://docs-react-dropzone-alternat.react-mediadrop.pages.dev |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@apps/docs/docs/index.mdx`:
- Around line 6-9: Update the introductory description of react-mediadrop to
explicitly state that the upload engine is opt-in via the transport option,
keeping the existing claims and useMediaDrop contract accurate.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro Plus
Run ID: c5332149-f251-4dce-a3a6-525c58e5904a
📒 Files selected for processing (2)
README.mdapps/docs/docs/index.mdx
57b59e9 to
73dd00e
Compare
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@skills/mediadrop/references/registry-blocks.md`:
- Around line 24-27: Update the GitHub-registry address explanation near the
`owner/repo/item` form to state that it requires an existing shadcn/ui project
initialized with `shadcn init`, while clarifying that no additional registry
setup is needed. Keep the pending `@mediadrop/<item-name>` namespace warning
unchanged.
- Around line 18-21: Pin the registry item references in the install examples by
appending the appropriate tag or commit after # in
skills/mediadrop/references/registry-blocks.md lines 18-21 and
skills/mediadrop/SKILL.md lines 105-109; apply the same update to the
corresponding examples in README.md lines 106-110, preserving the existing item
names.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro Plus
Run ID: 973df114-8dbd-401c-89bf-fe520240b24d
📒 Files selected for processing (3)
apps/docs/docs/getting-started/agent-skill.mdxskills/mediadrop/SKILL.mdskills/mediadrop/references/registry-blocks.md
…try install README and docs Introduction now state the react-dropzone comparison up front instead of burying it in the Why section. Also adds the AI-agent skill install command next to npm install, and a shadcn GitHub-registry install path for the four registry blocks.
…olling markup Adds references/registry-blocks.md documenting the four blocks and the GitHub-registry install command, wires it into SKILL.md's quickstarts + hard rules, and links it from the agent-skill docs page.
2f80a8c to
72595c6
Compare
Summary
npx shadcn@latest add autorender/react-mediadrop/dropzone) for the four registry blocks, documented in README, docs, and the agent skill (skills/mediadrop/references/registry-blocks.md). Holds off on the@mediadrop/dropzonenamespace form until shadcn-ui/ui#11239 merges.npx skills add autorender/react-mediadropand Context7 indexing next tonpm installon the README's first screen instead of only on its own docs page.shadcn initprerequisite next to the install-form explanation.Test plan