Repository navigation
docs: add a pdd-kit Product Model - #1332
Conversation
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
There was a problem hiding this comment.
Thanks, Itai. I checked the PR before merging:
- Every file is new and under
product-model/, and the repo's tracked-file guards pass. - All 433 code links resolve on
main. - A sample of about 45 capabilities, about 40 access rules and the main entity lifecycles match the code. No permission claim contradicts the endpoints.
Two claims need a fix before I merge:
capabilities/provision-farm/capability.mdsays the farm starts in UTC.provision-accounttakes--timezone,--localeand--currency(ProvisionAccountCliCommand.cs:13-18), so UTC is only the default when--timezoneis left out.business-rules/corrections-never-take-back-sold-eggs.mdandcapabilities/void-daily-entry/scenarios/refuse-voiding-a-day-with-sold-eggs.mdsay a void is refused if any eggs were sold. The guard compares produced with available (EggLot.cs:104-114), so eggs that were allocated or written off block a void too, not only sold ones.
One claim is right and the code is wrong. create-sales-order says an order date is never in the future, but only the web form enforces that. The API accepts a future date. I opened #1333 to fix the code, so please keep that part as it is.
I found a few smaller wording gaps too (the Locked state's depleted-flock exception, the Read-only role description, the CLI writing errors to stderr). I'll handle those myself after merge.
… Model Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
|
Thanks for checking it so carefully, @mforce. Both fixes are in 3d4e6fb:
I left the future-date line in I didn't rename any files, so the ids stay the same. It's a separate commit, so you can see just these changes. |
Closes #1302
This adds the Product Model proposed in #1302: plain Markdown under
product-model/. Only new files, no code, build or CI changes.product-model/README.mdis left out, as you asked.What's in it
Each file links to the code behind it. It was mapped from
mainat c50b939.Viewing it
The model was drafted with Claude Code using pdd-kit's
pdd-mapskill, then reviewed by me.🤖 Generated with Claude Code