pr feedbacks

CHANGELOG.md

@@ -10,6 +10,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ### Added
 
 -   [#40](https://github.com/green-code-initiative/creedengo-javascript/pull/40) Add rule `@creedengo/avoid-autoplay` (GCI36)
+-   [#46](https://github.com/green-code-initiative/creedengo-javascript/pull/46) Add rule `@creedengo/prefer-lighter-formats-for-image-files` (GCI31)
 
 ## [2.0.0] - 2025-01-22
 

eslint-plugin/docs/rules/prefer-lighter-formats-for-image-files.md

@@ -12,15 +12,17 @@ Some image formats are generally considered better for eco-friendly web design a
 
 We recommend using the following formats:
 
-- *WebP*, developed by Google, is a modern image format that provides high compression efficiency without significant loss of quality.
-- *AVIF* (AV1 Image File Format) is a relatively new and highly efficient image format that is based on the AV1 video codec.
-- *SVG* (Scalable Vector Graphics) is a vector image format that is based on XML.
-Files are lightweight and can be scaled without loss of quality.
+- **WebP**, developed by Google, is a modern image format that provides high compression efficiency without significant loss of quality.
+- **AVIF** (AV1 Image File Format) is a relatively new and highly efficient image format that is based on the AV1 video codec.
+- **SVG** (Scalable Vector Graphics) is a vector image format that is based on XML.
+  Files are lightweight and can be scaled without loss of quality.
 
 ```html
-<img src="./assets/images/cat.jpg" alt="Unoptimized image of a cat"/> // Non-compliant
+<img src="./assets/images/cat.jpg" alt="Unoptimized image of a cat" /> //
+Non-compliant
 
-<img src="./assets/images/cat.webp" alt="Optimized image of a cat"/> // Compliant
+<img src="./assets/images/cat.webp" alt="Optimized image of a cat" /> //
+Compliant
 ```
 
 Remember that the best image format may vary depending on the specific use case, content, and requirements of your website.
@@ -43,7 +45,7 @@ Format importantly impacts images size: on average, .webp images will be 30% lig
 images or .png images. .avif images can be up to 20% lighter than .webp image and 50% lighter than .jepg images.
 
 Don't forget to pay attention to browser support. .webp images will not be recognized by
-old browsers and will not be displayed. It is possible to provide several formats for the same image 
+old browsers and will not be displayed. It is possible to provide several formats for the same image
 to overcome this issue. Some server-side modules (such as Google's modPageSpeed, also available for Apache
 and Nginx) even allow you to provide the appropriate image for the browser that is calling the server.
 
@@ -58,13 +60,13 @@ Many tools will help you minimize images size:
 ### Example
 
 In this example, the DOM <picture> element informs the browser that there are two images: a .webp image and a
-.jpg image, which is used by default. The browser will decide which image will be downloaded. If the .webp format 
+.jpg image, which is used by default. The browser will decide which image will be downloaded. If the .webp format
 is supported, the image.webp image will be downloaded; otherwise, image.jpg image will be downloaded.
 
 ```html
 <picture>
-    <source srcset="image.webp" type="image/webp">
-    <img src="image.jpg" alt="..." loading="lazy">
+  <source srcset="image.webp" type="image/webp" />
+  <img src="image.jpg" alt="..." loading="lazy" />
 </picture>
 ```
 
@@ -76,25 +78,29 @@ To address this issue, you can supply multiple formats for the same image.
 
 ### Documentation
 
-- https://github.com/cnumr/best-practices/blob/main/chapters/BP_080_en.md[CNUMR best practices] - Optimize images
+- [CNUMR best practices](https://github.com/cnumr/best-practices/blob/main/chapters/BP_080_en.md) - Optimize images
 
 ### Articles & blog posts
 
-- https://greenspector.com/en/which-image-format-to-choose-to-reduce-its-energy-consumption-and-its-environmental-impact/[greenspector.com - Which image format choose to reduce energy consumption and environmental impact?]
-- https://dodonut.com/blog/use-cases-of-web-image-formats/[dodonut.com - The Most Efficient Web Image Formats. Use Cases For Different Types Of Images.]
+- [greenspector.com - Which image format choose to reduce energy consumption and environmental impact?](https://greenspector.com/en/which-image-format-to-choose-to-reduce-its-energy-consumption-and-its-environmental-impact/)
+- [dodonut.com - The Most Efficient Web Image Formats. Use Cases For Different Types Of Images.](https://dodonut.com/blog/use-cases-of-web-image-formats/)
 
 ### WSG
+
 - [UX15-2: Optimizing All Image Assets for a Variety of Different Resolutions](https://w3c.github.io/sustyweb/star.html#UX15-2)
 
 ### RGESN 5.1
 
 - https://ecoresponsable.numerique.gouv.fr/publications/referentiel-general-ecoconception/critere/5.1/
 
 #### Objective
+
 Reduce the size of files downloaded by users.
 
 #### How to implement
+
 Choose the image format best suited to the type of image and display context: use vector formats such as svg whenever possible (illustrations, icons, logos, graphs, etc.), jpg for photos and png for illustrations with solid colors.
 
 #### Means of test or control
-Evaluate the relevance of the image format displayed.
+
+Evaluate the relevance of the image format displayed.

eslint-plugin/lib/rules/prefer-lighter-formats-for-image-files.js

@@ -28,14 +28,14 @@ module.exports = {
       recommended: "warn",
     },
     messages: {
-      DefineFormatsForImageFiles:
-        "You should define a format for your image files and use light formats such as {{ eligibleExtensions }}",
       PreferLighterFormatsForImageFiles:
-        "You should use lighter formats for image files such as : {{ eligibleExtensions }}",
+        "You should use lighter formats for image files such as {{ eligibleExtensions }}",
     },
     schema: [],
   },
   create(context) {
+    const eligibleExtensions = ["webp", "avif", "svg", "jxl"];
+
     return {
       JSXOpeningElement(node) {
         const tagName = node.name.name;
@@ -44,8 +44,6 @@ module.exports = {
         const parentTagName = node.parent?.parent?.openingElement?.name?.name;
         if (parentTagName?.toLowerCase() === "picture") return;
 
-        const eligibleExtensions = ["webp", "avif", "svg", "jxl"];
-
         const srcAttribut = node.attributes.find(
           (attr) => attr.name.name === "src",
         );
@@ -57,14 +55,7 @@ module.exports = {
         srcValue = srcValue.substring(srcValue.lastIndexOf("/") + 1);
         const dotIndex = srcValue.lastIndexOf(".");
 
-        if (dotIndex === -1) {
-          context.report({
-            node,
-            messageId: "DefineFormatsForImageFiles",
-            data: { eligibleExtensions: eligibleExtensions.join(", ") },
-          });
-          return;
-        }
+        if (dotIndex === -1) return;
 
         const imgExtension = srcValue.substring(dotIndex + 1);
 

eslint-plugin/tests/lib/rules/prefer-lighter-formats-for-image-files.js

@@ -38,10 +38,7 @@ const ruleTester = new RuleTester({
     },
   },
 });
-const defineFormatsForImageFilesError = {
-  messageId: "DefineFormatsForImageFiles",
-  type: "JSXOpeningElement",
-};
+
 const preferLighterFormatsForImageFilesError = {
   messageId: "PreferLighterFormatsForImageFiles",
   type: "JSXOpeningElement",
@@ -82,23 +79,5 @@ ruleTester.run("prefer-lighter-formats-for-image-files", rule, {
       `,
       errors: [preferLighterFormatsForImageFilesError],
     },
-    {
-      code: `
-        <img src="./assets/images/cat" alt="A cat"/>
-      `,
-      errors: [defineFormatsForImageFilesError],
-    },
-    {
-      code: `
-        <img src="assets/images.dir/cat" alt="A cat"/>
-      `,
-      errors: [defineFormatsForImageFilesError],
-    },
-    {
-      code: `
-        <img src="./assets/images.dir/cat" alt="A cat"/>
-      `,
-      errors: [defineFormatsForImageFilesError],
-    },
   ],
 });

sonar-plugin/src/main/resources/org/greencodeinitiative/creedengo/profiles/javascript_profile.json

@@ -9,6 +9,7 @@
     "GCI26",
     "GCI29",
     "GCI30",
+    "GCI31",
     "GCI36",
     "GCI523",
     "GCI530"