Passing a SchemaRecord to ember `isEmpty` brings `Error: No field named unknownProperty on user`

[mkszepp]

While making tests with schema record / warp-drive (polaris mode) i was running into error Error: No field named unknownProperty on user.

This occors, if you pass a schema record to isEmpty(), imported from @ember/utils.

While i confirm, that using isEmpty make not really sense with warp-drive, its a possible issue for users which are moving an old app to warp-drive

Is there any was to fix this?

Here my used schemas:

const LoginUserSchema = withDefaults({
  type: 'login/user',
  fields: [
    {
      kind: 'field',
      name: 'guid',
      type: 'guid',
    },
    {
      kind: 'field',
      name: 'userName',
      type: 'string',
    },
    {
      kind: 'field',
      name: 'emailAddress',
      type: 'string',
    },
    {
      kind: 'field',
      name: 'language',
      type: 'number',
    },
    {
      kind: 'field',
      name: 'creationDate',
      type: 'date',
      options: { nullable: false },
    },
    {
      kind: 'field',
      name: 'modificationDate',
      type: 'date',
      options: { nullable: false },
    },
    {
      kind: 'belongsTo',
      name: 'privilegeRole',
      type: 'login/privilege-role',
      options: { async: false, inverse: 'users', linksMode: true },
    },
  ],
});

const LoginPrivilegeRoleSchema = withDefaults({
  type: 'login/privilege-role',
  fields: [
    {
      kind: 'field',
      name: 'name',
      type: 'string',
    },
    {
      kind: 'field',
      name: 'name',
      type: 'string',
    },
    {
      kind: 'field',
      name: 'privileges',
      type: 'array',
    },
    {
      kind: 'hasMany',
      name: 'users',
      type: 'login/user',
      options: { async: false, inverse: 'privilegeRole', linksMode: true },
    },
  ],
});

Ember: 6.6
EmberData / WarpDrive: 5.6

[runspired]

broad strokes: we intentionally error on access of anything not in the schema (sans our own special cased symbols, sans a few things that JS itself will often check for -- 'length', 'nodeType', 'then', 'setInterval', 'document', and sans a few "built-ins" we provide that can be overridden by schema 'constructor', 'prototype', '__proto__', 'toString', 'toJSON', 'toHTML', Symbol.toStringTag, Symbol.toPrimitive, Symbol.iterator).

by erring, we're helping to ensure your usage falls into standard patterns and isn't doing anything odd (such as storing or adding props to keep track of local state).

I think there's a case by which 'unknownProperty' is in the same category as our own internal symbols and language built-ins where we should guard for it. Feel free to PR adding it to IgnoredGlobalFields