diff --git a/docs/modernization/build-inventory.md b/docs/modernization/build-inventory.md index fa757bd..0efe60d 100644 --- a/docs/modernization/build-inventory.md +++ b/docs/modernization/build-inventory.md @@ -433,6 +433,10 @@ Known local toolchain state: - `pano_cli plan-display-file` exposes `pp_app_core` external file presentation planning as JSON for empty and non-empty paths; the live display-file command consumes the same contract before retained platform open-file bridges. +- `pano_cli plan-keyboard-visibility` exposes `pp_app_core` virtual keyboard + visibility planning as JSON for hidden and visible states; live show/hide + keyboard requests consume the same contract before retained mobile platform + keyboard bridges. - `pano_cli plan-cloud-upload` exposes `pp_app_core` cloud upload availability, new-document warning, publish prompt, and save-before-upload planning as JSON; the live cloud upload command consumes the same start contract before @@ -465,7 +469,8 @@ Known local toolchain state: share execution. - `pp_app_core_document_platform_io_tests` covers empty selected-path filtering and non-empty picked-path callback planning, plus empty/non-empty display-file - planning before platform picker/display callbacks. + planning before platform picker/display callbacks, plus virtual keyboard + show/hide planning before platform keyboard callbacks. - `pp_app_core_document_cloud_tests` covers cloud upload no-canvas, new-document warning, clean publish prompt, and dirty save-before-upload decisions, plus cloud browse no-canvas/show-browser and selected-download diff --git a/docs/modernization/capability-map.md b/docs/modernization/capability-map.md index 1f1af71..df547ca 100644 --- a/docs/modernization/capability-map.md +++ b/docs/modernization/capability-map.md @@ -68,7 +68,7 @@ and validation command. | Mouse/keyboard/touch/gestures | `App`, platform entrypoints | `pp_platform_*`, app | Synthetic event playback | | Wacom pressure | `WacomTablet` | `pp_platform_windows` | Adapter smoke with fallback | | Clipboard/file picker/share/display | `App` platform methods | `pp_app_core`, `pp_platform_*` | Share saved-path, picked-path, and display-file decision tests, platform smoke or mocked service | -| Virtual keyboard | platform entrypoints | `pp_platform_*` | Platform smoke | +| Virtual keyboard | `App`, platform entrypoints | `pp_app_core`, `pp_platform_*` | Keyboard visibility decision tests, platform smoke | | OpenVR desktop | `HMD`, `Vive`, `app_vr` | `pp_platform_vr`, app | Compile gate and mocked pose tests | | Quest/OVR | Android Quest files | `pp_platform_android_quest` | Compile/package gate | | Focus/Wave | Android Focus files | `pp_platform_android_wave` | Compile/package gate | diff --git a/docs/modernization/debt.md b/docs/modernization/debt.md index ac4d76b..9886e9b 100644 --- a/docs/modernization/debt.md +++ b/docs/modernization/debt.md @@ -22,7 +22,7 @@ agent or engineer to remove them without reconstructing context from chat. | --- | --- | --- | --- | --- | --- | --- | | DEBT-0001 | Open | Modernization | Existing platform build files remain alongside new CMake | Required for incremental migration without losing platform coverage | Existing platform builds plus new CMake configure | Remove after all platform builds consume shared CMake targets | | DEBT-0002 | Open | Modernization | Vendored SDK and patched libraries retained initially | Some dependencies are SDK-only, patched, or have platform-specific binaries | Dependency inventory and platform build smoke tests | Replace with vcpkg packages or document permanent vendored status after triplet evaluation | -| DEBT-0003 | Open | Modernization | Existing singletons remain during initial split; `App::open_document`, `App::request_close`, `App::share_file`, `App::cloud_upload`, `App::cloud_upload_all`, `App::cloud_browse`, `App::rec_start`, `App::rec_stop`, `App::rec_clear`, `App::rec_export`, file-menu save actions, `NodeCanvas` save hotkeys, new/open/browse dirty-document workflow prompts, new-document target/resolution/overwrite decisions, save-as document file naming and overwrite decisions, save-version target decisions, export start/target naming/path decisions, share-file saved-path decisions, file/image/save/directory picker selected-path decisions, display-file external-open decisions, recording lifecycle/export progress decisions, cloud-upload prompt/save-before-upload decisions, cloud-browse availability and selected-download decisions, bulk cloud-upload progress decisions, `pano_cli classify-open`, `pano_cli plan-open-route`, `pano_cli plan-new-document`, `pano_cli plan-document-file`, `pano_cli plan-document-version`, `pano_cli plan-export-start`, `pano_cli plan-export-target`, `pano_cli plan-recording-session`, `pano_cli plan-share-file`, `pano_cli plan-picked-path`, `pano_cli plan-display-file`, `pano_cli plan-cloud-upload`, `pano_cli plan-cloud-browse`, `pano_cli plan-cloud-upload-all`, and `pano_cli simulate-app-session` now consume pure `pp_app_core` route/session/export/recording/share/platform-I/O/display/cloud contracts, but document creation/loading, brush import execution, saving, export execution, platform share execution, picker callback execution, display-file platform execution, recording/MP4 execution, cloud upload execution, and cloud browse/download execution still reach legacy `Canvas::I`/UI/network/video/platform singletons | Avoid behavior changes while introducing component boundaries | App launch and component tests; `pp_app_core_document_route_tests`; `pp_app_core_document_export_tests`; `pp_app_core_document_recording_tests`; `pp_app_core_document_sharing_tests`; `pp_app_core_document_platform_io_tests`; `pp_app_core_document_cloud_tests`; `pp_app_core_document_session_tests`; `pano_cli classify-open --path D:/Paint/demo.ppi`; `pano_cli plan-open-route --path D:/Paint/demo.ppi --unsaved`; `pano_cli plan-new-document --work-dir D:/Paint --name demo --resolution-index 3 --target-exists`; `pano_cli plan-document-file --work-dir D:/Paint --name demo --target-exists`; `pano_cli plan-document-version --directory D:/Paint --doc-name demo.01 --existing-path D:/Paint/demo.02.ppi`; `pano_cli plan-export-start --requires-license --demo`; `pano_cli plan-export-target --kind file --work-dir D:/Paint --doc-name demo --extension .png`; `pano_cli plan-recording-session --running --frame-count 12`; `pano_cli plan-share-file --path D:/Paint/demo.ppi`; `pano_cli plan-picked-path --path D:/Paint/demo.ppi`; `pano_cli plan-display-file --path D:/Paint/export.png`; `pano_cli plan-cloud-upload --new-document --unsaved`; `pano_cli plan-cloud-browse --selected-file demo.ppi`; `pano_cli plan-cloud-upload-all --file-count 3`; `pano_cli simulate-app-session --unsaved --save-intent save-dirty-version`; `pano_cli simulate-app-session --no-canvas`; `ctest --preset desktop-fast --build-config Debug` | Replace singleton reaches with context/service injection at component boundaries | +| DEBT-0003 | Open | Modernization | Existing singletons remain during initial split; `App::open_document`, `App::request_close`, `App::share_file`, `App::cloud_upload`, `App::cloud_upload_all`, `App::cloud_browse`, `App::rec_start`, `App::rec_stop`, `App::rec_clear`, `App::rec_export`, file-menu save actions, `NodeCanvas` save hotkeys, new/open/browse dirty-document workflow prompts, new-document target/resolution/overwrite decisions, save-as document file naming and overwrite decisions, save-version target decisions, export start/target naming/path decisions, share-file saved-path decisions, file/image/save/directory picker selected-path decisions, display-file external-open decisions, virtual-keyboard visibility decisions, recording lifecycle/export progress decisions, cloud-upload prompt/save-before-upload decisions, cloud-browse availability and selected-download decisions, bulk cloud-upload progress decisions, `pano_cli classify-open`, `pano_cli plan-open-route`, `pano_cli plan-new-document`, `pano_cli plan-document-file`, `pano_cli plan-document-version`, `pano_cli plan-export-start`, `pano_cli plan-export-target`, `pano_cli plan-recording-session`, `pano_cli plan-share-file`, `pano_cli plan-picked-path`, `pano_cli plan-display-file`, `pano_cli plan-keyboard-visibility`, `pano_cli plan-cloud-upload`, `pano_cli plan-cloud-browse`, `pano_cli plan-cloud-upload-all`, and `pano_cli simulate-app-session` now consume pure `pp_app_core` route/session/export/recording/share/platform-I/O/display/keyboard/cloud contracts, but document creation/loading, brush import execution, saving, export execution, platform share execution, picker callback execution, display-file platform execution, keyboard platform execution, recording/MP4 execution, cloud upload execution, and cloud browse/download execution still reach legacy `Canvas::I`/UI/network/video/platform singletons | Avoid behavior changes while introducing component boundaries | App launch and component tests; `pp_app_core_document_route_tests`; `pp_app_core_document_export_tests`; `pp_app_core_document_recording_tests`; `pp_app_core_document_sharing_tests`; `pp_app_core_document_platform_io_tests`; `pp_app_core_document_cloud_tests`; `pp_app_core_document_session_tests`; `pano_cli classify-open --path D:/Paint/demo.ppi`; `pano_cli plan-open-route --path D:/Paint/demo.ppi --unsaved`; `pano_cli plan-new-document --work-dir D:/Paint --name demo --resolution-index 3 --target-exists`; `pano_cli plan-document-file --work-dir D:/Paint --name demo --target-exists`; `pano_cli plan-document-version --directory D:/Paint --doc-name demo.01 --existing-path D:/Paint/demo.02.ppi`; `pano_cli plan-export-start --requires-license --demo`; `pano_cli plan-export-target --kind file --work-dir D:/Paint --doc-name demo --extension .png`; `pano_cli plan-recording-session --running --frame-count 12`; `pano_cli plan-share-file --path D:/Paint/demo.ppi`; `pano_cli plan-picked-path --path D:/Paint/demo.ppi`; `pano_cli plan-display-file --path D:/Paint/export.png`; `pano_cli plan-keyboard-visibility --visible`; `pano_cli plan-cloud-upload --new-document --unsaved`; `pano_cli plan-cloud-browse --selected-file demo.ppi`; `pano_cli plan-cloud-upload-all --file-count 3`; `pano_cli simulate-app-session --unsaved --save-intent save-dirty-version`; `pano_cli simulate-app-session --no-canvas`; `ctest --preset desktop-fast --build-config Debug` | Replace singleton reaches with context/service injection at component boundaries | | DEBT-0004 | Open | Modernization | Android, Linux, WebGL, Apple, and AppX build files remain platform-specific until root CMake alignment reaches them | Prevent platform regressions during incremental migration; raw Windows `.sln/.vcxproj` files were removed on 2026-05-31 by user decision | `cmake --preset windows-msvc-default`; platform-specific configure/build smoke checks as each platform is migrated | Root CMake owns every platform source list and package path | | DEBT-0005 | Open | Modernization | Temporary local CTest harness is used before Catch2 is wired through vcpkg | `vcpkg` is not currently on PATH, but headless tests need to run now | `ctest --preset desktop-fast --build-config Debug` | Replace `tests/test_harness.h` tests with Catch2 tests once vcpkg toolchain/presets are validated | | DEBT-0007 | Open | Modernization | `vcpkg.json` and `windows-msvc-vcpkg-headless` are validated for the headless Windows component matrix, but app targets still use vendored libraries and Android/Apple triplets are not proven | Dependency migration must stay incremental while SDK/patched/vendor dependencies remain in use | `$env:VCPKG_ROOT="C:\Program Files\Microsoft Visual Studio\2022\Community\VC\vcpkg"; cmake --preset windows-msvc-vcpkg-headless`; `ctest --preset desktop-fast-vcpkg --build-config Debug` | Component targets consume vcpkg packages where reliable and desktop app, Android, and Apple triplets are validated or explicitly documented as permanent vendor exceptions | diff --git a/docs/modernization/roadmap.md b/docs/modernization/roadmap.md index b848859..2a173ee 100644 --- a/docs/modernization/roadmap.md +++ b/docs/modernization/roadmap.md @@ -457,6 +457,9 @@ platform callbacks or legacy picker bridges continue. `pano_cli plan-display-file` exposes the app-core external file presentation decision used by live display-file requests before retained platform open-file bridges continue. +`pano_cli plan-keyboard-visibility` exposes the app-core virtual keyboard +visibility decision used by live show/hide keyboard requests before retained +mobile platform keyboard bridges continue. `pano_cli plan-cloud-upload` exposes the app-core cloud upload decision used by the live cloud upload command for missing-canvas, new-document warning, publish prompt, and dirty-document save-before-upload states before legacy UI, canvas, @@ -946,13 +949,17 @@ Results: - `pp_app_core_document_platform_io_tests` passed, covering empty selected-path filtering and non-empty picked-path callback planning before platform picker callbacks, plus empty/non-empty display-file planning before platform - display callbacks. + display callbacks, plus virtual keyboard show/hide planning before platform + keyboard callbacks. - `pano_cli_plan_picked_path_empty_smoke` and `pano_cli_plan_picked_path_selected_smoke` passed and expose app-core picker selected-path decisions as JSON. - `pano_cli_plan_display_file_empty_smoke` and `pano_cli_plan_display_file_selected_smoke` passed and expose app-core display-file decisions as JSON. +- `pano_cli_plan_keyboard_visibility_hidden_smoke` and + `pano_cli_plan_keyboard_visibility_visible_smoke` passed and expose app-core + virtual keyboard decisions as JSON. - `panopainter_validate_shaders` passed, validating 25 shader programs and 7 shader includes for stage markers and include graph integrity. - `pp_renderer_gl_capabilities_tests` passed on default MSVC, vcpkg-headless, diff --git a/src/app_core/document_platform_io.h b/src/app_core/document_platform_io.h index 2a91f8d..e62c8a2 100644 --- a/src/app_core/document_platform_io.h +++ b/src/app_core/document_platform_io.h @@ -14,6 +14,11 @@ enum class DisplayFileAction { open_external_file, }; +enum class VirtualKeyboardAction { + show_keyboard, + hide_keyboard, +}; + [[nodiscard]] constexpr PickedPathAction plan_picked_path(std::string_view path) noexcept { return path.empty() @@ -28,4 +33,11 @@ enum class DisplayFileAction { : DisplayFileAction::open_external_file; } +[[nodiscard]] constexpr VirtualKeyboardAction plan_virtual_keyboard(bool visible) noexcept +{ + return visible + ? VirtualKeyboardAction::show_keyboard + : VirtualKeyboardAction::hide_keyboard; +} + } diff --git a/src/app_events.cpp b/src/app_events.cpp index 06dd0e2..cd2e571 100644 --- a/src/app_events.cpp +++ b/src/app_events.cpp @@ -19,6 +19,14 @@ void invoke_picked_path_if_selected( callback(path); } +[[nodiscard]] bool should_dispatch_keyboard_visibility(bool visible) noexcept +{ + const auto action = pp::app::plan_virtual_keyboard(visible); + return visible + ? action == pp::app::VirtualKeyboardAction::show_keyboard + : action == pp::app::VirtualKeyboardAction::hide_keyboard; +} + } #ifdef __ANDROID__ @@ -132,6 +140,8 @@ void App::showKeyboard() { LOG("show keyboard"); redraw = true; + if (!should_dispatch_keyboard_visibility(true)) + return; #ifdef __IOS__ dispatch_async(dispatch_get_main_queue(), ^{ [ios_view show_keyboard]; @@ -145,6 +155,8 @@ void App::hideKeyboard() { LOG("hide keyboard"); redraw = true; + if (!should_dispatch_keyboard_visibility(false)) + return; #ifdef __IOS__ dispatch_async(dispatch_get_main_queue(), ^{ [ios_view hide_keyboard]; diff --git a/tests/CMakeLists.txt b/tests/CMakeLists.txt index 984459f..d5a7122 100644 --- a/tests/CMakeLists.txt +++ b/tests/CMakeLists.txt @@ -627,6 +627,18 @@ if(TARGET pano_cli) LABELS "app;integration;desktop-fast" PASS_REGULAR_EXPRESSION "\"command\":\"plan-display-file\".*\"path\":\"D:/Paint/export.png\".*\"decision\":\"open-external-file\"") + add_test(NAME pano_cli_plan_keyboard_visibility_hidden_smoke + COMMAND pano_cli plan-keyboard-visibility) + set_tests_properties(pano_cli_plan_keyboard_visibility_hidden_smoke PROPERTIES + LABELS "app;integration;desktop-fast" + PASS_REGULAR_EXPRESSION "\"command\":\"plan-keyboard-visibility\".*\"visible\":false.*\"decision\":\"hide-keyboard\"") + + add_test(NAME pano_cli_plan_keyboard_visibility_visible_smoke + COMMAND pano_cli plan-keyboard-visibility --visible) + set_tests_properties(pano_cli_plan_keyboard_visibility_visible_smoke PROPERTIES + LABELS "app;integration;desktop-fast" + PASS_REGULAR_EXPRESSION "\"command\":\"plan-keyboard-visibility\".*\"visible\":true.*\"decision\":\"show-keyboard\"") + add_test(NAME pano_cli_simulate_app_session_clean_smoke COMMAND pano_cli simulate-app-session) set_tests_properties(pano_cli_simulate_app_session_clean_smoke PROPERTIES diff --git a/tests/app_core/document_platform_io_tests.cpp b/tests/app_core/document_platform_io_tests.cpp index 294f718..618d9c5 100644 --- a/tests/app_core/document_platform_io_tests.cpp +++ b/tests/app_core/document_platform_io_tests.cpp @@ -31,6 +31,20 @@ void display_file_opens_nonempty_path(pp::tests::Harness& harness) pp::app::plan_display_file("D:/Paint/export.png") == pp::app::DisplayFileAction::open_external_file); } +void virtual_keyboard_plans_show_action(pp::tests::Harness& harness) +{ + PP_EXPECT( + harness, + pp::app::plan_virtual_keyboard(true) == pp::app::VirtualKeyboardAction::show_keyboard); +} + +void virtual_keyboard_plans_hide_action(pp::tests::Harness& harness) +{ + PP_EXPECT( + harness, + pp::app::plan_virtual_keyboard(false) == pp::app::VirtualKeyboardAction::hide_keyboard); +} + } int main() @@ -40,5 +54,7 @@ int main() harness.run("picked path invokes callback for nonempty path", picked_path_invokes_callback_for_nonempty_path); harness.run("display file ignores empty path", display_file_ignores_empty_path); harness.run("display file opens nonempty path", display_file_opens_nonempty_path); + harness.run("virtual keyboard plans show action", virtual_keyboard_plans_show_action); + harness.run("virtual keyboard plans hide action", virtual_keyboard_plans_hide_action); return harness.finish(); } diff --git a/tools/pano_cli/main.cpp b/tools/pano_cli/main.cpp index cbae02c..c98c06a 100644 --- a/tools/pano_cli/main.cpp +++ b/tools/pano_cli/main.cpp @@ -174,6 +174,10 @@ struct PlanDisplayFileArgs { std::string path; }; +struct PlanKeyboardVisibilityArgs { + bool visible = false; +}; + struct SimulateAppSessionArgs { bool has_canvas = true; bool new_document = false; @@ -531,6 +535,18 @@ const char* display_file_action_name(pp::app::DisplayFileAction action) noexcept return "ignore-empty-path"; } +const char* virtual_keyboard_action_name(pp::app::VirtualKeyboardAction action) noexcept +{ + switch (action) { + case pp::app::VirtualKeyboardAction::show_keyboard: + return "show-keyboard"; + case pp::app::VirtualKeyboardAction::hide_keyboard: + return "hide-keyboard"; + } + + return "hide-keyboard"; +} + pp::foundation::Result parse_float_arg(std::string_view text) { float value = 0.0F; @@ -574,6 +590,7 @@ void print_help() << " plan-share-file [--path FILE]\n" << " plan-picked-path [--path FILE]\n" << " plan-display-file [--path FILE]\n" + << " plan-keyboard-visibility [--visible]\n" << " load-project --path FILE\n" << " parse-layout --path FILE\n" << " record-render [--width N] [--height N] [--exercise-clear]\n" @@ -2018,6 +2035,40 @@ int plan_display_file(int argc, char** argv) return 0; } +pp::foundation::Status parse_plan_keyboard_visibility_args( + int argc, + char** argv, + PlanKeyboardVisibilityArgs& args) +{ + for (int i = 2; i < argc; ++i) { + const std::string_view key(argv[i]); + if (key == "--visible") { + args.visible = true; + } else { + return pp::foundation::Status::invalid_argument("unknown option"); + } + } + + return pp::foundation::Status::success(); +} + +int plan_keyboard_visibility(int argc, char** argv) +{ + PlanKeyboardVisibilityArgs args; + const auto status = parse_plan_keyboard_visibility_args(argc, argv, args); + if (!status.ok()) { + print_error("plan-keyboard-visibility", status.message); + return 2; + } + + const auto decision = pp::app::plan_virtual_keyboard(args.visible); + std::cout << "{\"ok\":true,\"command\":\"plan-keyboard-visibility\"" + << ",\"state\":{\"visible\":" << json_bool(args.visible) + << "},\"decision\":\"" << virtual_keyboard_action_name(decision) + << "\"}\n"; + return 0; +} + pp::foundation::Status parse_plan_export_target_args( int argc, char** argv, @@ -4197,6 +4248,10 @@ int main(int argc, char** argv) return plan_display_file(argc, argv); } + if (command == "plan-keyboard-visibility") { + return plan_keyboard_visibility(argc, argv); + } + if (command == "load-project") { return load_project(argc, argv); }