feat: enhanced auto render and pr publish.#38
Merged
knightedcodemonkey merged 4 commits intomainfrom Mar 28, 2026
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This PR enhances the playground’s auto-render behavior (by implicitly creating an App entrypoint when possible) and updates the “Open Pull Request” flow to optionally strip or include the App wrapper when committing component source.
Changes:
- Add implicit
Appwrapper generation in the render runtime when auto-render is enabled, and tighten runtime expectations around anAppentrypoint. - Add an “Include App wrapper…” checkbox to the Open PR drawer; default behavior strips the top-level
Appwrapper from committed component source. - Introduce shared JSX transform/declaration helpers and update Playwright coverage for the new behaviors.
Reviewed changes
Copilot reviewed 12 out of 12 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| src/styles/ai-controls.css | Adds layout/styling for the Open PR drawer checkbox row. |
| src/modules/render-runtime.js | Implements implicit App wrapping for auto-render and updates runtime entrypoint expectations/errors. |
| src/modules/lint-diagnostics.js | Updates unused-binding allowlist to only ignore App. |
| src/modules/jsx-transform-runtime.js | Adds a cached loader for transformJsxSource from the CDN. |
| src/modules/jsx-top-level-declarations.js | Adds utilities to collect and query top-level function-like declarations. |
| src/modules/github-pr-drawer.js | Adds checkbox wiring and default behavior to strip top-level App wrapper before committing. |
| src/index.html | Adds the “Include App wrapper…” checkbox to the Open PR drawer UI. |
| src/app.js | Wires new checkbox + declaration collection into the PR drawer and auto-render behavior into the render runtime. |
| playwright/rendering-modes.spec.ts | Adds tests for preview clearing and implicit App wrapper behavior. |
| playwright/github-pr-drawer.spec.ts | Adds tests ensuring committed source strips/includes App wrapper per checkbox state. |
| playwright/diagnostics.spec.ts | Updates expectations for unused binding handling (now only ignores App). |
| docs/next-steps.md | Adds a new “Phase 2 UX/UI continuation” section outlining next work. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Contributor
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 12 out of 12 changed files in this pull request and generated 4 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
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.
No description provided.