Skip to content

Improve error message for missing franc-min dependency#60

Draft
Copilot wants to merge 4 commits intoclaude/configurable-news-reader-G1Gdxfrom
copilot/sub-pr-57-another-one
Draft

Improve error message for missing franc-min dependency#60
Copilot wants to merge 4 commits intoclaude/configurable-news-reader-G1Gdxfrom
copilot/sub-pr-57-another-one

Conversation

Copy link
Contributor

Copilot AI commented Feb 5, 2026

The error handler for missing franc-min logged "franc-min not available, using feed-declared language only" without telling users how to fix it.

Changes

  • Enhanced error message to include actionable resolution steps
  • Message now explicitly instructs users to run npm install when the package is missing
} catch (e) {
  console.error('WARNING: franc-min package not found. Language detection will be limited to feed-declared languages only.');
  console.error('To enable automatic language detection, please run: npm install');
  detectLang = () => null;
}

This addresses the review feedback that error messages should be informative and include instructions for resolution.


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI and others added 3 commits February 5, 2026 19:10
Co-authored-by: EthanThePhoenix38 <103653068+EthanThePhoenix38@users.noreply.github.com>
Co-authored-by: EthanThePhoenix38 <103653068+EthanThePhoenix38@users.noreply.github.com>
…ting

Co-authored-by: EthanThePhoenix38 <103653068+EthanThePhoenix38@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix issues based on feedback for news reader implementation Improve error message for missing franc-min dependency Feb 5, 2026
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

Successfully merging this pull request may close these issues.

2 participants