refactor: extract hardcoded thresholds and magic numbers to named constants

[Copilot]

Auto-QA flagged raw numeric literals scattered across five files that would be difficult to tune or reason about at a glance. All numbers are replaced with named, commented constants at the top of their respective files — zero behaviour change.

📌 Fixes

---

📝 Summary of Changes

Five files had inline magic numbers replaced with named module-level constants:

File	Constants added
CardHistory.tsx	MS_PER_MINUTE, MS_PER_HOUR, MS_PER_DAY, MS_PER_WEEK
Tour.tsx	TOOLTIP_GAP (was inline 12), NEAR_TOP_THRESHOLD (was inline 100)
RunbookProgress.tsx	MS_PER_SECOND
GPUDetailModal.tsx	GPU_UTIL_HIGH = 90, GPU_UTIL_WARN = 70 — used in getUtilizationColor() and both progress-bar JSX expressions
ClusterGrid.tsx	MIN_SPIN_DURATION_MS (default param for useMinSpin)

AlertRuleEditor.tsx and AlertDetail.tsx already had their numbers in named constants; no changes needed there.

Example — GPUDetailModal.tsx before/after:

// Before — same threshold repeated three times
if (percentage >= 90) return 'text-red-400'
if (percentage >= 70) return 'text-yellow-400'

// After — single source of truth
const GPU_UTIL_HIGH = 90 // Utilization % threshold for critical (red) status
const GPU_UTIL_WARN = 70 // Utilization % threshold for warning (yellow) status

if (percentage >= GPU_UTIL_HIGH) return 'text-red-400'
if (percentage >= GPU_UTIL_WARN) return 'text-yellow-400'

---

Changes Made

• Extracted raw ms time-interval literals in CardHistory.tsx to MS_PER_* constants
• Promoted gap = 12 local var and inline 100 threshold in Tour.tsx to module-level constants
• Added MS_PER_SECOND to RunbookProgress.tsx for duration display threshold
• Extracted GPU utilization thresholds in GPUDetailModal.tsx to GPU_UTIL_HIGH/GPU_UTIL_WARN
• Replaced useMinSpin default parameter literal in ClusterGrid.tsx with MIN_SPIN_DURATION_MS

---

Checklist

• I used a coding agent (Claude Code, Copilot, Gemini, or Codex) to generate/review this code
• I have reviewed the project's contribution guidelines
• New cards target console-marketplace, not this repo
• isDemoData is wired correctly (cards show Demo badge when using demo data)
• I have written unit tests for the changes (if applicable)
• I have tested the changes locally and ensured they work as expected
• All commits are signed with DCO (git commit -s)

---

Screenshots or Logs (if applicable)

N/A — pure refactor, no visual changes.

---

👀 Reviewer Notes

Pure rename refactor — no logic changes. Each constant is co-located with the code that uses it (not moved to a central file) since they are domain-specific to their component.

[netlify]

✅ Deploy Preview for kubestellarconsole ready!

Name	Link
🔨 Latest commit	dd661c6
🔍 Latest deploy log	https://app.netlify.com/projects/kubestellarconsole/deploys/69cc26dba54cae00085f9c81
😎 Deploy Preview	https://deploy-preview-4024.console-deploy-preview.kubestellar.io
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify project configuration.

[github-actions]

👋 Hey @Copilot — thanks for opening this PR!

🤖 This project is developed exclusively using AI coding assistants.

Please do not attempt to code anything for this project manually.
All contributions should be authored using an AI coding tool such as:

• Claude Code (Opus 4.5 / 4.6) — recommended
• GitHub Copilot
• Cursor
• Other AI coding assistants

This ensures consistency in code style, architecture patterns, test coverage,
and commit quality across the entire codebase.

---

This is an automated message.

[github-actions]

❌ PR Title Verification Failed

Your PR title does not follow the required format.

Current title: refactor: extract hardcoded thresholds and magic numbers to named constants

Required Format

PR titles must start with one of these emoji prefixes:

Emoji	Meaning
⚠️	Breaking change
✨	Non-breaking feature
🐛	Patch fix / Bug fix
📖	Documentation
🚀	Release
🌱	Infra/Tests/Other

How to Fix

Edit your PR title to start with the appropriate emoji. For example:

• ✨ Add new feature for user authentication
• 🐛 Fix crash when loading empty config
• 📖 Update installation guide

You can edit the title by clicking the Edit button next to your PR title.

---

This comment was automatically posted by the PR Title Verifier workflow.

[Copilot]

Copilot wasn't able to review any files in this pull request.

[kubestellar-prow]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please ask for approval from clubanderson. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details

Needs approval from an approver in each of these files:

• OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[kubestellar-prow]

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.

📝 Please follow instructions in the contributing guide to update your commits with the DCO

Full details of the Developer Certificate of Origin can be found at developercertificate.org.

The list of commits missing DCO signoff:

• 6da9a5d Initial plan

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

[github-actions]

Thank you for your contribution! Your PR has been merged.

Check out what's new:

• KubeStellar Console — Live multi-cluster dashboard
• Marketplace — Community extensions
• Knowledge Base — Troubleshooting and how-tos

Stay connected: Slack #kubestellar-dev | Multi-Cluster Survey