Scaffold UIHTML Console Error Rerouter - #3
Merged
PVDecker1 merged 11 commits intoMar 11, 2026
Conversation
Implemented the initial project structure as described in AGENTS.md, including the ES5-compatible JS shim, the MATLAB ConsoleErrorRerouter class, minimal HTML and script examples, and a MATLAB unit test suite covering constructor behavior, property toggling, error levels filtering, custom formatting, and clean teardown. Co-authored-by: PVDecker1 <89765682+PVDecker1@users.noreply.github.com>
Contributor
|
👋 Jules, reporting for duty! I'm here to lend a hand with this pull request. When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job! For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with New to Jules? Learn more at jules.google/docs. For security, I will only act on instructions from the user who triggered this task. |
Updated the ConsoleErrorRerouter class to dynamically inject consoleShim.js. The constructor now copies the shim to the target directory, reads the original HTMLSource, prepends the script tag, and writes the injected content to a temporary file. The temporary file is then assigned to the HTMLSource property. On destruction, the original HTMLSource is restored and both the temporary HTML file and copied shim are deleted. Cleaned up examples and tests by removing manual shim inclusions. Added tests for auto-delivery. Co-authored-by: PVDecker1 <89765682+PVDecker1@users.noreply.github.com>
Removed the ErrorLevels property entirely. The JavaScript shim now unconditionally intercepts error, warn, info, log, and debug. The default formatter routes these levels to native MATLAB streams: - error -> stderr (red text) - warn -> native warning() - info/log/debug -> stdout Updated FormatFcn signature to return void, putting output responsibility on the formatter itself. Removed outdated ErrorLevels code from tests and examples. Co-authored-by: PVDecker1 <89765682+PVDecker1@users.noreply.github.com>
Evaluated the use of `htmlTree` for HTML manipulation. Confirmed that it requires the Text Analytics Toolbox, which violates the strict constraint in AGENTS.md against introducing new MATLAB toolbox dependencies. The current implementation using `regexpi` and string manipulation is robust and relies only on base MATLAB functions. Co-authored-by: PVDecker1 <89765682+PVDecker1@users.noreply.github.com>
Extracted `MockUihtmlComponent` and `MockHTMLEventData` out of `tests/tConsoleErrorRerouter.m` into their own separate `.m` files to comply with MATLAB's rule against multiple `classdef` blocks in a single file. Added `.github/workflows/matlab-tests.yml` using `matlab-actions` to run automated unit tests on push and pull request events. Co-authored-by: PVDecker1 <89765682+PVDecker1@users.noreply.github.com>
Investigated the GitHub Actions CI failure (`MathWorks Licensing Error 1`). Confirmed that `matlab-actions/setup-matlab` fails on private repositories without explicit license tokens because the free cloud license is only automatically provisioned for public repositories. No code changes are required. The CI will pass once the repository is made public. Co-authored-by: PVDecker1 <89765682+PVDecker1@users.noreply.github.com>
Investigated the GitHub Actions CI failure (`MathWorks Licensing Error 1`). Confirmed that `matlab-actions/setup-matlab` fails on private repositories without explicit license tokens because the free cloud license is only automatically provisioned for public repositories. No code changes are required. The CI will pass once the repository is made public. Co-authored-by: PVDecker1 <89765682+PVDecker1@users.noreply.github.com>
Investigated the GitHub Actions CI failure (`MathWorks Licensing Error 1`). Confirmed that `matlab-actions/setup-matlab` fails on private repositories without explicit license tokens because the free cloud license is only automatically provisioned for public repositories. No code changes are required. The CI will pass once the repository is made public. Co-authored-by: PVDecker1 <89765682+PVDecker1@users.noreply.github.com>
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
1 participant
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.
Implemented the core functionality for intercepting JavaScript
console.errorcalls within MATLABuihtmlcomponents and routing them to the MATLAB Command Window.Files added:
src/js/consoleShim.js: ES5-compatible JavaScript shim for intercepting console methods and sending payloads to MATLAB.src/ConsoleErrorRerouter.m: Main MATLAB class handling theHTMLEventReceivedevents and formatting the output.examples/html/example_page.html: Sample HTML page with inline shim for manual testing.examples/basic_usage.m: Minimal example demonstrating how to use the rerouter.tests/html/test_page.html: HTML fixture for integration testing.tests/tConsoleErrorRerouter.m: Unit test suite for the MATLAB class.PR created automatically by Jules for task 12349836404906975249 started by @PVDecker1