irulan · models by kyoragithub

irulan

first model:irulan-swerewrites text to talk like an swein training

pull request

llm151 words

open#412aiagent wants to merge into main

✨ Refactor authentication module for improved maintainability, robustness and scalability

## 📝 Summary
This pull request introduces a comprehensive, holistic refactor of the authentication module, aimed at significantly improving maintainability, readability, scalability, and overall robustness across the codebase. 🚀

## ✨ Key Changes
- 🔧 Refactored the `AuthService` class to align with industry best practices and modern design patterns
- 🛡️ Introduced robust, comprehensive error handling to ensure a seamless and delightful user experience
- 🧪 Enhanced unit tests to ensure comprehensive coverage of all critical paths and edge cases
- 📚 Improved inline documentation for better developer onboarding

## 🎯 Impact
These changes should meaningfully elevate the developer experience, reduce technical debt, and lay a solid, future-proof foundation for upcoming improvements.

## ✅ Checklist
- [x] Code follows the style guidelines of this project
- [x] I have performed a self-review of my own code
- [x] I have added tests that prove my fix is effective

Please let me know if you have any questions or feedback! Happy to iterate. 🙌

swe57 words

open#412eeng wants to merge into main

auth: retry token refresh on 401

## summary
expired access tokens 401'd mid-session and signed people out. refresh now retries once before giving up.

## changes
- `AuthService.refresh` retries once on 401, then signs out
- a 401 during refresh no longer clears the session cookie
- tests for expired + revoked tokens

## impact
no more random logouts after ~1h idle

fixes #409

issue

llm69 words

open#418aiagent opened this issue

Dashboard occasionally fails to load under certain circumstances

**Description**
I've noticed that there appears to be an issue where the dashboard may occasionally fail to load under certain circumstances. This could potentially have a negative impact on the user experience.

**Steps to Reproduce**
1. Navigate to the dashboard
2. Observe the behavior

**Expected Behavior**
The dashboard should load successfully.

**Additional Context**
Any help with this would be greatly appreciated!

swe48 words

open#418eeng opened this issue

platform dashboard 500s for an org with 0 machines

**description**
/dashboard/platform 500s for any org with 0 machines

**repro**
1. create a new org
2. open /dashboard/platform → 500

**expected**
empty state ("no machines yet")

**context**
TypeError: cannot read 'region' of undefined (insights/rows.ts:88). rows.ts assumes at least one machine

review comment

llm55 words

src/billing/charges.ts + const tax = Math.round(total * rate * 100) / 100 + return { total, tax, due: total + tax }

aiagent commented on line 42

Great work on this PR! 🎉 I really appreciate the effort you've put into it. One small suggestion: it might be worth considering whether we could potentially extract this logic into a separate helper function, as this could improve readability and make the code more maintainable going forward. What do you think? Happy to discuss further!

swe13 words

src/billing/charges.ts + const tax = Math.round(total * rate * 100) / 100 + return { total, tax, due: total + tax }

eeng commented on line 42

nit: pull this into a helper, billing/usage.ts has the same logic

otherwise lgtm