Fix: Use nils for presence checking

Signed-off-by: Albert Callarisa <[email protected]>

Author: acroca

backend/runtimestate/runtimestate.go

@@ -12,6 +12,7 @@ import (
 	"github.com/dapr/durabletask-go/api"
 	"github.com/dapr/durabletask-go/api/helpers"
 	"github.com/dapr/durabletask-go/api/protos"
+	"github.com/dapr/kit/ptr"
 )
 
 var ErrDuplicateEvent = errors.New("duplicate event")
@@ -143,20 +144,20 @@ func ApplyActions(s *protos.OrchestrationRuntimeState, customStatus *wrapperspb.
 					// Create a router for the completion event that routes back to the parent
 					var completionRouter *protos.TaskRouter
 					if action.Router != nil {
-						var parentAppID string
+						var parentAppID *string
 
 						allEvents := append(s.OldEvents, s.NewEvents...)
 						for _, event := range allEvents {
 							if es := event.GetExecutionStarted(); es != nil && event.GetRouter() != nil {
-								parentAppID = event.GetRouter().GetSource()
+								parentAppID = ptr.Of(event.GetRouter().GetSource())
 								break
 							}
 						}
 
-						if parentAppID != "" {
+						if parentAppID != nil {
 							completionRouter = &protos.TaskRouter{
 								Source: action.Router.Source,
-								Target: &parentAppID,
+								Target: parentAppID,
 							}
 						} else {
 							completionRouter = action.Router

task/orchestrator.go

@@ -216,7 +216,7 @@ func (ctx *OrchestrationContext) processEvent(e *backend.HistoryEvent) error {
 			router := e.GetRouter()
 			// For cross-app suborchestrations, if we have a target, use that as our appID
 			// since that's where we're actually executing
-			if router.GetTarget() != "" {
+			if router.Target != nil {
 				ctx.appID = ptr.Of(router.GetTarget())
 			} else {
 				ctx.appID = ptr.Of(router.GetSource())