fix(server): generate durable thread titles - #5357
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
ApprovabilityVerdict: Approved ed1863e This PR improves prompts for thread title generation and consolidates a reasoning effort constant to the shared contracts package. The changes affect AI-generated title quality without modifying application logic, and the constant consolidation maintains the existing 'low' reasoning effort value. You can customize Macroscope's approvability policy. Learn more. |
b91f78e to
ed1863e
Compare
Dismissing prior approval to re-evaluate ed1863e
(cherry picked from commit d7950ac)
Thread titles often captured the requested artifact or workflow instead of the lasting subject, making conversations harder to find later.
This switches initial and regenerated thread titles to the evaluated durable-intent prompt. It also makes the default text generation selection explicitly use GPT-5.6 Luna with low reasoning effort, matching the tested configuration.
Testing
Built with GPT-5.6 Sol at high reasoning effort through the T3 Code Codex harness.
Note
Low Risk
Prompt-only title behavior change plus settings defaults; no auth or data-path changes, with coverage in prompt and settings tests.
Overview
Thread title generation is reworked so titles emphasize durable subject and outcome instead of restating the user message or naming artifacts (mocks, plans, PRs).
buildThreadTitlePromptnow uses a subject/outcome/incidental-instructions framing, editorial rules (length, reviews/research, regeneration across full thread), and optionalguidance/rulesLabelon the sharedbuildPromptFromMessagehelper. Tests and the Claude integration test expectations were updated for the new prompt text.Default text generation explicitly defaults to GPT-5.6 Luna with
reasoningEffort: low:DEFAULT_TEXT_GENERATION_REASONING_EFFORTis exported from contracts, wired intoServerSettings.textGenerationModelSelection, and used as the Codex CLI fallback when the selection omits effort.Reviewed by Cursor Bugbot for commit ed1863e. Bugbot is set up for automated code reviews on this repo. Configure here.
Note
Rewrite thread title prompts to produce durable, subject-focused titles
buildThreadTitlePromptin TextGenerationPrompts.ts with a new guidance section and editorial rules that direct the model to extract subject/outcome rather than summarizing the user's initial request verbatim.buildPromptFromMessageto accept optionalguidancelines and a customrulesLabel, used to inject the new editorial section.DEFAULT_TEXT_GENERATION_REASONING_EFFORT('low') from contracts and uses it as the defaultreasoningEffortin bothCodexTextGenerationand the defaulttextGenerationModelSelectioninServerSettings.ServerSettingsdecoded without an explicittextGenerationModelSelectionnow includes areasoningEffort: 'low'option by default.Macroscope summarized ed1863e.