src/Agent.Worker/Release/Artifacts/BuildArtifact.cs (3 lines):
	- line 31: // TODO: Write tests for this
	- line 85: // TODO: Should we download them parallely?
	- line 159: //TODO: If editing artifactDefinitionName is not allowed then we can remove this


src/Agent.Plugins/TFCliManager.cs (3 lines):
	- line 46: // TODO: Remove AddAsync after last-saved-checkin-metadata problem is fixed properly.
	- line 79: // TODO: Fix scorch. Scorch blows up if a root mapping does not exist.
	- line 173: // TODO: Remove parameter "move" after last-saved-checkin-metadata problem is fixed properly.


src/Agent.Worker/Handlers/PowerShellExeHandler.cs (3 lines):
	- line 60: // TODO: Write this file under the _work folder and clean it up at the beginning of the next build?
	- line 70: // TODO: If not rooted, WHICH the file if it doesn't contain any slashes.
	- line 102: // TODO: If script file begins and ends with a double-quote, trim quotes before making determination. Likewise when determining whether the file exists.


src/Agent.Listener/Configuration.macOS/MacOSServiceControlManager.cs (3 lines):
	- line 38: // TODO: encoding?
	- line 39: // TODO: Loc strings formatted into MSG_xxx vars in shellscript
	- line 51: //TODO: encoding?


src/Agent.Listener/Agent.cs (3 lines):
	- line 56: // TODO Unit test to cover this logic
	- line 62: // TODO: Invalid config prints usage
	- line 500: //TODO: make sure we don't mask more important exception


src/Agent.Worker/Build/TFCommandManager.cs (3 lines):
	- line 43: // TODO: Remove AddAsync after last-saved-checkin-metadata problem is fixed properly.
	- line 76: // TODO: Fix scorch. Scorch blows up if a root mapping does not exist.
	- line 170: // TODO: Remove parameter "move" after last-saved-checkin-metadata problem is fixed properly.


src/Agent.Worker/Build/TfsVCCommandManager.cs (3 lines):
	- line 28: // TODO: Remove AddAsync after last-saved-checkin-metadata problem is fixed properly.
	- line 37: // TODO: Remove parameter move after last-saved-checkin-metadata problem is fixed properly.
	- line 213: // TODO: Test whether the output encoding needs to be specified on a non-Latin OS.


src/Agent.Worker/DiagnosticLogManager.cs (2 lines):
	- line 479: builder.AppendLine($"Environment file created at(UTC): {DateTime.UtcNow}"); // TODO: Format this like we do in other places.
	- line 617: builder.AppendLine($"Environment file created at(UTC): {DateTime.UtcNow}"); // TODO: Format this like we do in other places.


src/Agent.Worker/Build/TeeCommandManager.cs (2 lines):
	- line 25: // TODO: Remove AddAsync after last-saved-checkin-metadata problem is fixed properly.
	- line 131: // TODO: Remove parameter move after last-saved-checkin-metadata problem is fixed properly.


src/Microsoft.VisualStudio.Services.Agent/JobServerQueue.cs (2 lines):
	- line 386: // TODO: upload all file in parallel
	- line 401: // TODO: figure out how should we retry paging log upload.


src/Agent.Plugins/TeeCliManager.cs (2 lines):
	- line 27: // TODO: Remove AddAsync after last-saved-checkin-metadata problem is fixed properly.
	- line 135: // TODO: Remove parameter move after last-saved-checkin-metadata problem is fixed properly.


src/Agent.Worker/TaskCommandExtension.cs (2 lines):
	- line 591: // TODO - remove this and just always throw once the feature has been fully rolled out.
	- line 678: // TODO - remove this and just always throw once the feature has been fully rolled out.


src/Agent.Plugins/PipelineArtifact/PipelineArtifactPlugin.cs (2 lines):
	- line 30: // TODO: Translate targetPath from container to host (Ting)
	- line 176: // TODO: use a constant for project id, which is currently defined in Microsoft.VisualStudio.Services.Agent.Constants.Variables.System.TeamProjectId (Ting)


src/Agent.Worker/Handlers/ProcessHandler.cs (2 lines):
	- line 44: // TODO: WHICH the command?
	- line 48: // TODO: If command begins and ends with a double-quote, trim quotes before making determination. Likewise when determining whether the file exists.


src/Agent.Listener/Configuration.Windows/NativeWindowsServiceHelper.cs (2 lines):
	- line 98: // TODO: Make sure to remove Old agent's group and registry changes made during auto upgrade to vsts-agent.
	- line 1093: // TODO Fix this. This is not yet available in coreclr (newer version?)


src/Agent.Worker/AgentPluginManager.cs (2 lines):
	- line 105: // TODO: look to see if there is a better way to not have translate these back
	- line 122: Container = containerInfo, //TODO: Figure out if this needs to have all the containers or just the one for the current step


src/Agent.Worker/Worker.cs (2 lines):
	- line 179: // TODO: Should we fail instead? Do any additional pains need to be taken here? Should the job message not be traced?
	- line 184: // TODO: Avoid adding redundant secrets. If the endpoint auth matches the system connection, then it's added as a value secret and as a regex secret. Once as a value secret b/c of the following code that iterates over each endpoint. Once as a regex secret due to the hint sent down in the job message.


src/Agent.Worker/Variables.cs (2 lines):
	- line 459: // TODO: A performance improvement could be made by short-circuiting if the non-expanded values are not dirty. It's unclear whether it would make a significant difference.
	- line 465: // TODO: Validate max size? No limit on *nix. Max of 32k per env var on Windows https://msdn.microsoft.com/en-us/library/windows/desktop/ms682653%28v=vs.85%29.aspx


src/Agent.Worker/Container/DockerCommandManager.cs (1 line):
	- line 151: // TODO: Remove fallback variable if stable


src/Agent.Plugins/GitSourceProvider.cs (1 line):
	- line 169: // TODO: how to deal with limited refs?


src/Agent.Listener/Program.cs (1 line):
	- line 48: // TODO: check that the right supporting tools are available for this platform


src/Agent.Worker/Handlers/LegacyPowerShellHandler.cs (1 line):
	- line 405: // TODO: use alias to find the trigger repo when we have the concept of triggering repo.


src/Misc/layoutbin/powershell/CapabilityHelpers/CapabilityFunctions.ps1 (1 line):
	- line 250: $escapeMappings = @( # TODO: WHAT ABOUT "="? WHAT ABOUT "%"?


release/rollrelease.js (1 line):
	- line 59: // TODO: Add other actions to take when rolling agent to specific rings


src/Agent.Plugins/PipelineArtifact/PipelineArtifactPluginV2.cs (1 line):
	- line 142: // TODO: use a constant for project id, which is currently defined in Microsoft.VisualStudio.Services.Agent.Constants.Variables.System.TeamProjectId (Ting)


tools/CredScanRegexes/PatternProcessor.cs (1 line):
	- line 17: // TODO: "(?<" also starts things other than named groups


src/Agent.Plugins/BuildArtifact/BuildArtifactPluginV1.cs (1 line):
	- line 162: // TODO: use a constant for project id, which is currently defined in Microsoft.VisualStudio.Services.Agent.Constants.Variables.System.TeamProjectId (Ting)


src/Agent.Plugins/PipelineCache/PipelineCacheTaskPluginBase.cs (1 line):
	- line 117: // TODO: Translate path from container to host (Ting)


src/Agent.Worker/Release/Artifacts/Definition/IArtifactDetails.cs (1 line):
	- line 17: // TODO: We may not need this, server may return / always, check and remove it


src/Agent.Worker/SimpleTimer.cs (1 line):
	- line 61: // TODO : Currently Telemetry is not support in PublishTestResults Library. Uncomment following line of code when we start supporting Telemetry.


src/Agent.Worker/TaskRunner.cs (1 line):
	- line 82: // TODO: Add a try catch here to give a better error message.


src/Agent.Worker/TaskManager.cs (1 line):
	- line 316: $"{task.Name}_{task.Id}_{task.Version}.zip"); // TODO: Move to shared string.


src/Agent.Plugins/SvnCliManager.cs (1 line):
	- line 644: // TODO: Test whether the output encoding needs to be specified on a non-Latin OS.


src/Agent.Worker/Release/Artifacts/FileShareArtifact.cs (1 line):
	- line 15: // TODO: Add test for this


src/Agent.Worker/Build/GitSourceProvider.cs (1 line):
	- line 199: // TODO: how to deal with limited refs?


src/Agent.Sdk/Knob/AgentKnobs.cs (1 line):
	- line 242: // TODO: Added 5/27/21. Please remove within a month or two


src/Agent.Service/Windows/Program.cs (1 line):
	- line 20: // TODO: LOC all strings.


src/Agent.Worker/AgentLogPlugin.cs (1 line):
	- line 146: // TODO: look to see if there is a better way to not have translate these back


src/Agent.Sdk/Util/StringUtil.cs (1 line):
	- line 175: // TODO: Log that string format failed. Consider moving this into a context base class if that's the only place it's used. Then the current trace scope would be available as well.


src/Agent.Plugins/TfsVCCliManager.cs (1 line):
	- line 230: // TODO: Test whether the output encoding needs to be specified on a non-Latin OS.


src/Agent.Worker/TestResults/ResultsCommandExtension.cs (1 line):
	- line 205: //TODO: Make a cleaner fix after TCM User Story 401703 is completed.


src/Microsoft.VisualStudio.Services.Agent/Terminal.cs (1 line):
	- line 57: // TODO: Consider using SecureString.


src/Agent.Worker/Build/BuildJobExtension.cs (1 line):
	- line 148: if (!string.IsNullOrEmpty(repoInfo.TriggeringRepository.Id)) // TODO: Move to const after source artifacts PR is merged.


src/Agent.Worker/Build/SvnCommandManager.cs (1 line):
	- line 735: // TODO: Test whether the output encoding needs to be specified on a non-Latin OS.


src/Agent.Listener/JobDispatcher.cs (1 line):
	- line 801: // TODO: We need send detailInfo back to DT in order to add an issue for the job


src/Microsoft.VisualStudio.Services.Agent/HostContext.cs (1 line):
	- line 547: // TODO: Dispose the trace listener also.


src/Agent.Worker/TestResults/Utils/PublisherInputValidator.cs (1 line):
	- line 59: // TODO check if we need to allow only codecoverage logs at build level


src/Agent.Listener/Configuration/NegotiateCredential.cs (1 line):
	- line 79: //TODO: use Validators.NTAccountValidator when it works on Linux


src/Misc/layoutbin/AgentService.js (1 line):
	- line 84: // TODO wait for 30 seconds and send a SIGKILL


src/Agent.Worker/Release/ZipStreamDownloader.cs (1 line):
	- line 27: // TODO: Add tests for this


src/Agent.Worker/Release/ReleaseJobExtension.cs (1 line):
	- line 345: // TODO: Should we also write to log in executionContext.Output methods? so that we don't have to repeat writing into logs?


src/Agent.Listener/Configuration.Windows/WindowsServiceControlManager.cs (1 line):
	- line 49: // TODO: Fix bug that exists in the legacy Windows agent where configuration using mirrored credentials causes an error, but the agent is still functional (after restarting). Mirrored credentials is a supported scenario and shouldn't manifest any errors.


src/Agent.Listener/Configuration/CredentialManager.cs (1 line):
	- line 10: // TODO: Refactor extension manager to enable using it from the agent process.


src/Agent.Listener/Configuration/ConfigurationManager.cs (1 line):
	- line 289: // TODO: Hook up to ctrl+c cancellation token.


src/Agent.Plugins/PipelineArtifact/PipelineArtifactPluginV1.cs (1 line):
	- line 244: // TODO: use a constant for project id, which is currently defined in Microsoft.VisualStudio.Services.Agent.Constants.Variables.System.TeamProjectId (Ting)


src/Agent.Listener/DistributedTask.Pipelines/Yaml/PipelineParser.cs (1 line):
	- line 66: // TODO: CHANGE THIS TO PUBLIC WHEN SWITCH RETURN TYPES