Revert check on husky

Author: tolyo

.husky/pre-commit

@@ -1,2 +1,5 @@
-
+make lint
+make check
+make types
+make pretty
 git add ./

@types/core/di/ng-module/ng-module.d.ts

@@ -1,6 +1,4 @@
 /** @private */
-export const INJECTOR_LITERAL: "$injector";
-/** @private */
 export const COMPILE_LITERAL: "$compileProvider";
 /** @private */
 export const ANIMATION_LITERAL: "$animateProvider";
@@ -13,10 +11,6 @@ export const CONTROLLER_LITERAL: "$controllerProvider";
  * controllers, directives, filters, etc. They provide recipes for the injector
  * to do the actual instantiation. A module itself has no behaviour but only state.
  * A such, it acts as a data structure between the Angular instance and the injector service.
- *
- * Since this is an internal structure that is exposed only via the Angular instance,
- * it contains no validation of the items it receives. It is up to the instantiator on
- * modules to do the actual validation.
  */
 export class NgModule {
   /**
@@ -49,7 +43,6 @@ export class NgModule {
   /** @type {!Array.<ng.Injectable<any>>} */
   runBlocks: Array<ng.Injectable<any>>;
   services: any[];
-  wasmModules: any[];
   restDefinitions: any[];
   /**
    * @param {string} name
@@ -123,11 +116,17 @@ export class NgModule {
    */
   filter(name: string, filterFn: ng.Injectable<any>): NgModule;
   /**
-   * @param {string} name
-   * @param {ng.Injectable<any>} ctlFn
+   * The $controller service is used by Angular to create new controllers.
+   * This provider allows controller registration via the register method.
+   *
+   * @param {string} name Controller name
+   * @param {ng.Injectable<ng.ControllerConstructor>} ctlFn Controller constructor fn (optionally decorated with DI annotations in the array notation)
    * @returns {NgModule}
    */
-  controller(name: string, ctlFn: ng.Injectable<any>): NgModule;
+  controller(
+    name: string,
+    ctlFn: ng.Injectable<ng.ControllerConstructor>,
+  ): NgModule;
   /**
    * Register a named WebAssembly module that will be instantiated via $provide.
    *

@types/core/scope/scope.d.ts

@@ -10,10 +10,16 @@ export function nextId(): number;
  */
 export function createScope(target?: any, context?: Scope): Scope;
 /**
+ * @ignore
+ * Checks if a target should be excluded from scope observability
  * @param {any} target
  * @returns {boolean}
  */
-export function isUnsafeGlobal(target: any): boolean;
+export function isNonScope(target: any): boolean;
+/**
+ * Decorator for excluding objects from scope observability
+ */
+export const NONSCOPE: "$nonscope";
 export const $postUpdateQueue: any[];
 export let rootScope: any;
 export class RootScopeProvider {
@@ -26,10 +32,6 @@ export class RootScopeProvider {
       ) => Scope)
   )[];
 }
-/**
- * Decorator for excluding objects from scope observability
- */
-export const NONSCOPE: "$nonscope";
 /**
  * Scope class for the Proxy. It intercepts operations like property access (get)
  * and property setting (set), and adds support for deep change tracking and
@@ -84,8 +86,6 @@ export class Scope {
    */
   $root: Scope;
   $parent: Scope;
-  /** @type {import('./interface.ts').AsyncQueueTask[]} */
-  $$asyncQueue: import("./interface.ts").AsyncQueueTask[];
   filters: any[];
   /** @type {boolean} */
   $$destroyed: boolean;

@types/directive/style/style.d.ts

@@ -1,4 +1,4 @@
 /**
- * @returns {import('../../interface.ts').Directive}
+ * @returns {ng.Directive}
  */
-export function ngStyleDirective(): import("../../interface.ts").Directive;
+export function ngStyleDirective(): ng.Directive;

@types/namespace.d.ts

@@ -40,6 +40,7 @@ import {
   DirectiveFactory as TDirectiveFactory,
   Component as TComponent,
   Controller as TController,
+  ControllerConstructor as TControllerConstructor,
   Injectable as TInjectable,
 } from "./interface.ts";
 import {
@@ -149,6 +150,7 @@ declare global {
     type StorageType = TStorageType;
     type StreamConnectionConfig = TStreamConnectionConfig;
     type CookieOptions = TCookieOptions;
+    type ControllerConstructor = TControllerConstructor;
     type CookieStoreOptions = TCookieStoreOptions;
     type RestService<T, ID> = TRestService<T, ID>;
     type RestDefinition<T> = TRestDefinition<T>;

@types/services/cookie/cookie.d.ts

@@ -34,6 +34,7 @@ export class CookieService {
    *
    * @param {string} key
    * @returns {string|null}
+   * @throws {URIError} – If decodeURIComponent fails.
    */
   get(key: string): string | null;
   /**
@@ -49,6 +50,7 @@ export class CookieService {
    * Returns an object containing all raw cookies.
    *
    * @returns {Record<string, string>}
+   * @throws {URIError} – If decodeURIComponent fails
    */
   getAll(): Record<string, string>;
   /**

@types/services/exception/exception.d.ts

@@ -37,7 +37,7 @@
  * }
  * ```
  *
- * @see {@link ng.ExceptionHandlerService ExceptionHandlerService}
+ * @see {@link ng.ExceptionHandlerService} ExceptionHandlerService
  */
 /**
  * Provider for the `$exceptionHandler` service.

@types/services/http/http.d.ts

@@ -108,6 +108,7 @@ export class HttpProvider {
     | ((
         $injector: ng.InjectorService,
         $sce: any,
+        $cookie: ng.CookieService,
       ) => {
         (requestConfig: any): Promise<any>;
         pendingRequests: any[];

@types/services/pubsub/pubsub.d.ts

@@ -15,6 +15,7 @@ export class PubSubProvider {
   $get: () => PubSub;
 }
 export class PubSub {
+  static $nonscope: any;
   /**
    * Runs a function asynchronously.
    *
@@ -153,7 +154,7 @@ export class PubSub {
    * the order in which they were added, passing all arguments along.
    *
    * If this object was created with async=true, subscribed functions are called
-   * via Promise.resolve().  Otherwise, the functions are called directly, and if
+   * via `queueMicrotask`.  Otherwise, the functions are called directly, and if
    * any of them throw an uncaught error, publishing is aborted.
    *
    * @param {string} topic Topic to publish to.

@types/services/rest/rest.d.ts

@@ -9,18 +9,14 @@ export class RestService<T, ID> {
    *
    * @param {ng.HttpService} $http Angular-like $http service
    * @param {string} baseUrl Base URL or URI template
-   * @param {{new(data: any): T}=} entityClass Optional constructor to map JSON to objects
-   * @param {Object=} options Optional settings (interceptors, headers, etc.)
+   * @param {ng.EntityClass<T>} [entityClass] Optional constructor to map JSON to objects
+   * @param {Object} [options] Optional settings (interceptors, headers, etc.)
    */
   constructor(
     $http: ng.HttpService,
     baseUrl: string,
-    entityClass?:
-      | {
-          new (data: any): T;
-        }
-      | undefined,
-    options?: any | undefined,
+    entityClass?: ng.EntityClass<T>,
+    options?: any,
   );
   /** @private */
   private $http;