meilisearch / meilisearch/scrapix

Improve errors logs

Open
#34 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

enhancement
Dominant language
TypeScript
Stars
32
Forks
9
PR merge metrics
No merged PRs in 30d

Description

In most of the places were errors are silently ignored, a console.log compensate with some information on it. These are not always very clear.

Exemple in Senders.ts where it is not clear that there was no tmp index to delete.

catch (e) {
 console.log("try to delete a tmp index if it exists");
}

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by reviewing the error-handling locations that silently ignore errors and the example in Senders.ts. Identify which console.log messages lack clear context, then make the affected logs explain what happened, including the temporary-index deletion case. Done means the relevant ignored-error logs are understandable to someone reading them.

Written by the indexing model from the issue text.

Assessment

Tech stack
typescript
Domain
backend
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.