chore: fix dev workspace type check

[shazarre]

Summary by CodeRabbit

• Tests
  • Updated test suites to use public package exports for type imports, improving consistency and maintainability.
  • No changes to app behavior, performance, or UI.

[coderabbitai]

Walkthrough

Type-only imports in three React Native test files were updated to reference DocumentCatalog from the @selfxyz/common barrel instead of a deep internal path. No runtime or logic changes were made.

Changes

Cohort / File(s)	Summary of changes
Type-only import path normalization (tests) packages/mobile-sdk-demo/tests/screens/DocumentsList.test.tsx, packages/mobile-sdk-demo/tests/screens/.../GenerateMock.test.tsx, packages/mobile-sdk-demo/tests/screens/.../RegisterDocument.test.tsx	Switched DocumentCatalog import from @selfxyz/common/dist/esm/src/utils/types.js to @selfxyz/common (barrel). No logic/runtime changes.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

• test: add mobile sdk demo vitest tests #1203 — Adjusts the same test files’ type-only imports; directly related to these path updates.
• SELF-808: Implement document camera MRZ demo screen #1204 — Similar migration from deep/dist imports to public barrel exports in mobile-sdk-demo.
• remove lazy loading #1018 — Changes DocumentCatalog imports in app code to public utils/barrel; closely aligned with this refactor.

Suggested labels

codex

Suggested reviewers

• remicolin

Poem

Three tests took a tidy stroll,
From deep paths to a single role.
Types aligned, the imports clean,
No shifting logic in between.
Barrel bound, they now convene—
🧪✨ a quieter diff, crisp and serene.

Pre-merge checks and finishing touches

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Title Check	❓ Inconclusive	The title uses generic phrasing (“fix dev workspace type check”) without specifying that the PR’s main purpose is to update the DocumentCatalog type imports to the public barrel export, making it too vague to convey the actual changes. It doesn’t reference the test files or modules altered and thus fails to clearly summarize the update for those scanning the history.	Rename the title to something like “chore: update DocumentCatalog type imports to use public barrel export” to clearly reflect the import path adjustments made across the test files.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage	✅ Passed	No functions found in the changes. Docstring coverage check skipped.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch shazarre/chore_fix_dev

---

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

_{Comment @coderabbitai help to get the list of available commands and usage tips.}