chore: remove tracedSVG (#37093) (#37137)

* chore: remove tracedSVG (#37093)

* init

* gri

* update shopify snapshot

* gatsby-plugin-sharp tests

* update gatsby-remark-images/gatsby-node tests

* update gatsby-plugin-image tests

* update schema print tests

* remove traceSVG unit tests as it was removed

* legacy gatsby-image fields

* update polyfill tests

* update cypress assertion

* update contentful cypress assertions

* drop few more packages from gatsby-plugin-sharp as they are no onger used

* correct IMAGE_CDN fallback for TRACE_SVG

* update generateImageData

* update e2e-prod/assertions

* update unit test

* update snapshot

* update e2e-dev/assertions

* drop more unused

* sync yarn.lock

* a bit more prod warnings

* adjust gatsby-remark-images plugin options warning

* add link to gatsby-remark-images warning

(cherry picked from commit 94c2d73)

* gatsby recipe tests

* cli tests

* babel config tests

* update assertion

* some reverts?

* update assertion

* contentful extend node type

* fallback in test

* fix tracedSVG fields

* fix(gatsby-cli): relax error location validation and ignore extra fields (#34559)

* package.json removals

Co-authored-by: Michal Piechowiak <[email protected]>

Author: gatsbybot

e2e-tests/contentful/cypress/integration/gatsby-image.js

@@ -52,7 +52,9 @@ describe(`gatsby-image`, () => {
   it(`fluid`, testConfig, () => testGatsbyImage(`fluid`, hasBase64Placeholder))
   it(`fixed`, testConfig, () => testGatsbyImage(`fixed`, hasBase64Placeholder))
   it(`webp`, testConfig, () => testGatsbyImage(`webp`, hasBase64Placeholder))
-  it(`traced`, testConfig, () => testGatsbyImage(`traced`, hasSVGPlaceholder))
+  it(`traced`, testConfig, () =>
+    testGatsbyImage(`traced`, hasBase64Placeholder)
+  )
   it(`sqip`, testConfig, () => testGatsbyImage(`sqip`, hasSVGPlaceholder))
 
   it(`english`, testConfig, () => {

e2e-tests/contentful/cypress/integration/gatsby-plugin-image.js

@@ -74,7 +74,7 @@ describe(`gatsby-plugin-image`, () => {
     testGatsbyPluginImage(`dominant-color`, hasColorPlaceholder)
   )
   it(`traced`, testConfig, () =>
-    testGatsbyPluginImage(`traced`, hasSVGPlaceholder)
+    testGatsbyPluginImage(`traced`, hasColorPlaceholder)
   )
   it(`blurred`, testConfig, () =>
     testGatsbyPluginImage(`blurred`, hasBase64Placeholder)

e2e-tests/contentful/cypress/integration/tags.js

@@ -19,7 +19,7 @@ describe(`tags`, () => {
       "have.text",
       "numberInteger"
     )
-    cy.get('[data-cy-id^="tag-"]').should("have.length", 2)
+    cy.get('[data-cy-id^="tag-"]').should("have.length", 5)
   })
   it(`Filtered Entries`, () => {
     cy.get('[data-cy-integers] [data-cy-id="number-integer"]').should(

e2e-tests/contentful/src/pages/gatsby-plugin-image.js

@@ -85,7 +85,9 @@ const GatsbyPluginImagePage = ({ data }) => {
         ))}
       </Grid>
 
-      <h2>gatsby-plugin-image: Traced SVG Placeholder</h2>
+      <h2>
+        gatsby-plugin-image: Traced SVG Placeholder (fallback to DOMINANT_COLOR)
+      </h2>
       <Grid data-cy="traced">
         {data.default.nodes.map(node => (
           <div>

e2e-tests/gatsby-static-image/cypress/integration/traced.js

@@ -5,12 +5,14 @@ describe(`fixed`, () => {
     cy.visit(`/traced`).waitForRouteChange()
   })
 
-  it(`renders a traced svg`, () => {
+  it(`traced svg (falls back to DOMINANT_COLOR)`, () => {
     cy.getTestElement(tracedTestId)
-      .find(`.gatsby-image-wrapper > img`)
-      .should(`have.attr`, `src`)
-      .and(src => {
-        ;[`data:image/svg+xml`].forEach(part => expect(src).to.include(part))
+      .find(`.gatsby-image-wrapper > [data-placeholder-image]`)
+      .first()
+      .should($el => {
+        // traced falls
+        expect($el.prop("tagName")).to.be.equal("DIV")
+        expect($el).to.be.empty
       })
   })
 

examples/using-contentful/package.json

@@ -30,6 +30,7 @@
   "scripts": {
     "develop": "gatsby develop",
     "build": "gatsby build",
+    "clean": "gatsby clean",
     "start": "gatsby serve"
   }
 }

integration-tests/gatsby-cli/__tests__/new.js

@@ -29,7 +29,11 @@ describe(`gatsby new`, () => {
   })
 
   it(`creates a gatsby site with the default starter`, () => {
-    const [code, logs] = GatsbyCLI.from(cwd).invoke([`new`, `gatsby-default`])
+    const [code, logs] = GatsbyCLI.from(cwd).invoke([
+      `new`,
+      `gatsby-default`,
+      `gatsbyjs/gatsby-starter-default#v3`,
+    ])
 
     logs.should.contain(
       `info Creating new site from git: https://github.com/gatsbyjs/gatsby-starter-default.git`
@@ -48,7 +52,7 @@ describe(`gatsby new`, () => {
     const [code, logs] = GatsbyCLI.from(cwd).invoke([
       `new`,
       `gatsby-blog`,
-      `gatsbyjs/gatsby-starter-blog`,
+      `gatsbyjs/gatsby-starter-blog#v3`,
     ])
 
     logs.should.contain(

packages/gatsby-cli/src/structured-errors/error-schema.ts

@@ -1,10 +1,12 @@
 import Joi from "joi"
 import { ILocationPosition, IStructuredError } from "./types"
 
-export const Position: Joi.ObjectSchema<ILocationPosition> = Joi.object().keys({
-  line: Joi.number(),
-  column: Joi.number(),
-})
+export const Position: Joi.ObjectSchema<ILocationPosition> = Joi.object()
+  .keys({
+    line: Joi.number(),
+    column: Joi.number(),
+  })
+  .unknown()
 
 export const errorSchema: Joi.ObjectSchema<IStructuredError> =
   Joi.object().keys({
@@ -27,7 +29,7 @@ export const errorSchema: Joi.ObjectSchema<IStructuredError> =
     location: Joi.object({
       start: Position.required(),
       end: Position,
-    }),
+    }).unknown(),
     docsUrl: Joi.string().uri({
       allowRelative: false,
       relativeOnly: false,

packages/gatsby-plugin-image/src/resolver-utils.ts

@@ -201,9 +201,9 @@ export function getGatsbyImageFieldConfig<TSource, TContext, TArgs>(
         type: ImagePlaceholderType.name,
         description: stripIndent`
             Format of generated placeholder image, displayed while the main image loads.
-            BLURRED: a blurred, low resolution image, encoded as a base64 data URI (default)
-            DOMINANT_COLOR: a solid color, calculated from the dominant color of the image.
-            TRACED_SVG: a low-resolution traced SVG of the image.
+            BLURRED: a blurred, low resolution image, encoded as a base64 data URI.
+            DOMINANT_COLOR: a solid color, calculated from the dominant color of the image (default).
+            TRACED_SVG: deprecated. Will use DOMINANT_COLOR.
             NONE: no placeholder. Set the argument "backgroundColor" to use a fixed background color.`,
       },
       formats: {

packages/gatsby-plugin-sharp/package.json

@@ -17,13 +17,10 @@
     "gatsby-telemetry": "^2.14.0",
     "got": "^11.8.2",
     "lodash": "^4.17.21",
-    "mini-svg-data-uri": "^1.3.3",
-    "potrace": "^2.1.8",
     "probe-image-size": "^6.0.0",
     "progress": "^2.0.3",
     "semver": "^7.3.5",
     "sharp": "^0.29.0",
-    "svgo": "1.3.2",
     "uuid": "3.4.0"
   },
   "devDependencies": {