refactor: moved some helper functions to github.com/ralvarezdev/go-strings

Author: ralvarezdev

go.mod

@@ -4,7 +4,7 @@ go 1.24.0
 
 require (
 	github.com/ralvarezdev/go-reflect v0.2.13
-	github.com/ralvarezdev/go-strings v0.1.17
+	github.com/ralvarezdev/go-strings v0.2.0
 	google.golang.org/genproto/googleapis/rpc v0.0.0-20251020155222-88f65dc88635
 )
 

go.sum

@@ -16,6 +16,8 @@ github.com/ralvarezdev/go-strings v0.1.16 h1:0yc7yamYEWVrFTK3C95IjSE5IqCs3H1alII
 github.com/ralvarezdev/go-strings v0.1.16/go.mod h1:8sFOqmPJpqzS7bTjf91EzUCITnwpmkfifwY80GxV5r8=
 github.com/ralvarezdev/go-strings v0.1.17 h1:HFFqe5fVqsYdaOsj+yoVtgIgkNXubDHSz5ubN8V6iW8=
 github.com/ralvarezdev/go-strings v0.1.17/go.mod h1:8sFOqmPJpqzS7bTjf91EzUCITnwpmkfifwY80GxV5r8=
+github.com/ralvarezdev/go-strings v0.2.0 h1:JJUbGRpsnmuoEaFVv+rTT3P4BTsMqGOzlLkQRdfbt6c=
+github.com/ralvarezdev/go-strings v0.2.0/go.mod h1:8sFOqmPJpqzS7bTjf91EzUCITnwpmkfifwY80GxV5r8=
 google.golang.org/genproto/googleapis/rpc v0.0.0-20251007200510-49b9836ed3ff h1:A90eA31Wq6HOMIQlLfzFwzqGKBTuaVztYu/g8sn+8Zc=
 google.golang.org/genproto/googleapis/rpc v0.0.0-20251007200510-49b9836ed3ff/go.mod h1:7i2o+ce6H/6BluujYR+kqX3GKH+dChPTQU19wjRPiGk=
 google.golang.org/genproto/googleapis/rpc v0.0.0-20251020155222-88f65dc88635 h1:3uycTxukehWrxH4HtPRtn1PDABTU331ViDjyqrUbaog=

mapper/constants.go

@@ -7,9 +7,6 @@ import (
 var (
 	ErrNilGenerator            = errors.New("generator cannot be nil")
 	ErrNilMapper               = errors.New("mapper cannot be nil")
-	ErrProtobufTagNotFound     = "missing protobuf tag for field: %s"
-	ErrProtobufTagNameNotFound = "missing protobuf tag name for field: %s"
-	ErrEmptyJSONTag            = "empty json tag for field: %s"
 	ErrNilStructInstance      = errors.New("struct instance cannot be nil")
 	ErrStructInstanceNotStruct = errors.New("struct instance must be a struct")
 	ErrInvalidStructInstance = errors.New("invalid struct instance")

mapper/errors.go

@@ -6,6 +6,7 @@ import (
 	"strings"
 
 	goreflect "github.com/ralvarezdev/go-reflect"
+	gostringsjson "github.com/ralvarezdev/go-strings/json"
 )
 
 type (
@@ -51,6 +52,11 @@ func (j JSONGenerator) NewMapper(structInstance any) (
 	*Mapper,
 	error,
 ) {
+	// Check if the struct instance is nil
+	if structInstance == nil {
+		return nil, ErrNilStructInstance
+	}
+	
 	// Reflection of data
 	reflectedType := goreflect.GetDereferencedType(structInstance)
 
@@ -68,7 +74,6 @@ func (j JSONGenerator) NewMapper(structInstance any) (
 		// Get the field type through reflection
 		structField := reflectedType.Field(i)
 		fieldType := structField.Type
-		fieldTag := structField.Tag
 		fieldName := structField.Name
 
 		// Check if the field is unexported
@@ -79,10 +84,13 @@ func (j JSONGenerator) NewMapper(structInstance any) (
 		}
 
 		// Get the JSON tag of the field
-		jsonTag := fieldTag.Get(JSONTag)
+		jsonTag, err := gostringsjson.GetJSONTag(structField, fieldName)
+		if err != nil {
+			return nil, err
+		}
 
 		// Get the JSON name from the tag
-		jsonName, err := GetJSONTagName(jsonTag, fieldName)
+		jsonName, err := gostringsjson.GetJSONTagName(jsonTag, fieldName)
 		if err != nil {
 			return nil, err
 		}
@@ -91,7 +99,7 @@ func (j JSONGenerator) NewMapper(structInstance any) (
 		rootMapper.AddFieldTagName(fieldName, jsonName)
 
 		// Check if the JSON tag is unassigned or if it contains 'omitempty', which means it is an optional field
-		if jsonTag == "-" || strings.Contains(jsonTag, JSONOmitempty) {
+		if jsonTag == "-" || strings.Contains(jsonTag, gostringsjson.JSONOmitempty) {
 			// Set field name as not required
 			rootMapper.SetFieldIsRequired(fieldName, false)
 

mapper/json.go

@@ -5,6 +5,7 @@ import (
 	"reflect"
 
 	goreflect "github.com/ralvarezdev/go-reflect"
+	gostringsprotobuf "github.com/ralvarezdev/go-strings/protobuf"
 )
 
 type (
@@ -49,7 +50,12 @@ func NewProtobufGenerator(logger *slog.Logger) *ProtobufGenerator {
 func (p ProtobufGenerator) NewMapper(structInstance any) (
 	*Mapper,
 	error,
-) {	
+) {
+	// Check if the struct instance is nil
+	if structInstance == nil {
+		return nil, ErrNilStructInstance
+	}
+	
 	// Reflection of data
 	reflectedType := goreflect.GetDereferencedType(structInstance)
 
@@ -70,20 +76,20 @@ func (p ProtobufGenerator) NewMapper(structInstance any) (
 		fieldName := structField.Name
 
 		// Omit protobuf internal fields, protobuf oneof fields or just unexported fields
-		if IsProtobufGeneratedField(fieldName) || IsProtobufOneOfField(structField) || !goreflect.IsStructFieldExported(structField) {
+		if gostringsprotobuf.IsProtobufGeneratedField(fieldName) || gostringsprotobuf.IsProtobufOneOfField(structField) || !goreflect.IsStructFieldExported(structField) {
 			// Set field as not required
 			rootMapper.SetFieldIsRequired(fieldName, false)
 			continue
 		}
 
 		// Get the Protobuf tag of the field
-		protobufTag, err := GetProtobufTag(structField, fieldName)
+		protobufTag, err := gostringsprotobuf.GetProtobufTag(structField, fieldName)
 		if err != nil {
 			return nil, err
 		}
 
 		// Get the field name from the Protobuf tag
-		protobufName, err := GetProtobufTagName(protobufTag, fieldName)
+		protobufName, err := gostringsprotobuf.GetProtobufTagName(protobufTag, fieldName)
 		if err != nil {
 			return nil, err
 		}
@@ -98,7 +104,7 @@ func (p ProtobufGenerator) NewMapper(structInstance any) (
 
 			// Check if the element type is not a struct and the tag to determine if it contains 'oneof', which means it
 			// is an optional struct field
-			if fieldType.Kind() != reflect.Struct || IsProtobufFieldOptional(protobufTag) {
+			if fieldType.Kind() != reflect.Struct || gostringsprotobuf.IsProtobufFieldOptional(protobufTag) {
 				// Set field as not required
 				rootMapper.SetFieldIsRequired(fieldName, false)