Skip to content

Commit 95a03f8

Browse files
author
DavidQ
committed
Separate app coordination and centralized logging for Preview Generator V2 - PR_26126_042-preview-generator-v2-app-and-logger-separation
1 parent c0cd318 commit 95a03f8

4 files changed

Lines changed: 457 additions & 825 deletions

File tree

‎docs/dev/reports/codex_changed_files.txt‎

Lines changed: 9 additions & 39 deletions
Original file line numberDiff line numberDiff line change
@@ -1,42 +1,12 @@
11
# git status --short
2+
M docs/dev/reports/codex_changed_files.txt
3+
M docs/dev/reports/codex_review.diff
24
M tools/preview-generator-v2/PreviewGeneratorV2App.js
3-
M tools/preview-generator-v2/PreviewGeneratorV2Ui.js
4-
M tools/preview-generator-v2/controls/AssetFolderControl.js
5-
M tools/preview-generator-v2/controls/CaptureModeControl.js
6-
M tools/preview-generator-v2/controls/LastGeneratedImageControl.js
7-
D tools/preview-generator-v2/controls/MenuSampleControl.js
8-
M tools/preview-generator-v2/controls/OutputSummaryControl.js
9-
M tools/preview-generator-v2/controls/PathsOrIdsControl.js
10-
M tools/preview-generator-v2/controls/PreviewFrameControl.js
11-
M tools/preview-generator-v2/controls/RenderControlsControl.js
12-
M tools/preview-generator-v2/controls/RepoDestinationControl.js
13-
D tools/preview-generator-v2/controls/StatusControl.js
14-
M tools/preview-generator-v2/controls/TargetSourceControl.js
15-
?? docs/dev/reports/preview_generator_v2_control_class_boundaries.txt
16-
?? tools/preview-generator-v2/controls/GeneratePreviewControl.js
17-
?? tools/preview-generator-v2/controls/StatusLogControl.js
5+
?? docs/dev/reports/preview_generator_v2_app_logger_separation.txt
186

19-
# PR_26126_041 diff stat excluding generated review artifact files
20-
.../preview-generator-v2/PreviewGeneratorV2App.js | 62 +++-------------------
21-
tools/preview-generator-v2/PreviewGeneratorV2Ui.js | 59 ++++++--------------
22-
.../controls/AssetFolderControl.js | 4 +-
23-
.../controls/CaptureModeControl.js | 4 +-
24-
.../controls/LastGeneratedImageControl.js | 15 ++----
25-
.../controls/MenuSampleControl.js | 25 ---------
26-
.../controls/OutputSummaryControl.js | 6 +--
27-
.../controls/PathsOrIdsControl.js | 4 +-
28-
.../controls/PreviewFrameControl.js | 4 +-
29-
.../controls/RenderControlsControl.js | 6 +--
30-
.../controls/RepoDestinationControl.js | 6 +--
31-
.../preview-generator-v2/controls/StatusControl.js | 13 -----
32-
.../controls/TargetSourceControl.js | 6 +--
33-
13 files changed, 49 insertions(+), 165 deletions(-)
34-
# New file diff stat: docs/dev/reports/preview_generator_v2_control_class_boundaries.txt
35-
...eview_generator_v2_control_class_boundaries.txt | 56 ++++++++++++++++++++++
36-
1 file changed, 56 insertions(+)
37-
# New file diff stat: tools/preview-generator-v2/controls/GeneratePreviewControl.js
38-
.../controls/GeneratePreviewControl.js | 25 ++++++++++++++++++++++
39-
1 file changed, 25 insertions(+)
40-
# New file diff stat: tools/preview-generator-v2/controls/StatusLogControl.js
41-
.../controls/StatusLogControl.js | 21 +++++++++++++++++++++
42-
1 file changed, 21 insertions(+)
7+
# PR_26126_042 diff stat excluding generated review artifact files
8+
.../preview-generator-v2/PreviewGeneratorV2App.js | 329 +--------------------
9+
1 file changed, 5 insertions(+), 324 deletions(-)
10+
# New file diff stat: docs/dev/reports/preview_generator_v2_app_logger_separation.txt
11+
.../preview_generator_v2_app_logger_separation.txt | 48 ++++++++++++++++++++++
12+
1 file changed, 48 insertions(+)

0 commit comments

Comments
 (0)