fix: [Note] Title not filled by default #13838

[bugisthegod]

Fixes #13838
When creating a note from the command menu side panel (e.g. clicking "Add Note" in a related notes section on an Opportunity/company/people page), the title field was not auto-focused — focus point went to body instead.

Root Cause

When a record opens in the side panel, there is no page navigation, so PageChangeEffect (which handles title auto-focus for full-page views) never runs. openNewRecordTitleCell() was simply never called for the side-panel path.

Fix

openRecordInCommandMenu is the single entry point for all side-panel record opens, so title auto-focus is handled there once for all callers.
Previously, useCreateNewIndexRecord called openRecordInCommandMenu and then called openNewRecordTitleCell separately, which would have caused a double invocation after this fix. The redundant call has been removed.

Before

before.mp4

After

after.mp4

[Copilot]

Pull request overview

Fixes a UX regression where creating a Note via the command menu side panel did not auto-focus the title field (typing went into the rich-text body), by ensuring the “new record title cell” is opened/focused when records are opened in the side panel flow.

Changes:

• Trigger title-cell focus for newly created records inside openRecordInCommandMenu (side panel entry point).
• Remove the redundant title-cell opening logic from useCreateNewIndexRecord to avoid double-invocation.
• Add test coverage asserting title-cell focus occurs only for isNewRecord: true.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File	Description
packages/twenty-front/src/modules/ui/utilities/focus/constants/DebugFocusStack.ts	Toggles focus-stack debugging (currently enabled).
packages/twenty-front/src/modules/object-record/record-table/hooks/useCreateNewIndexRecord.ts	Removes redundant title-cell focus call after opening side panel record.
packages/twenty-front/src/modules/command-menu/hooks/useOpenRecordInCommandMenu.ts	Centralizes new-record title focus for side panel record opens.
packages/twenty-front/src/modules/command-menu/hooks/tests/useOpenRecordInCommandMenu.test.tsx	Adds tests for title-cell focus behavior on new record opens.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[cubic-dev-ai]

No issues found across 4 files

[greptile-apps]

Greptile Summary

This PR successfully fixes the note title auto-focus issue by centralizing the logic in openRecordInCommandMenu, eliminating duplicate calls and ensuring title fields receive focus when notes are created in the side panel.

• Centralizes title auto-focus logic in useOpenRecordInCommandMenu to handle all side-panel record opens
• Removes redundant openRecordTitleCell call from useCreateNewIndexRecord to prevent double invocation
• Adds comprehensive tests to verify the new behavior works correctly
• CRITICAL: DEBUG_FOCUS_STACK flag accidentally left enabled - will cause excessive logging in production

Confidence Score: 2/5

• Critical issue with debug flag enabled - must be fixed before merge
• The core fix is well-implemented with proper testing, but the accidentally enabled debug flag is a critical issue that would cause excessive logging in production and potential performance impact
• DebugFocusStack.ts requires immediate attention - debug flag must be set to false

Important Files Changed

Filename	Overview
packages/twenty-front/src/modules/ui/utilities/focus/constants/DebugFocusStack.ts	Debug flag accidentally left enabled - critical issue that will cause excessive logging in production
packages/twenty-front/src/modules/command-menu/hooks/tests/useOpenRecordInCommandMenu.test.tsx	Added comprehensive tests for new title cell auto-focus behavior; minor improvement needed in assertion specificity
packages/twenty-front/src/modules/command-menu/hooks/useOpenRecordInCommandMenu.ts	Centralizes title cell auto-focus logic for side panel opens; correctly handles new record state with proper field lookup
packages/twenty-front/src/modules/object-record/record-table/hooks/useCreateNewIndexRecord.ts	Removes redundant title cell opening logic to prevent double invocation; correctly delegates to centralized implementation

_{Last reviewed commit: 8d826e6}

[greptile-apps]

_{4 files reviewed, 2 comments}

_{Edit Code Review Agent Settings | Greptile}

[github-actions]

🚀 Preview Environment Ready!

Your preview environment is available at: http://bore.pub:16874

This environment will automatically shut down when the PR is closed or after 5 hours.

[etiennejouan]

LGTM ! Thank you for your contribution !

[twenty-eng-sync]

Hey @FelixMalfait! After you've done the QA of your Pull Request, you can mark it as done here. Thank you!

[twenty-eng-sync]

Hey @FelixMalfait! After you've done the QA of your Pull Request, you can mark it as done here. Thank you!

[github-actions]

Thanks @bugisthegod for your contribution!
This marks your 5th PR on the repo. You're top 5% of all our contributors 🎉
See contributor page - Share on LinkedIn - Share on Twitter