Skip to content
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.

Sign up for GitHub

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

Jump to bottom

💡 [Feature]: improve /manage chat command so that it uses text output modes for all list CLI for M365 command types #375

Closed
Tracked by #353
Adam-it opened this issue Dec 9, 2024 · 0 comments · Fixed by #378
Closed
Tracked by #353

💡 [Feature]: improve /manage chat command so that it uses text output modes for all list CLI for M365 command types #375

Adam-it opened this issue Dec 9, 2024 · 0 comments · Fixed by #378
Assignees
@Adam-it
Labels
⭐ enhancement New feature or request 👨‍💻work in progress I am working on it
Milestone
v4.X

Comments

@Adam-it
Copy link
Member

Adam-it commented Dec 9, 2024 •
edited
Loading

Currently when the response is very big we may get Error: Message exceeds token limit. error. This is because the message exceeds 4k token limit. Currently for CLI commands we pass output md which for some list command types may be too much. We should use text output mode for list command types and md for get command types
@Adam-it Adam-it mentioned this issue Dec 9, 2024
Closed
3 tasks
@Adam-it Adam-it changed the title Currently when the response is very big we may get Error: Message exceeds token limit. error. This is because the message exceeds 4k token limit. Currently for CLI commands we pass output md here 💡 [Feature]: improve /manage chat command so that it uses text output modes for all list CLI for M365 command types Dec 9, 2024
@Adam-it Adam-it self-assigned this Dec 9, 2024
@Adam-it Adam-it added ⭐ enhancement New feature or request 👨‍💻work in progress I am working on it labels Dec 9, 2024
@Adam-it Adam-it added this to the v4.X milestone Dec 9, 2024
@Adam-it Adam-it mentioned this issue Dec 9, 2024
Merged
2 tasks
@Adam-it Adam-it linked a pull request Dec 9, 2024 that will close this issue
Updates /manage chat command to use text output mode for list CLI for M365 commands. Closes: #375 #378
Merged
2 tasks
Adam-it added a commit that referenced this issue Dec 9, 2024
@Adam-it
Updates /manage chat command to use text output mode for list CLI for…
b1e59fb 
… M365 commands. Closes: #375 (#378)

## 🎯 Aim

The aim is to update the command output mode to default to `text` for
`list` CLI for Microsoft 365 commands and `md` for `get` commands

## ✅ What was done

- [X] updated manage command grouding prompt
- [X] updated the output mode to use text for list commands

## 🔗 Related issue

Closes: #375
@Adam-it Adam-it closed this as completed Dec 9, 2024
Adam-it added a commit that referenced this issue Dec 10, 2024
@Adam-it
Updates /manage chat command to use text output mode for list CLI for…
8b41071 
… M365 commands. Closes: #375 (#378)

## 🎯 Aim

The aim is to update the command output mode to default to `text` for
`list` CLI for Microsoft 365 commands and `md` for `get` commands

## ✅ What was done

- [X] updated manage command grouding prompt
- [X] updated the output mode to use text for list commands

## 🔗 Related issue

Closes: #375
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Adam-it Adam-it

Labels
⭐ enhancement New feature or request 👨‍💻work in progress I am working on it
Projects
None yet
Milestone
v4.X
1 participant
@Adam-it