feat: configurable description for groups created via /new chat - #220
Open
maiqigh wants to merge 1 commit into
Open
feat: configurable description for groups created via /new chat#220maiqigh wants to merge 1 commit into
maiqigh wants to merge 1 commit into
Conversation
createBoundChat() already accepts a description and forwards it to channel.createChat(), but handleNewChat() never passed one, so groups created via /new chat always end up with an empty description. Add an optional preferences.newChatDescription config key and thread it through. A stable description marker keeps bridge-created groups identifiable (searchable in Lark, visible in chat settings) even after the group is renamed to a topic-specific name. Default behavior is unchanged when the key is unset. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Motivation
Groups created with
/new chatcurrently have an empty description. In practice it's useful to stamp bridge-created groups with a stable marker (e.g.Claude Code session group) so they stay identifiable — searchable in Lark, visible in chat settings — even after the group gets renamed to a topic-specific name later.The plumbing already exists:
createBoundChat()accepts adescriptionand forwards it tochannel.createChat(), buthandleNewChat()never passes one.Changes
preferences.newChatDescriptionconfig key (documented via schema JSDoc, consistent with the other preference keys).handleNewChat()threads it intocreateBoundChat().channel.createChat, and omitted when unset.Default behavior is unchanged when the key is not configured.
Testing
pnpm typecheckpassespnpm vitest run tests/unit/bot/group.test.ts— 3/3 passpnpm test: 618 passed; the 2 failures (tests/unit/cli/preflight.test.ts,tests/integration/cli/secrets-profile.test.ts) also fail on unmodifiedmainin my environment — they read the developer's real~/.lark-channelprofile, unrelated to this change.🤖 Generated with Claude Code