Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Clipper failing with parse error - Error("Failed to parse article"); #8

Open
ArlindNocaj opened this issue May 14, 2024 · 1 comment

Comments

@ArlindNocaj
Copy link

I tried the following command:

clipper clip -u https://aws-experience.com/emea/dach-cee/e/566ad/aws-deep-dive-days---generative-ai

It fails with:

 clipper clip -u https://aws-experience.com/emea/dach-cee/e/566ad/aws-deep-dive-days---generative-ai
/Users/arlnocaj/.nvm/versions/node/v18.16.1/lib/node_modules/@philschmid/clipper/dist/clipper.js:59
        throw new Error("Failed to parse article");
              ^

Error: Failed to parse article
    at extract_from_dom (/Users/arlnocaj/.nvm/versions/node/v18.16.1/lib/node_modules/@philschmid/clipper/dist/clipper.js:59:15)
    at extract_from_url (/Users/arlnocaj/.nvm/versions/node/v18.16.1/lib/node_modules/@philschmid/clipper/dist/clipper.js:86:12)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async Command.<anonymous> (/Users/arlnocaj/.nvm/versions/node/v18.16.1/lib/node_modules/@philschmid/clipper/dist/index.js:44:15)

Node.js v18.16.1

It seems that the readability plugin from mozialla is failing on that page. Ideally it should still give the output, with a warning that readability might not be optimal.

@TechupBusiness
Copy link

For such cases maybe a fallback mechanism to an LLM (e.g. local ollama) for markdown conversion could be helpful.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants