chore: fix typos (#1375)

danielrozenberg · web-flow · commit 6bb0a8c952ac · 2025-08-15T12:45:00.000-04:00
chore: fix typos
diff --git a/packages/cache-list/lib/AmpCaches.js b/packages/cache-list/lib/AmpCaches.js
@@ -60,7 +60,7 @@ class AmpCaches {
   }
 
   /**
-   * Retrieves the cache instance that maches the cacheId.
+   * Retrieves the cache instance that matches the cacheId.
    *
    * @param {string} cacheId The id of the cache to be retrieved.
    * @returns {Promise<object>} the Cache which the matching id or undefined, if a cache with the
diff --git a/packages/linter/src/rules/VideosAreSubtitled.ts b/packages/linter/src/rules/VideosAreSubtitled.ts
@@ -8,7 +8,7 @@ export class VideosAreSubtitled extends Rule {
 
     //Are there any <amp-video> tags? If so, is there at least 1 <track>?
     return numOfVideoElems > 0 && numOfSubtitleElems <= 0
-      ? this.warn(`One or more videos are missing HMTL subtitles`)
+      ? this.warn(`One or more videos are missing HTML subtitles`)
       : this.pass();
   }
   meta() {
diff --git a/packages/optimizer-docker/spec/index.spec.js b/packages/optimizer-docker/spec/index.spec.js
@@ -33,7 +33,7 @@ describe('Server integration tests', () => {
     });
   });
 
-  it('should run the provided HTML throught the optimizer', async () => {
+  it('should run the provided HTML through the optimizer', async () => {
     const {body} = await request({body: VALID_REQUEST, query: {canonical: 'http://example.com'}});
     expect(body).toMatch('<!doctype html>');
     expect(body).toMatch('<link data-auto rel="canonical" href="http://example.com">');
diff --git a/packages/optimizer-express/lib/AmpOptimizeMiddleware.js b/packages/optimizer-express/lib/AmpOptimizeMiddleware.js
@@ -40,7 +40,7 @@ class AmpOptimizerMiddleware {
         return;
       }
 
-      // This is a request for the canonical URL. Setup the middelware to transform the
+      // This is a request for the canonical URL. Setup the middleware to transform the
       // response using amp-optimizer.
       const chunks = [];
 
@@ -78,7 +78,7 @@ class AmpOptimizerMiddleware {
           chunks.push(chunk);
         }
 
-        // If end is called withouth any chunks, end the request.
+        // If end is called without any chunks, end the request.
         if (chunks.length === 0) {
           res.end();
           return;
@@ -104,7 +104,7 @@ class AmpOptimizerMiddleware {
    * Javascript or CSS file.
    *
    * @param {Request} req the request to be checked.
-   * @returns {boolean} true if the reqeust is for a resource.
+   * @returns {boolean} true if the request is for a resource.
    */
   static isResourceRequest_(req) {
     // Checks if mime-type for request is text/html. If mime type is unknown, assume text/html,
diff --git a/packages/optimizer-express/spec/lib/UrlMappingSpec.js b/packages/optimizer-express/spec/lib/UrlMappingSpec.js
@@ -30,7 +30,7 @@ describe('UrlMapping', () => {
       expect(result).toBe(true);
     });
 
-    it('correcty identifies non-AMP urls', () => {
+    it('correctly identifies non-AMP urls', () => {
       const result = urlMapping.isAmpUrl('/page');
       expect(result).toBe(false);
     });
diff --git a/packages/optimizer/lib/DomTransformer.js b/packages/optimizer/lib/DomTransformer.js
@@ -23,7 +23,7 @@ const fetchRuntimeParameters = require('./fetchRuntimeParameters');
 const cache = require('./cache');
 
 /**
- * AMP Optimizer Configuration only applying AMP validity perserving transformations.
+ * AMP Optimizer Configuration only applying AMP validity preserving transformations.
  */
 const TRANSFORMATIONS_AMP_FIRST = [
   // Adds missing AMP tags
diff --git a/packages/optimizer/lib/ParseLayout.js b/packages/optimizer/lib/ParseLayout.js
@@ -37,7 +37,7 @@ const SIZE_DEFINED_LAYOUTS = [
 ];
 const CSS_LENGTH_ONE_PX = cssLength('1', false, false);
 const CSS_LENGTH_AUTO = cssLength('auto', true, false);
-const CSS_LENGTH_FOURTY_FOUR_PX = cssLength('44px', false, false);
+const CSS_LENGTH_FORTY_FOUR_PX = cssLength('44px', false, false);
 const CSS_LENGTH_SIXTY_PX = cssLength('60px', false, false);
 
 function getLayoutClass(layout) {
@@ -62,7 +62,7 @@ function calculateHeight(inputLayout, inputHeight, tagName) {
       case 'amp-pixel':
         return CSS_LENGTH_ONE_PX;
       case 'amp-social-share':
-        return CSS_LENGTH_FOURTY_FOUR_PX;
+        return CSS_LENGTH_FORTY_FOUR_PX;
       default:
     }
   }
diff --git a/packages/optimizer/lib/transformers/AddBlurryImagePlaceholders.js b/packages/optimizer/lib/transformers/AddBlurryImagePlaceholders.js
@@ -103,7 +103,7 @@ class AddBlurryImagePlaceholders {
       this.cache_ = new Map();
     } else if (maxCacheSize > 0) {
       const LRU = require('lru-cache');
-      this.log_.debug('using LRU cache for regularily used placeholders', maxCacheSize);
+      this.log_.debug('using LRU cache for regularly used placeholders', maxCacheSize);
       // use a LRU cache otherwise
       this.cache_ = new LRU({
         max: maxCacheSize,
diff --git a/packages/optimizer/lib/transformers/AutoExtensionImporter.js b/packages/optimizer/lib/transformers/AutoExtensionImporter.js
@@ -43,12 +43,12 @@ const manualExtensions = Array.from(manualAttributeToExtensionMapping.values());
  *
  * - use validation rules to map used AMP tags to required AMP extensions.
  * - use validation rules to map used AMP attributes to required AMP extensions.
- * - manually specifiy attribute to extension mappings if this information is not available in the
+ * - manually specify attribute to extension mappings if this information is not available in the
  *   validation rules.
- * - mnullay implement AMP extension detection for a few corner cases.
+ * - manually implement AMP extension detection for a few corner cases.
  *
  * This importer also enables a shortcode `bindtext` instead of `data-amp-bind-text` for specifying
- * AMP bindings when the square bracket notation (`[text]`) is not available. To avoid accidently
+ * AMP bindings when the square bracket notation (`[text]`) is not available. To avoid accidentally
  * rewriting non-AMP attributes, the transformer uses the AMP validation rules to only rename bindable
  * attributes as specified in the validation rules.
  *
diff --git a/packages/optimizer/lib/transformers/OptimizeImages.js b/packages/optimizer/lib/transformers/OptimizeImages.js
@@ -70,7 +70,7 @@ class SrcsetWidth {
   }
 
   /**
-   * Sets the base width, i.e., renderered dimension measured in CSS pixels.
+   * Sets the base width, i.e., rendered dimension measured in CSS pixels.
    * Returns true if srcset is needed, that is, we'll resize the image to at
    * least 2 supported widths (@see SRCSET_WIDTH for a list of supported widths).
    *
diff --git a/packages/optimizer/lib/transformers/PreloadHeroImage.js b/packages/optimizer/lib/transformers/PreloadHeroImage.js
@@ -16,7 +16,7 @@
 
 'use strict';
 
-/* This transfomer has been deprecated, use OptimizeHeroImages.js instead */
+/* This transformer has been deprecated, use OptimizeHeroImages.js instead */
 
 /** @module PreloadHeroImage */
 module.exports = require('./OptimizeHeroImages');
diff --git a/packages/optimizer/lib/transformers/PreloadImages.js b/packages/optimizer/lib/transformers/PreloadImages.js
@@ -35,7 +35,7 @@ const MAX_PRELOADED_IMAGES = 5;
  *
  * This transformer supports the following option:
  *
- * * `imagePreloadCount`: specifies the maxinum number of images to preload. The default is 5.
+ * * `imagePreloadCount`: specifies the maximum number of images to preload. The default is 5.
  */
 class PreloadImages {
   transform(root, params) {
diff --git a/packages/optimizer/lib/transformers/RewriteAmpUrls.js b/packages/optimizer/lib/transformers/RewriteAmpUrls.js
@@ -118,7 +118,7 @@ class RewriteAmpUrls {
       node = node.nextSibling;
     }
 
-    // process preloads later to avoid accidently rewriting the URL
+    // process preloads later to avoid accidentally rewriting the URL
     for (const preload of preloads) {
       if (preload) {
         insertAfter(head, preload, referenceNode);
diff --git a/packages/optimizer/spec/transformers/TransformerSpec.js b/packages/optimizer/spec/transformers/TransformerSpec.js
@@ -21,15 +21,15 @@ log.verbose();
 const {getDirectories} = require('../helpers/Utils.js');
 const createSpec = require('../helpers/TransformerRunner.js');
 
-describe('Transfomers', () => {
+describe('Transformers', () => {
   ['experimental', 'valid'].forEach((subDir) => {
     loadTestConfigs(subDir).forEach(createSpec);
   });
 });
 
 function loadTestConfigs(subDir) {
-  const transfomerTestDirs = getDirectories(join(__dirname, subDir));
-  return transfomerTestDirs.map((testDir) => {
+  const transformerTestDirs = getDirectories(join(__dirname, subDir));
+  return transformerTestDirs.map((testDir) => {
     const transformerName = basename(testDir);
     const transformerPath = join('../../lib/transformers', transformerName + '.js');
 
diff --git a/packages/page-experience/lib/PageDataGatherer.js b/packages/page-experience/lib/PageDataGatherer.js
@@ -276,7 +276,7 @@ class PageAnalyzer {
         request.resourceType() === 'script' &&
         !request.url().startsWith('https://cdn.ampproject.org')
       ) {
-        // Only donwload AMP runtime scripts as they're need for layouting the page
+        // Only download AMP runtime scripts as they're need for layouting the page
         // Once self-hosting is a thing we'll have to change this
         // TODO: investigate whether we could cache these locally
         return request.abort();
diff --git a/packages/page-experience/lib/checks/earlyIframes.js b/packages/page-experience/lib/checks/earlyIframes.js
@@ -25,7 +25,7 @@ const earlyIframes = (pageData, result) => {
   }
 
   result.warn({
-    title: 'Avoid iframes at the begining of pages',
+    title: 'Avoid iframes at the beginning of pages',
     message:
       'You have iframes at the top of your page - move them below the initial viewport to improve your loading time',
   });
diff --git a/packages/page-experience/test-data/checks/earlyIframes/iframe-in-first-viewport.json b/packages/page-experience/test-data/checks/earlyIframes/iframe-in-first-viewport.json
@@ -16,8 +16,8 @@
     }
   },
   "earlyIframes": {
-    "title": "Avoid iframes at the begining of pages",
+    "title": "Avoid iframes at the beginning of pages",
     "message": "You have iframes at the top of your page - move them below the initial viewport to improve your loading time",
     "status": "WARN"
   }
-}
+}

Original file line number	Diff line number	Diff line change
`@@ -60,7 +60,7 @@ class AmpCaches {`
`60`	`60`	`}`
`61`	`61`
`62`	`62`	`/**`
`63`		`- * Retrieves the cache instance that maches the cacheId.`
	`63`	`+ * Retrieves the cache instance that matches the cacheId.`
`64`	`64`	`*`
`65`	`65`	`* @param {string} cacheId The id of the cache to be retrieved.`
`66`	`66`	`* @returns {Promise<object>} the Cache which the matching id or undefined, if a cache with the`
Original file line number	Diff line number	Diff line change
`@@ -8,7 +8,7 @@ export class VideosAreSubtitled extends Rule {`
`8`	`8`
`9`	`9`	`//Are there any <amp-video> tags? If so, is there at least 1 <track>?`
`10`	`10`	`return numOfVideoElems > 0 && numOfSubtitleElems <= 0`
`11`		- ? this.warn(`One or more videos are missing HMTL subtitles`)
	`11`	+ ? this.warn(`One or more videos are missing HTML subtitles`)
`12`	`12`	`: this.pass();`
`13`	`13`	`}`
`14`	`14`	`meta() {`
Original file line number	Diff line number	Diff line change
`@@ -43,12 +43,12 @@ const manualExtensions = Array.from(manualAttributeToExtensionMapping.values());`
`43`	`43`	`*`
`44`	`44`	`* - use validation rules to map used AMP tags to required AMP extensions.`
`45`	`45`	`* - use validation rules to map used AMP attributes to required AMP extensions.`
`46`		`- * - manually specifiy attribute to extension mappings if this information is not available in the`
	`46`	`+ * - manually specify attribute to extension mappings if this information is not available in the`
`47`	`47`	`* validation rules.`
`48`		`- * - mnullay implement AMP extension detection for a few corner cases.`
	`48`	`+ * - manually implement AMP extension detection for a few corner cases.`
`49`	`49`	`*`
`50`	`50`	* This importer also enables a shortcode `bindtext` instead of `data-amp-bind-text` for specifying
`51`		- * AMP bindings when the square bracket notation (`[text]`) is not available. To avoid accidently
	`51`	+ * AMP bindings when the square bracket notation (`[text]`) is not available. To avoid accidentally
`52`	`52`	`* rewriting non-AMP attributes, the transformer uses the AMP validation rules to only rename bindable`
`53`	`53`	`* attributes as specified in the validation rules.`
`54`	`54`	`*`
Original file line number	Diff line number	Diff line change
`@@ -70,7 +70,7 @@ class SrcsetWidth {`
`70`	`70`	`}`
`71`	`71`
`72`	`72`	`/**`
`73`		`- * Sets the base width, i.e., renderered dimension measured in CSS pixels.`
	`73`	`+ * Sets the base width, i.e., rendered dimension measured in CSS pixels.`
`74`	`74`	`* Returns true if srcset is needed, that is, we'll resize the image to at`
`75`	`75`	`* least 2 supported widths (@see SRCSET_WIDTH for a list of supported widths).`
`76`	`76`	`*`