feat: dashboard navigation, photo processing refactor, and UI fixes

[copyandpastecoder]

No description provided.

[Copilot]

Pull request overview

This PR implements dashboard navigation enhancements, refactors photo processing jobs to eliminate code duplication, and fixes several UI issues related to inventory cards and cycle expansion controls. It also adds support for preserving original uploaded photos alongside WebP variants.

• Dashboard stat cards are now clickable and navigate to relevant pages
• Photo processing jobs refactored using a generic base class, reducing ~460 lines of duplicate code
• UI fixes for inventory card text overflow and image error handling
• Cycles expand/collapse control restored after using "Expand All"

Reviewed changes

Copilot reviewed 30 out of 32 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
src/web/src/components/ui/stat-card.tsx	Added onClick prop and cursor pointer styling for clickable stat cards
src/web/src/components/ui/cycle-stats-chart.tsx	Removed unnecessary hsl() wrappers from CSS variable references
src/web/src/components/stage/stage-materials-section.tsx	Added max-height and scrolling to material selection dropdown
src/web/src/components/stage-form/index.tsx	Exported WATER_UNITS constant for reuse
src/web/src/components/cycle-card/types.ts	Added onExpandedChange callback prop
src/web/src/components/cycle-card/cycle-card.tsx	Implemented callback to reset expand state on manual toggle
src/web/src/app/(protected)/inventory/page.tsx	Fixed text overflow, added image error handling, increased padding
src/web/src/app/(protected)/dashboard/page.tsx	Added navigation onClick handlers to stat cards
src/web/src/app/(protected)/cycles/page.tsx	Added URL parameter handling for tab selection and expand state reset
src/web/src/app/(protected)/cycles/[id]/page.tsx	Added water amount input with unit conversion for next stage
src/web/package.json	Updated frontend dependencies to latest versions
src/api/MyUglyRocks.Infrastructure/Services/ImageProcessingService.cs	Added PreserveOriginalAsync method for phase 3 processing
src/api/MyUglyRocks.Infrastructure/MyUglyRocks.Infrastructure.csproj	Downgraded from non-existent .NET 10.0 to .NET 9.0
src/api/MyUglyRocks.Infrastructure/Migrations/*.cs	Added database migrations for original photo fields
src/api/MyUglyRocks.Infrastructure/Jobs/*.cs	Refactored photo processing jobs with generic base class
src/api/MyUglyRocks.Core/*.csproj	Corrected .NET version and dependencies
src/api/MyUglyRocks.Core/Entities/*.cs	Added original photo fields to Photo and InventoryPhoto entities
src/api/MyUglyRocks.Api/*.cs	Updated to three-phase photo processing and .NET 9.0
src/api/MyUglyRocks.Abstractions/Interfaces/IImageProcessingService.cs	Added OriginalPreservationResult record and PreserveOriginalAsync method

Files not reviewed (1)

• src/web/package-lock.json: Language not supported

---

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

[Copilot]

The duplicate migration files (20251231000000_AddOriginalPhotoFields.cs and 20251231203756_AddPhotoOriginalFields.cs) appear to have the same purpose but different timestamps. The earlier migration (20251231000000) adds slightly different column types for OriginalMimeType (character varying(50) vs text). This could cause migration conflicts if both are applied. Consider removing the duplicate migration or consolidating them into a single migration.

Suggested change

	// Add original photo fields to photos table
	migrationBuilder.AddColumn<string>(
	name: "OriginalStorageKey",
	table: "photos",
	type: "text",
	nullable: true);
	migrationBuilder.AddColumn<string>(
	name: "OriginalUrl",
	table: "photos",
	type: "text",
	nullable: true);
	migrationBuilder.AddColumn<string>(
	name: "OriginalMimeType",
	table: "photos",
	type: "character varying(50)",
	maxLength: 50,
	nullable: true);
	migrationBuilder.AddColumn<long>(
	name: "OriginalFileSizeBytes",
	table: "photos",
	type: "bigint",
	nullable: true);
	// Add original photo fields to inventory_photos table
	migrationBuilder.AddColumn<string>(
	name: "OriginalStorageKey",
	table: "inventory_photos",
	type: "text",
	nullable: true);
	migrationBuilder.AddColumn<string>(
	name: "OriginalUrl",
	table: "inventory_photos",
	type: "text",
	nullable: true);
	migrationBuilder.AddColumn<string>(
	name: "OriginalMimeType",
	table: "inventory_photos",
	type: "character varying(50)",
	maxLength: 50,
	nullable: true);
	migrationBuilder.AddColumn<long>(
	name: "OriginalFileSizeBytes",
	table: "inventory_photos",
	type: "bigint",
	nullable: true);
	}
	/// <inheritdoc />
	protected override void Down(MigrationBuilder migrationBuilder)
	{
	// Drop original photo fields from photos table
	migrationBuilder.DropColumn(
	name: "OriginalStorageKey",
	table: "photos");
	migrationBuilder.DropColumn(
	name: "OriginalUrl",
	table: "photos");
	migrationBuilder.DropColumn(
	name: "OriginalMimeType",
	table: "photos");
	migrationBuilder.DropColumn(
	name: "OriginalFileSizeBytes",
	table: "photos");
	// Drop original photo fields from inventory_photos table
	migrationBuilder.DropColumn(
	name: "OriginalStorageKey",
	table: "inventory_photos");
	migrationBuilder.DropColumn(
	name: "OriginalUrl",
	table: "inventory_photos");
	migrationBuilder.DropColumn(
	name: "OriginalMimeType",
	table: "inventory_photos");
	migrationBuilder.DropColumn(
	name: "OriginalFileSizeBytes",
	table: "inventory_photos");
	// Intentionally left empty.
	// A later migration (20251231203756_AddPhotoOriginalFields) is the
	// authoritative source for adding the original photo fields.
	}
	/// <inheritdoc />
	protected override void Down(MigrationBuilder migrationBuilder)
	{
	// Intentionally left empty.
	// The corresponding columns are managed by the later migration.