flowmade-one/aether
3
0
Fork 0
Code Issues 5 Pull Requests 7 Actions Packages Projects Releases Wiki Activity

[Issue #37] Replace interface{} with properly defined interfaces #42

Merged
HugoNijhuis merged 2 commits from issue-37-replace-interface into main 2026-01-10 17:46:12 +00:00
Conversation 4 Commits 2 Files Changed 3
+142 -43
2 Commits
Author SHA1 Message Date
HugoNijhuisandClaude Opus 4.5 b759c7fb97 Fix type assertion bug in handleClusterMessage
CI / build (pull_request) Successful in 16s
Details 
JSON unmarshal produces map[string]interface{}, not concrete types.
Added ModelPayload and MessagePayload concrete types that implement
RuntimeModel and RuntimeMessage interfaces respectively.

The handleClusterMessage now re-marshals and unmarshals the payload
to convert from map[string]interface{} to the proper concrete type.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
 2026-01-10 16:45:28 +01:00
HugoNijhuisandClaude Opus 4.5 eaff315782 Replace interface{} with properly defined interfaces 
- Add VirtualMachine interface with GetID(), GetActorID(), GetState()
- Add VMState type with idle/running/paused/stopped states
- Add RuntimeModel interface for event storming model contracts
- Add RuntimeMessage interface for actor message contracts
- Add VMProvider interface for decoupled VM access
- Update VMRegistry.GetActiveVMs() to return map[string]VirtualMachine
- Update Runtime interface to use RuntimeModel and RuntimeMessage
- Update DistributedVMRegistry to use VMProvider instead of interface{}
- Add SetVMProvider method to DistributedVM for dependency injection

This improves type safety, makes contracts explicit, and enables better
IDE support while avoiding import cycles through interface segregation.

Closes #37

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
 2026-01-10 16:45:28 +01:00