feature/save_and_loaaaaaad #32

2025-11-25T18:00:53+01:00

kziolkowski commented

2025-11-25 18:00:53 +01:00

Implemented and reworked the Save structure based on our first use cases and some pointers from the chat.

General:

Split the save system into two parts: Settings and User Data.
- Settings are for local stuff like window size, sound settings and so on.
- User Data is for game saves that will later be synced via the Steam Cloud (already set up via the folder structure).
Added Versioning to the save data that isn't hooked up for now but will be helpful when migrating in the future
simplified the save structure to a simply "id - string payload" matching to make it easier to maintain and debug
found a way to use Guids for saving: I'm adding a Metadata entry to the top node of each saveable prefab scene and add it to a group called "Saveable". Then in the containing scene, I run a tool node that iterates over all instances in this group and auto-generates the ID where necessary. Needs to be updated manually, but works fine so far.

Use cases implemented and synced via save (triggers on scene transition / restart):

Field and plant states
duck states
inventory state
item pickup state (both unique items and collectable ones)

Known issues:

The first run (when there is no save file yet) sometimes still triggers some ominous error messages. Could not repro last time I checked, but I cannot say for sure if it's gone for good.
There is still some cleaning up (todos) to do in the InteractionArea script, but that's something for another PR
Also there are still things we need to sync like player position, last scene loaded, dialogues played etc, but these will be added in the next batch
Also there are some known bugs with the current state of the dialogue system as a whole (looping, random triggers etc.) which will also be addressed in a different PR

Implemented and reworked the Save structure based on our first use cases and some pointers from the chat. General: * Split the save system into two parts: Settings and User Data. * Settings are for local stuff like window size, sound settings and so on. * User Data is for game saves that will later be synced via the Steam Cloud (already set up via the folder structure). * Added Versioning to the save data that isn't hooked up for now but will be helpful when migrating in the future * simplified the save structure to a simply "id - string payload" matching to make it easier to maintain and debug * found a way to use Guids for saving: I'm adding a Metadata entry to the top node of each saveable prefab scene and add it to a group called "Saveable". Then in the containing scene, I run a tool node that iterates over all instances in this group and auto-generates the ID where necessary. Needs to be updated manually, but works fine so far. ![image](/attachments/baee9c14-0c04-4671-88b6-741cfea5ce00) ![image](/attachments/4533d422-c982-4116-8782-fe0077f5f4cd) ![image](/attachments/07c0d5a3-054d-403a-8dd2-c06d333b83fa) Use cases implemented and synced via save (triggers on scene transition / restart): * Field and plant states * duck states * inventory state * item pickup state (both unique items and collectable ones) Known issues: * The first run (when there is no save file yet) sometimes still triggers some ominous error messages. Could not repro last time I checked, but I cannot say for sure if it's gone for good. * There is still some cleaning up (todos) to do in the InteractionArea script, but that's something for another PR * Also there are still things we need to sync like player position, last scene loaded, dialogues played etc, but these will be added in the next batch * Also there are some known bugs with the current state of the dialogue system as a whole (looping, random triggers etc.) which will also be addressed in a different PR

image.png

7.4 KiB

image.png

12 KiB

image.png

8.5 KiB

kziolkowski added 18 commits 2025-11-25 18:00:54 +01:00

🚧 WIP implementing save and load for the inventory 72079044bd

✨ Added Save and load support to the inventory 2a4425b9db

working intermediate state of the save system 2947011a1a

❇️ Added Settings classes to save and load local configurations ce29711614

Reworked existing save system for user data: Made path steam-compatible, added Versioning and try/catch to save 4ed6f4e7d9

Created ISaveable interface and made existing saveable classes inherit from it. 4bbaab1a2a

Savesystem rework fda2b95b38

🚧 implemented interaction area loading bee6249c6f

🚧 intermediate state with homework c68727aebe

:feature: implemented saving and loading for ducks, 🔥 removed save implementation from interactionArea c695115a4a

:feature: made GetComponent nullable in NodeExtension b4013b1ff2

📝 Added documentation 638ebaff46

♻️ reworked and debugged the SaveSystem. Removed unnecessary scene reference. e9cd4ce276

♻️ refactored field code to work with the new id system 9d818e5079

♻️ inventory works 88f3c90eec

✨ Implemented load check for unique inventory items 70383fc16e

✨ added saving and loading support to collectable inventory items f37f7c7ceb

♻️ removed Print statements 75e4bdd7fd

kziolkowski requested review from Jonathan 2025-11-25 18:01:40 +01:00

Jonathan approved these changes 2025-11-25 19:21:15 +01:00

Jonathan left a comment

Nur eine kleinigkeit

scripts/CSharp/Common/CharacterControls/InteractionArea2D.cs

						
				@@ -18,6 +17,7 @@ public partial class InteractionArea2D : Node2D

				    [Export] private int _id = -1; // TODO: remove

				    private Material[] _backupMaterials;

				    private int _interactionCounter;