feat: add webhook support for generation completion events#213
Conversation
👷 Deploy request for arcmind-ai pending review.Visit the deploys page to approve it
|
|
@SATYAM-PRATIBHAN Sir, I have finally completed the feature. It took me 2 days to finish as I also had an exam during that time. Please review my PR when you get time and let me know if any changes or improvements are required. I’ve tested the full flow including backend, frontend, and documentation. If possible, I would be grateful if this PR could be considered under level:critical since it includes a full-stack feature implementation with backend, frontend, and documentation changes. Thanks! |
|
Also Sir, I want to inform you before hand only that the frontend is in |
|
Hey @Abfa41, kindly solve the CI checks!! |
|
@SATYAM-PRATIBHAN Sir, I checked the failing CI/type-check issue. The failure is currently caused by existing unused The TypeScript error is:
This happens when a suppression comment exists but the actual TypeScript error no longer exists anymore, so CI fails during strict type checking. Since most of these files are unrelated to the webhook feature/issue, I wanted to confirm first whether you would prefer handling this cleanup separately, or should I remove/fix all these unused directives within this PR itself so the CI passes, even though it would include unrelated project-wide changes outside the webhook issue scope. If you want, we can also ignore this for this PR and handle it later separately. |
@SATYAM-PRATIBHAN Sir, Please let me know as early as possible.. |


Description
This PR adds webhook support to notify users when a system design generation completes (success or failure).
Previously, users had to wait for generation results inside the application. Now, external systems can receive real-time HTTP callbacks when a generation finishes, enabling integrations like automation tools, bots, and CI pipelines.
Type of Change
Related Issues
Closes #108
Features / Completed all Requirements
Screenshots ( Backend )
Screenshots ( Frontend ):
Checklist
Additional Notes