🚀 feat(inspect): Geti Inspect

[ashwinvaidya17]

📝 Description

• Merge feature branch into main

✨ Changes

Select what type of change your PR is:

• 🚀 New feature (non-breaking change which adds functionality)
• 🐞 Bug fix (non-breaking change which fixes an issue)
• 🔄 Refactor (non-breaking change which refactors the code base)
• ⚡ Performance improvements
• 🎨 Style changes (code style/formatting)
• 🧪 Tests (adding/modifying tests)
• 📚 Documentation update
• 📦 Build system changes
• 🚧 CI/CD configuration
• 🔧 Chore (general maintenance)
• 🔒 Security update
• 💥 Breaking change (fix or feature that would cause existing functionality to not work as expected)

✅ Checklist

Before you submit your pull request, please make sure you have completed the following steps:

• 📚 I have made the necessary updates to the documentation (if applicable).
• 🧪 I have written tests that support my changes and prove that my fix is effective or my feature works (if applicable).
• 🏷️ My PR title follows conventional commit format.

For more information about code review checklists, see the Code Review Checklist.

[Copilot]

Pull request overview

Copilot reviewed 229 out of 448 changed files in this pull request and generated 1 comment.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

[nitpick] CSS custom property value should be converted to a string. While JavaScript may coerce the number, explicit conversion with .toString() or template literal is more reliable and matches CSS custom property best practices.

Suggested change

	'--zoom-scale': zoom.scale,
	'--zoom-scale': `${zoom.scale}`,

[Copilot]

Pull request overview

Copilot reviewed 229 out of 448 changed files in this pull request and generated 1 comment.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

The fire-and-forget task creation could lead to unhandled exceptions being silently ignored. Consider storing task exceptions and logging them. For example: task = asyncio.create_task(training_service.train_pending_job()); task.add_done_callback(lambda t: logger.error(f'Task failed: {t.exception()}') if t.exception() else None)

Suggested change

	running_tasks.add(asyncio.create_task(training_service.train_pending_job()))
	task = asyncio.create_task(training_service.train_pending_job())
	task.add_done_callback(
	lambda t: logger.error(f"Task failed: {t.exception()}", exc_info=True) if t.exception() else None
	)
	running_tasks.add(task)

[Copilot]

Pull request overview

Copilot reviewed 231 out of 454 changed files in this pull request and generated no new comments.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Pull request overview

Copilot reviewed 230 out of 454 changed files in this pull request and generated no new comments.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Pull request overview

Copilot reviewed 228 out of 454 changed files in this pull request and generated 1 comment.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

These TODO comments indicate incomplete functionality for zoom restrictions. Consider creating tickets to track this technical debt or implementing the scale and translate restrictions.

[Copilot]

Pull request overview

Copilot reviewed 228 out of 454 changed files in this pull request and generated 2 comments.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

The regex replacement replace(/}:/g, '}//:') modifying the OpenAPI spec appears to be a workaround. Add a comment explaining why this transformation is necessary for MSW to properly handle the spec, as this is not immediately obvious.

Suggested change

	const handlers = await fromOpenApi(JSON.stringify(spec).replace(/}:/g, '}//:'));
	// MSW's OpenAPI parser struggles with paths that have an operation suffix
	// immediately after a path parameter, e.g. `/api/models/{model_name}:activate`.
	// To work around this, we rewrite `}:` to `}//:` in the serialized spec so that
	// `@mswjs/source/open-api` can parse these routes, and then later escape the
	// colon at runtime in `post` via `path.replace('}:', '}\\:')`.
	const handlers = await fromOpenApi(
	JSON.stringify(spec).replace(/}:/g, '}//:')
	);

[Copilot]

The # type: ignore[override] comment suggests a type mismatch. Consider using proper type annotations that match the expected signature instead of suppressing the type checker.

Suggested change

	async def _save_file(filename: str, content: bytes) -> str: # type: ignore[override]
	async def _save_file(filename: str, content: bytes) -> str:

[Copilot]

Pull request overview

Copilot reviewed 228 out of 455 changed files in this pull request and generated 1 comment.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

The fetch request doesn't check for HTTP errors. Add response.ok validation and throw an error for failed requests to prevent silent failures.