From 7bcc102f1b0515b67c5b1486560c4c78578ff112 Mon Sep 17 00:00:00 2001
From: MrWaradana <muhammadridhowaradana@gmail.com>
Date: Wed, 4 Feb 2026 17:26:22 +0700
Subject: [PATCH] Refactor attribute access for clarity and robustness, and
 enhance subprocess command execution by using a list for arguments.

---
 src/__pycache__/exceptions.cpython-311.pyc    | Bin 7492 -> 7437 bytes
 .../__pycache__/service.cpython-311.pyc       | Bin 41581 -> 41416 bytes
 src/equipment/service.py                      |  10 +++++-----
 src/exceptions.py                             |   5 ++---
 src/modules/equipment/Prediksi.py             |  12 ++++++------
 .../__pycache__/Prediksi.cpython-311.pyc      | Bin 50593 -> 50384 bytes
 src/modules/plant/run.py                      |  12 ++++++++++--
 src/plant_fs_transaction_data/router.py       |   2 +-
 src/plant_fs_transaction_data/service.py      |   2 +-
 src/uploaded_file/router.py                   |   2 +-
 10 files changed, 26 insertions(+), 19 deletions(-)

diff --git a/src/__pycache__/exceptions.cpython-311.pyc b/src/__pycache__/exceptions.cpython-311.pyc
index 7d640202233d987df0160c50a164d1772cc8afb0..ceda2ab59c7eb33cd8786b437229754ef95dd049 100644
GIT binary patch
delta 443
zcmX?N)oaDOoR^o20SM$In==hJ^3LJ3_`t-#FrA@<5hMx$H4G_?%YdS*ffxdc*lQTF
z;41PMQ<!QQH^1ksXW_|6EKV#bDJrrCDx6%y*P`tLWY`07aUoEB1H)Gi1{LiUrW@2Q
zDqCJrw!9$fJRx*S)P$%VB@KR?4fu;0Ig(f)_D^o)mfCz&fQgx<hA~Tc@_pe#cD@?+
z*$gTClfy;i1qEuDfo$d)rr8X0SyEV11Si)DDzK#prLfm9PrfLqH2J)UKcnzudC|9%
zrN!LjK~5;*2N8TAf&)Z2fe1?w;WfEI%uE^N)LWeK@yR)f#l`XQMbaQXZ+v`7QDSmx
zQetv;eEj6|V%ChVlexte8NDX!iU)JW0@cI=aq*_jUE(Jg*=~qQUJ+BDoGAI8(P?s^
z)MG72pk$E<h!6u2Dj>oFL_lnFtw>HSD9OyvD~g>QCoRjExw%HVlo80VU10x#0Z5?}
J4wHFgEdZJoZM^^h

delta 440
zcmeCRI%37UoR^o20SFG}@??5%<ekH*^^b{xVLC$zBS;hiY8X-&Q<#=9Gcc?MVhE^Z
zEMcBpBP7Md$gue#XFUsFdTL2xNlDQy)`FtUypqW|d@Tm9K&AFTTwDe;tbySx2ZNaG
zjL-#cD_ky$8eS1KZ1B6w!r7sBkwx+fi{u3s$<5OI#f*$)oBIWnnI%fpp*8`j8pag9
z6n2OMCT|liWaqD8pUsdWFj-GTUQn=x8OUa?VVcb_mnDTIMQHK^AqA!s;mHjmrHmqz
zzlpr%zr`CLUs9BqoSKxFoE;xO`LU>*yaUkOB7P9T2O>B?gfob+1QFhoJ;cl;K_0lp
z86TgVlUQ6FA73O5lClF5lNX9vGrCQ_E2hZkJ^8;_FjpK<Fad~*cW;gnKf%a<7wFUE
z1@;?UE{a)R5wo1ECHbDwd9sw$V@9XR|D+@YAtt+4B&QaXWaj4;#Z6Y1mSxP^>?&Q#
WC@sfmyTJYf1CT-|oF-qDu>b&XZghYE

diff --git a/src/equipment/__pycache__/service.cpython-311.pyc b/src/equipment/__pycache__/service.cpython-311.pyc
index 83de7cf5bdae46315b14395d3c8c9fb7f0539ec5..9559c6373c5bfa7f025fb4521588aaa5e2c1f75d 100644
GIT binary patch
delta 1957
zcmZ{kdu&s66vyxB<F@N#o49rZncLcpwOhO4Hqea?*j(b|F(4puNX9xkgs$D>*E=H8
zIZ=WIaiGV?BtG_!Jc1@H_n%>iJPPtiz$y@Qh!YbNBTkkX)J+Ng&TT7{;7#r)=l4C|
z-|u&R_nv!4PN+^tROY>Avq9k3rL0%hwGNnDb5)-i<@LG48c^esf|?#(D8rXQ2D&=H
ztLe6cGB%THiD_}w8dt4JYGz!`Oj5JrYF3guSt?JA1tDa4j_<VxdIaGBB;@WXNfv9&
zlH_<O$&q;<$J8W8)_ok=cR6%IXfj#w&uQ79MN46E+Kfawyvkgu3<qkndYDyu^p5PC
znutTWdwBJCPQX-Nu{&3qmB{lYCV6$-rRFYv$1V_d!49qT-?;@rs!8Mo!IzVWrSy1&
zYwcznx20m*vNTM0d}z!~SiE|?CRy;jqYCnHAm5HIr+%R=k>&3by7NPNrVHtP_9V3+
zuIAsZZg)%b6JtRLyVrm>1vfj(VF7w<rJQfr?D#{C6<ZuGoM*R^UUV56aeaL$r^iuD
zs=4vb8a@XG?6YvJxE~BMx=f&DDxQ_gODX}J7${u@FXOq=Z0vXE;TP_JJXU%bEU$43
z+d+7p&`a2fkw@miOPE@A9k$>^*)qdnE}!Rjut|Qp`~y`_dBs6@xb~u{Bx))-x@1%>
zq&P?Qg2UP0O{zU=vJd8w`Y=~r?tL|+7Di2lgC0_CTrGC(Q&yN3HBB2-R><ROS^2?i
z^17oY_t9GNI&K<-w2Wv*MKq&w5WY&Sx|muOmOq&}T?0qtt5rV%Y{f|REI1+GuHFxD
z4EN8@pR|>i>eMhV9p)q<g2S_IhSNNJfww|%9Y(vXm^WtwoRRzHoKnGAY^f`QZHRRR
zup1B6*&aMg)6Q@ViK{yUfzA%T#&2~txFA39__${JIZ9V>IIT>im_A|lDuFtdEhcOu
zP-C$#v8tgcvyX>ETEC~Y&Ch%!zeCuLn;LS&uPKaRf5TiDmajEDA;KkeE~;Mk4e8$!
zHWADOWiMJRwsi`RhSb5X0J}nwG1%sBTP>|x!#V@^RutP64=u_`S2p~f7P(A_;LnRH
z;ks;Fd`xS)l#m<QIlS?d#rOXQBj-Ie4w0r5s?<nD-F;jdrV^-b_8lROK)*(I6+4#C
zgc14U<qOpi#^0L`Km<Qp@i31+Ur_=#<&hQ7gKmsl8`&uOR~CtX(6ljpe`OWik}t1p
zhWfuq8Rg(DP)M=zGR7&IAXSVtMr<M%hK%bvf^AG%<@HD&_BZ+d!HvysP|16ncY{&Q
zTZYk=EQ||UrhpEgZ3#kHzS3ehh-ONrqNaT#Z=4s~+Y|NNXs@<tDM!B><|4QW$8oN|
z$Y`L<n=}@&%U_zOqVX*nH*xTuQd&1mLTq<D=^qi(xhrVJ+htjDG++?<lerRfLZPgc
zcEHqAWL`=Txy`9g>i$o=CPR^Ys%wu9CdnP$Kc*Xxlk)^&H$KxFugIiK$6dW8>29hh
zLgQva0iN&O0xn#-a|6`MXLnA~>Th%1sbaPG>s#}oUViND0TqnnFYo5S1RDCD6qT;n
zg-v~)9{PQ-BLrnf`VQF~!Ue)G;W}Z2aFd`s3+1WksF1a!6;f2hF{Ekj?DVr*8rKu(
eLu4}uGdU*o)qzfLm%nW`^YAPPa^-s!DgOddYTmK{

delta 2108
zcma)7eN0nV6o02JZK1E=2jxR;sUSkZmQhC)C}`^k>M|SwO~el@wr&;*diz|QmUM2K
z&1q3{G`N{9**`9e*#>*dLb5GkiW4;1O4gXkJ~T`Amr67y8g(<fr?o>!|JcoYznpu{
z@7(k8PH#spD((y`^q=VU83JF;hPQWpHlS}eD@>xi$9zKtRoGNq7oI6u$edWAY_H0j
zc1|e>=lPphUEodYF@&-YPG_o9GILTeH7S|7#0*ahuT4qKOL1s=N@l)Po=jX2LWbQu
z`E$U_;csH~%$dRn=cJ&_okGb?!I(FNk(YvDoWjUY!7zo4p?Mkm`Pb<w2+i?k;K0K2
zENe1Pp1oNr!;hBd)w3L_Vp8qTOUfbh)Y6#3tLrgKi<5SK#4N9ft`fuCCDMNl5Co}e
zlHs3=drGeB<|Q>=J&swjaaU<A7;#`hQF>8G%QPXa--KhOh9w0_9-j*6DSl=%Kd*hy
zDy>NF1>s%mPVi=;(Xtp;qNlKe<3Y;-l8YK}W!VW>&9(XXQ&SF?3W{9%)sw#d1-QG}
zgm>#QF<kxyWXTnaM8F5O!ZfDfEX=D2$jhzO0L8f9)&}q3xXq~5yy}xYlEmc072ks4
zP0nKn3Em=jo8S;`tXv0&@#D%{a0DavmW=PX`bYi-Ps+FKA1cCbSOG)ENW%$NG`D)N
za!jm?iFHx_v~0Mm6w)nswZf96{ec9*)(>N%Jto?tqJ3m0*-hMD>m0Nu?Dl@wnCOU!
zj;QDuX(D?ex7XATJ_}IU&jVD&#Hy%RHR2+B!F*0|fg6VOi6fQ{&L0z<G0_?2Pjibr
zQMFbDL-Nv^zW|P+$yp1R<W}dW02feIzhKr;UW!G<yj1KD1jATUUkIbPwf;0*k?+@E
zR=`#K#$|_IoNyJxS<H49&b~^_D;&nvJGujb?k?`O)?ElOxzqhf^#b*Wy~x3m!_*`j
z2#yiZ@UT}1dI@F_MDcV(S?>E>98c@?wEH@lpVWN>$MIT2f%<0>hw(|na=0lw*EFc%
z8XjtNhOd);oZtX~o**8J4oiei=hC<`xHrIlBS{zZb^3NlZ97?aV0uQAQJcxVCGOeX
z<zvgZWYHv=6i?*_*?$pK9EWkm`f9i>_pJXuEqpDZ7lcMO*@N8w)dTXJR?e@ai`Nt{
zhw^{EGqlYlpc2{d97L8yVm84aIJ&_B6JKt8SqTwr+;kp>@$n`Tm-Aa~@IYSEx(_t>
zIkS=7!^@k?)DOw)K0e%B15ai9mTj=|5ozvm-~&=Za{RkJCg}-j)rpQI9Jyj#x2r4Y
zV^W*fBYD{0#QO(hTdkm!727_5g(`lpFbc^c9D=-if|)Po-~t~c=JQBCro{_wL5RqX
z?L`^tO!COWHvb!Z@UHtil1Z-Xa2nKPtE1Bv5zuAGF5=maGM$brNP7)NJ1XWYY5$b=
zn>g^9j88k|z!E8;x$_RplrsY*YThMB&;q6M_TVvCHbR_c0u4XIqI~{$!nHyA<D0C&
z%SVe>P|m3*_@xuvhfS^7^0K{0G%#Bp4gEPocae^|L~s_rjy%g$q`d}jMQk%{l<6?-
zw-GEvOYb4D;^)1ugIg{>J||5pP#y~A#)eZb!73R~4JhC-uIej*C+O>2tB#M*N&KqM
zvxROdHbfAQI7$*tGK&%1B)Cm5LGXYe-lKT8w3Nteq$?$fN*Y(ScXxL(H|<y9o-;;h
Kmye&RPX8Afv-9Er

diff --git a/src/equipment/service.py b/src/equipment/service.py
index 5a4a8ad..002f3fc 100644
--- a/src/equipment/service.py
+++ b/src/equipment/service.py
@@ -250,19 +250,19 @@ async def get_master_by_assetnum(
         y = yeardata_dict.get(year)
 
         asset_crit_ens_energy_not_served = (
-            getattr(y, "asset_crit_ens_energy_not_served", 0) if y is not None else 0
+            y.asset_crit_ens_energy_not_served if y is not None else 0
         )
         asset_crit_bpp_system = (
-            getattr(y, "asset_crit_bpp_system", 0) if y is not None else 0
+            y.asset_crit_bpp_system if y is not None else 0
         )
         asset_crit_bpp_pembangkit = (
-            getattr(y, "asset_crit_bpp_pembangkit", 0) if y is not None else 0
+            y.asset_crit_bpp_pembangkit if y is not None else 0
         )
         asset_crit_marginal_cost = (
-            getattr(y, "asset_crit_marginal_cost", 0) if y is not None else 0
+            y.asset_crit_marginal_cost if y is not None else 0
         )
         asset_crit_dmn_daya_mampu_netto = (
-            getattr(y, "asset_crit_dmn_daya_mampu_netto", 0) if y is not None else 0
+            y.asset_crit_dmn_daya_mampu_netto if y is not None else 0
         )
 
         # Convert to floats and compute criticality safely
diff --git a/src/exceptions.py b/src/exceptions.py
index 769acc0..8c9b589 100644
--- a/src/exceptions.py
+++ b/src/exceptions.py
@@ -64,8 +64,7 @@ def handle_sqlalchemy_error(error: SQLAlchemyError):
     """
     Handle SQLAlchemy errors and return user-friendly error messages.
     """
-    original_error = getattr(error, 'orig', None)
-    print(original_error)
+    original_error = error.orig if hasattr(error, 'orig') else None
     
     if isinstance(error, IntegrityError):
         if "unique constraint" in str(error).lower():
@@ -144,7 +143,7 @@ def handle_exception(request: Request, exc: Exception):
 
     # Log unexpected errors
     error_message = f"{exc.__class__.__name__}: {str(exc)}"
-    error_traceback = getattr(exc, '__traceback__', None)
+    error_traceback = exc.__traceback__
     
     # Get file and line info if available
     if error_traceback:
diff --git a/src/modules/equipment/Prediksi.py b/src/modules/equipment/Prediksi.py
index b9cf0ca..44a621f 100644
--- a/src/modules/equipment/Prediksi.py
+++ b/src/modules/equipment/Prediksi.py
@@ -599,7 +599,7 @@ class Prediksi:
             # Try to sign in again
             try:
                 signin_res = await self.sign_in()
-                if getattr(self, "access_token", None):
+                if self.access_token:
                     return signin_res
             except Exception as signin_exc:
                 print(f"Sign-in failed after sign-out: {signin_exc}")
@@ -610,7 +610,7 @@ class Prediksi:
 
         On success updates self.access_token and returns it. Returns None on failure.
         """
-        if not getattr(self, "refresh_token", None):
+        if not self.refresh_token:
             print("No refresh token available to refresh access token.")
             return None
         try:
@@ -641,7 +641,7 @@ class Prediksi:
         endpoint = f"{url}/main/number-of-failures/{assetnum}/{int(year)}/{int(year)}"
         async with httpx.AsyncClient() as client:
             try:
-                current_token = getattr(self, "access_token", None)
+                current_token = self.access_token
                 response = await client.get(
                     endpoint,
                     timeout=30.0,
@@ -650,7 +650,7 @@ class Prediksi:
                 response.raise_for_status()
                 return response.json()
             except httpx.HTTPStatusError as e:
-                status = getattr(e.response, "status_code", None)
+                status = e.response.status_code
                 # If we get a 401 or 403, try to refresh the access token and retry once
                 if status in (401, 403):
                     print(f"Received {status} from reliability API, attempting to refresh/re-login...")
@@ -660,7 +660,7 @@ class Prediksi:
                     if not new_access:
                         print("Refresh failed, attempting full sign-in...")
                         await self.sign_in()
-                        new_access = getattr(self, "access_token", None)
+                        new_access = self.access_token
 
                     if new_access:
                         try:
@@ -985,7 +985,7 @@ async def main(RELIABILITY_APP_URL=RELIABILITY_APP_URL, assetnum=None, token=Non
         # If token not provided, sign in to obtain access_token/refresh_token
         if token is None:
             signin_res = await prediksi.sign_in()
-            if not getattr(prediksi, "access_token", None):
+            if not prediksi.access_token:
                 print("Failed to obtain access token; aborting.")
                 return
         else:
diff --git a/src/modules/equipment/__pycache__/Prediksi.cpython-311.pyc b/src/modules/equipment/__pycache__/Prediksi.cpython-311.pyc
index e8f9e3a30b890e9c31e909c74feb7a745af09f24..a7fa153308de23c0aa5d66c23bb0a1f995a15542 100644
GIT binary patch
delta 2364
zcmY*beQaCR6@T|#KR^4~FaAvYk-APA2jW7TCQXy7A87eb3Y5asp=|^&*zW2yv6JiP
zj;8iP2SQ|MQoHlV5KYvUBGA+_lDt-9YQ~li@=>7eSgJ*4tYTstP*rRb{4+6a_c#h^
zp5>p<`Q3BwJ?Gqe&-1w#>5XBkzOJf@z#qTw*+lWUdNoGs_2rmbBt`n2LoX3SR_HP7
zqlfJxvB9r;`XF@VB>9(q`A8o%F1tKSt`O9ublFR16=#PFZl|gby6BukX={+?N}X-B
z(%crQgEO@fG7S{v-J9A&;(UN~c<Fov;mmwU=}@Hkt<DZfS`g5FL6SJLprHMNOX~1K
zs^8<E76su!fe614cvKj4;Ym9*3gxH$aIHTYprc~U5R>j6KP`-k2@*42lwtos5Pq_~
z0!|J{M21t%6%ML7R_76fEY+OwzfPa|yD?Li9u?R@xPHki#!O&j_YyN~Wj;@#Sm>hB
zHOH6&PqJ3%Vh*^(x&oHfk&wUwIcc4x6>2W|pgrv;Dm<0`4U}s;$bR^aHcqX~2TCRY
z_h%}NGkgX%3y)`_&H<kEr~LDb@J~gsyO(U``pwK?v2DcHQY&0;s%r_-nXr?~Gkfe@
zp0zr-JR4M6!t!jGqI^F=c}qdAwY6@f?+|CJkG|uRS~tS8Pv0ZX8Q{XxFXL7lpzBzr
zhh~jTE|k%@QoJNd2p5fT;&?AegN?f%%9*mJ9`kCJJ>S$MXU>{q@=@U=VHVA^IxYyj
zoaWO)%sfir_Fz+0PRT0iZR-q_*N7lo;NO*p@Ety@@*Ua)?+#jxsPfp_kKbpbg}9WJ
zxC$}-GFY_g#_PQ!w|-?y)&i&M-O!bNSiL(Uz7M$rAm`M;-QL@BrgdSV)ezsQ@m!hz
z3SWi2i|`(Yu*fD+exC#UH}{su#$2d>58<~6ZzFi&y&rFJ=PhZiTg#*)(NrwXLO5{!
z=R?Kcx`EOo9P%{7dh@1uGS;6;BsGTnpI4(9#^OmWqNR?+lWYl%t8u`%G8?-F2ZtUe
zqi}xcsByV0v{Y6DRnPp%wCwYb_?LXuU_bFFX@Ka7YU6Tcc&V~pKX+nY9J13XYh!U!
zoj6@*?^LB3Lt|}+MVf7BRN8!oIYKx$CmK5Kbk1jM56E*>u6D0Hx0|56$G}bJ1+<wb
zXfyAXI6ogKL4JFIuS0&p;^a090avF<TG)pCLJjhZrecXJiz-FMu5|k4#Q^E_>t8=t
zNy$h0rWbzX-tj4B`gerSF<si7JjU?0vCmL79ZIoyv^%Y_CgkpL2+QmXxHeMa_%|Au
z`*@XmBZm?i`%?dE<dV4I6SOg}O!*IVDy$%(^cnpA%m)4AGigHprXM_4PRNJwagPgr
z`igr8uFh_A$QxO_Ki;k76)hD>X8PhR5iQJ-H}u83lRU!S-n=2vn~JhK@W)r0$PFkt
zza{WjbhY%ecq|ds5~*aGeT6d;e#{w#ADs7+FW}hu<5rmyc|$bCQ~Gk^RjZl&8Pqqb
z-74ByaV!`)fRh|aCt%9$*|`x1MsTWb1P7{5qLf3JM!1epIEx>m<V2Ht{mdIL5i$Wg
z->fGOLF&yq`>?%-x0jVCy;+kBee_nX1orXmrlgz|hwTv&%Ex?E%o4muMzb(%KPy3F
zONjxUVPGswtZ;fPAk!@2ciaHK8`}%nW(TO_4g8%^?(ss`xDUP_Yd2)+Srbf+7gsn}
z*A#?{^7@;flVc{{USv2IC;{6<G2z#@xdJ*SY8~dBqWRW5dsKLFqLc*S^@*^m3_JPa
zmMm`!F)MsDQTn+0oGEAdp^&p`!8IS=o{BEoUgJ4m_$Kcbq3h8nxRz6MMOyh?wnZ`f
zdP`QEErL@&^FXrQR>G@O6mzb{le1;5XPq!r;)XMNSX~<s_-VSS--|T~C7UV)&APTm
zp<zjyyvD>_`~~p`(Y+tcm!BZF^h1}Q7s&^3=hr(o?!isG#ZAM;!gl8+eqa*)EB&Ni
zZ+`19Vnd;$U}}T5R>o4SERihh>rNzhm4)35Uv(?T!;4a(4^^U64_#Ld+Y7!=p?)92
z0^Gdv0BHmFWXRf#OSd4j5_o^J2fjaPQy$~`62F&eGiZ~K@V^74ORg&YgUPq)5a}!v
zrYbjZoEFQQLc(+fGSeZYS*6oki<?b!hDw~7F-e@6QJpC7+Qf0Tp{0udEC$KGy~1Ld
zvM)#%H#_(F>Ea#(XRi7s&Rh*rl=u1`ETMT*B!U@6B7@$?GRd;|KqdjUsps#(biy^P
z1as-&q2`VK-N{&Yn(g9DUgFod{}}sRzcN)pjF<{*fKR3m1PfKzj!prDQV#zy?cpsY
P(;I()CG~I5>=*wJt1e^E

delta 2639
zcmZuz4Nz3q6@K@<ef#$RpIsJ2KtWv;Y(N)Alqw(sCZ@3&JM9>^8{Qh1z%Jf>Sc7kc
z{ER8h)asncgh_NnlQGQ5B#>tl6H}}FEf{UGQ<`>YNR#Pbe%drM8QYmmJJWl1*;r<J
znfL9v-#Pc*bM8I&T;3ZL&OakqU$$CJ9BbeH{S(p0$E`PI(yskZ_V6TM`}@HmlCKn_
zHrrHAmSl2K;J8bIHh9Rv69+us*=<~q9*HPnMWG-)dWw9l4L#Z|*caUeLz{+Ij(O7V
z;JiBs=C~%U5~fV9)*_gWZ|pA;W}K#$T5+b()lx0aREn*PsTPr`6;Q6<u&atsttG9i
zgj6MAOsdM%>Jd|Qu2#D^YeM^3yU3VX58BTbiOjNt<eX){68oNH-&gPR9JiYz+;g0g
z)7u%(;vq7hjR#^qJ|e)0p2OR8XL(ta^(xn?XGWR!0(X{=5?LBGWUX`jlbp;S6ZUb>
z2`6+(Ugd`+IDKy!6M&3+q35vE?#`LA<pjk8;gf#h`?3*64;K?7+&a7#di(6&q)ye*
z3RS0aA#-kkBEYLV3!s|T5({+F?qW+$Q^wJvq^PiS$Tiujif}sU18bsy*kE1aO+EE9
z<sSGru^D{IxL~D|@QzYM{O~6wpr2woIt}jLj>0~+H=nS-0q%~8bNEgvL!l8lZLgWP
z*JwZPJ;b+M^fzzie%ZLGIUr2dxR^YpcQAR%>qI$VYOax{Y6O&n1m)U{yw$#QoiJ^7
z?OZKPuNHT%gW%CMJTJn5qtE9n#+=`oGZ18M=?wmTwA`mt^;F92zdpO`F%soQCAjo#
z0qlJYI)0cmEE{j={AM=Ukeewtf6}NLW#bX<6rp)aUXJBB*`yj@;AHbz;UY_jOSz<A
zDWY=2W;orn3P$@Zo}@`NCAKUZXn~Tx#sb1E^MRA^#&E4~y<L~_-<0)#$#>VWpTVth
zqn91t=A?PqoK}@B8T7z4)d-8nn(hDVs7*C2>9)Xe)tZ~)TWlBpzgd*nVO_FhLp~ny
zRfJmzw;2R@`ZmhHVz6LNuq)e}2zA8e2t1s;;<HiARr&*jUm#pVn1p+dR|aVr%I_ge
zAlzUOu+RdO(+Kx7SP*(?*McDulY8RPm_n!E{BgJYeN=BmwLZHdeHT*4?;~$QUjGsM
zV#(SO-AL?;oGxjaFKL3){l9fC`b&nLFBObDbfr1%-!bps0X+i`@NEPp1~%Fk%QuWX
zIM#4|Q@Xr$zPwfY*FcKzuM%!pb`&=@@DmM=w)Ntqtx?|^5U2BYumYo-DJG1YDbcl6
z2{X0!UG-8*@7@)ZQhtJRfsUD`R-#R+7;REP5&3#A<7b`7&$^JG4Y*LQcekw(=WNK&
z*^!@HlP6+ly?~u6Q(K)hS5MmNw7pN23*<xX=+i&)?Ee&Z;jajv<5mda*kcs$FZ~R=
zhJ$e$=?Etj+KAj23<47U66JQ(o8_<)4kaQq8cEQvkTG_%W9teXj4Jeww)@Ow-uGu5
zWX!Jd369JxDMI176+Hciw&~mvLjIy%8ZIT|1Gs&D?Phd@{tYM7(@0MwtSp$6cqrD}
z&6erNj9$=nN5V1YY**KUF4`6EpkF}XLL*5-*M-X9A91L;heqURhZ2p)67-)qBhM}q
z)giQC_cFp+gjR$v;oS?zEC!}t&~?OFz<0ELFItRb5jsXU>TNj83PYpib;T&2!69J;
z7j_SzltjoRVJ00j>G}b3ZZxUY?u-r*G6N@H35xp$9m6DdnG2FSMPznMZP3ALf3EKQ
z=M^8}Azkf;TVpk4F-heI9ieZGRYIRp@GV(tvY@hAgC3oc8+5?qHD0|+1|6LukajP0
zz3yfOc=6RDNv9H4Vi-HN4bFt@@P}#<r{@(Xl=|}F+?bz8OKS$%Q*guUS)+JzYj7NU
zYFI#bOH)#kjY)$lvGR1on+GdAPF|HDyaJyYJN#t4S~Mk1N+ArW9{A0;n-s&R;{mIA
z(8ZjvsH_&rRw#b0@O!qW3`z4NT+*VH-d&7UN&BedB6|uhu`k=2xrZLu&w5E~(x$A=
zv8-a-Wan~Any1>p`BN|ae3!$^cF!ie@$t#Fq+PWPyD>cY__ctwFvoLTrw4r6+RWbE
zr%9!@yP8am6qoI}C-cDh;iUGul(&asb%U16`^iVzi<eLE<kwK~M%~J7m;xU$RY0Ff
zmIaYzShOcwy=t|AH$LQ9@`m0jD9xizW|qyPNt7}Lt{kOI;jD#c-#X;Tl(vVFT#qmZ
z)~gL<7u<Wb%+icY?L=rHaJ$0i-@bi2#o*Yb2}fh}yG%9DZg|24qgU_G{5MoLd|>Rs
zw0q0Edy7Wiz9#gW+bXyl8&@!#u#`4daTApU<tkH?Pnf97YjO#bR*^B2E|D>lJ{Oax
zbgLLnlje2oXD&c?OWa(AsXZXfZF04j2{-jR#@s9u8FMorF!>$Hzq>?OFoZ(rS18n%
zzqdD55;@!(g@)@VtQcb89_q#T+8}*BpM0);c6~L`V*s=c+!Nm^&TK;sS`;B*lwTXR
SvI^AO6=|Sx?TLv!{C@%Q(yaji

diff --git a/src/modules/plant/run.py b/src/modules/plant/run.py
index 0ade782..2180cf7 100644
--- a/src/modules/plant/run.py
+++ b/src/modules/plant/run.py
@@ -276,8 +276,16 @@ else:
 # Mapping dari kolom Excel ke nama kolom database sesuai gambar 2
 
 libreoffice_path = "soffice"  # Sesuaikan dengan lokasi di Linux/Windows
-command = f'"{libreoffice_path}" --headless --convert-to xlsx {file_ref} --outdir {ROOT_DIR}/hasil'
-subprocess.run(command, shell=True, check=True)
+command = [
+    libreoffice_path,
+    "--headless",
+    "--convert-to",
+    "xlsx",
+    file_ref,
+    "--outdir",
+    f"{ROOT_DIR}/hasil",
+]
+subprocess.run(command, check=True)
 
 print("recalculate OK")
 
diff --git a/src/plant_fs_transaction_data/router.py b/src/plant_fs_transaction_data/router.py
index ec735d8..c67c34c 100644
--- a/src/plant_fs_transaction_data/router.py
+++ b/src/plant_fs_transaction_data/router.py
@@ -58,7 +58,7 @@ async def import_fs_charts(
     updated, missing = await update_fs_charts_from_matrix(
         db_session=db_session,
         payload=payload,
-        updated_by=getattr(current_user, "user_id", None) if current_user else None,
+        updated_by=current_user.user_id if current_user else None,
     )
 
     msg = "Data imported successfully."
diff --git a/src/plant_fs_transaction_data/service.py b/src/plant_fs_transaction_data/service.py
index 16ce785..957c0c4 100644
--- a/src/plant_fs_transaction_data/service.py
+++ b/src/plant_fs_transaction_data/service.py
@@ -21,7 +21,7 @@ def _resolve_user_id(user: Optional[CurrentUser]) -> Optional[str]:
     if user is None:
         return None
     # UserBase guarantees user_id
-    return getattr(user, "user_id", None)
+    return user.user_id
 
 
 async def get(
diff --git a/src/uploaded_file/router.py b/src/uploaded_file/router.py
index 2c81054..53d9625 100644
--- a/src/uploaded_file/router.py
+++ b/src/uploaded_file/router.py
@@ -137,7 +137,7 @@ async def delete_uploaded_file(db_session: DbSession, uploaded_file_id: str):
         )
 
     # Attempt to delete the file on disk if a file_url is present
-    file_url = getattr(uploaded_file, "file_url", None)
+    file_url = uploaded_file.file_url
     if file_url:
         try:
             # file_url is stored like "/uploads/<timestamp>/<filename>"