Skip to content

feat: add goose configure flow for adjusting amt of tool output shown#1048

Merged
laanak08 merged 1 commit intomainfrom
marcelle/cfg-tool-output-verbose
Feb 13, 2025
Merged

feat: add goose configure flow for adjusting amt of tool output shown#1048
laanak08 merged 1 commit intomainfrom
marcelle/cfg-tool-output-verbose

Conversation

@laanak08
Copy link
Contributor

@laanak08 laanak08 commented Feb 3, 2025

This change adds a goose configure flow for selecting a tool-output verbosity.
new behavior is to default to verbose logging.
leaves it up to users to decide to launch configure and update if desired.

@laanak08 laanak08 linked an issue Feb 3, 2025 that may be closed by this pull request
@michaelneale
Copy link
Collaborator

I think it would be better to pick a default if current is not right, and not add more configuration to it ideally.

@laanak08
Copy link
Contributor Author

laanak08 commented Feb 3, 2025

I think it would be better to pick a default if current is not right, and not add more configuration to it ideally.

i misspoke, it is as you say it should be. it defaults to the verbose logging, which is what we want.

@Owaisashraf001
Copy link

Successful

@laanak08 laanak08 closed this Feb 13, 2025
@laanak08 laanak08 force-pushed the marcelle/cfg-tool-output-verbose branch from 2ecf069 to 744a5cf Compare February 13, 2025 19:21
@laanak08 laanak08 reopened this Feb 13, 2025
@laanak08 laanak08 merged commit a8dcb09 into main Feb 13, 2025
2 checks passed
@laanak08 laanak08 deleted the marcelle/cfg-tool-output-verbose branch February 13, 2025 19:27
@laanak08 laanak08 restored the marcelle/cfg-tool-output-verbose branch February 14, 2025 00:46
@laanak08 laanak08 deleted the marcelle/cfg-tool-output-verbose branch February 14, 2025 01:42
ahau-square pushed a commit that referenced this pull request May 2, 2025
* main:
  feat: add goose configure flow for adjusting amt of tool output shown (#1048)
  docs: fixing exchange url to public version (#67)
  docs: Update CONTRIBUTING.md (#69)
  goose->goose-ai in install instructions (#7)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Lack of context-aware system message.

4 participants