.NET 8 Support and Dependency Fixes
Hi, I briefly had a look at some of the changes and it mostly looks good. Due to the size of the PR I need some time to review it thoroughly. Right now I'm quite busy.
Are you done with your changes though?
Hi, aren't we all... busy ... 😉 And... "mostly"?? 😁
Anyway, I found some very minor issues, would like to fix them and then be done. Hopefully this week. Btw. would you mind if I put it on Nexusmods? Ofcourse with proper credit. I think license is MIT, but still..
Btw. would you mind if I put it on Nexusmods? Ofcourse with proper credit. I think license is MIT, but still..
The project is already on NexusMods and Thunderstore: https://www.nexusmods.com/sonsoftheforest/mods/41 https://thunderstore.io/c/sons-of-the-forest/p/codengine/SOTFEdit/
Ahh, right, it's actually a cheating mod! I normally filter on no cheating. 🙂
Can you split the changes into multiple MRs? It makes reviewing and bisecting easier.
Hi Stefan,
I am done with the game and my motivation to continue updating the code is quite zero, sorry about that.
But first of all thanks for the great tool, I almost exklusively used it for the map feature. In my opinion it's THE feature, even more with my enhanced "Done" (And I checked other tools and websites). Depends how one plays I suppose.
To be honest, I hardly looked at the code, this was more of a getting to know how good AI programming with VSCode works. Did partly quite well, it would have needed more guidance in places. Partly it was frustrating. That was also one of the reasons I went for yaml. Having multiple changes, trying to keep the languages in synch plus a tiny error in the structure can be a real nightmare to fix with a deeply nested json like that.
Language hot swapping works quite well, made some further enhancements in the latest edits. Not sure what you mean the issue is or could be.
Sorry about the many changes, I don't normally with pull requests. It apparently kept adding commits to the request. This is probably what you would have to do, make a cherry pick on each. I tried to keep the commits meaningful, but I did not do any nice feature branching, so it's not too great. Maybe just take it all and fix what you don't like. As I used it as a permanent companion and I was fine I don't think it's too bad.
Only one thing, the last commit is a bit rough. Tried fixing the Autoload, but could not get it to work. I was tempted to take a deeper code dive, but I just finished the game (and the end game got me very annoyed), so... no. 😔
You know, if you want to completely just ignore my changes, it's fine with me. I myself got pretty much what interested me. Was a bit more effort then I expected. I asume you might also have lost interest a while ago, and the game is pretty much past it's prime? But the implementation is surely a nice reference for future tools. 🙂
Kind regards Alexander
On December 4, 2025 8:05:19 AM GMT+01:00, Stefan Hueg @.***> wrote:
codengine left a comment (codengine/SOTFEdit#44)
Can you split the changes into multiple MRs? It makes reviewing and bisecting easier.
-- Reply to this email directly or view it on GitHub: https://github.com/codengine/SOTFEdit/pull/44#issuecomment-3610608095 You are receiving this because you authored the thread.
Message ID: @.***>
@Za-Pa-Al First of all, thank you for your interest and the kind words for the editor. It was a passion-driven project and it was quite fun to work on the map and figure out all the things that make it work.
However, as you already mentioned, my passion has soon dropped because Endnight stopped working on the game and to be honest - the more I worked at the editor, the less I played the game because it lost its "magic".
I already suspected that the changes were Ai driven, also hinted by the specific Claude settings. Which, to be honest, leads to trust issues on my end. Not specific because of your PR, but because everyone can create such PRs which look good on the surface but won't deliver thoroughly planned and executed changes. Its too tempting. But thank you for your honesty at least.
I will have a look which parts can be reused.
Hi,
I have been in IT and developing in many languages for many years, only recently I have start playing around with AI driven approaches. I must say, having AI support does significantly lower the inhibition threshold to even attempt to jump into a foreign project and do some changes "fast", without knowing the project at all.
I understand your trust issues, and I also don't like other people messing up my code. ☺ In this case I am pretty sure I never even wrote a single line myself, I find it amazing how much AI can do already, and the code pieces it shows in diff seem quite ok. Having experience in software engineering likely helped in supporting it. But it can get very, very lost. I never gave into looking at the code though, I just told Claude, it does not work, fix it. Maybe suggest a different approach ... ☺
If you ever get around to verifying the changes, please drop me a little feedback ... good, ok, or a total disaster. As I said, I think it did not perform horribly, actually it even did quite ok. Most of the changes were obviously straight forward, but maybe it is really only on the surface.
Regards, Alexander
From: Stefan Hueg @.***>
Sent: Thursday, December 4, 2025 at 18:38 UTC+1
To: codengine/SOTFEdit @.***>
Cc: Za-Pa-Al @.>, Mention @.>
Subject: RE: [codengine/SOTFEdit] .NET 8 Support and Dependency Fixes (PR #44)
codengine left a comment (codengine/SOTFEdit#44) https://github.com/codengine/SOTFEdit/pull/44#issuecomment-3613479075
@Za-Pa-Al https://github.com/Za-Pa-Al First of all, thank you for your interest and the kind words for the editor. It was a passion-driven project and it was quite fun to work on the map and figure out all the things that make it work.
However, as you already mentioned, my passion has soon dropped because Endnight stopped working on the game and to be honest - the more I worked at the editor, the less I played the game because it lost its "magic".
I already suspected that the changes were Ai driven, also hinted by the specific Claude settings. Which, to be honest, leads to trust issues on my end. Not specific because of your PR, but because everyone can create such PRs which look good on the surface but won't deliver thoroughly planned and executed changes. Its too tempting. But thank you for your honesty at least.
I will have a look which parts can be reused.
— Reply to this email directly, view it on GitHub https://github.com/codengine/SOTFEdit/pull/44#issuecomment-3613479075, or unsubscribe https://github.com/notifications/unsubscribe-auth/BYB5MRJCFHVLLTB6OOKPIZ34ABWRJAVCNFSM6AAAAACK3S4ZLSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTMMJTGQ3TSMBXGU. You are receiving this because you were mentioned.Message ID: @.***>
I'll update it to .net8.0, but differently. Thanks anyway for your interest in the project :)