Skip to content

feat(balance): high cook skill = more food fun#8539

Draft
scarf005 wants to merge 1 commit intocataclysmbn:mainfrom
scarf005:fix/cooking-enjoy-bonus
Draft

feat(balance): high cook skill = more food fun#8539
scarf005 wants to merge 1 commit intocataclysmbn:mainfrom
scarf005:fix/cooking-enjoy-bonus

Conversation

@scarf005
Copy link
Copy Markdown
Member

@scarf005 scarf005 commented Apr 16, 2026

Purpose of change (The Why)

image

idea by @jeremy7986 https://discord.com/channels/830879262763909202/830916451517857894/1494335607927869511

Improve cooked food morale so higher cooking skill gives better enjoyment and softens negative enjoyment.

Describe the solution (The How)

  • Store cooking skill on cooked and dehydrated craft results, then scale final fun by 10% per level with flooring.
  • Apply the same scaling to negative fun so bad cooked food is less unpleasant.
  • Fall back to the eater's current cooking skill for items that only carry the COOKED flag.

Testing

  • cmake --build --preset linux-full --target cataclysm-bn-tiles cata_test-tiles
  • ./out/build/linux-full/tests/cata_test-tiles "[fun_for][food][cooking]"

Checklist

Mandatory

  • I wrote the PR title in conventional commit format.
  • I ran the code formatter.
  • I linked any relevant issues using github keyword syntax.
  • I've committed my changes to a new branch that isn't main.

PR opened by gpt-5.4 high on opencode

@github-actions github-actions Bot added src changes related to source code. tests changes related to tests labels Apr 16, 2026
@scarf005 scarf005 changed the title fix(balance): scale cooked food enjoyment by skill feat(balance): high cook skill = more food fun Apr 16, 2026
Assisted-by: openai/gpt-5.4 on opencode
Co-authored-by: chatgpt-codex-connector[bot] <199175422+chatgpt-codex-connector[bot]@users.noreply.github.com>
@scarf005 scarf005 force-pushed the fix/cooking-enjoy-bonus branch from 202a22e to df361e0 Compare April 23, 2026 09:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

src changes related to source code. tests changes related to tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant