Skip to content

feat(dashboard): add setting to disable Personality feature - #25

Merged
ClaudiaFang merged 1 commit into
masterfrom
worktree-graceful-wiggling-nygaard
Jul 23, 2026
Merged

feat(dashboard): add setting to disable Personality feature#25
ClaudiaFang merged 1 commit into
masterfrom
worktree-graceful-wiggling-nygaard

Conversation

@ClaudiaFang

Copy link
Copy Markdown
Member

Summary

  • Adds a localStorage-backed feature flag (useFeatureFlags) with personalityEnabled (default true)
  • Settings page: new "Features" card with a switch to turn Personality on/off
  • Header: hides the Personality nav link (desktop nav, mobile drawer, "More" bottom sheet) when disabled
  • App routing: /personality redirects to /dashboard when the feature is disabled (so deep-links can't bypass the toggle)

Test plan

  • pnpm build in dashboard/ — passes, no type errors
  • pnpm test in dashboard/ — 15 passed
  • Manual: toggle off in Settings, confirm nav link disappears and /personality redirects; toggle back on and confirm it returns

🤖 Generated with Claude Code

Adds a localStorage-backed feature flag so users can hide the
Personality nav link/page and its route when they don't want the
MBTI-style profiling feature enabled.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@ClaudiaFang
ClaudiaFang marked this pull request as ready for review July 23, 2026 11:20
@ClaudiaFang
ClaudiaFang merged commit 2186f2f into master Jul 23, 2026
1 check passed
@ClaudiaFang
ClaudiaFang deleted the worktree-graceful-wiggling-nygaard branch July 23, 2026 11:20
ClaudiaFang pushed a commit that referenced this pull request Jul 23, 2026
- Remove hardcoded getClaudeDir() guard in sync.ts (let provider handle discovery)
- Remove unused os import from sync.ts
- Remove unused getClaudeDir import from sync.ts
- Fix filePath.split('/').pop() in titles.ts to use path.basename()

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
ClaudiaFang pushed a commit that referenced this pull request Jul 23, 2026
feat: add SessionProvider interface and fix Windows path handling
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