Skip to content

Implement Phase 13B: Deep Linking (FR-027)#12

Merged
jeremyeder merged 2 commits intomainfrom
feature/phase-13b-deep-linking
Nov 27, 2025
Merged

Implement Phase 13B: Deep Linking (FR-027)#12
jeremyeder merged 2 commits intomainfrom
feature/phase-13b-deep-linking

Conversation

@jeremyeder
Copy link
Copy Markdown
Contributor

Summary

This PR implements Phase 13B: Deep Linking, adding universal deep linking support to enable users to open specific screens via custom URLs.

Changes

Deep Linking Infrastructure

  • URL scheme configuration in app.json ()
  • URL parsing and validation ()
  • Navigation handlers for different URL types ()
  • Analytics integration to track deep link usage ()
  • React hook for deep link events ()

Supported Deep Link Patterns

  • acpmobile://session/:id - Open specific session
  • acpmobile://notifications - Open notifications screen
  • acpmobile://repo/:id - Open repository details
  • Future-ready for chat, settings, and other screens

Documentation

  • Android App Links setup guide
  • Deep linking architecture documentation
  • URL pattern reference

Testing

  • Comprehensive test coverage for URL parsing
  • Handler tests for navigation logic
  • Analytics tracking verification

Functional Requirements

  • FR-027: Universal deep linking support

Integration

  • Integrated with app/_layout.tsx for app-wide deep link handling
  • Works with existing navigation (Expo Router)
  • Compatible with PostHog analytics (Phase 13D)

Related

  • Part of Phases 11-13 implementation sequence
  • Enables future push notification deep links
  • Foundation for sharing features

🤖 Generated with Claude Code

jeremyeder and others added 2 commits November 27, 2025 04:03
- Add universal deep linking support (acpmobile:// URLs)
- Implement URL parsing and validation
- Add navigation handlers for sessions, notifications, repos
- Integrate analytics tracking for deep link events
- Update app.json with URL scheme configuration
- Add comprehensive documentation

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
- Keep both useLinking and initializeTelemetry imports
- Deep link analytics already integrated in performance monitoring
@jeremyeder jeremyeder merged commit 5ce31d5 into main Nov 27, 2025
@jeremyeder jeremyeder deleted the feature/phase-13b-deep-linking branch November 27, 2025 09:04
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.

1 participant