user/aggregate.ts (9 lines):
	- line 14: // TODO: expose a combined 'teams in' which is maint + member
	- line 15: // TODO: how to expose a -pending- org invite membership waiting?
	- line 16: // TODO: expose a 'in' org that is owner and member of
	- line 17: // TODO: object should expose each aspect as promises, allowing intra-middleware smart use & reuse in a request
	- line 18: // TODO: remove 'linkmanager' concept
	- line 37: // TODO: implement these newer ones
	- line 42: // TODO: the newer types, also...
	- line 53: // TODO: rename repos to repositories
	- line 63: // TODO: rename to UserAggregations class


business/team.ts (4 lines):
	- line 55: // TODO: cleanup intentional memory leak
	- line 126: // TODO: remove assignKnownFieldsPrefixed concept, use newer field definitions instead?
	- line 233: // TODO: move beyond setting with this approach
	- line 277: // TODO: validating typing here - number or int?


business/operations/index.ts (4 lines):
	- line 534: // TODO: Ported code from account.ts, remains unimp. It's OK.
	- line 558: // TODO: Ported code from account.ts, remains unimp. It's OK.
	- line 1096: // TODO: telemetry
	- line 1169: // TODO: consider looking at the options as to how to include/exclude properties etc.


business/organization.ts (4 lines):
	- line 487: // TODO: create repository options interface
	- line 619: // TODO: hook up this new change
	- line 681: getOwners(options?: IPagedCacheOptions): Promise<OrganizationMember[] /* TODO: validate return type */> {
	- line 707: // TODO: consider using the generic GET request method below


business/account.ts (4 lines):
	- line 133: // TODO: looks like we need to be able to resolve the link in here, too, to set instance.link
	- line 274: // TODO: should this throw then?
	- line 284: // TODO: can we only ignore WHEN WE KNOW ITS a 404 no link vs a 500?
	- line 388: // TODO: implement getOrganizationMemberships, with caching; reuse below code


routes/reposPager.ts (3 lines):
	- line 82: // TODO: Validate the type
	- line 114: // TODO: Validate the language value is in the Linguist list
	- line 197: reposDataAgeInformation: null, // ageInformation, // TODO: can 'ageInformation' be recovered?


routes/index-authenticated.ts (3 lines):
	- line 100: // TODO: bring back sudoers
	- line 104: twoFactorOff: false, // TODO: RESTORE: Reinstate two-factor off functionality
	- line 130: // TODO: This would redirect to the organization /security-check endpoint


lib/linkProviders/table/tableLinkProvider.ts (3 lines):
	- line 199: // TODO: NOT NORMALIZED
	- line 200: // TODO: VALUES in the current table have usernames that are MIXED CASE!!!!!!
	- line 266: // TODO: not sure if this one is normalized or not...


routes/org/membership.ts (3 lines):
	- line 17: // TODO: Was this ever a possible situation? What's going on here? Probably was v1 (single-org)
	- line 54: // TODO: Was this ever a possible situation? What's going on here? Probably was v1 (single-org)
	- line 67: // TODO: validate this works, since it is blindly calling now!


app.ts (3 lines):
	- line 64: // TODO: automatically track elapsed job time
	- line 68: // TODO: insights metric and event, if a prefix exists
	- line 138: // TODO: insights metric for job time


webhooks/tasks/team.ts (2 lines):
	- line 18: // TODO: connect to query cache
	- line 19: // TODO: consider whether to slowly kick off Redis cache updates, too


entities/teamJoinApproval/teamJoinApproval.ts (2 lines):
	- line 75: ticketType: 'ticketType', // TODO: remove tt
	- line 229: [Field.ticketType, 'tickettype'], // TODO: remove ticket type from team join approvals


lib/linkProviders/postgres/postgresLinkProvider.ts (2 lines):
	- line 173: // TODO: is an order by of interest here?
	- line 281: // TODO: how to validate updates?


api/people/links.ts (2 lines):
	- line 244: // TODO: not implemented for performance reasons now
	- line 300: // TODO: this is a cross-org map!? validate return type...


webhooks/tasks/member.ts (2 lines):
	- line 47: // TODO: then have to lookup the level of permission!
	- line 60: // TODO: may need to support the new 5 levels vs 3...


routes/link.ts (2 lines):
	- line 47: // TODO: graph provider non-guest check should be middleware and in the link business process
	- line 179: // TODO: is this code still ever really used?


jobs/refreshQueryCache/task.ts (2 lines):
	- line 42: // TODO: we mark when the slow walk starts for the process, use it to set the max age value when requesting entities (?)
	- line 43: // TODO: make max age seconds accept a function that could make it dynamic and functional


api/createRepo.ts (2 lines):
	- line 201: // TODO: we also want to store corporate manager information, eventually
	- line 335: // req.approvalRequest['ms.approvalId'] = requestId; // TODO: is this ever used?


routes/unlink.ts (2 lines):
	- line 35: // TODO: consider insights here, but we do not want to halt progress
	- line 101: // TODO: validate


api/client/organization/repos.ts (2 lines):
	- line 182: // TODO: aggressive in-memory caching for each org
	- line 193: // TODO: const language = null;


business/graphManager.ts (2 lines):
	- line 28: // TODO: rename to CachedCrossOrganizationProxy
	- line 110: const everything = await this.getUserTeams(githubId, {}); // TODO:CONFIRM: should this pass options down or not?


business/teamMember.ts (1 line):
	- line 120: // TODO: CONSIDER: a better design...


interfaces/github/members.ts (1 line):
	- line 19: type?: string; // TODO: should be an enum eventually


business/memberSearch.ts (1 line):
	- line 112: // TODO: cleanup isTeamMember approach


webhooks/tasks/auditLog.ts (1 line):
	- line 93: // TODO: corporate link?


api/client/leakyLocalCache.ts (1 line):
	- line 12: // TODO: use one of many NPMs to do this better and cleanup behind the scenes


interfaces/operations.ts (1 line):
	- line 61: orgs?: ICrossOrganizationMembershipBasics[]; // TODO: WARNING: This typing is incorrect. The object properties are the org name.


views/org/team/index.pug (1 line):
	- line 8: //- TODO: Add back team maintainer management


middleware/apiAad.ts (1 line):
	- line 14: // TODO: Caching of signing keys


entities/repositoryMetadata/repositoryMetadataProvider.ts (1 line):
	- line 83: // TODO: which storage error code would this be?


business/operations/core.ts (1 line):
	- line 298: console.log(`TODO: consider investigating the callback here as to why the getAuthorizationHeader call was not provided a purpose for the ${organizationName} org. falling back to: purpose=${purpose}`);


routes/org/team/index.ts (1 line):
	- line 35: // TODO: PERFORMANCE: The ability to lookup the e-mail address for a member should happen through a dedicated /people/login endpoint that would allow offloading the e-mail lookup functions until actually needed.


views/org/team/approvals.pug (1 line):
	- line 64: //- TODO: This is actually a upn not email


business/repository.ts (1 line):
	- line 433: // TODO: what if the branch already exists? Should let this keep running to update more PRs until done.


api/client/context/repos.ts (1 line):
	- line 47: // TODO: would be nice for team permission for repos to also store the team slug in the query cache!


api/client/banner.ts (1 line):
	- line 14: // TODO: move to modern w/administration experience, optionally


webhooks/tasks/repository.ts (1 line):
	- line 48: // TODO: Verify what happens to forks...


default-assets-package/Gruntfile.js (1 line):
	- line 122: // TODO:


features/publicReposFastFilter.ts (1 line):
	- line 9: // TODO: refresh occassionally.


middleware/links/index.ts (1 line):
	- line 88: // TODO: are multiple links selected through a session or web context setting, or ?


jobs/repositories.ts (1 line):
	- line 38: // TODO: query all, remove any not processed [recently]


routes/orgAdmin.ts (1 line):
	- line 609: // TODO: FEATURE FLAG: add a feature flag whether this API is available.


user/index.ts (1 line):
	- line 293: // TODO: if the user hasn't linked, we need to access their session/individual context's github identity here!


views/teams/index.pug (1 line):
	- line 124: //- TODO: consider fixing this to be consistent in cross-org vs single org


routes/settings/authorizations.ts (1 line):
	- line 115: // TODO: Actually delete this token/authorization


views/repos/history.pug (1 line):
	- line 45: //- TODO: consider pagination


transitional.ts (1 line):
	- line 133: // TODO: support new collaboration values as they come online for Enterprise Cloud!


lib/linkProviders/index.ts (1 line):
	- line 20: // TODO: implement a new queryByThirdPartyIds to retrieve a set of links for a set of known IDs more effectively


lib/caching/blob.ts (1 line):
	- line 83: // TODO: remove temporary message


routes/org/join.ts (1 line):
	- line 145: // TODO: is this the right approach to use with asyncHandler and sub-awaits and sub-routes?


jobs/reports/teams.ts (1 line):
	- line 261: // TODO: use the operations e-mail


lib/github/core.ts (1 line):
	- line 261: // TODO: use proper next tick to kick this off?


api/client/teams.ts (1 line):
	- line 42: // TODO: need to do lots of caching to make this awesome!


middleware/business/authentication.ts (1 line):
	- line 136: // TODO: when and where does this happen?


api/client/jsonPager.ts (1 line):
	- line 49: // TODO: this can go past the end, i.e. search while on page 7, it will not return page 1 results


lib/graphProvider/microsoftGraphProvider.ts (1 line):
	- line 66: // TODO: refactor for efficient use of Microsoft Graph's checkMemberObjects https://docs.microsoft.com/en-us/graph/api/group-checkmemberobjects?view=graph-rest-1.0&tabs=http


jobs/reports/repositories.ts (1 line):
	- line 566: // TODO: validate if this is ever even a case in the new reviewer model


middleware/apiVstsAuth.ts (1 line):
	- line 14: // TODO: consider better caching


middleware/session.ts (1 line):
	- line 66: // TODO: 2020: consider SameSite setting requirements here that are compatible with the IdP


routes/settings/approvals.ts (1 line):
	- line 26: // TODO: move to team object?


lib/staticBlobCacheFallback.ts (1 line):
	- line 58: // TODO: remove after validation


routes/placeholders.ts (1 line):
	- line 13: // TODO: remove from open source version since not helpful having random routes in place


routes/link-cleanup.ts (1 line):
	- line 21: // TODO: revisit implementation


business/application.ts (1 line):
	- line 54: // TODO: 404 vs error


api/client/organization/teams.ts (1 line):
	- line 69: // TODO: need to do lots of caching to make this awesome!


business/organizationMember.ts (1 line):
	- line 15: // TODO: review this decision, it's used by MemberSearch APIs.


business/repoSearch.ts (1 line):
	- line 204: // TODO: a more official flywheel attach vs adding an uninterfaced property