WIP - Prevent mutations from secondary side of relation

client/collection_description.go

@@ -150,6 +150,10 @@ func (col CollectionDescription) GetFieldByRelation(
 	otherCollectionName string,
 	otherFieldName string,
 ) (CollectionFieldDescription, bool) {
+	if relationName == "" {
+		return CollectionFieldDescription{}, false
+	}
+
 	for _, field := range col.Fields {
 		if field.RelationName.Value() == relationName &&
 			!(col.Name.Value() == otherCollectionName && otherFieldName == field.Name) &&

client/definitions.go

@@ -166,7 +166,7 @@ func NewFieldDefinition(local CollectionFieldDescription, global SchemaFieldDesc
 		Kind:              kind,
 		RelationName:      local.RelationName.Value(),
 		Typ:               global.Typ,
-		IsPrimaryRelation: kind.IsObject() && !kind.IsArray(),
+		IsPrimaryRelation: local.RelationName.HasValue() && !kind.IsArray(),
 		DefaultValue:      local.DefaultValue,
 	}
 }

client/document.go

@@ -692,6 +692,11 @@ func (doc *Document) Set(field string, value any) error {
 	if !exists {
 		return NewErrFieldNotExist(field)
 	}
+
+	if fd.RelationName != "" && !fd.IsPrimaryRelation {
+		return NewErrCannotSetRelationFromSecondarySide(field)
+	}
+
 	if fd.Kind.IsObject() && !fd.Kind.IsArray() {
 		if !strings.HasSuffix(field, request.RelatedObjectID) {
 			field = field + request.RelatedObjectID

client/errors.go

@@ -34,6 +34,7 @@ const (
 	errCanNotTurnNormalValueIntoArray      string = "can not turn normal value into array"
 	errCanNotMakeNormalNilFromFieldKind    string = "can not make normal nil from field kind"
 	errFailedToParseKind                   string = "failed to parse kind"
+	errCannotSetRelationFromSecondarySide  string = "cannot set relation from secondary side"
 )
 
 // Errors returnable from this package.
@@ -61,6 +62,7 @@ var (
 	ErrCanNotMakeNormalNilFromFieldKind     = errors.New(errCanNotMakeNormalNilFromFieldKind)
 	ErrCollectionNotFound                   = errors.New(errCollectionNotFound)
 	ErrFailedToParseKind                    = errors.New(errFailedToParseKind)
+	ErrCannotSetRelationFromSecondarySide   = errors.New(errCannotSetRelationFromSecondarySide)
 )
 
 // NewErrFieldNotExist returns an error indicating that the given field does not exist.
@@ -190,3 +192,7 @@ func ReviveError(message string) error {
 		return fmt.Errorf("%s", message)
 	}
 }
+
+func NewErrCannotSetRelationFromSecondarySide(name string) error {
+	return errors.New(errCannotSetRelationFromSecondarySide, errors.NewKV("Name", name))
+}

internal/request/graphql/schema/generate.go

@@ -566,6 +566,10 @@ func (g *Generator) buildMutationInputTypes(collections []client.CollectionDefin
 					continue
 				}
 
+				if field.RelationName != "" && !field.IsPrimaryRelation {
+					continue
+				}
+
 				var ttype gql.Type
 				if field.Kind.IsObject() {
 					if field.Kind.IsArray() {

tests/integration/mutation/update/field_kinds/one_to_one/with_simple_test.go

@@ -260,7 +260,6 @@ func TestMutationUpdateOneToOne_RelationIDToLinkFromPrimarySide(t *testing.T) {
 }
 
 func TestMutationUpdateOneToOne_RelationIDToLinkFromSecondarySide(t *testing.T) {
-	author1ID := "bae-53eff350-ad8e-532c-b72d-f95c4f47909c"
 	author2ID := "bae-c058cfd4-259f-5b08-975d-106f13a143d5"
 
 	test := testUtils.TestCase{
@@ -280,13 +279,9 @@ func TestMutationUpdateOneToOne_RelationIDToLinkFromSecondarySide(t *testing.T)
 			},
 			testUtils.CreateDoc{
 				CollectionID: 0,
-				Doc: fmt.Sprintf(
-					`{
-						"name": "Painted House",
-						"author_id": "%s"
-					}`,
-					author1ID,
-				),
+				Doc: `{
+					"name": "Painted House"
+				}`,
 			},
 			testUtils.UpdateDoc{
 				CollectionID: 0,
@@ -297,7 +292,7 @@ func TestMutationUpdateOneToOne_RelationIDToLinkFromSecondarySide(t *testing.T)
 					}`,
 					author2ID,
 				),
-				ExpectedError: "target document is already linked to another document.",
+				ExpectedError: "cannot set relation from secondary side",
 			},
 		},
 	}