Skip to content

feat: bake pies in a round pie tin and cut them into slices - #111

Merged
JustinasLa merged 1 commit into
mainfrom
feat/pie-tray
Oct 31, 2026
Merged

JustinasLa merged 1 commit into
mainfrom
feat/pie-tray

Conversation

@JustinasLa

Copy link
Copy Markdown
Contributor

Summary

Pies, baked in a round pie tin (new pies.yml, new pie package):

  1. Right-click a placed Pie Tin with dough to line it, then with 1 to 3 fillings: fruit, meat, mashed potatoes, cut or chopped veg, cheese. An empty hand takes the last one back out.
  2. Sneak-carry the tin into a lit oven. A tin with only dough is refused. It cooks at bake.cook-seconds and burns at bake.burn-seconds, counting only while the oven has heat. A new filling restarts the count.
  3. An empty hand takes the whole pie out, Hot, even from inside the oven.
  4. On the cutting board the knife cuts a pie into 4 slices (cut_pie). New recipe option share-food gives each slice a quarter of the pie's food; slices keep its kind, name and tags.
  • Kinds (pies.yml kinds, SaladKind): Shepherd's, Meat, Apple, Key Lime, Mixed Berry, or named by its fillings. New kind rules: only-origins, min-origins and name.
  • The pie's value:
    • Food is the dough and fillings summed. Each filling's own cooking is left out (SandwichItems.applyTotals option); the pie's bake applies once.
    • 3 fillings make it Hearty. New composition context PIE.
  • Oven: it accepts and bakes pie tins as well as baking trays and pizza peels. A broken tin drops the baked pie whole, or what's in it.

The pie tin furniture, its model and the pie types/looks are server config (Dev only so far).

Documentation impact

  • Central Cooking documentation: no change. pies.yml documents its keys, and the README lists the feature.

Contract

  • Affected behaviour:
    • Clicking or breaking a pie tin.
    • The oven taking pie tins; its refusal message now names them.
    • Cutting pies on the board.
    • Trays and peels are unchanged.
  • Tests run: mvn -o -B clean verify; all tests pass and line coverage is still 100%. New PieCoverageTest.
  • Tested on Dev: deployed on Dev since 2026-10-31; not confirmed in game yet.
  • Player wiki (ProvinceSystem /wiki/cooking or related pages): no change.

🤖 Generated with Claude Code

A pie tin (pies.yml) is the bread tray's round, one-mould cousin: dough lines
it, up to three fillings go in under its top crust, and an empty hand takes
back the last one. Carried into a lit oven, a filled pie bakes (cooked, then
burnt) and an empty hand takes it out whole and hot. Its food is its dough
and fillings summed like a sandwich's, with the pie's bake in place of each
filling's own cooking; its kind (Apple, Key Lime, Mixed Berry, Meat,
Shepherd's, or named by its fillings) gives it its origin and name, and three
fillings make it Hearty. Ovens take empty tins and filled pies, not dough
alone. Kinds gain only-origins, min-origins and name rules, and crafting
recipes a share-food option so a pie's four slices hold a quarter each.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Oct 31, 2026

Copy link
Copy Markdown

Important

  • Trigger review

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

Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 9cc10abc-bc50-4963-ae53-364791685c09
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

@github-actions github-actions Bot added the size:XL 500-999 effective changed lines (tests excluded in mixed PRs). label Oct 31, 2026
@JustinasLa
JustinasLa merged commit 2d8cfaa into main Oct 31, 2026
3 checks passed
@JustinasLa
JustinasLa deleted the feat/pie-tray branch October 31, 2026 18:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XL 500-999 effective changed lines (tests excluded in mixed PRs).

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant