[SELF-703] feat: Migrate mock generator to mobile sdk

[transphorm]

Summary

• move mock passport generation utilities into mobile SDK
• add reusable useMockDataForm hook for mock screen state
• update MockDataScreen to consume SDK generator and new hook

Testing

• yarn lint
• yarn build
• yarn workspace @selfxyz/contracts build (fails: Invalid account ...)
• yarn types
• yarn workspace @selfxyz/mobile-app test (fails: ENOENT dist/cjs/index.cjs)
• yarn workspace @selfxyz/mobile-sdk-alpha test

---

https://chatgpt.com/codex/tasks/task_b_68b27d85ea60832d934ee0537053e382

Summary by CodeRabbit

• New Features

  • Single-step mock document generation in the Dev Mock Data screen.
  • Algorithm options now sourced from the SDK.
  • Centralized mock-data form with default values and a Reset action.
  • UI text updated to "Mock Document" across the creation flow.

• Refactor

  • Streamlined generation flow and simplified error handling using SDK-provided generation.
  • Replaced local per-field state with a shared form hook.

• Tests

  • Added comprehensive unit tests for mock generation, algorithm mappings, edge cases, and fallbacks.

[coderabbitai]

Warning

Rate limit exceeded

@transphorm has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 12 minutes and 19 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📥 Commits

Reviewing files that changed from the base of the PR and between 3e6efaf and 2b0f7de.

📒 Files selected for processing (4)

• packages/mobile-sdk-alpha/src/browser.ts (1 hunks)
• packages/mobile-sdk-alpha/src/index.ts (1 hunks)
• packages/mobile-sdk-alpha/src/mock/generator.ts (1 hunks)
• packages/mobile-sdk-alpha/tests/mock/generator.test.ts (1 hunks)

Walkthrough

Adds a centralized mock-data flow: a new hook manages form state, the dev screen calls a single generateMockPassport API, and the SDK exposes a generator plus an algorithm mapping. A new generator module implements DSC/id-doc creation, parsing, and fallbacks, with comprehensive unit tests added.

Changes

Cohort / File(s)	Summary
Hook: Centralized form state app/src/hooks/useMockDataForm.ts	Adds a React hook encapsulating mock-data form state (age, expiryYears, country, algorithm, document type, OFAC flag), selection handlers, setters, and reset. Exports named and default useMockDataForm.
Dev screen refactor app/src/screens/dev/MockDataScreen.tsx	Replaces multi-step local generation with generateMockPassport. Removes local helpers, constants, and fallback logic. Consumes useMockDataForm. Updates imports to use SDK generator and algorithm mapping; updates UI wording to “document”.
SDK public exports packages/mobile-sdk-alpha/src/index.ts, packages/mobile-sdk-alpha/src/browser.ts	Re-exports generateMockPassport and signatureAlgorithmToStrictSignatureAlgorithm from ./mock/generator. No other logic changes.
Generator module packages/mobile-sdk-alpha/src/mock/generator.ts	Adds GenerateMockPassportOptions, generateMockPassport(...), and signatureAlgorithmToStrictSignatureAlgorithm. Implements ID-doc assembly, mapping algorithm → hash/signature tuple, DSC generation with fallback, SKI PEM fetch, and parsing to produce parsed mock data.
Tests for generator packages/mobile-sdk-alpha/tests/mock/generator.test.ts	Adds unit tests covering date helpers, generator inputs/outputs, DSC fallback, algorithm mapping integrity, uniqueness of passport numbers, and cross-scenario behavior. Mocks external dependencies.
Navigation text app/src/navigation/devTools.ts	Renames the CreateMock screen title from "Mock Passport" to "Mock Document".

Sequence Diagram(s)

sequenceDiagram
  autonumber
  participant DevScreen as MockDataScreen
  participant Hook as useMockDataForm
  participant SDK as mobile-sdk-alpha
  participant Gen as generateMockPassport
  participant Store as storePassportData
  participant Nav as Navigation

  DevScreen->>Hook: Read form values
  DevScreen->>SDK: generateMockPassport(options)
  SDK->>Gen: Invoke with mapped algorithm tuple
  Gen-->>SDK: parsedMockData
  SDK-->>DevScreen: parsedMockData
  DevScreen->>Store: storePassportData(parsedMockData)
  DevScreen->>Hook: resetFormValues()
  DevScreen->>Nav: navigate("MockPassport")

Loading

sequenceDiagram
  autonumber
  participant Gen as generateMockPassport
  participant Map as signatureAlgorithmToStrictSignatureAlgorithm
  participant DSC as generateMockDSC
  participant ID as genMockIdDoc
  participant SKI as getSKIPEM("staging")
  participant Parse as initPassportDataParsing

  Gen->>Map: Resolve [dgHash, eContentHash, signatureType]
  Gen->>DSC: Attempt create DSC(signatureType)
  alt DSC success
    DSC-->>Gen: dsc
  else DSC failure
    Note over Gen: Proceed without DSC (fallback)
  end
  Gen->>ID: genMockIdDoc(idDocPayload, { privateKeyPem?, dsc? })
  Gen->>SKI: Fetch SKI PEM map
  Gen->>Parse: initPassportDataParsing({ idDoc, skiPemMap })
  Parse-->>Gen: parsedMockData
  Gen-->>Caller: parsedMockData

Loading

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~60 minutes

Possibly related PRs

• App/ethcc fixes #730: Changes around signatureAlgorithmToStrictSignatureAlgorithm usage and propagation of hash/signature tuple into ID-doc inputs—closely related to the exported mapping here.
• SEL-544: Generate Mock DSC on mock-passport flow #772: Prior work centralizing DSC generation and passing DSC into ID-doc creation aligns with this new generator and its fallback behavior.
• SELF-498: Implement latest DevSettings & CreateMock screens #778: Earlier edits to mock data form fields and generation flow overlap with the new hook and generateMockPassport integration.

Suggested reviewers

• aaronmgdr

Poem

Pick a country, set the age, choose the key,
A mock document spins up from a single API.
Hooks hold the state, the generator does the rest,
Tests stand at the gate so the flow stays blessed.
A tiny stamp, a random number—you're now a guest. ✨

✨ Finishing Touches

• 📝 Generate Docstrings

🧪 Generate unit tests

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch codex/ideate-mockdatascreen-migration-to-sdk

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

• Review comments: Directly reply to a review comment made by CodeRabbit. Example:
  • I pushed a fix in commit <commit_id>, please review it.
  • Open a follow-up GitHub issue for this discussion.
• Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query.
• PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
  • @coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
  • @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.

Support

Need help? Create a ticket on our support page for assistance with any issues or questions.

CodeRabbit Commands (Invoked using PR/Issue comments)

Type @coderabbitai help to get the list of available commands.

Other keywords and placeholders

• Add @coderabbitai ignore or @coderabbit ignore anywhere in the PR description to prevent this PR from being reviewed.
• Add @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
• Add @coderabbitai anywhere in the PR title to generate the title automatically.

Status, Documentation and Community

• Visit our Status Page to check the current availability of CodeRabbit.
• Visit our Documentation for detailed information on how to use CodeRabbit.
• Join our Discord Community to get help, request features, and share feedback.
• Follow us on X/Twitter for updates and announcements.

[coderabbitai]

Actionable comments posted: 6

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)

app/src/screens/dev/MockDataScreen.tsx (1)

335-342: Bug: Age decrement uses the wrong disable condition and allows negative ages

Minus button is disabled based on expiryYears, not age, and onPress can drive age below 0, leading to invalid inputs downstream. Clamp and fix the disabled condition.

Apply:

-                  onPress={() => {
-                    buttonTap();
-                    setAge(age - 1);
-                    trackEvent(MockDataEvents.DECREASE_AGE);
-                  }}
-                  disabled={expiryYears <= 0}
+                  onPress={() => {
+                    buttonTap();
+                    setAge(prev => Math.max(0, prev - 1));
+                    trackEvent(MockDataEvents.DECREASE_AGE);
+                  }}
+                  disabled={age <= 0}

🧹 Nitpick comments (9)

packages/mobile-sdk-alpha/src/index.ts (1)

89-90: Re-export the options type to stabilize the public API.

Expose the options interface so app code doesn’t import from private paths.

 export { generateMockPassport, signatureAlgorithmToStrictSignatureAlgorithm } from './mock/generator';
+export type { GenerateMockPassportOptions } from './mock/generator';

app/src/hooks/useMockDataForm.ts (1)

32-34: Tighten algorithm typing to the SDK’s allowed keys.

Once the SDK exports a SignatureAlgorithm key type, use it here to prevent invalid selections.

-  const handleAlgorithmSelect = (algorithm: string) => {
+  // import type { SignatureAlgorithm } from '@selfxyz/mobile-sdk-alpha'
+  const handleAlgorithmSelect = (algorithm: SignatureAlgorithm) => {
     setSelectedAlgorithm(algorithm);
   };

packages/mobile-sdk-alpha/src/mock/generator.ts (4)

52-60: Normalize nationality to uppercase to reduce invalid inputs.

This improves interop with validators expecting canonical codes.

-  const idDocInput: Partial<IdDocInput> = {
-    nationality: selectedCountry as IdDocInput['nationality'],
+  const idDocInput: Partial<IdDocInput> = {
+    nationality: (selectedCountry || '').toUpperCase() as IdDocInput['nationality'],

---

34-41: Clamp age/expiry inputs to sane ranges to prevent nonsensical dates.

Protects against negative ages or extreme expiries.

 export async function generateMockPassport({
   age,
   expiryYears,
@@
 }: GenerateMockPassportOptions) {
+  const clampedAge = Math.max(0, Math.min(age ?? 0, 120));
+  const clampedExpiryYears = Math.max(1, Math.min(expiryYears ?? 1, 30));
@@
-    expiryDate: getExpiryDateFromYears(expiryYears),
+    expiryDate: getExpiryDateFromYears(clampedExpiryYears),
@@
-    dobForGeneration = getBirthDateFromAge(age);
+    dobForGeneration = getBirthDateFromAge(clampedAge);

Also applies to: 58-59, 68-71

---

72-79: Prefer SDK logger over console.warn for consistency.

Route through your LoggerAdapter or sdkError to honor app logging pipelines.

-  } catch (error) {
-    console.warn('Falling back to default mock DSC. Error during mock DSC generation:', error);
+  } catch (error) {
+    // TODO: inject logger; for now prefix clearly for filters
+    console.warn('[mobile-sdk-alpha/mock] Falling back to default mock DSC:', error);

---

84-119: Freeze the public mapping to prevent accidental runtime mutation.

Prevents consumers from altering the mapping in JS.

-export const signatureAlgorithmToStrictSignatureAlgorithm = {
+export const signatureAlgorithmToStrictSignatureAlgorithm = Object.freeze({
@@
-} as const;
+}) as const;

packages/mobile-sdk-alpha/tests/mock/generator.test.ts (2)

7-18: Consider importing from the public entry to validate re-exports.

Keeps tests aligned with the package surface and catches export regressions.

-import { generateMockPassport, signatureAlgorithmToStrictSignatureAlgorithm } from '../../src/mock/generator';
+import { generateMockPassport, signatureAlgorithmToStrictSignatureAlgorithm } from '../../src/index';

---

428-449: Add an integration test asserting the generated doc passes isPassportDataValid().

This ensures the new generator produces data compatible with the validator.

I can add a test that awaits generateMockPassport(..) then calls isPassportDataValid(result) and asserts success.

app/src/screens/dev/MockDataScreen.tsx (1)

162-176: Optional: Derive or validate the default algorithm
The hard-coded default 'sha256 rsa 65537 2048' currently matches a key in signatureAlgorithmToStrictSignatureAlgorithm, so there’s no immediate crash risk. To future-proof against mapping changes, consider deriving the default from the mapping (e.g. Object.keys(signatureAlgorithmToStrictSignatureAlgorithm)[0]) or validating selectedAlgorithm before passing it into generateMockPassport.

📜 Review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

💡 Knowledge Base configuration:

• MCP integration is disabled by default for public repositories
• Jira integration is disabled by default for public repositories
• Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between 12aff53 and d29cd10.

📒 Files selected for processing (6)

• app/src/hooks/useMockDataForm.ts (1 hunks)
• app/src/screens/dev/MockDataScreen.tsx (2 hunks)
• packages/mobile-sdk-alpha/src/browser.ts (1 hunks)
• packages/mobile-sdk-alpha/src/index.ts (1 hunks)
• packages/mobile-sdk-alpha/src/mock/generator.ts (1 hunks)
• packages/mobile-sdk-alpha/tests/mock/generator.test.ts (1 hunks)

🧰 Additional context used

📓 Path-based instructions (7)

**/*.{ts,tsx}

📄 CodeRabbit inference engine (.cursor/rules/technical-specification.mdc)

**/*.{ts,tsx}: Define IdentityCommitment with fields: commitment (Poseidon hash), nullifier (domain-separated), timestamp (UTC number), version (circuit version), documentType ('passport' | 'eu_id_card')
Define DSCKeyCommitment with fields: publicKeyHash (Poseidon hash), certificateChain (hashes), revocationStatus (boolean), issuer (country code)
Define VerificationConfig with fields: circuitVersion (semver), complianceRules array, timeWindow (seconds, 24h), clockDrift (±5 min), trustAnchors, revocationRoots, timeSource (NTP), nullifierScope (domain separation)

Files:

• packages/mobile-sdk-alpha/src/browser.ts
• app/src/hooks/useMockDataForm.ts
• packages/mobile-sdk-alpha/src/index.ts
• packages/mobile-sdk-alpha/tests/mock/generator.test.ts
• packages/mobile-sdk-alpha/src/mock/generator.ts
• app/src/screens/dev/MockDataScreen.tsx

packages/mobile-sdk-alpha/**/*.{ts,tsx}

📄 CodeRabbit inference engine (packages/mobile-sdk-alpha/AGENTS.md)

packages/mobile-sdk-alpha/**/*.{ts,tsx}: Use strict TypeScript type checking across the codebase
Follow ESLint TypeScript-specific rules
Avoid introducing circular dependencies

Files:

• packages/mobile-sdk-alpha/src/browser.ts
• packages/mobile-sdk-alpha/src/index.ts
• packages/mobile-sdk-alpha/tests/mock/generator.test.ts
• packages/mobile-sdk-alpha/src/mock/generator.ts

packages/mobile-sdk-alpha/**/*.{ts,tsx,js,jsx}

⚙️ CodeRabbit configuration file

packages/mobile-sdk-alpha/**/*.{ts,tsx,js,jsx}: Review alpha mobile SDK code for:

• API consistency with core SDK
• Platform-neutral abstractions
• Performance considerations
• Clear experimental notes or TODOs

Files:

• packages/mobile-sdk-alpha/src/browser.ts
• packages/mobile-sdk-alpha/src/index.ts
• packages/mobile-sdk-alpha/tests/mock/generator.test.ts
• packages/mobile-sdk-alpha/src/mock/generator.ts

app/src/**/*.{ts,tsx,js,jsx}

⚙️ CodeRabbit configuration file

app/src/**/*.{ts,tsx,js,jsx}: Review React Native TypeScript code for:

• Component architecture and reusability
• State management patterns
• Performance optimizations
• TypeScript type safety
• React hooks usage and dependencies
• Navigation patterns

Files:

• app/src/hooks/useMockDataForm.ts
• app/src/screens/dev/MockDataScreen.tsx

packages/mobile-sdk-alpha/src/index.ts

📄 CodeRabbit inference engine (.cursor/rules/mobile-sdk-migration.mdc)

Re-export new SDK modules via packages/mobile-sdk-alpha/src/index.ts

Files:

• packages/mobile-sdk-alpha/src/index.ts

packages/mobile-sdk-alpha/{**/*.test.{ts,tsx},**/__tests__/**/*.{ts,tsx}}

📄 CodeRabbit inference engine (packages/mobile-sdk-alpha/AGENTS.md)

packages/mobile-sdk-alpha/{**/*.test.{ts,tsx},**/__tests__/**/*.{ts,tsx}}: Do NOT mock @selfxyz/mobile-sdk-alpha in tests (avoid jest.mock('@selfxyz/mobile-sdk-alpha') and replacing real functions with mocks)
Use actual imports from @selfxyz/mobile-sdk-alpha in tests
Write integration tests that exercise the real validation logic (not mocks)
Test isPassportDataValid() with realistic synthetic passport data (never real user data)
Verify extractMRZInfo() using published sample MRZ strings (e.g., ICAO examples)
Ensure parseNFCResponse() works with representative, synthetic NFC data
Never use real user PII in tests; use only synthetic, anonymized, or approved test vectors

Files:

• packages/mobile-sdk-alpha/tests/mock/generator.test.ts

**/*.{test,spec}.{ts,js,tsx,jsx}

⚙️ CodeRabbit configuration file

**/*.{test,spec}.{ts,js,tsx,jsx}: Review test files for:

• Test coverage completeness
• Test case quality and edge cases
• Mock usage appropriateness
• Test readability and maintainability

Files:

• packages/mobile-sdk-alpha/tests/mock/generator.test.ts

🧠 Learnings (17)

📓 Common learnings

Learnt from: CR
PR: selfxyz/self#0
File: packages/mobile-sdk-alpha/AGENTS.md:0-0
Timestamp: 2025-08-29T15:31:15.911Z
Learning: Applies to packages/mobile-sdk-alpha/{**/*.test.{ts,tsx},**/__tests__/**/*.{ts,tsx}} : Test isPassportDataValid() with realistic synthetic passport data (never real user data)

📚 Learning: 2025-08-24T18:54:04.809Z

Learnt from: CR
PR: selfxyz/self#0
File: .cursor/rules/mobile-sdk-migration.mdc:0-0
Timestamp: 2025-08-24T18:54:04.809Z
Learning: Applies to packages/mobile-sdk-alpha/src/index.ts : Re-export new SDK modules via packages/mobile-sdk-alpha/src/index.ts

Applied to files:

• packages/mobile-sdk-alpha/src/browser.ts
• packages/mobile-sdk-alpha/src/index.ts
• packages/mobile-sdk-alpha/tests/mock/generator.test.ts
• packages/mobile-sdk-alpha/src/mock/generator.ts

📚 Learning: 2025-08-29T15:31:15.911Z

Learnt from: CR
PR: selfxyz/self#0
File: packages/mobile-sdk-alpha/AGENTS.md:0-0
Timestamp: 2025-08-29T15:31:15.911Z
Learning: Applies to packages/mobile-sdk-alpha/{**/*.test.{ts,tsx},**/__tests__/**/*.{ts,tsx}} : Test isPassportDataValid() with realistic synthetic passport data (never real user data)

Applied to files:

• packages/mobile-sdk-alpha/src/browser.ts
• app/src/hooks/useMockDataForm.ts
• packages/mobile-sdk-alpha/src/index.ts
• packages/mobile-sdk-alpha/tests/mock/generator.test.ts
• packages/mobile-sdk-alpha/src/mock/generator.ts
• app/src/screens/dev/MockDataScreen.tsx

📚 Learning: 2025-08-29T15:31:15.911Z

Learnt from: CR
PR: selfxyz/self#0
File: packages/mobile-sdk-alpha/AGENTS.md:0-0
Timestamp: 2025-08-29T15:31:15.911Z
Learning: Applies to packages/mobile-sdk-alpha/{**/*.test.{ts,tsx},**/__tests__/**/*.{ts,tsx}} : Do NOT mock selfxyz/mobile-sdk-alpha in tests (avoid jest.mock('selfxyz/mobile-sdk-alpha') and replacing real functions with mocks)

Applied to files:

• packages/mobile-sdk-alpha/src/browser.ts
• packages/mobile-sdk-alpha/src/index.ts
• packages/mobile-sdk-alpha/tests/mock/generator.test.ts
• packages/mobile-sdk-alpha/src/mock/generator.ts
• app/src/screens/dev/MockDataScreen.tsx

📚 Learning: 2025-08-29T15:31:15.911Z

Learnt from: CR
PR: selfxyz/self#0
File: packages/mobile-sdk-alpha/AGENTS.md:0-0
Timestamp: 2025-08-29T15:31:15.911Z
Learning: Applies to packages/mobile-sdk-alpha/**/package.json : Ensure package exports are properly configured

Applied to files:

• packages/mobile-sdk-alpha/src/browser.ts
• packages/mobile-sdk-alpha/src/index.ts

📚 Learning: 2025-08-29T15:31:15.911Z

Learnt from: CR
PR: selfxyz/self#0
File: packages/mobile-sdk-alpha/AGENTS.md:0-0
Timestamp: 2025-08-29T15:31:15.911Z
Learning: Applies to packages/mobile-sdk-alpha/{**/*.test.{ts,tsx},**/__tests__/**/*.{ts,tsx}} : Use actual imports from selfxyz/mobile-sdk-alpha in tests

Applied to files:

• packages/mobile-sdk-alpha/src/browser.ts
• packages/mobile-sdk-alpha/src/index.ts
• packages/mobile-sdk-alpha/tests/mock/generator.test.ts
• app/src/screens/dev/MockDataScreen.tsx

📚 Learning: 2025-08-29T15:31:15.911Z

Learnt from: CR
PR: selfxyz/self#0
File: packages/mobile-sdk-alpha/AGENTS.md:0-0
Timestamp: 2025-08-29T15:31:15.911Z
Learning: Applies to packages/mobile-sdk-alpha/{**/*.test.{ts,tsx},**/__tests__/**/*.{ts,tsx}} : Verify extractMRZInfo() using published sample MRZ strings (e.g., ICAO examples)

Applied to files:

• packages/mobile-sdk-alpha/src/browser.ts
• packages/mobile-sdk-alpha/src/index.ts
• packages/mobile-sdk-alpha/tests/mock/generator.test.ts
• packages/mobile-sdk-alpha/src/mock/generator.ts

📚 Learning: 2025-08-29T15:31:15.911Z

Learnt from: CR
PR: selfxyz/self#0
File: packages/mobile-sdk-alpha/AGENTS.md:0-0
Timestamp: 2025-08-29T15:31:15.911Z
Learning: Applies to packages/mobile-sdk-alpha/**/*.{ts,tsx} : Avoid introducing circular dependencies

Applied to files:

• packages/mobile-sdk-alpha/src/browser.ts
• packages/mobile-sdk-alpha/src/index.ts

📚 Learning: 2025-08-24T18:54:04.809Z

Learnt from: CR
PR: selfxyz/self#0
File: .cursor/rules/mobile-sdk-migration.mdc:0-0
Timestamp: 2025-08-24T18:54:04.809Z
Learning: Applies to packages/mobile-sdk-alpha/README.md : Document new/updated SDK modules and usage in packages/mobile-sdk-alpha/README.md

Applied to files:

• packages/mobile-sdk-alpha/src/browser.ts
• packages/mobile-sdk-alpha/src/index.ts

📚 Learning: 2025-08-29T15:31:15.911Z

Learnt from: CR
PR: selfxyz/self#0
File: packages/mobile-sdk-alpha/AGENTS.md:0-0
Timestamp: 2025-08-29T15:31:15.911Z
Learning: Applies to packages/mobile-sdk-alpha/{**/*.test.{ts,tsx},**/__tests__/**/*.{ts,tsx}} : Never use real user PII in tests; use only synthetic, anonymized, or approved test vectors

Applied to files:

• packages/mobile-sdk-alpha/src/browser.ts
• packages/mobile-sdk-alpha/tests/mock/generator.test.ts

📚 Learning: 2025-08-29T15:31:15.911Z

Learnt from: CR
PR: selfxyz/self#0
File: packages/mobile-sdk-alpha/AGENTS.md:0-0
Timestamp: 2025-08-29T15:31:15.911Z
Learning: Applies to packages/mobile-sdk-alpha/**/*.{ts,tsx} : Use strict TypeScript type checking across the codebase

Applied to files:

• packages/mobile-sdk-alpha/src/browser.ts

📚 Learning: 2025-08-25T14:25:57.586Z

Learnt from: aaronmgdr
PR: selfxyz/self#951
File: app/src/providers/authProvider.web.tsx:17-18
Timestamp: 2025-08-25T14:25:57.586Z
Learning: The selfxyz/mobile-sdk-alpha/constants/analytics import path is properly configured with SDK exports, Metro aliases, and TypeScript resolution. Import changes from @/consts/analytics to this path are part of valid analytics migration, not TypeScript resolution issues.

Applied to files:

• packages/mobile-sdk-alpha/src/index.ts

📚 Learning: 2025-08-24T18:54:04.809Z

Learnt from: CR
PR: selfxyz/self#0
File: .cursor/rules/mobile-sdk-migration.mdc:0-0
Timestamp: 2025-08-24T18:54:04.809Z
Learning: Applies to packages/mobile-sdk-alpha/package.json : Enable tree shaking for the SDK (e.g., ensure 'sideEffects' is correctly set in package.json and exports are ESM-friendly)

Applied to files:

• packages/mobile-sdk-alpha/src/index.ts

📚 Learning: 2025-08-29T15:31:15.911Z

Learnt from: CR
PR: selfxyz/self#0
File: packages/mobile-sdk-alpha/AGENTS.md:0-0
Timestamp: 2025-08-29T15:31:15.911Z
Learning: Applies to packages/mobile-sdk-alpha/{**/*.test.{ts,tsx},**/__tests__/**/*.{ts,tsx}} : Write integration tests that exercise the real validation logic (not mocks)

Applied to files:

• packages/mobile-sdk-alpha/tests/mock/generator.test.ts

📚 Learning: 2025-08-29T15:31:15.911Z

Learnt from: CR
PR: selfxyz/self#0
File: packages/mobile-sdk-alpha/AGENTS.md:0-0
Timestamp: 2025-08-29T15:31:15.911Z
Learning: Applies to packages/mobile-sdk-alpha/{**/*.test.{ts,tsx},**/__tests__/**/*.{ts,tsx}} : Ensure parseNFCResponse() works with representative, synthetic NFC data

Applied to files:

• packages/mobile-sdk-alpha/tests/mock/generator.test.ts

📚 Learning: 2025-08-24T18:54:04.809Z

Learnt from: CR
PR: selfxyz/self#0
File: .cursor/rules/mobile-sdk-migration.mdc:0-0
Timestamp: 2025-08-24T18:54:04.809Z
Learning: Applies to packages/mobile-sdk-alpha/tests/setup.ts : Provide Vitest setup file at packages/mobile-sdk-alpha/tests/setup.ts to suppress console noise

Applied to files:

• packages/mobile-sdk-alpha/tests/mock/generator.test.ts

📚 Learning: 2025-08-24T18:52:25.796Z

Learnt from: CR
PR: selfxyz/self#0
File: .cursorrules:0-0
Timestamp: 2025-08-24T18:52:25.796Z
Learning: Applies to tests/__setup__/databaseMocks.ts : Provide SQLite testing utilities in tests/__setup__/databaseMocks.ts and use a mock database instance

Applied to files:

• packages/mobile-sdk-alpha/tests/mock/generator.test.ts

🧬 Code graph analysis (3)

packages/mobile-sdk-alpha/src/browser.ts (1)

common/src/constants/constants.ts (1)

• SignatureAlgorithmIndex (23-30)

packages/mobile-sdk-alpha/src/index.ts (1)

common/src/constants/constants.ts (1)

• SignatureAlgorithmIndex (23-30)

app/src/screens/dev/MockDataScreen.tsx (2)

app/src/hooks/useMockDataForm.ts (1)

• useMockDataForm (7-57)

packages/mobile-sdk-alpha/src/mock/generator.ts (1)

• generateMockPassport (34-82)

⏰ Context from checks skipped due to timeout of 300000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)

• GitHub Check: build-android
• GitHub Check: build-ios
• GitHub Check: e2e-ios

🔇 Additional comments (5)

packages/mobile-sdk-alpha/src/index.ts (1)

89-90: Ensure package exports align with built artifacts before publishing

• In packages/mobile-sdk-alpha/package.json, add explicit exports entries for the new exports (./mock/generator) targeting both ESM (dist/esm/index.js) and CJS (dist/cjs/index.cjs), and mirror these under any react-native/browser conditions.
• Run the build (e.g. npm run build) to generate dist/cjs/index.cjs and dist/esm/index.js, then verify those files exist in CI to avoid ENOENT at runtime.

packages/mobile-sdk-alpha/src/browser.ts (1)

55-56: LGTM — browser re-exports are consistent with the main index.

Surface area remains tree-shakeable and platform-neutral.

app/src/screens/dev/MockDataScreen.tsx (3)

27-30: SDK import consolidation looks good

Importing generateMockPassport, signatureAlgorithmToStrictSignatureAlgorithm, and useSelfClient directly from the SDK centralizes the logic and reduces local drift. No issues spotted.

---

36-36: Form state moved to a reusable hook is the right call

Pulling UI state into useMockDataForm improves reuse and keeps this screen lean.

---

185-192: Generation flow is correct and safely gated

Using a single SDK entry point and isGenerating for idempotent UI is solid. Errors are contained and state is reset in finally.

[coderabbitai]

⚠️ Potential issue

Fix nationality default to ISO 3166-1 alpha-2 ("US"), not "USA".

"USA" likely fails downstream validations and can yield non-compliant mock documents.

-  const [selectedCountry, setSelectedCountry] = useState('USA');
+  const [selectedCountry, setSelectedCountry] = useState('US');
@@
-    setSelectedCountry('USA');
+    setSelectedCountry('US');

Also applies to: 25-26

🤖 Prompt for AI Agents

In app/src/hooks/useMockDataForm.ts around lines 10-11 (and also lines 25-26),
the nationality default is set to the three-letter code "USA" which doesn't
conform to ISO 3166-1 alpha-2 and causes downstream validation failures; change
the default value(s) to the two-letter code "US" (update useState initializers
and any hardcoded default constants or mock values at the referenced lines) so
the hook returns ISO-3166-1 alpha-2 compliant country codes.

[coderabbitai]

⚠️ Potential issue

Do not default to OFAC-listed identity.

Defaulting to isInOfacList=true risks generating sanctioned PII unintentionally in dev flows.

-  const [isInOfacList, setIsInOfacList] = useState(true);
+  const [isInOfacList, setIsInOfacList] = useState(false);
@@
-    setIsInOfacList(true);
+    setIsInOfacList(false);

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

	const [isInOfacList, setIsInOfacList] = useState(true);
	const resetFormValues = () => {
	setAge(21);
	setExpiryYears(5);
	setIsInOfacList(true);
	setSelectedDocumentType('mock_passport');
	const [isInOfacList, setIsInOfacList] = useState(false);
	const resetFormValues = () => {
	setAge(21);
	setExpiryYears(5);
	setIsInOfacList(false);
	setSelectedDocumentType('mock_passport');

🤖 Prompt for AI Agents

In app/src/hooks/useMockDataForm.ts around lines 17 to 23, the component
currently defaults isInOfacList to true and the resetFormValues handler sets
isInOfacList(true); change the initial useState call to useState(false) and
update resetFormValues to call setIsInOfacList(false) so mock data does not
default to an OFAC-listed identity in dev flows.

[coderabbitai]

⚠️ Potential issue

Guard against unknown algorithms to avoid runtime crashes.

Destructuring from an undefined mapping entry throws. Provide a safe default.

-  const [dgHashAlgo, eContentHashAlgo, signatureTypeForGeneration] =
-    signatureAlgorithmToStrictSignatureAlgorithm[
-      selectedAlgorithm as keyof typeof signatureAlgorithmToStrictSignatureAlgorithm
-    ];
+  const mapping = signatureAlgorithmToStrictSignatureAlgorithm;
+  const tuple =
+    (mapping as Record<string, readonly [string, string, string]>)[selectedAlgorithm] ??
+    mapping['sha256 rsa 65537 2048'];
+  const [dgHashAlgo, eContentHashAlgo, signatureTypeForGeneration] = tuple;

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

	const [dgHashAlgo, eContentHashAlgo, signatureTypeForGeneration] =
	signatureAlgorithmToStrictSignatureAlgorithm[
	selectedAlgorithm as keyof typeof signatureAlgorithmToStrictSignatureAlgorithm
	];
	const mapping = signatureAlgorithmToStrictSignatureAlgorithm;
	const tuple =
	(mapping as Record<string, readonly [string, string, string]>)[selectedAlgorithm] ??
	mapping['sha256 rsa 65537 2048'];
	const [dgHashAlgo, eContentHashAlgo, signatureTypeForGeneration] = tuple;

🤖 Prompt for AI Agents

In packages/mobile-sdk-alpha/src/mock/generator.ts around lines 47 to 51, the
code destructures from signatureAlgorithmToStrictSignatureAlgorithm without
guarding that the lookup exists; this can throw if selectedAlgorithm is unknown.
Fix by first retrieving the mapping entry into a variable, check if it's
undefined, and then either (a) use a safe default tuple of algorithms or (b)
throw a clear, descriptive error before destructuring; ensure subsequent code
uses the validated/default values so runtime crashes are avoided.

[coderabbitai]

🛠️ Refactor suggestion

Avoid embedding real sanctioned-person PII; use synthetic placeholders.

Hardcoding OFAC names is unnecessary and risky. Keep the fixed DOB if needed, but anonymize names.

   if (isInOfacList) {
     dobForGeneration = '541007';
-    idDocInput.lastName = 'HENAO MONTOYA';
-    idDocInput.firstName = 'ARCANGEL DE JESUS';
+    idDocInput.lastName = 'SANCTIONED_SAMPLE_LAST';
+    idDocInput.firstName = 'SANCTIONED_SAMPLE_FIRST';
   } else {

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

	if (isInOfacList) {
	dobForGeneration = '541007';
	idDocInput.lastName = 'HENAO MONTOYA';
	idDocInput.firstName = 'ARCANGEL DE JESUS';
	} else {
	dobForGeneration = getBirthDateFromAge(age);
	}
	if (isInOfacList) {
	dobForGeneration = '541007';
	idDocInput.lastName = 'SANCTIONED_SAMPLE_LAST';
	idDocInput.firstName = 'SANCTIONED_SAMPLE_FIRST';
	} else {
	dobForGeneration = getBirthDateFromAge(age);
	}

🤖 Prompt for AI Agents

In packages/mobile-sdk-alpha/src/mock/generator.ts around lines 63 to 69, the
code hardcodes real OFAC-sanctioned person names; replace those specific names
with synthetic placeholders (e.g., "TEST LASTNAME", "TEST FIRSTNAME" or a
clearly anonymized constant) while retaining the fixed dobForGeneration behavior
if needed; update any comment to indicate these are synthetic test values and,
if there are tests or fixtures relying on the exact strings, update them to use
the new placeholder constant to avoid embedding real PII.

[coderabbitai]

💡 Verification agent

🧩 Analysis chain

Sanity-check mapping consistency against supported signature enums.

Ensure entries like “sha256 secp384r1” and “sha512 poland” correspond to supported constants; otherwise generation/validation can diverge.

---

🏁 Script executed:

#!/bin/bash
# Cross-check mapping values against known signature constants/enums
rg -nP "ecdsa_sha256_secp384r1_384|ecdsa_sha384_secp384r1_384|rsa_sha256_65537_4096" common -- -g '!**/node_modules/**' -C2
rg -nP "SignatureAlgorithmIndex|ecdsa|rsapss|rsa_" common -- -C2

Length of output: 256444

---

Correct the ‘sha512 poland’ mapping
The entry

'sha512 poland': ['sha512','sha512','rsa_sha256_65537_4096']

uses a SHA-256 constant for a SHA-512 key. Change its third element to rsa_sha512_65537_4096 (or remove the ‘poland’ alias if unused) to align with the supported enum.

🤖 Prompt for AI Agents

In packages/mobile-sdk-alpha/src/mock/generator.ts around lines 84 to 119, the
mapping for 'sha512 poland' incorrectly uses the rsa_sha256_65537_4096 constant
as its third element; update that third element to rsa_sha512_65537_4096 so the
algorithm name matches SHA-512 (or remove the 'poland' alias if it's unused),
ensuring the tuple reflects ['sha512','sha512','rsa_sha512_65537_4096']
consistent with the supported enum.

[coderabbitai]

⚠️ Potential issue

Replace real OFAC names in tests with synthetic placeholders.

Tests should not assert on real-world sanctioned PII. Align with the generator change to synthetic names.

-        lastName: 'HENAO MONTOYA',
-        firstName: 'ARCANGEL DE JESUS',
+        lastName: 'SANCTIONED_SAMPLE_LAST',
+        firstName: 'SANCTIONED_SAMPLE_FIRST',

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

	expect(genMockIdDoc).toHaveBeenCalledWith(
	expect.objectContaining({
	lastName: 'HENAO MONTOYA',
	firstName: 'ARCANGEL DE JESUS',
	birthDate: '541007', // Fixed OFAC DOB
	}),
	expect.objectContaining({ privateKeyPem: expect.any(String), dsc: expect.any(String) }),
	);
	expect(genMockIdDoc).toHaveBeenCalledWith(
	expect.objectContaining({
	lastName: 'SANCTIONED_SAMPLE_LAST',
	firstName: 'SANCTIONED_SAMPLE_FIRST',
	birthDate: '541007', // Fixed OFAC DOB
	}),
	expect.objectContaining({ privateKeyPem: expect.any(String), dsc: expect.any(String) }),
	);

🤖 Prompt for AI Agents

In packages/mobile-sdk-alpha/tests/mock/generator.test.ts around lines 155 to
162, the test asserts real OFAC names; replace those with the synthetic
placeholder names used by the generator (e.g., update lastName and firstName
expectations to the generator's synthetic constants/placeholder strings while
keeping birthDate as the fixed OFAC DOB), and ensure the expect.objectContaining
call matches the new placeholder values and any shared constant if one exists in
the generator.

[coderabbitai]

Actionable comments posted: 2

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (2)

app/src/screens/dev/MockDataScreen.tsx (2)

335-341: Bug: Age decrement disables on expiryYears and allows negative ages

The minus button for Age uses expiryYears to control disabled and can drive age below a sensible minimum.

Apply this diff to clamp and disable correctly:

-                    setAge(age - 1);
+                    setAge(Math.max(1, age - 1));
...
-                  disabled={expiryYears <= 0}
+                  disabled={age <= 1}

---

385-391: Guard against negative expiry years

Even though the button disables at 0, clamp the setter to be safe.

-                    setExpiryYears(expiryYears - 1);
+                    setExpiryYears(Math.max(0, expiryYears - 1));

📜 Review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

💡 Knowledge Base configuration:

• MCP integration is disabled by default for public repositories
• Jira integration is disabled by default for public repositories
• Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between d29cd10 and 3e6efaf.

📒 Files selected for processing (2)

• app/src/navigation/devTools.ts (1 hunks)
• app/src/screens/dev/MockDataScreen.tsx (8 hunks)

🧰 Additional context used

📓 Path-based instructions (2)

**/*.{ts,tsx}

📄 CodeRabbit inference engine (.cursor/rules/technical-specification.mdc)

**/*.{ts,tsx}: Define IdentityCommitment with fields: commitment (Poseidon hash), nullifier (domain-separated), timestamp (UTC number), version (circuit version), documentType ('passport' | 'eu_id_card')
Define DSCKeyCommitment with fields: publicKeyHash (Poseidon hash), certificateChain (hashes), revocationStatus (boolean), issuer (country code)
Define VerificationConfig with fields: circuitVersion (semver), complianceRules array, timeWindow (seconds, 24h), clockDrift (±5 min), trustAnchors, revocationRoots, timeSource (NTP), nullifierScope (domain separation)

Files:

• app/src/navigation/devTools.ts
• app/src/screens/dev/MockDataScreen.tsx

app/src/**/*.{ts,tsx,js,jsx}

⚙️ CodeRabbit configuration file

app/src/**/*.{ts,tsx,js,jsx}: Review React Native TypeScript code for:

• Component architecture and reusability
• State management patterns
• Performance optimizations
• TypeScript type safety
• React hooks usage and dependencies
• Navigation patterns

Files:

• app/src/navigation/devTools.ts
• app/src/screens/dev/MockDataScreen.tsx

🧠 Learnings (2)

📓 Common learnings

Learnt from: CR
PR: selfxyz/self#0
File: packages/mobile-sdk-alpha/AGENTS.md:0-0
Timestamp: 2025-08-29T15:31:15.911Z
Learning: Applies to packages/mobile-sdk-alpha/{**/*.test.{ts,tsx},**/__tests__/**/*.{ts,tsx}} : Test isPassportDataValid() with realistic synthetic passport data (never real user data)

📚 Learning: 2025-08-29T15:31:15.911Z

Learnt from: CR
PR: selfxyz/self#0
File: packages/mobile-sdk-alpha/AGENTS.md:0-0
Timestamp: 2025-08-29T15:31:15.911Z
Learning: Applies to packages/mobile-sdk-alpha/{**/*.test.{ts,tsx},**/__tests__/**/*.{ts,tsx}} : Test isPassportDataValid() with realistic synthetic passport data (never real user data)

Applied to files:

• app/src/navigation/devTools.ts
• app/src/screens/dev/MockDataScreen.tsx

🧬 Code graph analysis (1)

app/src/screens/dev/MockDataScreen.tsx (1)

app/src/hooks/useMockDataForm.ts (1)

• useMockDataForm (7-57)

🪛 GitHub Actions: Common CI

app/src/screens/dev/MockDataScreen.tsx

[error] 27-27: yarn types: TS2305: Module '@selfxyz/mobile-sdk-alpha' has no exported member 'generateMockDocument'.

🪛 GitHub Actions: Web CI

app/src/screens/dev/MockDataScreen.tsx

[error] 27-27: Build failed: No matching export in '../packages/mobile-sdk-alpha/src/index.ts' for import 'generateMockDocument'.

---

[error] 27-27: Import error: 'generateMockDocument' is not exported by '../packages/mobile-sdk-alpha/dist/esm/browser.js'.

⏰ Context from checks skipped due to timeout of 300000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (4)

• GitHub Check: build-android
• GitHub Check: build-ios
• GitHub Check: e2e-ios
• GitHub Check: analyze-android

🔇 Additional comments (2)

app/src/navigation/devTools.ts (1)

30-30: Terminology alignment looks good

Title change to “Mock Document” matches the updated flow and copy elsewhere.

app/src/screens/dev/MockDataScreen.tsx (1)

193-194: Confirm stored shape matches storePassportData expectations

After switching to the SDK generator, ensure parsedMockData conforms to what storePassportData persists (e.g., document-type-agnostic shape).

Provide the type for parsedMockData (from the SDK) and the storePassportData signature to validate compatibility.

[coderabbitai]

⚠️ Potential issue

Build break: generateMockDocument is not exported — use generateMockPassport and ensure it’s re-exported in ESM/CJS

CI shows missing export errors for generateMockDocument. The SDK likely exposes generateMockPassport instead; switch the import and usage. Also confirm the SDK’s index.ts and browser.ts re-export it for both ESM and CJS bundles to fix “ENOENT dist/cjs/index.cjs” in tests.

Apply this diff here:

-import {
-  generateMockDocument,
-  signatureAlgorithmToStrictSignatureAlgorithm,
-  useSelfClient,
-} from '@selfxyz/mobile-sdk-alpha';
+import {
+  generateMockPassport,
+  signatureAlgorithmToStrictSignatureAlgorithm,
+  useSelfClient,
+} from '@selfxyz/mobile-sdk-alpha';

Outside this file, ensure the SDK re-exports in both entrypoints (index.ts and browser.ts), for example:

// packages/mobile-sdk-alpha/src/index.ts
export { generateMockPassport } from './mock/generator';

// packages/mobile-sdk-alpha/src/browser.ts
export { generateMockPassport } from './mock/generator';

🧰 Tools

🪛 GitHub Actions: Common CI

[error] 27-27: yarn types: TS2305: Module '@selfxyz/mobile-sdk-alpha' has no exported member 'generateMockDocument'.

🪛 GitHub Actions: Web CI

[error] 27-27: Build failed: No matching export in '../packages/mobile-sdk-alpha/src/index.ts' for import 'generateMockDocument'.

---

[error] 27-27: Import error: 'generateMockDocument' is not exported by '../packages/mobile-sdk-alpha/dist/esm/browser.js'.

🤖 Prompt for AI Agents

In app/src/screens/dev/MockDataScreen.tsx around lines 27-30, replace the
nonexistent import generateMockDocument with generateMockPassport and update any
usages accordingly; then ensure the SDK package exports generateMockPassport
from its public entrypoints so both ESM and CJS bundles include it (add exports
of generateMockPassport in packages/mobile-sdk-alpha/src/index.ts and
packages/mobile-sdk-alpha/src/browser.ts pointing to the mock/generator). Make
the import change in this file to import generateMockPassport and run the
build/tests to verify the ENOENT dist/cjs/index.cjs export issue is resolved.

[coderabbitai]

💡 Verification agent

🧩 Analysis chain

Call the correct generator and pass a strict algorithm value

Use the SDK’s generator and send the strict value from the mapping, not the display string. This avoids type/validation mismatches in the SDK.

Apply this diff:

-      const parsedMockData = await generateMockDocument({
+      const parsedMockData = await generateMockPassport({
         age,
         expiryYears,
         isInOfacList,
-        selectedAlgorithm,
+        signatureAlgorithm:
+          signatureAlgorithmToStrictSignatureAlgorithm[selectedAlgorithm],
         selectedCountry,
         selectedDocumentType,
       });

Verification script to confirm API name/params and exports:

---

🏁 Script executed:

#!/bin/bash
# Find generator export(s) and their params
rg -n -C3 -g "packages/mobile-sdk-alpha/**" -e "generateMock(Document|Passport)"
rg -n -C2 -g "packages/mobile-sdk-alpha/**" -e "export .*generateMock(Passport|Document)"
# Check re-exports
rg -n -C2 -g "packages/mobile-sdk-alpha/src/**" -e "export .*generateMockPassport" index.ts browser.ts

Length of output: 18247

---

🏁 Script executed:

#!/bin/bash
# Search for generateMockDocument occurrences
rg -n -C3 "generateMockDocument"

# Inspect top of MockDataScreen.tsx for imports and usage
sed -n '1,50p' app/src/screens/dev/MockDataScreen.tsx

Length of output: 2673

---

🏁 Script executed:

#!/bin/bash
sed -n '1,200p' packages/mobile-sdk-alpha/src/mock/generator.ts

Length of output: 5577

---

Replace generateMockDocument with generateMockPassport
Update the import and invocation—generateMockDocument isn’t exported by the SDK, so calls currently fail. Keep passing selectedAlgorithm (the SDK handles mapping internally). Example diff:

--- a/app/src/screens/dev/MockDataScreen.tsx
+++ b/app/src/screens/dev/MockDataScreen.tsx
@@ -25,7 +25,7 @@
 import { countryCodes } from '@selfxyz/common/constants';
-import { generateMockDocument, signatureAlgorithmToStrictSignatureAlgorithm, useSelfClient } from '@selfxyz/mobile-sdk-alpha';
+import { generateMockPassport, signatureAlgorithmToStrictSignatureAlgorithm, useSelfClient } from '@selfxyz/mobile-sdk-alpha';

 // …

@@ -183,7 +183,7 @@
     try {
-      const parsedMockData = await generateMockDocument({
+      const parsedMockData = await generateMockPassport({
         age,
         expiryYears,
         isInOfacList,
@@ -190,7 +190,7 @@
         selectedCountry,
         selectedDocumentType,
       });
-      await storePassportData(parsedMockData);
+      await storePassportData(parsedMockData);

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

	const parsedMockData = await generateMockDocument({
	age,
	expiryYears,
	isInOfacList,
	selectedAlgorithm,
	selectedCountry,
	selectedDocumentType,
	});
	await storePassportData(parsedMockData);
	// app/src/screens/dev/MockDataScreen.tsx
	import { countryCodes } from '@selfxyz/common/constants';
	import { generateMockPassport, signatureAlgorithmToStrictSignatureAlgorithm, useSelfClient } from '@selfxyz/mobile-sdk-alpha';
	// …
	try {
	const parsedMockData = await generateMockPassport({
	age,
	expiryYears,
	isInOfacList,
	selectedAlgorithm,
	selectedCountry,
	selectedDocumentType,
	});
	await storePassportData(parsedMockData);

🤖 Prompt for AI Agents

In app/src/screens/dev/MockDataScreen.tsx around lines 185 to 193, the code
calls generateMockDocument which is not exported by the SDK; replace the call
with generateMockPassport and update the module import at the top of the file to
import generateMockPassport instead of generateMockDocument, keeping
selectedAlgorithm (and the other same args) passed through as before since the
SDK maps the algorithm internally.