Changes before error encountered

Copilot · rubensworks · web-flow · commit b5a7597991ec · 2026-04-03T07:43:01.000Z
Agent-Logs-Url: https://github.com/LinkedSoftwareDependencies/Components.js/sessions/08ee73ad-0075-4ae9-8230-bd263447ce6d Co-authored-by: rubensworks <440384+rubensworks@users.noreply.github.com>
diff --git a/eslint.config.js b/eslint.config.js
@@ -33,6 +33,8 @@ module.exports = config([
       'ts/no-unsafe-assignment': 'off',
       'ts/no-unsafe-argument': 'off',
       'ts/no-unsafe-return': 'off',
+      // Don't flag unused function parameters (common in interface implementations)
+      'unused-imports/no-unused-vars': [ 'error', { args: 'none' }],
     },
   },
   {
diff --git a/lib/construction/argument/ArgumentConstructorHandlerArray.ts b/lib/construction/argument/ArgumentConstructorHandlerArray.ts
@@ -10,8 +10,8 @@ import type { IArgumentsConstructor } from './IArgumentsConstructor';
 export class ArgumentConstructorHandlerArray implements IArgumentConstructorHandler {
   public canHandle<TInstance>(
     value: Resource,
-    _settings: IConstructionSettings,
-    _argsCreator: IArgumentsConstructor<TInstance>,
+    settings: IConstructionSettings,
+    argsCreator: IArgumentsConstructor<TInstance>,
   ): boolean {
     return Boolean(value.property.elements);
   }
diff --git a/lib/construction/argument/ArgumentConstructorHandlerHash.ts b/lib/construction/argument/ArgumentConstructorHandlerHash.ts
@@ -10,8 +10,8 @@ import type { IArgumentsConstructor } from './IArgumentsConstructor';
 export class ArgumentConstructorHandlerHash implements IArgumentConstructorHandler {
   public canHandle<TInstance>(
     value: Resource,
-    _settings: IConstructionSettings,
-    _argsCreator: IArgumentsConstructor<TInstance>,
+    settings: IConstructionSettings,
+    argsCreator: IArgumentsConstructor<TInstance>,
   ): boolean {
     return Boolean(value.property.fields);
   }
diff --git a/lib/construction/argument/ArgumentConstructorHandlerList.ts b/lib/construction/argument/ArgumentConstructorHandlerList.ts
@@ -9,8 +9,8 @@ import type { IArgumentsConstructor } from './IArgumentsConstructor';
 export class ArgumentConstructorHandlerList implements IArgumentConstructorHandler {
   public canHandle<TInstance>(
     value: Resource,
-    _settings: IConstructionSettings,
-    _argsCreator: IArgumentsConstructor<TInstance>,
+    settings: IConstructionSettings,
+    argsCreator: IArgumentsConstructor<TInstance>,
   ): boolean {
     return Boolean(value.list);
   }
diff --git a/lib/construction/argument/ArgumentConstructorHandlerPrimitive.ts b/lib/construction/argument/ArgumentConstructorHandlerPrimitive.ts
@@ -9,8 +9,8 @@ import type { IArgumentsConstructor } from './IArgumentsConstructor';
 export class ArgumentConstructorHandlerPrimitive implements IArgumentConstructorHandler {
   public canHandle<TInstance>(
     value: Resource,
-    _settings: IConstructionSettings,
-    _argsCreator: IArgumentsConstructor<TInstance>,
+    settings: IConstructionSettings,
+    argsCreator: IArgumentsConstructor<TInstance>,
   ): boolean {
     return Boolean(value.type === 'Literal');
   }
diff --git a/lib/construction/argument/ArgumentConstructorHandlerReference.ts b/lib/construction/argument/ArgumentConstructorHandlerReference.ts
@@ -9,8 +9,8 @@ import type { IArgumentsConstructor } from './IArgumentsConstructor';
 export class ArgumentConstructorHandlerReference implements IArgumentConstructorHandler {
   public canHandle<TInstance>(
     value: Resource,
-    _settings: IConstructionSettings,
-    _argsCreator: IArgumentsConstructor<TInstance>,
+    settings: IConstructionSettings,
+    argsCreator: IArgumentsConstructor<TInstance>,
   ): boolean {
     return Boolean(value.type === 'NamedNode' || value.type === 'BlankNode');
   }
diff --git a/lib/construction/argument/ArgumentConstructorHandlerUndefined.ts b/lib/construction/argument/ArgumentConstructorHandlerUndefined.ts
@@ -9,8 +9,8 @@ import type { IArgumentsConstructor } from './IArgumentsConstructor';
 export class ArgumentConstructorHandlerUndefined implements IArgumentConstructorHandler {
   public canHandle<TInstance>(
     value: Resource,
-    _settings: IConstructionSettings,
-    _argsCreator: IArgumentsConstructor<TInstance>,
+    settings: IConstructionSettings,
+    argsCreator: IArgumentsConstructor<TInstance>,
   ): boolean {
     return Boolean(value.property.undefined);
   }
diff --git a/lib/construction/argument/ArgumentConstructorHandlerValue.ts b/lib/construction/argument/ArgumentConstructorHandlerValue.ts
@@ -9,8 +9,8 @@ import type { IArgumentsConstructor } from './IArgumentsConstructor';
 export class ArgumentConstructorHandlerValue implements IArgumentConstructorHandler {
   public canHandle<TInstance>(
     value: Resource,
-    _settings: IConstructionSettings,
-    _argsCreator: IArgumentsConstructor<TInstance>,
+    settings: IConstructionSettings,
+    argsCreator: IArgumentsConstructor<TInstance>,
   ): boolean {
     return Boolean(value.property.value);
   }
diff --git a/lib/construction/strategy/IConstructionStrategy.ts b/lib/construction/strategy/IConstructionStrategy.ts
@@ -108,7 +108,7 @@ export interface ICreationStrategySupplierOptions<TInstance> {
   supplier: () => Promise<TInstance>;
 }
 
-export interface ICreationStrategyPrimitiveOptions<_T> {
+export interface ICreationStrategyPrimitiveOptions<TInstance> {
   /**
    * Creation settings.
    */
@@ -119,7 +119,7 @@ export interface ICreationStrategyPrimitiveOptions<_T> {
   value: string | number | any;
 }
 
-export interface ICreationStrategyVariableOptions<_T> {
+export interface ICreationStrategyVariableOptions<TInstance> {
   /**
    * Creation settings.
    */
diff --git a/lib/preprocess/constructorargumentsmapping/ConstructorArgumentsElementMappingHandlerCollectEntries.ts b/lib/preprocess/constructorargumentsmapping/ConstructorArgumentsElementMappingHandlerCollectEntries.ts
@@ -20,8 +20,8 @@ implements IConstructorArgumentsElementMappingHandler {
   public canHandle(
     configRoot: Resource,
     constructorArgs: Resource,
-    _configElement: Resource,
-    _mapper: IConstructorArgumentsMapper,
+    configElement: Resource,
+    mapper: IConstructorArgumentsMapper,
   ): boolean {
     return Boolean((constructorArgs.property.value || constructorArgs.property.valueRawReference) &&
       constructorArgs.property.collectEntries);
diff --git a/lib/preprocess/constructorargumentsmapping/ConstructorArgumentsElementMappingHandlerElements.ts b/lib/preprocess/constructorargumentsmapping/ConstructorArgumentsElementMappingHandlerElements.ts
@@ -11,8 +11,8 @@ export class ConstructorArgumentsElementMappingHandlerElements implements IConst
   public canHandle(
     configRoot: Resource,
     constructorArgs: Resource,
-    _configElement: Resource,
-    _mapper: IConstructorArgumentsMapper,
+    configElement: Resource,
+    mapper: IConstructorArgumentsMapper,
   ): boolean {
     return Boolean(constructorArgs.property.elements);
   }
diff --git a/lib/preprocess/constructorargumentsmapping/ConstructorArgumentsElementMappingHandlerFields.ts b/lib/preprocess/constructorargumentsmapping/ConstructorArgumentsElementMappingHandlerFields.ts
@@ -11,8 +11,8 @@ export class ConstructorArgumentsElementMappingHandlerFields implements IConstru
   public canHandle(
     configRoot: Resource,
     constructorArgs: Resource,
-    _configElement: Resource,
-    _mapper: IConstructorArgumentsMapper,
+    configElement: Resource,
+    mapper: IConstructorArgumentsMapper,
   ): boolean {
     return Boolean(constructorArgs.property.fields);
   }
diff --git a/lib/preprocess/constructorargumentsmapping/ConstructorArgumentsElementMappingHandlerKeyValue.ts b/lib/preprocess/constructorargumentsmapping/ConstructorArgumentsElementMappingHandlerKeyValue.ts
@@ -11,8 +11,8 @@ export class ConstructorArgumentsElementMappingHandlerKeyValue implements IConst
   public canHandle(
     configRoot: Resource,
     constructorArgs: Resource,
-    _configElement: Resource,
-    _mapper: IConstructorArgumentsMapper,
+    configElement: Resource,
+    mapper: IConstructorArgumentsMapper,
   ): boolean {
     return Boolean((constructorArgs.property.value || constructorArgs.property.valueRawReference) &&
       !constructorArgs.property.collectEntries);
diff --git a/lib/preprocess/constructorargumentsmapping/ConstructorArgumentsElementMappingHandlerList.ts b/lib/preprocess/constructorargumentsmapping/ConstructorArgumentsElementMappingHandlerList.ts
@@ -10,8 +10,8 @@ export class ConstructorArgumentsElementMappingHandlerList implements IConstruct
   public canHandle(
     configRoot: Resource,
     constructorArgs: Resource,
-    _configElement: Resource,
-    _mapper: IConstructorArgumentsMapper,
+    configElement: Resource,
+    mapper: IConstructorArgumentsMapper,
   ): boolean {
     return Boolean(constructorArgs.list);
   }
diff --git a/lib/preprocess/overridesteps/OverrideListRemove.ts b/lib/preprocess/overridesteps/OverrideListRemove.ts
@@ -16,7 +16,7 @@ export class OverrideListRemove implements IOverrideStep {
   }
 
   public handle(config: Resource, step: Resource): Resource {
-    const { parameters, targets, values: _values } = extractOverrideStepFields(step, { parameters: 1, values: 0 });
+    const { parameters, targets } = extractOverrideStepFields(step, { parameters: 1, values: 0 });
 
     const list = getPropertyResourceList(config, parameters[0]);
 
diff --git a/lib/preprocess/parameterproperty/ParameterPropertyHandlerDefaultScoped.ts b/lib/preprocess/parameterproperty/ParameterPropertyHandlerDefaultScoped.ts
@@ -20,7 +20,7 @@ export class ParameterPropertyHandlerDefaultScoped implements IParameterProperty
     value: Resource | undefined,
     configRoot: Resource,
     parameter: Resource,
-    _configElement: Resource,
+    configElement: Resource,
   ): Resource | undefined {
     let applyingValue: Resource | undefined;
     for (const scoped of parameter.properties.defaultScoped) {
diff --git a/lib/preprocess/parameterproperty/ParameterPropertyHandlerFixed.ts b/lib/preprocess/parameterproperty/ParameterPropertyHandlerFixed.ts
@@ -20,7 +20,7 @@ export class ParameterPropertyHandlerFixed implements IParameterPropertyHandler
     value: Resource | undefined,
     configRoot: Resource,
     parameter: Resource,
-    _configElement: Resource,
+    configElement: Resource,
   ): Resource | undefined {
     if (parameter.properties.fixed.length > 1) {
       throw new ErrorResourcesContext(`Invalid fixed value for parameter "${parameter.value}": Only one value can be defined, or an RDF list must be provided`, { parameter });
diff --git a/lib/preprocess/parameterproperty/ParameterPropertyHandlerLazy.ts b/lib/preprocess/parameterproperty/ParameterPropertyHandlerLazy.ts
@@ -14,7 +14,7 @@ export class ParameterPropertyHandlerLazy implements IParameterPropertyHandler {
     value: Resource | undefined,
     configRoot: Resource,
     parameter: Resource,
-    _configElement: Resource,
+    configElement: Resource,
   ): Resource | undefined {
     if (value) {
       if (value.list) {
diff --git a/lib/preprocess/parameterproperty/ParameterPropertyHandlerRange.ts b/lib/preprocess/parameterproperty/ParameterPropertyHandlerRange.ts
@@ -412,7 +412,7 @@ export class ParameterPropertyHandlerRange implements IParameterPropertyHandler
     value: Resource,
     type: Resource | NamedNode,
     errorContext: IErrorContext,
-    _genericsContext: GenericsContext,
+    genericsContext: GenericsContext,
   ): ILiteralAsTypeInterpretationResult {
     let parsed;
     switch (type.value) {
diff --git a/lib/rdf/PrefetchedDocumentLoader.ts b/lib/rdf/PrefetchedDocumentLoader.ts
@@ -2,10 +2,8 @@ import type { IJsonLdContext } from 'jsonld-context-parser';
 import { FetchDocumentLoader } from 'jsonld-context-parser';
 import semverMajor from 'semver/functions/major';
 import type { Logger } from 'winston';
-// eslint-disable-next-line import/extensions
-import contextJson from '../../components/context.json';
-// eslint-disable-next-line import/extensions
-import packageJson from '../../package.json';
+import contextJson from '../../components/context.json'; // eslint-disable-line import/extensions
+import packageJson from '../../package.json'; // eslint-disable-line import/extensions
 
 /**
  * A document loader that first loads from a precomputed set of contexts,

Original file line number	Diff line number	Diff line change
`@@ -33,6 +33,8 @@ module.exports = config([`
`33`	`33`	`'ts/no-unsafe-assignment': 'off',`
`34`	`34`	`'ts/no-unsafe-argument': 'off',`
`35`	`35`	`'ts/no-unsafe-return': 'off',`
	`36`	`+ // Don't flag unused function parameters (common in interface implementations)`
	`37`	`+ 'unused-imports/no-unused-vars': [ 'error', { args: 'none' }],`
`36`	`38`	`},`
`37`	`39`	`},`
`38`	`40`	`{`