@league-of-foundry-developers/foundry-vtt-types 12.331.4-beta → 12.331.5

package/src/foundry/client/data/documents/actor-delta.d.mts

@@ -1,9 +1,9 @@
+import type { AnyObject, Identity, Merge, NullishProps, RequiredProps } from "#utils";
 import type { DataSchema } from "../../../common/data/fields.d.mts";
 import type { BaseActorDelta } from "../../../common/documents/_module.d.mts";
 import type Document from "../../../common/abstract/document.d.mts";
 import type { fields } from "../../../common/data/module.d.mts";
 import type { ConfiguredActorDelta } from "../../../../configuration/index.d.mts";
-import type { Merge } from "../../../../utils/index.d.mts";
 import type DataModel from "../../../common/abstract/data.d.mts";
 
 declare global {
@@ -15,8 +15,16 @@ declare global {
 
     /**
      * The arguments to construct the document.
-     */
-    type ConstructorArgs = Document.ConstructorParameters<CreateData, Parent>;
+     * @privateRemarks This is off-template, as ActorDelta throws if not provided a valid TokenDocument
+     * parent in the construction context for any construction, not just `.create`ion
+     */
+    interface ConstructorArgs
+      extends Identity<
+        [
+          data: CreateData | undefined,
+          context: RequiredProps<Document.ConstructionContext<TokenDocument.Implementation>, "parent">,
+        ]
+      > {}
 
     /**
      * The documents embedded within `ActorDelta`.
@@ -211,7 +219,7 @@ declare global {
     type Collection = never;
 
     /**
-     * An instance of `ActorDelta` that comes from the database but failed validation meaining that
+     * An instance of `ActorDelta` that comes from the database but failed validation meaning that
      * its `system` and `_source` could theoretically be anything.
      */
     interface Invalid extends Document.Invalid<ActorDelta.Implementation> {}
@@ -280,16 +288,19 @@ declare global {
 
       /**
        * The name override, if any.
+       * @defaultValue `null`
        */
       name: fields.StringField<{ required: false; nullable: true; initial: null }>;
 
       /**
        * The type override, if any.
+       * @defaultValue `null`
        */
       type: fields.StringField<{ required: false; nullable: true; initial: null }>;
 
       /**
        * The image override, if any.
+       * @defaultValue `null`
        */
       img: fields.FilePathField<{ categories: ["IMAGE"]; nullable: true; initial: null; required: false }>;
 
@@ -313,6 +324,7 @@ declare global {
 
       /**
        * Ownership overrides.
+       * @defaultValue `null`
        */
       ownership: fields.DocumentOwnershipField<{ required: false; nullable: true; initial: null }>;
 
@@ -520,6 +532,17 @@ declare global {
      * @deprecated {@link ActorDelta.Implementation | `ActorDelta.Implementation`}
      */
     type ConfiguredInstance = Implementation;
+
+    /** @internal */
+    type _InitializeOptions = NullishProps<{
+      /**
+       * @remarks Is this initialization part of a {@link Scene.reset | `Scene#reset`} call? (skips further initialization if truthy)
+       * @defaultValue `false`
+       */
+      sceneReset: boolean;
+    }>;
+
+    interface InitializeOptions extends Document.InitializeOptions, _InitializeOptions {}
   }
 
   /**
@@ -535,9 +558,11 @@ declare global {
      */
     constructor(...args: ActorDelta.ConstructorArgs);
 
-    protected override _configure(options?: { pack?: string | null }): void;
+    // options: not null (parameter default only, destructured in super)
+    protected override _configure(options?: Document.ConfigureOptions): void;
 
-    protected override _initialize(options?: any): void;
+    // options: not null (destructured)
+    protected override _initialize(options?: ActorDelta.InitializeOptions): void;
 
     /** Pass-through the type from the synthetic Actor, if it exists. */
     _type: string;
@@ -545,17 +570,20 @@ declare global {
     /**
      * Apply this ActorDelta to the base Actor and return a synthetic Actor.
      * @param context - Context to supply to synthetic Actor instantiation.
+     * @remarks Forwards `context` to {@link BaseActorDelta.applyDelta | `this.constructor.applyDelta(this, this.parent.baseActor, context)`}
      */
-    apply(context: unknown): Actor.Implementation;
+    apply(context?: BaseActorDelta.ApplyDeltaContext | null): Actor.Implementation | null;
 
     /** @remarks `"The synthetic actor prepares its items in the appropriate context of an actor. The actor delta does not need to prepare its items, and would do so in the incorrect context."` */
     override prepareEmbeddedDocuments(): void;
 
+    // TODO: accurately type changes and return type
+    // changes, options: not null (parameter default only)
     override updateSource(
       // Note(LukeAbby): This must be valid for both `new ActorDelta.implementation(actorChanges, { parent: this.parent });` and `super.updateSource`.
       // However it's likely the overlap between these two types is pretty high.
       changes?: ActorDelta.Source,
-      options?: DataModel.UpdateSourceOptions,
+      options?: DataModel.UpdateOptions,
     ): object;
 
     override reset(): void;
@@ -589,9 +617,7 @@ declare global {
      */
     _handleDeltaCollectionUpdates(doc: Document.Any): void;
 
-    /**
-     * @privateRemarks _onUpdate and _onDelete are all overridden but with no signature changes from BaseActorDelta.
-     */
+    // _onUpdate and _onDelete are all overridden but with no signature changes from BaseActorDelta.
 
     protected override _dispatchDescendantDocumentEvents(
       event: ClientDocument.LifeCycleEventName,
@@ -614,7 +640,7 @@ declare global {
 
     /**
      * @remarks To make it possible for narrowing one parameter to jointly narrow other parameters
-     * this method must be overriden like so:
+     * this method must be overridden like so:
      * ```typescript
      * class SwadeActorDelta extends ActorDelta {
      *   protected override _preCreateDescendantDocuments(...args: ActorDelta.PreCreateDescendantDocumentsArgs) {
@@ -632,7 +658,7 @@ declare global {
 
     /**
      * @remarks To make it possible for narrowing one parameter to jointly narrow other parameters
-     * this method must be overriden like so:
+     * this method must be overridden like so:
      * ```typescript
      * class GurpsActorDelta extends ActorDelta {
      *   protected override _onCreateDescendantDocuments(...args: ActorDelta.OnCreateDescendantDocumentsArgs) {
@@ -650,7 +676,7 @@ declare global {
 
     /**
      * @remarks To make it possible for narrowing one parameter to jointly narrow other parameters
-     * this method must be overriden like so:
+     * this method must be overridden like so:
      * ```typescript
      * class LancerActorDelta extends ActorDelta {
      *   protected override _preUpdateDescendantDocuments(...args: ActorDelta.OnUpdateDescendantDocuments) {
@@ -668,7 +694,7 @@ declare global {
 
     /**
      * @remarks To make it possible for narrowing one parameter to jointly narrow other parameters
-     * this method must be overriden like so:
+     * this method must be overridden like so:
      * ```typescript
      * class Ptr2eActorDelta extends ActorDelta {
      *   protected override _onUpdateDescendantDocuments(...args: ActorDelta.OnUpdateDescendantDocumentsArgs) {
@@ -686,7 +712,7 @@ declare global {
 
     /**
      * @remarks To make it possible for narrowing one parameter to jointly narrow other parameters
-     * this method must be overriden like so:
+     * this method must be overridden like so:
      * ```typescript
      * class KultActorDelta extends ActorDelta {
      *   protected override _preDeleteDescendantDocuments(...args: ActorDelta.PreDeleteDescendantDocumentsArgs) {
@@ -704,7 +730,7 @@ declare global {
 
     /**
      * @remarks To make it possible for narrowing one parameter to jointly narrow other parameters
-     * this method must be overriden like so:
+     * this method must be overridden like so:
      * ```typescript
      * class BladesActorDelta extends ActorDelta {
      *   protected override _onDeleteDescendantDocuments(...args: ActorDelta.OnUpdateDescendantDocuments) {
@@ -720,23 +746,28 @@ declare global {
      */
     protected override _onDeleteDescendantDocuments(...args: ActorDelta.OnDeleteDescendantDocumentsArgs): void;
 
+    // context: not null (destructured)
     static override defaultName(
-      context: Document.DefaultNameContext<ActorDelta.SubType, NonNullable<ActorDelta.Parent>>,
+      context?: Document.DefaultNameContext<"ActorDelta", NonNullable<ActorDelta.Parent>>,
     ): string;
 
+    /** @remarks `context.parent` is required as creation requires one */
     static override createDialog(
-      data: Document.CreateDialogData<ActorDelta.CreateData>,
-      context: Document.CreateDialogContext<ActorDelta.SubType, NonNullable<ActorDelta.Parent>>,
+      data: Document.CreateDialogData<ActorDelta.CreateData> | undefined,
+      context: Document.CreateDialogContext<"ActorDelta", NonNullable<ActorDelta.Parent>>,
     ): Promise<ActorDelta.Stored | null | undefined>;
 
+    // options: not null (parameter default only)
     static override fromDropData(
       data: Document.DropData<ActorDelta.Implementation>,
-      options?: Document.FromDropDataOptions,
+      options?: AnyObject,
     ): Promise<ActorDelta.Implementation | undefined>;
 
     static override fromImport(
       source: ActorDelta.Source,
-      context?: Document.FromImportContext<ActorDelta.Parent>,
+      context?: Document.FromImportContext<ActorDelta.Parent> | null,
     ): Promise<ActorDelta.Implementation>;
+
+    override _onClickDocumentLink(event: MouseEvent): ClientDocument.OnClickDocumentLinkReturn;
   }
 }

package/src/foundry/client/data/documents/actor.d.mts

@@ -1,7 +1,6 @@
-import type { AnyObject, InexactPartial, NullishProps, Merge } from "fvtt-types/utils";
+import type { AnyObject, InexactPartial, NullishProps, Merge } from "#utils";
 import type { documents } from "../../../client-esm/client.d.mts";
 import type Document from "../../../common/abstract/document.d.mts";
-import type EmbeddedCollection from "../../../common/abstract/embedded-collection.d.mts";
 import type BaseActor from "../../../common/documents/actor.d.mts";
 import type { fields, PrototypeToken } from "../../../common/data/module.d.mts";
 import type { ConfiguredActor } from "../../../../configuration/index.d.mts";
@@ -227,7 +226,7 @@ declare global {
     type Collection = Actors.Configured;
 
     /**
-     * An instance of `Actor` that comes from the database but failed validation meaining that
+     * An instance of `Actor` that comes from the database but failed validation meaning that
      * its `system` and `_source` could theoretically be anything.
      */
     interface Invalid<out SubType extends Actor.SubType = Actor.SubType> extends Document.Invalid<OfType<SubType>> {}
@@ -296,16 +295,18 @@ declare global {
       _id: fields.DocumentIdField;
 
       /** The name of this Actor */
-      name: fields.StringField<{ required: true; blank: false; textSearch: true }>;
+      // FIXME: required with no initial, assignment type override required
+      name: fields.StringField<{ required: true; blank: false; textSearch: true }, string>;
 
       /** An Actor subtype which configures the system data model applied */
+      // TODO: required with no initial, needs assignment type override
       type: fields.DocumentTypeField<typeof BaseActor>;
 
       /**
        * An image file path which provides the artwork for this Actor
-       * @defaultValue `null`
+       * @defaultValue `Actor.DEFAULT_ICON`
        */
-      img: fields.FilePathField<{ categories: "IMAGE"[]; initial: (data: unknown) => string }>;
+      img: fields.FilePathField<{ categories: ["IMAGE"]; initial: (data: unknown) => string }>;
 
       /**
        * The system data object which is defined by the system template.json model
@@ -315,7 +316,7 @@ declare global {
 
       /**
        * Default Token settings which are used for Tokens created from this Actor
-       * @defaultValue see {@link PrototypeToken | `PrototypeToken`}
+       * @defaultValue see {@linkcode PrototypeToken.Schema}
        */
       prototypeToken: fields.EmbeddedDataField<typeof PrototypeToken>;
 
@@ -508,55 +509,47 @@ declare global {
       | Document.OnDeleteDescendantDocumentsArgs<Actor.Stored, Actor.DirectDescendant, Actor.Metadata.Embedded>
       | Item.OnDeleteDescendantDocumentsArgs;
 
-    /**
-     * @deprecated {@link Actor.Database | `Actor.DatabaseOperation`}
-     */
-    // eslint-disable-next-line @typescript-eslint/no-deprecated
-    interface DatabaseOperations extends Document.Database.Operations<Actor.Implementation> {}
-
-    /**
-     * @deprecated {@link Actor.SubType | `Actor.SubType`}
-     */
-    type TypeNames = Actor.SubType;
-
-    /**
-     * @deprecated {@link Actor.CreateData | `Actor.CreateData`}
-     */
-    interface ConstructorData extends Actor.CreateData {}
-
-    /**
-     * @deprecated {@link Actor.implementation | `Actor.ImplementationClass`}
-     */
-    type ConfiguredClass = ImplementationClass;
+    interface GetDefaultArtworkReturn {
+      img: string;
+      texture: GetDefaultArtworkTextureReturn;
+    }
 
-    /**
-     * @deprecated {@link Actor.Implementation | `Actor.Implementation`}
-     */
-    type ConfiguredInstance = Implementation;
+    interface GetDefaultArtworkTextureReturn {
+      src: string;
+    }
 
     type ItemTypes = {
-      [SubType in Game.Model.TypeNames<"Item">]: Array<Item.OfType<SubType>>;
+      [SubType in Item.SubType]: Array<Item.OfType<SubType>>;
     };
 
-    interface RollInitiativeOptions {
+    type GetActiveTokensReturn<Document extends boolean | null | undefined> = Document extends true
+      ? TokenDocument.Implementation[]
+      : Token.Object[];
+
+    /** @internal */
+    type _RollInitiativeOptions = NullishProps<{
       /**
        * Create new Combatant entries for Tokens associated with this actor.
        * @defaultValue `false`
        */
-      createCombatants?: boolean;
+      createCombatants: boolean;
 
       /**
        * Re-roll the initiative for this Actor if it has already been rolled.
        * @defaultValue `false`
        */
-      rerollInitiative?: boolean;
+      rerollInitiative: boolean;
+    }> &
+      InexactPartial<{
+        /**
+         * Additional options passed to the Combat#rollInitiative method.
+         * @defaultValue `{}`
+         * @remarks Can't be `null` as it only has a parameter default
+         */
+        initiativeOptions: Combat.InitiativeOptions;
+      }>;
 
-      /**
-       * Additional options passed to the Combat#rollInitiative method.
-       * @defaultValue `{}`
-       */
-      initiativeOptions?: Combat.InitiativeOptions;
-    }
+    interface RollInitiativeOptions extends _RollInitiativeOptions {}
 
     /** @internal */
     type _ToggleStatusEffectOptions = NullishProps<{
@@ -574,7 +567,63 @@ declare global {
        */
       overlay: boolean;
     }>;
+
     interface ToggleStatusEffectOptions extends _ToggleStatusEffectOptions {}
+
+    /** @internal */
+    type _RequestTokenImagesOptions = NullishProps<{
+      /**
+       * The name of the compendium the actor is in.
+       * @defaultValue `null`
+       * @remarks The default comes from the `"requestTokenImages"` socket handler in `dist/database/documents/actor.mjs` where it's the parameter default
+       */
+      pack: string;
+    }>;
+
+    interface RequestTokenImagesOptions extends _RequestTokenImagesOptions {}
+
+    /** @internal */
+    type _GetDependentTokensOptions = NullishProps<{
+      /**
+       * A single Scene, or list of Scenes to filter by.
+       * @defaultValue `Array.from(this._dependentTokens.keys())`
+       */
+      scenes: Scene.Implementation | Scene.Implementation[];
+
+      /**
+       * Limit the results to tokens that are linked to the actor.
+       * @defaultValue `false`
+       */
+      linked: boolean;
+    }>;
+
+    interface GetDependentTokensOptions extends _GetDependentTokensOptions {}
+
+    /**
+     * @deprecated {@link Actor.Database | `Actor.DatabaseOperation`}
+     */
+    // eslint-disable-next-line @typescript-eslint/no-deprecated
+    interface DatabaseOperations extends Document.Database.Operations<Actor.Implementation> {}
+
+    /**
+     * @deprecated {@link Actor.SubType | `Actor.SubType`}
+     */
+    type TypeNames = Actor.SubType;
+
+    /**
+     * @deprecated {@link Actor.CreateData | `Actor.CreateData`}
+     */
+    interface ConstructorData extends Actor.CreateData {}
+
+    /**
+     * @deprecated {@link Actor.implementation | `Actor.ImplementationClass`}
+     */
+    type ConfiguredClass = ImplementationClass;
+
+    /**
+     * @deprecated {@link Actor.Implementation | `Actor.Implementation`}
+     */
+    type ConfiguredInstance = Implementation;
   }
 
   /**
@@ -609,18 +658,23 @@ declare global {
      */
     constructor(...args: Actor.ConstructorArgs);
 
-    static override metadata: Actor.Metadata;
-
-    protected override _configure(options?: { pack?: string | null }): void;
+    // options: not null (parameter default only, destructured in super)
+    protected override _configure(options?: Document.ConfigureOptions): void;
 
     /**
      * Maintain a list of Token Documents that represent this Actor, stored by Scene.
+     * @remarks Doesn't exist prior to being `defineProperty`'d in {@link Actor._configure | `Actor#_configure`}
      */
-    protected _dependentTokens: foundry.utils.IterableWeakMap<Scene.Implementation, TokenDocument.Implementation>;
+    protected _dependentTokens?: foundry.utils.IterableWeakMap<
+      Scene.Implementation,
+      foundry.utils.IterableWeakSet<TokenDocument.Implementation>
+    >;
 
+    /** @remarks `||=`s the `prototypeToken`'s `name` and `texture.src` fields with the main actor's values */
+    // options: not null (parameter default only)
     protected override _initializeSource(
       data: this | Actor.CreateData,
-      options?: Omit<foundry.abstract.DataModel.DataValidationOptions, "parent">,
+      options?: Document.InitializeSourceOptions,
     ): Actor.Source;
 
     /**
@@ -638,6 +692,7 @@ declare global {
      * A cached array of image paths which can be used for this Actor's token.
      * Null if the list has not yet been populated.
      * @defaultValue `null`
+     * @private
      */
     protected _tokenImages: string[] | null;
 
@@ -650,7 +705,7 @@ declare global {
     /**
      * Provide a thumbnail image path used to represent this document.
      */
-    get thumbnail(): this["img"];
+    get thumbnail(): string;
 
     /**
      * Provide an object which organizes all embedded Item instances by their type
@@ -670,7 +725,7 @@ declare global {
     /**
      * An array of ActiveEffect instances which are present on the Actor which have a limited duration.
      */
-    get temporaryEffects(): EmbeddedCollection<ActiveEffect.Implementation, Actor.Implementation>;
+    get temporaryEffects(): ActiveEffect.Implementation[];
 
     /**
      * Return a reference to the TokenDocument which owns this Actor as a synthetic override
@@ -694,13 +749,14 @@ declare global {
      *
      * @param linked   - Limit results to Tokens which are linked to the Actor. Otherwise return all
      *                   Tokens even those which are not linked. (default: `false`)
-     * @param document - Return the Document instance rather than the PlaceableObject (default: `false`)
+     * @param document - Return the Document instance rather than the PlaceableObject
+     *                   (default: `false`)
      * @returns An array of Token instances in the current Scene which reference this Actor.
      */
-    getActiveTokens<ReturnDocument extends boolean | undefined = undefined>(
-      linked?: boolean,
+    getActiveTokens<ReturnDocument extends boolean | null | undefined = false>(
+      linked?: boolean | null,
       document?: ReturnDocument,
-    ): ReturnDocument extends true ? TokenDocument.Implementation[] : Token.Object[];
+    ): Actor.GetActiveTokensReturn<ReturnDocument>;
 
     /**
      * Get all ActiveEffects that may apply to this Actor.
@@ -721,7 +777,11 @@ declare global {
      * @param data - Additional data, such as x, y, rotation, etc. for the created token data (default: `{}`)
      * @returns The created TokenData instance
      */
-    getTokenDocument(data?: TokenDocument.CreateData): Promise<TokenDocument.Implementation>;
+    // data, options: not null (parameter defaults only)
+    getTokenDocument(
+      data?: TokenDocument.CreateData,
+      options?: Document.ConstructionContext<TokenDocument.Parent>,
+    ): Promise<TokenDocument.Implementation>;
 
     /**
      * Get an Array of Token images which could represent this Actor
@@ -737,7 +797,13 @@ declare global {
      * @param isBar     - Whether the new value is part of an attribute bar, or just a direct value (default: `true`)
      * @returns The updated Actor document
      */
-    modifyTokenAttribute(attribute: string, value: number, isDelta: boolean, isBar: boolean): Promise<this | undefined>;
+    modifyTokenAttribute(
+      attribute: string,
+      // TODO: tighten Combatant.Resource with the justification of this being simply `number`
+      value: number,
+      isDelta?: boolean,
+      isBar?: boolean,
+    ): Promise<this | undefined>;
 
     override prepareData(): void;
 
@@ -751,6 +817,7 @@ declare global {
      * @param options - Configuration for how initiative for this Actor is rolled.
      * @returns A promise which resolves to the Combat document once rolls are complete.
      */
+    // options: not null (destructured)
     rollInitiative(options?: Actor.RollInitiativeOptions): Promise<void>;
 
     /**
@@ -772,59 +839,44 @@ declare global {
     /**
      * Request wildcard token images from the server and return them.
      * @param actorId - The actor whose prototype token contains the wildcard image path.
-     * @internal
+     * @private
      */
-    protected static _requestTokenImages(
-      actorId: string,
-      options?: {
-        /** The name of the compendium the actor is in. */
-        pack: string;
-      },
-    ): Promise<string[]>;
+    // options: not null (parameter default only)
+    protected static _requestTokenImages(actorId: string, options?: Actor.RequestTokenImagesOptions): Promise<string[]>;
 
     /**
      * Get this actor's dependent tokens.
      * If the actor is a synthetic token actor, only the exact Token which it represents will be returned.
      */
-    getDependentTokens(
-      options?: InexactPartial<{
-        /**
-         * A single Scene, or list of Scenes to filter by.
-         */
-        scenes: Scene.Implementation | Scene.Implementation[];
-        /**
-         * Limit the results to tokens that are linked to the actor.
-         * @defaultValue `false`
-         */
-        linked: boolean;
-      }>,
-    ): TokenDocument.Implementation[];
+    // options: not null (destructured)
+    getDependentTokens(options?: Actor.GetDependentTokensOptions): TokenDocument.Implementation[];
 
     /**
      * Register a token as a dependent of this actor.
      * @param token - The Token
+     * @internal
      */
     protected _registerDependantToken(token: TokenDocument.Implementation): void;
 
     /**
      * Remove a token from this actor's dependents.
      * @param token - The Token
+     * @internal
      */
     protected _unregisterDependentToken(token: TokenDocument.Implementation): void;
 
     /**
      * Prune a whole scene from this actor's dependent tokens.
      * @param scene - The scene
+     * @internal
      */
     protected _unregisterDependentScene(scene: Scene.Implementation): void;
 
-    /**
-     * @privateRemarks _preCreate and _onUpdate are all overridden but with no signature changes from BaseActor.
-     */
+    // _onUpdate is overridden but with no signature changes from BaseActor.
 
     /**
      * @remarks To make it possible for narrowing one parameter to jointly narrow other parameters
-     * this method must be overriden like so:
+     * this method must be overridden like so:
      * ```typescript
      * class GurpsActor extends Actor {
      *   protected override _onCreateDescendantDocuments(...args: Actor.OnCreateDescendantDocumentsArgs) {
@@ -842,7 +894,7 @@ declare global {
 
     /**
      * @remarks To make it possible for narrowing one parameter to jointly narrow other parameters
-     * this method must be overriden like so:
+     * this method must be overridden like so:
      * ```typescript
      * class Ptr2eActor extends Actor {
      *   protected override _onUpdateDescendantDocuments(...args: Actor.OnUpdateDescendantDocumentsArgs) {
@@ -860,7 +912,7 @@ declare global {
 
     /**
      * @remarks To make it possible for narrowing one parameter to jointly narrow other parameters
-     * this method must be overriden like so:
+     * this method must be overridden like so:
      * ```typescript
      * class BladesActor extends Actor {
      *   protected override _onDeleteDescendantDocuments(...args: Actor.OnUpdateDescendantDocuments) {
@@ -878,7 +930,6 @@ declare global {
 
     /**
      * Additional workflows to perform when any descendant document within this Actor changes.
-     * @internal
      */
     protected _onEmbeddedDocumentChange(): void;
 
@@ -886,11 +937,9 @@ declare global {
      * Update the active TokenDocument instances which represent this Actor.
      * @param update  - The update delta.
      * @param options - The update context.
+     * @remarks Forwards to {@link Token._onUpdateBaseActor | `Token#_onUpdateBaseActor`}
      */
-    protected _updateDependentTokens(
-      update: TokenDocument.UpdateData,
-      options: TokenDocument.Database.UpdateOperation,
-    ): void;
+    protected _updateDependentTokens(update: Actor.UpdateData, options: Actor.Database.UpdateOperation): void;
 
     /*
      * After this point these are not really overridden methods.
@@ -906,7 +955,7 @@ declare global {
 
     /**
      * @remarks To make it possible for narrowing one parameter to jointly narrow other parameters
-     * this method must be overriden like so:
+     * this method must be overridden like so:
      * ```typescript
      * class SwadeActor extends Actor {
      *   protected override _preCreateDescendantDocuments(...args: Actor.PreCreateDescendantDocumentsArgs) {
@@ -924,7 +973,7 @@ declare global {
 
     /**
      * @remarks To make it possible for narrowing one parameter to jointly narrow other parameters
-     * this method must be overriden like so:
+     * this method must be overridden like so:
      * ```typescript
      * class LancerActor extends Actor {
      *   protected override _preUpdateDescendantDocuments(...args: Actor.OnUpdateDescendantDocuments) {
@@ -942,7 +991,7 @@ declare global {
 
     /**
      * @remarks To make it possible for narrowing one parameter to jointly narrow other parameters
-     * this method must be overriden like so:
+     * this method must be overridden like so:
      * ```typescript
      * class KultActor extends Actor {
      *   protected override _preDeleteDescendantDocuments(...args: Actor.PreDeleteDescendantDocumentsArgs) {
@@ -958,21 +1007,26 @@ declare global {
      */
     protected override _preDeleteDescendantDocuments(...args: Actor.PreDeleteDescendantDocumentsArgs): void;
 
-    static override defaultName(context?: Document.DefaultNameContext<Actor.SubType, Actor.Parent>): string;
+    // context: not null (destructured)
+    static override defaultName(context?: Document.DefaultNameContext<"Actor", Actor.Parent>): string;
 
+    // data: not null (parameter default only), context: not null (destructured)
     static override createDialog(
       data?: Document.CreateDialogData<Actor.CreateData>,
-      context?: Document.CreateDialogContext<Actor.SubType, Actor.Parent>,
+      context?: Document.CreateDialogContext<"Actor", Actor.Parent>,
     ): Promise<Actor.Stored | null | undefined>;
 
+    // options: not null (parameter default only)
     static override fromDropData(
       data: Document.DropData<Actor.Implementation>,
-      options?: Document.FromDropDataOptions,
+      options?: AnyObject,
     ): Promise<Actor.Implementation | undefined>;
 
     static override fromImport(
       source: Actor.Source,
-      context?: Document.FromImportContext<Actor.Parent>,
+      context?: Document.FromImportContext<Actor.Parent> | null,
     ): Promise<Actor.Implementation>;
+
+    override _onClickDocumentLink(event: MouseEvent): ClientDocument.OnClickDocumentLinkReturn;
   }
 }