[engsys][monitor-query] upgrade typescript version to ~4.8.0 (Azure#24582)

to be consistent with most packages in the repository.

- Fix compiler errors. Use `Extract<Thing, Record<PropertyName,
unknown>>` instead of `Thing & Record<PropertyName, unknown>`

Author: jeremymeng

common/config/rush/pnpm-lock.yaml

@@ -144,7 +144,7 @@
     "prettier": "^2.5.1",
     "rimraf": "^3.0.0",
     "source-map-support": "^0.5.9",
-    "typescript": "~4.6.0",
+    "typescript": "~4.8.0",
     "@azure-tools/test-credential": "^1.0.0"
   },
   "//sampleConfiguration": {

sdk/monitor/monitor-query/package.json

@@ -47,7 +47,7 @@ export function convertIntervalToTimeIntervalObject(timespan: string): QueryTime
 export function objectHasProperty<Thing, PropertyName extends string>(
   thing: Thing,
   property: PropertyName
-): thing is Thing & Record<PropertyName, unknown> {
+): thing is Extract<Thing, Record<PropertyName, unknown>> {
   return typeof thing === "object" && property in (thing as Record<string, unknown>);
 }
 
@@ -70,7 +70,7 @@ export function isDefined<T>(thing: T | undefined | null): thing is T {
 export function isObjectWithProperties<Thing, PropertyName extends string>(
   thing: Thing,
   properties: PropertyName[]
-): thing is Thing & Record<PropertyName, unknown> {
+): thing is Extract<Thing, Record<PropertyName, unknown>> {
   if (!isDefined(thing) || typeof thing !== "object") {
     return false;
   }