8000 feat: onedrive word write tool by iwilltry42 · Pull Request #413 · obot-platform/tools · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

feat: onedrive word write tool #413

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

Merged
merged 8 commits into from
Feb 7, 2025

Conversation

iwilltry42
Copy link
Contributor
@iwilltry42 iwilltry42 commented Feb 5, 2025

Adds a new Write Doc tool that allows for creating new docx documents or updating existing ones in OneDrive.
It works with OneDrive item IDs or filepaths within OneDrive, creating directories as needed.
Conversion from Markdown to Docx is done either via Pandoc or LibreOffice, whichever is available.
In our docker images we definitely have LibreOffice (soffice binary) available.

Example of a Doc after:

  • having the AI create a writing guide
  • tool call to create a file writingguide.docx in root
  • tool call to add some more content to it (third section)
  • tool call to write that same content to a file in a nested folder

Screenshot 2025-02-05 at 17-06-49 mynewdoc docx

Issue obot-platform/obot#1468

Note: It may make sense to also rely on pandoc/soffice instead of docconv to transform back from docx to markdown when reading a file.

@iwilltry42 iwilltry42 force-pushed the feat/word-write branch 2 times, most recently from 36dd4b6 to 03af393 Compare February 5, 2025 16:24
@@ -16,6 +16,8 @@ Tools: ../../oauth2
"Fields.Read",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think you should replace Fields.Read to File.Read. I think `"Fields.Read" are added accidentlly as a typo. The scope doesn't exist

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, this was my fault.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess they could be removed as we have ReadWrite below, but I'll leave them here as the Azure permissions played tricks on me more than once :D

Copy link
Contributor
@thedadams thedadams left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good to me once the invalid Fields scopes are removed.

iwilltry42 and others added 8 commits February 7, 2025 10:03
Co-authored-by: Grant Linville <grant@acorn.io>
Co-authored-by: Grant Linville <grant@acorn.io>
Co-authored-by: Grant Linville <grant@acorn.io>
Co-authored-by: Grant Linville <grant@acorn.io>
@iwilltry42 iwilltry42 merged commit 7a90bd0 into obot-platform:main Feb 7, 2025
1 check passed
@iwilltry42 iwilltry42 deleted the feat/word-write branch February 7, 2025 10:17
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.

4 participants
0