improvement(mothership): whitespace only deltas need to be preserved, update docs for theshold billing#4212
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
PR SummaryMedium Risk Overview Fixes a Mothership chat rendering issue by no longer trimming Reviewed by Cursor Bugbot for commit e890972. Configure here. |
Greptile SummaryThis PR makes two independent fixes: it corrects the whitespace-trimming guard in Confidence Score: 5/5Safe to merge — both changes are targeted and low-risk. The whitespace fix is a one-line guard change with no behavioural side-effects beyond the intended preservation of whitespace deltas. The billing threshold update is applied consistently across the constant, env default, and docs, with no other call-sites left at the old value. No P0 or P1 findings. No files require special attention. Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[ContentBlock arrives\ntype = 'text'] --> B{block.content\nfalsily empty?}
B -- "null / undefined / ''" --> C[skip block\ncontinue]
B -- "non-empty\n(including whitespace-only)" --> D[process block]
D --> E{block.subagent set?}
E -- yes --> F[append to agent group]
E -- no --> G[append to top-level segments]
style C fill:#f5a5a5,stroke:#c0392b
style D fill:#a5d6a7,stroke:#2e7d32
Reviews (1): Last reviewed commit: "update overage threshold docs" | Re-trigger Greptile |
Summary
Type of Change
Testing
Tested manually
Checklist