Domain Modeling

I added Customer and User to GLOSSARY.md, with "Account" on the avoid list for both. Order.account_id in src/orders.py:21 still uses the old word. Renaming it to customer_id would bring the code in line with the glossary, and I can do that if you want. I left the code alone for now because the rename may affect the database and any APIs that use the field.

One question on User: can one person log in for more than one Customer, for example a consultant who orders for two of your clients? If yes, then "User belongs to a Customer" is wrong, and the definition needs to say so.

"Cancelled": this needs a decision from you

The code and support are using one word for two different actions:

• Code (src/orders.py:25-28): cancelling applies to the whole Order, and it's refused if any line has shipped.
• Support: "cancel" also means "stop the lines that haven't shipped yet and refund them," on an Order that stays partly delivered.

Here's a scenario to test the options. An Order has 3 lines: line A has shipped, and lines B and C haven't. The Customer says, "Don't send the rest."

1. What happens to the Order? Is its status CANCELLED even though line A was delivered and paid for? Or does it count as complete (just A), with B and C individually cancelled? Today the code only has PLACED / SHIPPED / CANCELLED on the Order, and only a shipped flag on each line, so there's no way to record "B and C were cancelled."
2. Is the refund part of the cancellation, or a separate thing? Support refunds by hand today. Should cancelling unshipped lines automatically trigger a refund for them, or is a Refund its own concept with its own rules (for example, it could also be used for damaged goods)?
3. What if the Customer wants to cancel just line C and still get line B? If that should be allowed, the concept works per line, not as "the rest of the Order."

My recommendation: use two terms.

• Order Cancellation: stopping an entire Order before any line has shipped. This is what the code already does.
• Line Cancellation: stopping one or more unshipped lines while the rest of the Order goes ahead. A Refund for the cancelled lines follows from it.

Support would then say "cancel lines B and C," never "cancel the order," when something has already shipped. This also tells you what to change in the code: lines need their own cancelled state, and partial cancellation should no longer go through a manual refund.

Answer questions 1–3 (and the User question) and I'll add the cancellation terms to the glossary. If you go with per-line cancellation and automatic refunds, I'd also suggest a short ADR. Changing the Order/line state model is hard to undo, and someone reading the code later could reasonably wonder why Order.cancel() refuses partly shipped Orders when line cancellation exists.

Change recorded in the trial sandbox (git diff of GLOSSARY.md):

diff --git a/GLOSSARY.md b/GLOSSARY.md
index a218c96..a1a541a 100644
--- a/GLOSSARY.md
+++ b/GLOSSARY.md
@@ -7,3 +7,11 @@ Receives customer orders and tracks them until delivery.
 **Order**:
 A request from a Customer to buy one or more products, placed in a single checkout.
 _Avoid_: Purchase, basket
+
+**Customer**:
+The company that places Orders and pays us for them.
+_Avoid_: Account, client, company
+
+**User**:
+A person who logs in to act on behalf of a Customer.
+_Avoid_: Account, login, member