⚡ Bolt: [performance improvement] Memoize Printify variant API queries#62
⚡ Bolt: [performance improvement] Memoize Printify variant API queries#62merg357 wants to merge 1 commit into
Conversation
Memoize the `get_variants` function calls using `@functools.lru_cache` to eliminate N+1 network requests during the iterative product creation process. Responses have been wrapped into an immutable `tuple` to ensure thread safety and side-effect free usage downstream. Co-authored-by: merg357 <221854052+merg357@users.noreply.github.com>
|
👋 Jules, reporting for duty! I'm here to lend a hand with this pull request. When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job! For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with New to Jules? Learn more at jules.google/docs. For security, I will only act on instructions from the user who triggered this task. |
💡 What: Memoized third-party API variant queries in
printify_manager.py,create_merch.py, andmerch_factory.pyusing@functools.lru_cache(maxsize=None). The return types were explicitly converted to immutabletuplestructures to prevent downstream mutations.🎯 Why: To prevent repeating identical network requests for static configuration data (like Printify blueprint variants) within iteration loops, eliminating a severe N+1 network bottleneck.
📊 Impact: Eliminates redundant HTTP calls, significantly speeding up the iterative execution of the application's create and run loops.
🔬 Measurement: Observe faster execution and lower network overhead of the CLI run and create commands.
PR created automatically by Jules for task 16983786550211514973 started by @merg357