Lint:fix

reactiveweb/rollup.config.mjs

@@ -1,6 +1,7 @@
+import { Addon } from "@embroider/addon-dev/rollup";
+
 import { babel } from "@rollup/plugin-babel";
 import copy from "rollup-plugin-copy";
-import { Addon } from "@embroider/addon-dev/rollup";
 
 const addon = new Addon({
   srcDir: "src",

reactiveweb/src/-private/types.ts

@@ -14,9 +14,26 @@ export interface Class<Instance> {
   new (...args: unknown[]): Instance;
 }
 
-// This is a bit of a lie, because we don't have an easy way to
-// represent an object with no properties.
-export type EmptyObject = {};
+// --- Type utilities for component signatures --- //
+// Type-only "symbol" to use with `EmptyObject` below, so that it is *not*
+// equivalent to an empty interface.
+declare const Empty: unique symbol;
+
+/**
+ * This provides us a way to have a "fallback" which represents an empty object,
+ * without the downsides of how TS treats `{}`. Specifically: this will
+ * correctly leverage "excess property checking" so that, given a component
+ * which has no named args, if someone invokes it with any named args, they will
+ * get a type error.
+ *
+ * @internal This is exported so declaration emit works (if it were not emitted,
+ *   declarations which fall back to it would not work). It is *not* intended for
+ *   public usage, and the specific mechanics it uses may change at any time.
+ *   The location of this export *is* part of the public API, because moving it
+ *   will break existing declarations, but is not legal for end users to import
+ *   themselves, so ***DO NOT RELY ON IT***.
+ */
+export type EmptyObject = { [Empty]?: true };
 
 export type GetOrElse<Obj, K, Fallback> = K extends keyof Obj ? Obj[K] : Fallback;
 

reactiveweb/src/ember-concurrency.ts

@@ -1,5 +1,4 @@
 /* eslint-disable @typescript-eslint/no-explicit-any */
-/* eslint-disable @typescript-eslint/ban-types */
 /* eslint-disable ember/no-get */
 import { assert } from '@ember/debug';
 import { associateDestroyableChild, registerDestructor } from '@ember/destroyable';

reactiveweb/src/link.ts

@@ -126,7 +126,7 @@ function linkDecoratorFactory(child: Class<unknown>) {
 
 function linkDecorator(
   _prototype: object,
-  key: string | Symbol,
+  key: string | symbol,
   descriptor: Stage1DecoratorDescriptor | undefined,
   explicitChild?: Class<unknown>
 ): void {

reactiveweb/src/resource/modifier/index.ts

@@ -200,6 +200,7 @@ export function modifier<S extends { Element?: Element; Args?: object }>(
 export function modifier(fn: (element: Element, ...args: unknown[]) => void): ModifierLike<{
   Element: Element;
   Args: {
+    // eslint-disable-next-line @typescript-eslint/no-empty-object-type
     Named: {};
     Positional: [];
   };
@@ -211,6 +212,7 @@ export function modifier(fn: (element: Element, ...args: unknown[]) => void): Mo
   return fn as unknown as ModifierLike<{
     Element: Element;
     Args: {
+      // eslint-disable-next-line @typescript-eslint/no-empty-object-type
       Named: {};
       Positional: [];
     };

reactiveweb/src/utils.ts

@@ -1,4 +1,3 @@
-/* eslint-disable @typescript-eslint/ban-types */
 import type { ArgsWrapper, Thunk } from 'ember-resources';
 
 export const DEFAULT_THUNK = () => [];

reactiveweb/unpublished-development-types/index.d.ts

@@ -4,12 +4,3 @@
 import 'ember-source/types';
 import 'ember-source/types/preview';
 import '@glint/environment-ember-loose';
-
-declare module '@glint/environment-ember-loose/registry' {
-  // Remove this once entries have been added! 👇
-  // eslint-disable-next-line @typescript-eslint/no-empty-interface
-  export default interface Registry {
-    // Add any registry entries from other addons here that your addon itself uses (in non-strict mode templates)
-    // See https://typed-ember.gitbook.io/glint/using-glint/ember/using-addons
-  }
-}

tests/test-app/tests/utils/ember-concurrency/js-test.ts

@@ -1,3 +1,4 @@
+/* eslint-disable @typescript-eslint/no-unused-expressions */
 /* eslint-disable @typescript-eslint/no-explicit-any */
 import { tracked } from '@glimmer/tracking';
 import { settled } from '@ember/test-helpers';

tests/test-app/tests/utils/function/js-test.ts

@@ -1,3 +1,4 @@
+/* eslint-disable @typescript-eslint/no-unused-expressions */
 import { tracked } from '@glimmer/tracking';
 import { destroy, isDestroyed, isDestroying } from '@ember/destroyable';
 import { settled } from '@ember/test-helpers';

tests/test-app/types/global.d.ts

@@ -3,6 +3,7 @@ import '@glint/environment-ember-loose';
 import type PageTitle from 'ember-page-title/template-registry';
 
 declare module '@glint/environment-ember-loose/registry' {
+  // eslint-disable-next-line @typescript-eslint/no-empty-object-type
   export default interface Registry extends PageTitle {
     /* your local loose-mode entries here */
   }