Align guided Inspector and export tutorials - #1
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: e021160ff2
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| blocks, | ||
| connections, | ||
| polyVoiceBlankets, | ||
| hardwareConfig, |
There was a problem hiding this comment.
When a project has Field mappings and the user later switches the target to Seed or Pod, setHardwareConfig deliberately retains fieldControlMappings; forwarding the complete configuration here makes CodeGenerator.generate() run buildFieldMappingConflictErrors() on those inactive mappings unconditionally. Consequently, a retained mapping whose target now has a graph connection blocks an otherwise valid Seed/Pod export, even though getFieldControlMappings() correctly ignores mappings for non-Field targets. Gate conflict validation on the active platform or pass only active mappings.
Useful? React with 👍 / 👎.
| 7. When processing succeeds, compare **Raw (DVPE)** and **AI Corrected** in the | ||
| preview before downloading. |
There was a problem hiding this comment.
After a successful Advanced Export, the UI only displays aiCorrectedCode.mainCpp; handleDownload and downloadViaBrowser always package generatedCode.mainCpp and generatedCode.makefile, so the corrected C++ and corrected Makefile cannot be downloaded or fully diffed as this new workflow directs. Users following the tutorial will unknowingly compile the raw export instead of the reviewed correction; either make Download Files use the selected result (including its Makefile) or document that the corrected result is preview-only.
Useful? React with 👍 / 👎.
1 participant
Aligns the public tutorial path with the original BASE while preserving the clean repository scope. Adds an illustrated Inspector/Hardware/Export/Advanced tutorial, links every tutorial from the README, and fixes export preview construction so the saved hardware target, mapping, and Poly Voice Blankets reach code generation.
Verification: