Configure Slim logger at startup and fix icon type issues#395
Conversation
Apply window.config.logger through Slim's logger at boot, simplify Button icon prop typing, and add React DOMAttributes augmentation for @ant-design/icons pointer-capture handlers.
|
|
Overall Grade |
Security Reliability Complexity Hygiene |
Code Review Summary
| Analyzer | Status | Updated (UTC) | Details |
|---|---|---|---|
| JavaScript | Jul 6, 2026 5:14p.m. | Review ↗ |
Important
AI Review is run only on demand for your team. We're only showing results of static analysis review right now. To trigger AI Review, comment @deepsourcebot review on this thread.
|
|
Visit the preview URL for this PR (updated for commit f51bbc8): https://idc-external-006--pr395-fix-slim-logger-and-bjthz516.web.app (expires Mon, 13 Jul 2026 17:16:34 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: 88aacecd98ba54d2f9c8d201a9444e43d1ad8307 |



Summary
window.config.loggerat application startup vialogger.configure()inindex.tsxButtonicon prop typing to acceptIconTypeand generic React componentstypes/react-antd-icons.d.tsto augment ReactDOMAttributeswith optional pointer-capture handlers required by@ant-design/icons@4under@types/reactts5.0Notes
master, notfeat/viv-loadertypes/dicom-microscopy-viewer/index.d.tshunk from the staged diff was omitted becausemasterdoes not definesetLogLevelthereTest plan
pnpm run typecheckpassespnpm startloads withREACT_APP_CONFIG=exampleand respectsconfig.loggersettings