docs: restructure README into concise overview with docs/ reference pages#26
Merged
docs: restructure README into concise overview with docs/ reference pages#26
Conversation
…vents references Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Contributor
Original prompt from Ryan |
Contributor
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
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.
docs: restructure README into concise overview with docs/ reference pages
Summary
Slims down the README from ~340 lines to ~180 lines by moving detailed content into a new
docs/folder. The README now serves as a quick-start guide with a command summary table, linking out to dedicated reference pages for deeper information.README changes:
OpenSeaAPIErrorerror handling exampleNew
docs/pages:docs/cli-reference.md— full command reference with all options and flags (moved from README)docs/examples.md— real-world usage examples for every command (moved from README)docs/sdk.md— full SDK API reference with all methods, config options, error handling, and exportsdocs/pagination.md— pagination patterns for CLI and SDK, including a table of all paginated commands and an iteration exampledocs/events.md— event type reference table with CLI and SDK usageNo code changes — docs only.
Updates since last revision
The initial commit added detailed content directly into the README, making it quite long. Per reviewer feedback, a second commit restructured the approach:
docs/cli-reference.mddocs/examples.mddocs/Review & Testing Checklist for Human
docs/cli-reference.md,docs/examples.md,docs/sdk.md,docs/pagination.md,docs/events.md; and cross-links within docs/ likedocs/cli-reference.mdlinking toevents.mdandpagination.md)docs/cli-reference.mdmatch actual behavior — especially--creator,--include-hidden,--after,--beforeflags which were sourced from command source files but not previously documenteddocs/sdk.mdexamples againstsrc/sdk.tsclass signatures for accuracy (note: the SDK examples use repeated destructuring likeconst { nfts, next }in multiple places which is fine as documentation but would need unique names in real code)Notes
src/commands/*.tsandsrc/sdk.tsdocs/pagination.mdnotes thattokenscommands use--cursorwhile all others use--next— this is an existing inconsistency in the codebase, not introduced by this PR