Summary: 81 instances, 66 unique

Text	Count
// 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.	1
// TODO: upload all file in parallel	1
// TODO: Translate path from container to host (Ting)	1
// TODO: Add test for this	1
// TODO: Remove parameter "move" after last-saved-checkin-metadata problem is fixed properly.	2
// TODO: CHANGE THIS TO PUBLIC WHEN SWITCH RETURN TYPES	1
//TODO: encoding?	1
// TODO: If command begins and ends with a double-quote, trim quotes before making determination. Likewise when determining whether the file exists.	1
//TODO: use Validators.NTAccountValidator when it works on Linux	1
// TODO: Should we also write to log in executionContext.Output methods? so that we don't have to repeat writing into logs?	1
// TODO: Remove fallback variable if stable	1
// TODO: how to deal with limited refs?	2
// TODO: Add other actions to take when rolling agent to specific rings	1
// TODO: check that the right supporting tools are available for this platform	1
// TODO Unit test to cover this logic	1
// TODO: If script file begins and ends with a double-quote, trim quotes before making determination. Likewise when determining whether the file exists.	1
//TODO: make sure we don't mask more important exception	1
// 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	1
// TODO: Added 5/27/21. Please remove within a month or two	1
// TODO: encoding?	1
// TODO: Make sure to remove Old agent's group and registry changes made during auto upgrade to vsts-agent.	1
Container = containerInfo, //TODO: Figure out if this needs to have all the containers or just the one for the current step	1
// TODO: Test whether the output encoding needs to be specified on a non-Latin OS.	4
// TODO: Translate targetPath from container to host (Ting)	1
// TODO: Should we fail instead? Do any additional pains need to be taken here? Should the job message not be traced?	1
// TODO: look to see if there is a better way to not have translate these back	1
// TODO: Loc strings formatted into MSG_xxx vars in shellscript	1
// TODO: Invalid config prints usage	1
// TODO: use alias to find the trigger repo when we have the concept of triggering repo.	1
//TODO: Make a cleaner fix after TCM User Story 401703 is completed.	1
// TODO: Dispose the trace listener also.	1
// TODO - remove this and just always throw once the feature has been fully rolled out.	2
// TODO: WHICH the command?	1
$"{task.Name}_{task.Id}_{task.Version}.zip"); // TODO: Move to shared string.	1
// TODO: Refactor extension manager to enable using it from the agent process.	1
// 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.	1
// TODO: We may not need this, server may return / always, check and remove it	1
// TODO: look to see if there is a better way to not have translate these back	1
// TODO: We need send detailInfo back to DT in order to add an issue for the job	1
// TODO: Remove parameter move after last-saved-checkin-metadata problem is fixed properly.	1
// TODO: Should we download them parallely?	1
// TODO: Fix scorch. Scorch blows up if a root mapping does not exist.	2
// TODO: If not rooted, WHICH the file if it doesn't contain any slashes.	1
// TODO: Add a try catch here to give a better error message.	1
// 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.	1
// TODO: figure out how should we retry paging log upload.	1
// TODO : Currently Telemetry is not support in PublishTestResults Library. Uncomment following line of code when we start supporting Telemetry.	1
// 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.	1
// TODO check if we need to allow only codecoverage logs at build level	1
// TODO: Consider using SecureString.	1
// TODO wait for 30 seconds and send a SIGKILL	1
// TODO: Remove parameter move after last-saved-checkin-metadata problem is fixed properly.	2
// TODO: Write this file under the _work folder and clean it up at the beginning of the next build?	1
// TODO: Hook up to ctrl+c cancellation token.	1
//TODO: If editing artifactDefinitionName is not allowed then we can remove this	1
builder.AppendLine($"Environment file created at(UTC): {DateTime.UtcNow}"); // TODO: Format this like we do in other places.	2
// TODO: Remove AddAsync after last-saved-checkin-metadata problem is fixed properly.	5
if (!string.IsNullOrEmpty(repoInfo.TriggeringRepository.Id)) // TODO: Move to const after source artifacts PR is merged.	1
// TODO: use a constant for project id, which is currently defined in Microsoft.VisualStudio.Services.Agent.Constants.Variables.System.TeamProjectId (Ting)	1
// TODO: Write tests for this	1
// TODO Fix this. This is not yet available in coreclr (newer version?)	1
// TODO: use a constant for project id, which is currently defined in Microsoft.VisualStudio.Services.Agent.Constants.Variables.System.TeamProjectId (Ting)	3
// TODO: LOC all strings.	1
// TODO: Add tests for this	1
$escapeMappings = @( # TODO: WHAT ABOUT "="? WHAT ABOUT "%"?	1
// TODO: "(?<" also starts things other than named groups	1