8000 fix: update workspace read/write method, so it's clear it only supports plain text format by tybalex · Pull Request #487 · obot-platform/tools · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

fix: update workspace read/write method, so it's clear it only supports plain text format #487

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
Merged
merged 5 commits into from
Mar 13, 2025
Merged

fix: update workspace read/write method, so it's clear it only supports plain text format #487

merged 5 commits into from
Mar 13, 2025

Conversation

tybalex
Copy link
Contributor
@tybalex tybalex commented Mar 11, 2025 •
edited
Loading

@tybalex tybalex self-assigned this Mar 11, 2025
cjellick
cjellick reviewed Mar 11, 2025
View reviewed changes
cjellick
cjellick requested changes Mar 11, 2025
View reviewed changes
@tybalex tybalex requested a review from cjellick March 11, 2025 21:37
@tybalex
Copy link
Contributor Author
tybalex commented Mar 11, 2025 •
edited
Loading

it is very interesting that, now that I prompt the Obot to NOT read non-plain texts with workspace_read, it uses knowledge tool to find the answer:
Screenshot 2025-03-11 at 3 20 21 PM
Screenshot 2025-03-11 at 3 22 34 PM

@tybalex @cjellick
fix: Update workspace-files/main.go comment
c020f1b 
Co-authored-by: Craig Jellick <craig.jellick@gmail.com>
@tybalex tybalex requested a review from cjellick March 13, 2025 04:38
@cjellick cjellick merged commit d0823e2 into obot-platform:main Mar 13, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cjellick cjellick cjellick approved these changes

@thedadams thedadams Awaiting requested review from thedadams

@njhale njhale Awaiting requested review from njhale

@StrongMonkey StrongMonkey Awaiting requested review from StrongMonkey

@g-linville g-linville Awaiting requested review from g-linville

Assignees

@tybalex tybalex

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@tybalex @cjellick
0