Add simple AI Harness#1352
Conversation
stevepolitodesign
force-pushed
the
sp-ai-harness
branch
from
fda7475 to
dfbdf28
Compare
| empty_directory ".claude" | ||
| empty_directory ".claude/rules" | ||
|
|
||
| get "#{base_url}/CLAUDE.md", ".claude/CLAUDE.md" |
There was a problem hiding this comment.
One issue here is this is Claude-specific. That works for thoughtbot, but do we want to support other services?
There was a problem hiding this comment.
FYI it's being discussed here thoughtbot/guides#786 (comment)
stevepolitodesign
force-pushed
the
sp-ai-harness
branch
from
dfbdf28 to
cb31d09
Compare
|
This looks great! Annoyingly, this will include the usage section which we've effectively already done. I can't think of a non-fragile way to include these. Unless perhaps we add a README.md in the same guides folder with these instructions and then the CLAUDE.md is "ready" to use, they just need to update the app name and add the sentence on app context. @laicuRoot what do you think? |
|
@jaredlt @laicuRoot alternatively, would it be worth making Suspenders the source of truth for this? Then, we could update the Guides to link to these files? |
|
Hey folks! I think extracting the instructions to a README makes sense @jaredlt 👍
🤔 I'm not sure about this @stevepolitodesign. Shouldn't our guides be the source of truth for how we work and how we recommend writing code? In any case, if you think that would make the rules more "maintainable" I'm ok with it. By the way, now rails ships with an AGENT.md file. |
|
I believe the AGENTS.md file in the Rails repo is more about contributing to the rails repo, and not so much about developing apps using rails. I can't decide what is best about the source of truth for where this should live :) |
|
@jaredlt @laicuRoot I've opened thoughtbot/guides#789 to address the concerns mentioned in this PR. Once that merges, we can merge this. |
Relates to #1350 and #1351. We introduced [thoughtbot agents rules](thoughtbot/guides#783) to our guides a few months ago. This commit simply pulls those files into newly generated Suspenders applications. There's a case to be made that these should be kept separate, but I like having a single source of truth.
stevepolitodesign
force-pushed
the
sp-ai-harness
branch
from
cb31d09 to
89b440a
Compare
|
@jaredlt @laicuRoot now that thoughtbot/guides#789 has merged, I think we're in a good state. You can also manually test this by pulling down the project and running We also have coverage in CI. |
jaredlt
left a comment
jaredlt
left a comment
There was a problem hiding this comment.
Looks good!
I think we can explore supporting other agents etc in a future iteration.
3 participants
Relates to #1350 and
#1351.
We introduced thoughtbot agents
rules to our guides a
few months ago. This commit simply pulls those files into newly
generated Suspenders applications. There's a case to be made that these
should be kept separate, but I like having a single source of truth.