chore: replace local isSnapId definition with isSnapId from @metamask/snaps-utils

[hmalik88]

Description

Write a short description of the changes included in this pull request, also include relevant motivation and context. Have in mind the following questions:

1. What is the reason for the change? Deduplication of code
2. What is the improvement/solution? Using the isSnapId function from the @metamask/snaps-utils package.

Related issues

Fixes: #29280

Pre-merge author checklist

• I've followed MetaMask Contributor Docs and MetaMask Extension Coding Standards.
• I've completed the PR template to the best of my ability
• I’ve included tests if applicable
• I’ve documented my code using JSDoc format if applicable
• I’ve applied the right labels on the PR (see labeling guidelines). Not required for external contributors.

Pre-merge reviewer checklist

• I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
• I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.

[github-actions]

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

[metamaskbot]

Builds ready [1dd71a1]

• builds: chrome, firefox
• builds (beta): chrome
• builds (flask): chrome, firefox
• builds (MMI): chrome, firefox
• builds (test): chrome, firefox
• builds (test-flask): chrome, firefox
• build viz: Build System
• mv3: Bundle Size Stats
• mv2: E2e Actions Stats
• code coverage: Report
• storybook: Storybook
• typescript migration: Dashboard
• all artifacts
• bundle viz:

  • background: 0, 1, 2, 3, 4, 5, 6, 7
  • common: 0, 1, 10, 11, 12, 2, 3, 4, 5, 6, 7, 8, 9
  • content-script: 0
  • offscreen: 0
  • ui: 0, 1, 10, 11, 12, 13, 2, 3, 4, 5, 6, 7, 8, 9

Page Load Metrics (1863 ± 291 ms)

Platform	Page	Metric	Min (ms)	Max (ms)	Average (ms)	StandardDeviation (ms)	MarginOfError (ms)
Chrome	Home	firstPaint	349	3098	1667	578	277
		domContentLoaded	1402	3607	1839	603	289
		load	1407	3633	1863	606	291
		domInteractive	22	231	57	50	24
		backgroundConnect	8	60	23	15	7
		firstReactRender	15	101	62	30	15
		getState	5	79	23	26	12
		initialActions	0	1	0	0	0
		loadScripts	1028	2949	1397	515	247
		setupStore	6	74	15	18	8
		uiStartup	1676	4093	2250	651	313

[metamaskbot]

Builds ready [7baee13]

• builds: chrome, firefox
• builds (beta): chrome
• builds (flask): chrome, firefox
• builds (MMI): chrome, firefox
• builds (test): chrome, firefox
• builds (test-flask): chrome, firefox
• build viz: Build System
• mv3: Bundle Size Stats
• mv2: E2e Actions Stats
• code coverage: Report
• storybook: Storybook
• typescript migration: Dashboard
• all artifacts
• bundle viz:

  • background: 0, 1, 2, 3, 4, 5, 6, 7
  • common: 0, 1, 10, 11, 12, 2, 3, 4, 5, 6, 7, 8, 9
  • content-script: 0
  • offscreen: 0
  • ui: 0, 1, 10, 11, 12, 13, 2, 3, 4, 5, 6, 7, 8, 9

Page Load Metrics (1700 ± 84 ms)

Platform	Page	Metric	Min (ms)	Max (ms)	Average (ms)	StandardDeviation (ms)	MarginOfError (ms)
Chrome	Home	firstPaint	284	2076	1494	504	242
		domContentLoaded	1425	2034	1676	173	83
		load	1435	2039	1700	175	84
		domInteractive	22	115	45	28	13
		backgroundConnect	9	94	24	23	11
		firstReactRender	15	93	36	25	12
		getState	5	100	25	24	12
		initialActions	0	1	0	0	0
		loadScripts	1014	1545	1246	145	70
		setupStore	6	60	13	14	7
		uiStartup	1616	2644	1975	257	123