ECHOES-1377 Add icons for Architecture 'system components'

[jeremy-davis-sonarsource]

---

Summary by Gitar

• New components:
  • Added IconComputer, IconDataTable, and IconDatabase to the icon library.
  • Exported the new icon components via the index.ts barrel file.

_{This will update automatically on new commits.}

[netlify]

✅ Deploy Preview for echoes-react ready!

Name	Link
🔨 Latest commit	20a3e21
🔍 Latest deploy log	https://app.netlify.com/projects/echoes-react/deploys/6a314e75adda8800084e966c
😎 Deploy Preview	https://deploy-preview-706--echoes-react.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.
🤖 Make changes	Run an agent on this branch

---

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

[hashicorp-vault-sonar-prod]

ECHOES-1377

[gitar-bot]

Code Review ✅ Approved 1 resolved / 1 findings

Adds new IconComputer, IconDataTable, and IconDatabase components to the icon library. Please replace the deprecated forwardRef usage in these components with the updated React ref forwarding pattern.

✅ 1 resolved

✅ Quality: New icon components use deprecated forwardRef

📄 src/components/icons/IconComputer.tsx:21 📄 src/components/icons/IconComputer.tsx:24-31 📄 src/components/icons/IconDataTable.tsx 📄 src/components/icons/IconDatabase.tsx
The three new icon components (IconComputer, IconDataTable, IconDatabase) are introduced using forwardRef from React. Per the project's component conventions, forwardRef is deprecated in React 19 (ref is now a regular prop) and should not be used in newly added components — ref should be accepted directly in the props destructuring, e.g. function IconComputer({ ref, ...props }: IconProps).

Note: these new components faithfully follow the established convention used by every existing icon in src/components/icons/ (IconClock, IconComment, IconDashboard, etc.) and the shared IconMaterialWrapper/IconBase wrappers, which all still rely on forwardRef. Because the entire icon family is consistent, this is flagged as a minor migration candidate rather than a blocker — ideally the whole icon set (including the wrapper) migrates away from forwardRef together so the pattern stays uniform.

Suggested fix (applies to each new file):

Options

Auto-apply is off → Gitar will not commit updates to this branch.
Display: compact → Showing less information.

Comment with these commands to change:

Auto-apply	Compact
gitar auto-apply:on	gitar display:verbose

_{Was this helpful? React with 👍 / 👎 | Gitar}

[sonarqube-next]

Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
0 Dependency risks
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarQube