i18n: use IcuMessage objects instead of string IDs (#10630)

GoogleChrome · Sep 17, 2020 · 378a31f · 378a31f
1 parent d4fc00c
commit 378a31f
Show file tree

Hide file tree

Showing 56 changed files with 708 additions and 390 deletions.
diff --git a/clients/test/extension/settings-controller-test.js b/clients/test/extension/settings-controller-test.js
@@ -8,14 +8,18 @@
 /* eslint-env jest */
 
 const SettingsController = require('../../extension/scripts/settings-controller.js');
-const Config = require('../../../lighthouse-core/config/config.js');
 const defaultConfig = require('../../../lighthouse-core/config/default-config.js');
 const i18n = require('../../../lighthouse-core/lib/i18n/i18n.js');
 
 describe('Lighthouse chrome extension SettingsController', () => {
   it('default categories should be correct', () => {
-    const categories = Config.getCategories(defaultConfig);
-    categories.forEach(cat => cat.title = i18n.getFormatted(cat.title, 'en-US'));
+    const categories = Object.entries(defaultConfig.categories)
+      .map(([id, category]) => {
+        return {
+          id,
+          title: i18n.getFormatted(category.title, 'en-US'),
+        };
+      });
     expect(SettingsController.DEFAULT_CATEGORIES).toEqual(categories);
   });
 });
diff --git a/lighthouse-cli/run.js b/lighthouse-cli/run.js
@@ -21,7 +21,7 @@ const assetSaver = require('../lighthouse-core/lib/asset-saver.js');
 
 const open = require('open');
 
-/** @typedef {import('../lighthouse-core/lib/lh-error.js')} LighthouseError */
+/** @typedef {Error & {code: string, friendlyMessage?: string}} ExitError */
 
 const _RUNTIME_ERROR_CODE = 1;
 const _PROTOCOL_TIMEOUT_EXIT_CODE = 67;
@@ -92,7 +92,7 @@ function printProtocolTimeoutErrorAndExit() {
 }
 
 /**
- * @param {LighthouseError} err
+ * @param {ExitError} err
  * @return {never}
  */
 function printRuntimeErrorAndExit(err) {
@@ -104,7 +104,7 @@ function printRuntimeErrorAndExit(err) {
 }
 
 /**
- * @param {LighthouseError} err
+ * @param {ExitError} err
  * @return {never}
  */
 function printErrorAndExit(err) {
@@ -239,7 +239,6 @@ async function runLighthouse(url, flags, config) {
       return printErrorAndExit({
         name: 'LHError',
         friendlyMessage: runtimeError.message,
-        lhrRuntimeError: true,
         code: runtimeError.code,
         message: runtimeError.message,
       });

diff --git a/lighthouse-core/audits/audit.js b/lighthouse-core/audits/audit.js
@@ -242,8 +242,8 @@ class Audit {
 
   /**
    * @param {typeof Audit} audit
-   * @param {string} errorMessage
-   * @return {LH.Audit.Result}
+   * @param {string | LH.IcuMessage} errorMessage
+   * @return {LH.RawIcu<LH.Audit.Result>}
    */
   static generateErrorAuditResult(audit, errorMessage) {
     return Audit.generateAuditResult(audit, {
@@ -255,7 +255,7 @@ class Audit {
   /**
    * @param {typeof Audit} audit
    * @param {LH.Audit.Product} product
-   * @return {LH.Audit.Result}
+   * @return {LH.RawIcu<LH.Audit.Result>}
    */
   static generateAuditResult(audit, product) {
     if (product.score === undefined) {

diff --git a/lighthouse-core/audits/byte-efficiency/byte-efficiency-audit.js b/lighthouse-core/audits/byte-efficiency/byte-efficiency-audit.js
@@ -27,9 +27,9 @@ const WASTED_MS_FOR_SCORE_OF_ZERO = 5000;
  * @property {Array<LH.Audit.ByteEfficiencyItem>} items
  * @property {Map<string, number>=} wastedBytesByUrl
  * @property {LH.Audit.Details.Opportunity['headings']} headings
- * @property {string} [displayValue]
- * @property {string} [explanation]
- * @property {Array<string>} [warnings]
+ * @property {LH.IcuMessage} [displayValue]
+ * @property {LH.IcuMessage} [explanation]
+ * @property {Array<string | LH.IcuMessage>} [warnings]
  */
 
 /**

diff --git a/lighthouse-core/audits/byte-efficiency/render-blocking-resources.js b/lighthouse-core/audits/byte-efficiency/render-blocking-resources.js
@@ -270,7 +270,7 @@ class RenderBlockingResources extends Audit {
   static async audit(artifacts, context) {
     const {results, wastedMs} = await RenderBlockingResources.computeResults(artifacts, context);
 
-    let displayValue = '';
+    let displayValue;
     if (results.length > 0) {
       displayValue = str_(i18n.UIStrings.displayValueMsSavings, {wastedMs});
     }

diff --git a/lighthouse-core/audits/content-width.js b/lighthouse-core/audits/content-width.js
@@ -59,7 +59,7 @@ class ContentWidth extends Audit {
       };
     }
 
-    let explanation = '';
+    let explanation;
     if (!widthsMatch) {
       explanation = str_(UIStrings.explanation,
         {innerWidth: artifacts.ViewportDimensions.innerWidth,

diff --git a/lighthouse-core/audits/deprecations.js b/lighthouse-core/audits/deprecations.js
@@ -87,7 +87,7 @@ class Deprecations extends Audit {
     ];
     const details = Audit.makeTableDetails(headings, deprecations);
 
-    let displayValue = '';
+    let displayValue;
     if (deprecations.length > 0) {
       displayValue = str_(UIStrings.displayValue, {itemCount: deprecations.length});
     }

diff --git a/lighthouse-core/audits/dobetterweb/external-anchors-use-rel-noopener.js b/lighthouse-core/audits/dobetterweb/external-anchors-use-rel-noopener.js
@@ -49,7 +49,7 @@ class ExternalAnchorsUseRelNoopenerAudit extends Audit {
    * @return {LH.Audit.Product}
    */
   static audit(artifacts) {
-    /** @type {string[]} */
+    /** @type {LH.IcuMessage[]} */
     const warnings = [];
     const pageHost = new URL(artifacts.URL.finalUrl).host;
     const failingAnchors = artifacts.AnchorElements

diff --git a/lighthouse-core/audits/dobetterweb/no-vulnerable-libraries.js b/lighthouse-core/audits/dobetterweb/no-vulnerable-libraries.js
@@ -52,15 +52,15 @@ const str_ = i18n.createMessageInstanceIdFn(__filename, UIStrings);
 
 const SEMVER_REGEX = /^(\d+\.\d+\.\d+)[^-0-9]+/;
 
-/** @type {Object<string, string>} */
+/** @type {Record<string, LH.IcuMessage>} */
 const rowMap = {
   'low': str_(UIStrings.rowSeverityLow),
   'medium': str_(UIStrings.rowSeverityMedium),
   'high': str_(UIStrings.rowSeverityHigh),
 };
 
 /** @typedef {{npm: Object<string, Array<{id: string, severity: string, semver: {vulnerable: Array<string>}}>>}} SnykDB */
-/** @typedef {{severity: string, numericSeverity: number, library: string, url: string}} Vulnerability */
+/** @typedef {{severity: LH.IcuMessage, numericSeverity: number, library: string, url: string}} Vulnerability */
 
 class NoVulnerableLibrariesAudit extends Audit {
   /**
@@ -158,7 +158,7 @@ class NoVulnerableLibrariesAudit extends Audit {
 
   /**
    * @param {Array<Vulnerability>} vulnerabilities
-   * @return {string}
+   * @return {LH.IcuMessage}
    */
   static highestSeverity(vulnerabilities) {
     const sortedVulns = vulnerabilities
@@ -181,7 +181,7 @@ class NoVulnerableLibrariesAudit extends Audit {
     }
 
     let totalVulns = 0;
-    /** @type {Array<{highestSeverity: string, vulnCount: number, detectedLib: LH.Audit.Details.LinkValue}>} */
+    /** @type {Array<{highestSeverity: LH.IcuMessage, vulnCount: number, detectedLib: LH.Audit.Details.LinkValue}>} */
     const vulnerabilityResults = [];
 
     for (const lib of foundLibraries) {
@@ -206,7 +206,7 @@ class NoVulnerableLibrariesAudit extends Audit {
       }
     }
 
-    let displayValue = '';
+    let displayValue;
     if (totalVulns > 0) {
       displayValue = str_(UIStrings.displayValue, {itemCount: totalVulns});
     }

diff --git a/lighthouse-core/audits/dobetterweb/uses-http2.js b/lighthouse-core/audits/dobetterweb/uses-http2.js
@@ -210,7 +210,7 @@ class UsesHTTP2Audit extends Audit {
     const resources = UsesHTTP2Audit.determineNonHttp2Resources(networkRecords);
     const wastedMs = UsesHTTP2Audit.computeWasteWithTTIGraph(resources, graph, simulator);
 
-    let displayValue = '';
+    let displayValue;
     if (resources.length > 0) {
       displayValue = str_(UIStrings.displayValue, {itemCount: resources.length});
     }

diff --git a/lighthouse-core/audits/font-display.js b/lighthouse-core/audits/font-display.js
@@ -115,7 +115,7 @@ class FontDisplay extends Audit {
   /**
    * Some pages load many fonts we can't check, so dedupe on origin.
    * @param {Array<string>} warningUrls
-   * @return {Array<string>}
+   * @return {Array<LH.IcuMessage>}
    */
   static getWarningsForFontUrls(warningUrls) {
     /** @type {Map<string, number>} */

diff --git a/lighthouse-core/audits/image-aspect-ratio.js b/lighthouse-core/audits/image-aspect-ratio.js
@@ -56,7 +56,7 @@ class ImageAspectRatio extends Audit {
 
   /**
    * @param {WellDefinedImage} image
-   * @return {Error|{url: string, displayedAspectRatio: string, actualAspectRatio: string, doRatiosMatch: boolean}}
+   * @return {LH.IcuMessage|{url: string, displayedAspectRatio: string, actualAspectRatio: string, doRatiosMatch: boolean}}
    */
   static computeAspectRatios(image) {
     const url = URL.elideDataURI(image.src);
@@ -68,7 +68,7 @@ class ImageAspectRatio extends Audit {
 
     if (!Number.isFinite(actualAspectRatio) ||
       !Number.isFinite(displayedAspectRatio)) {
-      return new Error(str_(UIStrings.warningCompute, {url}));
+      return str_(UIStrings.warningCompute, {url});
     }
 
     return {
@@ -88,7 +88,7 @@ class ImageAspectRatio extends Audit {
   static audit(artifacts) {
     const images = artifacts.ImageElements;
 
-    /** @type {string[]} */
+    /** @type {LH.IcuMessage[]} */
     const warnings = [];
     /** @type {Array<{url: string, displayedAspectRatio: string, actualAspectRatio: string, doRatiosMatch: boolean}>} */
     const results = [];
@@ -109,8 +109,8 @@ class ImageAspectRatio extends Audit {
     }).forEach(image => {
       const wellDefinedImage = /** @type {WellDefinedImage} */ (image);
       const processed = ImageAspectRatio.computeAspectRatios(wellDefinedImage);
-      if (processed instanceof Error) {
-        warnings.push(processed.message);
+      if (i18n.isIcuMessage(processed)) {
+        warnings.push(processed);
         return;
       }
 

diff --git a/lighthouse-core/audits/is-on-https.js b/lighthouse-core/audits/is-on-https.js
@@ -90,7 +90,7 @@ class HTTPS extends Audit {
           .filter(record => !HTTPS.isSecureRecord(record))
           .map(record => URL.elideDataURI(record.url));
 
-      /** @type {Array<{url: string, resolution?: string}>}  */
+      /** @type {Array<{url: string, resolution?: LH.IcuMessage|string}>}  */
       const items = Array.from(new Set(insecureURLs)).map(url => ({url, resolution: undefined}));
 
       /** @type {LH.Audit.Details.Table['headings']} */
@@ -119,7 +119,7 @@ class HTTPS extends Audit {
         if (!item.resolution) item.resolution = str_(UIStrings.allowed);
       }
 
-      let displayValue = '';
+      let displayValue;
       if (items.length > 0) {
         displayValue = str_(UIStrings.displayValue, {itemCount: items.length});
       }

diff --git a/lighthouse-core/audits/load-fast-enough-for-pwa.js b/lighthouse-core/audits/load-fast-enough-for-pwa.js
@@ -84,9 +84,7 @@ class LoadFastEnough4Pwa extends Audit {
 
     const score = Number(tti.timing < MAXIMUM_TTI);
 
-    /** @type {string|undefined} */
     let displayValue;
-    /** @type {string|undefined} */
     let explanation;
     if (!score) {
       displayValue = str_(displayValueTemplate, {timeInMs: tti.timing});

diff --git a/lighthouse-core/audits/non-composited-animations.js b/lighthouse-core/audits/non-composited-animations.js
@@ -85,7 +85,7 @@ const ACTIONABLE_FAILURE_REASONS = [
  * We can check if a specific bit is true in the failure coding using bitwise and '&' with the flag.
  * @param {number} failureCode
  * @param {string[]} unsupportedProperties
- * @return {string[]}
+ * @return {LH.IcuMessage[]}
  */
 function getActionableFailureReasons(failureCode, unsupportedProperties) {
   return ACTIONABLE_FAILURE_REASONS

diff --git a/lighthouse-core/audits/offline-start-url.js b/lighthouse-core/audits/offline-start-url.js
@@ -65,6 +65,7 @@ class OfflineStartUrl extends Audit {
     // The StartUrl artifact provides no explanation if a response was received from a service
     // worker with a non-200 status code. In that case, this audit provides its own explanation.
     // In all other cases it defers to the artifact explanation.
+    /** @type {string|LH.IcuMessage|undefined} */
     let explanation = artifacts.StartUrl.explanation;
     if (!explanation && artifacts.StartUrl.statusCode !== -1 && !hasOfflineStartUrl) {
       explanation = str_(UIStrings.errorLoading, {

diff --git a/lighthouse-core/audits/performance-budget.js b/lighthouse-core/audits/performance-budget.js
@@ -27,7 +27,7 @@ const UIStrings = {
 const str_ = i18n.createMessageInstanceIdFn(__filename, UIStrings);
 
 /** @typedef {import('../computed/resource-summary.js').ResourceEntry} ResourceEntry */
-/** @typedef {{resourceType: LH.Budget.ResourceType, label: string, requestCount: number, transferSize: number, sizeOverBudget: number | undefined, countOverBudget: string | undefined}} BudgetItem */
+/** @typedef {{resourceType: LH.Budget.ResourceType, label: LH.IcuMessage, requestCount: number, transferSize: number, sizeOverBudget: number | undefined, countOverBudget: LH.IcuMessage | undefined}} BudgetItem */
 
 class ResourceBudget extends Audit {
   /**

diff --git a/lighthouse-core/audits/resource-summary.js b/lighthouse-core/audits/resource-summary.js
@@ -55,7 +55,7 @@ class ResourceSummary extends Audit {
     ];
 
 
-    /** @type {Record<LH.Budget.ResourceType,string>} */
+    /** @type {Record<LH.Budget.ResourceType, LH.IcuMessage>} */
     const strMappings = {
       'total': str_(i18n.UIStrings.totalResourceType),
       'document': str_(i18n.UIStrings.documentResourceType),

diff --git a/lighthouse-core/audits/seo/hreflang.js b/lighthouse-core/audits/seo/hreflang.js
@@ -7,7 +7,7 @@
 
 /** @typedef {string|LH.Audit.Details.NodeValue|undefined} Source */
 /** @typedef {{source: Source, subItems: {type: 'subitems', items: SubItem[]}}} InvalidHreflang */
-/** @typedef {{reason: string}} SubItem */
+/** @typedef {{reason: LH.IcuMessage}} SubItem */
 
 const Audit = require('../audit.js');
 const VALID_LANGS = importValidLangs();

diff --git a/lighthouse-core/audits/service-worker.js b/lighthouse-core/audits/service-worker.js
@@ -107,7 +107,7 @@ class ServiceWorker extends Audit {
    * contolled by the scopeUrl.
    * @param {LH.Artifacts['WebAppManifest']} WebAppManifest
    * @param {string} scopeUrl
-   * @return {string|undefined}
+   * @return {LH.IcuMessage|undefined}
    */
   static checkStartUrl(WebAppManifest, scopeUrl) {
     if (!WebAppManifest) {

diff --git a/lighthouse-core/audits/third-party-summary.js b/lighthouse-core/audits/third-party-summary.js
@@ -52,7 +52,7 @@ const PASS_THRESHOLD_IN_MS = 250;
  * @typedef {{
  *   transferSize: number,
  *   blockingTime: number,
- *   url: string,
+ *   url: string | LH.IcuMessage,
  * }} URLSummary
  */
 

diff --git a/lighthouse-core/audits/timing-budget.js b/lighthouse-core/audits/timing-budget.js
@@ -24,7 +24,7 @@ const UIStrings = {
 
 const str_ = i18n.createMessageInstanceIdFn(__filename, UIStrings);
 
-/** @typedef {{metric: LH.Budget.TimingMetric, label: string, measurement?: LH.Audit.Details.NumericValue|number, overBudget?: LH.Audit.Details.NumericValue|number}} BudgetItem */
+/** @typedef {{metric: LH.Budget.TimingMetric, label: LH.IcuMessage, measurement?: LH.Audit.Details.NumericValue|number, overBudget?: LH.Audit.Details.NumericValue|number}} BudgetItem */
 
 class TimingBudget extends Audit {
   /**
@@ -42,7 +42,7 @@ class TimingBudget extends Audit {
 
   /**
    * @param {LH.Budget.TimingMetric} timingMetric
-   * @return {string}
+   * @return {LH.IcuMessage}
    */
   static getRowLabel(timingMetric) {
     /** @type {Record<LH.Budget.TimingMetric, string>} */

diff --git a/lighthouse-core/audits/user-timings.js b/lighthouse-core/audits/user-timings.js
@@ -100,7 +100,6 @@ class UserTimings extends Audit {
 
       const details = Audit.makeTableDetails(headings, tableRows);
 
-      /** @type {string|undefined} */
       let displayValue;
       if (userTimings.length) {
         displayValue = str_(UIStrings.displayValue, {itemCount: userTimings.length});

diff --git a/lighthouse-core/audits/uses-rel-preconnect.js b/lighthouse-core/audits/uses-rel-preconnect.js
@@ -108,7 +108,7 @@ class UsesRelPreconnectAudit extends Audit {
     const settings = context.settings;
 
     let maxWasted = 0;
-    /** @type {string[]} */
+    /** @type {Array<LH.IcuMessage>} */
     const warnings = [];
 
     const [networkRecords, mainResource, loadSimulator, traceOfTab, pageGraph] = await Promise.all([

diff --git a/lighthouse-core/audits/uses-rel-preload.js b/lighthouse-core/audits/uses-rel-preload.js
@@ -221,7 +221,7 @@ class UsesRelPreloadAudit extends Audit {
     // sort results by wastedTime DESC
     results.sort((a, b) => b.wastedMs - a.wastedMs);
 
-    /** @type {Array<string>|undefined} */
+    /** @type {Array<LH.IcuMessage>|undefined} */
     let warnings;
     const failedURLs = UsesRelPreloadAudit.getURLsFailedToPreload(graph);
     if (failedURLs.size) {

diff --git a/lighthouse-core/config/config-helpers.js b/lighthouse-core/config/config-helpers.js
@@ -8,6 +8,7 @@
 const path = require('path');
 const Audit = require('../audits/audit.js');
 const Runner = require('../runner.js');
+const i18n = require('../lib/i18n/i18n.js');
 
 /**
  * If any items with identical `path` properties are found in the input array,
@@ -53,19 +54,19 @@ function assertValidAudit(auditDefinition) {
     throw new Error(`${auditName} has no meta.id property, or the property is not a string.`);
   }
 
-  if (typeof implementation.meta.title !== 'string') {
+  if (!i18n.isStringOrIcuMessage(implementation.meta.title)) {
     throw new Error(`${auditName} has no meta.title property, or the property is not a string.`);
   }
 
   // If it'll have a ✔ or ✖ displayed alongside the result, it should have failureTitle
   if (
-    typeof implementation.meta.failureTitle !== 'string' &&
+    !i18n.isStringOrIcuMessage(implementation.meta.failureTitle) &&
     implementation.meta.scoreDisplayMode === Audit.SCORING_MODES.BINARY
   ) {
     throw new Error(`${auditName} has no failureTitle and should.`);
   }
 
-  if (typeof implementation.meta.description !== 'string') {
+  if (!i18n.isStringOrIcuMessage(implementation.meta.description)) {
     throw new Error(
       `${auditName} has no meta.description property, or the property is not a string.`
     );