AG-31746 stop using getText for logging

Merge in ADGUARD-FILTERS/scriptlets from fix/AG-31746 to master

Squashed commit of the following:

commit 6b69abd
Author: Maxim Topciu <mtopciu@adguard.com>
Date:   Wed Apr 17 12:51:45 2024 +0300

    AG-31746 fix changelog

commit da6f4f4
Author: Maxim Topciu <mtopciu@adguard.com>
Date:   Wed Apr 17 12:05:49 2024 +0300

    AG-31746 add changelog

commit 577f04c
Author: Maxim Topciu <mtopciu@adguard.com>
Date:   Tue Apr 16 13:01:47 2024 +0300

    AG-31746 fix typo

commit 4da4ad5
Author: Maxim Topciu <mtopciu@adguard.com>
Date:   Tue Apr 16 12:34:30 2024 +0300

    AG-31746 stop using getText for logging

CHANGELOG.md

@@ -15,6 +15,10 @@ The format is based on [Keep a Changelog], and this project adheres to [Semantic
   without needing the full rule text [#420]
 - `trusted-dispatch-event` scriptlet [#382]
 
+### Deprecated
+
+- `ruleText` option in the `IConfiguration`
+
 [Unreleased]: https://github.com/AdguardTeam/Scriptlets/compare/v1.10.25...HEAD
 [#420]: https://github.com/AdguardTeam/Scriptlets/issues/420
 [#382]: https://github.com/AdguardTeam/Scriptlets/issues/382

README.md

@@ -448,7 +448,7 @@ where:
  * @property {'extension'|'corelibs'} engine — Defines the final form of scriptlet string presentation.
  * @property {string} [version] — Extension version.
  * @property {boolean} [verbose] — Flag to enable debug information printing to console.
- * @property {string} [ruleText] — Source rule text, needed for debug purposes.
+ * @property {string} [ruleText] — Deprecated: Source rule text, needed for debug purposes.
  * @property {string} [domainName] — Domain name where scriptlet is applied, needed for debug purposes.
  */
 ```
@@ -494,7 +494,7 @@ scriptlets.isValidScriptletName(name);
  *
  * @returns {boolean} — True if scriptlet name is valid in rule.
  */
-scriptlets.isValidScriptletRule(input);
+scriptlets.isValidScriptletRule(ruleText);
 ```
 
 

src/helpers/converter.ts

@@ -568,7 +568,7 @@ export const isValidScriptletRule = (ruleText: string): boolean => {
  * @param modifiers rule modifiers
  * @param redirectsData validator.REDIRECT_RULE_TYPES.(UBO|ADG)
  * @param rule rule string
- * @returns merker data object
+ * @returns marker data object
  */
 const getMarkerData = (modifiers: string[], redirectsData: RedirectsData, rule: string): MarkerData => {
     const { redirectRuleMarker, redirectMarker } = redirectsData;

src/helpers/hit.ts

@@ -22,22 +22,11 @@ export const hit = (source: Source) => {
         const log = console.log.bind(console);
         const trace = console.trace.bind(console);
 
-        let prefix = source.ruleText || '';
-
+        let prefix = '';
         if (source.domainName) {
-            const AG_SCRIPTLET_MARKER = '#%#//';
-            const UBO_SCRIPTLET_MARKER = '##+js';
-            let ruleStartIndex;
-            if (source.ruleText.includes(AG_SCRIPTLET_MARKER)) {
-                ruleStartIndex = source.ruleText.indexOf(AG_SCRIPTLET_MARKER);
-            } else if (source.ruleText.includes(UBO_SCRIPTLET_MARKER)) {
-                ruleStartIndex = source.ruleText.indexOf(UBO_SCRIPTLET_MARKER);
-            }
-            // delete all domains from ruleText and leave just rule part
-            const rulePart = source.ruleText.slice(ruleStartIndex);
-            // prepare applied scriptlet rule for specific domain
-            prefix = `${source.domainName}${rulePart}`;
+            prefix += `${source.domainName}`;
         }
+        prefix += `#%#//scriptlet('${source.name}', '${source.args.join(', ')}')`;
 
         log(`${prefix} trace start`);
         if (trace) {

types/scriptlets.d.ts

@@ -35,9 +35,11 @@ declare module '@adguard/scriptlets' {
         verbose: boolean;
 
         /**
-         * Source rule text is used for debugging purposes
+         * Source rule text is used for debugging purposes.
+         *
+         * @deprecated since it is not used in the code anymore.
          */
-        ruleText: string;
+        ruleText?: string;
 
         /**
          * Domain name, used to improve logging