agent/api/container/container.go (5 lines):
	- line 116: // TODO: [ecs-exec] Change variable name from Status to KnownStatus in future PR to avoid noise
	- line 221: // TODO DesiredStatusUnsafe should probably be private with appropriately written
	- line 229: // TODO KnownStatusUnsafe should probably be private with appropriately written
	- line 266: // TODO SentStatusUnsafe should probably be private with appropriately written
	- line 1158: // TODO return error to differentiate between error and default mode .


agent/dockerclient/dockerapi/docker_client.go (5 lines):
	- line 181: // TODO ListPluginsWithFilters can be removed since ListPlugins takes in filters
	- line 229: // TODO Remove clientfactory field once all API calls are migrated to sdkclientFactory
	- line 270: // TODO Remove clientfactory parameter once migration to Docker SDK is complete.
	- line 578: // TODO Remove ContainerInspect call
	- line 890: // TODO: move the error check into StartListening() to keep event streaming and error handling in one place.


agent/api/task/task.go (5 lines):
	- line 184: // TODO DesiredStatusUnsafe should probably be private with appropriately written
	- line 195: // TODO KnownStatusUnsafe should probably be private with appropriately written
	- line 215: // TODO SentStatusUnsafe should probably be private with appropriately written
	- line 1956: // TODO: Create a mapping of resource status to the corresponding task status and use it here
	- line 2101: // TODO: what's the point of locking if we are returning a pointer?


agent/engine/execcmd/manager_init_task.go (3 lines):
	- line 68: // TODO: [ecs-exec] Should we validate the ssm agent binaries & certs are valid and fail here if they're not? (bind mount will succeed even if files don't exist in host)
	- line 118: // TODO: [ecs-exec] This requirement will be removed for SSM agent V2
	- line 155: // TODO [ecs-exec] : verify that returning the default session limit (2) is ok in case of any errors, misconfiguration


agent/engine/execcmd/manager_start.go (2 lines):
	- line 62: //TODO: [ecs-exec] retry only for certain exit codes?
	- line 181: if !inspect.Running { //TODO: [ecs-exec] retry only for certain exit codes?


agent/engine/docker_task_engine.go (2 lines):
	- line 295: // TODO: [ecs-exec]add jitter between containers to not overload docker with top calls
	- line 1328: // TODO: Add a state to the apicontainer.Container for the status of the metadata file (Whether it needs update) and


agent/engine/docker_image_manager.go (2 lines):
	- line 127: // TODO add setter and getter for modifying this
	- line 298: //TODO: change image createdTime to image lastUsedTime when docker support it in the future


agent/engine/task_manager.go (2 lines):
	- line 251: // TODO: make this idempotent on agent restart
	- line 1385: // TODO we should probably panic here


agent/handlers/v2/response.go (2 lines):
	- line 196: // TODO: remove includeV4Metadata from NewContainerResponseFromState/NewContainerResponse
	- line 218: // TODO: remove includeV4Metadata from NewContainerResponse


agent/doctor/docker_runtime_healthcheck.go (1 line):
	- line 58: // TODO pass in context as an argument


agent/tcs/client/error.go (1 line):
	- line 44: // TODO: Populate this list when the json is updated with actual exceptions.


agent/dockerclient/dockerapi/types.go (1 line):
	- line 106: // TODO Remove type when migration is complete


misc/volumes-test/build.ps1 (1 line):
	- line 15: # TODO be smart about this and only rebuild if necessary


agent/logger/eventlog_windows.go (1 line):
	- line 25: TODO: Make this whole thing better


agent/stats/engine.go (1 line):
	- line 206: // TODO ensure that this is done only once per engine object


agent/taskresource/credentialspec/credentialspec_windows.go (1 line):
	- line 635: // TODO: Use registry to setup credentialspec resource location


misc/windows-deploy/amazon-ecs-agent.ps1 (1 line):
	- line 20: ## TODO: This is a very basic logging function. We should make it better, or replace it with


agent/app/agent.go (1 line):
	- line 479: // TODO Handle default cluster in a sane and unified way across the codebase


agent/app/agent_windows.go (1 line):
	- line 166: // TODO: Pre-start hooks go here (unclear if we need any yet)


agent/eni/networkutils/utils_linux.go (1 line):
	- line 92: // a timeout here. TODO: Fix RetryWithBackoffCtx to return ctx.Err() on context Done()


agent/eventhandler/task_handler.go (1 line):
	- line 332: // TODO: wire in the context here. Else, we have go routine leaks in tests


agent/api/testutils/container_equal.go (1 line):
	- line 28: // TODO: use reflection along with `equal:"unordered"` annotations on slices to


agent/tcs/client/client.go (1 line):
	- line 87: // TODO make this context inherited from the handler


agent/statemanager/state_manager_windows.go (1 line):
	- line 145: // TODO Investigate performance differences between FlushFileBuffers and unbuffered IO.


agent/dockerclient/dockerapi/docker_events_buffer.go (1 line):
	- line 24: // TODO  add support for filter in go-dockerclient


agent/engine/execcmd/manager_init_task_linux.go (1 line):
	- line 77: // TODO: [ecs-exec] seelog config needs to be implemented following a similar approach to ss, config


agent/taskresource/asmauth/asmauth.go (1 line):
	- line 63: // TODO: Refactor this struct so that each ASMAuthData gets associated with


agent/app/agent_capability.go (1 line):
	- line 217: // TODO: gate this on docker api version when ecs supported docker includes


agent/handlers/v4/response.go (1 line):
	- line 179: // TODO this is hard-coded to `0` for now. Once backend starts populating


agent/api/task/task_linux.go (1 line):
	- line 130: // TODO: DefaultCPUPeriod only permits 10VCPUs.


agent/api/statechange.go (1 line):
	- line 261: // TODO Move the Attachment statechange check into this method


agent/eni/watcher/watcher.go (1 line):
	- line 203: // a timeout here. TODO: Fix RetryWithBackoffCtx to return ctx.Err() on context Done()


agent/wsclient/mock/utils/utils.go (1 line):
	- line 25: // TODO replace with gomock