[senamakel]

Summary

• Enable the Memory tab (Intelligence page) in the bottom tab bar; it was previously hidden behind a Hide Memory tab from bottom navigation bar #976 gate.
• The underlying /intelligence route and IntelligenceMemoryTab component already exist — this just unhides the entry point.

Problem

The Memory tab has been live in code (/intelligence route, IntelligenceMemoryTab component) but commented out in BottomTabBar.tsx pending the Intelligence feature being ready. It's ready enough to surface to users.

Solution

Uncomment the Memory tab entry in app/src/components/BottomTabBar.tsx.

Submission Checklist

• N/A: nav-only re-enablement of an existing route; covered by existing tab-bar rendering tests
• N/A: no new logic — diff is a comment-out removal, no measurable lines changed
• N/A: no feature behaviour added or removed; existing Intelligence/Memory rows already cover this surface
• N/A: not tied to a matrix feature ID
• N/A: no network dependencies touched
• N/A: navigation surface only; release smoke already covers tab navigation
• N/A: no linked issue

Impact

• Desktop UI: Memory tab becomes visible in the bottom tab bar.
• No runtime, performance, security, or migration impact.

Related

• Closes:
• Follow-up PR(s)/TODOs: surface memory ingestion running state in the Memory tab + add singleton guard around ingestion in core.

Summary by CodeRabbit

• New Features
  • The Memory tab is now active and accessible in the bottom navigation menu, allowing quick access to memory and intelligence capabilities from the main app interface.

[coderabbitai]

📝 Walkthrough

Walkthrough

The Bottom Tab Bar's navigation configuration is updated to include an active Memory/Intelligence tab entry with id: 'intelligence', label: 'Memory', and path: '/intelligence'. A previously commented-out intelligence tab block is removed. No other navigation logic, locking behavior, or badge functionality changed.

Changes

Navigation Tab Configuration

Layer / File(s)	Summary
Tab Configuration app/src/components/BottomTabBar.tsx	The intelligence tab entry is uncommented and activated in the tabs array, defining its id, label, and navigation path.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

• #361: Modifies the same BottomTabBar.tsx tabs configuration to add/enable a tab entry.
• #992: Directly related—toggles the Memory/Intelligence tab visibility (one activates, the other disables).
• #868: Modifies BottomTabBar.tsx and changes the Intelligence/Memory tab configuration.

Poem

🐰 A memory awakens, no longer in shade,
The intelligence tab has finally been made,
From comments of old to navigation so bright,
A path to /intelligence shines in the light! ✨

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title 'feat(nav): enable Memory tab in BottomTabBar' directly and clearly describes the primary change: uncommenting/enabling the Memory tab navigation entry in the BottomTabBar component.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

---

_{Review rate limit: 4/5 reviews remaining, refill in 12 minutes.}

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

[coderabbitai]

🧹 Nitpick comments (1)

app/src/components/BottomTabBar.tsx (1)

56-56: ⚡ Quick win

Remove or update the stale “hidden” comment.

Line 56 says the Memory tab is hidden, but it is now active directly below. Keeping this comment will mislead future maintenance.

Suggested cleanup

-  // Memory tab hidden until Intelligence feature is ready (`#976`)
+  // Memory tab

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@app/src/components/BottomTabBar.tsx` at line 56, The inline comment "//
Memory tab hidden until Intelligence feature is ready (`#976`)" is stale and
misleading in BottomTabBar.tsx because the Memory tab is now rendered active;
remove or update that comment near the Memory tab JSX (the Tab.Screen /
Memory-related element) so it accurately reflects current behavior (e.g., delete
the comment or replace it with a note about why Memory is shown or any relevant
issue reference).

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In `@app/src/components/BottomTabBar.tsx`:
- Line 56: The inline comment "// Memory tab hidden until Intelligence feature
is ready (`#976`)" is stale and misleading in BottomTabBar.tsx because the Memory
tab is now rendered active; remove or update that comment near the Memory tab
JSX (the Tab.Screen / Memory-related element) so it accurately reflects current
behavior (e.g., delete the comment or replace it with a note about why Memory is
shown or any relevant issue reference).

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 00d89b33-7383-4f72-aae7-06116f9f84cf

📥 Commits

Reviewing files that changed from the base of the PR and between dc69803 and 624b84a.

📒 Files selected for processing (1)

• app/src/components/BottomTabBar.tsx