@league-of-foundry-developers/foundry-vtt-types 13.340.1 → 13.345.1-beta.20250704.94554

package/src/foundry/common/documents/region.d.mts

@@ -1,8 +1,8 @@
-import type { DataField, SchemaField } from "#common/data/fields.mjs";
-import type { AnyObject } from "fvtt-types/utils";
+import type { AnyMutableObject } from "#utils";
 import type DataModel from "../abstract/data.d.mts";
 import type Document from "../abstract/document.mts";
 import type { LogCompatibilityWarningOptions } from "../utils/logging.d.mts";
+import type { DataField, SchemaField } from "#common/data/fields.mjs";
 
 /**
  * The Region Document.
@@ -13,14 +13,14 @@ declare abstract class BaseRegion extends Document<"Region", BaseRegion.Schema,
    * @param data    - Initial data from which to construct the `BaseRegion`
    * @param context - Construction context options
    *
-   * @deprecated Constructing `BaseRegion` directly is not advised. The base document classes exist in
+   * @remarks Constructing `BaseRegion` directly is not advised. The base document classes exist in
    * order to use documents on both the client (i.e. where all your code runs) and behind the scenes
    * on the server to manage document validation and storage.
    *
    * You should use {@link RegionDocument.implementation | `new RegionDocument.implementation(...)`} instead which will give you
    * a system specific implementation of `RegionDocument`.
    */
-  constructor(...args: RegionDocument.ConstructorArgs);
+  constructor(data: RegionDocument.CreateData, context?: RegionDocument.ConstructionContext);
 
   /**
    * @defaultValue
@@ -35,7 +35,7 @@ declare abstract class BaseRegion extends Document<"Region", BaseRegion.Schema,
    *     RegionBehavior: "behaviors"
    *   },
    *   schemaVersion: "12.324"
-   * }
+   * })
    * ```
    */
   static override metadata: BaseRegion.Metadata;
@@ -54,47 +54,45 @@ declare abstract class BaseRegion extends Document<"Region", BaseRegion.Schema,
 
   /* Document overrides */
 
-  static " fvtt_types_internal_document_name_static": "Region";
-
   // Same as Document for now
-  protected static override _initializationOrder(): Generator<[string, DataField.Any]>;
+  protected static override _initializationOrder(): Generator<[string, DataField.Any], void, undefined>;
 
-  readonly parentCollection: RegionDocument.ParentCollectionName | null;
+  override readonly parentCollection: RegionDocument.ParentCollectionName | null;
 
-  readonly pack: string | null;
+  override readonly pack: string | null;
 
-  static get implementation(): RegionDocument.ImplementationClass;
+  static override get implementation(): RegionDocument.ImplementationClass;
 
-  static get baseDocument(): typeof BaseRegion;
+  static override get baseDocument(): typeof BaseRegion;
 
-  static get collectionName(): RegionDocument.ParentCollectionName;
+  static override get collectionName(): RegionDocument.ParentCollectionName;
 
-  static get documentName(): RegionDocument.Name;
+  static override get documentName(): RegionDocument.Name;
 
-  static get TYPES(): CONST.BASE_DOCUMENT_TYPE[];
+  static override get TYPES(): CONST.BASE_DOCUMENT_TYPE[];
 
-  static get hasTypeData(): undefined;
+  static override get hasTypeData(): undefined;
 
-  static get hierarchy(): RegionDocument.Hierarchy;
+  static override get hierarchy(): RegionDocument.Hierarchy;
 
   override parent: BaseRegion.Parent;
 
-  static createDocuments<Temporary extends boolean | undefined = false>(
+  static override createDocuments<Temporary extends boolean | undefined = undefined>(
     data: Array<RegionDocument.Implementation | RegionDocument.CreateData> | undefined,
     operation?: Document.Database.CreateOperation<RegionDocument.Database.Create<Temporary>>,
   ): Promise<Array<Document.TemporaryIf<RegionDocument.Implementation, Temporary>>>;
 
-  static updateDocuments(
+  static override updateDocuments(
     updates: RegionDocument.UpdateData[] | undefined,
     operation?: Document.Database.UpdateDocumentsOperation<RegionDocument.Database.Update>,
   ): Promise<RegionDocument.Implementation[]>;
 
-  static deleteDocuments(
+  static override deleteDocuments(
     ids: readonly string[] | undefined,
     operation?: Document.Database.DeleteDocumentsOperation<RegionDocument.Database.Delete>,
   ): Promise<RegionDocument.Implementation[]>;
 
-  static override create<Temporary extends boolean | undefined = false>(
+  static override create<Temporary extends boolean | undefined = undefined>(
     data: RegionDocument.CreateData | RegionDocument.CreateData[],
     operation?: RegionDocument.Database.CreateOperation<Temporary>,
   ): Promise<Document.TemporaryIf<RegionDocument.Implementation, Temporary> | undefined>;
@@ -132,22 +130,24 @@ declare abstract class BaseRegion extends Document<"Region", BaseRegion.Schema,
     // operation?: Document.Database.CreateOperation<Document.Database.CreateForName<EmbeddedName>>,
     // However this causes a number of errors.
     operation?: object,
-  ): Promise<Array<Document.Stored<Document.ImplementationFor<EmbeddedName>>> | undefined>;
+  ): Promise<Array<Document.StoredForName<EmbeddedName>> | undefined>;
 
   override updateEmbeddedDocuments<EmbeddedName extends RegionDocument.Embedded.Name>(
     embeddedName: EmbeddedName,
     updates: Document.UpdateDataForName<EmbeddedName>[] | undefined,
     operation?: Document.Database.UpdateOperationForName<EmbeddedName>,
-  ): Promise<Array<Document.Stored<Document.ImplementationFor<EmbeddedName>>> | undefined>;
+  ): Promise<Array<Document.StoredForName<EmbeddedName>> | undefined>;
 
   override deleteEmbeddedDocuments<EmbeddedName extends RegionDocument.Embedded.Name>(
     embeddedName: EmbeddedName,
     ids: Array<string>,
     operation?: Document.Database.DeleteOperationForName<EmbeddedName>,
-  ): Promise<Array<Document.Stored<Document.ImplementationFor<EmbeddedName>>>>;
+  ): Promise<Array<Document.StoredForName<EmbeddedName>>>;
 
   // Same as Document for now
-  override traverseEmbeddedDocuments(_parentPath?: string): Generator<[string, Document.AnyChild<this>]>;
+  override traverseEmbeddedDocuments(
+    _parentPath?: string,
+  ): Generator<[string, Document.AnyChild<this>], void, undefined>;
 
   override getFlag<Scope extends RegionDocument.Flags.Scope, Key extends RegionDocument.Flags.Key<Scope>>(
     scope: Scope,
@@ -165,118 +165,144 @@ declare abstract class BaseRegion extends Document<"Region", BaseRegion.Schema,
     key: Key,
   ): Promise<this>;
 
-  protected _preCreate(
+  protected override _preCreate(
     data: RegionDocument.CreateData,
     options: RegionDocument.Database.PreCreateOptions,
     user: User.Implementation,
   ): Promise<boolean | void>;
 
-  protected _onCreate(
+  protected override _onCreate(
     data: RegionDocument.CreateData,
     options: RegionDocument.Database.OnCreateOperation,
     userId: string,
   ): void;
 
-  protected static _preCreateOperation(
+  protected static override _preCreateOperation(
     documents: RegionDocument.Implementation[],
     operation: Document.Database.PreCreateOperationStatic<RegionDocument.Database.Create>,
     user: User.Implementation,
   ): Promise<boolean | void>;
 
-  protected static _onCreateOperation(
+  protected static override _onCreateOperation(
     documents: RegionDocument.Implementation[],
     operation: RegionDocument.Database.Create,
     user: User.Implementation,
   ): Promise<void>;
 
-  protected _preUpdate(
+  protected override _preUpdate(
     changed: RegionDocument.UpdateData,
     options: RegionDocument.Database.PreUpdateOptions,
     user: User.Implementation,
   ): Promise<boolean | void>;
 
-  protected _onUpdate(
+  protected override _onUpdate(
     changed: RegionDocument.UpdateData,
     options: RegionDocument.Database.OnUpdateOperation,
     userId: string,
   ): void;
 
-  protected static _preUpdateOperation(
+  protected static override _preUpdateOperation(
     documents: RegionDocument.Implementation[],
     operation: RegionDocument.Database.Update,
     user: User.Implementation,
   ): Promise<boolean | void>;
 
-  protected static _onUpdateOperation(
+  protected static override _onUpdateOperation(
     documents: RegionDocument.Implementation[],
     operation: RegionDocument.Database.Update,
     user: User.Implementation,
   ): Promise<void>;
 
-  protected _preDelete(
+  protected override _preDelete(
     options: RegionDocument.Database.PreDeleteOptions,
     user: User.Implementation,
   ): Promise<boolean | void>;
 
-  protected _onDelete(options: RegionDocument.Database.OnDeleteOperation, userId: string): void;
+  protected override _onDelete(options: RegionDocument.Database.OnDeleteOperation, userId: string): void;
 
-  protected static _preDeleteOperation(
+  protected static override _preDeleteOperation(
     documents: RegionDocument.Implementation[],
     operation: RegionDocument.Database.Delete,
     user: User.Implementation,
   ): Promise<boolean | void>;
 
-  protected static _onDeleteOperation(
+  protected static override _onDeleteOperation(
     documents: RegionDocument.Implementation[],
     operation: RegionDocument.Database.Delete,
     user: User.Implementation,
   ): Promise<void>;
 
-  static get hasSystemData(): undefined;
-
   // These data field things have been ticketed but will probably go into backlog hell for a while.
   // We'll end up copy and pasting without modification for now I think. It makes it a tiny bit easier to update though.
-  protected static _addDataFieldShims(data: AnyObject, shims: AnyObject, options?: Document.DataFieldShimOptions): void;
 
-  protected static _addDataFieldMigration(
-    data: AnyObject,
+  // options: not null (parameter default only in _addDataFieldShim)
+  protected static override _addDataFieldShims(
+    data: AnyMutableObject,
+    shims: Record<string, string>,
+    options?: Document.DataFieldShimOptions,
+  ): void;
+
+  // options: not null (parameter default only)
+  protected static override _addDataFieldShim(
+    data: AnyMutableObject,
     oldKey: string,
     newKey: string,
-    apply?: (data: AnyObject) => unknown,
-  ): unknown;
+    options?: Document.DataFieldShimOptions,
+  ): void;
+
+  protected static override _addDataFieldMigration(
+    data: AnyMutableObject,
+    oldKey: string,
+    newKey: string,
+    apply?: ((data: AnyMutableObject) => unknown) | null,
+  ): boolean;
 
-  protected static _logDataFieldMigration(
+  // options: not null (destructured where forwarded)
+  protected static override _logDataFieldMigration(
     oldKey: string,
     newKey: string,
     options?: LogCompatibilityWarningOptions,
   ): void;
 
-  protected static _onCreateDocuments(
+  /**
+   * @deprecated since v12, will be removed in v14
+   * @remarks "The `Document._onCreateDocuments` static method is deprecated in favor of {@link Document._onCreateOperation | `Document._onCreateOperation`}"
+   */
+  protected static override _onCreateDocuments(
     documents: RegionDocument.Implementation[],
     context: Document.ModificationContext<RegionDocument.Parent>,
   ): Promise<void>;
 
-  protected static _onUpdateDocuments(
+  /**
+   * @deprecated since v12, will be removed in v14
+   * @remarks "The `Document._onUpdateDocuments` static method is deprecated in favor of {@link Document._onUpdateOperation | `Document._onUpdateOperation`}"
+   */
+  protected static override _onUpdateDocuments(
     documents: RegionDocument.Implementation[],
     context: Document.ModificationContext<RegionDocument.Parent>,
   ): Promise<void>;
 
-  protected static _onDeleteDocuments(
+  /**
+   * @deprecated since v12, will be removed in v14
+   * @remarks "The `Document._onDeleteDocuments` static method is deprecated in favor of {@link Document._onDeleteOperation | `Document._onDeleteOperation`}"
+   */
+  protected static override _onDeleteDocuments(
     documents: RegionDocument.Implementation[],
     context: Document.ModificationContext<RegionDocument.Parent>,
   ): Promise<void>;
 
   /* DataModel overrides */
 
-  protected static _schema: SchemaField<RegionDocument.Schema>;
+  protected static override _schema: SchemaField<RegionDocument.Schema>;
 
-  static get schema(): SchemaField<RegionDocument.Schema>;
+  static override get schema(): SchemaField<RegionDocument.Schema>;
 
-  static validateJoint(data: RegionDocument.Source): void;
+  static override validateJoint(data: RegionDocument.Source): void;
 
+  // options: not null (parameter default only, destructured in super)
   static override fromSource(
     source: RegionDocument.CreateData,
-    { strict, ...context }?: DataModel.FromSourceOptions,
+    context?: DataModel.FromSourceOptions,
   ): RegionDocument.Implementation;
 
   static override fromJSON(json: string): RegionDocument.Implementation;
@@ -286,6 +312,7 @@ export default BaseRegion;
 
 declare namespace BaseRegion {
   export import Name = RegionDocument.Name;
+  export import ConstructionContext = RegionDocument.ConstructionContext;
   export import ConstructorArgs = RegionDocument.ConstructorArgs;
   export import Hierarchy = RegionDocument.Hierarchy;
   export import Metadata = RegionDocument.Metadata;
@@ -300,7 +327,6 @@ declare namespace BaseRegion {
   export import Invalid = RegionDocument.Invalid;
   export import Stored = RegionDocument.Stored;
   export import Source = RegionDocument.Source;
-  export import PersistedData = RegionDocument.PersistedData;
   export import CreateData = RegionDocument.CreateData;
   export import InitializedData = RegionDocument.InitializedData;
   export import UpdateData = RegionDocument.UpdateData;
@@ -308,20 +334,12 @@ declare namespace BaseRegion {
   export import DatabaseOperation = RegionDocument.Database;
   export import Flags = RegionDocument.Flags;
 
-  /**
-   * @deprecated This type is used by Foundry too vaguely.
-   * In one context the most correct type is after initialization whereas in another one it should be
-   * before but Foundry uses it interchangeably.
-   */
-  type Properties = SchemaField.InitializedData<Schema>;
-
-  /**
-   * @deprecated {@link foundry.data.fields.SchemaField | `SchemaField<BaseRegionDocument.Schema>`}
-   */
-  type SchemaField = foundry.data.fields.SchemaField<Schema>;
-
-  /**
-   * @deprecated {@link BaseRegion.CreateData | `BaseRegionDocument.CreateData`}
-   */
-  type ConstructorData = BaseRegion.CreateData;
+  namespace Internal {
+    // Note(LukeAbby): The point of this is to give the base class of `RegionDocument` a name.
+    // The expression `CanvasDocumentMixin(BaseRegion)` is more intuitive but it has worse
+    // caching, likely due to the majority of tsc's caching working off of names.
+    // See https://gist.github.com/LukeAbby/18a928fdc35c5d54dc121ed5dbf412fd.
+    interface CanvasDocument extends foundry.documents.abstract.CanvasDocumentMixin.Mix<typeof BaseRegion> {}
+    const CanvasDocument: CanvasDocument;
+  }
 }

package/src/foundry/common/documents/roll-table.d.mts

@@ -1,4 +1,4 @@
-import type { AnyMutableObject, AnyObject } from "fvtt-types/utils";
+import type { AnyMutableObject } from "#utils";
 import type DataModel from "../abstract/data.d.mts";
 import type Document from "../abstract/document.mts";
 import type { DataField, SchemaField } from "../data/fields.d.mts";
@@ -16,26 +16,56 @@ declare abstract class BaseRollTable extends Document<"RollTable", BaseRollTable
    * @param data    - Initial data from which to construct the `BaseRollTable`
    * @param context - Construction context options
    *
-   * @deprecated Constructing `BaseRollTable` directly is not advised. The base document classes exist in
+   * @remarks Constructing `BaseRollTable` directly is not advised. The base document classes exist in
    * order to use documents on both the client (i.e. where all your code runs) and behind the scenes
    * on the server to manage document validation and storage.
    *
    * You should use {@link RollTable.implementation | `new RollTable.implementation(...)`} instead which will give you
    * a system specific implementation of the `RollTable` document.
    */
-  constructor(...args: RollTable.ConstructorArgs);
+  constructor(data: RollTable.CreateData, context?: RollTable.ConstructionContext);
 
+  /**
+   * @defaultValue
+   * ```js
+   * mergeObject(super.metadata, {
+   *   name: "RollTable",
+   *   collection: "tables",
+   *   indexed: true,
+   *   compendiumIndexFields: ["_id", "name", "description", "img", "sort", "folder"],
+   *   embedded: {TableResult: "results"},
+   *   label: "DOCUMENT.RollTable",
+   *   labelPlural: "DOCUMENT.RollTables",
+   *   schemaVersion: "13.341"
+   * })
+   * ```
+   */
   static override metadata: BaseRollTable.Metadata;
 
-  static override defineSchema(): BaseRollTable.Schema;
+  /** @defaultValue `["DOCUMENT", "TABLE"]` */
+  static override LOCALIZATION_PREFIXES: string[];
 
   /**
    * The default icon used for newly created Macro documents
+   * @defaultValue `"icons/svg/d20-grey.svg"`
+   * @remarks "Macro" comes from foundry's comment
    */
-  static DEFAULT_ICON: "icons/svg/d20-grey.svg";
+  static DEFAULT_ICON: string;
+
+  static override defineSchema(): BaseRollTable.Schema;
+
+  protected override _initialize(options?: Document.InitializeOptions): void;
 
+  /**
+   * @remarks
+   * Migrations:
+   * - `flags.core.sourceId` to `_stats.compendiumSource` (since v12, no specified end)
+   */
   static override migrateData(source: AnyMutableObject): AnyMutableObject;
 
+  /** @remarks `source` instead of the parent's `data` here */
+  static override shimData(source: AnyMutableObject, options?: DataModel.ShimDataOptions): AnyMutableObject;
+
   /*
    * After this point these are not really overridden methods.
    * They are here because Foundry's documents are complex and have lots of edge cases.
@@ -48,10 +78,8 @@ declare abstract class BaseRollTable extends Document<"RollTable", BaseRollTable
 
   /* Document overrides */
 
-  static " fvtt_types_internal_document_name_static": "RollTable";
-
   // Same as Document for now
-  protected static override _initializationOrder(): Generator<[string, DataField.Any]>;
+  protected static override _initializationOrder(): Generator<[string, DataField.Any], void, undefined>;
 
   readonly parentCollection: RollTable.ParentCollectionName | null;
 
@@ -73,7 +101,7 @@ declare abstract class BaseRollTable extends Document<"RollTable", BaseRollTable
 
   override parent: RollTable.Parent;
 
-  static createDocuments<Temporary extends boolean | undefined = false>(
+  static createDocuments<Temporary extends boolean | undefined = undefined>(
     data: Array<RollTable.Implementation | RollTable.CreateData> | undefined,
     operation?: Document.Database.CreateOperation<RollTable.Database.Create<Temporary>>,
   ): Promise<Array<Document.TemporaryIf<RollTable.Implementation, Temporary>>>;
@@ -88,7 +116,7 @@ declare abstract class BaseRollTable extends Document<"RollTable", BaseRollTable
     operation?: Document.Database.DeleteDocumentsOperation<RollTable.Database.Delete>,
   ): Promise<RollTable.Implementation[]>;
 
-  static override create<Temporary extends boolean | undefined = false>(
+  static override create<Temporary extends boolean | undefined = undefined>(
     data: RollTable.CreateData | RollTable.CreateData[],
     operation?: RollTable.Database.CreateOperation<Temporary>,
   ): Promise<Document.TemporaryIf<RollTable.Implementation, Temporary> | undefined>;
@@ -100,7 +128,7 @@ declare abstract class BaseRollTable extends Document<"RollTable", BaseRollTable
 
   override delete(operation?: RollTable.Database.DeleteOperation): Promise<this | undefined>;
 
-  static get(documentId: string, options?: RollTable.Database.GetOptions): RollTable.Implementation | null;
+  static override get(documentId: string, options?: RollTable.Database.GetOptions): RollTable.Implementation | null;
 
   static override getCollectionName<CollectionName extends RollTable.Embedded.Name>(
     name: CollectionName,
@@ -123,22 +151,24 @@ declare abstract class BaseRollTable extends Document<"RollTable", BaseRollTable
     // operation?: Document.Database.CreateOperation<Document.Database.CreateForName<EmbeddedName>>,
     // However this causes a number of errors.
     operation?: object,
-  ): Promise<Array<Document.Stored<Document.ImplementationFor<EmbeddedName>>> | undefined>;
+  ): Promise<Array<Document.StoredForName<EmbeddedName>> | undefined>;
 
   override updateEmbeddedDocuments<EmbeddedName extends RollTable.Embedded.Name>(
     embeddedName: EmbeddedName,
     updates: Document.UpdateDataForName<EmbeddedName>[] | undefined,
     operation?: Document.Database.UpdateOperationForName<EmbeddedName>,
-  ): Promise<Array<Document.Stored<Document.ImplementationFor<EmbeddedName>>> | undefined>;
+  ): Promise<Array<Document.StoredForName<EmbeddedName>> | undefined>;
 
   override deleteEmbeddedDocuments<EmbeddedName extends RollTable.Embedded.Name>(
     embeddedName: EmbeddedName,
     ids: Array<string>,
     operation?: Document.Database.DeleteOperationForName<EmbeddedName>,
-  ): Promise<Array<Document.Stored<Document.ImplementationFor<EmbeddedName>>>>;
+  ): Promise<Array<Document.StoredForName<EmbeddedName>>>;
 
   // Same as Document for now
-  override traverseEmbeddedDocuments(_parentPath?: string): Generator<[string, Document.AnyChild<this>]>;
+  override traverseEmbeddedDocuments(
+    _parentPath?: string,
+  ): Generator<[string, Document.AnyChild<this>], void, undefined>;
 
   override getFlag<Scope extends RollTable.Flags.Scope, Key extends RollTable.Flags.Key<Scope>>(
     scope: Scope,
@@ -219,35 +249,60 @@ declare abstract class BaseRollTable extends Document<"RollTable", BaseRollTable
     user: User.Implementation,
   ): Promise<void>;
 
-  static get hasSystemData(): undefined;
-
   // These data field things have been ticketed but will probably go into backlog hell for a while.
   // We'll end up copy and pasting without modification for now I think. It makes it a tiny bit easier to update though.
-  protected static _addDataFieldShims(data: AnyObject, shims: AnyObject, options?: Document.DataFieldShimOptions): void;
 
-  protected static _addDataFieldMigration(
-    data: AnyObject,
+  // options: not null (parameter default only in _addDataFieldShim)
+  protected static override _addDataFieldShims(
+    data: AnyMutableObject,
+    shims: Record<string, string>,
+    options?: Document.DataFieldShimOptions,
+  ): void;
+
+  // options: not null (parameter default only)
+  protected static override _addDataFieldShim(
+    data: AnyMutableObject,
     oldKey: string,
     newKey: string,
-    apply?: (data: AnyObject) => unknown,
-  ): unknown;
+    options?: Document.DataFieldShimOptions,
+  ): void;
 
-  protected static _logDataFieldMigration(
+  protected static override _addDataFieldMigration(
+    data: AnyMutableObject,
+    oldKey: string,
+    newKey: string,
+    apply?: ((data: AnyMutableObject) => unknown) | null,
+  ): boolean;
+
+  // options: not null (destructured where forwarded)
+  protected static override _logDataFieldMigration(
     oldKey: string,
     newKey: string,
     options?: LogCompatibilityWarningOptions,
   ): void;
 
+  /**
+   * @deprecated since v12, will be removed in v14
+   * @remarks "The `Document._onCreateDocuments` static method is deprecated in favor of {@link Document._onCreateOperation | `Document._onCreateOperation`}"
+   */
   protected static _onCreateDocuments(
     documents: RollTable.Implementation[],
     context: Document.ModificationContext<RollTable.Parent>,
   ): Promise<void>;
 
+  /**
+   * @deprecated since v12, will be removed in v14
+   * @remarks "The `Document._onUpdateDocuments` static method is deprecated in favor of {@link Document._onUpdateOperation | `Document._onUpdateOperation`}"
+   */
   protected static _onUpdateDocuments(
     documents: RollTable.Implementation[],
     context: Document.ModificationContext<RollTable.Parent>,
   ): Promise<void>;
 
+  /**
+   * @deprecated since v12, will be removed in v14
+   * @remarks "The `Document._onDeleteDocuments` static method is deprecated in favor of {@link Document._onDeleteOperation | `Document._onDeleteOperation`}"
+   */
   protected static _onDeleteDocuments(
     documents: RollTable.Implementation[],
     context: Document.ModificationContext<RollTable.Parent>,
@@ -259,11 +314,13 @@ declare abstract class BaseRollTable extends Document<"RollTable", BaseRollTable
 
   static get schema(): SchemaField<RollTable.Schema>;
 
+  /** @remarks Not actually overridden, still a no-op, typed for ease of subclassing */
   static validateJoint(data: RollTable.Source): void;
 
+  // options: not null (parameter default only, destructured in super)
   static override fromSource(
     source: RollTable.CreateData,
-    { strict, ...context }?: DataModel.FromSourceOptions,
+    context?: DataModel.FromSourceOptions,
   ): RollTable.Implementation;
 
   static override fromJSON(json: string): RollTable.Implementation;
@@ -273,6 +330,7 @@ export default BaseRollTable;
 
 declare namespace BaseRollTable {
   export import Name = RollTable.Name;
+  export import ConstructionContext = RollTable.ConstructionContext;
   export import ConstructorArgs = RollTable.ConstructorArgs;
   export import Hierarchy = RollTable.Hierarchy;
   export import Metadata = RollTable.Metadata;
@@ -287,7 +345,6 @@ declare namespace BaseRollTable {
   export import Invalid = RollTable.Invalid;
   export import Stored = RollTable.Stored;
   export import Source = RollTable.Source;
-  export import PersistedData = RollTable.PersistedData;
   export import CreateData = RollTable.CreateData;
   export import InitializedData = RollTable.InitializedData;
   export import UpdateData = RollTable.UpdateData;
@@ -295,20 +352,12 @@ declare namespace BaseRollTable {
   export import DatabaseOperation = RollTable.Database;
   export import Flags = RollTable.Flags;
 
-  /**
-   * @deprecated This type is used by Foundry too vaguely.
-   * In one context the most correct type is after initialization whereas in another one it should be
-   * before but Foundry uses it interchangeably.
-   */
-  type Properties = SchemaField.InitializedData<Schema>;
-
-  /**
-   * @deprecated {@link foundry.data.fields.SchemaField | `SchemaField<BaseRollTable.Schema>`}
-   */
-  type SchemaField = foundry.data.fields.SchemaField<Schema>;
-
-  /**
-   * @deprecated {@link BaseRollTable.CreateData | `BaseRollTable.CreateData`}
-   */
-  type ConstructorData = BaseRollTable.CreateData;
+  namespace Internal {
+    // Note(LukeAbby): The point of this is to give the base class of `RollTable` a name.
+    // The expression `ClientDocumentMixin(BaseRollTable)` is more intuitive but it has worse
+    // caching, likely due to the majority of tsc's caching working off of names.
+    // See https://gist.github.com/LukeAbby/18a928fdc35c5d54dc121ed5dbf412fd.
+    interface ClientDocument extends foundry.documents.abstract.ClientDocumentMixin.Mix<typeof BaseRollTable> {}
+    const ClientDocument: ClientDocument;
+  }
 }