Remove chat session title truncation

[Copilot]

Chat session titles were truncated to 30 characters in editor tabs, cutting off longer session names.

Changes

• Removed truncation logic in ChatEditorInput.getName() - now returns full title for both custom and default titles
• Removed unused constant ChatEditorTitleMaxLength from constants.ts
• Cleaned up imports - removed truncate function import

Chat session titles now display in full without ellipsis.

Original prompt

There is truncation of the chat session title somewhere in the codebase. Remove that.

The user has attached the following file paths as relevant context:

• .github/instructions/learnings.instructions.md
• .github/copilot-instructions.md
• AGENTS.md

Created from VS Code.

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

[nidhishgajjar]

Orb Code Review (powered by GLM 5.1 on Orb Cloud)

Summary

This PR cleanly removes chat session title truncation in VS Code editor tabs, allowing full display of session names. The implementation is straightforward and well-executed.

Architecture

✅ Clean Removal: Properly removes both the truncation logic and unused constants
✅ Minimal Impact: Changes are isolated to chat editor functionality
✅ No Breaking Changes: Maintains existing API surface while improving UX
✅ Code Cleanup: Removes unused imports and constants

Issues Found

✅ No Issues: The changes are clean and correct
✅ Import Cleanup: Properly removes unused truncate function import
✅ Constant Cleanup: Removes unused ChatEditorTitleMaxLength constant
✅ Logic Simplification: Simplified the conditional logic in getName() method

Cross-file Impact

✅ Isolated Changes: Only affects chat editor title display
✅ No Dependencies: No other code depends on the removed constant
✅ Backward Compatibility: Maintains compatibility - just displays more text

Code Quality

✅ TypeScript Safety: Changes maintain type safety
✅ Consistent Style: Follows VS Code codebase conventions
✅ Clear Intent: The change is obvious and purposeful
✅ Minimal Diff: Only changes what's necessary

User Experience

✅ UX Improvement: Users can now see full chat session titles
✅ No Regressions: No negative impact on existing functionality
✅ Better Accessibility: Full titles improve context and navigation

Assessment

Recommendation: ✅ APPROVE

This is an excellent, well-crafted change that:

1. Solves the problem: Removes unwanted title truncation
2. Clean implementation: No leftover code or unused imports
3. Minimal footprint: Only touches what's necessary
4. Improves UX: Users get more context with full titles
5. Follows best practices: Proper cleanup of unused constants

The change is exactly what you'd expect from the description - simple, clean, and effective. The code diff shows good attention to detail with proper import cleanup.

No action required - this is ready to merge as-is.