Skip to content

Conversation

@MichaelHinrichs
Copy link
Contributor

SteamDB app page links to a few games using this

https://steamdb.info/app/3404520

Brief explanation of the change

This game, and likely others, have only this library as evidence that they use Godot.

@SteamDB-Tracker

This comment was marked as outdated.

@SteamDB-Tracker
Copy link
Member

After running changed rules in this pull request (this bot only tests rules.ini changes):

Changed technologies: Engine.Godot

3189 apps still match.

These apps now match: (15) 523630 1876280 2200690 2800490 3144730 3606900 3719900 3884910 3891190 3984130 4002180 4041270 4071380 4114490 4142580

Please confirm that all these changed apps are correct.

Copy link
Contributor

@akien-mga akien-mga left a comment

Choose a reason for hiding this comment

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

Seems ok, though it's pretty niche.

Godot Boy seems to be a GameBoy emulator extension for Godot, which basically lets users ship GameBoy games (ROMs I assume) using Godot. I assume most of those first-party games are made with GBStudio and thus it would likely make more sense for these games to be detected as made with GB Studio (but there's no marker of it in the depots), at least alongside Godot.

But I'm not against merging this, technically those games do use Godot in some capacity.

Comment on lines +119 to +120
Godot[] = (?:^|/)godot_boy\.windows\.x86_64\.dll$
Godot[] = (?:^|/)libgodot_boy\.linux\.x86_64\.so$
Copy link
Contributor

Choose a reason for hiding this comment

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

Could be simplified like the above checks:

Suggested change
Godot[] = (?:^|/)godot_boy\.windows\.x86_64\.dll$
Godot[] = (?:^|/)libgodot_boy\.linux\.x86_64\.so$
Godot[] = (?:^|/)(?:lib)godot_boy

Copy link
Contributor

Choose a reason for hiding this comment

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

That being said if this can use globbing it might make sense to make it more specific like:

Suggested change
Godot[] = (?:^|/)godot_boy\.windows\.x86_64\.dll$
Godot[] = (?:^|/)libgodot_boy\.linux\.x86_64\.so$
Godot[] = (?:^|/)(?:lib)godot_boy.*\.(?:dll|dylib|so)$

and likewise for godotsteam and steamsdk-godot extension libraries.

Comment on lines +1388 to +1391
godot_boy.linux.x86_64.dll
libgodot_boy.windows.x86_64.so
godot_boy.windows.x86_64
godot_boy.linux.x86_64.so
Copy link
Contributor

Choose a reason for hiding this comment

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

With my suggestion above these tests won't be non-matching anymore, but I'm not sure there's much value in testing these which should never be found in the wild (unless someone explicitly renames their libs to confuse us).

@akien-mga
Copy link
Contributor

Please confirm that all these changed apps are correct.

There are a few matches here which are indeed using Godot, but not through this PR.
I assume it's just that it's comparing to a slightly older sync and those games just happen to have had their depot data made available recently?

E.g. https://steamdb.info/app/3606900/depots/ (detected through libgodotsteam) or https://steamdb.info/app/3719900/depots/ (detected through .pck and/or libgodotsteam).

@xPaw
Copy link
Member

xPaw commented Jan 19, 2026

Yeah this doesn't quite seem worth it at the moment.

@xPaw xPaw closed this Jan 19, 2026
@github-project-automation github-project-automation bot moved this from To Triage to Done in File rules Jan 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants