-
Notifications
You must be signed in to change notification settings - Fork 2
fix(2495): Karen humanizes EDS list output before showing to user #360
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
base: main
Are you sure you want to change the base?
Changes from all commits
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 |
|---|---|---|
| @@ -1 +1 @@ | ||
| 1.2.269 | ||
| 1.2.271 |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -28,6 +28,11 @@ | |
| better not to mention it at all. Policy document path (such as /support/summary) might be useful for admin setup, | ||
| but it is not useful for a regular user who asks a question, so do not mention it. | ||
|
|
||
| When EDS tools return raw data (JSON, markdown tables, field names like `eds_id` or `eds_type`, timestamps, | ||
| `eds_scan_problem`), NEVER paste it verbatim. Translate into plain English: one line per source with a friendly name, | ||
| type ("Google Drive folder", "website crawl", "uploaded file"), last scan time in relative phrasing ("scanned today", | ||
|
Contributor
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.
|
||
| "last scanned 3 days ago"), and any blocker in plain words — hide all raw IDs, hashes, and technical field names. | ||
|
Contributor
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. we may not need 4 lines of prompt for this, instruction can be a bit shorter |
||
|
|
||
| Pay attention to which messengers permit tables, and what markup they use. Avoid using double asterisks, | ||
| that almost never works in messengers (not in Slack, not in Telegram). Follow the messenger-specific formatting | ||
| rules carefully. | ||
|
|
||
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.
don't directly mention English, user may be talking in diff language