I just wanted to confirm whether appending "Controller" as a suffix is
simply a naming convention, or if there are any potential consequences
for not following it
Well, yes you are partially correct its a naming convernsion tough. However, In ASP.NET Core, while it is not mandatory for a controller class to have the suffix "Controller," but it is highly recommended to follow this convention. According to the official documentation from Microsoft you can refer here.
If there are any potential consequences for not following it. Even
though my .NET 6 application seems to function without issue
currently.
In general there wouldn't be any issue but if you use few advance level feature like reflection, you might encounter an issue.
Because, the framework uses reflection to discover controller classes. While it can find classes without the "Controller" suffix, using the suffix helps the framework easily identify and differentiate controller classes from other classes. If the suffix is omitted, you need to explicitly mark the class with the [Controller] attribute.
By default, ASP.NET Core searches for classes ending with "Controller" when setting up the MVC middleware.
Apart from that, convention-based routing, filters, and dependency injection, might not work as expected if the controller naming conventions are not followed.
Additionally, if you use scaffolding feature, so scaffolding usually serach for "Controller" suffix while generating views and other files for you may potentially leading to issues in automation and code generation.
Nonetheless, while your current setup can work, it is generally advisable to adhere to the naming convention of appending "Controller" to your controller classes. This helps avoid potential issues and aligns your code with common practices in the ASP.NET Core community. So, it is recommended to rename your class to HomeController for consistency and clarity.