textbrowser 0.51.0 → 0.52.1

package/resources/templates/index.js

@@ -38,7 +38,6 @@ const Templates = {
         (installationDialog).showModal();
         const container = /** @type {HTMLElement} */ ($('#dialogContainer'));
         container.hidden = false;
-      // eslint-disable-next-line no-unused-vars -- Ok
       } catch (err) {
         // May already be open
       }

package/resources/templates/workDisplay.js

@@ -1121,7 +1121,6 @@ export default {
                         const url = replaceHash(paramsCopy) + `&work=${work}&${work}-startEnd1=%s`; // %s will be escaped if set as param; also add changeable workName here
                         try {
                           await navigator.clipboard.writeText(url);
-                        // eslint-disable-next-line no-unused-vars -- Okay to ignore
                         } catch (err) {
                           // User rejected
                         }

package/resources/user-sample.js

@@ -1,3 +1,4 @@
+// eslint-disable-next-line sonarjs/no-internal-api-use -- Demo
 import TextBrowser from '../node_modules/textbrowser/dist/index-es.js';
 
 const tb = new TextBrowser({

package/resources/utils/Metadata.js

@@ -3,7 +3,7 @@ import {getPreferredLanguages} from './Languages.js';
 import JsonRefs from 'json-refs'; // // eslint-disable-line import/order
 
 const getCurrDir = () => {
-  return window.location.href.replace(/(index\.html)?#.*$/, '');
+  return globalThis.location.href.replace(/(index\.html)?#.*$/, '');
 };
 
 /**
@@ -105,6 +105,7 @@ export const getMetadata = async (file, property, basePath) => {
       url.toString(),
       {
         loaderOptions: {
+          /* eslint-disable jsdoc/reject-any-type -- How to resolve? */
           /**
            * @param {{
            *   text: string,
@@ -113,6 +114,7 @@ export const getMetadata = async (file, property, basePath) => {
            * @param {(err?: Error, cbValue: any) => void} callback
            */
           processContent (res, callback) {
+            /* eslint-enable jsdoc/reject-any-type -- How to resolve? */
             callback(undefined, JSON.parse(
               res.text ||
                             // `.metadata` not a recognized extension, so

package/resources/utils/Plugin.js

@@ -40,6 +40,12 @@ export const escapePlugin = ({pluginName, applicableField, targetLanguage}) => {
         (targetLanguage ? '-' + escapePluginComponent(targetLanguage) : '');
 };
 
+/* eslint-disable jsdoc/reject-any-type -- How to resolve? */
+/**
+ * @typedef {any} MetaValue
+ */
+/* eslint-enable jsdoc/reject-any-type -- Arbitrary */
+
 /**
  * @todo Complete
  * @typedef {{
@@ -66,7 +72,7 @@ export const escapePlugin = ({pluginName, applicableField, targetLanguage}) => {
  *     },
  *     $p: import('./IntlURLSearchParams.js').default,
  *     thisObj: import('../index.js').default|import('../../server/main.js').ResultsDisplayServerContext
- *   }) => void,
+ *   }) => string|Integer,
  *   done: (info: {
  *     $p: import('./IntlURLSearchParams.js').default,
  *     applicableField: string|undefined,
@@ -87,7 +93,7 @@ export const escapePlugin = ({pluginName, applicableField, targetLanguage}) => {
  *     targetLanguage: string,
  *     applicableField: string,
  *     applicableFieldI18N: string|string[]|import("../../server/main.js").LocalizationStrings,
- *     meta: any,
+ *     meta: MetaValue,
  *     metaApplicableField: {
  *       [key: string]: string
  *     },
@@ -100,7 +106,7 @@ export const escapePlugin = ({pluginName, applicableField, targetLanguage}) => {
  * @typedef {{
  *   path: string,
  *   lang: string,
- *   meta: any,
+ *   meta: MetaValue,
  *   onByDefault: boolean
  * }} PluginInfo
  */
@@ -157,7 +163,7 @@ export class PluginsForWork {
    *     [applicableField: string]: {
    *       targetLanguage: string|string[],
    *       onByDefault: boolean,
-   *       meta: any
+   *       meta: MetaValue
    *     }
    *   },
    *   pluginName: string,
@@ -199,7 +205,7 @@ export class PluginsForWork {
    *   [applicableField: string]: {
    *     targetLanguage: string|string[],
    *     onByDefault: boolean,
-   *     meta: any
+   *     meta: MetaValue
    *   }
    * }} applicableFields
    * @param {(cfg: {

package/resources/utils/ServiceWorker.js

@@ -125,7 +125,7 @@ export const respondToState = async ({
   // We use this promise for rejecting (inside a listener)
   //    to a common catch and to prevent continuation by
   //    failing to return
-  return new Promise(async () => { // eslint-disable-line no-async-promise-executor, sonarjs/no-misused-promises -- See above
+  return new Promise(async () => { // eslint-disable-line no-async-promise-executor -- See above
     navigator.serviceWorker.addEventListener('message', ({data}) => {
       const {message, type, name, errorType} = data;
       console.log('msg1', message, r);
@@ -292,7 +292,6 @@ export const registerServiceWorker = async ({
         type: 'module'
       }
     );
-  // eslint-disable-next-line no-unused-vars -- Ok
   } catch (err) {
     console.log('serviceWorkerPath', serviceWorkerPath);
     await dialogs.alert(`

package/resources/utils/WorkInfo.js

@@ -93,7 +93,7 @@ export const getWorkFiles = async function getWorkFiles (files) {
   filesObj.groups.forEach((fileGroup) => {
     fileGroup.files.forEach((fileData) => {
       const {file, schemaFile, metadataFile} =
-                getFilePaths(filesObj, fileGroup, fileData);
+        getFilePaths(filesObj, fileGroup, fileData);
       dataFiles.push(file, schemaFile, metadataFile);
     });
   });

package/resources/utils/getLocaleFallbackResults.js

@@ -35,5 +35,7 @@ export default async function getLocaleFallbackResults ({
   if (!$p.l10n) {
     $p.l10n = l;
   }
-  return l;
+
+  // Not sure why cast needed here, but works...
+  return /** @type {import('intl-dom').I18NCallback} */ (l);
 }

package/server/main.js

@@ -3,8 +3,7 @@
 /* eslint-env node -- Environment here */
 import http from 'node:http';
 
-// @ts-expect-error Todo: Needs Types
-import statik from '@brettz9/node-static';
+import * as statik from '@node-static/node-static';
 import fetch from 'node-fetch';
 // @ts-expect-error Todo: Needs Types
 import commandLineArgs from 'command-line-args';
@@ -93,13 +92,19 @@ const port = 'port' in userParams ? userParams.port : 8000;
 const domain = userParams.domain || `localhost`;
 const basePath = userParams.basePath || `http://${domain}${port ? ':' + port : ''}/`;
 
+/* eslint-disable jsdoc/reject-any-type -- Arbitrary */
+/**
+ * @typedef {any} AnyValue
+ */
+/* eslint-enable jsdoc/reject-any-type -- Arbitrary */
+
 /**
  * @typedef {import('../resources/utils/ServiceWorker.js').ServiceWorkerConfig &
  *   UserOptions & {
  *     lang: string[],
  *     langs: LanguageInfo[],
  *     fallbackLanguages: string[],
- *     log: (...args: any[]) => void,
+ *     log: (...args: AnyValue[]) => void,
  *     nodeActivate?: boolean,
  *     port?: number,
  *     skipIndexedDB: false,
@@ -117,7 +122,7 @@ const userParamsWithDefaults = {
   }),
   ...userParams,
   /**
-   * @param {...any} args
+   * @param {...AnyValue} args
    */
   log (...args) {
     console.log(...args);
@@ -272,7 +277,7 @@ const srv = http.createServer(async (req, res) => {
   const languages = (req.headers['accept-language']?.replace(/;q=.*$/, '') ?? 'en-US').split(',');
   // @ts-expect-error Polyglot reasons
   // eslint-disable-next-line n/no-unsupported-features/node-builtins -- Polyglot reasons
-  global.navigator = {
+  globalThis.navigator = {
     language: languages[0],
     languages
   };

package/sw-sample.js

@@ -1,8 +1,9 @@
 /* eslint-env serviceworker -- Service worker */
 
+// eslint-disable-next-line sonarjs/no-internal-api-use -- Demo
 import swHelper from './node_modules/textbrowser/dist/sw-helper.js';
 
 // IMPORTANT: Keep this comment and increment this number to trigger
 //              a worker change: 1
 
-swHelper(self);
+swHelper(globalThis);