Skip to content

Conversation

harshgupta2125
Copy link

feat(sites): add Ko‑fi, StackBlitz, Modrinth

Summary

Add three new platforms to Sherlock: Ko‑fi, StackBlitz, Modrinth. Detection uses status_code: 404 with a browser-like User-Agent. Verified locally. Part of Hacktoberfest 2025.

Sites Added

Verification

Claimed examples:

Run locally with your data.json:

python -m sherlock_project stackblitz --site StackBlitz --local
python -m sherlock_project modrinth --site Modrinth --local
python -m sherlock_project <ko_fi_handle> --site "Ko-fi" --local
python -m sherlock_project thisuserdoesnotexist12345 --site "Ko-fi" --site StackBlitz --site Modrinth --local

Checks

  • JSON valid and loads
  • Claimed links resolve
  • Unclaimed returns not found
  • Ruff passes (or tox -e lint)
  • pytest passes

Copy link
Contributor

github-actions bot commented Oct 8, 2025

Automatic validation of changes

Target F+ Check F- Check
Modrinth ✔️   Pass ❌   Fail
StackBlitz ✔️   Pass ❌   Fail
AtCoder ✔️   Pass ❌   Fail
Ko-fi ✔️   Pass ❌   Fail

Failures were detected on at least one updated target. Commits containing accuracy failures will often not be merged (unless a rationale is provided, such as false negatives due to regional differences).

@akh7177
Copy link
Contributor

akh7177 commented Oct 8, 2025

@harshgupta2125 Atcoder is already present in data.json. Moreover, all the F- tests seem to be failing. Kindly make relevant changes only in data.json and keep the commit tree clean

Thanks

Copy link
Contributor

github-actions bot commented Oct 8, 2025

Automatic validation of changes

Target F+ Check F- Check
Ko-fi ❌   Fail ❌   Fail
AtCoder ✔️   Pass ❌   Fail
Modrinth ✔️   Pass ❌   Fail
StackBlitz ✔️   Pass ❌   Fail

Failures were detected on at least one updated target. Commits containing accuracy failures will often not be merged (unless a rationale is provided, such as false negatives due to regional differences).

Copy link
Contributor

github-actions bot commented Oct 8, 2025

Automatic validation of changes

Target F+ Check F- Check
AtCoder ✔️   Pass ❌   Fail
Ko-fi ❌   Fail ❌   Fail
Modrinth ✔️   Pass ❌   Fail
StackBlitz ✔️   Pass ❌   Fail

Failures were detected on at least one updated target. Commits containing accuracy failures will often not be merged (unless a rationale is provided, such as false negatives due to regional differences).

Copy link
Contributor

github-actions bot commented Oct 8, 2025

Automatic validation of changes

Target F+ Check F- Check
Ko-fi ❌   Fail ❌   Fail
Modrinth ✔️   Pass ✔️   Pass
StackBlitz ✔️   Pass ✔️   Pass

Failures were detected on at least one updated target. Commits containing accuracy failures will often not be merged (unless a rationale is provided, such as false negatives due to regional differences).

Copy link
Contributor

github-actions bot commented Oct 8, 2025

Automatic validation of changes

Target F+ Check F- Check
Modrinth ✔️   Pass ✔️   Pass
StackBlitz ✔️   Pass ✔️   Pass
Ko-fi ❌   Fail ❌   Fail

Failures were detected on at least one updated target. Commits containing accuracy failures will often not be merged (unless a rationale is provided, such as false negatives due to regional differences).

Copy link
Contributor

github-actions bot commented Oct 8, 2025

Automatic validation of changes

Target F+ Check F- Check
Modrinth ✔️   Pass ✔️   Pass
StackBlitz ✔️   Pass ✔️   Pass

@harshgupta2125
Copy link
Author

@harshgupta2125 Atcoder is already present in data.json. Moreover, all the F- tests seem to be failing. Kindly make relevant changes only in data.json and keep the commit tree clean

Thanks

well i removed atcoder coz it already presents.. and removed ko-fi also because it get many errors and without give relevent socials because of signup policy, now all set.. you can merge it now

Copy link
Member

@ppfeister ppfeister left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A lot of misc. files are committed and not to be merged into master. If you could, please set to the merge-base and commit only the additional sites (ideally, without the whitespace between them)

If a new PR is easier than a force push, that's fine as well

Thanks!

Copy link
Contributor

Automatic validation of changes

Target F+ Check F- Check
Ifunny ✔️   Pass ✔️   Pass
MuseScore ✔️   Pass ✔️   Pass
StackBlitz ✔️   Pass ✔️   Pass
Vjudge ✔️   Pass ❌   Fail
imood ✔️   Pass ✔️   Pass
kaskus ✔️   Pass ✔️   Pass
Modrinth ✔️   Pass ✔️   Pass
Status Cafe ✔️   Pass ✔️   Pass
Discord.bio ✔️   Pass ✔️   Pass

Failures were detected on at least one updated target. Commits containing accuracy failures will often not be merged (unless a rationale is provided, such as false negatives due to regional differences).

Copy link
Contributor

Automatic validation of changes

Target F+ Check F- Check
Vjudge ✔️   Pass ❌   Fail
Discord.bio ✔️   Pass ✔️   Pass
kaskus ✔️   Pass ✔️   Pass
Ifunny ✔️   Pass ✔️   Pass
MuseScore ✔️   Pass ✔️   Pass
imood ✔️   Pass ✔️   Pass
Status Cafe ✔️   Pass ✔️   Pass
StackBlitz ✔️   Pass ✔️   Pass
Modrinth ✔️   Pass ✔️   Pass

Failures were detected on at least one updated target. Commits containing accuracy failures will often not be merged (unless a rationale is provided, such as false negatives due to regional differences).

Copy link
Contributor

Automatic validation of changes

Target F+ Check F- Check
Ifunny ✔️   Pass ✔️   Pass
Discord.bio ✔️   Pass ✔️   Pass
Vjudge ✔️   Pass ❌   Fail
kaskus ✔️   Pass ✔️   Pass
Status Cafe ✔️   Pass ✔️   Pass
Modrinth ✔️   Pass ✔️   Pass
imood ✔️   Pass ✔️   Pass
StackBlitz ✔️   Pass ✔️   Pass
MuseScore ✔️   Pass ✔️   Pass

Failures were detected on at least one updated target. Commits containing accuracy failures will often not be merged (unless a rationale is provided, such as false negatives due to regional differences).

@harshgupta2125
Copy link
Author

@ppfeister with all my trying i add and removed all the errors from my contribution of the two webssites but i commit and then found that their is still one error which is not by my side because i fetch and pull all the latest branch and then work on it and it succesfully worked, i tried to solve this but it cant be resolve coz one again thats not by my side so hope you'll understand and merged this pr into main OR if you find out where and how the problem is occuring then tell me i can do all fix it.

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.

3 participants