[REBASED → #481] fix(mockup-generator): auditoria exaustiva — 10 bugs corrigidos#478
Closed
adm01-debug wants to merge 3 commits into
Closed
[REBASED → #481] fix(mockup-generator): auditoria exaustiva — 10 bugs corrigidos#478adm01-debug wants to merge 3 commits into
adm01-debug wants to merge 3 commits into
Conversation
… limit 200, T10 thumbnail_url
… userId delete, T9 URL params T1: 7 async handlers wrapped in useCallback → useMemo output is now truly stable. T2: Correct dependency arrays eliminate stale-closure bugs. T3: historyPushTimeout + draftNoticeTimeoutRef cleaned up on unmount (no memory leaks). T5: Batch DB saves parallelised with Promise.allSettled (was sequential for-await). T6: deleteMockupFromDb receives user?.id for owner-scoped DELETE. T9: URL param cleanup preserves non-processed search params.
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Warning Review limit reached
More reviews will be available in 48 minutes and 11 seconds. Learn how PR review limits work. Your organization has run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After more reviews become available, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available. Please see our Fair Usage Limits Policy for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
This pull request has been ignored for the connected project Preview Branches by Supabase. |
Copilot stopped reviewing on behalf of
adm01-debug due to an error
May 26, 2026 20:39
Contributor
There was a problem hiding this comment.
Pull request overview
Note
Copilot was unable to run its full agentic suite in this review.
This PR hardens the mockup generator hook against stale closures and timer leaks, improves batch history persistence performance, and tightens history deletion/URL-param handling.
Changes:
- Stabilizes async handlers with
useCallback, fixes dependency arrays, and adds unmount cleanup for debounced timers. - Parallelizes batch history saves using
Promise.allSettledand scopes DB deletion byuserId. - Updates URL param cleanup to only remove processed keys.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| src/hooks/mockup/useMockupGenerator.ts | Stabilizes handlers, improves batch save behavior, and cleans up timers/URL params. |
| src/hooks/mockup/mockupGenerationService.ts | File content appears replaced with a base64-like single-line payload, breaking the TS module. |
Comments suppressed due to low confidence (2)
src/hooks/mockup/mockupGenerationService.ts:1
- This file appears to have been replaced by a base64-encoded (or otherwise escaped) single-line payload rather than valid TypeScript source. As-is, the module will not compile and all imports from this service will break. Please restore the file as plain TypeScript (normal UTF-8 text), ensuring the repository contains the real source code rather than an encoded artifact.
LyoqCiAqIG1vY2t1cEdlbmVyYXRpb25TZXJ2aWNlIOKAlCBIYW5kbGVzIG1vY2t1cCBnZW5lcmF0aW9uIEFQSSBjYWxscyBhbmQgaGlzdG9yeSBwZXJzaXN0ZW5jZS4KICogRXh0cmFjdGVkIGZyb20gdXNlTW9ja3VwR2VuZXJhdG9yIHRvIHJlZHVjZSBob29rIGNvbXBsZXhpdHkuCiAqCiAqIEZpeGVzIChhdWRpdCAyNi8wNS8yMDI2KToKICogVDQ6IHBvc2l0aW9uX3gsIHBvc2l0aW9uX3ksIGxvZ29fdXJsIHBlcnNpc3RlZCBhcyB0b3AtbGV2ZWwgY29sdW1ucy4KICogVDc6IGdldFRlY2huaXF1ZVByb21wdCBza2lwcyAiZGVmYXVsdCIgaW4gc2VhcmNoIGxvb3AuCiAqIFQ4OiBmZXRjaE1vY2t1cEhpc3RvcnkgbGltaXRlZCB0byAyMDAgcmVjb3Jkcy4KICogVDEwOiB0aHVtYm5haWxfdXJsIG5vdyBzdG9yZXMgbW9ja3VwVXJsIChub3QgbG9nb1VybCkuCiAqLwppbXBvcnQgeyBzdXBhYmFzZSB9IGZyb20gJ0AvaW50ZWdyYXRpb25zL3N1cGFiYXNlL2NsaWVudCc7CmltcG9ydCB7IHVwbG9hZExvZ29Ub1N0b3JhZ2UsIGRvd25sb2FkSW1hZ2VBc1BkZkZyb21VcmwgfSBmcm9tICdAL2xpYi9tb2NrdXAtc3RvcmFnZSc7CmltcG9ydCB7IHRvYXN0IH0gZnJvbSAnc29ubmVyJzsKaW1wb3J0IHR5cGUgeyBQZXJzb25hbGl6YXRpb25BcmVhIH0gZnJvbSAnQC9jb21wb25lbnRzL21vY2t1cC9NdWx0aUFyZWFNYW5hZ2VyJzsKCmV4cG9ydCBpbnRlcmZhY2UgVGVjaG5pcXVlIHsKICBpZDogc3RyaW5nOwogIG5hbWU6IHN0cmluZzsKICBjb2RlOiBzdHJpbmcgfCBudWxsOwogIFtrZXk6IHN0cmluZ106IHVua25vd247Cn0KCmV4cG9ydCBpbnRlcmZhY2UgR2VuZXJhdGVkTW9ja3VwIHsKICBpZDogc3RyaW5nOwogIHByb2R1Y3RfaWQ6IHN0cmluZyB8IG51bGw7CiAgcHJvZHVjdF9uYW1lOiBzdHJpbmc7CiAgcHJvZHVjdF9za3U6IHN0cmluZyB8IG51bGw7CiAgdGVjaG5pcXVlX2lkOiBzdHJpbmcgfCBudWxsOwogIHRlY2huaXF1ZV9uYW1lOiBzdHJpbmc7CiAgbW9ja3VwX3VybDogc3RyaW5nOwogIGxheW91dF91cmw/OiBzdHJpbmcgfCBudWxsOwogIGxvZ29fdXJsOiBzdHJpbmc7CiAgcG9zaXRpb25feDogbnVtYmVyIHwgbnVsbDsKICBwb3NpdGlvbl95OiBudW1iZXIgfCBudWxsOwogIGxvZ29fd2lkdGhfY206IG51bWJlciB8IG51bGw7CiAgbG9nb19oZWlnaHRfY206IG51bWJlciB8IG51bGw7CiAgbG9jYXRpb25fbmFtZT86IHN0cmluZyB8IG51bGw7CiAgY29sb3JzX2NvdW50PzogbnVtYmVyIHwgbnVsbDsKICBhbm5vdGF0aW9ucz86IEFycmF5PFJlY29yZDxzdHJpbmcsIHVua25vd24+PiB8IG51bGw7CiAgY2xpZW50X25hbWU/OiBzdHJpbmcgfCBudWxsOwogIGNyZWF0ZWRfYXQ6IHN0cmluZzsKICBjbGllbnRfaWQ6IHN0cmluZyB8IG51bGw7Cn0KCmNvbnN0IFRFQ0hOSVFVRV9QUk9NUFRTOiBSZWNvcmQ8c3RyaW5nLCBzdHJpbmc+ID0gewogIGJvcmRhZG86ICdhcyBwcm9mZXNzaW9uYWwgbWFjaGluZSBlbWJyb2lkZXJ5IHdpdGggdmlzaWJsZSB0aHJlYWQgc3RpdGNoIHRleHR1cmUnLAogIHNpbGs6ICdhcyBzY3JlZW4gcHJpbnRlZCB3aXRoIGZsYXQgc29saWQgY29sb3JzLCBtYXR0ZSBmaW5pc2gnLAogIGR0ZjogJ2FzIERURiBwcmludGVkIHRyYW5zZmVyIHdpdGggdmlicmFudCBjb2xvcnMsIHNsaWdodCBnbG9zc3kgZmluaXNoJywKICBsYXNlcjogJ2FzIGxhc2VyIGVuZ3JhdmVkLCBldGNoZWQgaW50byB0aGUgbWF0ZXJpYWwgc3VyZmFjZSwgbW9ub2Nocm9tYXRpYycsCiAgbGFzZXJfY28yOiAnYXMgQ08yIGxhc2VyIGVuZ3JhdmVkIHdpdGggcHJlY2lzZSBldGNoaW5nIG9uIG9yZ2FuaWMgbWF0ZXJpYWxzJywKICBsYXNlcl9maWJyYTogJ2FzIGZpYmVyIGxhc2VyIG1hcmtlZCBvbiBtZXRhbCB3aXRoIGhpZ2gtY29udHJhc3QgcGVybWFuZW50IG1hcmsnLAogIHN1YmxpbWFjYW86ICdhcyBzdWJsaW1hdGlvbiBwcmludGVkLCBjb2xvcnMgYWJzb3JiZWQgc2VhbWxlc3NseSBpbnRvIHRoZSBtYXRlcmlhbCcsCiAgdGFtcG9ncmFmaWE6ICdhcyBwYWQgcHJpbnRlZCB3aXRoIHNsaWdodGx5IGdsb3NzeSBpbmssIHByZWNpc2Ugc21hbGwgZGV0YWlscycsCiAgaG90X3N0YW1waW5nOiAnYXMgaG90IHN0YW1wZWQgd2l0aCBtZXRhbGxpYyBmb2lsIGZpbmlzaCwgc2hpbnkgcmVmbGVjdGl2ZSBzdXJmYWNlJywKICBhZGVzaXZvOiAnYXMgdmlueWwgc3RpY2tlci9kZWNhbCBhcHBsaWVkIHRvIHN1cmZhY2UnLAogIHV2OiAnYXMgVVYgcHJpbnRlZCB3aXRoIHJhaXNlZCBpbmsgdGV4dHVyZSwgdmlicmFudCBjb2xvcnMnLAogIHRyYW5zZmVyOiAnYXMgaGVhdCB0cmFuc2ZlciB2aW55bCwgc21vb3RoIGZpbmlzaCB3aXRoIHNsaWdodCBzaGVlbicsCiAgZGVmYXVsdDogJ2FzIHByb2Zlc3Npb25hbGx5IHByaW50ZWQvYXBwbGllZCBsb2dvJywKfTsKCi8vIFQ3IEZJWDogc2tpcCAiZGVmYXVsdCIgaW4gdGhlIGxvb3AgdG8gYXZvaWQgZmFsc2Ugc3Vic3RyaW5nIG1hdGNoZXMuCmV4cG9ydCBmdW5jdGlvbiBnZXRUZWNobmlxdWVQcm9tcHQodGVjaG5pcXVlOiBUZWNobmlxdWUpOiBzdHJpbmcgewogIGNvbnN0IGNvZGUgPSB0ZWNobmlxdWUuY29kZT8udG9Mb3dlckNhc2UoKSB8fCB0ZWNobmlxdWUubmFtZS50b0xvd2VyQ2FzZSgpOwogIGZvciAoY29uc3QgW2tleSwgcHJvbXB0XSBvZiBPYmplY3QuZW50cmllcyhURUNITklRVUVfUFJPTVBUUykpIHsKICAgIGlmIChrZXkgPT09ICdkZWZhdWx0JykgY29udGludWU7CiAgICBpZiAoY29kZS5pbmNsdWRlcyhrZXkpIHx8IHRlY2huaXF1ZS5uYW1lLnRvTG93ZXJDYXNlKCkuaW5jbHVkZXMoa2V5KSkgcmV0dXJuIHByb21wdDsKICB9CiAgcmV0dXJuIFRFQ0hOSVFVRV9QUk9NUFRTLmRlZmF1bHQ7Cn0KCi8vIFQ4IEZJWDogbGltaXQgdG8gMjAwIHJlY29yZHMgdG8gcHJldmVudCB1bmJvdW5kZWQgcGF5bG9hZCBncm93dGguCmV4cG9ydCBhc3luYyBmdW5jdGlvbiBmZXRjaE1vY2t1cEhpc3RvcnkodXNlcklkPzogc3RyaW5nKTogUHJvbWlzZTxHZW5lcmF0ZWRNb2NrdXBbXT4gewogIGxldCBxdWVyeSA9IHN1cGFiYXNlCiAgICAuZnJvbSgnZ2VuZXJhdGVkX21vY2t1cHMnKQogICAgLnNlbGVjdCgKICAgICAgJ2lkLCBwcm9kdWN0X2lkLCBwcm9kdWN0X25hbWUsIHByb2R1Y3Rfc2t1LCB0ZWNobmlxdWVfaWQsIHRlY2huaXF1ZV9uYW1lLCAnICsKICAgICAgJ21vY2t1cF91cmwsIGxvZ29fdXJsLCBwb3NpdGlvbl94LCBwb3NpdGlvbl95LCBsb2dvX3dpZHRoX2NtLCBsb2dvX2hlaWdodF9jbSwgJyArCiAgICAgICdjbGllbnRfaWQsIGNsaWVudF9uYW1lLCBsb2NhdGlvbl9uYW1lLCBjb2xvcnNfY291bnQsIGFubm90YXRpb25zLCBjcmVhdGVkX2F0JywKICAgICkKICAgIC5vcmRlcignY3JlYXRlZF9hdCcsIHsgYXNjZW5kaW5nOiBmYWxzZSB9KQogICAgLmxpbWl0KDIwMCk7CiAgaWYgKHVzZXJJZCkgcXVlcnkgPSBxdWVyeS5lcSgndXNlcl9pZCcsIHVzZXJJZCk7CiAgY29uc3QgeyBkYXRhLCBlcnJvciB9ID0gYXdhaXQgcXVlcnk7CiAgaWYgKGVycm9yKSB0aHJvdyBlcnJvcjsKICByZXR1cm4gKGRhdGEgfHwgW10pIGFzIHVua25vd24gYXMgR2VuZXJhdGVkTW9ja3VwW107Cn0KCmV4cG9ydCBpbnRlcmZhY2UgU2F2ZU1vY2t1cFBhcmFtcyB7CiAgdXNlcklkOiBzdHJpbmc7CiAgcHJvZHVjdDogeyBpZDogc3RyaW5nOyBuYW1lOiBzdHJpbmc7IHNrdT86IHN0cmluZyB8IG51bGwgfTsKICB0ZWNobmlxdWU6IFRlY2huaXF1ZTsKICBjbGllbnQ6IHsgaWQ/OiBzdHJpbmc7IG5hbWU/OiBzdHJpbmc7IG5vbWVfZmFudGFzaWE/OiBzdHJpbmc7IHJhemFvX3NvY2lhbD86IHN0cmluZyB9IHwgbnVsbDsKICBhcmVhOiBQZXJzb25hbGl6YXRpb25BcmVhOwogIG1vY2t1cFVybDogc3RyaW5nOwogIGFubm90YXRpb25zPzogeyBpZDogc3RyaW5nOyB4OiBudW1iZXI7IHk6IG51bWJlcjsgdGV4dDogc3RyaW5nIH1bXTsKICBleHRyYT86IHsgbGF5b3V0VXJsPzogc3RyaW5nOyBsb2NhdGlvbk5hbWU/OiBzdHJpbmc7IGNvbG9yc0NvdW50PzogbnVtYmVyIH07Cn0KCi8vIFQ0IEZJWDogcG9zaXRpb25feCwgcG9zaXRpb25feSwgbG9nb191cmwsIGxvZ29fd2lkdGhfY20sIGxvZ29faGVpZ2h0X2NtIHBlcnNpc3RlZCB0b3AtbGV2ZWwuCi8vIFQxMCBGSVg6IHRodW1ibmFpbF91cmwgPSBtb2NrdXBVcmwgKHdhcyBpbmNvcnJlY3RseSBzZXQgdG8gbG9nb1VybCkuCmV4cG9ydCBhc3luYyBmdW5jdGlvbiBzYXZlTW9ja3VwVG9EYihwYXJhbXM6IFNhdmVNb2NrdXBQYXJhbXMpOiBQcm9taXNlPHN0cmluZyB8IG51bGw+IHsKICBjb25zdCB7IHVzZXJJZCwgcHJvZHVjdCwgdGVjaG5pcXVlLCBjbGllbnQsIGFyZWEsIG1vY2t1cFVybCwgYW5ub3RhdGlvbnMsIGV4dHJhIH0gPSBwYXJhbXM7CgogIHRyeSB7CiAgICBsZXQgbG9nb1VybCA9IGFyZWEubG9nb1ByZXZpZXcgfHwgJyc7CiAgICBpZiAoYXJlYS5sb2dvUHJldmlldz8uc3RhcnRzV2l0aCgnZGF0YTonKSkgewogICAgICBjb25zdCB1cGxvYWRlZFVybCA9IGF3YWl0IHVwbG9hZExvZ29Ub1N0b3JhZ2UoCiAgICAgICAgdXNlcklkLAogICAgICAgIGFyZWEubG9nb1ByZXZpZXcsCiAgICAgICAgYCR7cHJvZHVjdC5za3UgfHwgJ3Byb2R1Y3QnfS0ke3RlY2huaXF1ZS5jb2RlIHx8ICd0ZWNoJ31gLAogICAgICApOwogICAgICBsb2dvVXJsID0gdXBsb2FkZWRVcmwgfHwgJyc7CiAgICB9CgogICAgbGV0IHNhZmVQcm9kdWN0SWQ6IHN0cmluZyB8IG51bGwgPSBudWxsOwogICAgaWYgKHByb2R1Y3QuaWQpIHsKICAgICAgY29uc3QgeyBkYXRhOiBwcm9kdWN0Um93IH0gPSBhd2FpdCBzdXBhYmFzZQogICAgICAgIC5mcm9tKCdwcm9kdWN0cycpCiAgICAgICAgLnNlbGVjdCgnaWQnKQogICAgICAgIC5lcSgnaWQnLCBwcm9kdWN0LmlkKQogICAgICAgIC5tYXliZVNpbmdsZSgpOwogICAgICBpZiAocHJvZHVjdFJvdykgc2FmZVByb2R1Y3RJZCA9IHByb2R1Y3QuaWQ7CiAgICB9CgogICAgY29uc3Qgc2FmZVRlY2huaXF1ZUlkOiBzdHJpbmcgfCBudWxsID0gdGVjaG5pcXVlLmlkIHx8IG51bGw7CiAgICBjb25zdCBjbGllbnROYW1lID0gY2xpZW50Py5ub21lX2ZhbnRhc2lhIHx8IGNsaWVudD8ucmF6YW9fc29jaWFsIHx8IGNsaWVudD8ubmFtZSB8fCBudWxsOwoKICAgIGNvbnN0IHsgZGF0YTogaW5zZXJ0ZWRSb3csIGVycm9yIH0gPSBhd2FpdCBzdXBhYmFzZQogICAgICAuZnJvbSgnZ2VuZXJhdGVkX21vY2t1cHMnKQogICAgICAuaW5zZXJ0KHsKICAgICAgICB1c2VyX2lkOiB1c2VySWQsCiAgICAgICAgcHJvZHVjdF9pZDogc2FmZVByb2R1Y3RJZCwKICAgICAgICBwcm9kdWN0X25hbWU6IHByb2R1Y3QubmFtZSwKICAgICAgICBwcm9kdWN0X3NrdTogcHJvZHVjdC5za3UgfHwgbnVsbCwKICAgICAgICB0ZWNobmlxdWVfaWQ6IHNhZmVUZWNobmlxdWVJZCwKICAgICAgICB0ZWNobmlxdWVfbmFtZTogdGVjaG5pcXVlLm5hbWUsCiAgICAgICAgbW9ja3VwX3VybDogbW9ja3VwVXJsLAogICAgICAgIHRodW1ibmFpbF91cmw6IG1vY2t1cFVybCB8fCBudWxsLAogICAgICAgIGxvZ29fdXJsOiBsb2dvVXJsIHx8IG51bGwsCiAgICAgICAgcG9zaXRpb25feDogYXJlYS5wb3NpdGlvblgsCiAgICAgICAgcG9zaXRpb25feTogYXJlYS5wb3NpdGlvblksCiAgICAgICAgbG9nb193aWR0aF9jbTogYXJlYS5sb2dvV2lkdGgsCiAgICAgICAgbG9nb19oZWlnaHRfY206IGFyZWEubG9nb0hlaWdodCwKICAgICAgICBhcmVhX25hbWU6IGV4dHJhPy5sb2NhdGlvbk5hbWUgfHwgYXJlYS5uYW1lIHx8ICdGcmVudGUnLAogICAgICAgIGFpX21vZGVsX3VzZWQ6IHRlY2huaXF1ZS5jb2RlIHx8IHRlY2huaXF1ZS5uYW1lIHx8ICdjdXN0b20nLAogICAgICAgIGFyZWFfY29uZmlnOiB7CiAgICAgICAgICBwb3NpdGlvblg6IGFyZWEucG9zaXRpb25YLAogICAgICAgICAgcG9zaXRpb25ZOiBhcmVhLnBvc2l0aW9uWSwKICAgICAgICAgIGxvZ29XaWR0aDogYXJlYS5sb2dvV2lkdGgsCiAgICAgICAgICBsb2dvSGVpZ2h0OiBhcmVhLmxvZ29IZWlnaHQsCiAgICAgICAgICBsb2dvVXJsLAogICAgICAgICAgY2xpZW50TmFtZSwKICAgICAgICAgIGNvbG9yc0NvdW50OiBleHRyYT8uY29sb3JzQ291bnQgfHwgbnVsbCwKICAgICAgICAgIGFubm90YXRpb25zOiBhbm5vdGF0aW9ucyAmJiBhbm5vdGF0aW9ucy5sZW5ndGggPiAwID8gYW5ub3RhdGlvbnMgOiBudWxsLAogICAgICAgIH0sCiAgICAgIH0pCiAgICAgIC5zZWxlY3QoJ2lkJykKICAgICAgLnNpbmdsZSgpOwoKICAgIGlmIChlcnJvcikgdGhyb3cgZXJyb3I7CiAgICByZXR1cm4gaW5zZXJ0ZWRSb3c/LmlkIHx8IG51bGw7CiAgfSBjYXRjaCAoZXJyb3IpIHsKICAgIGNvbnNvbGUuZXJyb3IoJ0Vycm9yIHNhdmluZyB0byBoaXN0b3J5OicsIGVycm9yKTsKICAgIHJldHVybiBudWxsOwogIH0KfQoKZXhwb3J0IGludGVyZmFjZSBHZW5lcmF0ZU1vY2t1cFBhcmFtcyB7CiAgcHJvZHVjdEltYWdlOiBzdHJpbmc7CiAgcHJvZHVjdE5hbWU6IHN0cmluZzsKICB0ZWNobmlxdWU6IFRlY2huaXF1ZTsKICBhcmVhczogUGVyc29uYWxpemF0aW9uQXJlYVtdOwp9CgpleHBvcnQgaW50ZXJmYWNlIEdlbmVyYXRlTW9ja3VwUmVzdWx0IHsKICBzaW5nbGVVcmw6IHN0cmluZyB8IG51bGw7CiAgYmF0Y2hSZXN1bHRzOiB7IGFyZWFOYW1lOiBzdHJpbmc7IHVybDogc3RyaW5nIH1bXTsKfQoKZXhwb3J0IGFzeW5jIGZ1bmN0aW9uIGdlbmVyYXRlTW9ja3VwQXBpKAogIHBhcmFtczogR2VuZXJhdGVNb2NrdXBQYXJhbXMsCik6IFByb21pc2U8R2VuZXJhdGVNb2NrdXBSZXN1bHQ+IHsKICBjb25zdCB7IHByb2R1Y3RJbWFnZSwgcHJvZHVjdE5hbWUsIHRlY2huaXF1ZSwgYXJlYXMgfSA9IHBhcmFtczsKICBjb25zdCBhcmVhc1dpdGhMb2dvcyA9IGFyZWFzLmZpbHRlcigoYSkgPT4gYS5sb2dvUHJldmlldyk7CiAgY29uc3QgdGVjaG5pcXVlUHJvbXB0ID0gZ2V0VGVjaG5pcXVlUHJvbXB0KHRlY2huaXF1ZSk7CgogIGlmIChhcmVhc1dpdGhMb2dvcy5sZW5ndGggPT09IDEpIHsKICAgIGNvbnN0IGFyZWEgPSBhcmVhc1dpdGhMb2dvc1swXTsKICAgIGNvbnN0IGlzTG9nb1VybCA9IGFyZWEubG9nb1ByZXZpZXc/LnN0YXJ0c1dpdGgoJ2h0dHAnKTsKCiAgICBjb25zdCByZXNwb25zZSA9IGF3YWl0IHN1cGFiYXNlLmZ1bmN0aW9ucy5pbnZva2UoJ2dlbmVyYXRlLW1vY2t1cCcsIHsKICAgICAgYm9keTogewogICAgICAgIHByb2R1Y3RJbWFnZVVybDogcHJvZHVjdEltYWdlLAogICAgICAgIGxvZ29CYXNlNjQ6IGlzTG9nb1VybCA/IHVuZGVmaW5lZCA6IGFyZWEubG9nb1ByZXZpZXcsCiAgICAgICAgbG9nb1VybDogaXNMb2dvVXJsID8gYXJlYS5sb2dvUHJldmlldyA6IHVuZGVmaW5lZCwKICAgICAgICB0ZWNobmlxdWVOYW1lOiB0ZWNobmlxdWUubmFtZSwKICAgICAgICB0ZWNobmlxdWVQcm9tcHQsCiAgICAgICAgcG9zaXRpb25YOiBhcmVhLnBvc2l0aW9uWCwKICAgICAgICBwb3NpdGlvblk6IGFyZWEucG9zaXRpb25ZLAogICAgICAgIGxvZ29XaWR0aENtOiBhcmVhLmxvZ29XaWR0aCwKICAgICAgICBsb2dvSGVpZ2h0Q206IGFyZWEubG9nb0hlaWdodCwKICAgICAgICBsb2dvUm90YXRpb246IGFyZWEubG9nb1JvdGF0aW9uIHx8IDAsCiAgICAgICAgbG9nb1NjYWxlOiBhcmVhLmxvZ29TY2FsZSA/PyAxMDAsCiAgICAgICAgcHJvZHVjdE5hbWUsCiAgICAgICAgYXJlYXM6IGFyZWFzV2l0aExvZ29zLm1hcCgoYSkgPT4gKHsKICAgICAgICAgIG5hbWU6IGEubmFtZSwKICAgICAgICAgIHBvc2l0aW9uWDogYS5wb3NpdGlvblgsCiAgICAgICAgICBwb3NpdGlvblk6IGEucG9zaXRpb25ZLAogICAgICAgICAgbG9nb1dpZHRoOiBhLmxvZ29XaWR0aCwKICAgICAgICAgIGxvZ29IZWlnaHQ6IGEubG9nb0hlaWdodCwKICAgICAgICAgIGxvZ29Sb3RhdGlvbjogYS5sb2dvUm90YXRpb24gfHwgMCwKICAgICAgICAgIGxvZ29TY2FsZTogYS5sb2dvU2NhbGUgPz8gMTAwLAogICAgICAgIH0pKSwKICAgICAgfSwKICAgIH0pOwoKICAgIGlmIChyZXNwb25zZS5lcnJvcikgewogICAgICBjb25zdCBlcnJEYXRhID0gcmVzcG9uc2UuZGF0YSB8fCByZXNwb25zZS5lcnJvcjsKICAgICAgaWYgKGVyckRhdGE/LmVycm9yQ29kZSA9PT0gJ1NWR19OT1RfU1VQUE9SVEVEJykgewogICAgICAgIHRocm93IG5ldyBFcnJvcihlcnJEYXRhLmVycm9yIHx8ICdMb2dvcyBTVkcgbsOjbyBzw6NvIHN1cG9ydGFkb3MuIFVzZSBQTkcgb3UgSlBHLicpOwogICAgICB9CiAgICAgIHRocm93IHJlc3BvbnNlLmVycm9yOwogICAgfQogICAgaWYgKCFyZXNwb25zZS5kYXRhPy5tb2NrdXBVcmwpIHRocm93IG5ldyBFcnJvcignTmVuaHVtYSBpbWFnZW0gcmV0b3JuYWRhJyk7CiAgICByZXR1cm4geyBzaW5nbGVVcmw6IHJlc3BvbnNlLmRhdGEubW9ja3VwVXJsLCBiYXRjaFJlc3VsdHM6IFtdIH07CiAgfQoKICAvLyBCQVRDSCDigJQgQVBJIGNhbGxzIHNlcXVlbnRpYWwgKGNvbnN0cmFpbnQpLCBEQiBzYXZlcyBoYW5kbGVkIGluIHBhcmFsbGVsIGJ5IHRoZSBob29rIChUNSkuCiAgY29uc3QgcmVzdWx0czogeyBhcmVhTmFtZTogc3RyaW5nOyB1cmw6IHN0cmluZyB9W10gPSBbXTsKICBjb25zdCBmYWlsZWRBcmVhczogc3RyaW5nW10gPSBbXTsKCiAgZm9yIChjb25zdCBhcmVhIG9mIGFyZWFzV2l0aExvZ29zKSB7CiAgICBjb25zdCBpc0xvZ29VcmwgPSBhcmVhLmxvZ29QcmV2aWV3Py5zdGFydHNXaXRoKCdodHRwJyk7CiAgICB0b2FzdC5pbmZvKGBHZXJhbmRvICR7YXJlYS5uYW1lfS4uLmAsIHsgZHVyYXRpb246IDIwMDAgfSk7CgogICAgY29uc3QgcmVzcG9uc2UgPSBhd2FpdCBzdXBhYmFzZS5mdW5jdGlvbnMuaW52b2tlKCdnZW5lcmF0ZS1tb2NrdXAnLCB7CiAgICAgIGJvZHk6IHsKICAgICAgICBwcm9kdWN0SW1hZ2VVcmw6IHByb2R1Y3RJbWFnZSwKICAgICAgICBsb2dvQmFzZTY0OiBpc0xvZ29VcmwgPyB1bmRlZmluZWQgOiBhcmVhLmxvZ29QcmV2aWV3LAogICAgICAgIGxvZ29Vcmw6IGlzTG9nb1VybCA/IGFyZWEubG9nb1ByZXZpZXcgOiB1bmRlZmluZWQsCiAgICAgICAgdGVjaG5pcXVlTmFtZTogdGVjaG5pcXVlLm5hbWUsCiAgICAgICAgdGVjaG5pcXVlUHJvbXB0LAogICAgICAgIHBvc2l0aW9uWDogYXJlYS5wb3NpdGlvblgsCiAgICAgICAgcG9zaXRpb25ZOiBhcmVhLnBvc2l0aW9uWSwKICAgICAgICBsb2dvV2lkdGhDbTogYXJlYS5sb2dvV2lkdGgsCiAgICAgICAgbG9nb0hlaWdodENtOiBhcmVhLmxvZ29IZWlnaHQsCiAgICAgICAgbG9nb1JvdGF0aW9uOiBhcmVhLmxvZ29Sb3RhdGlvbiB8fCAwLAogICAgICAgIGxvZ29TY2FsZTogYXJlYS5sb2dvU2NhbGUgPz8gMTAwLAogICAgICAgIHByb2R1Y3ROYW1lLAogICAgICAgIGFyZWFzOiBbewogICAgICAgICAgbmFtZTogYXJlYS5uYW1lLAogICAgICAgICAgcG9zaXRpb25YOiBhcmVhLnBvc2l0aW9uWCwKICAgICAgICAgIHBvc2l0aW9uWTogYXJlYS5wb3NpdGlvblksCiAgICAgICAgICBsb2dvV2lkdGg6IGFyZWEubG9nb1dpZHRoLAogICAgICAgICAgbG9nb0hlaWdodDogYXJlYS5sb2dvSGVpZ2h0LAogICAgICAgICAgbG9nb1JvdGF0aW9uOiBhcmVhLmxvZ29Sb3RhdGlvbiB8fCAwLAogICAgICAgICAgbG9nb1NjYWxlOiBhcmVhLmxvZ29TY2FsZSA/PyAxMDAsCiAgICAgICAgfV0sCiAgICAgIH0sCiAgICB9KTsKCiAgICBpZiAocmVzcG9uc2UuZXJyb3IpIHsKICAgICAgY29uc29sZS5lcnJvcihgRXJyb3IgZ2VuZXJhdGluZyAke2FyZWEubmFtZX06YCwgcmVzcG9uc2UuZXJyb3IpOwogICAgICBmYWlsZWRBcmVhcy5wdXNoKGFyZWEubmFtZSk7CiAgICAgIGNvbnRpbnVlOwogICAgfQogICAgaWYgKHJlc3BvbnNlLmRhdGE/Lm1vY2t1cFVybCkKICAgICAgcmVzdWx0cy5wdXNoKHsgYXJlYU5hbWU6IGFyZWEubmFtZSwgdXJsOiByZXNwb25zZS5kYXRhLm1vY2t1cFVybCB9KTsKICB9CgogIGlmIChmYWlsZWRBcmVhcy5sZW5ndGggPiAwKSB7CiAgICB0b2FzdC53YXJuaW5nKAogICAgICBgJHtmYWlsZWRBcmVhcy5sZW5ndGh9IMOhcmVhKHMpIGZhbGhhcmFtOiAke2ZhaWxlZEFyZWFzLmpvaW4oJywgJyl9YCwKICAgICAgeyBkdXJhdGlvbjogNTAwMCB9LAogICAgKTsKICB9CgogIGlmIChyZXN1bHRzLmxlbmd0aCA9PT0gMCkgdGhyb3cgbmV3IEVycm9yKCdOZW5odW0gbW9ja3VwIGdlcmFkbyBubyBiYXRjaCcpOwogIHJldHVybiB7IHNpbmdsZVVybDogcmVzdWx0c1swXS51cmwsIGJhdGNoUmVzdWx0czogcmVzdWx0cyB9Owp9CgpleHBvcnQgYXN5bmMgZnVuY3Rpb24gZG93bmxvYWRNb2NrdXBBc1BkZihtb2NrdXBVcmw6IHN0cmluZywgc2t1Pzogc3RyaW5nLCB0ZWNobmlxdWVOYW1lPzogc3RyaW5nKSB7CiAgY29uc3Qgc2FmZVNrdSA9IChza3UgfHwgJ3Byb2R1dG8nKS5yZXBsYWNlKC9bXmEtekEtWjAtOS1fXS9nLCAnLScpOwogIGNvbnN0IHNhZmVUZWNobmlxdWUgPSAodGVjaG5pcXVlTmFtZSB8fCAndGVjbmljYScpLnJlcGxhY2UoL1teYS16QS1aMC05LV9dL2csICctJyk7CiAgY29uc3QgZmlsZU5hbWUgPSBgbW9ja3VwLSR7c2FmZVNrdX0tJHtzYWZlVGVjaG5pcXVlfS5wZGZgOwogIGF3YWl0IGRvd25sb2FkSW1hZ2VBc1BkZkZyb21VcmwobW9ja3VwVXJsLCBmaWxlTmFtZSk7Cn0KCmV4cG9ydCBhc3luYyBmdW5jdGlvbiBkZWxldGVNb2NrdXBGcm9tRGIoaWQ6IHN0cmluZywgdXNlcklkPzogc3RyaW5nKTogUHJvbWlzZTx2b2lkPiB7CiAgbGV0IHF1ZXJ5ID0gc3VwYWJhc2UuZnJvbSgnZ2VuZXJhdGVkX21vY2t1cHMnKS5kZWxldGUoKS5lcSgnaWQnLCBpZCk7CiAgaWYgKHVzZXJJZCkgcXVlcnkgPSBxdWVyeS5lcSgndXNlcl9pZCcsIHVzZXJJZCk7CiAgY29uc3QgeyBlcnJvciB9ID0gYXdhaXQgcXVlcnk7CiAgaWYgKGVycm9yKSB0aHJvdyBlcnJvcjsKfQoKZXhwb3J0IGNvbnN0IGNyZWF0ZURlZmF1bHRBcmVhID0gKCk6IFBlcnNvbmFsaXphdGlvbkFyZWEgPT4gKHsKICBpZDogY3J5cHRvLnJhbmRvbVVVSUQoKSwKICBuYW1lOiAnRnJlbnRlJywKICBwb3NpdGlvblg6IDUwLAogIHBvc2l0aW9uWTogNTAsCiAgbG9nb1dpZHRoOiA1LAogIGxvZ29IZWlnaHQ6IDMsCiAgbG9nb1JvdGF0aW9uOiAwLAogIGxvZ29TY2FsZTogMTAwLAogIGxvZ29QcmV2aWV3OiBudWxsLAp9KTsK
src/hooks/mockup/useMockupGenerator.ts:1
- Both effects are unmount-only cleanup blocks and can be merged into a single
useEffect(() => () => { ... }, [])to reduce duplicated lifecycle code. This is optional, but it makes the cleanup behavior easier to audit/extend (e.g., if more refs are added later).
/**
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
+623
to
+628
| const batchSaveResults = await Promise.allSettled( | ||
| result.batchResults.map((r, i) => { | ||
| const area = areasWithLogos.find((a) => a.name === r.areaName) || areasWithLogos[i]; | ||
| return saveMockupToHistory(r.url, area).then((recordId) => ({ recordId, r })); | ||
| }), | ||
| ); |
… (T4,T7,T8,T10 already merged via #473)
adm01-debug
changed the title
adm01-debug
added a commit
that referenced
this pull request
May 26, 2026
…D main (#484) Incorpora PR #478 (fix/mockup-generator-audit-10tasks) sobre HEAD atual do main. T1 — useMemo ineficaz: 7 handlers → useCallback com deps corretas T2 — Closures staladas: deps explícitas em cada useCallback async T3 — 2 memory leaks: historyPushTimeout + draftNoticeTimeoutRef com cleanup T4 — position_x/y, logo_url, logo_width/height_cm adicionados ao INSERT top-level T5 — Batch saves sequenciais → Promise.allSettled (escrita paralela) T6 — deleteMockupFromDb sem userId → filtro de owner adicionado T7 — getTechniquePrompt: skip key 'default' no loop T8 — fetchMockupHistory: .limit(200) T9 — window.history.replaceState: preserva outros query params T10 — thumbnail_url = mockupUrl (era logoUrl incorretamente) Co-authored-by: Claude <noreply@anthropic.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.
Fechado: branch rebased sobre HEAD atual do main em
rebase/478-mockup. PR substituto: #481 (será aberto agora). O conflito era causado pela divergência entre a base do PR (commit617a55c) e os 33+ commits que o Lovable fez no main desde então. Os 2 arquivos únicos desta branch (mockupGenerationService.tseuseMockupGenerator.ts) foram aplicados limpos sobre o HEAD atual.