Skip to content

context.modify_hand_chips#1314

Draft
stupxd wants to merge 2 commits intoSteamodded:mainfrom
stupxd:main
Draft

context.modify_hand_chips#1314
stupxd wants to merge 2 commits intoSteamodded:mainfrom
stupxd:main

Conversation

@stupxd
Copy link
Copy Markdown
Collaborator

@stupxd stupxd commented Apr 6, 2026

Yayy new context.

This one was added to make implementing hand score cap / modifiers easier (your "score cannot exceed 50% blind size" effects and all that).

What I still need to add:

  • Handle effects returned by SMODS.calculate_context(), current impl is very rushed.
  • Supply final hand score with context.after, or temporarily store in a global variable like SMODS.last_hand_oneshot (that one doesn't "reset", but this one should)

My question for people who would want to use this -

What kind of effects would you even need?

Is current impl (score is stored in context table and can be modified by anything) "good enough"
or do you want something like

if context.modify_hand_chips and <your conditions> then
  -- Set score to 1234
  return {
    set_chips = 1234,
  }
end

And beyond that, there's also a question of how should this reflect on flames with SMODS.calculate_context(true) -
should the flames function use hand chips from global variable during hand animation, or is it "fine" if it doesn't?

Additional Info:

  • I didn't modify api's or I've made a PR to the wiki repo.
  • I didn't modify api's or I've updated lsp definitions.
  • I didn't make new lovely files or all new lovely files have appropriate priority.

@stupxd stupxd changed the title context.modify_hand_chips and slight scoring calc optimization context.modify_hand_chips Apr 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant