Skip to content

chore(IT-Wallet): [SIW-3754] Bump CIE SDK version + CIE Playgrounds refactoring#7863

Open
mastro993 wants to merge 7 commits intomasterfrom
SIW-3754-cie-sdk-update
Open

chore(IT-Wallet): [SIW-3754] Bump CIE SDK version + CIE Playgrounds refactoring#7863
mastro993 wants to merge 7 commits intomasterfrom
SIW-3754-cie-sdk-update

Conversation

@mastro993
Copy link
Contributor

@mastro993 mastro993 commented Feb 11, 2026

Short description

This PR updates the CIE SDK adding support for logs (iOS) and engagement sounds (Android).

List of changes proposed in this pull request

Navigation and Structure Refactoring:

  • Introduced a new CiePlaygroundsNavigator that encapsulates all CIE-related playground screens, replacing multiple individual stack routes with a single nested navigator for better organization and maintainability. (ts/features/settings/devMode/playgrounds/Cie/navigation/CiePlaygroundsNavigator.tsx, ts/features/settings/common/navigation/SettingsNavigator.tsx, ts/features/settings/common/navigation/routes.ts, ts/features/settings/common/navigation/params/SettingsParamsList.ts, [1] [2] [3] [4] [5] [6] [7] [8]

  • Removed legacy CIE IAS+MRTD playground screens and their associated navigation routes, consolidating all related functionality under the new navigator. (ts/features/settings/devMode/playgrounds/Cie/CieIasAndMrtdPlayground.tsx, [1] [2]

Feature Additions and Improvements:

  • Added a new CieAttributesScreen allowing users to read CIE attributes, with appropriate NFC event handling and error management. (ts/features/settings/devMode/playgrounds/Cie/screens/CieAttributesScreen.tsx, ts/features/settings/devMode/playgrounds/Cie/screens/CieAttributesScreen.tsxR1-R103)

  • Refactored the existing CIE authentication playground into CieAuthenticationScreen, improved naming, and ensured compatibility with the new navigator structure. (ts/features/settings/devMode/playgrounds/Cie/screens/CieAuthenticationScreen.tsx, [1] [2]

User Interface Updates:

Dependency Update:

  • Updated the @pagopa/io-react-native-cie dependency from version 1.3.2 to 1.3.3. (package.json, package.jsonL64-R64)

How to test

On Android:

  • Navigate to Settings → Playgrounds → CIE SDK
  • Start any CIE reading test.
  • When the CIE is engaged, verify that the device emits a sound.

On iOS:

  • Navigate to Settings → Playgrounds → CIE SDK
  • Start any CIE reading test.
  • Return to the main CIE Playgrounds screen.
  • Tap View Logs.
  • Verify that logs are displayed.

Preview

Android iOS
screen-20260211-150815-1770818837864.2.mp4
ScreenRecording_02-12-2026.11-46-59_1.MP4

@mastro993 mastro993 self-assigned this Feb 11, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Feb 11, 2026

Jira Pull Request Link

This Pull Request refers to Jira issues:

@github-actions
Copy link
Contributor

PR Title Validation for conventional commit type

All good! PR title follows the conventional commit type.

@codecov
Copy link

codecov bot commented Feb 11, 2026

Codecov Report

❌ Patch coverage is 11.17318% with 159 lines in your changes missing coverage. Please review.
✅ Project coverage is 60.39%. Comparing base (a991ba4) to head (c03e18c).
⚠️ Report is 2 commits behind head on master.

Files with missing lines Patch % Lines
...e/playgrounds/Cie/screens/CieCertificateScreen.tsx 8.33% 44 Missing ⚠️
...devMode/playgrounds/Cie/screens/CiePlaygrounds.tsx 2.63% 37 Missing ⚠️
...de/playgrounds/Cie/screens/CieAttributesScreen.tsx 6.45% 29 Missing ⚠️
ts/hooks/useAppStateActive.ts 7.14% 13 Missing ⚠️
...evMode/playgrounds/Cie/screens/CieResultScreen.tsx 7.69% 12 Missing ⚠️
...ygrounds/Cie/screens/CieInternalAuthMrtdScreen.tsx 25.00% 6 Missing ⚠️
.../devMode/playgrounds/Cie/screens/CieMrtdScreen.tsx 25.00% 6 Missing ⚠️
...grounds/Cie/navigation/CiePlaygroundsNavigator.tsx 37.50% 5 Missing ⚠️
...laygrounds/Cie/screens/CieAuthenticationScreen.tsx 33.33% 4 Missing ⚠️
.../playgrounds/Cie/screens/CieInternalAuthScreen.tsx 25.00% 3 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #7863      +/-   ##
==========================================
- Coverage   60.46%   60.39%   -0.07%     
==========================================
  Files        1942     1945       +3     
  Lines       42861    42936      +75     
  Branches     9827     9830       +3     
==========================================
+ Hits        25916    25932      +16     
- Misses      16867    16926      +59     
  Partials       78       78              
Files with missing lines Coverage Δ
...s/settings/common/navigation/SettingsNavigator.tsx 100.00% <ø> (ø)
ts/features/settings/common/navigation/routes.ts 100.00% <ø> (ø)
...ttings/devMode/components/DeveloperModeSection.tsx 46.80% <ø> (+0.32%) ⬆️
...playgrounds/Cie/components/ReadStatusComponent.tsx 50.00% <ø> (ø)
...tings/devMode/playgrounds/Cie/navigation/routes.ts 100.00% <100.00%> (ø)
.../playgrounds/Cie/screens/CieInternalAuthScreen.tsx 4.44% <25.00%> (ø)
...laygrounds/Cie/screens/CieAuthenticationScreen.tsx 8.64% <33.33%> (ø)
...grounds/Cie/navigation/CiePlaygroundsNavigator.tsx 37.50% <37.50%> (ø)
...ygrounds/Cie/screens/CieInternalAuthMrtdScreen.tsx 4.00% <25.00%> (ø)
.../devMode/playgrounds/Cie/screens/CieMrtdScreen.tsx 6.25% <25.00%> (ø)
... and 5 more

... and 2 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1891e18...c03e18c. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant