app/containers/markdown/components/Paragraph.tsx (2)

36-42: Clarify the use case and add stories for this implementation.

This early return handles timestamps at the paragraph start, but the use case and test coverage remain unclear.

As noted in the previous review: What's the specific use case that requires this early return? Please add Storybook stories to demonstrate the TIMESTAMP rendering behavior.

---

45-45: Optimize the type check and clarify the margin logic.

The inline map operation is inefficient and the marginTop logic needs clarification:

1. value?.map(x=> x.type).includes('TIMESTAMP') creates an unnecessary intermediate array. Use some instead: value?.some(x => x.type === 'TIMESTAMP')
2. Missing space in arrow function: x=> should be x =>
3. The logic is confusing: lines 36-42 early return when value[0] is TIMESTAMP, so this marginTop only applies when TIMESTAMP appears at a non-zero index. Please clarify why mixed content with TIMESTAMP requires different spacing.

Apply this diff to optimize the type check:

-		<Text style={[styles.text, { color: themes[theme].fontDefault, marginTop: value?.map(x=> x.type).includes('TIMESTAMP') ? 2 : 0 }]}>
+		<Text style={[styles.text, { color: themes[theme].fontDefault, marginTop: value?.some(x => x.type === 'TIMESTAMP') ? 2 : 0 }]}>

Note: The previous reviewer asked "Why do you have to add this margin top?" - please provide justification for this spacing adjustment.

app/containers/markdown/components/Timestamp.tsx (2)

56-60: Add font styling to match the web version.

The component is missing font styling (family, weight) that should align with the web implementation for consistency across platforms.

As noted in the previous review: Please verify the web version's font styling and apply the same here. Consider extracting these styles to a StyleSheet for better performance instead of using inline styles.

#!/bin/bash
# Description: Find timestamp styling in web codebase to ensure mobile matches

# Search for timestamp-related styles in the web codebase
rg -n -C3 "timestamp.*style|Timestamp.*css" --type-add 'web:*.{tsx,ts,css,scss}' -t web

---

51-54: Extract the duplicate timestamp calculation.

The timestamp calculation parseInt(value.timestamp) * 1000 is duplicated on lines 18 and 52, violating DRY principles.

As noted in the previous review, extract this to a memoized constant:

     const formatDate = React.useMemo(()=>{
         const timestamp = parseInt(value.timestamp) * 1000;
         // ... formatting logic
     }, [value]);
+
+    const timestampMs = React.useMemo(() => parseInt(value.timestamp) * 1000, [value.timestamp]);

     const handlePress = () => {
-        const message = moment(parseInt(value.timestamp) * 1000).format('dddd, MMM DD, YYYY hh:mm A');
+        const message = moment(timestampMs).format('LLL');
         EventEmitter.emit(LISTENER, { message });
     }

Note: Also updated the format string to 'LLL' to use locale-aware formatting (see previous comment).

app/containers/markdown/components/Timestamp.tsx (1)

20-46: Consider using a switch statement for better readability.

The sequence of seven if statements would be more readable and maintainable as a switch statement.

Apply this diff:

     const formatDate = React.useMemo(()=>{
         const timestamp = parseInt(value.timestamp) * 1000;
 
-        if(value.format === "t"){
-            return moment(timestamp).format('LT');
-        }
-
-        if(value.format === "T"){
-            return moment(timestamp).format('LTS');
-        }
-
-        if(value.format === "d"){
-            return moment(timestamp).format('L');
-        }
-
-        if(value.format === "D"){
-            return moment(timestamp).format('LL');
-        }
-
-        if(value.format === "f"){
-            return moment(timestamp).format('LLL');
-        }
-
-        if(value.format === "F"){
-            return moment(timestamp).format('LLLL');
-        }
-
-        if(value.format === "R"){
-            return moment(timestamp).fromNow();
-        }
-
-        return "Invalid Date";
+        switch(value.format) {
+            case "t":
+                return moment(timestamp).format('LT');
+            case "T":
+                return moment(timestamp).format('LTS');
+            case "d":
+                return moment(timestamp).format('L');
+            case "D":
+                return moment(timestamp).format('LL');
+            case "f":
+                return moment(timestamp).format('LLL');
+            case "F":
+                return moment(timestamp).format('LLLL');
+            case "R":
+                return moment(timestamp).fromNow();
+            default:
+                return "Invalid Date";
+        }
     }, [value]);

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

Disabled knowledge base sources:

• Jira integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.