src/AutoRest.CSharp/Common/Output/Builders/JsonSerializationMethodsBuilder.cs (10 lines): - line 197: // TODO -- make the options and iPersistableModelTInterface parameter non-nullable again when we remove the `UseModelReaderWriter` flag. - line 265: // TODO -- make the options parameter non-nullable again when we remove the `UseModelReaderWriter` flag. - line 268: // TODO -- update this so that we could decide if we want to write the extra check of checking if this thing is in the raw data field dictionary - line 323: // TODO -- make the options parameter non-nullable again when we remove the `UseModelReaderWriter` flag. - line 597: // TODO -- make the options parameter non-nullable again when we remove the `UseModelReaderWriter` flag. - line 636: // TODO -- make the options parameter non-nullable again when we remove the `use-model-reader-writer` flag - line 645: // TODO -- make the options parameter non-nullable again when we remove the `UseModelReaderWriter` flag. - line 714: // TODO -- make the options parameter non-nullable again when we remove the `UseModelReaderWriter` flag. - line 984: // TODO -- make options parameter non-nullable again when we remove the `use-model-reader-writer` flag - line 1027: // TODO -- make options parameter non-nullable again when we remove the `use-model-reader-writer` flag src/AutoRest.CSharp/Mgmt/Generation/MgmtClientBaseWriter.cs (6 lines): - line 81: _writer.Line(); // TODO -- add this here to minimize the amount of code changes, this could be removed after future refactor - line 424: // TODO -- since we are combining multiple operations under different parents, which description should we leave here - line 425: // TODO -- find a better way to get this type - line 577: // TODO -- since we are combining multiple operations under different parents, which description should we leave here? - line 578: // TODO -- find a way to properly get the LRO response type here. Temporarily we are using the first one - line 579: // TODO -- we need to write multiple branches for a LRO operation src/AutoRest.CSharp/Mgmt/AutoRest/MgmtOutputLibrary.cs (5 lines): - line 120: // TODO -- considering provide a customized comparer - line 192: //TODO handle expandable request paths. We assume that this is fine since if all of the expanded - line 341: // TODO -- remove this condition completely when remove the UseModelReaderWriter flag - line 367: "Unknown", //TODO: do we need to support extensible enum / int values? - line 689: // TODO -- introduce a flag that suppress the exception here to be thrown which notice the user to assign a proper name in config src/AutoRest.CSharp/Common/Output/Builders/SerializationBuilder.cs (5 lines): - line 107: // TODO: add binary serialization - line 622: /*TODO: This is a temporary implementation. We need to revisit this and make it more robust. - line 701: /*TODO: need to update to use InputType to identify if it is a Multipart File or not. Current we will set contentType for Bytes and Stream*/ - line 705: valueSerialization.ContentType = "application/octet-stream"; //TODO: need to set the right content type from InputType - line 711: valueSerialization.ContentType = "application/octet-stream"; //TODO: need to set the right content type from InputType src/AutoRest.CSharp/Common/Input/Configuration.cs (4 lines): - line 60: // TODO - this configuration only exists here because we would like a rolling update for all libraries for this feature since it changes so many files. - line 62: // TODO - this configuration only exists here because we would like a rolling update for all libraries for this feature since it changes so many files. - line 168: //TODO Remove after resolving https://github.com/Azure/autorest.csharp/issues/3151 - line 403: examplesDirectory: null, // TODO -- what we put here? src/AutoRest.CSharp/LowLevel/Output/LowLevelClient.cs (4 lines): - line 85: // TODO -- we should report a warning diagnostic here indicating that we are changing something in the input. - line 124: //TODO: get all custom methods using similar method to GetEffectiveCtor - line 320: //TODO: This method is needed because we allow roslyn code gen attributes to be run AFTER the writers do their job but before - line 350: //TODO: Currently skipping ctors which use models from the library due to constructing with all empty lists. src/AutoRest.CSharp/Common/Output/Models/Types/SchemaObjectType.cs (4 lines): - line 427: // TODO: remove the order by - line 448: // TODO -- this is not really getting the serialized name of the properties as advertised in the method name - line 723: // TODO -- this should always return true when use model reader writer is enabled. - line 729: // TODO -- this should always return true when use model reader writer is enabled. src/AutoRest.CSharp/Common/Output/Models/Types/ModelTypeProvider.cs (4 lines): - line 223: //TODO if no one inherits from this we can omit the virtual - line 614: valueSerialization = new MultipartValueSerialization(dictionaryValueType, SerializationFormat.Default, true);//TODO support dictionary type - line 688: // TODO -- this should always return true when use model reader writer is enabled. - line 694: // TODO -- this should always return true when use model reader writer is enabled. src/AutoRest.CSharp/LowLevel/Generation/DpgClientWriter.cs (3 lines): - line 85: //TODO: Write example docs for custom methods - line 173: //TODO: make this a field?? - line 554: // TODO -- create wiki links to provide guidance here: https://github.com/Azure/autorest.csharp/issues/3624 src/AutoRest.CSharp/Common/Output/Models/Types/ModelFactoryTypeProvider.cs (3 lines): - line 34: // TODO: remove this intermediate state once we generate it before output types - line 44: // TODO: remove this intermediate state once we generate it before output types - line 123: // TODO: enable this for DPG, and check Configuration.Generate1ConvenientClient to disable it for HLC src/AutoRest.CSharp/Common/Output/Models/Types/SignatureType.cs (3 lines): - line 224: // TODO: handle missing type - line 234: // TODO: handle missing method return type from MgmtOutputLibrary - line 238: // TODO: handle missing parameter type from MgmtOutputLibrary src/AutoRest.CSharp/Mgmt/Models/MgmtClientOperation.cs (3 lines): - line 131: // TODO -- we need a better way to get the name of this - line 134: // TODO -- we need a better way to get the description of this - line 198: // TODO -- we need a better way to get this src/AutoRest.CSharp/Common/AutoRest/Plugins/GeneratedCodeWorkspace.cs (3 lines): - line 137: if (needProcessGeneratedDocs) // TODO -- this is a workaround. In HLC, in some cases, there are multiple documents with the same name added in this list, and we get "dictionary same key has been added" exception - line 236: // TODO: Currently the outputDirectory is expected to be generated folder. We will handle the customization folder if there is a case. - line 271: // TODO: This is not the right way to construct the workspace but it works src/AutoRest.CSharp/Common/AutoRest/Plugins/CSharpProj.cs (2 lines): - line 13: // TODO -- move this somewhere else because it is no longer a "plugin" - line 55: //TODO adding to workspace makes the formatting messed up since its a raw xml document src/AutoRest.CSharp/Common/Output/Expressions/Snippets.Serialization.cs (2 lines): - line 50: // TODO -- make the options parameter non-nullable again when we remove the `UseModelReaderWriter` flag. - line 109: // TODO -- make the options parameter non-nullable again when we remove the `UseModelReaderWriter` flag. src/AutoRest.CSharp/Common/AutoRest/Communication/StandaloneGeneratorRunner.cs (2 lines): - line 28: //TODO this is only here for back compat we should consider removing it - line 66: // TODO - add support for DataFactoryElement lookup src/assets/Azure.Core.Shared/NextLinkOperationImplementation.cs (2 lines): - line 100: // TODO: Once we remove NextLinkOperationImplementation from internal shared and make it internal to Azure.Core only in https://github.com/Azure/azure-sdk-for-net/issues/43260 - line 225: // TODO: Once we remove NextLinkOperationImplementation from internal shared and make it internal to Azure.Core only in https://github.com/Azure/azure-sdk-for-net/issues/43260 src/AutoRest.CSharp/Common/Generation/Writers/RequestWriterHelpers.cs (2 lines): - line 107: //TODO: Duplicate code between query and header parameter processing logic - line 278: //TODO: Duplicate code between query and header parameter processing logic src/AutoRest.CSharp/Common/Output/Builders/XmlSerializationMethodsBuilder.cs (2 lines): - line 102: // TODO -- make the options parameter non-nullable again when we remove the `UseModelReaderWriter` flag. - line 255: // TODO -- make the options parameter non-nullable again when we remove the `UseModelReaderWriter` flag. src/AutoRest.CSharp/Common/Output/Models/CmcRestClientBuilder.cs (2 lines): - line 119: // TODO: why are there duplicates in the list? - line 350: if (requestParameter.DefaultValue is not null) // TODO: we're using default value to store constant value, this should be fixed. src/AutoRest.CSharp/Mgmt/Decorator/SchemaExtensions.cs (2 lines): - line 88: // TODO: we may reuse the IsResourceModel instead of creating this method, but the result for flattened properties is different as although models with matched flattened properties are not treated as Resource but they still inherit from ResourceData. We should probably consider to align the behavior before we can refactor the methods. - line 99: // TODO: we may reuse the IsResourceModel instead of creating this method, but the result for flattened properties is different as although models with matched flattened properties are not treated as Resource but they still inherit from ResourceData. We should probably consider to align the behavior before we can refactor the methods. src/AutoRest.CSharp/AzurePlugin/AzurePluginTarget.cs (2 lines): - line 13: // TODO: serialize configuration.json and serialize inputNamespace to tspCodeModel.json - line 14: // TODO: spawn a child process to invoke MTG or Azure plugin src/AutoRest.CSharp/Common/Output/Models/Types/HelperTypeProviders/System/ClientUriBuilderProvider.cs (2 lines): - line 194: // TODO -- this is temporary, in the future, TypeFormatters will be a standalone static class, but now it is a nested type inside ModelSerializationExtensions - line 271: // TODO -- this is temporary, in the future, TypeFormatters will be a standalone static class, but now it is a nested type inside ModelSerializationExtensions src/AutoRest.CSharp/Common/Output/Builders/BuilderHelpers.cs (2 lines): - line 180: // TODO: Clean up these helper methods in https://github.com/Azure/autorest.csharp/issues/4767 - line 384: // TODO -- we only construct additional properties when the type is verifiable, because we always need the property to fall into the bucket of serialized additional raw data field when it does not fit the additional properties. Directory.Build.props (2 lines): - line 19: - line 21: src/AutoRest.CSharp/Common/Output/Models/Types/HelperTypeProviders/TypeFormattersProvider.cs (2 lines): - line 27: // TODO -- this type provider is temprorarily an inner class therefore it is not singleton yet. We need to change this to singleton when we decide to emit the entire TypeFormatters class. - line 324: // TODO -- figure out how to write this line src/AutoRest.CSharp/Common/Generation/Writers/PageableMethodsWriterExtensions.cs (2 lines): - line 22: // TODO -- this class could be a "PageableMethodsBuilder" in the future - line 276: // TODO -- we no longer need this once we remove the UseModelReaderWriter flag src/AutoRest.CSharp/LowLevel/Output/DpgOutputLibraryBuilder.cs (2 lines): - line 118: // TODO -- remove this condition completely when remove the UseModelReaderWriter flag - line 139: "Unknown", //TODO: do we need to support extensible enum / int values? src/TypeSpec.Extension/Emitter.Csharp/src/emitter.ts (1 line): - line 160: /* TODO: when we support to emit decorator list https://github.com/Azure/autorest.csharp/issues/4887, we will update to use emitted decorator to identify if it is azure-arm */ src/AutoRest.CSharp/Common/Input/CodeModelSerialization.cs (1 line): - line 58: //TODO: Handle custom type dictionaries. src/AutoRest.CSharp/Common/Input/InputTypes/BodyMediaType.cs (1 line): - line 41: // TODO: This is a temporary solution. We will move this part to some common place. src/AutoRest.CSharp/Common/Utilities/NamedTypeSymbolExtensions.cs (1 line): - line 16: // TODO -- to be removed when we refactor the code of sample generator in DPG. This method duplicated the existing method TypeFactory.CreateType src/AutoRest.CSharp/Common/Output/Models/Types/HelperTypeProviders/ModelSerializationExtensionsProvider.cs (1 line): - line 696: yield return _typeFormattersProvider; // TODO -- we should remove this and move `TypeFormattersProvider.Instance` to the list of helper providers src/AutoRest.CSharp/Common/Output/Expressions/Snippets.Convert.cs (1 line): - line 131: // TODO: Here we only consider the case when body is string type. We will add support for other types. src/AutoRest.CSharp/Common/Input/InputTypes/InputPrimitiveType.cs (1 line): - line 59: public static InputPrimitiveType Stream { get; } = new(InputPrimitiveTypeKind.Stream, "stream", "TypeSpec.Stream"); // TODO -- this is not a builtin type src/AutoRest.CSharp/Common/Input/InputTypes/Serialization/TypeSpecInputModelTypeConverter.cs (1 line): - line 237: /* TODO: in Multipart body model, if the property is of type Bytes, it should be converted to Stream src/AutoRest.CSharp/Common/Output/Models/Types/SerializableObjectType.cs (1 line): - line 64: // TODO -- despite this is actually a field if present, we have to make it a property to work properly with other functionalities in the generator, such as the `CodeWriter.WriteInitialization` method src/AutoRest.CSharp/Common/Output/Expressions/Snippets.cs (1 line): - line 105: // TODO: https://github.com/Azure/autorest.csharp/issues/4633 src/AutoRest.CSharp/Common/Input/CodeModelConverter.cs (1 line): - line 421: // TODO -- change it to the throw exception version when https://github.com/Azure/azure-rest-api-specs/issues/29320 and https://github.com/Azure/azure-rest-api-specs/issues/29321 are resolved src/AutoRest.CSharp/Common/Output/Models/Types/HelperTypeProviders/ChangeTrackingDictionaryProvider.cs (1 line): - line 313: //TODO: This line will not honor the generic type of the array eng/Generate.ps1 (1 line): - line 60: # TODO -- remove the flag when the feature is generally available src/AutoRest.CSharp/Mgmt/Output/Resource.cs (1 line): - line 459: // TODO -- what if the response type is not the same? Also we need to verify they have the same parameters before we could union those together src/AutoRest.CSharp/Mgmt/Report/ResourceItem.cs (1 line): - line 39: // assume there is no circle in resource hirachy. TODO: handle it if it's not true src/AutoRest.CSharp/LowLevel/Output/ConvenienceMethod.cs (1 line): - line 112: // TODO -- currently we only need to build response handling for long running method and normal method src/AutoRest.CSharp/Common/Input/InputTypes/Serialization/TypeSpecInputParameterConverter.cs (1 line): - line 62: || reader.TryReadBoolean("isResourceParameter", ref isResourceParameter) // TODO - this should be removed src/AutoRest.CSharp/Common/Output/Models/Types/ModelTypeProviderFields.cs (1 line): - line 159: // TODO -- when we consolidate the schemas into input types, we should remove this method and move it into BuilderHelpers src/AutoRest.CSharp/Common/Input/InputTypes/InputParameter.cs (1 line): - line 16: // TODO: This should be removed, tracking it in https://github.com/Azure/autorest.csharp/issues/4779 src/AutoRest.CSharp/Mgmt/Decorator/ParentDetection.cs (1 line): - line 85: // TODO -- enhence this to support the new arm-id format src/AutoRest.CSharp/Common/Input/InputTypes/Transformation/InputNamespaceTransformer.cs (1 line): - line 10: // TODO: Remove this when we have a better way to remove operations, tracking in https://github.com/Azure/typespec-azure/issues/964 src/AutoRest.CSharp/Mgmt/Decorator/Transformer/CodeModelValidator.cs (1 line): - line 26: // TODO: this might be able to be removed after https://github.com/Azure/autorest.csharp/issues/1917 is resolved. src/AutoRest.CSharp/Common/Input/InputTypes/Transformation/InputPluralEnumTransformer.cs (1 line): - line 13: // TODO: Add items to skip this transformation src/AutoRest.CSharp/Common/Input/InputTypes/InputType.cs (1 line): - line 50: //TODO: Remove this until the SDK nullable is enabled, traking in https://github.com/Azure/autorest.csharp/issues/4780 src/AutoRest.CSharp/Mgmt/Decorator/ParameterMappingBuilder.cs (1 line): - line 219: // TODO: The deserialize type value logic is existing in multiple writers, similar but slightly different, src/AutoRest.CSharp/Common/Output/Models/Types/SystemObjectType.cs (1 line): - line 127: // TODO -- this might be unnecessary, need to investigate the ctor of CSharpType to make sure it could correctly handle typeof System.Nullable Directory.Build.targets (1 line): - line 76: src/AutoRest.CSharp/Common/Input/InputTypes/InputLiteralType.cs (1 line): - line 6: internal record InputLiteralType(InputType ValueType, object Value) : InputType("Literal") // TODO -- name? src/AutoRest.CSharp/Mgmt/AutoRest/MgmtTarget.cs (1 line): - line 234: // TODO: fix the overriden src/AutoRest.CSharp/Common/Generation/Writers/CSProjWriter.cs (1 line): - line 147: //TODO if the properties is empty, this part will genearet a self -closing tag. And can not be elimitted after trying many ways. src/AutoRest.CSharp/Common/Input/InputTypes/InputOperation.cs (1 line): - line 164: //TODO: Remove this until the SDK nullable is enabled, tracking in https://github.com/Azure/autorest.csharp/issues/4780 src/AutoRest.CSharp/LowLevel/Output/LowLevelClientMethod.cs (1 line): - line 24: LongRunningResultRetrievalMethod? LongRunningResultRetrievalMethod) // TODO: move `LongRunningResultRetrievalMethod` under output model of long running, currently we're using src/AutoRest.CSharp/Mgmt/Models/ResourceTypeSegment.cs (1 line): - line 164: return DoAllButLastItemMatch(other); //TODO: limit matching to the enum values src/AutoRest.CSharp/Mgmt/Decorator/BodyParameterNormalizer.cs (1 line): - line 46: // TODO -- we need to add a check here to see if this rename introduces parameter name collisions src/AutoRest.CSharp/Common/Output/Models/Types/ExpressionTypeProvider.cs (1 line): - line 18: // TODO -- eventually we should combine everything in this class into TypeProvider src/AutoRest.CSharp/Mgmt/Decorator/ReferenceTypePropertyChooser.cs (1 line): - line 72: //TODO for core generation this list is small enough we can simply define each of them here. src/AutoRest.CSharp/Mgmt/Output/MgmtExtension.cs (1 line): - line 96: // TODO -- these logic needs a thorough refactor -- the values MgmtRestOperation consumes here are actually coupled together src/AutoRest.CSharp/Common/Output/Models/Types/AspDotNetExtensionTypeProvider.cs (1 line): - line 33: //TODO: very bad design that this list is empty when we leave the constructor and is filled in at some point in the future. src/AutoRest.CSharp/Common/AutoRest/Plugins/NewProjectScaffolding.cs (1 line): - line 82: //TODO handle existing ci where multiple projects are in the same service directory src/AutoRest.CSharp/Common/Input/InputTypes/InputTypeSerialization.cs (1 line): - line 6: // TODO: Remove this class and consume Decorators information instead src/AutoRest.CSharp/Common/Generation/Writers/CodeWriter.cs (1 line): - line 339: // TODO Temporary -- maybe we should consolidate this into the CodeWriter.AppendXmlDocumentation method as well src/assets/Azure.Core.Shared/RawRequestUriBuilder.cs (1 line): - line 80: // TODO: Find a better way to map schemes to default ports src/AutoRest.CSharp/Common/Input/InputTypes/Serialization/TypeSpecInputEnumTypeConverter.cs (1 line): - line 63: // TODO: roll back to throw JSON error when there is linter on the upstream to check enum without @doc src/AutoRest.CSharp/Common/Generation/Writers/ModelWriter.cs (1 line): - line 99: // TODO -- this is workaround because we are declaring fields and properties in different way, and this raw data field and AdditionalProperties property could be converted from each other. src/AutoRest.CSharp/Common/Generation/Writers/LongRunningOperationWriter.cs (1 line): - line 222: // TODO -- why we have a hard-coded Product.DeserializeProduct here??? src/AutoRest.CSharp/Mgmt/AutoRest/MgmtConfiguration.cs (1 line): - line 178: // TODO: A unified way to load from both readme and configuration.json src/assets/Azure.Core.Shared/DiagnosticScope.cs (1 line): - line 140: // TODO (limolkova) when we start targeting .NET 8 we should put src/AutoRest.CSharp/Common/Output/Models/Serialization/SerializationInterfaces.cs (1 line): - line 17: // TODO -- includeSerializer could be removed after the use-model-reader-writer configuration is removed because it is always true src/AutoRest.CSharp/Common/Output/Models/Types/ObjectTypeProperty.cs (1 line): - line 317: // TODO -- need to fix this so that we could have the reference of unioned models. src/AutoRest.CSharp/Common/Generation/Writers/FormattableStringHelpers.cs (1 line): - line 175: // TODO: Here we only consider the case when body is string type. We will add support for other types. src/AutoRest.CSharp/Common/Generation/Types/CSharpType.cs (1 line): - line 75: // TODO -- these are azure-specific. We might need to generalize these using ApiTypes