@itwin/rpcinterface-full-stack-tests 5.13.0-dev.1 → 5.13.0-dev.2

package/lib/dist/bundled-tests.js

@@ -105332,7 +105332,7 @@ class BriefcaseConnection extends _IModelConnection__WEBPACK_IMPORTED_MODULE_5__
      */
     async pushChanges(description) {
         this.requireTimeline();
-        return _IpcApp__WEBPACK_IMPORTED_MODULE_6__.IpcApp.appFunctionIpc.pushChanges(this.key, description);
+        return this.changeset = await _IpcApp__WEBPACK_IMPORTED_MODULE_6__.IpcApp.appFunctionIpc.pushChanges(this.key, description);
     }
     /** The current graphical editing scope, if one is in progress.
      * @see [[enterEditingScope]] to begin graphical editing.
@@ -207453,8 +207453,17 @@ class ToolAdmin {
     }
     /** Handler for keyboard events. */
     static _keyEventHandler = (ev) => {
-        if (!ev.repeat) // we don't want repeated keyboard events. If we keep them they interfere with replacing mouse motion events, since they come as a stream.
+        // we don't want repeated keyboard events. If we keep them they interfere with replacing mouse motion events, since they come as a stream.
+        if (!ev.repeat) {
+            // Suppress default browser behavior for iTwin.js Ctrl shortcuts (Z, Y, C, X, V, F2) on keydown when focus allows.
+            if (ev.type === "keydown" && !ev.isComposing && !ev.defaultPrevented) {
+                if (_IModelApp__WEBPACK_IMPORTED_MODULE_5__.IModelApp.toolAdmin.isCtrlKeyShortcut(ev) && _IModelApp__WEBPACK_IMPORTED_MODULE_5__.IModelApp.toolAdmin.isFocusValidForShortcuts()) {
+                    if (_IModelApp__WEBPACK_IMPORTED_MODULE_5__.IModelApp.toolAdmin.shouldPreventCtrlDefault(ev))
+                        ev.preventDefault();
+                }
+            }
             ToolAdmin.addEvent(ev);
+        }
     };
     /** Handler for modifier key transitions captured before focused UI elements can stop propagation. */
     static _keyEventCaptureHandler = async (event) => {
@@ -208251,9 +208260,6 @@ class ToolAdmin {
     static isContextMenuOpen() {
         return document.querySelector(".core-context-menu-opened") !== null;
     }
-    static isElement(target) {
-        return target instanceof Element;
-    }
     static isEditable(element) {
         const tagName = element.tagName.toLowerCase();
         const editableTags = ["input", "textarea", "select"];
@@ -208263,6 +208269,11 @@ class ToolAdmin {
             return true;
         return false;
     }
+    /** Check if there is currently an active text selection on the page. */
+    static hasActiveTextSelection() {
+        const selection = window.getSelection();
+        return selection !== null && selection.toString().length > 0;
+    }
     static getModifierKey(event) {
         switch (event.key) {
             case "Alt": return _Tool__WEBPACK_IMPORTED_MODULE_13__.BeModifierKeys.Alt;
@@ -208271,22 +208282,75 @@ class ToolAdmin {
         }
         return _Tool__WEBPACK_IMPORTED_MODULE_13__.BeModifierKeys.None;
     }
+    /** Check if focus allows shortcuts to run — i.e., focus is Home or AccuDraw, or focus is not on an editable element and context menu is not open.
+     * Called synchronously in _keyEventHandler to decide whether to call preventDefault(), and in onKeyTransition to gate shortcut processing.
+     * @return true to allow shortcuts
+     */
+    isFocusValidForShortcuts() {
+        // Always allow shortcuts when focus is Home (document.body) or AccuDraw has focus
+        if (ToolAdmin.isFocusHome() || undefined !== _IModelApp__WEBPACK_IMPORTED_MODULE_5__.IModelApp.accuDraw.getFocusItem())
+            return true;
+        // Block shortcuts when context menu is open or focus is on an editable element
+        if (ToolAdmin.isContextMenuOpen())
+            return false;
+        const active = document.activeElement;
+        if (active !== null && ToolAdmin.isEditable(active))
+            return false;
+        return true;
+    }
+    /** Check if a key is part of a Ctrl key shortcut handled by iTwin.js (Ctrl+Z, Ctrl+Y, Ctrl+C, Ctrl+X, Ctrl+V, Ctrl+F2).
+     * @note Subclasses can override to add custom Ctrl shortcuts and handle them with onCtrlKeyPressed.
+     * @return true if key is a Ctrl key shortcut
+     */
+    isCtrlKeyShortcut(keyEvent) {
+        if (!keyEvent.ctrlKey || keyEvent.altKey || keyEvent.metaKey)
+            return false;
+        const lower = keyEvent.key.toLowerCase();
+        return (lower === "z" || lower === "y" || lower === "c" || lower === "x" || lower === "v" || lower === "f2");
+    }
+    /** Determine if the default browser action should be prevented for a Ctrl key shortcut.
+     * By default, allows browser copy/cut of selected text to work naturally, and prevents paste.
+     * Only called when focus is valid for shortcuts (Home, AccuDraw, not on an editable element, context menu not open).
+     * @note This is called from _keyEventHandler to decide whether to call preventDefault().
+     * @note Subclasses can override to customize preventDefault behavior for Ctrl shortcuts.
+     * @return true if preventDefault() should be called; false to allow browser default
+     */
+    shouldPreventCtrlDefault(keyEvent) {
+        const lower = keyEvent.key.toLowerCase();
+        // For copy/cut, only prevent if no active text selection (allow browser to handle text operations)
+        if (lower === "c" || lower === "x") {
+            return !ToolAdmin.hasActiveTextSelection();
+        }
+        // Prevent default for all other iTwin.js shortcuts (z, y, v, F2, custom)
+        // Caller already checked that focus is valid for shortcuts, so v (paste) should be prevented
+        return true;
+    }
     /** Process key down events while the Ctrl key is pressed */
     async onCtrlKeyPressed(keyEvent) {
         let handled = false;
         let result = false;
-        switch (keyEvent.key) {
+        switch (keyEvent.key.toLowerCase()) {
             case "z":
-            case "Z":
                 result = await this.doUndoOperation();
                 handled = true;
                 break;
             case "y":
-            case "Y":
                 result = await this.doRedoOperation();
                 handled = true;
                 break;
-            case "F2":
+            case "c":
+            case "x":
+                // If browser already handled copy/cut (text was selected, so default was not prevented),
+                // mark as handled to skip processShortcutKey.
+                if (!keyEvent.defaultPrevented) {
+                    handled = true;
+                }
+                break;
+            case "v":
+                // Paste is prevented when focus is valid for shortcuts; processShortcutKey can handle Ctrl+V.
+                // When not prevented (e.g., in editable element), browser handles naturally.
+                break;
+            case "f2":
                 result = _IModelApp__WEBPACK_IMPORTED_MODULE_5__.IModelApp.uiAdmin.showKeyinPalette();
                 handled = true;
                 break;
@@ -208300,27 +208364,22 @@ class ToolAdmin {
         return false;
     }
     /** Event for every key down and up transition.
-     * @note Called before processShortcutKey.
+     * @note Called before Ctrl shortcut and focus checks in onKeyTransition.
+     * For custom Ctrl shortcuts, override isCtrlKeyShortcut and onCtrlKeyPressed.
+     * For other shortcuts, override processShortcutKey.
      * @return true if handled and no further processing of event should occur.
      */
     processKeyboardEvent(keyEvent, _wentDown) {
-        if (ToolAdmin.isFocusHome() || undefined !== _IModelApp__WEBPACK_IMPORTED_MODULE_5__.IModelApp.accuDraw.getFocusItem())
-            return false; // Focus is Home or AccuDraw, allow shortcuts...
-        if (keyEvent.defaultPrevented || keyEvent.isComposing)
-            return true; // Respect UI handling / IME composition; don't process shortcuts
         // NOTE: Provide a convenient way for the user to move focus to Home to use shortcuts.
         // Escape is the only practical choice with its default behavior that can cancel/close/blur.
         // Intentionally not checking wentDown as some ui elements stop propagation of down but not up (moving to capture is not a good option).
         // Apps that want to use Escape to start the default tool still can with Home focus in processShortcutKey.
-        if (keyEvent.key === "Escape" && _ToolSettings__WEBPACK_IMPORTED_MODULE_14__.ToolSettings.escapeMovesFocusToHome) {
+        // Let AccuDraw handle Escape when it has focus—AccuDrawViewportUI.onKeyboardEvent manages this directly.
+        if (keyEvent.key === "Escape" && _ToolSettings__WEBPACK_IMPORTED_MODULE_14__.ToolSettings.escapeMovesFocusToHome && !ToolAdmin.isFocusHome() && undefined === _IModelApp__WEBPACK_IMPORTED_MODULE_5__.IModelApp.accuDraw.getFocusItem()) {
             ToolAdmin.setFocusHome();
             return true;
         }
-        if (ToolAdmin.isContextMenuOpen())
-            return true; // Don't allow shortcuts if context menu is open...
-        if (ToolAdmin.isElement(keyEvent.target) && ToolAdmin.isEditable(keyEvent.target))
-            return true; // Don't allow shortcuts when event target is editable...
-        return false;
+        return false; // Not handled; continue processing
     }
     /** Need to check for modifier changes on capture phase as a ui element that calls stopPropagation
      * prevents onKeyTransition from being called when bubbling.
@@ -208334,9 +208393,24 @@ class ToolAdmin {
     /** Event for every key down and up transition. */
     async onKeyTransition(event, wentDown) {
         const keyEvent = event.ev;
+        // Respect IME composition; don't process shortcuts
+        if (keyEvent.isComposing)
+            return _Tool__WEBPACK_IMPORTED_MODULE_13__.EventHandled.Yes;
         if (this.processKeyboardEvent(keyEvent, wentDown))
             return _Tool__WEBPACK_IMPORTED_MODULE_13__.EventHandled.Yes;
-        if (wentDown && keyEvent.ctrlKey) {
+        // Respect if any other handler has handled this event, except for Ctrl shortcuts we manage ourselves.
+        // _keyEventHandler calls preventDefault() for our Ctrl shortcuts, but defaultPrevented should not block them.
+        // For all other keys, defaultPrevented signals that another UI handler has claimed the event.
+        // Exception: AccuDraw's input fields call preventDefault() but still want shortcuts to flow through.
+        const isCtrlShortcut = wentDown && this.isCtrlKeyShortcut(keyEvent);
+        const hasAccuDrawFocus = undefined !== _IModelApp__WEBPACK_IMPORTED_MODULE_5__.IModelApp.accuDraw.getFocusItem();
+        if (keyEvent.defaultPrevented && !isCtrlShortcut && !hasAccuDrawFocus)
+            return _Tool__WEBPACK_IMPORTED_MODULE_13__.EventHandled.Yes; // Respect other UI handling
+        // Check if focus is valid for shortcuts (Home, AccuDraw, not editable, no context menu)
+        if (!this.isFocusValidForShortcuts())
+            return _Tool__WEBPACK_IMPORTED_MODULE_13__.EventHandled.No; // Focus is on an editable element or context menu is open; don't allow shortcuts
+        // Process Ctrl shortcuts if focus is valid
+        if (isCtrlShortcut) {
             const { handled, result } = await this.onCtrlKeyPressed(keyEvent);
             if (handled)
                 return result;
@@ -342151,7 +342225,7 @@ class TestContext {
         this.initializeRpcInterfaces({ title: this.settings.Backend.name, version: this.settings.Backend.version });
         const iModelClient = new imodels_client_management_1.IModelsClient({ api: { baseUrl: `https://${process.env.IMJS_URL_PREFIX ?? ""}api.bentley.com/imodels` } });
         await core_frontend_1.NoRenderApp.startup({
-            applicationVersion: "5.13.0-dev.1",
+            applicationVersion: "5.13.0-dev.2",
             applicationId: this.settings.gprid,
             authorizationClient: new frontend_1.TestFrontendAuthorizationClient(this.serviceAuthToken),
             hubAccess: new imodels_access_frontend_1.FrontendIModelsAccess(iModelClient),
@@ -366988,7 +367062,7 @@ var loadLanguages = instance.loadLanguages;
 (module) {
 
 "use strict";
-module.exports = /*#__PURE__*/JSON.parse('{"name":"@itwin/core-frontend","version":"5.13.0-dev.1","description":"iTwin.js frontend components","main":"lib/cjs/core-frontend.js","module":"lib/esm/core-frontend.js","typings":"lib/cjs/core-frontend","license":"MIT","scripts":{"build":"npm run -s copy:public && npm run -s build:cjs && npm run -s build:esm && npm run -s webpackWorkers && npm run -s copy:workers && npm run -s copy:draco","build:cjs":"npm run -s copy:js:cjs && tsc 1>&2 --outDir lib/cjs","build:esm":"npm run -s copy:js:esm && tsc 1>&2 --module ES2022 --outDir lib/esm","clean":"rimraf -g lib .rush/temp/package-deps*.json","copy:public":"cpx \\"./src/public/**/*\\" ./lib/public","copy:js:cjs":"cpx \\"./src/**/*.js\\" ./lib/cjs","copy:js:esm":"cpx \\"./src/**/*.js\\" ./lib/esm","copy:workers":"cpx \\"./lib/workers/webpack/parse-imdl-worker.js\\" ./lib/public/scripts","copy:draco":"cpx \\"./node_modules/@loaders.gl/draco/dist/libs/*\\" ./lib/public/scripts","docs":"betools docs --json=../../generated-docs/core/core-frontend/file.json --tsIndexFile=./core-frontend.ts --onlyJson --excludes=webgl/**/*,**/map/*.d.ts,**/tile/*.d.ts,**/*-css.ts && npm run -s extract","extract":"betools extract --fileExt=ts --extractFrom=./src/test/example-code --recursive --out=../../generated-docs/extract","extract-api":"betools extract-api --entry=core-frontend && npm run extract-extension-api","extract-extension-api":"eslint --no-inline-config -c extraction.eslint.config.js  \\"./src/**/*.ts\\" 1>&2","lint":"eslint \\"./src/**/*.ts\\" 1>&2","lint-fix":"eslint --fix -f visualstudio \\"./src/**/*.ts\\" 1>&2","lint-deprecation":"eslint --fix -f visualstudio --no-inline-config -c ../../common/config/eslint/eslint.config.deprecation-policy.js \\"./src/**/*.ts\\"","pseudolocalize":"betools pseudolocalize --englishDir ./src/public/locales/en --out ./public/locales/en-PSEUDO","test":"npm run webpackTestWorker && vitest --run","cover":"npm run webpackTestWorker && vitest --run","webpackTests":"webpack --config ./src/test/utils/webpack.config.js 1>&2 && npm run -s webpackTestWorker","webpackTestWorker":"webpack --config ./src/test/worker/webpack.config.js 1>&2 && cpx \\"./lib/test/test-worker.js\\" ./lib/test","webpackWorkers":"webpack --config ./src/workers/ImdlParser/webpack.config.js 1>&2"},"repository":{"type":"git","url":"https://github.com/iTwin/itwinjs-core.git","directory":"core/frontend"},"keywords":["Bentley","BIM","iModel","digital-twin","iTwin"],"author":{"name":"Bentley Systems, Inc.","url":"http://www.bentley.com"},"peerDependencies":{"@itwin/appui-abstract":"workspace:*","@itwin/core-bentley":"workspace:*","@itwin/core-common":"workspace:*","@itwin/core-geometry":"workspace:*","@itwin/core-orbitgt":"workspace:*","@itwin/core-quantity":"workspace:*","@itwin/ecschema-metadata":"workspace:*","@itwin/ecschema-rpcinterface-common":"workspace:*"},"//devDependencies":["NOTE: All peerDependencies should also be listed as devDependencies since peerDependencies are not considered by npm install","NOTE: All tools used by scripts in this package must be listed as devDependencies"],"devDependencies":{"@bentley/aec-units-schema":"^1.0.3","@bentley/formats-schema":"^1.0.0","@bentley/units-schema":"^1.0.10","@itwin/appui-abstract":"workspace:*","@itwin/build-tools":"workspace:*","@itwin/core-bentley":"workspace:*","@itwin/core-common":"workspace:*","@itwin/core-geometry":"workspace:*","@itwin/core-orbitgt":"workspace:*","@itwin/core-quantity":"workspace:*","@itwin/ecschema-metadata":"workspace:*","@itwin/ecschema-rpcinterface-common":"workspace:*","@itwin/object-storage-core":"^3.0.4","@itwin/eslint-plugin":"^6.0.0","@types/chai-as-promised":"^7","@types/draco3d":"^1.4.10","@types/node":"~20.17.0","@types/sinon":"^17.0.2","@vitest/browser-playwright":"^4.1.10","@vitest/coverage-v8":"^4.1.10","cpx2":"^8.0.0","eslint":"^9.31.0","glob":"^10.5.0","playwright":"~1.56.1","rimraf":"^6.0.1","sinon":"^17.0.2","source-map-loader":"^5.0.0","typescript":"~5.6.2","vitest":"^4.1.10","vite-multiple-assets":"^1.3.1","vite-plugin-static-copy":"2.2.0","webpack":"^5.97.1"},"//dependencies":["NOTE: these dependencies should be only for things that DO NOT APPEAR IN THE API","NOTE: core-frontend should remain UI technology agnostic, so no react/angular dependencies are allowed"],"dependencies":{"@itwin/core-i18n":"workspace:*","@itwin/webgl-compatibility":"workspace:*","@loaders.gl/core":"~4.3.4","@loaders.gl/draco":"~4.3.4","fuse.js":"^3.3.0","wms-capabilities":"0.4.0"}}');
+module.exports = /*#__PURE__*/JSON.parse('{"name":"@itwin/core-frontend","version":"5.13.0-dev.2","description":"iTwin.js frontend components","main":"lib/cjs/core-frontend.js","module":"lib/esm/core-frontend.js","typings":"lib/cjs/core-frontend","license":"MIT","scripts":{"build":"npm run -s copy:public && npm run -s build:cjs && npm run -s build:esm && npm run -s webpackWorkers && npm run -s copy:workers && npm run -s copy:draco","build:cjs":"npm run -s copy:js:cjs && tsc 1>&2 --outDir lib/cjs","build:esm":"npm run -s copy:js:esm && tsc 1>&2 --module ES2022 --outDir lib/esm","clean":"rimraf -g lib .rush/temp/package-deps*.json","copy:public":"cpx \\"./src/public/**/*\\" ./lib/public","copy:js:cjs":"cpx \\"./src/**/*.js\\" ./lib/cjs","copy:js:esm":"cpx \\"./src/**/*.js\\" ./lib/esm","copy:workers":"cpx \\"./lib/workers/webpack/parse-imdl-worker.js\\" ./lib/public/scripts","copy:draco":"cpx \\"./node_modules/@loaders.gl/draco/dist/libs/*\\" ./lib/public/scripts","docs":"betools docs --json=../../generated-docs/core/core-frontend/file.json --tsIndexFile=./core-frontend.ts --onlyJson --excludes=webgl/**/*,**/map/*.d.ts,**/tile/*.d.ts,**/*-css.ts && npm run -s extract","extract":"betools extract --fileExt=ts --extractFrom=./src/test/example-code --recursive --out=../../generated-docs/extract","extract-api":"betools extract-api --entry=core-frontend && npm run extract-extension-api","extract-extension-api":"eslint --no-inline-config -c extraction.eslint.config.js  \\"./src/**/*.ts\\" 1>&2","lint":"eslint \\"./src/**/*.ts\\" 1>&2","lint-fix":"eslint --fix -f visualstudio \\"./src/**/*.ts\\" 1>&2","lint-deprecation":"eslint --fix -f visualstudio --no-inline-config -c ../../common/config/eslint/eslint.config.deprecation-policy.js \\"./src/**/*.ts\\"","pseudolocalize":"betools pseudolocalize --englishDir ./src/public/locales/en --out ./public/locales/en-PSEUDO","test":"npm run webpackTestWorker && vitest --run","cover":"npm run webpackTestWorker && vitest --run","webpackTests":"webpack --config ./src/test/utils/webpack.config.js 1>&2 && npm run -s webpackTestWorker","webpackTestWorker":"webpack --config ./src/test/worker/webpack.config.js 1>&2 && cpx \\"./lib/test/test-worker.js\\" ./lib/test","webpackWorkers":"webpack --config ./src/workers/ImdlParser/webpack.config.js 1>&2"},"repository":{"type":"git","url":"https://github.com/iTwin/itwinjs-core.git","directory":"core/frontend"},"keywords":["Bentley","BIM","iModel","digital-twin","iTwin"],"author":{"name":"Bentley Systems, Inc.","url":"http://www.bentley.com"},"peerDependencies":{"@itwin/appui-abstract":"workspace:*","@itwin/core-bentley":"workspace:*","@itwin/core-common":"workspace:*","@itwin/core-geometry":"workspace:*","@itwin/core-orbitgt":"workspace:*","@itwin/core-quantity":"workspace:*","@itwin/ecschema-metadata":"workspace:*","@itwin/ecschema-rpcinterface-common":"workspace:*"},"//devDependencies":["NOTE: All peerDependencies should also be listed as devDependencies since peerDependencies are not considered by npm install","NOTE: All tools used by scripts in this package must be listed as devDependencies"],"devDependencies":{"@bentley/aec-units-schema":"^1.0.3","@bentley/formats-schema":"^1.0.0","@bentley/units-schema":"^1.0.10","@itwin/appui-abstract":"workspace:*","@itwin/build-tools":"workspace:*","@itwin/core-bentley":"workspace:*","@itwin/core-common":"workspace:*","@itwin/core-geometry":"workspace:*","@itwin/core-orbitgt":"workspace:*","@itwin/core-quantity":"workspace:*","@itwin/ecschema-metadata":"workspace:*","@itwin/ecschema-rpcinterface-common":"workspace:*","@itwin/object-storage-core":"^3.0.4","@itwin/eslint-plugin":"^6.0.0","@types/chai-as-promised":"^7","@types/draco3d":"^1.4.10","@types/node":"~20.17.0","@types/sinon":"^17.0.2","@vitest/browser-playwright":"^4.1.10","@vitest/coverage-v8":"^4.1.10","cpx2":"^8.0.0","eslint":"^9.31.0","glob":"^13.0.6","playwright":"~1.56.1","rimraf":"^6.0.1","sinon":"^17.0.2","source-map-loader":"^5.0.0","typescript":"~5.6.2","vitest":"^4.1.10","vite-multiple-assets":"^1.3.1","vite-plugin-static-copy":"2.2.0","webpack":"^5.97.1"},"//dependencies":["NOTE: these dependencies should be only for things that DO NOT APPEAR IN THE API","NOTE: core-frontend should remain UI technology agnostic, so no react/angular dependencies are allowed"],"dependencies":{"@itwin/core-i18n":"workspace:*","@itwin/webgl-compatibility":"workspace:*","@loaders.gl/core":"~4.3.4","@loaders.gl/draco":"~4.3.4","fuse.js":"^3.3.0","wms-capabilities":"0.4.0"}}');
 
 /***/ },