[Bug] GET /api/artstudio 500s for any user with pre-#353 asset documents — "Document element is missing for required non-nullable property 'GalleryStatus'" #369

New Issue

2026-07-05T00:10:44Z

spikerj commented

2026-07-05 00:10:44 +00:00

Found by the #359 Angular-in-the-loop validation on the Mac stack (2026-07-04).

Symptom

GET /api/artstudio?page=1&pageSize=20 returns 500 and the student sees a permanent "Failed to list art assets" toast + an empty My Assets grid — for every asset, forever:

{"error":"Failed to list art assets","details":"Document element is missing for required non-nullable property 'GalleryStatus'."}

After backfilling galleryStatus the next missing field fails the same way:

{"error":"Failed to list art assets","details":"Document element is missing for required non-nullable property 'FlaggedForReview'."}

Root cause

The list path reads art-assets through the EF Core Mongo mapping, which requires every non-nullable property to be present in the document. Documents created before the #349/#352 (flaggedForReview, quarantinedArtifacts) and #353 (galleryStatus) merges lack those elements, so a single legacy document poisons the whole list query. (The Bson-attribute model SpikerSoft.Data/Mongos/ArtStudio/ArtAsset.cs has C# defaults, but EF's required-property check fires before defaults help.)

On the Mac dev DB, 7 of 13 documents lacked galleryStatus and 3 lacked flaggedForReview/quarantinedArtifacts. Production is exposed the same way: any asset created before those deploys breaks its owner's My Assets list.

Local unblock used (documented in docs/mac-local-stack.md §6)

docker exec mongo-router mongosh spikersoft --quiet --eval \
  'db.getCollection("art-assets").updateMany({galleryStatus:{$exists:false}},{$set:{galleryStatus:"None"}});
   db.getCollection("art-assets").updateMany({flaggedForReview:{$exists:false}},{$set:{flaggedForReview:false}});
   db.getCollection("art-assets").updateMany({quarantinedArtifacts:{$exists:false}},{$set:{quarantinedArtifacts:[]}})'

Proper fix options

Make the EF mapping tolerate missing elements (nullable + coalesce, or configure defaults), and/or
A one-time migration that backfills the three fields, and
A convention/test so future required fields added to ArtAsset ship with a migration (this has now happened twice in one week).

Part of epic #346; found during #359.

Found by the #359 Angular-in-the-loop validation on the Mac stack (2026-07-04). ## Symptom `GET /api/artstudio?page=1&pageSize=20` returns **500** and the student sees a permanent "Failed to list art assets" toast + an empty My Assets grid — for every asset, forever: ``` {"error":"Failed to list art assets","details":"Document element is missing for required non-nullable property 'GalleryStatus'."} ``` After backfilling `galleryStatus` the next missing field fails the same way: ``` {"error":"Failed to list art assets","details":"Document element is missing for required non-nullable property 'FlaggedForReview'."} ``` ## Root cause The list path reads `art-assets` through the EF Core Mongo mapping, which requires every non-nullable property to be present in the document. Documents created **before** the #349/#352 (`flaggedForReview`, `quarantinedArtifacts`) and #353 (`galleryStatus`) merges lack those elements, so a single legacy document poisons the whole list query. (The Bson-attribute model `SpikerSoft.Data/Mongos/ArtStudio/ArtAsset.cs` has C# defaults, but EF's required-property check fires before defaults help.) On the Mac dev DB, 7 of 13 documents lacked `galleryStatus` and 3 lacked `flaggedForReview`/`quarantinedArtifacts`. **Production is exposed the same way**: any asset created before those deploys breaks its owner's My Assets list. ## Local unblock used (documented in docs/mac-local-stack.md §6) ```bash docker exec mongo-router mongosh spikersoft --quiet --eval \ 'db.getCollection("art-assets").updateMany({galleryStatus:{$exists:false}},{$set:{galleryStatus:"None"}}); db.getCollection("art-assets").updateMany({flaggedForReview:{$exists:false}},{$set:{flaggedForReview:false}}); db.getCollection("art-assets").updateMany({quarantinedArtifacts:{$exists:false}},{$set:{quarantinedArtifacts:[]}})' ``` ## Proper fix options 1. Make the EF mapping tolerate missing elements (nullable + coalesce, or configure defaults), and/or 2. A one-time migration that backfills the three fields, and 3. A convention/test so future required fields added to `ArtAsset` ship with a migration (this has now happened twice in one week). Part of epic #346; found during #359.

spikerj referenced this issue

2026-07-05 00:11:29 +00:00

[ArtStudio M4] Entire stack on the MacBook — native ArtPipeProcessor (no GPU in Docker on macOS), compose.mac, dev-mode GpuCoordinator #359

spikerj commented

2026-07-05 00:49:28 +00:00

Resolved in spikersoft-backend PR #75 (merged to master). Root cause: MongoDB.EntityFrameworkCore throws on any missing non-nullable mapped member (and rejects [BsonDefaultValue]), so the post-P1 FlaggedForReview/GalleryStatus fields 500'd list queries containing legacy docs. Fixed schema-tolerantly (nullable backing fields + normalizing accessors; docs heal on next write; API JSON unchanged) with 11 regression tests driving era-exact BSON through the provider's real pipeline, plus a model-guard test forbidding future post-P1 non-nullables on these entities. Optional prod backfill one-liner is in the PR/report if wanted. Closing.

Resolved in spikersoft-backend PR #75 (merged to `master`). Root cause: MongoDB.EntityFrameworkCore throws on any missing non-nullable mapped member (and rejects `[BsonDefaultValue]`), so the post-P1 `FlaggedForReview`/`GalleryStatus` fields 500'd list queries containing legacy docs. Fixed schema-tolerantly (nullable backing fields + normalizing accessors; docs heal on next write; API JSON unchanged) with 11 regression tests driving era-exact BSON through the provider's real pipeline, plus a model-guard test forbidding future post-P1 non-nullables on these entities. Optional prod backfill one-liner is in the PR/report if wanted. Closing.

spikerj closed this issue

2026-07-05 00:49:28 +00:00

spikerj referenced this issue

2026-07-05 01:59:21 +00:00

[Epic] ProArt 3D Art Studio — student asset pipeline (concept → model → texture → rig → animate → game-ready GLB) #346

spikerj referenced this issue

2026-07-05 07:43:41 +00:00

[ArtStudio] ProArt capability parity — user/predefined workflows, multi-method stages, full PBR map sets, rig-method chain, curation (epic #346 phase 2) #378

spikerj referenced this issue

2026-07-16 01:37:13 +00:00

[Bug][Backend][Quiz] GET /api/Quiz 500s — Quiz.DegradedPageNumbers (#559) & UnvalidatedQuestionDropCount (#564) are required non-nullable, so pre-deploy quizzes fail to materialize #606

spikerj referenced this issue