Fix delete method implementation and naming collisions

- Fixed ObjectQL.delete() to return boolean directly from driver
- Removed incorrect comment about driver returning record/null
- Fixed MockDriver.delete() to return boolean per DriverInterface spec
- Renamed QueryFilter/QueryOptions to DataEngineFilter/DataEngineQueryOptions to avoid naming collisions with existing data protocol types
- All tests passing

Co-authored-by: hotlong <50353452+hotlong@users.noreply.github.com>

Author: Copilot

packages/objectql/src/index.ts

@@ -398,8 +398,7 @@ export class ObjectQL implements IDataEngine {
     hookContext.result = result;
     await this.triggerHooks('afterDelete', hookContext);
 
-    // Return boolean - true if deletion was successful
-    // The driver.delete should return the deleted record or null
-    return hookContext.result !== null && hookContext.result !== undefined;
+    // Driver.delete() already returns boolean per DriverInterface spec
+    return hookContext.result;
   }
 }

test-dataengine-interface.ts

@@ -48,7 +48,7 @@ class MockDriver {
 
   async delete(object: string, id: any) {
     console.log(`[MockDriver] delete(${object}, ${id})`);
-    return { id };
+    return true; // Return boolean as per DriverInterface
   }
 }