[mforce]

Why

specs/product/specs.md §10.5 presents the canonical sales_order_items schema, and two entries in it were wrong. Both were found while building #720 and deliberately left out of that slice to keep it scoped — it was already carrying a migration, a UI change and three locales.

What changed

Removed line_total_cents. There is no such column. LineTotal is a computed property on the entity and is explicitly ignored by EF Core:

// LineTotal is computed — ignored by EF Core
builder.Ignore(i => i.LineTotal);

A short paragraph now says so under the block, so the column does not get re-added by someone reconciling the spec against a query result.

Added egg_grade_id. It is real, required and FK-constrained — builder.Property(i => i.EggGradeId).IsRequired(), a foreign key to EggGrades with ON DELETE RESTRICT, and IX_SalesOrderItems_EggGradeId. It is placed where the generated schema puts it, after product_type_snapshot, and annotated as present on every line rather than only on egg products, since Guid EggGradeId is non-nullable for all product types.

Verification

Both claims checked at both layers before editing, not from the issue text:

• docs/schema/public.SalesOrderItems.md (generated and CI-checked) lists 16 columns. EggGradeId is there, Nullable = false. No LineTotal appears anywhere in it.
• SalesOrderConfiguration.cs:117-118 for the ignore, :65 and :71 for the requirement and the FK.

Scope

Documentation only, one file. The issue's third item — the §10.2/§10.4 discount-vocabulary collision — is not fixed here. It needs a vocabulary decision rather than a correction, and #737 itself says to handle it alongside #730, which must touch the same prose and would otherwise write over it. That has been recorded on #730.

Note this is not a documentation-only PR by CI's definition: specs/ is classified as code on purpose (#782), because web/src/routes/helpGlossary.test.ts reads from that tree. No test reads specs.md itself.

Closes #737

[coderabbitai]

Important

• 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: 2d6741e6-530f-419a-9e4f-342b352985f7

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}