[BUG] Convert HTML to Markdown Unexpected token '||=' #6220

[lcaresia]

WHAT

🤖 Generated by Copilot at 1824e67

This pull request fixes a dependency issue and updates the convert-html-to-markdown action in the formatting component. It pins the jsdom version to 21.1.1 and uses consistent import syntax across the component files.

🤖 Generated by Copilot at 1824e67

Oh we're the crew of the formatting ship, and we work with code and yarn
We've pinned the jsdom to twenty-one, and we've raised the action barn
Heave away, me hearties, heave away with me
We've fixed the dependency issue, and we've made the markdown free

WHY

HOW

🤖 Generated by Copilot at 1824e67

• Update jsdom dependency to 21.1.1 and use alias syntax to avoid conflict with other components (link, link, link, link, link)
• Remove unused omniconvert component from pnpm-lock.yaml file (link)

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
pipedream-docs	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	May 10, 2023 4:09pm
pipedream-docs-redirect-do-not-edit	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	May 10, 2023 4:09pm

[alysonturing]

We had to set a fixed version of JSDOM because the newer version doesn't work with Node 14;

Following Pipedream's docs, we imported the package on the code like this:
import jsdom from "jsdom@21.1.1";

And following this StackOverflow answer https://stackoverflow.com/a/56495651
We used an alias for the import on the package.json like this:
"jsdom@21.1.1": "npm:jsdom@21.1.1",

Result:

[GTFalcao]

testing permission to merge - ignore this approval

Was testing permission to merge without all checks passing

[dannyroosevelt]

@lcaresia what is the status of this?

[lcaresia]

@lcaresia what is the status of this?

@andrewjschuang are working on it

[andrewjschuang]

@dannyroosevelt I'm working with @lcaresia and @alysonturing to see if we can make the GH Action Check pass without ignoring the pnpm cache so the workflow time execution is not compromised. We are doing this to enable the usage of pinned versions of npm packages.

[alysonturing]

This was required to use linkedom package
reference

[GTFalcao]

Looks good to me!

[vunguyenhung]

Hi, I see that the bug is fixed

[lcaresia]

/approve