A possible way to improve the stack deep merge by ...
# atmos-devr
RB
03/15/2026, 3:43 AMA possible way to improve the stack deep merge by 3.5 times
https://github.com/cloudposse/atmos/pull/2201/changes
👀 1
✅ 1
merged 1
e
Erik Osterman (Cloud Posse)
03/15/2026, 4:30 AM@Andriy Knysh (Cloud Posse) would need to review this one
r
RB
03/23/2026, 5:27 AMFriendly ping here. I iterated on this with coderabbit and copilot to improve these metrics
Copy code
| Score | Number | Grade | Delta vs previous |
|---|---|---|---|
| Merge Safety | 96/100 | A | 0 |
| Test Coverage | 78/100 | B- | +1 |
| Code Quality | 93/100 | A- | 0 |
| Security Posture | 92/100 | A- | 0 |e
Erik Osterman (Cloud Posse)
03/23/2026, 3:20 PM@Andriy Knysh (Cloud Posse) LGTM
👀 1
Erik Osterman (Cloud Posse)
03/23/2026, 3:21 PMJust the usual trepidation changing core functionality. 😃
r
RB
03/23/2026, 3:58 PMIf you’d like to wait and do more tests, please feel free. I think I exhausted both copilot and coderabbit here lol but im sure there is room for improvement, given that the scores are not all 100 out of 100.
RB
03/23/2026, 3:58 PMThank you for considering.
e
Erik Osterman (Cloud Posse)
03/23/2026, 4:04 PM@Andriy Knysh (Cloud Posse) has the most experience with this area of the code, so I will defer to his expertise
🙌 1
Erik Osterman (Cloud Posse)
03/23/2026, 4:05 PMLong as you know, we can patch quickly if there’s any issues that’s all
💪 1
a
Andriy Knysh (Cloud Posse)
03/23/2026, 4:10 PMi’m going to review it, thanks for the contribution RB
🤘 1
Andriy Knysh (Cloud Posse)
03/24/2026, 3:04 PM@RB i improved your PR and merged to main. It will be in the next release.
Thank you for the contribution, it should speed up atmos stack processing.
I hope you don't mind me changing your PRs (I have added your name to all your PRs in the last release instead of the AI)
r
RB
03/24/2026, 3:31 PMOf course, I do not mind at all. Thank you very much for considering the change and improving it 🙂