From 00e7ae4604b746dab3d0ced077bc4473a157dbe8 Mon Sep 17 00:00:00 2001
From: Cristian Magherusan-Stanciu <cristi@leanercloud.com>
Date: Fri, 24 Apr 2026 23:01:33 +0200
Subject: [PATCH] ux(settings): clarify Savings Plans card covers SageMaker +
 Lambda (#22)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

The "Compute Savings Plans" card in Settings → Purchasing actually
governs defaults for all four SavingsPlansType values fetched by
providers/aws/recommendations/parser_sp.go: ComputeSp (covers EC2,
Fargate, Lambda), Ec2InstanceSp, SagemakerSp, and DatabaseSp. Users
reported SageMaker and Lambda looked unrepresented in settings.

Rename the card header to "Savings Plans" and add a small hint line
listing the covered services, so the coverage is self-documenting.
No extra cards are added: a Lambda-specific card would be misleading
(Lambda SPs do not exist as a standalone product — Lambda usage is
covered by Compute SP), and SageMaker SPs share the same term/payment
knobs as the other types. The underlying aws-savingsplans-term and
aws-savingsplans-payment inputs keep the same IDs so settings wiring
and the existing save/load paths are unchanged.
---
 frontend/src/index.html          | 3 ++-
 frontend/src/styles/settings.css | 7 +++++++
 2 files changed, 9 insertions(+), 1 deletion(-)

diff --git a/frontend/src/index.html b/frontend/src/index.html
index 3ac89e98..edb424a1 100644
--- a/frontend/src/index.html
+++ b/frontend/src/index.html
@@ -484,7 +484,8 @@ <h5>Redshift Reserved Nodes</h5>
                                 <label>Payment: <select id="aws-redshift-payment"><option value="no-upfront">No Upfront</option><option value="partial-upfront">Partial</option><option value="all-upfront" selected>All Upfront</option></select></label>
                             </div>
                             <div class="service-default-card">
-                                <h5>Compute Savings Plans</h5>
+                                <h5>Savings Plans</h5>
+                                <p class="service-default-hint">Compute (EC2, Fargate, Lambda), EC2 Instance, SageMaker, and Database (RDS) plans share these defaults.</p>
                                 <label>Term: <select id="aws-savingsplans-term"><option value="1">1 Year</option><option value="3" selected>3 Years</option></select></label>
                                 <label>Payment: <select id="aws-savingsplans-payment"><option value="no-upfront">No Upfront</option><option value="partial-upfront">Partial</option><option value="all-upfront" selected>All Upfront</option></select></label>
                             </div>
diff --git a/frontend/src/styles/settings.css b/frontend/src/styles/settings.css
index 6850d6bd..8f1794e8 100644
--- a/frontend/src/styles/settings.css
+++ b/frontend/src/styles/settings.css
@@ -198,6 +198,13 @@
     color: #333;
 }
 
+.service-default-card .service-default-hint {
+    margin: -0.5rem 0 0.75rem;
+    font-size: 0.75rem;
+    line-height: 1.3;
+    color: #666;
+}
+
 .service-default-card label {
     display: flex;
     align-items: center;