Incorrect codegen error - DataStore does not support 1 to 1 connection with both sides of connection as optional field

[anthonymoretti]

Describe the bug
Codegen produces the following error even though both sides of the connection are actually non-optional:
"DataStore does not support 1 to 1 connection with both sides of connection as optional field"

To Reproduce

1. Create the schema below
2. Add api, push, then run amplify codegen models
3. See error

Expected behavior
Generates the code

Environment:

• Amplify Framework Version: 4.21.0
• Dependency Manager: Cocoapods
• Swift Version : 5.0

Device Information:

• Device: Simulator
• iOS Version: iOS 13.5

Additional context
Schema:

type PhoneCall @model {
  id: ID!
  caller: Person! @connection(name: "PhoneCallCaller")
  callee: Person! @connection(name: "PhoneCallCallee")
  transcript: Transcript! @connection(name: "PhoneCallTranscript")
}

type Person @model {
  id: ID!
  name: String!
  callerOf: [PhoneCall!] @connection(name: "PhoneCallCaller")
  calleeOf: [PhoneCall!] @connection(name: "PhoneCallCallee")
}

type Transcript @model {
  id: ID!
  text: String!
  language: String
  phoneCall: PhoneCall! @connection(name: "PhoneCallTranscript")
}

[watanabethais]

I'm having the same issue on amplify-android library.
Maybe this is related to amplify-cli...

[wooklym]

+1

[wooklym]

type TestModel @model {
  id: ID!
  name: String
}

type TestModel2 @model {
  id: ID!
  test1: TestModel! @connection
}

this works well, but

type TestModel @model {
  id: ID!
  name: String
}

type TestModel2 @model {
  id: ID!
  test1Id: ID!
  test1: TestModel! @connection(fields: ["test1Id"])
}

this causes error: DataStore does not support 1 to 1 connection with both sides of connection as optional field

But,

type TestModel @model {
  id: ID!
  name: String
}

type TestModel2 @model {
  id: ID!
  test1Id: ID!
  test1: TestModel @connection(fields: ["test1Id"]) # `!` removed
}

This works!!! 🙃 @anthonymoretti @watanabethais

[anthonymoretti]

Thanks guys. I've been told it's being looked into, has to do with which side of the relationship should be the "owner". I've made a feature request that possibly addresses this and other issues RFC - Simplified @connection directive

[danrivett]

I just ran into this, and can confirm paradoxically removing the ! from the connected model (TestModel in the example above) avoids the error as well for me.

The downside is now the generated TypeScript binding has the field as a nullable field when I want it non-nullable, especially as the key (test1Id in the example above) is defined as non-nullable, so it doesn't make sense that the connected field is nullable.

I wonder if there's any update to this bug?

[danrivett]

Isn't the bug here?: https://github.com/aws-amplify/amplify-cli/blob/master/packages/amplify-codegen-appsync-model-plugin/src/utils/process-connections.ts#L201

There are only 3 branches in the if statement checking if both sides of the connection are null or not:

• if (!field.isNullable && otherSideField.isNullable)
• else if (field.isNullable && !otherSideField.isNullable)
• else

But there are 4 separate possibilities, and 2 of them are being swallowed up by the last else clause. It looks like there should be a separate clause checking that both are not null and not throwing an error in that case.

I confirmed by monkey patch that if there was this 4th branch added (!field.isNullable && !otherSideField.isNullable), then it would fall into it and not error out for the schema @wooklym provided above.

@drochetti is the above known, and any idea if this will be fixed any time soon if so?

[github-actions]

This issue is stale because it has been open 14 days with no activity. Please, provide an update or this will be automatically closed in 7 days.

[drochetti]

This is still a relevant issue and needs to be triaged and prioritized.