Skip to content

Conversation

@el-feo
Copy link
Owner

@el-feo el-feo commented Nov 10, 2025

Summary

  • Corrected API response documentation to show actual synchronous completion response with images array, pages_converted count, and metadata
  • Updated project description from "asynchronous" to "synchronous" PDF processing throughout both README and CLAUDE.md
  • Clarified that webhook notifications are optional and operate in fire-and-forget mode
  • Added missing environment variables to README documentation (CONVERSION_DPI, PNG_COMPRESSION, MAX_PAGES, VIPS_WARNING)
  • Reorganized dependencies section with proper Development subsection for rubycritic
  • Added simplecov to testing dependencies
  • Removed AWS_REGION requirement from basic test command (already handled by spec_helper default)

Test plan

  • Review README.md for accuracy against actual codebase implementation
  • Review CLAUDE.md for consistency with README.md
  • Verify environment variables match template.yaml configuration
  • Confirm dependencies match Gemfile entries
  • Validate API response format matches ResponseBuilder implementation

🤖 Generated with Claude Code

- Corrected API response documentation to show actual synchronous completion response with images array, page count, and metadata
- Changed description from "asynchronous" to "synchronous" PDF processing throughout
- Clarified webhook notifications are optional and fire-and-forget
- Added missing environment variables to README (CONVERSION_DPI, PNG_COMPRESSION, MAX_PAGES, VIPS_WARNING)
- Reorganized dependencies section to include Development subsection with rubycritic
- Added simplecov to testing dependencies
- Removed AWS_REGION requirement from basic test command (handled by spec_helper)

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>
@el-feo el-feo merged commit 908a17e into main Nov 10, 2025
1 check 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