fix: [#4204] Fix remaining eslint warnings - botbuilder-dialogs (1/2)

[ceciliaavila]

Addresses #4204
#minor

Description

This PR fixes the ESLint and JSDoc issues in the botbuilder-dialogs library.

Note: we excluded the @typescript-eslint/no-explicit-any and @typescript-eslint/explicit-module-boundary-types to tackle those issues as a separate task as they require deeper analysis and testing.

Specific Changes

• Applied auto-fixes with yarn lint --fix
  • Add trailing commas.
  • Fix spacing.
  • Fix indentation.
  • Add blank line in between comments.
  • Fix quoting.
• Applied manual fixes for:
  • Missing documentation.
  • Unused parameters and imports.
  • Replacing hasOwnProperty calls.
  • Changing obsolete tags with deprecated.
  • Using template tag to document generic types.
  • Disabling detect-non-literal-fs-filename rule to allow dynamic path for files.
  • Disabling no-constant-condition rule to allow while(true) loop.
  • Fixing lodash imports.
• Added uuid as dependency (excluding it from the .depcheckrc as it incorrectly marks it as unused).
• Updated methods documentation in botbuilder-dialogs.api.md.

Testing

Here we can see the eslint status before and after.

[coveralls]

Pull Request Test Coverage Report for Build 2665212552

• 13 of 18 (72.22%) changed or added relevant lines in 14 files are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage decreased (-0.003%) to 84.479%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
libraries/botbuilder-dialogs/src/dialog.ts	0	1	0.0%
libraries/botbuilder-dialogs/src/memory/scopes/botStateMemoryScope.ts	0	1	0.0%
libraries/botbuilder-dialogs/src/memory/scopes/dialogMemoryScope.ts	1	2	50.0%
libraries/botbuilder-dialogs/src/dialogManager.ts	0	2	0.0%

Totals
Change from base Build 2656737347:	-0.003%
Covered Lines:	19904
Relevant Lines:	22312

---

💛 - Coveralls