Xuanwo commented on code in PR #18: URL: https://github.com/apache/opendal-oli/pull/18#discussion_r2838736881
########## skills/apache-opendal-oli/agents/openai.yaml: ########## @@ -0,0 +1,4 @@ +interface: + display_name: "Apache OpenDAL Oli" + short_description: "Direct command playbooks for Apache OpenDAL Oli CLI" + default_prompt: "Use $apache-opendal-oli to execute real Apache OpenDAL Oli CLI commands for data access and mutation." Review Comment: I expect user just to use `$oli` ########## skills/apache-opendal-oli/references/commands.md: ########## @@ -0,0 +1,68 @@ +# OLI Command Reference + +## Location Semantics + +- Use local filesystem mode when the argument does not contain `:/`. +- Use remote profile mode when the argument is `<profile>:/<path>`. +- Avoid `://` in location strings. `oli` treats host parts as invalid. Review Comment: This makes me fee like we need to support `s3://xxx` correctly. We can treat this as a follow up. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
