Conversation
There was a problem hiding this comment.
Pull request overview
Adds an in-app diagnostics experience to the playground, including TypeScript-aware transpilation and an on-demand TypeScript typecheck flow, with new UI indicators for status/diagnostics severity.
Changes:
- Add a Diagnostics drawer UI (toggle, close, clear actions) and supporting styles.
- Introduce TypeScript runtime loading + lib
.d.tsfetching to run in-browser type diagnostics. - Improve status styling (neutral/pending/error) and add Playwright coverage for TS stripping and new status/diagnostics behaviors.
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
| src/styles.css | Adds status severity styles, diagnostics toggle/drawer styling, and a loading indicator for buttons. |
| src/index.html | Adds Diagnostics toggle + drawer markup and a new “Typecheck” button. |
| src/cdn.js | Adds TypeScript CDN import spec and helper to build lib .d.ts URL fallbacks. |
| src/app.js | Implements diagnostics state/rendering, TypeScript compiler+lib loading, typecheck execution, TS-stripping transpile fallback, and status severity handling. |
| playwright/app.spec.ts | Adds E2E coverage for TS annotation transpilation and new status/diagnostics interactions. |
| docs/next-steps.md | Updates roadmap with modularization follow-up items. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| const shouldAttemptTranspileFallback = error => error instanceof SyntaxError | ||
|
|
There was a problem hiding this comment.
For reference later:
Keeping this broad as SyntaxError now intentionally to preserve TS/JSX recovery behavior. Error shape matters but it will be addressed during the planned move to oxc-transform to avoid churn and regressions. Will revisit narrowing/merging fallback errors once the new transform path is in place and covered by tests.
No description provided.