Skip to content

Expand documentation with comprehensive usage guide and contributing guidelines - #159

Merged
sergioteula merged 1 commit into
masterfrom
claude/project-documentation-review-dhmfgl
Sep 4, 2026
Merged

Expand documentation with comprehensive usage guide and contributing guidelines#159
sergioteula merged 1 commit into
masterfrom
claude/project-documentation-review-dhmfgl

Conversation

@sergioteula

Copy link
Copy Markdown
Owner

Summary

This PR significantly expands the project documentation to provide comprehensive guidance for both users and contributors. The changes include a detailed usage guide covering all client features, configuration options, and error handling, along with a new contributing guide for developers.

Key Changes

Documentation Improvements

  • Expanded usage guide (docs/pages/usage-guide.md): Transformed from a basic overview into a comprehensive reference covering:

    • Installation requirements (Python 3.9+, async extra)
    • Credentials and API versions with detailed tables
    • Supported countries and marketplace mappings
    • All client methods with examples and explanations
    • Configuration options (throttling, timeout, retries, custom endpoints)
    • Error handling with exception types and examples
    • Async support with usage patterns
    • Context manager usage for connection pooling
  • Updated README.md: Aligned with the expanded usage guide, including:

    • Table of contents for better navigation
    • Credentials section with parameter descriptions
    • Reorganized configuration section
    • Improved formatting and clarity
  • New CONTRIBUTING.md: Added comprehensive contributor guide including:

    • Project setup instructions using uv
    • Make commands reference
    • Project layout and structure
    • Code style conventions
    • Testing requirements (98% coverage minimum)
    • Documentation expectations
    • Pull request submission checklist

Documentation Structure Updates

  • Enhanced Sphinx configuration (docs/conf.py):

    • Added Markdown heading anchors for cross-page linking
    • Configured member ordering by source for better readability
  • Improved RST documentation files:

    • docs/index.rst: Added introductory example and support section
    • docs/amazon_creatorsapi.rst: Clarified main API module purpose
    • docs/amazon_creatorsapi.aio.rst: Added usage examples for async client
    • docs/amazon_creatorsapi.core.rst: Documented core utilities
    • docs/amazon_creatorsapi.errors.rst: Explained error hierarchy
    • docs/amazon_creatorsapi.models.rst: Clarified model re-exports

Migration Guide Updates

  • Clarified API version requirements and credential changes
  • Improved method signature comparison tables
  • Better explanation of parameter changes between PAAPI and Creators API

Minor Fixes

  • Fixed typo in bug report template ("currenctly" → "currently")
  • Updated CHANGELOG.md with documentation additions

Notable Details

  • Documentation maintains consistency between README.md and the detailed usage guide
  • All configuration options are now documented with examples and default values
  • Error types are comprehensively documented with when each is raised
  • Contributing guide enforces code quality standards (98% test coverage, type hints, docstrings)
  • Sphinx configuration optimized for better documentation readability

https://claude.ai/code/session_01HkMXgQgzMKJBkEE7Cw3jLi

The migration guide told users that `include_unavailable`, `availability`
and `delivery_flags` were gone, when the Creators API accepts all three,
imported `get_asin` from a private path and listed only part of the
exceptions of the module.

The accepted values of `version`, the countries and the marketplace each
one maps to, and the `marketplace` argument had no documentation at all,
so the only way to find them was to read the code.

The usage guide and the README had also drifted apart, each one holding
sections the other lacked, and both promised the same methods in the two
clients without telling that `close` only exists in the synchronous one.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01HkMXgQgzMKJBkEE7Cw3jLi
@sergioteula
sergioteula merged commit a6a7693 into master Sep 4, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants