-
Notifications
You must be signed in to change notification settings - Fork 2
Agent guidance #15
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
base: main
Are you sure you want to change the base?
Agent guidance #15
Conversation
dawsontoth
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reads logically to me. Some of these I think we could roll into core in v5 so we don't have to guide the LLM, like the static bit. I also wonder if we can describe the config yaml schema better (for AI and users)
heskew
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great guidelines. Just some minor suggestions for this initial doc. Some examples and even more recommendations might help but I'm curious to see how this alone affects agents when blindly bootstrapping a Harper app...
| @@ -0,0 +1,17 @@ | |||
| --- | |||
| name: docs_agent | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
seems okay to keep the front matter (metadata) but should the name be more generic?
| ## Your role | ||
| - Build high-quality Harper applications that demonstrate best practices. | ||
|
|
||
| ## Guidelines for agents building Harper applications |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should we mention what globals are available? e.g.
## Available Globals
- `tables` - Access tables from the default `data` database (e.g., `const { User } = tables;`)
- `Resource` - Base class for custom resources
For sure, this will be the default in v5, the static flag has been the best thing I could come up with as a preparation for this change in v5, as awkward as it is.
Perhaps; I guess my belief has been that AI actually has seen and fully understands the config.yaml. But has conflated it with harperdb-config.yaml and in doing so has concluded that it can get very creative in what config it can add there. But I could be wrong about that.
Yeah, is there a good way we can start testing this? I know we have been making some guesses about if/how valuable examples might be and corrective guidance. Ultimately it would be nice to have empirical evidence of what works well here, since I am probably wrong about a lot of this :). Maybe we start building out the LLM APIs we will use and create some test scripts where we can try out different guidance? |
Co-authored-by: Nathan Heskew <heskew@pm.me>
Co-authored-by: Nathan Heskew <heskew@pm.me>
Co-authored-by: Nathan Heskew <heskew@pm.me>
Co-authored-by: Nathan Heskew <heskew@pm.me>
Co-authored-by: Nathan Heskew <heskew@pm.me>
Co-authored-by: Nathan Heskew <heskew@pm.me>
Just need to try it. I've been asking various bots what they think would be most useful but we just need to really try it. Also have docs for folks to try themselves and have a clear feedback loop. This is probably going to continuously evolve too...so it'll be something else to maintain (but could have a great devexp roi...) |
No description provided.