From 44b9ed1ab1f2fab5a20046112da22da548537146 Mon Sep 17 00:00:00 2001
From: Jarvis <jarvis@johnmaguire.me>
Date: Sat, 21 Feb 2026 04:14:49 +0000
Subject: [PATCH] Fix: property ownership now tracked on buy and auction

- Direct buy: set property_owner when buy confirmed via checkpoint
- Auction: set property_owner when 'It goes to' parsed
- Historical log replay still passes (1551/1553)
---
 __pycache__/monop_parser.cpython-310.pyc | Bin 23372 -> 23546 bytes
 cardinal-plugin/monop/monop_parser.py    |  15 +++++++++++++++
 monop_parser.py                          |  15 +++++++++++++++
 3 files changed, 30 insertions(+)

diff --git a/__pycache__/monop_parser.cpython-310.pyc b/__pycache__/monop_parser.cpython-310.pyc
index a3a2d8a5fb5932f83ebd8bda260ee48be54d1a33..38d2321b17cca2bef111b5b3870e9765f9528545 100644
GIT binary patch
delta 5779
zcma)AYiwM_6~1%#?!)W#bN$M$*LEI`oy19SUIYl>;E<4zKteEikgznFyw+xe#}1<H
zT-}13AC#t8+EPeS)HQ8UDWIygprT42lv1fLR6t052wG?(A<+sVgy;iGzd18|b8lkO
zCi2`lXTEdh%sFRf&g`$BVgLF*t36s%leF+J-GB1PE6;wnw!3TNQjjV94dQQXce7<l
zt6;f~bi~<lCLPWuY;jgP1=X^0gxgb0wD5>@@^<l$#BsoBFTPrmT_-(*<PkRpJ@Il>
zvMJ`qWnBDZJR#%dDqcIqCYY=Hbr`FH?xdTPN%3R!s?jD3Db;JU+-g|`s%F?0PmR~g
zgamJbQ4bi^AGXAkGU-@aVzuSjRzAgJZmigi>Fdh%Qx>nk%Mwo)b3vGoYV;?6nI?aE
zCX|ecw_zUs3Jq?GH^`J@MXX^a{=yq!z}rQeS;ZJe8>QzI(lWKi(*EITc&S=ClT5r1
zbDGBfjkcL6<;kWgHp#}+NWf1R9XZcaO1@PYT4X%!*2o&r2b4a1*UB2K9erLm%WAVc
zx0c*f>pKd=+ED-x$Qp8$fv>idAByvgQ3s5wW~Hjtx~L{#jV3~0N}8}-&&oI9W96HR
z_d#uoiGQ7}!}QOA)5_Dwuxjxf)>wyZKn5Ti>Sete8@4g>0LfRh93<~O7+Aet_v7RR
zwBZF&{EH1{ML!@A+8J#<XX*-N9s;2p{mTocdY!bO4NpbBtIYHcVcwBm1r&CRKNSpX
z(>|HBMM7HTSyL7>JB~LaTQWSo-NLR*AG3fc0$<6E2H614$ayj?BojlMl?~-8PdOB>
zMVGfK*P1?UI^woXv4hSY$8B`eZj;+Q-d@hhW^BV2*<vQ$s7x5|kc~1uVY{ueNw(%w
zvIT8}Z055NBwOs^G0>gxRabeo#7xY>zHsv19BIK#b9o)5#9Y0aLWVb!*gvtJc_t_H
zdLARAE@E^fVn@i>HcZwn8+qmgo3zKi5il@dghvxvh54aL{x;Jjwyo5Ll)gB#((xX(
z&Iz{8+FQTJc4xsSt%VxdNK1NC&ceFq-~@nQM!i>fAR@+?-jFdLD6a39ER;mXaGDmf
z_(lys>1HG{0XYy6bw*1t4;;~1S(4ir47cZ|gAsR|DlwlS(h;$Zg0;W|YXO3@usoZ$
z<Gbh)iz6@4cASb*bSW)W0A!kXp0M4l%o+n9^u3l<=buW5tP(-uWGQj7e&R&LT}Yg4
zzVjk3jfgJ<TpAhu?1dS1$a33d7Uy2ma0DKDT(-mPF2(U+@od@jh;<5=5=_Vz8e|KB
zeng--(kb^T?i3>=R%npzP;};^{+KHpAu~^=(RTCs<;AjHX*rb$EY@qm>Kcmiu+4$h
zww#whj=)C3#gRrHsQ$TRij1B$+szS;+G~^vFrN1iI){aHPC8?`DxFVOVTl(=HAf7Z
zT%bcXm?p^moZBIDvg4qQ>|LsQtvGh_mzZSh59t~8lkNQTg2CvcGDB^P%;Y_|oA;7+
zM|mR{4X`|u@5JGd$uBd?muuw^`aFr=D>Le7<qIm;2u?u87m}9yjm(OWLFM4W+)jyu
zC||9R9u*{uaIre#2~3S5g%mK}N9w+3)U6Dud%m(h;8$rpuII(1<wd&0HP<R!8j?E`
zGNTJev$tAysdO;YTo!_V7z}}{;9>OT0d!PUHnP4UFGODvLNC<1J$>rWpnOdTzV?5Z
za%Bj;*(~QO&7F)2j=)95z6(O^Eu%<Zt)0l1!0Aze`05tfN)$zVsV+Zw4`oT@f-D*E
zCwqwGA4E-cE|avBI+U!!S|9mskD%Y8gj#0KwMyDu<HLSVsGpg%s~GZ0>Ib!)t1hgl
zc**!;ov-b$r0q@8=B4;0W?kC0YXibDkqjlw>`)NqKut_UwNY}8(Go!|wcbp*A*Act
zur4_p$m=}bFWFrd6!nw3KC{LJ=d0t(fMdW75^ms=aB%NNv(lK@4{Fn7Y-}fXmX+Uh
z2+q%6uZ%-zK&$-U49N9?9RN5E2=H(#L<0ORp|zY0tz~Y=FY^qg-wmB)E;53OvCfdM
zZzxBx|53}RNX5j{A*J0UIBf*G14oTjxG`kR$1}*iDl<sl*ciSvad%*16}I>ExbWVS
z-Fzvso>>1!o-5&xW>9Jswg!xfi(JU4`G%qr59i-}*@=T3Ri>_gDvyh}=r*adEu<`T
zFSuc4Jy6y|%9@SrO(EF_DraT5@Gy0US9#y5lIdKr&?|c<<Kl$uCFk{M=K(==4vl2O
zPb=__tH+fZD@M$Ds){xJy3%`VyEY7%hmEN>2k*bQ_|qrZii8}Z>Lot*EcKToWO4Be
zz5-4hQZ){F;1(QaR(=PyJIxeBd<9`&<I6+XU4-&>)8|$L-UADf^^}kaQKhDu#5WX6
zRd!XB{zrnsgnE5LT$z;+apLYa+`S>4FoJY0*E*^G$AfC2uQVZgC8+#4`?b=3;#Lg3
zO)JG@14iM1nJ#afoDh5QbAZKHPAHRh0Cl_mmOrJQFdAL<_MIaN`+d<PbYR?WRC1sH
z8EndgxElw@0htM$4hJ>D`0ak4#l@VVH#Le*I`a?tI=&hlt<?XA{o%*xIf-W{iB%q?
z>V!y$XM=%&k1D7mzVuIxTp?u8Zwz-h#Qhz(9x4}<4pk@}65`JUQ}imL-}328iGJ6o
zR}=jYpS}!qmC#OA@|P2x@W-zpy56VP5Z%Vpo+_Ona3wLie85#i_xkkJMC)o#;RlFb
zt;hLVq+OM`hJrTY%Una4O+GzH^a;v}b;yb;aX;Swg%Q=E3t@zYt~EpWIfft(wdThe
zcM<k<7amq^`cb8Ruqj8RD&VTbeX><%P`pzQ4=yWzhfcJ~s!`#av7WHslx)1n*LxHA
ziR8FF%X0f<AKyU2UzdGK)EBsp1fKHgjYL1~)0>F?p-<yFg9$JAbdqSD1Il;RM88Vo
zo?>h!#%sO+eldfBGd{hA=)e2)4Me~1(>D_Rkxy?0ofP_QsPu0my2c;BiRflU>-u*)
zF>*fOW}@|7pcD)dz1SbWh3J(&y@Tj04SiIFgvW`uA;iPW2Zn7^ym6--<vXRvrfjhE
zS;fj)bQi(6f*rE>?njxsST07)Q|=O#ta1sBA2YE{juybZliy0*<CM;ORDbL^O>PxN
z&Gt-+FBx<0<a-JI70PIS8`VHbamoxwc%Fuz2p$z`jHgQ-E1ypahTlHLb~L3oRxT<v
zth#sl;Z?aP6QyUaDAw*$gF93s6k_h0b)}ItcTB&&W_g4yEj12w$7B1B9y@e+ytHbd
zFB%;i%^#V*eW04LFP28u_76ICiaD&7C0PuADVDS|tR5WubG44m>TT7F*-7Fh>=<hU
zCWik_krYdTuC?o@f4lY}I~4;FV;Sby$&aTGt-IW=?t$H+S~a=Slj}R!ThqT<|89)s
zN~gChVINGty=Bm5L(@yQ-orX_3bjNvo7GCF*{XUUq8Xlkaob^*?x?8yL*xj*TeO2L
zy|kq{isRk#ruT1ufwBJ5f}we_ZUvlE+CFrcJzhFJbees6`qQ_ZW^ACeYUiWuj?(XU
nzQC51zP4){`%dZXt{mG~s@uIjwoyrKF1=BoE){pb#uon<{qHcX

delta 5605
zcmai2d2C$88GrNMdhK0byFTJwU$NsOiJg;NBqYI!6Nd!Hxtz;67U(p`nl5<Op)H!n
zgSsIFiE7cRM+zZpR8=6MR*@@EP}))=sy6BYs#-)^Dj_Naf3yl!TB(Ji{^onL&+nNu
z0cCc+Z@%yMoilI7|9OS|`ej!2KxJjrz<(F#o;_Lq%7v<~wyg_6R^VSh{$>AMXBg5*
z8+J&B#GB(xhB%ur#hWseRwFA|+?-@$7Pn+5WoBL_P6(Xl%#ZW3?W|LjG}5Jo63Hf+
z9hPD7>Ufz97b3iBl1(tXl;+h~D+6xSj>w3(h*>$tNV-DJ5{4a>WuPiYP4VJ*l`NIf
zgfYQr1{pOUHN*=t5;9zk+YQr5RXCY;xL;j}O&UD*kRdK)k~2jzG0*4~GJ0jEgc}@>
zBgAqU$}#bKV8*joG1d^J(oi$WaxAM>e1Ya1CbrFpH6;G23}zUQ+vTzx0R0LMzq4gI
ze8-$uCk;egLD5lcG{dSJCuCb?IYp|a-DoqN+NFMtRO2Y9hMPk{SVPq)=~a|CL~Eqt
zG}dIE0Bo%;U!|;s`Sak^@%SmkEY2gqN)!_XL@}#mm0BA$vGOJE_+gGw^0L0@-MG(C
z7BJo*BIr|+ze_S0Z{g~4<sQnQ%JhN^IbNb&Q<n$^<4sKchGa0_+!P$EcXI+$A4sDx
zYtmqPv+!E<N&}DYaSZI3_$ec2ia{t8TccW5gR^ahQXMIUF(zcSn^2@y&FWQZ^_=z$
zac51kW1+(#J7&l28oPEpS!k5C*hO_xl}6c|>%a$Nyh+Ane8RNrWsR&)RmeJw&9as^
zph~7#!^5CkP?pxhYzb4$;0ZX<CJkhuomW#wv@0aYH1Jx|dK=-)(LLdY^Yt4z?F78G
zEYl)yX91`~#(456mNT<o^Ev2uLK!sDodF|PZeWC&ra}#1zT{#$6txd@yi3J7!8RF3
zVuww;0XeBpSIQX0^qOo?aWf->yo_DozJgHnM!GxT%pVlj%S)re$i4^s1b*KqzN5~w
zgiPpq{7u97`@j<aEyiTW{Pniu{)*c*#hTxs(w10E#hR;&H5YlCSD4M498BgvV{lX(
zW6QY`s;*l}6hOv#>r<wklu2^X5?QYlNtGw`LsDs=Z<5qENw04#F^s-Rrdn>2QA;={
zd?~fY4t~C+jvIEPOyb<D8MShjI3pY3H_WCh;BcF)dB!-0?+IK;rkiCQ$vjIk?J`v8
zQQS#Jk_b>sHUj6en-83S)V*FpbUx<D7?e6>9AhW%D)h=m1#_-!X0(~opv%5!XU*sz
zOZ*kRv_Qtu2WqZ*$#I*>8vC8zZkD*_rj0rQ-t%q}{{)e=<gDx`5x1d?^g>^&g+#mV
z1v+H?U4q_kvYTa-Y(8ef;373^z(!3iRu)wKmjr-zc+EERz+kzM*e(+^*2x4aUB~Cb
zIdJ(rMZ@t#ss(35BDGXE@@+0QWbPx)dt^eLntZ<gia4s(81EsNlUiw6z@bO{k=ZR0
z2T`h=B-LlN7oXLTcmk$zzi_#H0ioqI+VTL}WHCI*ui$Y<nFq{5f_aI);b7CjvNE7}
zIp9Vsj^<UeRds`wzC9p))n5Yp^U|xew4;v3kofo(;A?!~C8F#W;CBS1XWvX()qib3
zyhq2g&gJGx#70!EMD*RD#;VfmNuKdVNP3AtecchwN)%Olv3vdC-P9$PuS=-@M*2?A
z*eL?Y;z~8$O0Jq)ls-@L8zlKSHPlkQjf-u!!ISzvNj<OIuIPw|?x0is@Ecu4Ta~3m
z{7C1b--COd;9k>kOLSbWYnuYP9|lSqEJ7Qhyblyd-A3CqjD;)JW^Ho72No6`K`=QR
zl3Vyv$sRItVNngZu!w4>=Fddqw?U3UqMhVg{c=vK0;!%hdzX$hEH<Dsw&udwmDpNF
zYU^=)YE#>kb0ohN{eL$kw)u7dq;Woghp&R=;|~Tx`Eno>r+9AB4aT0M7u|b*3;BzV
zpo>;ZAlcgkp!oo32kAYn^*V|dR@&4<0ljyf8z@f61iCkz#V;A|@EKN#e#>3QZwFtD
zt|!hd`doF0(*4s6Ox^!G0$zO*aH>;NZiZT9r)<GVPM2?ec&VpFsjzhC?DS!k2EPk-
z5UWdeI<kwvYP9mMfb!1bVR_Swhq^IM-xa!LH=G%g-ITj7U+#?bZWkTv2emzCPrzW|
z{~1JE4bv5*H2W~kmm%?`;z_<7nHW|V6$)c7K0ZcjAC3F9iDAB+WS`^90<s5`l#@D0
z!XsMv5ZpoU(*7?MSClE3xe_Rdx+@exe<ne}pnC!gbJwk361N!R;Q;1!jd`6go#L}-
zo)7dR2`V-1zYF>g;uak};)3SN1S3HFYA24m8O2H~-s1*eF=1kkSko!@w{VYoh`7>D
z;ofN?eV?b=L%TkuA-~}9??4xqiPhNsTqb;b{EMyx<*iH%@<x1K8|Yn(t1WH3V;;sK
z->OqwKJG0ar$;4TD5O?=R;mNJOg!u_1aefAI^k(gXuXpGhrX$~=>Yfa^!Go4EmS5Z
zNle`bRucW9N3SCKf=Ay@^hJ+eP4v$_dJWOP^5{E=e#@iR68#R|zj_a^BgT84z<Q!T
z^5{E>{@A1Yi2k=r^9|_m2u=WXRF8Tx8zB=B(|4tQqKBv}HlZsbVo-hmN7aN*^-)^d
z>@H!YTCL!ztVaD;5_`V7_v3Yxa9>91FDC^{&L`+?vR)=|1&`qAs$NY->VCJ;a*?rR
zo?8PXdrY!%y#EI#(6^_}q+$2S9=?T?x$IG@p2A(Ekn!lPL_g%w_z1z|BOZM>(T{s{
zlxVjHRB_9R{<d0o==L{AjORQB{K*P{Kk(@7ME}sEhlqaJqjwPfQ;*(B^e;Sm7tz1<
z=-ovB5j5Wa%A-BR_=_hnO!NmHy_e{}d-Oh{|K-v9iT+H}8C4Qq3SLx{1$cPAfJcY@
zc&Jdq4<P<Y6Rf6y*g+EOQoQ2^A9<eHy|NcIudo*=cjW?FU#V*w9qR@60e*<M0~F9<
zHO~$Zs*!$BZ_lXMq3t}tkC60II_3CLy2(VvQN3*O6fNKHJ1Xd;;PL#kD=z0xuIx1h
z%RjyHG}|*(v#Py>jpdiG&Ezkv?Pmw`*Vc}ei2eBU<kY?EmRW3J{#$)r`Pch;r>^%!
z7<(}P*@mUdD$Ee8Vo?^xzX}#L8(EB*k5+|D7BkI9!)6q882{H<6|BMxvnsP{N^Ja&
zIaR-TjalBs3`3Nwp(#JIrIo!i_0=uc!mKHu9$LgcoH{$yZ?b`@cXvL<TAP&A0yPZ6
zBX`)TW}jj>JSBIZV71LfxU1Gl{uR*(GVcu4m*AK-9-N8~zronz{M&mw!yQVnEzkCy
zVBgE9_g!MAr{?az#KP+pe=z^pP;LIx15dJ}`7;OKU`z7bM|QD7{`^P_+mipu$Uu0D
PEB0QjHs5^cUDo<P{j3GO

diff --git a/cardinal-plugin/monop/monop_parser.py b/cardinal-plugin/monop/monop_parser.py
index 9b913a4..7fab73e 100644
--- a/cardinal-plugin/monop/monop_parser.py
+++ b/cardinal-plugin/monop/monop_parser.py
@@ -285,6 +285,7 @@ class MonopParser:
                 
                 if not is_info and g._buy_pending:
                     # Resolve: use checkpoint peek if available
+                    bought = False
                     m_ck = self.CHECKPOINT_RE.match(msg)
                     if m_ck:
                         ck_money = int(m_ck.group(3))
@@ -296,6 +297,7 @@ class MonopParser:
                             if ck_name == cp_buy.name:
                                 if abs(ck_money - expected) < abs(ck_money - cp_buy.money):
                                     cp_buy.money -= g.pending_buy_cost
+                                    bought = True
                             else:
                                 # Checkpoint is for next player; can't peek buyer's money
                                 # Use last user input: if "n" or "no", declined
@@ -303,10 +305,17 @@ class MonopParser:
                                     pass  # declined
                                 else:
                                     cp_buy.money -= g.pending_buy_cost
+                                    bought = True
                     else:
                         cp_buy = g.current_player
                         if cp_buy and g.pending_buy_cost:
                             cp_buy.money -= g.pending_buy_cost
+                            bought = True
+                    # Assign ownership if purchase confirmed
+                    if bought and cp_buy:
+                        sq_id = cp_buy.location
+                        if 0 <= sq_id < 40:
+                            g.property_owner[sq_id] = cp_buy.number
                     g._buy_pending = False
                     g.pending_buy_cost = None
 
@@ -759,6 +768,12 @@ class MonopParser:
             buyer = g.get_player(name=name, number=num)
             if buyer:
                 buyer.money -= price
+                # The auctioned property is at the current player's location
+                cp = g.current_player
+                if cp:
+                    sq_id = cp.location
+                    if 0 <= sq_id < 40:
+                        g.property_owner[sq_id] = num
             return
 
         if self.NOBODY_RE.match(msg):
diff --git a/monop_parser.py b/monop_parser.py
index 9b913a4..7fab73e 100644
--- a/monop_parser.py
+++ b/monop_parser.py
@@ -285,6 +285,7 @@ class MonopParser:
                 
                 if not is_info and g._buy_pending:
                     # Resolve: use checkpoint peek if available
+                    bought = False
                     m_ck = self.CHECKPOINT_RE.match(msg)
                     if m_ck:
                         ck_money = int(m_ck.group(3))
@@ -296,6 +297,7 @@ class MonopParser:
                             if ck_name == cp_buy.name:
                                 if abs(ck_money - expected) < abs(ck_money - cp_buy.money):
                                     cp_buy.money -= g.pending_buy_cost
+                                    bought = True
                             else:
                                 # Checkpoint is for next player; can't peek buyer's money
                                 # Use last user input: if "n" or "no", declined
@@ -303,10 +305,17 @@ class MonopParser:
                                     pass  # declined
                                 else:
                                     cp_buy.money -= g.pending_buy_cost
+                                    bought = True
                     else:
                         cp_buy = g.current_player
                         if cp_buy and g.pending_buy_cost:
                             cp_buy.money -= g.pending_buy_cost
+                            bought = True
+                    # Assign ownership if purchase confirmed
+                    if bought and cp_buy:
+                        sq_id = cp_buy.location
+                        if 0 <= sq_id < 40:
+                            g.property_owner[sq_id] = cp_buy.number
                     g._buy_pending = False
                     g.pending_buy_cost = None
 
@@ -759,6 +768,12 @@ class MonopParser:
             buyer = g.get_player(name=name, number=num)
             if buyer:
                 buyer.money -= price
+                # The auctioned property is at the current player's location
+                cp = g.current_player
+                if cp:
+                    sq_id = cp.location
+                    if 0 <= sq_id < 40:
+                        g.property_owner[sq_id] = num
             return
 
         if self.NOBODY_RE.match(msg):