[BUG] ITelemetryTrace parentId cannot be set to undefined

[zmarty]

App Insights Browser (JS) SDK

I think this is a bug in your TypeScript types and parentId shoud be nullable. I verified in the browser that the first time an operation starts its parent is set to undefined, so why can't I do the same? I specifically want to be able to set parentId to undefined for my purposes, since I track router changes myself.

The ITelemetryTrace interface says parentId should be string:

But TelemetryTrace (which implements ITelemetryTrace) allows setting it to undefined:

Here's the ugly workaround I used:

// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
this.appInsights.context.telemetryTrace.parentID = parentId!;

(where parentId can be undefined)

[MSNev]

I think the compile failure would be because of your eslint config, by default we are only using
"eslint:recommended",
"plugin:@typescript-eslint/recommended",
"plugin:security/recommended"

Having said that after looking at the code, I can't see any reason why this can't be optional

[MSNev]

v2.7.0 is not fully deployed to NPM and all CDN endpoints

[github-actions]

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.