fix(hooks): only trace hooks that use param-name service injection

A hook declaring nothing but hookArgs (or no parameters) already follows
the recommended pattern - resolving services through the typed container
- and must not be reported as deprecated usage.

Author: edusperoni

lib/common/services/hooks-service.ts

@@ -246,11 +246,19 @@ export class HooksService implements IHooksService {
 					projectDataHookArg;
 			}
 
-			reportDeprecation({
-				api: "hooks.param-name-signature",
-				detail: hook.fullPath,
-				logger: this.$logger,
-			});
+			// Only param-name *service* injection is on the deprecation track; a
+			// hook declaring nothing but `hookArgs` (or no parameters) already
+			// follows the recommended pattern and must not be flagged.
+			const usesParamNameInjection = (<string[]>(
+				hookEntryPoint.$inject.args
+			)).some((argument) => argument !== this.hookArgsName);
+			if (usesParamNameInjection) {
+				reportDeprecation({
+					api: "hooks.param-name-signature",
+					detail: hook.fullPath,
+					logger: this.$logger,
+				});
+			}
 
 			const maybePromise = this.$injector.resolve(
 				hookEntryPoint,

test/compat/legacy-hooks.ts

@@ -138,6 +138,9 @@ describe("legacy hook contract", () => {
 		});
 
 		assert.deepEqual(args, ["assembleDebug", "--offline"]);
+		// A hookArgs-only signature is the recommended pattern and must not be
+		// reported as param-name injection.
+		assert.notInclude(logger().traceOutput, "hooks.param-name-signature");
 	});
 
 	it("treats a rejection carrying stopExecution + errorAsWarning as a warning, not a failure", async () => {