[PM-37811] llm: Add audit build warnings command#2674
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2674 +/- ##
==========================================
- Coverage 87.25% 86.56% -0.69%
==========================================
Files 1918 2136 +218
Lines 172164 186042 +13878
==========================================
+ Hits 150216 161050 +10834
- Misses 21948 24992 +3044 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
|
||
| ## Phase 3: Report | ||
|
|
||
| Write a temporary markdown report to `/tmp/build_warnings_report.md` (NOT to `Docs/` or any tracked directory — this is an ephemeral audit artifact). |
There was a problem hiding this comment.
🤔 Even though I as the user shouldn't be required to open the file, I still would like it to be in an easy-to-access location; I instruct Claude to put all of those in the .claude/output directory, which is similarly ephemeral and untracked. I'm not sure what Claude would generate given those two strings of instructions, but it would be nice if we had some way of managing "if the user has a place they want it, use that; otherwise put it here" sorts of instructions
There was a problem hiding this comment.
As far as I see .claude/output is not included in .gitignore so currently it's being tracked.
I think we had a discussion to leave output tracked in case we wanted to share requirements/plans in the repo.
We can revisit that discussion or create a .claude/tmp folder to put these outputs. What do you think?
There was a problem hiding this comment.
Ah, right, I have that untracked in my global gitignore 😅
This feels like the sort of thing @bitwarden/team-ai-sme should weigh in on, and see if we can get some sort of consistent standard across all the Bitwarden repos 🤔
But I very much do want there to be some .claude/[something] folder in each repo that's ephemeral and untracked, for this sort of thing. I'm just very ambivalent on what it is, so long as it's the standard.
🎟️ Tracking
PM-37811
📔 Objective
Adds an LLM command
audit-build-warningsto check build warnings, categorize them and try to fix them.