- 
                Notifications
    You must be signed in to change notification settings 
- Fork 8
Add prompt loader and root server prompt #18
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?
Conversation
| 
 | 
c00af81    to
    86834fe      
    Compare
  
    | @JReinhold @shilman I am not locked into the current prompt, but it does work as-intended. I just took a first-pass loosely-modeled after what I've seen Notion use. Their MCP server has worked well for me. Looking forward to hearing your thoughts! | 
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.
Nice!! I'll let @JReinhold test but the change makes sense and LGTM. However, we'd also want to update the README as part of this change too, right?
| Ohhhh yeah good call! I'll push up the change 👍 | 
| Just some added context! I was having trouble getting the MCP server to be used automatically, but I think Claude is just acting a little strange. It wasn't using the tool automatically, but now that I've made a few stories, it seems to have picked up that I want to use the tool. That being said, I do think this prompt change will be helpful! Been using the MCP server to add a bunch of stories to my blog site this weekend and it's been going well! | 
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.
Oh this is really interesting, thank you! I didn't know MCP servers supported root instructions like this, is there any way I can read up on this? I can't really find documentation about this anywhere. Is this supported by all clients?
| @JReinhold sorry for the delay here! I haven't been able to poke at MCP stuff as much as I would have liked to recently, but as far as I know, that  The documentation also doesn't really mention what the  | 
| I've been testing this out lately with the update to Sonnet 4.5 and have been seeing some issues where it's not applying the prompt anymore. When I ask it why didn't use the server, it gives me the usual "You're absolutely right!" junk. But it goes on to say that it jumped right into evaluating the code, which seems to somehow be making it opt to not use the server unless explicitly asked. Doing some testing to see if I can tweak the prompt! I have switched it to using Markdown locally, but I don't think that's the issue here - there have been some updates to the models I've been using, so I suspect it's related. | 
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.
I think something weird happened when resolving the file conflicts earlier - I will update this! Just need to make sure we don't include docs on modifying their AGENT.md and/or their CLAUDE.md files since it should no longer be required. Need to step out for some errands, but will get to this later!
This PR aims to:
Testing
pnpm run storybookget_ui_building_instructionstoolScreenshot:

Additional Context
This prompt setup is modeled after the setup I used in a side project MCP server I've been working on here:
Through testing, I can confirm that the prompt added like this does in fact make the MCP server be used very proactively. It was actually a bit too strong at first and would get used for things it wasn't exactly intended for.
Why is the prompt so long?
I modeled the prompt loosely after Notion's MCP server tools which are very specific and quite lengthy. That being said, it seems this approach is generally better since it removes most of the need for the user to have lengthy back-and-forths with the tool to get a usable response.
Going further
Each
descriptionfield on the tools should probably have a similarly-specific prompt, but I wanted to start with a baseline that we can iterate on first. Let me know what you think!