@itwin/core-backend 5.14.3 → 5.14.5

package/CHANGELOG.md

@@ -1,6 +1,16 @@
 # Change Log - @itwin/core-backend
 
-This log was last generated on Mon, 12 Oct 2026 17:37:54 GMT and should not be manually modified.
+This log was last generated on Tue, 27 Oct 2026 12:19:48 GMT and should not be manually modified.
+
+## 5.14.5
+Tue, 27 Oct 2026 12:19:48 GMT
+
+_Version update only_
+
+## 5.14.4
+Fri, 23 Oct 2026 19:58:24 GMT
+
+_Version update only_
 
 ## 5.14.3
 Mon, 12 Oct 2026 17:37:54 GMT

package/lib/esm/test/hubaccess/SchemaImportOrphanColumns.test.d.ts

@@ -0,0 +1,2 @@
+export {};
+//# sourceMappingURL=SchemaImportOrphanColumns.test.d.ts.map

package/lib/esm/test/hubaccess/SchemaImportOrphanColumns.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"SchemaImportOrphanColumns.test.d.ts","sourceRoot":"","sources":["../../../../src/test/hubaccess/SchemaImportOrphanColumns.test.ts"],"names":[],"mappings":""}

package/lib/esm/test/hubaccess/SchemaImportOrphanColumns.test.js

@@ -0,0 +1,142 @@
+/*---------------------------------------------------------------------------------------------
+* Copyright (c) Bentley Systems, Incorporated. All rights reserved.
+* See LICENSE.md in the project root for license information.
+*--------------------------------------------------------------------------------------------*/
+import { DbResult } from "@itwin/core-bentley";
+import * as chai from "chai";
+import * as chaiAsPromised from "chai-as-promised";
+import { HubWrappers, KnownTestLocations } from "..";
+import { HubMock } from "../../internal/HubMock";
+chai.use(chaiAsPromised);
+/**
+ * Repro for Bug 2115705: "DbTable 'bis_ElementMultiAspect' has N non-virtual columns, but the physical table has M columns."
+ *
+ * No semantic rebase, schema sync or dynamic schema involved. A schema adds an ElementMultiAspect subclass wide enough to
+ * ALTER TABLE ADD shared columns to bis_ElementMultiAspect. discardChanges() then reverses the EcSchema txn (ec_* rows)
+ * but skips the Ddl txn (TxnManager::ApplyTxnChanges returns early for TxnType::Ddl), so the physical columns remain
+ * with no ec_Column rows. The next regular schema import used to fail the strict column-count check; it must now succeed.
+ *
+ * The dynamic-schema tests show that deleting properties/classes (allowed for dynamic schemas) is not a cause.
+ */
+describe("Schema import discard leaves orphan physical columns (Bug 2115705)", function () {
+    this.timeout(90000);
+    const tableName = "bis_ElementMultiAspect";
+    const extraColumns = 6;
+    let briefcase;
+    before(() => HubMock.startup("SchemaImportOrphanColumns", KnownTestLocations.outputDir));
+    after(() => HubMock.shutdown());
+    afterEach(() => {
+        briefcase?.close();
+        briefcase = undefined;
+    });
+    const physicalColumns = (db) => db.withSqliteStatement(`PRAGMA table_info([${tableName}])`, (stmt) => {
+        const names = [];
+        while (stmt.step() === DbResult.BE_SQLITE_ROW)
+            names.push(stmt.getValue(1).getString());
+        return names;
+    });
+    const mappedNonVirtualColumnCount = (db) => db.withSqliteStatement(`SELECT COUNT(*) FROM ec_Column c JOIN ec_Table t ON t.Id = c.TableId WHERE t.Name = '${tableName}' AND c.IsVirtual = 0`, (stmt) => {
+        chai.expect(stmt.step()).to.equal(DbResult.BE_SQLITE_ROW);
+        return stmt.getValue(0).getInteger();
+    });
+    const schemaWithWideMultiAspect = (propCount) => `<?xml version="1.0" encoding="UTF-8"?>
+    <ECSchema schemaName="WideAspect" alias="wa" version="01.00.00" xmlns="http://www.bentley.com/schemas/Bentley.ECXML.3.2">
+      <ECSchemaReference name="BisCore" version="01.00.00" alias="bis"/>
+      <ECEntityClass typeName="WideMultiAspect">
+        <BaseClass>bis:ElementMultiAspect</BaseClass>
+        ${Array.from({ length: propCount }, (_, i) => `<ECProperty propertyName="P${i + 1}" typeName="string"/>`).join("\n        ")}
+      </ECEntityClass>
+    </ECSchema>`;
+    const unrelatedSchema = `<?xml version="1.0" encoding="UTF-8"?>
+    <ECSchema schemaName="Unrelated" alias="unr" version="01.00.00" xmlns="http://www.bentley.com/schemas/Bentley.ECXML.3.2">
+      <ECSchemaReference name="BisCore" version="01.00.00" alias="bis"/>
+      <ECEntityClass typeName="Thing">
+        <BaseClass>bis:GraphicalElement2d</BaseClass>
+        <ECProperty propertyName="Name" typeName="string"/>
+      </ECEntityClass>
+    </ECSchema>`;
+    const dynamicSchema = (version, propCount) => `<?xml version="1.0" encoding="UTF-8"?>
+    <ECSchema schemaName="DynTest" alias="dyn" version="${version}" xmlns="http://www.bentley.com/schemas/Bentley.ECXML.3.2">
+      <ECSchemaReference name="BisCore" version="01.00.00" alias="bis"/>
+      <ECSchemaReference name="CoreCustomAttributes" version="01.00.03" alias="CoreCA"/>
+      <ECCustomAttributes>
+        <DynamicSchema xmlns="CoreCustomAttributes.01.00.03"/>
+      </ECCustomAttributes>
+      ${propCount === undefined ? "" : `<ECEntityClass typeName="WideMultiAspect">
+        <BaseClass>bis:ElementMultiAspect</BaseClass>
+        ${Array.from({ length: propCount }, (_, i) => `<ECProperty propertyName="P${i + 1}" typeName="string"/>`).join("\n        ")}
+      </ECEntityClass>`}
+    </ECSchema>`;
+    it("deleting dynamic-schema properties/classes keeps ec_Column and physical columns in sync", async () => {
+        const accessToken = "user1";
+        const iModelId = await HubMock.createNewIModel({ accessToken, iTwinId: HubMock.iTwinId, iModelName: "DynDelete" });
+        briefcase = await HubWrappers.downloadAndOpenBriefcase({ accessToken, iTwinId: HubMock.iTwinId, iModelId });
+        const db = briefcase;
+        const assertInSync = (step) => chai.expect(physicalColumns(db).length).to.equal(mappedNonVirtualColumnCount(db), `out of sync after ${step}`);
+        const sharedColumnCount = physicalColumns(db).filter((name) => /^ps\d+$/i.test(name)).length;
+        const wide = sharedColumnCount + extraColumns;
+        await db.importSchemaStrings([dynamicSchema("01.00.00", wide)]);
+        assertInSync("wide import");
+        await db.importSchemaStrings([dynamicSchema("02.00.00", 1)]); // delete P2..Pn
+        assertInSync("property deletion");
+        await db.importSchemaStrings([dynamicSchema("03.00.00", undefined)]); // delete the class
+        assertInSync("class deletion");
+        await db.importSchemaStrings([unrelatedSchema]); // strict validation passes
+        assertInSync("unrelated import");
+    });
+    it("dynamic-schema deletions pushed and pulled keep the pulling briefcase in sync", async () => {
+        const iModelId = await HubMock.createNewIModel({ accessToken: "user1", iTwinId: HubMock.iTwinId, iModelName: "DynDeletePull" });
+        const writer = await HubWrappers.downloadAndOpenBriefcase({ accessToken: "user1", iTwinId: HubMock.iTwinId, iModelId });
+        const reader = await HubWrappers.downloadAndOpenBriefcase({ accessToken: "user2", iTwinId: HubMock.iTwinId, iModelId });
+        try {
+            const assertInSync = (db, step) => chai.expect(physicalColumns(db).length).to.equal(mappedNonVirtualColumnCount(db), `${db === writer ? "writer" : "reader"} out of sync after ${step}`);
+            const wide = physicalColumns(writer).filter((name) => /^ps\d+$/i.test(name)).length + extraColumns;
+            const steps = [
+                ["wide import", dynamicSchema("01.00.00", wide)],
+                ["property deletion", dynamicSchema("02.00.00", 1)],
+                ["class deletion", dynamicSchema("03.00.00", undefined)],
+            ];
+            for (const [step, xml] of steps) {
+                await writer.importSchemaStrings([xml]);
+                await writer.pushChanges({ description: step });
+                assertInSync(writer, step);
+                await reader.pullChanges();
+                assertInSync(reader, step);
+            }
+            // Strict validation on the pulling briefcase passes.
+            await reader.importSchemaStrings([unrelatedSchema]);
+            assertInSync(reader, "unrelated import");
+        }
+        finally {
+            writer.close();
+            reader.close();
+        }
+    });
+    it("schema import succeeds after discarding a schema import that added columns", async () => {
+        const accessToken = "user1";
+        const iModelId = await HubMock.createNewIModel({ accessToken, iTwinId: HubMock.iTwinId, iModelName: "DiscardOrphan" });
+        briefcase = await HubWrappers.downloadAndOpenBriefcase({ accessToken, iTwinId: HubMock.iTwinId, iModelId });
+        const baselineColumns = physicalColumns(briefcase);
+        chai.expect(baselineColumns.length).to.equal(mappedNonVirtualColumnCount(briefcase), "baseline must be consistent");
+        // Plain (non-dynamic) schema whose aspect needs more shared columns than the table has -> ALTER TABLE ADD COLUMN.
+        const sharedColumnCount = baselineColumns.filter((name) => /^ps\d+$/i.test(name)).length;
+        await briefcase.importSchemaStrings([schemaWithWideMultiAspect(sharedColumnCount + extraColumns)]);
+        chai.expect(physicalColumns(briefcase).length).to.equal(baselineColumns.length + extraColumns, "import should add physical columns");
+        chai.expect(mappedNonVirtualColumnCount(briefcase)).to.equal(baselineColumns.length + extraColumns);
+        // Abandon the un-pushed schema import: ec_* is reverted, the DDL is not.
+        await briefcase.discardChanges();
+        const schemaCount = briefcase.withSqliteStatement("SELECT COUNT(*) FROM ec_Schema WHERE Name = 'WideAspect'", (stmt) => {
+            stmt.step();
+            return stmt.getValue(0).getInteger();
+        });
+        chai.expect(schemaCount).to.equal(0, "schema should be gone after discardChanges");
+        const mapped = mappedNonVirtualColumnCount(briefcase);
+        const physical = physicalColumns(briefcase).length;
+        chai.expect(mapped).to.equal(baselineColumns.length);
+        chai.expect(physical - mapped).to.equal(extraColumns, `orphan columns left in ${tableName}`);
+        // A subsequent regular schema import must tolerate the unused shared columns left behind.
+        await briefcase.importSchemaStrings([unrelatedSchema]);
+        chai.expect(physicalColumns(briefcase).length).to.equal(physical);
+    });
+});
+//# sourceMappingURL=SchemaImportOrphanColumns.test.js.map

package/lib/esm/test/hubaccess/SchemaImportOrphanColumns.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"SchemaImportOrphanColumns.test.js","sourceRoot":"","sources":["../../../../src/test/hubaccess/SchemaImportOrphanColumns.test.ts"],"names":[],"mappings":"AAAA;;;+FAG+F;AAC/F,OAAO,EAAE,QAAQ,EAAE,MAAM,qBAAqB,CAAC;AAC/C,OAAO,KAAK,IAAI,MAAM,MAAM,CAAC;AAC7B,OAAO,KAAK,cAAc,MAAM,kBAAkB,CAAC;AAEnD,OAAO,EAAE,WAAW,EAAE,kBAAkB,EAAE,MAAM,IAAI,CAAC;AAErD,OAAO,EAAE,OAAO,EAAE,MAAM,wBAAwB,CAAC;AAEjD,IAAI,CAAC,GAAG,CAAC,cAAc,CAAC,CAAC;AAEzB;;;;;;;;;GASG;AACH,QAAQ,CAAC,oEAAoE,EAAE;IAC7E,IAAI,CAAC,OAAO,CAAC,KAAK,CAAC,CAAC;IACpB,MAAM,SAAS,GAAG,wBAAwB,CAAC;IAC3C,MAAM,YAAY,GAAG,CAAC,CAAC;IACvB,IAAI,SAAkC,CAAC;IAEvC,MAAM,CAAC,GAAG,EAAE,CAAC,OAAO,CAAC,OAAO,CAAC,2BAA2B,EAAE,kBAAkB,CAAC,SAAS,CAAC,CAAC,CAAC;IACzF,KAAK,CAAC,GAAG,EAAE,CAAC,OAAO,CAAC,QAAQ,EAAE,CAAC,CAAC;IAChC,SAAS,CAAC,GAAG,EAAE;QACb,SAAS,EAAE,KAAK,EAAE,CAAC;QACnB,SAAS,GAAG,SAAS,CAAC;IACxB,CAAC,CAAC,CAAC;IAEH,MAAM,eAAe,GAAG,CAAC,EAAe,EAAY,EAAE,CACpD,EAAE,CAAC,mBAAmB,CAAC,sBAAsB,SAAS,IAAI,EAAE,CAAC,IAAI,EAAE,EAAE;QACnE,MAAM,KAAK,GAAa,EAAE,CAAC;QAC3B,OAAO,IAAI,CAAC,IAAI,EAAE,KAAK,QAAQ,CAAC,aAAa;YAC3C,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAC,SAAS,EAAE,CAAC,CAAC;QAC3C,OAAO,KAAK,CAAC;IACf,CAAC,CAAC,CAAC;IAEL,MAAM,2BAA2B,GAAG,CAAC,EAAe,EAAU,EAAE,CAC9D,EAAE,CAAC,mBAAmB,CACpB,wFAAwF,SAAS,uBAAuB,EACxH,CAAC,IAAI,EAAE,EAAE;QACP,IAAI,CAAC,MAAM,CAAC,IAAI,CAAC,IAAI,EAAE,CAAC,CAAC,EAAE,CAAC,KAAK,CAAC,QAAQ,CAAC,aAAa,CAAC,CAAC;QAC1D,OAAO,IAAI,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAC,UAAU,EAAE,CAAC;IACvC,CAAC,CAAC,CAAC;IAEP,MAAM,yBAAyB,GAAG,CAAC,SAAiB,EAAE,EAAE,CAAC;;;;;UAKjD,KAAK,CAAC,IAAI,CAAC,EAAE,MAAM,EAAE,SAAS,EAAE,EAAE,CAAC,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC,8BAA8B,CAAC,GAAG,CAAC,uBAAuB,CAAC,CAAC,IAAI,CAAC,YAAY,CAAC;;gBAEpH,CAAC;IAEf,MAAM,eAAe,GAAG;;;;;;;gBAOV,CAAC;IAEf,MAAM,aAAa,GAAG,CAAC,OAAe,EAAE,SAA6B,EAAE,EAAE,CAAC;0DAClB,OAAO;;;;;;QAMzD,SAAS,KAAK,SAAS,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC;;UAE7B,KAAK,CAAC,IAAI,CAAC,EAAE,MAAM,EAAE,SAAS,EAAE,EAAE,CAAC,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC,8BAA8B,CAAC,GAAG,CAAC,uBAAuB,CAAC,CAAC,IAAI,CAAC,YAAY,CAAC;uBAC7G;gBACP,CAAC;IAEf,EAAE,CAAC,yFAAyF,EAAE,KAAK,IAAI,EAAE;QACvG,MAAM,WAAW,GAAG,OAAO,CAAC;QAC5B,MAAM,QAAQ,GAAG,MAAM,OAAO,CAAC,eAAe,CAAC,EAAE,WAAW,EAAE,OAAO,EAAE,OAAO,CAAC,OAAO,EAAE,UAAU,EAAE,WAAW,EAAE,CAAC,CAAC;QACnH,SAAS,GAAG,MAAM,WAAW,CAAC,wBAAwB,CAAC,EAAE,WAAW,EAAE,OAAO,EAAE,OAAO,CAAC,OAAO,EAAE,QAAQ,EAAE,CAAC,CAAC;QAC5G,MAAM,EAAE,GAAG,SAAS,CAAC;QACrB,MAAM,YAAY,GAAG,CAAC,IAAY,EAAE,EAAE,CACpC,IAAI,CAAC,MAAM,CAAC,eAAe,CAAC,EAAE,CAAC,CAAC,MAAM,CAAC,CAAC,EAAE,CAAC,KAAK,CAAC,2BAA2B,CAAC,EAAE,CAAC,EAAE,qBAAqB,IAAI,EAAE,CAAC,CAAC;QAEjH,MAAM,iBAAiB,GAAG,eAAe,CAAC,EAAE,CAAC,CAAC,MAAM,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,UAAU,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC,MAAM,CAAC;QAC7F,MAAM,IAAI,GAAG,iBAAiB,GAAG,YAAY,CAAC;QAE9C,MAAM,EAAE,CAAC,mBAAmB,CAAC,CAAC,aAAa,CAAC,UAAU,EAAE,IAAI,CAAC,CAAC,CAAC,CAAC;QAChE,YAAY,CAAC,aAAa,CAAC,CAAC;QAC5B,MAAM,EAAE,CAAC,mBAAmB,CAAC,CAAC,aAAa,CAAC,UAAU,EAAE,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,gBAAgB;QAC9E,YAAY,CAAC,mBAAmB,CAAC,CAAC;QAClC,MAAM,EAAE,CAAC,mBAAmB,CAAC,CAAC,aAAa,CAAC,UAAU,EAAE,SAAS,CAAC,CAAC,CAAC,CAAC,CAAC,mBAAmB;QACzF,YAAY,CAAC,gBAAgB,CAAC,CAAC;QAC/B,MAAM,EAAE,CAAC,mBAAmB,CAAC,CAAC,eAAe,CAAC,CAAC,CAAC,CAAC,2BAA2B;QAC5E,YAAY,CAAC,kBAAkB,CAAC,CAAC;IACnC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,+EAA+E,EAAE,KAAK,IAAI,EAAE;QAC7F,MAAM,QAAQ,GAAG,MAAM,OAAO,CAAC,eAAe,CAAC,EAAE,WAAW,EAAE,OAAO,EAAE,OAAO,EAAE,OAAO,CAAC,OAAO,EAAE,UAAU,EAAE,eAAe,EAAE,CAAC,CAAC;QAChI,MAAM,MAAM,GAAG,MAAM,WAAW,CAAC,wBAAwB,CAAC,EAAE,WAAW,EAAE,OAAO,EAAE,OAAO,EAAE,OAAO,CAAC,OAAO,EAAE,QAAQ,EAAE,CAAC,CAAC;QACxH,MAAM,MAAM,GAAG,MAAM,WAAW,CAAC,wBAAwB,CAAC,EAAE,WAAW,EAAE,OAAO,EAAE,OAAO,EAAE,OAAO,CAAC,OAAO,EAAE,QAAQ,EAAE,CAAC,CAAC;QACxH,IAAI,CAAC;YACH,MAAM,YAAY,GAAG,CAAC,EAAe,EAAE,IAAY,EAAE,EAAE,CACrD,IAAI,CAAC,MAAM,CAAC,eAAe,CAAC,EAAE,CAAC,CAAC,MAAM,CAAC,CAAC,EAAE,CAAC,KAAK,CAAC,2BAA2B,CAAC,EAAE,CAAC,EAAE,GAAG,EAAE,KAAK,MAAM,CAAC,CAAC,CAAC,QAAQ,CAAC,CAAC,CAAC,QAAQ,sBAAsB,IAAI,EAAE,CAAC,CAAC;YAExJ,MAAM,IAAI,GAAG,eAAe,CAAC,MAAM,CAAC,CAAC,MAAM,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,UAAU,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC,MAAM,GAAG,YAAY,CAAC;YACnG,MAAM,KAAK,GAA4B;gBACrC,CAAC,aAAa,EAAE,aAAa,CAAC,UAAU,EAAE,IAAI,CAAC,CAAC;gBAChD,CAAC,mBAAmB,EAAE,aAAa,CAAC,UAAU,EAAE,CAAC,CAAC,CAAC;gBACnD,CAAC,gBAAgB,EAAE,aAAa,CAAC,UAAU,EAAE,SAAS,CAAC,CAAC;aACzD,CAAC;YACF,KAAK,MAAM,CAAC,IAAI,EAAE,GAAG,CAAC,IAAI,KAAK,EAAE,CAAC;gBAChC,MAAM,MAAM,CAAC,mBAAmB,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC;gBACxC,MAAM,MAAM,CAAC,WAAW,CAAC,EAAE,WAAW,EAAE,IAAI,EAAE,CAAC,CAAC;gBAChD,YAAY,CAAC,MAAM,EAAE,IAAI,CAAC,CAAC;gBAC3B,MAAM,MAAM,CAAC,WAAW,EAAE,CAAC;gBAC3B,YAAY,CAAC,MAAM,EAAE,IAAI,CAAC,CAAC;YAC7B,CAAC;YAED,qDAAqD;YACrD,MAAM,MAAM,CAAC,mBAAmB,CAAC,CAAC,eAAe,CAAC,CAAC,CAAC;YACpD,YAAY,CAAC,MAAM,EAAE,kBAAkB,CAAC,CAAC;QAC3C,CAAC;gBAAS,CAAC;YACT,MAAM,CAAC,KAAK,EAAE,CAAC;YACf,MAAM,CAAC,KAAK,EAAE,CAAC;QACjB,CAAC;IACH,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,4EAA4E,EAAE,KAAK,IAAI,EAAE;QAC1F,MAAM,WAAW,GAAG,OAAO,CAAC;QAC5B,MAAM,QAAQ,GAAG,MAAM,OAAO,CAAC,eAAe,CAAC,EAAE,WAAW,EAAE,OAAO,EAAE,OAAO,CAAC,OAAO,EAAE,UAAU,EAAE,eAAe,EAAE,CAAC,CAAC;QACvH,SAAS,GAAG,MAAM,WAAW,CAAC,wBAAwB,CAAC,EAAE,WAAW,EAAE,OAAO,EAAE,OAAO,CAAC,OAAO,EAAE,QAAQ,EAAE,CAAC,CAAC;QAE5G,MAAM,eAAe,GAAG,eAAe,CAAC,SAAS,CAAC,CAAC;QACnD,IAAI,CAAC,MAAM,CAAC,eAAe,CAAC,MAAM,CAAC,CAAC,EAAE,CAAC,KAAK,CAAC,2BAA2B,CAAC,SAAS,CAAC,EAAE,6BAA6B,CAAC,CAAC;QAEpH,kHAAkH;QAClH,MAAM,iBAAiB,GAAG,eAAe,CAAC,MAAM,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,UAAU,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC,MAAM,CAAC;QACzF,MAAM,SAAS,CAAC,mBAAmB,CAAC,CAAC,yBAAyB,CAAC,iBAAiB,GAAG,YAAY,CAAC,CAAC,CAAC,CAAC;QACnG,IAAI,CAAC,MAAM,CAAC,eAAe,CAAC,SAAS,CAAC,CAAC,MAAM,CAAC,CAAC,EAAE,CAAC,KAAK,CAAC,eAAe,CAAC,MAAM,GAAG,YAAY,EAAE,oCAAoC,CAAC,CAAC;QACrI,IAAI,CAAC,MAAM,CAAC,2BAA2B,CAAC,SAAS,CAAC,CAAC,CAAC,EAAE,CAAC,KAAK,CAAC,eAAe,CAAC,MAAM,GAAG,YAAY,CAAC,CAAC;QAEpG,yEAAyE;QACzE,MAAM,SAAS,CAAC,cAAc,EAAE,CAAC;QACjC,MAAM,WAAW,GAAG,SAAS,CAAC,mBAAmB,CAAC,0DAA0D,EAAE,CAAC,IAAI,EAAE,EAAE;YACrH,IAAI,CAAC,IAAI,EAAE,CAAC;YACZ,OAAO,IAAI,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAC,UAAU,EAAE,CAAC;QACvC,CAAC,CAAC,CAAC;QACH,IAAI,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC,EAAE,CAAC,KAAK,CAAC,CAAC,EAAE,4CAA4C,CAAC,CAAC;QAEnF,MAAM,MAAM,GAAG,2BAA2B,CAAC,SAAS,CAAC,CAAC;QACtD,MAAM,QAAQ,GAAG,eAAe,CAAC,SAAS,CAAC,CAAC,MAAM,CAAC;QACnD,IAAI,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,EAAE,CAAC,KAAK,CAAC,eAAe,CAAC,MAAM,CAAC,CAAC;QACrD,IAAI,CAAC,MAAM,CAAC,QAAQ,GAAG,MAAM,CAAC,CAAC,EAAE,CAAC,KAAK,CAAC,YAAY,EAAE,0BAA0B,SAAS,EAAE,CAAC,CAAC;QAE7F,0FAA0F;QAC1F,MAAM,SAAS,CAAC,mBAAmB,CAAC,CAAC,eAAe,CAAC,CAAC,CAAC;QACvD,IAAI,CAAC,MAAM,CAAC,eAAe,CAAC,SAAS,CAAC,CAAC,MAAM,CAAC,CAAC,EAAE,CAAC,KAAK,CAAC,QAAQ,CAAC,CAAC;IACpE,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC","sourcesContent":["/*---------------------------------------------------------------------------------------------\r\n* Copyright (c) Bentley Systems, Incorporated. All rights reserved.\r\n* See LICENSE.md in the project root for license information.\r\n*--------------------------------------------------------------------------------------------*/\r\nimport { DbResult } from \"@itwin/core-bentley\";\r\nimport * as chai from \"chai\";\r\nimport * as chaiAsPromised from \"chai-as-promised\";\r\nimport { Suite } from \"mocha\";\r\nimport { HubWrappers, KnownTestLocations } from \"..\";\r\nimport { BriefcaseDb } from \"../../core-backend\";\r\nimport { HubMock } from \"../../internal/HubMock\";\r\n\r\nchai.use(chaiAsPromised);\r\n\r\n/**\r\n * Repro for Bug 2115705: \"DbTable 'bis_ElementMultiAspect' has N non-virtual columns, but the physical table has M columns.\"\r\n *\r\n * No semantic rebase, schema sync or dynamic schema involved. A schema adds an ElementMultiAspect subclass wide enough to\r\n * ALTER TABLE ADD shared columns to bis_ElementMultiAspect. discardChanges() then reverses the EcSchema txn (ec_* rows)\r\n * but skips the Ddl txn (TxnManager::ApplyTxnChanges returns early for TxnType::Ddl), so the physical columns remain\r\n * with no ec_Column rows. The next regular schema import used to fail the strict column-count check; it must now succeed.\r\n *\r\n * The dynamic-schema tests show that deleting properties/classes (allowed for dynamic schemas) is not a cause.\r\n */\r\ndescribe(\"Schema import discard leaves orphan physical columns (Bug 2115705)\", function (this: Suite) {\r\n  this.timeout(90000);\r\n  const tableName = \"bis_ElementMultiAspect\";\r\n  const extraColumns = 6;\r\n  let briefcase: BriefcaseDb | undefined;\r\n\r\n  before(() => HubMock.startup(\"SchemaImportOrphanColumns\", KnownTestLocations.outputDir));\r\n  after(() => HubMock.shutdown());\r\n  afterEach(() => {\r\n    briefcase?.close();\r\n    briefcase = undefined;\r\n  });\r\n\r\n  const physicalColumns = (db: BriefcaseDb): string[] =>\r\n    db.withSqliteStatement(`PRAGMA table_info([${tableName}])`, (stmt) => {\r\n      const names: string[] = [];\r\n      while (stmt.step() === DbResult.BE_SQLITE_ROW)\r\n        names.push(stmt.getValue(1).getString());\r\n      return names;\r\n    });\r\n\r\n  const mappedNonVirtualColumnCount = (db: BriefcaseDb): number =>\r\n    db.withSqliteStatement(\r\n      `SELECT COUNT(*) FROM ec_Column c JOIN ec_Table t ON t.Id = c.TableId WHERE t.Name = '${tableName}' AND c.IsVirtual = 0`,\r\n      (stmt) => {\r\n        chai.expect(stmt.step()).to.equal(DbResult.BE_SQLITE_ROW);\r\n        return stmt.getValue(0).getInteger();\r\n      });\r\n\r\n  const schemaWithWideMultiAspect = (propCount: number) => `<?xml version=\"1.0\" encoding=\"UTF-8\"?>\r\n    <ECSchema schemaName=\"WideAspect\" alias=\"wa\" version=\"01.00.00\" xmlns=\"http://www.bentley.com/schemas/Bentley.ECXML.3.2\">\r\n      <ECSchemaReference name=\"BisCore\" version=\"01.00.00\" alias=\"bis\"/>\r\n      <ECEntityClass typeName=\"WideMultiAspect\">\r\n        <BaseClass>bis:ElementMultiAspect</BaseClass>\r\n        ${Array.from({ length: propCount }, (_, i) => `<ECProperty propertyName=\"P${i + 1}\" typeName=\"string\"/>`).join(\"\\n        \")}\r\n      </ECEntityClass>\r\n    </ECSchema>`;\r\n\r\n  const unrelatedSchema = `<?xml version=\"1.0\" encoding=\"UTF-8\"?>\r\n    <ECSchema schemaName=\"Unrelated\" alias=\"unr\" version=\"01.00.00\" xmlns=\"http://www.bentley.com/schemas/Bentley.ECXML.3.2\">\r\n      <ECSchemaReference name=\"BisCore\" version=\"01.00.00\" alias=\"bis\"/>\r\n      <ECEntityClass typeName=\"Thing\">\r\n        <BaseClass>bis:GraphicalElement2d</BaseClass>\r\n        <ECProperty propertyName=\"Name\" typeName=\"string\"/>\r\n      </ECEntityClass>\r\n    </ECSchema>`;\r\n\r\n  const dynamicSchema = (version: string, propCount: number | undefined) => `<?xml version=\"1.0\" encoding=\"UTF-8\"?>\r\n    <ECSchema schemaName=\"DynTest\" alias=\"dyn\" version=\"${version}\" xmlns=\"http://www.bentley.com/schemas/Bentley.ECXML.3.2\">\r\n      <ECSchemaReference name=\"BisCore\" version=\"01.00.00\" alias=\"bis\"/>\r\n      <ECSchemaReference name=\"CoreCustomAttributes\" version=\"01.00.03\" alias=\"CoreCA\"/>\r\n      <ECCustomAttributes>\r\n        <DynamicSchema xmlns=\"CoreCustomAttributes.01.00.03\"/>\r\n      </ECCustomAttributes>\r\n      ${propCount === undefined ? \"\" : `<ECEntityClass typeName=\"WideMultiAspect\">\r\n        <BaseClass>bis:ElementMultiAspect</BaseClass>\r\n        ${Array.from({ length: propCount }, (_, i) => `<ECProperty propertyName=\"P${i + 1}\" typeName=\"string\"/>`).join(\"\\n        \")}\r\n      </ECEntityClass>`}\r\n    </ECSchema>`;\r\n\r\n  it(\"deleting dynamic-schema properties/classes keeps ec_Column and physical columns in sync\", async () => {\r\n    const accessToken = \"user1\";\r\n    const iModelId = await HubMock.createNewIModel({ accessToken, iTwinId: HubMock.iTwinId, iModelName: \"DynDelete\" });\r\n    briefcase = await HubWrappers.downloadAndOpenBriefcase({ accessToken, iTwinId: HubMock.iTwinId, iModelId });\r\n    const db = briefcase;\r\n    const assertInSync = (step: string) =>\r\n      chai.expect(physicalColumns(db).length).to.equal(mappedNonVirtualColumnCount(db), `out of sync after ${step}`);\r\n\r\n    const sharedColumnCount = physicalColumns(db).filter((name) => /^ps\\d+$/i.test(name)).length;\r\n    const wide = sharedColumnCount + extraColumns;\r\n\r\n    await db.importSchemaStrings([dynamicSchema(\"01.00.00\", wide)]);\r\n    assertInSync(\"wide import\");\r\n    await db.importSchemaStrings([dynamicSchema(\"02.00.00\", 1)]); // delete P2..Pn\r\n    assertInSync(\"property deletion\");\r\n    await db.importSchemaStrings([dynamicSchema(\"03.00.00\", undefined)]); // delete the class\r\n    assertInSync(\"class deletion\");\r\n    await db.importSchemaStrings([unrelatedSchema]); // strict validation passes\r\n    assertInSync(\"unrelated import\");\r\n  });\r\n\r\n  it(\"dynamic-schema deletions pushed and pulled keep the pulling briefcase in sync\", async () => {\r\n    const iModelId = await HubMock.createNewIModel({ accessToken: \"user1\", iTwinId: HubMock.iTwinId, iModelName: \"DynDeletePull\" });\r\n    const writer = await HubWrappers.downloadAndOpenBriefcase({ accessToken: \"user1\", iTwinId: HubMock.iTwinId, iModelId });\r\n    const reader = await HubWrappers.downloadAndOpenBriefcase({ accessToken: \"user2\", iTwinId: HubMock.iTwinId, iModelId });\r\n    try {\r\n      const assertInSync = (db: BriefcaseDb, step: string) =>\r\n        chai.expect(physicalColumns(db).length).to.equal(mappedNonVirtualColumnCount(db), `${db === writer ? \"writer\" : \"reader\"} out of sync after ${step}`);\r\n\r\n      const wide = physicalColumns(writer).filter((name) => /^ps\\d+$/i.test(name)).length + extraColumns;\r\n      const steps: Array<[string, string]> = [\r\n        [\"wide import\", dynamicSchema(\"01.00.00\", wide)],\r\n        [\"property deletion\", dynamicSchema(\"02.00.00\", 1)],\r\n        [\"class deletion\", dynamicSchema(\"03.00.00\", undefined)],\r\n      ];\r\n      for (const [step, xml] of steps) {\r\n        await writer.importSchemaStrings([xml]);\r\n        await writer.pushChanges({ description: step });\r\n        assertInSync(writer, step);\r\n        await reader.pullChanges();\r\n        assertInSync(reader, step);\r\n      }\r\n\r\n      // Strict validation on the pulling briefcase passes.\r\n      await reader.importSchemaStrings([unrelatedSchema]);\r\n      assertInSync(reader, \"unrelated import\");\r\n    } finally {\r\n      writer.close();\r\n      reader.close();\r\n    }\r\n  });\r\n\r\n  it(\"schema import succeeds after discarding a schema import that added columns\", async () => {\r\n    const accessToken = \"user1\";\r\n    const iModelId = await HubMock.createNewIModel({ accessToken, iTwinId: HubMock.iTwinId, iModelName: \"DiscardOrphan\" });\r\n    briefcase = await HubWrappers.downloadAndOpenBriefcase({ accessToken, iTwinId: HubMock.iTwinId, iModelId });\r\n\r\n    const baselineColumns = physicalColumns(briefcase);\r\n    chai.expect(baselineColumns.length).to.equal(mappedNonVirtualColumnCount(briefcase), \"baseline must be consistent\");\r\n\r\n    // Plain (non-dynamic) schema whose aspect needs more shared columns than the table has -> ALTER TABLE ADD COLUMN.\r\n    const sharedColumnCount = baselineColumns.filter((name) => /^ps\\d+$/i.test(name)).length;\r\n    await briefcase.importSchemaStrings([schemaWithWideMultiAspect(sharedColumnCount + extraColumns)]);\r\n    chai.expect(physicalColumns(briefcase).length).to.equal(baselineColumns.length + extraColumns, \"import should add physical columns\");\r\n    chai.expect(mappedNonVirtualColumnCount(briefcase)).to.equal(baselineColumns.length + extraColumns);\r\n\r\n    // Abandon the un-pushed schema import: ec_* is reverted, the DDL is not.\r\n    await briefcase.discardChanges();\r\n    const schemaCount = briefcase.withSqliteStatement(\"SELECT COUNT(*) FROM ec_Schema WHERE Name = 'WideAspect'\", (stmt) => {\r\n      stmt.step();\r\n      return stmt.getValue(0).getInteger();\r\n    });\r\n    chai.expect(schemaCount).to.equal(0, \"schema should be gone after discardChanges\");\r\n\r\n    const mapped = mappedNonVirtualColumnCount(briefcase);\r\n    const physical = physicalColumns(briefcase).length;\r\n    chai.expect(mapped).to.equal(baselineColumns.length);\r\n    chai.expect(physical - mapped).to.equal(extraColumns, `orphan columns left in ${tableName}`);\r\n\r\n    // A subsequent regular schema import must tolerate the unused shared columns left behind.\r\n    await briefcase.importSchemaStrings([unrelatedSchema]);\r\n    chai.expect(physicalColumns(briefcase).length).to.equal(physical);\r\n  });\r\n});\r\n"]}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@itwin/core-backend",
-  "version": "5.14.3",
+  "version": "5.14.5",
   "description": "iTwin.js backend components",
   "main": "lib/cjs/core-backend.js",
   "module": "lib/esm/core-backend.js",
@@ -27,11 +27,11 @@
   },
   "peerDependencies": {
     "@opentelemetry/api": "^1.0.4",
-    "@itwin/core-bentley": "5.14.3",
-    "@itwin/core-common": "5.14.3",
-    "@itwin/core-geometry": "5.14.3",
-    "@itwin/core-quantity": "5.14.3",
-    "@itwin/ecschema-metadata": "5.14.3"
+    "@itwin/core-bentley": "5.14.5",
+    "@itwin/core-quantity": "5.14.5",
+    "@itwin/core-common": "5.14.5",
+    "@itwin/ecschema-metadata": "5.14.5",
+    "@itwin/core-geometry": "5.14.5"
   },
   "peerDependenciesMeta": {
     "@opentelemetry/api": {
@@ -57,7 +57,7 @@
     "@types/sinon": "^17.0.2",
     "@types/touch": "^3.1.2",
     "@types/ws": "^7.0.0",
-    "@xmldom/xmldom": "~0.8.13",
+    "@xmldom/xmldom": "~0.8.15",
     "chai": "^4.3.10",
     "chai-as-promised": "^7.1.1",
     "cpx2": "^8.0.0",
@@ -71,19 +71,19 @@
     "typescript": "~5.6.2",
     "marked": "^14.1.3",
     "sql-formatter": "^15.4.6",
-    "vite": "^6.4.3",
-    "@itwin/core-geometry": "5.14.3",
-    "@itwin/core-quantity": "5.14.3",
-    "@itwin/build-tools": "5.14.3",
-    "@itwin/core-bentley": "5.14.3",
-    "@itwin/ecschema-metadata": "5.14.3",
-    "internal-tools": "3.0.0-dev.69",
-    "@itwin/ecsql-common": "5.14.3",
-    "@itwin/core-common": "5.14.3",
-    "@itwin/ecschema-locaters": "5.14.3"
+    "vite": "^6.4.4",
+    "@itwin/core-geometry": "5.14.5",
+    "@itwin/core-bentley": "5.14.5",
+    "@itwin/build-tools": "5.14.5",
+    "@itwin/ecschema-locaters": "5.14.5",
+    "@itwin/core-common": "5.14.5",
+    "@itwin/core-quantity": "5.14.5",
+    "@itwin/ecsql-common": "5.14.5",
+    "@itwin/ecschema-metadata": "5.14.5",
+    "internal-tools": "3.0.0-dev.69"
   },
   "dependencies": {
-    "@bentley/imodeljs-native": "5.14.48",
+    "@bentley/imodeljs-native": "5.14.55",
     "@itwin/object-storage-azure": "^3.0.4",
     "@azure/storage-blob": "^12.28.0",
     "form-data": "^4.0.6",