Move export validation from backend errors to frontend warnings#415
Move export validation from backend errors to frontend warnings#415
Conversation
|
Azure Static Web Apps: Your stage site is ready! Visit it here: https://blue-flower-0c36fd81e-415.westus2.5.azurestaticapps.net |
There was a problem hiding this comment.
Pull request overview
Moves export-time validation from backend exceptions/toast notifications into frontend “Problems” warnings to reduce noisy autosave UX, while keeping exports best-effort.
Changes:
- Frontend: add SBML/SBOL export validation warnings in
ProblemsComponentand fix its interval cleanup. - Frontend: stop posting auto-export errors to the embedded parent and reduce SBML debounce to 1000ms.
- Backend: make SBOL/SBML exporters skip incomplete/invalid elements instead of throwing; adjust SBML parameter naming to match iBioSim.
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 9 comments.
Show a summary per file
| File | Description |
|---|---|
| SBOLCanvasFrontend/src/app/problems/problems.component.ts | Adds new validation checks + clears the validation timer on destroy. |
| SBOLCanvasFrontend/src/app/graph.service.ts | Removes embedded error postMessage calls and reduces SBML debounce interval. |
| SBOLCanvasFrontend/src/app/graph-helpers.ts | Exposes event/interaction dict accessors publicly for UI validation. |
| SBOLCanvasBackend/src/utils/MxToSBOL.java | Wraps SBOL export steps in try/catch to skip invalid elements and warn to stderr. |
| SBOLCanvasBackend/src/utils/MxToSBML.java | Skips invalid SBML constructs instead of throwing; renames parameters; adjusts error handling behavior. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
Azure Static Web Apps: Your stage site is ready! Visit it here: https://blue-flower-0c36fd81e-415.westus2.5.azurestaticapps.net |
|
Azure Static Web Apps: Your stage site is ready! Visit it here: https://blue-flower-0c36fd81e-415.westus2.5.azurestaticapps.net |

Export errors during auto-save created a stream of toast notifications in SynBioSuite.
MxToSBML.java,MxToSBOL.java) skips incomplete elements (missing promoters, disconnected edges, null species) and logs to stderr instead of throwingProblemsComponentchecks for the same conditions and shows warnings in the Problems dropdownpostMessagefrom auto-export sending all the toast notifications to SynBioSuitesetIntervalleak inProblemsComponent, timer was never cleared on destroykr_f_<id>tokr_<id>_fto match iBioSim