fixed address error and wrapped up big portion of the PR

actor/deadletter_test.go

@@ -3,13 +3,14 @@ package actor
 import (
 	"bytes"
 	"fmt"
-	"github.com/anthdm/hollywood/log"
-	"github.com/stretchr/testify/assert"
 	"log/slog"
 	"os"
 	"sync"
 	"testing"
 	"time"
+
+	"github.com/anthdm/hollywood/log"
+	"github.com/stretchr/testify/assert"
 )
 
 // TestDeadLetterDefault tests the default deadletter handling.
@@ -49,6 +50,8 @@ func TestDeadLetterCustom(t *testing.T) {
 	e.Poison(a1).Wait() // poison the a1 actor
 	// should be in deadletter
 	fmt.Println("==== sending message via a1 to deadletter ====")
+	fmt.Println(e.Registry)
+	fmt.Println("ID=> ", dl.PID())
 	e.Send(a1, testMessage{"bar"})
 	time.Sleep(time.Millisecond) // a flush would be nice here :-)
 	resp, err := e.Request(dl.PID(), &customDeadLetterFetch{flush: true}, time.Millisecond*10).Result()

actor/engine.go

@@ -1,6 +1,8 @@
 package actor
 
 import (
+	"log/slog"
+	reflect "reflect"
 	"sync"
 	"time"
 
@@ -37,10 +39,10 @@ type Engine struct {
 func NewEngine(opts ...func(*Engine)) *Engine {
 	e := &Engine{}
 	e.Registry = newRegistry(e) // need to init the registry in case we want a custom deadletter
+	e.address = LocalLookupAddr
 	for _, o := range opts {
 		o(e)
 	}
-	e.address = LocalLookupAddr
 	if e.remote != nil {
 		e.address = e.remote.Address()
 	}
@@ -157,8 +159,12 @@ func (e *Engine) send(pid *PID, msg any, sender *PID) {
 		return
 	}
 	if e.remote == nil {
-		e.logger.Errorw("failed sending messsage",
-			"err", "engine has no remote configured")
+		slog.Error("failed sending messsage",
+			"err", "engine has no remote configured",
+			"to", pid,
+			"type", reflect.TypeOf(msg),
+			"msg", msg,
+		)
 		return
 	}
 	e.remote.Send(pid, msg, sender)