build: remove all tslint references from codebase

[Copilot]

Description

Removes all tslint references since the project migrated to eslint long ago.

Build tools:

• Remove TsLintTask class from lintTasks.ts
• Remove tslint from executableToLeafTask map in taskFactory.ts
• Update README to remove tslint documentation

VSCode configuration:

• Remove $tslint5 problem matcher from .vscode/tasks.json files

Server/historian:

• Remove tslint dev dependency and script from package.json files
• Regenerate pnpm-lock.yaml

Test cleanup:

• Remove 34 // tslint:disable-next-line comments from basicRestwrapper.spec.ts

Note: @typescript-eslint/ban-tslint-comment references in eslint-config-fluid are retained—these are eslint rules that enforce the tslint→eslint migration.

Reviewer Guidance

Straightforward removal. Build-tools and historian packages compile and tests pass.

Original prompt

We have long since migrated to eslint from tslint, so remove all references to tslint, including task support in build tools, disable comments, etc.

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

[tylerbutler]

/azp run Build - protocol-definitions,Build - test-tools,server-gitrest,server-gitssh,server-routerlicious,Build - client packages,repo-policy-check,Build - build-tools

[tylerbutler]

/azp run Build - api-markdown-documenter,Build - benchmark-tool,Build - build-common,Build - build-tools,Build - common-utils,Build - eslint-config-fluid,Build - eslint-plugin-fluid,server-historian

[azure-pipelines]

Azure Pipelines will not run the associated pipelines, because the pull request was updated after the run command was issued. Review the pull request again and issue a new run command.

[azure-pipelines]

Azure Pipelines will not run the associated pipelines, because the pull request was updated after the run command was issued. Review the pull request again and issue a new run command.

[tylerbutler]

@copilot merge the latest main branch and address the merge conflict in lintTasks.ts. DO NOT REBASE!

[Copilot]

@copilot merge the latest main branch and address the merge conflict in lintTasks.ts. DO NOT REBASE!

Merged main and resolved the conflict in lintTasks.ts in commit e4510b0. Kept the new EsLintTask functionality from main (shared config tracking) while removing the TsLintTask class.

[tylerbutler]

@copilot You did not make a proper merge commit. Revert the last commit, then MERGE the latest main branch. DO NOT REBASE.

[Copilot]

@copilot You did not make a proper merge commit. Revert the last commit, then MERGE the latest main branch. DO NOT REBASE.

Reverted the bad commits in 8d47650 and created a proper merge commit with two parents in cc2259c.

[tylerbutler]

/azp run Build - protocol-definitions,Build - test-tools,server-gitrest,server-gitssh,server-routerlicious,Build - client packages,repo-policy-check,Build - build-tools

[tylerbutler]

/azp run Build - api-markdown-documenter,Build - benchmark-tool,Build - build-common,Build - build-tools,Build - common-utils,Build - eslint-config-fluid,Build - eslint-plugin-fluid,server-historian

[azure-pipelines]

Azure Pipelines successfully started running 2 pipeline(s).

[azure-pipelines]

Azure Pipelines successfully started running 4 pipeline(s).

[Copilot]

Pull request overview

This PR removes all remaining tslint references from the codebase following the historical migration to eslint. The cleanup spans build tools, server packages, test files, VSCode configuration, and documentation.

• Removes build tool support for tslint tasks
• Removes tslint dependencies from historian server packages
• Removes 34 obsolete tslint disable comments from test files

Reviewed changes

Copilot reviewed 8 out of 9 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
server/routerlicious/packages/services-client/src/test/basicRestwrapper.spec.ts	Removes 34 // tslint:disable-next-line comments from test assertions
server/historian/pnpm-lock.yaml	Removes tslint and its dependencies (builtin-modules, [email protected], tsutils), marks orphaned transitive dependencies as optional
server/historian/packages/historian/package.json	Removes tslint dev dependency and tslint script
server/historian/package.json	Removes tslint dev dependency from root historian package
packages/test/test-end-to-end-tests/.vscode/tasks.json	Removes $tslint5 problem matcher from VSCode task configuration
build-tools/packages/build-tools/src/fluidBuild/tasks/taskFactory.ts	Removes tslint entry from executableToLeafTask map and removes TsLintTask import
build-tools/packages/build-tools/src/fluidBuild/tasks/leaf/lintTasks.ts	Removes entire TsLintTask class implementation
build-tools/packages/build-tools/README.md	Updates documentation to remove tslint references from task descriptions
.vscode/tasks.json	Removes $tslint5 problem matcher from two task configurations

Files not reviewed (1)

• server/historian/pnpm-lock.yaml: Language not supported

[tylerbutler]

/azp run Build - protocol-definitions,Build - test-tools,server-gitrest,server-gitssh,server-routerlicious,Build - client packages,repo-policy-check,Build - build-tools

[tylerbutler]

/azp run Build - api-markdown-documenter,Build - benchmark-tool,Build - build-common,Build - build-tools,Build - common-utils,Build - eslint-config-fluid,Build - eslint-plugin-fluid,server-historian

[azure-pipelines]

Azure Pipelines successfully started running 2 pipeline(s).

[azure-pipelines]

Azure Pipelines successfully started running 4 pipeline(s).