Skip to content

Update telemetry client and dependencies #1236

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Jun 20, 2025

Conversation

polatengin
Copy link
Collaborator

  • Replaced @microsoft/applicationinsights-web and @microsoft/applicationinsights-react-js with updated versions.
  • Added @microsoft/applicationinsights-clickanalytics-js for enhanced click analytics.
  • Configured click analytics plugin with auto capture enabled.
  • Removed custom console error handling to simplify telemetry tracking.
  • Deleted obsolete package-lock.json file.

- Replaced @microsoft/applicationinsights-web and @microsoft/applicationinsights-react-js with updated versions.
- Added @microsoft/applicationinsights-clickanalytics-js for enhanced click analytics.
- Configured click analytics plugin with auto capture enabled.
- Removed custom console error handling to simplify telemetry tracking.
- Deleted obsolete package-lock.json file.
@polatengin polatengin self-assigned this Jun 19, 2025
@polatengin polatengin requested a review from BSamodien as a code owner June 19, 2025 20:15
- Updated error tracking in FeedbackBoardContainer, WorkService, and dataService to simplify exception handling by removing the properties wrapper in appInsights.trackException calls.
- Removed the ClickAnalyticsPlugin from telemetryClient and its related configurations, streamlining the telemetry setup.
- Cleaned up package.json and package-lock.json by removing the Click Analytics dependency and updating related packages.
@codecov-commenter
Copy link

codecov-commenter commented Jun 19, 2025

Codecov Report

Attention: Patch coverage is 45.83333% with 13 lines in your changes missing coverage. Please review.

Project coverage is 51.94%. Comparing base (5350004) to head (426309c).
Report is 4 commits behind head on main.

Files with missing lines Patch % Lines
src/frontend/components/feedbackBoardContainer.tsx 33.33% 4 Missing ⚠️
src/frontend/dal/dataService.tsx 20.00% 4 Missing ⚠️
src/frontend/components/boardSummaryTable.tsx 0.00% 3 Missing ⚠️
src/frontend/dal/itemDataService.tsx 33.33% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1236      +/-   ##
==========================================
+ Coverage   50.41%   51.94%   +1.52%     
==========================================
  Files          46       46              
  Lines        3360     3344      -16     
  Branches      898      899       +1     
==========================================
+ Hits         1694     1737      +43     
+ Misses       1666     1607      -59     
Files with missing lines Coverage Δ
src/frontend/dal/azureDevOpsWorkService.tsx 96.15% <100.00%> (-0.28%) ⬇️
src/frontend/dal/boardDataService.tsx 96.25% <ø> (-0.05%) ⬇️
src/frontend/dal/reflectBackendService.tsx 53.33% <100.00%> (+0.05%) ⬆️
src/frontend/utilities/telemetryClient.ts 100.00% <100.00%> (ø)
src/frontend/dal/itemDataService.tsx 25.26% <33.33%> (+0.08%) ⬆️
src/frontend/components/boardSummaryTable.tsx 37.07% <0.00%> (-0.21%) ⬇️
src/frontend/components/feedbackBoardContainer.tsx 22.29% <33.33%> (+0.03%) ⬆️
src/frontend/dal/dataService.tsx 50.87% <20.00%> (+2.49%) ⬆️

... and 1 file with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@codecov-commenter
Copy link

codecov-commenter commented Jun 19, 2025

Bundle Report

Changes will decrease total bundle size by 1.0kB (-0.06%) ⬇️. This is within the configured threshold ✅

Detailed changes
Bundle name Size Change
retrospective-extension-webpack-bundle-array-push 1.68MB -1.0kB (-0.06%) ⬇️

Affected Assets, Files, and Routes:

view changes for bundle: retrospective-extension-webpack-bundle-array-push

Assets Changed:

Asset Name Size Change Total Size Change (%)
./reflect-bundle.js -1.0kB 1.68MB -0.06%

Files in ./reflect-bundle.js:

  • ./dal/reflectBackendService.tsx → Total Size: 19.24kB

  • ./components/feedbackBoardContainer.tsx → Total Size: 128.89kB

  • ./dal/itemDataService.tsx → Total Size: 50.87kB

  • ./components/boardSummaryTable.tsx → Total Size: 30.93kB

  • ./dal/boardDataService.tsx → Total Size: 15.89kB

  • ./utilities/telemetryClient.ts → Total Size: 2.7kB

  • ./dal/dataService.tsx → Total Size: 13.17kB

  • ./dal/azureDevOpsWorkService.tsx → Total Size: 7.02kB

@polatengin polatengin merged commit 85366e2 into main Jun 20, 2025
8 checks passed
@polatengin polatengin deleted the polatengin/fix-application-insights branch June 20, 2025 23:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants