Trelent / Trelent-IntelliJ-Extension

JetBrains Plugin for Trelent
https://plugins.jetbrains.com/plugin/18716-trelent--ai-docstrings-on-demand
8 stars 1 forks source link

'We do not support this language yet' error when trying to use this on C# code within a Unity project - not sure if bug or undocumented intentional behaviour #1

Closed 11BelowStudio closed 2 years ago

11BelowStudio commented 2 years ago

So, just noticed your plugin on the Jetbrains plugin page, and decided to install it for Rider (2022.1), attempted to use it to generate a docstring for a C# file (within a Unity project), and was greeted with the 'We do not support this language yet' error, despite C# apparently being supported.

Not sure if this was an oversight (maybe Rider internally treats C# files within a Unity project as different to normal C# files, causing your tool to not realize that they are C# files), or if this was intentional (like if you're planning on treating C# files within a Unity project differently to ordinary C# files).

Please could you either update the plugin to treat Unity C# files as C# files, or update the description of the plugin to explicitly state that C# files within Unity projects are unsupported? Thanks.

calum-bird commented 2 years ago

Hey there, thank you for reporting this. First of all, it's very cool to see Trelent used in the game development space!

To answer your question, this is completely unintentional, and I apologize for the inconvenience this may have caused. I will try to reproduce this tonight, and include a fix in the next build. This should be going out later this week/weekend.

Thank you for your patience!

calum-bird commented 2 years ago

Writing again to confirm I found the bug and it will be patched in the next release. We were incorrectly retrieving the language in the opened file, which was returning "C#" instead of "csharp". Fix is live internally, just waiting on a couple bugs to be fixed prior to the next release.

calum-bird commented 2 years ago

Hi again! Just wanted to drop in and say that we are just awaiting JetBrains approval for the next release to go live. In the meantime you can install Version 1.7.0 (this release) ahead of approval if you would like. I've attached the latest build below.

Thanks again for reporting this bug!

Trelent - AI Docstrings on Demand-1.7.0.zip