[typescript-operations] Conditional directives are ignored on fields with alias

[jnous-5]

Describe the bug

The generated types for fields that use aliases don't contain the optional operator ?. This throws a ts error when strictNullChecks is enabled.

Your Example Website or App

https://stackblitz.com/edit/github-hxvpzz-f9camz?file=types.ts

Steps to Reproduce the Bug or Issue

1. Go to document.graphql and see that we have 2 fields with and without aliases.
2. Run yarn codegen
3. Go to types.ts and notice that UserQuery.alias isn't an optional property whereas UserQuery.user works correctly.

Expected behavior

Expected to see something like below. Fields with conditional directives (with or without aliases) should work similary.

export type UserQuery = {
  __typename?: 'Query';
  alias?: { __typename?: 'User'; id: string; username: string; email: string };
  user?: { __typename?: 'User'; id: string; username: string; email: string };
};

Screenshots or Videos

No response

Platform

• OS: macOS
• NodeJS: 16.16.0
• graphql version: 16.6.0
• @graphql-codegen/typescript-operations version(s): 2.5.3

Codegen Config File

No response

Additional context

Maybe we can pass the isConditional variable in L532. Currently, only the name has access on this.

const isConditional = hasConditionalDirectives(field) || inlineFragmentConditional;
linkFields.push({
- alias: field.alias ? this._processor.config.formatNamedField(field.alias.value, selectedFieldType) : undefined,
+ alias: field.alias ? this._processor.config.formatNamedField(field.alias.value, selectedFieldType, isConditional) : undefined,