runtime/protocol/http-client-engines/http-client-engine-ktor/jvm/src/aws/smithy/kotlin/runtime/http/engine/ktor/KtorRequestAdapter.kt (4 lines): - line 29: // TODO - we could probably consider allowing this be set from the environment - line 35: // TODO: Remove following annotation after https://youtrack.jetbrains.com/issue/KTOR-3001 is resolved - line 85: // FIXME - ensure the `source` is closed? - line 88: // FIXME - instead of reading and writing bytes we could probably proxy the underlying channel smithy-kotlin-codegen/src/main/kotlin/software/amazon/smithy/kotlin/codegen/rendering/ShapeValueGenerator.kt (4 lines): - line 171: // TODO - deal with document shapes - line 242: // TODO - We need to decide non-JVM only symbols to generate for these before we know how to assign values to them - line 246: // TODO - deal with document shapes - line 256: // TODO - deal with document shapes runtime/build.gradle.kts (3 lines): - line 30: // FIXME - Workaround for unspecified kotlin target error after 1.4 upgrade. - line 39: // FIXME - Workaround for unspecified kotlin target error after 1.4 upgrade. - line 46: // TODO - the group to publish under needs negotiated still smithy-kotlin-codegen/src/main/kotlin/software/amazon/smithy/kotlin/codegen/rendering/GradleGenerator.kt (3 lines): - line 200: // TODO - can we make kotlin dependencies not specify a version e.g. kotlin("kotlin-test") - line 201: // TODO - Kotlin MPP setup (pass through KotlinSettings) - maybe separate gradle writers - line 233: // FIXME ~ new codewriter should use settings from parent. Support from Smithy is needed runtime/serde/common/src/aws/smithy/kotlin/runtime/serde/SdkSerializable.kt (2 lines): - line 11: // FIXME - baby steps - line 29: // FIXME - this causes backing classes to be generated behind the scenes and contributes to the overall jar size smithy-kotlin-codegen/src/main/kotlin/software/amazon/smithy/kotlin/codegen/rendering/protocol/HttpBindingProtocolGenerator.kt (2 lines): - line 609: // TODO - deal with document members - line 1062: // TODO - implement document support runtime/protocol/http/common/src/aws/smithy/kotlin/runtime/http/Url.kt (2 lines): - line 44: // FIXME - the userinfo and fragment are raw at this point and need escaped as well probably - line 154: // TODO - when we get to other platforms we will likely just roll our own - for now we are going to punt and use JVM runtime/serde/serde-xml/common/src/aws/smithy/kotlin/runtime/serde/xml/XmlSerializer.kt (2 lines): - line 16: // TODO - mark class internal and remove integration tests once serde is stable - line 19: // FIXME - clean up stack to distinguish between mutable/immutable and move to utils? (e.g. MutableStack = mutableStackOf()) runtime/serde/serde-form-url/common/src/aws/smithy/kotlin/runtime/serde/formurl/FormUrlSerializer.kt (1 line): - line 142: // FIXME - do we even use this function in any of the formats? It seems like we go through `field(.., SdkSerializable)` ?? smithy-kotlin-codegen/src/main/kotlin/software/amazon/smithy/kotlin/codegen/core/KotlinWriter.kt (1 line): - line 341: // TODO: https://github.com/awslabs/smithy-kotlin/issues/136 runtime/serde/serde-xml/common/src/aws/smithy/kotlin/runtime/serde/xml/XmlFieldTraits.kt (1 line): - line 71: * NOTE/FIXME: This type was written to handle the restXml protocol handling but could be refactored to be more runtime/runtime-core/common/src/aws/smithy/kotlin/runtime/retries/impl/StandardRetryPolicy.kt (1 line): - line 70: // TODO Write logic to find connection errors, timeouts, stream faults, etc. runtime/protocol/http-client-engines/http-client-engine-ktor/jvm/src/aws/smithy/kotlin/runtime/http/engine/ktor/KtorEngine.kt (1 line): - line 77: // TODO: Remove following annotation after https://youtrack.jetbrains.com/issue/KTOR-3001 is resolved smithy-kotlin-codegen/src/main/kotlin/software/amazon/smithy/kotlin/codegen/rendering/serde/SerializeUnionGenerator.kt (1 line): - line 131: // FIXME - this doesn't account for unboxed primitives runtime/io/common/src/aws/smithy/kotlin/runtime/io/SdkByteReadChannel.kt (1 line): - line 130: // TODO - we could pool these runtime/io/common/src/aws/smithy/kotlin/runtime/io/SdkByteBuffer.kt (1 line): - line 415: // TODO - remove in favor of implementing Appendable in such a way as to not allocate an entire new byte array smithy-kotlin-codegen/src/main/kotlin/software/amazon/smithy/kotlin/codegen/rendering/serde/AbstractSerdeDescriptorGenerator.kt (1 line): - line 57: // FIXME - decompose these symbols directly when they are emitted runtime/io/common/src/aws/smithy/kotlin/runtime/io/Allocator.kt (1 line): - line 14: // FIXME - we should revisit this - Kotlin/Native is only place where we would actually be manually managing memory runtime/runtime-core/common/src/aws/smithy/kotlin/runtime/config/IdempotencyTokenProvider.kt (1 line): - line 44: * TODO: Implement a real function. See https://www.pivotaltracker.com/story/show/174214013 smithy-kotlin-codegen/src/main/kotlin/software/amazon/smithy/kotlin/codegen/rendering/serde/DeserializeStructGenerator.kt (1 line): - line 105: // TODO ~ Not yet implemented runtime/runtime-core/common/src/aws/smithy/kotlin/runtime/client/ExecutionContext.kt (1 line): - line 13: // TODO - propagate Job() and/or coroutine context? runtime/protocol/http-client-engines/http-client-engine-ktor/jvm/src/aws/smithy/kotlin/runtime/http/engine/ktor/KtorUtils.kt (1 line): - line 22: // TODO: Remove following annotation after https://youtrack.jetbrains.com/issue/KTOR-3001 is resolved runtime/protocol/http/common/src/aws/smithy/kotlin/runtime/http/operation/HttpSerde.kt (1 line): - line 11: // FIXME - if SAM interfaces support suspend soon we should consider updating HttpSerialize/Deserialize to use them instead runtime/serde/common/src/aws/smithy/kotlin/runtime/serde/Serializer.kt (1 line): - line 31: // FIXME - we should commonize how we deal with buffers internally and rely on `SdkBuffer` smithy-kotlin-codegen/src/main/kotlin/software/amazon/smithy/kotlin/codegen/rendering/serde/XmlSerdeDescriptorGenerator.kt (1 line): - line 31: // FIXME - we should be able to remove special casing of errors here which is protocol specific dokka-smithy/build.gradle.kts (1 line): - line 20: allWarningsAsErrors = false // FIXME Dokka bundles stdlib into the classpath, causing an unfixable warning runtime/protocol/http/common/src/aws/smithy/kotlin/runtime/http/request/HttpRequestBuilder.kt (1 line): - line 79: // TODO - we have no way to know the http version at this level to set HTTP/x.x runtime/protocol/http/build.gradle.kts (1 line): - line 9: // FIXME - atomicfu gradle plugin fails on transformJvmMainAtomicfu task for some reason, for now just use it as a library without the bytecode transform smithy-kotlin-codegen/src/main/kotlin/software/amazon/smithy/kotlin/codegen/rendering/ServiceGenerator.kt (1 line): - line 17: // FIXME - rename file and class to ServiceClientGenerator runtime/testing/common/src/aws/smithy/kotlin/runtime/testing/runTest.kt (1 line): - line 11: // FIXME - this can go away if/when this lands: https://github.com/Kotlin/kotlinx.coroutines/issues/1996 runtime/runtime-core/common/src/aws/smithy/kotlin/runtime/client/ClientOptionsBuilder.kt (1 line): - line 19: // TODO - currently can only have nullable (T?) values delegated. Look at providing either a default/initial value runtime/protocol/http/common/src/aws/smithy/kotlin/runtime/http/SdkHttpClient.kt (1 line): - line 44: // FIXME - can we relocate to engine? runtime/runtime-core/common/src/aws/smithy/kotlin/runtime/time/Instant.kt (1 line): - line 10: // FIXME - remove in favor of kotlinx-datetime before GA (assuming it's available). For now smithy-kotlin-codegen/src/main/kotlin/software/amazon/smithy/kotlin/codegen/rendering/serde/SerializeStructGenerator.kt (1 line): - line 39: // FIXME - refactor to just take a CodegenContext rather than the more specific protocol generator context. Serde should be protocol agnostic (ideally) runtime/io/common/src/aws/smithy/kotlin/runtime/io/MutableBuffer.kt (1 line): - line 12: // TODO - could implement Appendable for convenience, need writeUtf8Char() extension