chore: remove @microsoft/applicationinsights-common dependency#255
Conversation
|
@copilot Check whether BreezeChannelIdentifier can be imported from one of the remaining applicationinsights packages. |
Yes — |
…ore-js Both @microsoft/1ds-post-js@4.4.1 and @microsoft/1ds-core-js@4.4.1 pin @microsoft/applicationinsights-core-js to exactly 3.4.1, while the top-level dependency resolves to 3.4.2. npm installs nested duplicate copies, causing TypeScript to see structurally distinct but incompatible IXHROverride types from different module resolution paths. Adding the npm override forces all packages to use the same version, eliminating duplicate nested copies and fixing the TS2345 compile error.
…plicate @microsoft/1ds-core-js@4.4.2 and @microsoft/1ds-post-js@4.4.2 updated their pinned dependency on @microsoft/applicationinsights-core-js from 3.4.1 to 3.4.2, matching the top-level version. npm can now naturally deduplicate to a single copy, eliminating the TypeScript type mismatch without needing the override workaround.
Done in commit |
|
@copilot Also follow this |
@microsoft/1ds-core-js is deprecated; its functionality has been merged into @microsoft/applicationinsights-core-js. Update 1dsClientFactory.ts to import AppInsightsCore and IExtendedConfiguration directly from @microsoft/applicationinsights-core-js and remove the 1ds-core-js dependency from package.json.
Done in commit |
Drops
@microsoft/applicationinsights-common— the sole usage was importingBreezeChannelIdentifier, a string constant whose value is"AppInsightsChannelPlugin".Changes
src/common/appInsightsClientFactory.ts: ReplaceBreezeChannelIdentifierimport with the inlined literal"AppInsightsChannelPlugin"package.json/package-lock.json: Remove@microsoft/applicationinsights-commonfrom dependencies