-
Notifications
You must be signed in to change notification settings - Fork 3k
[codex] Add explicit channel memory for messaging channels #6051
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
fbbae08
ba20001
3d22859
e4a20a2
b189884
3094e0a
7313074
f242509
c20d24c
bb97da9
e23d5c4
558fd70
192559b
26f050f
6de49e0
045c4f7
7846f88
6215333
c5fcb95
a3d01c7
26448df
b763af3
b7e62f0
3efbf44
5c2151c
84b231d
fbbee48
b97d9e6
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -26,6 +26,7 @@ package-lock.json | |
| .qoder | ||
| .claude | ||
| .codex | ||
| .worktrees/ | ||
|
|
||
| # Qwen Code Configs | ||
| .qwen/* | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -83,6 +83,18 @@ Controls how conversation sessions are managed: | |
| - **`thread`** — One session per thread/topic. Useful for group chats with threads. | ||
| - **`single`** — One shared session for all users. Everyone shares the same conversation. | ||
|
|
||
| ### Channel Memory | ||
|
|
||
| Channel memory lets an authorized channel member save stable context for one chat or thread. Qwen Code injects that memory when a fresh channel session starts, including after `/clear`. | ||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. [Suggestion] Docs don't mention that memory injection is restricted to authorized users This paragraph describes channel memory injection on fresh sessions but doesn't mention that it only applies to prompts from users listed in Consider adding a sentence like: "Channel memory is only injected into prompts from authorized users listed in |
||
|
|
||
| Commands: | ||
|
|
||
| - `/remember-channel <text>` saves a memory line for the current chat or thread. | ||
| - `/channel-memory` shows saved memory for the current chat or thread. | ||
| - `/forget-channel confirm` clears saved memory for the current chat or thread. | ||
|
|
||
| Only users listed in `allowedUsers` can read, write, or clear channel memory. If `allowedUsers` is empty, channel memory commands are disabled for everyone. | ||
|
|
||
| ### Token Security | ||
|
|
||
| Bot tokens should not be stored directly in `settings.json`. Instead, use environment variable references: | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[Suggestion] The
.worktrees/entry appears unrelated to channel memory. Consider moving it to a separate PR to keep this diff focused.— glm-5.2 via Qwen Code /review