Skip to content

Commit 336ed4a

Browse files
lmjabreuclaude
andauthored
docs(thread): document group IDs in --notify help text (#34)
* docs(thread): document group IDs in --notify help text `tdc thread reply/create --notify` already accepts custom group IDs (partitioned via resolveNotifyIds → groups), but the help text only mentioned "user IDs". Since Comms group IDs are non-numeric base58 strings that look nothing like user IDs, the capability was effectively undiscoverable from --help. Mention groups in both option descriptions and add a group-notify example to the reply command's existing example block (alongside --close / --file). No behaviour change. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> * docs(skill): sync SKILL_CONTENT --notify examples with group support Per AGENTS.md, src/lib/skills/content.ts must track command-description and example changes. The reply/create --notify examples said "users" only; make them mention groups and add a group-notify example (base58 group ID next to a numeric user ID). Regenerated skills/comms-cli/SKILL.md via `sync:skill`; check:skill-sync passes. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
1 parent a33da5e commit 336ed4a

3 files changed

Lines changed: 11 additions & 8 deletions

File tree

skills/comms-cli/SKILL.md

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -90,14 +90,15 @@ tdc thread view <ref> --raw # Show raw markdown
9090
tdc thread create <channel-ref> "Title" "content" # Create a new thread
9191
tdc thread create <channel-ref> "Title" "content" --json # Create and return as JSON
9292
tdc thread create <channel-ref> "Title" "content" --json --full # Include all thread fields
93-
tdc thread create <channel-ref> "Title" "content" --notify 123,456 # Notify specific users
93+
tdc thread create <channel-ref> "Title" "content" --notify 123,456 # Notify specific users and/or groups by ID
9494
tdc thread create <channel-ref> "Title" "content" --unarchive # Land thread in author's Inbox (overrides default Comms auto-archive)
9595
tdc thread create <channel-ref> "Title" "content" --no-unarchive # Force archive even when userSettings.unarchiveNewThreads=true
9696
tdc thread create <channel-ref> "Title" "content" --dry-run # Preview without posting
9797
tdc thread create <channel-ref> "Title" --file ./a.png # Attach a file (repeatable; content optional)
9898
tdc thread reply <ref> "content" # Post a comment (notifies EVERYONE_IN_THREAD by default)
9999
tdc thread reply <ref> "content" --notify EVERYONE # Notify all workspace members
100-
tdc thread reply <ref> "content" --notify 123,id:456 # Notify specific user IDs
100+
tdc thread reply <ref> "content" --notify 123,id:456 # Notify specific users by ID
101+
tdc thread reply <ref> "content" --notify 123,Cbzzm11ZeYZoJYD4a6rti # Notify a user and a group (group IDs are base58, not numeric)
101102
tdc thread reply <ref> "content" --json # Post and return comment as JSON
102103
tdc thread reply <ref> "content" --json --full # Include all comment fields
103104
tdc thread reply <ref> "content" --close # Reply and close the thread
@@ -128,7 +129,7 @@ tdc thread update <ref> "New body" --json # Update and return { id, content } a
128129
tdc thread update <ref> "New body" --json --full # Update and return full thread as JSON
129130
```
130131

131-
Default `--notify` for reply is EVERYONE_IN_THREAD, which may notify more people than intended. Before posting, confirm with the user whether specific people should be notified instead (via `--notify <user-ids>`). Options: EVERYONE, EVERYONE_IN_THREAD, or comma-separated ID refs.
132+
Default `--notify` for reply is EVERYONE_IN_THREAD, which may notify more people than intended. Before posting, confirm with the user whether specific people should be notified instead (via `--notify <ids>`). Options: EVERYONE, EVERYONE_IN_THREAD, or comma-separated user and/or group IDs.
132133

133134
`--notify` automatically resolves IDs: group IDs are routed to the `groups` API field, user IDs to `recipients`. No special syntax needed.
134135

src/commands/thread/index.ts

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@ Examples:
5050
withUnvalidatedChoices(
5151
new Option(
5252
'--notify <recipients>',
53-
'Notification recipients: EVERYONE, EVERYONE_IN_THREAD, or comma-separated user IDs (default: EVERYONE_IN_THREAD)',
53+
'Notification recipients: EVERYONE, EVERYONE_IN_THREAD, or comma-separated user and/or group IDs (default: EVERYONE_IN_THREAD)',
5454
),
5555
['EVERYONE', 'EVERYONE_IN_THREAD'],
5656
),
@@ -68,6 +68,7 @@ Examples:
6868
tdc thread reply 12345 "Sounds good!"
6969
echo "Long reply" | tdc thread reply 12345
7070
tdc thread reply 12345 "Done" --close --json
71+
tdc thread reply 12345 "Heads up" --notify 67890,Cbzzm11ZeYZoJYD4a6rti
7172
tdc thread reply 12345 "See attached" --file ./diagram.png
7273
tdc thread reply 12345 --file ./a.png --file ./b.pdf`,
7374
)
@@ -76,7 +77,7 @@ Examples:
7677
thread
7778
.command('create <channel-ref> <title> [content]')
7879
.description('Create a new thread in a channel')
79-
.option('--notify <recipients>', 'Comma-separated user IDs to notify')
80+
.option('--notify <recipients>', 'Comma-separated user and/or group IDs to notify')
8081
.option(
8182
'--unarchive',
8283
'Unarchive after creation so the thread appears in your Inbox (overrides userSettings.unarchiveNewThreads when false)',

src/lib/skills/content.ts

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -94,14 +94,15 @@ tdc thread view <ref> --raw # Show raw markdown
9494
tdc thread create <channel-ref> "Title" "content" # Create a new thread
9595
tdc thread create <channel-ref> "Title" "content" --json # Create and return as JSON
9696
tdc thread create <channel-ref> "Title" "content" --json --full # Include all thread fields
97-
tdc thread create <channel-ref> "Title" "content" --notify 123,456 # Notify specific users
97+
tdc thread create <channel-ref> "Title" "content" --notify 123,456 # Notify specific users and/or groups by ID
9898
tdc thread create <channel-ref> "Title" "content" --unarchive # Land thread in author's Inbox (overrides default Comms auto-archive)
9999
tdc thread create <channel-ref> "Title" "content" --no-unarchive # Force archive even when userSettings.unarchiveNewThreads=true
100100
tdc thread create <channel-ref> "Title" "content" --dry-run # Preview without posting
101101
tdc thread create <channel-ref> "Title" --file ./a.png # Attach a file (repeatable; content optional)
102102
tdc thread reply <ref> "content" # Post a comment (notifies EVERYONE_IN_THREAD by default)
103103
tdc thread reply <ref> "content" --notify EVERYONE # Notify all workspace members
104-
tdc thread reply <ref> "content" --notify 123,id:456 # Notify specific user IDs
104+
tdc thread reply <ref> "content" --notify 123,id:456 # Notify specific users by ID
105+
tdc thread reply <ref> "content" --notify 123,Cbzzm11ZeYZoJYD4a6rti # Notify a user and a group (group IDs are base58, not numeric)
105106
tdc thread reply <ref> "content" --json # Post and return comment as JSON
106107
tdc thread reply <ref> "content" --json --full # Include all comment fields
107108
tdc thread reply <ref> "content" --close # Reply and close the thread
@@ -132,7 +133,7 @@ tdc thread update <ref> "New body" --json # Update and return { id, content } a
132133
tdc thread update <ref> "New body" --json --full # Update and return full thread as JSON
133134
\`\`\`
134135
135-
Default \`--notify\` for reply is EVERYONE_IN_THREAD, which may notify more people than intended. Before posting, confirm with the user whether specific people should be notified instead (via \`--notify <user-ids>\`). Options: EVERYONE, EVERYONE_IN_THREAD, or comma-separated ID refs.
136+
Default \`--notify\` for reply is EVERYONE_IN_THREAD, which may notify more people than intended. Before posting, confirm with the user whether specific people should be notified instead (via \`--notify <ids>\`). Options: EVERYONE, EVERYONE_IN_THREAD, or comma-separated user and/or group IDs.
136137
137138
\`--notify\` automatically resolves IDs: group IDs are routed to the \`groups\` API field, user IDs to \`recipients\`. No special syntax needed.
138139

0 commit comments

Comments
 (0)