chore: remove tracking and export config variable to env file (#4)

Co-authored-by: Gabriel Saillard <gabriel@saillard.dev>

.env.example

@@ -0,0 +1,2 @@
+MAPBOX_ACCESS_TOKEN="xyz"
+MAP_SERVICES_API_URL="https://valhalla1.openstreetmap.de"

README.md

@@ -25,12 +25,7 @@ If you're curious, feel free to dig around!
 
 This is a basic managed expo app, and there are a few scripts in `package.json` that should help you find your way.
 
-Secrets and API keys have been cut out of this release, and you might need to get new ones and put them in `app.config.ts` to succesfully build the app.
-All removed secrets have been replaced with the `[CENSORED]` tag to easily identify where they're missing.
-
-## Monitoring
-
-The implementation contains leftovers of our monitoring and tracking solutions (UXCam, Sentry). The API keys related to those have been deleted from the source code, and the related accounts on the monitoring platforms have been closed.
+You need to provide your own secrets to access the map through [Mapbox](https://docs.mapbox.com/help/getting-started/access-tokens/) and provide support for other map services like snap-to-road path tracing. Copy the `.env.example` file to `.env` and set them up there.
 
 ## Licensing
 The untitled map app source code is shared under a Creative Commons [BY-NC 4.0](https://creativecommons.org/licenses/by-nc/4.0/) license.

app.config.ts

@@ -19,22 +19,6 @@ export default ({ config }: ConfigContext): ExpoConfig => ({
 		resizeMode: 'cover',
 		backgroundColor: '#FFFFFF',
 	},
-	updates: {
-		fallbackToCacheTimeout: 0,
-		url: '[CENSORED]',
-	},
-	hooks: {
-		postPublish: [
-			{
-				file: 'sentry-expo/upload-sourcemaps',
-				config: {
-					organization: 'reality-co-open-source',
-					project: 'uma',
-					setCommits: true,
-				},
-			},
-		],
-	},
 	plugins: [
 		[
 			'expo-build-properties',
@@ -44,13 +28,6 @@ export default ({ config }: ConfigContext): ExpoConfig => ({
 				},
 			},
 		],
-		'sentry-expo',
-		[
-			'expo-updates',
-			{
-				username: 'reality-co',
-			},
-		],
 		[
 			'expo-location',
 			{
@@ -68,7 +45,7 @@ export default ({ config }: ConfigContext): ExpoConfig => ({
 			'@rnmapbox/maps',
 			{
 				RNMapboxMapsImpl: 'mapbox',
-				RNMapboxMapsDownloadToken: '[CENSORED]',
+				RNMapboxMapsDownloadToken: process.env.MAPBOX_ACCESS_TOKEN,
 			},
 		],
 	],
@@ -101,12 +78,7 @@ export default ({ config }: ConfigContext): ExpoConfig => ({
 		],
 	},
 	extra: {
-		eas: {
-			projectId: '[CENSORED]',
-		},
-		mapServicesApiUrl: '[CENSORED]',
-		mapboxAccessToken: '[CENSORED]',
-		uxCamAppKey: '[CENSORED]',
-		sentryDsn: '[CENSORED]',
+		mapServicesApiUrl: process.env.MAP_SERVICES_API_URL,
+		mapboxAccessToken: process.env.MAPBOX_ACCESS_TOKEN,
 	},
 })

app/_layout.tsx

@@ -3,19 +3,16 @@ import { InterTight_700Bold } from '@expo-google-fonts/inter-tight'
 import { Lexend_400Regular, Lexend_500Medium } from '@expo-google-fonts/lexend'
 import { safeBottomSheetViewScreenOptions } from '@lib/components/generic/SafeBottomSheetView'
 import BackgroundLocationServiceController from '@lib/services/BackgroundLocationServiceController'
-import { setupRoutingInstrumentation } from '@lib/services/monitoring'
 import TracingCrawlerServiceProvider from '@lib/services/tracing-crawler/TracingCrawlerServiceProvider'
 import WebWorkerThreadProvider from '@lib/services/web-worker/WebWorkerThreadProvider'
 import DatabaseProvider, {
 	initializeDatabase,
 } from '@lib/utils/DatabaseProvider'
 import SettingsProvider from '@lib/utils/SettingsProvider'
 import * as Font from 'expo-font'
-import { SplashScreen, Stack, useRootNavigation } from 'expo-router'
+import { SplashScreen, Stack } from 'expo-router'
 import { StatusBar } from 'expo-status-bar'
-import * as Updates from 'expo-updates'
 import { useEffect, useState } from 'react'
-import RNUxcam from 'react-native-ux-cam'
 
 SplashScreen.preventAutoHideAsync()
 
@@ -61,37 +58,6 @@ export default function RootLayout() {
 		}
 	}, [appIsReady])
 
-	Updates.useUpdateEvents((e) => {
-		console.debug('eas update event', e)
-	})
-	useEffect(() => {
-		console.log('Expo updates status', {
-			channel: Updates.channel,
-			runtime: Updates.runtimeVersion,
-		})
-	}, [])
-
-	const navigation = useRootNavigation()
-
-	useEffect(() => {
-		if (navigation) {
-			setupRoutingInstrumentation(navigation)
-		}
-
-		const onStateChange = () => {
-			const screenName = navigation?.getCurrentRoute()?.name
-			if (!screenName) return
-
-			RNUxcam.tagScreenName(screenName)
-		}
-
-		navigation?.addListener('state', onStateChange)
-
-		return () => {
-			navigation?.removeListener('state', onStateChange)
-		}
-	}, [navigation])
-
 	if (!appIsReady) {
 		return null
 	}

lib/components/DebugInfo.tsx

@@ -4,7 +4,6 @@ import Text from '@lib/components/generic/Text'
 import LocationPoint from '@lib/entities/LocationPoint'
 import dataSource from '@lib/utils/data-source'
 import Constants from 'expo-constants'
-import * as Updates from 'expo-updates'
 import { useEffect, useState } from 'react'
 import { StyleSheet } from 'react-native'
 
@@ -39,9 +38,7 @@ export default function DebugInfo() {
 					'unknown'
 				}`}
 				{'\n'}
-				{`build ${Constants.expoConfig?.version ?? 'unknown'}-${(
-					Updates.createdAt ?? new Date()
-				).toISOString()}`}
+				{`build ${Constants.expoConfig?.version ?? 'unknown'}`}
 				{'\n'}
 				database: {dataSource.options.database}
 				{' / '}

lib/services/index.ts

@@ -5,11 +5,8 @@ import {
 	backgroundLocationService,
 	backgroundLocationTaskName,
 } from './background-location'
-import { setupMonitoring } from './monitoring'
 
 export function registerServices() {
-	setupMonitoring()
-
 	TaskManager.defineTask(backgroundLocationTaskName, backgroundLocationService)
 	setupAppWideEvents()
 }

lib/services/tracing-crawler/index.ts

@@ -1,6 +1,5 @@
 import ComputedFogShape from '@lib/entities/ComputedFogShape'
 import ComputedWalkTrace from '@lib/entities/ComputedWalkTrace'
-import * as Monitoring from '@lib/services/monitoring'
 import ThreadBridgeEventBus from '@lib/services/web-worker/thread-bridge'
 import dataSource from '@lib/utils/data-source'
 import * as turf from '@turf/turf'
@@ -36,31 +35,17 @@ export default async function crawlTrace(
 }> {
 	console.group('crawlTrace: start')
 	console.time('crawlTrace')
-	const crawlRunTransaction = Monitoring.startTransaction({
-		op: 'tracingCrawler.crawlLoop',
-		name: 'Tracing crawler crawl loop',
-	})
 
 	try {
 		// TARGET
 		console.time('crawlTrace: query')
-		const querySpan = crawlRunTransaction.startChild({
-			op: 'tracingCrawler.query',
-		})
-
 		const nextTraceState = await getNextTracingCrawlerState()
 
 		if (!nextTraceState) {
 			console.info('crawlTrace: no more points to trace')
-
 			console.timeEnd('crawlTrace: query')
 			console.timeEnd('crawlTrace')
 			console.groupEnd()
-			querySpan.setStatus(Monitoring.Status.Ok)
-			querySpan.finish()
-			crawlRunTransaction.setStatus(Monitoring.Status.Ok)
-			crawlRunTransaction.finish()
-
 			return { moreWorkAhead: false }
 		}
 
@@ -82,15 +67,9 @@ export default async function crawlTrace(
 		)
 
 		console.timeEnd('crawlTrace: query')
-		querySpan.setStatus(Monitoring.Status.Ok)
-		querySpan.finish()
-
 		// TRACE
 
 		console.time('crawlTrace: trace')
-		const traceSpan = crawlRunTransaction.startChild({
-			op: 'tracingCrawler.trace',
-		})
 
 		let fogShape: FogShapeGeometries
 		let walkTrace: WalkTraceGeometries
@@ -130,9 +109,6 @@ export default async function crawlTrace(
 			// EXPAND OR CREATE
 
 			console.time('crawlTrace: expand geometries')
-			const expandSpan = crawlRunTransaction.startChild({
-				op: 'tracingCrawler.expand',
-			})
 
 			fogShape = await workerThread.waitFor(
 				workerThread.submitJobRequest(ThreadedFnName.ExpandOrCreateFog, [
@@ -147,8 +123,6 @@ export default async function crawlTrace(
 			)
 
 			console.timeEnd('crawlTrace: expand geometries')
-			expandSpan.setStatus(Monitoring.Status.Ok)
-			expandSpan.finish()
 		}
 
 		const justDrawnPoint = latestUntracedPoint
@@ -189,16 +163,11 @@ export default async function crawlTrace(
 		}
 
 		console.timeEnd('crawlTrace: trace')
-		traceSpan.setStatus(Monitoring.Status.Ok)
-		traceSpan.finish()
 
 		// MERGE
 		// TODO: move from using tracing time to using geographical proximity to find geometries to merge
 
 		console.time('crawlTrace: merge geometries')
-		const mergeSpan = crawlRunTransaction.startChild({
-			op: 'tracingCrawler.merge',
-		})
 
 		let hasMergedFogShapes = false
 		let hasMergedWalkTraces = false
@@ -278,15 +247,10 @@ export default async function crawlTrace(
 		}
 
 		console.timeEnd('crawlTrace: merge geometries')
-		mergeSpan.setStatus(Monitoring.Status.Ok)
-		mergeSpan.finish()
 
 		// COMMIT
 
 		console.time('crawlTrace: write new state')
-		const commitSpan = crawlRunTransaction.startChild({
-			op: 'tracingCrawler.commit',
-		})
 
 		await dataSource
 			.transaction(async (entityManager) => {
@@ -303,26 +267,19 @@ export default async function crawlTrace(
 			})
 			.catch((error) => {
 				console.error('crawlTrace: error saving new state', error)
-				Monitoring.captureException(error)
 			})
 
 		console.timeEnd('crawlTrace: write new state')
-		commitSpan.setStatus(Monitoring.Status.Ok)
-		commitSpan.finish()
 
 		console.timeEnd('crawlTrace')
 		console.groupEnd()
-		crawlRunTransaction.setStatus(Monitoring.Status.Ok)
-		crawlRunTransaction.finish()
 
 		return {
 			moreWorkAhead: true,
 			cursorTime: justDrawnPoint.timestamp,
 		}
 	} catch (error) {
 		console.error('crawlTrace: error', error)
-		crawlRunTransaction.setStatus(Monitoring.Status.InternalError)
-		Monitoring.captureException(error)
 	}
 
 	return {