Pull Request
indent
bot
commented 3 min ago
Issues
Potential issues found
decimal.Decimal amounts truncated to float in calculate_proration(). round (amount, 2) on a Decimal returns a float, silently losing precision. For a $1,247.635 proration this rounds to $1,247.64 as a float but subsequent multiplication by tax rate compounds the error. Use amount.quantize (Decimal('0.01')) instead.
Autofix issue →
Resolved (1 issue)
In-memory rate limiter won't work behind a load balancer — each worker tracks its own counters independently, making the effective limit 4× the configured value. (fixed by commit 2046dde)
CI checks
You have two failing Cl jobs. The first one will be resolved with a rebase, but the second one is real — we haven't updated the tests according to the new functionality. More details below.
Typecheck
•
Autofix issue →
pytest
•
Autofix issue →
Autofix All
Takes ownership
System rules
Always use design tokens for colors, never plain hexcodes
Always use semantic values in components
Never introduce spacing values that aren't in the design system
Ran 9 commands, Searched 5 patterns, Read 17 files
Posted 4 comments
Indent Code is addressing 3 comments
indent
bot
commented
indent
bot
left a comment
Author
Partial token adoption: The boxShadow property mixes token and hardcoded values. The first shadow layer correctly uses --shadow-color-sm, but the second layer contains a raw oklch value, bypassing the design system. ( src/components/Tooltip.tsx line 112)
Change proposed by
Indent Code
112
-
0 10px 24px -4px oklch(0.25 0.02 260 / 0.18)
112
-
0 10px 24px -4px var(--shadow-color-lg)
Accept change
Reject
Standards
DEPTH
Feedback
autofix
Works where you do
Your engineers can ask questions about code changes, discuss issues and potential solutions, and direct Indent to their preferred fixes right on the diff, without ever switching contexts.
jsegaran 1 minute ago
@indent can we re-use the exact tab list so if we update it in the future we don't forget to do it elsewhere?
👀
1
Indent
bot
1 minute ago
Done - now iterating over the REVIEW_TABS constant instead of hardcoding the triggers, matching the pattern in GithubRepoListContent .
❤️
1
Quick on the uptake
Indent follows your style guidelines, aligns with your existing patterns, and learns from your feedback to help you keep your work clean and consistent.
Indent
bot
3 weeks ago
Suggested change
−
throw new Error('User not found')
+
throw new AppError('User not found')
jsegaran 3 weeks ago
We wrap errors with AppError — keeps stack traces clean for Sentry.
💾
1
3 weeks later
Indent
bot
just now
Suggested change
−
throw new Error('Invoice not found')
+
throw new AppError('Invoice not found', {
+
code: 'INVOICE_NOT_FOUND',
+
statusCode: 404
+
})
Used AppError with structured metadata to match your error handling pattern.
Reliable and consistent
Indent does all the housekeeping, iterating on lint, merge conflicts, and minor test failures, so your engineers can focus on the important parts of a change.
Some checks were not successful
2 failing, 28 successful checks
1 failing check
typechck
Failed in 42s
pytest / unit
Failed in 1m 18s
28 passed checks
build / compile
Successful in 2m 14s
lint / eslint
Successful in 34s
lint / prettier
Successful in 12s
jest / unit
Successful in 1m 47s
integration / api
Successful in 3m 22s
fix: resolve type errors, update test snapshots
#2845
Changes
4 files
+42
19
main
←
indent/fix-type-errors
