remove invalid request validation error

Validation error type and status code enforcement should be handled at a 
layer above the adaptor.

deduplicator_test.go

@@ -17,7 +17,7 @@ func TestDeduplicator(t *testing.T) {
 			t.Error(err)
 		}
 		if expect != ok {
-			t.Errorf("deduplication expection was not met: %v vs %v", expect, ok)
+			t.Errorf("expectation not met: %v vs %v", expect, ok)
 		}
 	}
 

error.go

@@ -94,23 +94,6 @@ type InvalidRequestError struct {
 	error
 }
 
-// TODO: add slog.LogValue with more details including "request validation failed"
-func NewInvalidRequestError(fromError error) *InvalidRequestError {
-	return &InvalidRequestError{fromError}
-}
-
-func (e *InvalidRequestError) Error() string {
-	return e.error.Error()
-}
-
-func (e *InvalidRequestError) Unwrap() error {
-	return e.error
-}
-
-func (e *InvalidRequestError) HyperTextStatusCode() int {
-	return http.StatusUnprocessableEntity
-}
-
 type DecodingError struct {
 	error
 }

middleware/session/errors.go

@@ -1,6 +1,9 @@
 package session
 
-import "fmt"
+import (
+	"fmt"
+	"net/http"
+)
 
 type Error uint8
 
@@ -9,6 +12,17 @@ const (
 	ErrLargeCookie
 )
 
+func (e Error) HyperTextStatusCode() int {
+	switch e {
+	case ErrNoSessionInContext:
+		return http.StatusForbidden
+	case ErrLargeCookie:
+		return http.StatusUnprocessableEntity
+	default:
+		return http.StatusInternalServerError
+	}
+}
+
 func (e Error) Error() string {
 	switch e {
 	case ErrNoSessionInContext:

unary.go

@@ -83,7 +83,7 @@ func (a *UnaryFuncAdaptor[T, V, O]) executeDomainCall(
 
 	ctx := r.Context()
 	if err = request.Validate(ctx); err != nil {
-		return NewInvalidRequestError(err)
+		return err
 	}
 
 	response, err := a.domainCall(ctx, request)

void.go

@@ -69,7 +69,7 @@ func (a *VoidFuncAdaptor[T, V]) executeDomainCall(
 
 	ctx := r.Context()
 	if err = request.Validate(ctx); err != nil {
-		return NewInvalidRequestError(err)
+		return err
 	}
 	if err = a.domainCall(ctx, request); err != nil {
 		return err