search

jacob-ai-bot / jacob

Just Another Coding Bot
https://jacb.ai
Apache License 2.0
96 stars 13 forks source link

added tailwind styles to the prompt #19

Closed kleneway closed 6 months ago

vercel[bot] commented 6 months ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
otto-mvp ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 22, 2024 11:13pm
cpirich commented 6 months ago

@kleneway: A few question on my post-merge review:

(lots of detail in tailwind config here)

  1. Can we do any of this by default? (locate the presence of a tailwind config file / directory)
  2. Do you think that including a full tailwind.config.ts is the best way to pass "styles" to the LLM? (In some projects I've worked on, this file can be quite large and could use up a lot of the context window) I also think some tasks that are less front-end focused may not benefit from this, though I don't know we detect this.
  3. The new directories config option for tailwindConfig would seem to imply to me the "directory" that contains the tailwind config - but not the actual name of the file. But the code seems to assume that this "directory" config includes the file name, because we don't append a file name to that path. Also, if it were just a directory - how to do we know if it should be tailwind.config.ts vs tailwind.config.js (or some other custom name)?