From 2239b332f4348154695fa89894097b881abc7c98 Mon Sep 17 00:00:00 2001
From: Arieh Schneier <15041913+AriehSchneier@users.noreply.github.com>
Date: Thu, 20 Jun 2024 09:26:44 +1000
Subject: [PATCH] proto->sysl: filter out duplicate enum values (#925)

---
 .../bundles/assets/import_proto_cli.arraiz    | Bin 38613 -> 38862 bytes
 pkg/importer/importer_test.go                 |  17 +++++++++++
 pkg/importer/proto/proto_to_sysl.arrai        |  13 ++++++++-
 pkg/importer/proto/proto_to_sysl_test.arrai   |  27 +++++++++++++++++-
 pkg/importer/proto/tests/enum.proto           |  15 ++++++++++
 pkg/importer/proto/tests/enum.sysl            |  12 ++++++++
 .../proto/tests/errors/dulicate_enum.proto    |   6 ++++
 7 files changed, 88 insertions(+), 2 deletions(-)
 create mode 100644 pkg/importer/proto/tests/enum.proto
 create mode 100644 pkg/importer/proto/tests/enum.sysl
 create mode 100644 pkg/importer/proto/tests/errors/dulicate_enum.proto

diff --git a/internal/bundles/assets/import_proto_cli.arraiz b/internal/bundles/assets/import_proto_cli.arraiz
index db053f4af508a829f6926b52e6d42166463013df..e375ab4abf521e14c39759118127f5cb1a43e3c7 100644
GIT binary patch
delta 4998
zcmV;16M5{_tpd)k0<d&$e|&9o+qRbfdw+^k*;A2YmHz+l-suaba-8PIo$R!6r*XHt
zQ)euQge=As!2+NgU0M6tch12BL5gzhCbP4<6TPtn;5-00=XVaCTyhOjdc*Y_w&JR0
z8J__D{F!C9>;`|ZsuJf?tS)L@h$<*}DOM|9=?SDom6c7-U$1H*fARN1$Um9P-<C`*
zdHyrg%Lyb&62lZuPBzI&ql<F;?7+ybWw9!Bafh<>y5_&|#RO8dR^?l^;HcIF(u}Qm
z`I0G)g5Or`mgA3*>q&ikb5f87T#^LkMX<U?K9}N!`SKH2%5<(~QY&A+!;eYHiW4nF
zsZLm3S8T;SQ_1rtf9sIk6nfdrlT56T@EcZG`b<<>7V~bmC9iTWFV{8y6?OTo(973V
zSycQ*Rf#={;GleC2)U$R*DH}<yAtg1!695OIjCl_C_aD|!15gb=p_eM*JY70oJPQ|
zfLQF5>m}2$6lJaeCj(T?vtnUuRYJ;R#f!RrYh+AeguPHBe-b{cjp5Lk&L9)j9hX`G
z16u(~%c8m=)!>emMc$EbY6C_yQn5=+T7-gN3RkYl$%*2BB&wN5qhvIJ(e-HJ+Qvpj
z$(1PYthvrWaVE1RS0rfBlqD>R5=H8zf}(<%u_`r#g^(aBZU$+^w5L)_AvKtGM3|tQ
z!V16^K^-s!e}%?gb0$dwgIWqLNNzx>X24*@YMdu#)uQ&ua3%5vTjNl4I)r0krP;sg
z0000RI-REw*(&&R$)y`KG*nhp%2rMwIg8_GM4gE$hYfwjk7blX<XEbrx+!^_oK1YA
z>l98-xRgRhN3&lmqa~Y{ZZuLj+L&)!sD#Fz{q9N>e@ssyIRhbcF6V1V(sTmbce9wd
z8$+MrfV{LiqYcg=Tk`CdI;EFf0)w|}RT@h!6y$tSR6I9BPLn)e+v!J>Tp3tUC%kG_
z3MT?CmxTgWD$!a=#g*pyKx3~Cuk55}=7;k$Bax#h%MDc_Qq&L;$k2t(NxJul7DBp-
z&!9gDe>nevCUE6=)5v}GRTNYvt{2bK&DNSiat2dyG;n{vBOpr7?pWFIiEkChaDP9<
zp2lV$Z|jZxj+Jkk)tt*2NUj@c?3+Z8<wHfagi3@QO<i)NtSl7`6BdHxwd9Id21^2B
z8m(3mj*+eK#%-mvHXfxSPC-n9f;>(kpM)lOf5Y{qfkQKWrUz<g8h0?(`5Jk)n`lK4
z!&)G~FEcb<Z$EjFArmIoc1&;i+BG0_35|_ac`%r0%8)b742Cdm<>1(K`{8WJ14o6J
z5oQTA&DIbsL|{8s*EVy2X$PUBHm4xcF0MeOQs~Cws8Voz48M1R&AS-3Xp*g%gJ%am
ze<mRr#?hZ8H-X8UK3vsQMk2%1A~Nz#Z3;)TBl~@OeYA1Z+t~!M%>VRZWWgn`GM*dk
zaZ}H%G4`gJ=*J3mH^A^g)BKQ?;Ivnw`o<?i&~hh2gA%hoB($vpbMezp8WhU6BJZ&b
zs6BIL`+&Jl;Lk!YZPY=wlaKR#Li{>}f0@ceZFR^RDG{`&$PJ!j@4$)_GNV9<UfA3O
zlLneRn==?Q&;lC8p$Oo#KtF4$-09Mh>X0>@T(TwO5^!Y)%|}<bGTMeVhAbMzy(L;R
zHv&B9MmoVkj|m~~?`^dK&9E+Yec862Rv9<lwR=lAY2{m2>zM2tOlOKJS3PT4e=>s@
zLKZhg#mfFLlgai$ju63^jt3Q|XDG5WjA3rlHl{=-f9-Mr2yrF}QUf!forOn~oW*H3
z7mAXziA^?Cyp3HHde#%VKtaBYpuVq(q1*|DmUooI6R;EJW1;&T!ujie|LIi<8j}eE
z;U&v%6L@LNu3TGl#T3x3U|`7<e-_P4zETK{ioAeggdb8=tH_o?K~J#FRSa2V{&bQc
zEQ%BOZOJR6-V)Q&O8HTP43~Qv=G;$?&0qg4$`VXfPb#H&l+N1l)59{t0Nc^L<HGP8
z8jw{ayM<hE8Zlm}MmjuOSiO5v3q{&^#~u<e@rpC8`Ks0(H4Qdccw$8re^|2nY>YW>
zePL`9-`1o$QPSCm<O_^=-YXPa@itIiY0!y{me14i*u>gSxD04nU0=GmHthv&*BHFN
zU(DanAd{SF?t)M|A0X!Md8T2`6wd*@`;HZ*Nzto?kSmG+9cR9nzqf8&wbs^K9T@39
z@QO>ODVUJ53n^9sUK9&3f7zPu5MnA2S&~O41ry4AQurO2zl$e5MMA?yN3OglW~?lE
zj;jC~mmunOJ*hk7Ub*N&vA+M5mQm;ZEbr@lU<L^LPCzG<@@?V?3`_pkM(Hq@u3Yx-
zKG%KSeXWf!+HAx`0}~sVZlnot(Iz52*`FWw6zq1gQ-z<FVHi8`f49j#9G>^MUhi@m
z(B6Fx;qu}i7b*OJxuaY6y9~&>37JZv$3eDWQu93=&xHj3fvxJ2r|^0K-?M*GxFpE$
zE*o7=buPpnW~57@Kyd;o<Yn~%wG>U6BP5>-k$lhoImlusRs}QSgH`&_7~!Y4G0<_m
ziSvZ`VOI!01G5x<e;&Km*!O8|r&96WmfJo0U5D1Z*$#B01^#WYeEsD7|4VlN-<#dL
zN&8EAeNkzC!zEg2>o?<~(&Ne4)|qer@;CGM+rN)nOBy$>rm<CH<x12}<3;g-=U8XI
zn)bfbe*O0EV<OoO?$sGrhwoC@jORj>$PE8-R*h+CVV9iIfB5^0iyy~<)wh(c&nblM
zU%$B=ZoV0t-4qm+i)LPO>Om=(?uk7A@#4i^CT6!6NtKpJ*pdGB@|V|dei)#g!Z>G|
zk5eiPDE{%{^40$Kx>zAiul?ogpN446Xs$JN4F9VBSpLW5tGC0#=0(Nidfe;I4uL_V
zU=`YiK<3UXf2^rajB!t_S9CT>(bmrkUgk40`^F}M3oP1M&CzzLJ5TW2Y1*cNh(Qj3
zkXj4Rau}do!$LtRGN#)N&ki<qH^%~^J`6hQ{~HKIvqD>KPk|uv?nP&xvSf2!nlq!+
zHlU(jTpvkZ<2oQ8r4T9pM}uRUdN7O@32OgX8|y{Kf03Z`K~|3bGDkQ*g-Fyo1uYxi
z=P&smO(9A7zsSF3xb1k^Ut||L87`n+Xf0KD2N;WSsVR$jgKnpmOq%tEJ%T)WRW+-V
zUw(dRS7OG>1g_bT-Y(C`M_MjSx&*<j>+FHRt^<ns`|hy^nV#1bO%I#2TfLi+5{=CG
zHD+^Ue<dmG9_HE&T$2kO+r1bue{WN??iPA&7v9D=HzV5SxVFV8f~64gQl~WaSP0Wo
z5_~i}+9YRNn8MMfdkF1LAmZ&Tj@u(5r_!So%gpI@pjpmUCJPc^yWTtLjvS2_WF{Yt
zhJ5+oP)Z21x_gj8;6UI;vt{wdf^&dC4qn5Ze|t0Gj`iLt|AM*uF}FKUX#L>~JOzV2
zdPgA#zqon=ejJ~gJss<ho6&nEs%y<|tO*WZeCUaF_86M|fMT$TQmdh}d0A(#b1Mq{
ztvw=(yoYTc&HNL6%BHVTN~00?oU3!0pb~P$%D%sdr~8=2QNwxPI_rx|ebl8s#hco#
zfA_Prdpg%M^EPT5n@f8x-D&?MH?+nkDt@uJ9J06ysO}LKN9}C*CXz3P_8APVJ2RBI
z(BMA7%#h73Ut>OjlOz_`M;l_2oNZ_A!J4KMiYeGCbMizZY;H14Pf;i@^G<qdrr?`X
zh$`9|?wBzmG-7AZT9e0`Xsg_vja-qafB3anq=D}cXxja}Q1{OP%)u&lU0b(PZQ^BI
z$f_Bgr8vwJ@1t(tj^LQpgYHFU`ElP`vHA;$e|FyrCxvjF)ql;ey4cO?<jfyM+3oh1
zC1({hHO!mB-sV{r%C2(~c<XkyEmh>GL2+CmV2`dPU!3@T6K7r%-_rWQX6!Plf3dOe
z@i?Bq*epL*tiHbG>rM@&LT$;`GK_b857*9mH}tUjvF;rB^NQ7ODHb`!M;#rlchYDY
zEV|`uoG!D0za0Pv^A{J>mZg)Dm^AbF)HF#H*NMxuKI*#KT(Ao+bkc#f^F(r2f7>q1
zy}BOzzsOfVn}cb))$GP#uXFXhf68tc4y8oiwZhedU)j4>#v(gyO(b(yQam>wDoni#
z6s~m;yB-Q#U3T8!gKH}kcpngN>YaGk`m`7SKTU@ftIt0&3ciWcB%nF%0OqHuVor*z
z^IzwDq5PO1pStcJpSI$ir>5qPk69{BASFibNUCa91D=Y-#!b<n=x5S4e@D}$7LRmL
zYTNTq+StVMDO}Ot?j=HRdL()<vqA()&RPHp)}xNV;yvWNmYifGiK(+YNYA5jTRV=Q
zC-dj&f|V+~sc6q*r%o&UeSbD->F({FjyA&@$m~o{f`y4i+}0DZg|U#J6&9ut6`HS9
z3XxU8+%dXZbP~&mltmMXf2_*xDy@Tim*~a*4mR%|lI}U;tsoOkrC|#H-|V<f;1EPr
zu7OutuIWx9G2Wq!Dyzg-NjqM#Wp|3~+I!z*2b9FDno<{N?sNsDWe-Oi*M+Tt(iDz1
zdt}h9@a=5iX!ff`5}JhTkh|)kX+2i1>6&HR@M(Rcp(0BPS-0)ne}!9VFWYuy?H|qF
zUH3Tpf3ca^wjUUFi~~xgA!22{WOJ^IjFsg&hL1|Q4is0nMGZyw=E*jWhaAvf;h@GI
zVDefNl@2ZzRp1IiG}>%;$~q(TkacFjL)Mwa4%^g+YQK>!#cDUs?>W2fov=FzZJxXj
zyRg-6YVm~RzyO`?f5a57kYO9n7d!VGOM0)eE(OcY^79tHLnD9i5m4mMC2$}e!Zi;Z
z0!A@iTZsG1S%lmHUPbj$R+L4K2-HR?7PO3m{`(hJlO9%*um`|V)|pRHP}F8rGp)Jo
zoecaQjj&*J^UKdKNAbHj4i=;}lWc`ejKon<)r}t6w`B3Vf3~60E8Vv152>W}MQSH0
zj^8<Z$3Gito%$}DeScQhKk_?XUf5#`COI(3L|JmWqK_6t&!<=k?r4K7w&I+Yw{)kH
zj#_hMr#iM>B5PWd>+tPWU#Po-FuoDj_Ewd>KQMmJ?pRQ^wKwH&xPGz2<jN`N{`&VC
z-=%$-1_J_Je>*S`_Q4~8;_|M@><ZX?H!f*^?0EKmV5n4!3FaQ*0Il=v21j=v3OL!H
zj0B<ZDO@?eSC!g_1r92vd+s;7@j81qkmaA8NM0sQRfsBTqdFzmem*^eZeEmbw%xnW
zyXLfO9`0N&ezr%L`}XyJ?wX3DK+h&iwCmZO=OCR!e}*TVL++e@JQ_nnqMqJi6=hG|
z@%$WHYC9q}b{Gc^NxKQ*XFV|8)nm7-ILsHr=eSt)aXkYDpQ5X0Z%<$2;RM|}SO2xI
zyHgybj&}9re>;3vX~)TW&Lr#1uU8xet^$1!T?}1QjoT-S^jtPrpt-$1^OO*ZN{e<U
zMJ47Df1-Wr@X-6iZ0PNwfzeuuj4Q=+%WcKh7=G+4NP2VTH*z{j)qZLJvokbzzgIOi
z(g6RUd6lb!LwIGrQMh*Z9_gn0!J%yh=R&~d;1B@Zr52zodN5}a*8qR}_SwN9Y!41?
zn8fylzg_RHTkukGQ;(|xBrQyY%=5x>b?5vxe<@HlD`0tUGK6y>uq$uHiZ8{VX2G#T
z9#Y;&(bTv`q+R3i9Z&FtN6~ZeHDstvS%!HCXmLYNXWJ3{wa%I5u>0CPp-1WP%_+2L
z?1j8^uRlzov5sLQDuWlSm-f`<%Gs^m3;SqV?KjG&5M@KJTBoO-u(f)#_u&+-dpajB
ze_eFJ&xXj&=I?_$KZG>f4(;)2+tc<9?aEN@0T0h04)amB&tf;QdQWYqr?7drcPUXj
z{gX4dBNc+4AJ=;$D!{K7b**L|Qj|lH@GjwvefvO<`_+(__i^co{%!V?j4tlA>aKA(
zW18BI(wGN(Ksz_hEkQ8OwST3v`y>b2e=KUxtJn4)4?wUUYhT&!f4vj5_3sA<?fd?I
z*9O{1yb0v>;&bO01ScoGRL%8aixxbP9y;kkk~y$F^Kfhqv>c$NIG6_Bvk0FuY_^kd
zRQI|NcsGJJhuAfI-@^6FhX2;NIvj@+c+A|>FcR*Lc0y}wn`IrJw&~pNb|y_Te+-TD
z&_Ju+2uvOv0e>+4qOR>3ExFjt`>=8cJ1dd%(!Nl){z<+$?7HNUW~)yNdcnwZyAy4D
zVh$fk8oF@woH9*yDPDb`TqnhurGJDnfcnzBoI`jCVg#$}4%+h%h0+K#^lD4_9B^<Y
z1k<+t*I)M>#MrN^H&1Lk4XpS0e-`wcrp~M{+JY{fHEvHwzwMMGng87-3vyLY>dON1
zPMPNr+gJ3#0?iH$S?#v_+;HF@h9qb3)py1v+X<r`M2E^4UYrkFc8Bm2&0CP4Dnj|-
z-CdiyD4bDBEZd}{r<D00JQ4l!D}@PM-NDr@+~L57o_$MaH>g2$H=$ilB)_{kY4^5h
zGmBJ?Qe&L)!G8q+0RR6`O9u!!CRph45&!@uNs|$T9FqqfD3cD950mhf3bWRLRTK(2
zCRph45&!@uNs~2|gaUMdlMo&+la7=Ilkk-^0t|(d5FRg+UW5yiV3sNZsEU&i9u<?|
Qi5`=`mJkMVkpKVy035`=LjV8(

delta 4723
zcmV-(5{&K6uL9Mr0<d&$e>`n-+cuNm{VTSzry?gRJ@?!*{lHYN(>!q}H*MT$-0R)c
z84DsIi!n*C0O-b#qWkSPyLcf;QI4HtZtiZPFO~r83l{r^1>llvh|+7WU$Kg-hGl#P
z`12Q*-LPx?!RlI^Nl~3Qx)62H@LW_Cuk{SlqRz@T=P#>9Nc_DLfATLT^VcO)Yo7nY
z^m+zKlEg5F<Ks<o-0Gs-K0PqB8(CC^E^d*QzH9ied^Lkq-Kp|5TXAG-25H7BUOs1v
zBjMLIyW#jF#CqJ^Tpt(20hdHUc@nH?5YL5pX1@H)l`@lSnAFO*@8Dxnvf@|^QL1Cs
zG&QTZM=E*VW<8MWe?qU@Ws->s0l#5|h0jE-WwGo}Tk<;R^5U-HzacNb7kd4&E{mE!
zt81}`5fsWd29QhobybP{%C+Ex4-VmC%|W%RRq+nA0G8+YN3S`srYVbz;W7eF1;lEn
zU9XvjwJ381xEP=so)s(Gs}f=!JDxSoYeQoW6P$&b5b;H0e-wvgx`0g7w_Iuk3~UD|
zEsOe^Sc6+u7I}}p=?xGqh{ZlIaS;-NIb6CS$H$8QnW%OdO_IqBCRdZ08yg!IC0C-n
zwd%S6#hJ|3ToIvFTb8gYN+hY*3W^#QMyu2URziZPxe-#uw1-k_AvKtB1ehS5!V<s{
zK_4&$g~nNPe<q0ngGLH1h;9I>7Qmol4XzWjY|(f`Sc$yF(I|@Egiscin*FN|003~H
z<9QB|?SjA7TsonlqOzh^wsQu_X&gT#=uFf(Z0IX~ETR-5hf)>wb;;x8bmj+LrEq-A
zr4%wcTKrZUF4?kl(n#TGW4>*n78+;vhig$VKY`>Fe}v4rT;4&FrZd>SS;U0hDEa~g
z^4#)_I=Fyr&9fVtlwNZQ3|`-<(r9v_Am^*1=D86$E%NftE<dVdW!!=~=5<>sTnM;W
z7YbOZM5iS+SDNP|gFQREwu_#bAI{DUMUJE_He`iBkwf%A1}-#C;(Y+L64EVv216m>
z`UjlAf2G4slKbkbDCkVwEWS@STdNAmDa^sa!29<*1ft~hmX$4^`B8BU@86H1C)w=7
zZLN{tvhr11ExBBP<hqqczlj8CKGf7pUWq(M+msw3YYRnU!bXt1kzDcG_>usaq}5SE
z8QBi6+?GnG<54Q&6vQlO$m109S*U{7TwfS>e`uD^%s}ld<BpGYwnmoi7Fv;uVVxtu
zFAG#%uRnQ_ArdC<tW0nCof|;jB~&(6=YcR$l_6)E86U#5rGv8R&%^1EcN{hPj4(@}
zWwwf7T?CF}O=B|$nD;z%)a4We+V?9^s}%aNII0yqc>;g*g3X&4wy2UFnS-YXJ|-a;
zf5y>YBsYP{t3F&d)J7!3)FLwUO>YWEizEAedv&yN(A&ifeVPC1!^nzDUS~WvzQ-*+
zv&uMFW`dt8<lQ)icbe9Rv;>#E7WFqi8G;Tw0U9r{7`lXRRG=<?`H6!<`B7v&76FZC
zPHpc|*BSg(=(UYHh<5g2ypM=qhp<qYe`qWZSt}(vEoySYbLgG3B8H3<$fFlFH^HQV
z7SHAkMh&!pR&gi-cUqvGwRP@z=}~o_HC$ZMC1VnBX@%yaD_k0GLmfjDt>Ru2t(qGG
z-swg<L7~UwA>Y5Z-NtE#eW{zvwzIU_nCY(GTfkXI-<n!aWnW=BQ`ET{Xv>-ze~%$#
zab47`9EzD#cIf2DBN)~3PQ~#VlI)2w%uTw%l*r^ST@C=DpGkt$zzDRr@raVsIPK>`
zQF1!7$%cxzv5P`a2Sis$$hVQJA97+KcS51X9VPJ$?1K4N=st&V_VPb|evyL4WP+UV
znq@Z$JU41r-dT0U6wvKpV9gXZe=SR1DRhmBtbk&K7b&t;Wb2?|K-lIg1}rjvdPxvA
z#R>er=CxsOjcIAEywo7W?Vg4u_lslm*T0Ig1k*L3N+}+tvo`z;d>Q!wJJGyn!tgsP
zkYyyhfn0Es7_U_;oj+Syz6VqbMcjD99wRXGj5Doy)#x6Z#y42^#ELr5e`NRB7<JtG
z#@Hslol5nhq|*<{R~YfURVY^RE>K=-(20$f->1{5iM73O8Njlpxo~l9J_y`yFnD`E
zU%p*HCOOmG1)**|KrG+#Ov92Xo&#F<Eh|csqSq@SD~bR;V?JNLwPsv*+BVo780tUr
znoFiBm=LooDXPF<6e}><f12(AVk!_>kOw9OGs=8Y_ydu@iDv^wLd8ZyuDvB@tSots
zy8x0)5cT?&)EBvDF1nYjpFgE#G<m<u`z9Z#0lIxhpqEMcG4Tw>DgRrmbeK!mF8g<%
zt0C{c*G3p^3Ng{h!Un1vX>z!z6Oo?n&kqL-_9xlv!cT`Vj2-yfe`FudpAWEJ?P40?
z-hB??;`|@yDg21Jquckp49J=ZsY;=zLAGF0^F0{PgarPMRZYoLc)5Zf*gq*;lFRRJ
z8(mIyCd3|O#7m(-adJ|K%kl$iE!r|imwYBf@&o(lD2km}6|9I4R_T2*!q1>E&@;T5
zvxNA5QwTpJs}z17f4bM$_vv(}R`K4E+dceUh1$H?j!dI-{M%sr`pNnKm+bz3HoJF=
z_EC9#QEPt9C2DDBH`Ail)7jManQ#C4H}m(~zfU_&nzpW|wOv!?TGY<qRq>AJ*k`|*
z@xIl5{r2xu0@;q%>WrzwcPVV9OCd@`hJQJ$r?j=OTh3_uf5Z9tPt!o_J4iR@6vFW@
zUtNq3-%L$41xe+iU6!0?Pzt68B+q_2f3_EisrDkM(h>oC)L&ox`tsF}Bd~Lr=1lWx
zN{s=;Kb~E@*gswu6~YY0U%dQz48}<FPSeEjuNunoKQCUq9w)XeY9{ZdgX!!57;hBp
zLc0*i+<Apnf7O{$?wPfUUXc`a{jA_+z96-4Z6dhBrk&j!l}p`uf-k3On+Bo}at;Wg
zweTp%9h4hbC@4k7bXV}~`KEqxEC3q1K?nW+!voQ(P^s-PE{Lpq)vHsMY{^S=W|Z0i
z)HI0e6UiIg2jr6!BE|n~QKqSPhS4fP?jNdSgXlOBe{?>`%E?D#geNBuiAJZOWy^>7
zCI7Q6Br*Sy_&bE#o|Szhy3ojQ1NBU6srnUQY{spoES4>romw+#_8ayH^7uvFR>!~o
z^4#vkjFt)9vmw0Qo{^1oSeSGPg4x&E1A$!w6w9~$V-He2&nv1P4(Uq0n~4&w%=i^%
zbEG9Hf9xLS+5)c0g`U`2j99+4DO$gT-q?+|QO?bT${bg=8A-4dB3|m0mL3~nenNzg
z7Dt=pbPIDh+Vl^h-3dgzUBq#BMC4d{5M!A+y$(3bxyocg1nkrYC*6^Q@r2CcqtcKq
z|2t9%-K_2(WRP<p=SHh#{f%|bfd_K#HLTp5e+74__b&MtpS$1Z_SXruKU{&w_+an7
zqY#7NTs^{nJUKBn9cz!9$y+7rE6uL03XWfV7?5@P7^?jUW3Y)*%c0YG*<>$sOA76+
zJ0gp`g>4^<{1a`;rms;-(ufDf)w@hk3t6#pXfNXNHfH^(!Mtyu^~Iz<Xi}eIO<n2z
ze>Cl%&JEPOi`qu#(t%0$#y`pot+I)lpRX>)G_D5fdw|B#I2*nR<cp$x21V<y3}r4P
z+()PxqPgLBm`~s$iPhE7hL9wu+eLS<rs;%Y4z|mjJduRWO@{di66IyyOHa)bd~*U(
zO{L+U8l#6s-#O6M<e@6sF?VYtS0rkFe<fCFU_0bAZ9OmK{c||xU>AGWShG`I;$=+8
zvKgJGDCU{BQCGJkcg&hm^CGqUux(YW`QpSsyKRMwLVlbzA7@v6-_4rj)E`CJa{Kd=
zvl`k4mTh5g^Q;SH_c;l?c9m@l6**{79Crvfqbtc*$G&dj)NAHPT07WGT_!a(fBHS0
z#xt0j?MKC$s~djT>!DPrYuQ?a@s91`-dXR49@ae6oCABVSmU;0kz;()<I!3tNz?eE
z8-9n&WeWJ)5$9n3;%3^SbW{?PW<EJFLlVVx;&QDInyxk%?1BrObYSf)k<8WKwhMEw
zzQz6@venOKVA__NoecIG*TAZ5f5C7pCGw^f?jHQk-i<OE*&AyjnY)tWvH4J8`dy%K
zql4IWU)buS^9mnaTcN;v4|&t?$h*-egZTexJXEYX`+yYu5XVUXbJBB|pT>$gDYC|Y
zmGhPIGJkU7rvK!mBkwFVHFtc>R%r$)A#z7jbz6;CDmEK4MTMfDNjDr-f0sr)&_HSI
zz&@$4iS2W^q`zHdsmoJF@*`p;Oz*AV>hu{$7&!-{urY<G(7aNqna^mthHfo6?YvsB
z$#nq6(Z#*Z0I%~#6txbnPt@QVLA2Twb!3x~jAWA;5y>XA60$?RL-OT1DXLv>w&#Ry
zaE|CGblJeZ6~a-w4)hT&f6y$aox~h25n<=FRy()2O1hPQR|=M!B>M)wOW|m9?^B1!
zS6@+(b~!|Q&rwGd)8TA9Oxj@%W4Ma?gQzH0PDtq{DK>P7gZcY+B(s4o6R~@gS2me<
z#7Js0X_(er4(gwOL=$Y7T>tvZ^GW<Bj)SzjVUkri#88|Rb<^sJe|<3%zv%`lUFmMz
zP^6NwsMIb}9KUh;j(;{ZI`vaF)lJqkKk-{$p4;LqlN=aiqAWRGL`DswM<T2QSHLBW
ztvRP8o^FoOfl`j>R8O}{L`_={9lkIb5_Ko7(`#{M?~d40(&<}v%YwG8y$^iN^|Ku$
zmySXA*WXJ2E*;V|e?B1KwdV%HIe1T?xV$Ygo9CPF#w6{J9n0SD7;4q|1an(z1lC!0
zi_+ca1sv^9_5>mEIb1rtm$lmG3!JN%@43b0<aPRHB$qipmb^^bx)62L)$J*T@ayRj
zSQR%N==S?JJa^S;S3TV6L;Q3PG5783|Jqd*2Z5GNnrO53f1OM3y;ABUPFr?rybtyv
zA<%&Du#57bzC1ri;b4cv#)@&|D9De+&*wcB+T~++swn33@zW~o`Y@jXf=}Vqqqk>F
zF5?>n9cx^}H;wM*WRND=95G0njDJ^Y&&USGB%919P!0lfT^~eOW0&#b?qL=^3JlUv
zSJ5yJrl6>`f9Up$Dlt#3?1k?8URh;hFRY9kZKTM!QarcVDt3qA$G-KVmo<L7+DoeT
zQ~TeYp}AX(s%;Pk_&d$(Tpb+33-gV_wY%L#_jwNv?I<`C0yYPS0N{@9fi5D!l1Z4u
zzx?gnrw50yJvg*s62}+*4zW8_=B47MpVc{m7A8XGe|cfCy3<UX6eyb&usk;z!kG{_
zmDi%;Yw?$<2<?D}kk?YQ4bN?DEPSmHJXBEh==BaVWTq^`JOs43rU!zZ1b=ICraA1s
zsZHqVEqrqVT^f5XFWen>(`d9~T#3ryo!*75EnhmlwLAU~rq#Y2IEN@3d%HJ3>4mN8
z)!yeJf4CFs)s_3+1wR`CHzlrn_qXI}wmsM<C*4fDmx7gn+~Yhvf;h}a-EPuu(+3X<
z=O?hae|Ba|==e`g-M%|?dS0&Y4XMC?J!=}Z@GeC;FB0CEy0$mG=?SwMv+|)|dZc}u
zf{o$Dy~*2E4yR0WJ5iE(C&S(Qv=$(cbK~D4f9yUyfG&&L^R};hUJb#1tb0qi|BXU0
z*1wV()$jN3cXgnP#G8QLAU^kgK~Ntaq-t&sTh!o@^w3cclFX6&!S~1JfXfkBii2rj
zJ*)7-_GUW^rMlOIz?%_tImE8w`w?zlHvD(S)x$WRz+>i~hLLbr+6nEgPhCm6OXpG;
ze@WY+TjfyBV}e!#3C!-5fIohF)-?7cicD<bZCJVTbtQ6M+IP6tKFK!6O_wZ^c4-(5
z8YUIbZN1IT#2m2@H8kPqd&)G`g?RCfa-9@cmj2zth}W0qeGs~rAV#pJ>3Mtpu232s
z4ZTfKJ_j7#3BkM@|Mk}c12OvR`prYaf8GFVJ$?lJrlm9ci>{#$XO-K-(Qmut2<CtH
z!Gc`XgZj1ryw~P2#P+RlutBq;A+6o^A61R~TZH5kzWUCXWVc{3nuqhs7+#zWT6Bl-
zGp$>YpDOb5!Rw+fby2vYlvs92$$%;IpGG43<#!4*xV(kS8@NTmhn9Uq6>8)lDY~6e
z-P3I~OtH%qkP#iS9HmA%<AeVW00960P)i301ZItr_z?gA3`UbL2OP7GftC~s1ZItr
z_z?gA3`UbKmV^R(fRo{rHIwO<Gy)TZli`#vlTMd40<Vda;gl7VnwKsHd5{1A007nP
BN>u;=

diff --git a/pkg/importer/importer_test.go b/pkg/importer/importer_test.go
index a62092d5..e311dfef 100644
--- a/pkg/importer/importer_test.go
+++ b/pkg/importer/importer_test.go
@@ -219,6 +219,23 @@ func TestLoadProtobufFromTestFiles(t *testing.T) {
 	})
 }
 
+func TestLoadProtobufTestErrors(t *testing.T) {
+	t.Parallel()
+
+	runImportErrorTest(t, errTestConfig{
+		testConfig: testConfig{
+			name:          "TestLoadProtobufTestErrors",
+			testDir:       "proto/tests/errors",
+			testExtension: ".proto",
+			format:        Protobuf.Name,
+		},
+
+		cases: map[string]string{
+			"dulicate_enum": "enum with multiple names for value 0 found",
+		},
+	})
+}
+
 func TestLoadProtobufDirFromTestDir(t *testing.T) {
 	runImportDirEqualityTests(t, testConfig{
 		name:          "TestLoadProtobufDirFromTestDir",
diff --git a/pkg/importer/proto/proto_to_sysl.arrai b/pkg/importer/proto/proto_to_sysl.arrai
index b526ad82..40c04a2c 100644
--- a/pkg/importer/proto/proto_to_sysl.arrai
+++ b/pkg/importer/proto/proto_to_sysl.arrai
@@ -241,10 +241,21 @@ let type = \message \file \appName \imports \prefix
 # `enum` the enum being converted.
 # `file` the file the enum is located in.
 let enum = \enum \file
+    let fields = enum.@value.fields?:{} => (@: .@, @value: evalNumber(.@value.value), deprecated: .@value.options?('deprecated')?.b?:false);
+    let nested = fields nest|@,deprecated|e;
     (
         name: syslSafeName(enum.@),
         attrs: baseAttrs(enum, file),
-        enum: (items: enum.@value.fields?:{} => (@: .@, @value: evalNumber(.@value.value)))
+        enum: (items: (fields where \f
+            (let n = ((nested where .@value = f.@value) single).e;
+            cond {
+                n count = 1: true, # only entry for value
+                (n where !.deprecated) count > 1: //error($`enum with multiple names for value ${f.@value} found: ${(n where !.deprecated) => .@}`),
+                !f.deprecated: true, # this is the only non-deprecated
+                (n where !.deprecated) count = 1: false, # this is deprecated and there is a non-deprecated value
+                (n => .@ orderby .)(0) = f.@: true, # use the first (alphabetically) deprecated value
+                _: false, # skip it
+            })) => (@: .@, @value: .@value))
     );
 
 # `endpoint` creates an endpoint out of a rpc.
diff --git a/pkg/importer/proto/proto_to_sysl_test.arrai b/pkg/importer/proto/proto_to_sysl_test.arrai
index b8610a07..0afd54fd 100644
--- a/pkg/importer/proto/proto_to_sysl_test.arrai
+++ b/pkg/importer/proto/proto_to_sysl_test.arrai
@@ -1,4 +1,4 @@
-let (:import, ...) = //{./proto_parser};
+let (:import, :parseFile, ...) = //{./proto_parser};
 let (:module, ...) = //{./proto_to_sysl};
 
 (
@@ -21,5 +21,30 @@ let (:module, ...) = //{./proto_to_sysl};
             let actual = sysl.apps('SearchService').attrs('proto_options').a.elt(0).s;
             # Strings are not escaped in memory, only on serialization.
             //test.assert.equal(`php_namespace = Foo\Bar`, actual),
+    ),
+
+    enum: (
+        dupicates_filter:
+            let model = {'f.proto': parseFile(`
+                syntax = "proto3";
+                package enums;
+                enum e {
+                    E0 = 0;
+                    E1 = 1;
+                    E1_DEP = 1 [deprecated = true];
+                    E2 = 2;
+                    E2_DEP1 = 2 [deprecated = true];
+                    E2_DEP2 = 2 [deprecated = true];
+                    E3_DEP = 3 [deprecated = true];
+                    E4_DEP1 = 4 [deprecated = true];
+                    E4_DEP2 = 4 [deprecated = true];
+                    E5_DEP2 = 5 [deprecated = true];
+                    E5_DEP1 = 5 [deprecated = true];
+                }`
+            )};
+            let sysl = module((), model);
+            let expected = {'E0': 0, 'E1': 1, 'E2': 2, 'E3_DEP': 3, 'E4_DEP1': 4, 'E5_DEP1': 5};
+            let actual = sysl.apps('enums').types('e').enum.items;
+            //test.assert.equal(expected, actual),
     )
 )
diff --git a/pkg/importer/proto/tests/enum.proto b/pkg/importer/proto/tests/enum.proto
new file mode 100644
index 00000000..2b5ee52b
--- /dev/null
+++ b/pkg/importer/proto/tests/enum.proto
@@ -0,0 +1,15 @@
+syntax = "proto3";
+package enums;
+enum e {
+  E0 = 0;
+  E1 = 1;
+  E1_DEP = 1 [deprecated = true];
+  E2 = 2;
+  E2_DEP1 = 2 [deprecated = true];
+  E2_DEP2 = 2 [deprecated = true];
+  E3_DEP = 3 [deprecated = true];
+  E4_DEP1 = 4 [deprecated = true];
+  E4_DEP2 = 4 [deprecated = true];
+  E5_DEP2 = 5 [deprecated = true];
+  E5_DEP1 = 5 [deprecated = true];
+}
diff --git a/pkg/importer/proto/tests/enum.sysl b/pkg/importer/proto/tests/enum.sysl
new file mode 100644
index 00000000..5392104c
--- /dev/null
+++ b/pkg/importer/proto/tests/enum.sysl
@@ -0,0 +1,12 @@
+# Code generated by Sysl. DO NOT EDIT.
+
+enums:
+    @package = "enums"
+
+    !enum e [source_path="proto/tests/enum.proto"]:
+        E0 : 0
+        E1 : 1
+        E2 : 2
+        E3_DEP : 3
+        E4_DEP1 : 4
+        E5_DEP1 : 5
diff --git a/pkg/importer/proto/tests/errors/dulicate_enum.proto b/pkg/importer/proto/tests/errors/dulicate_enum.proto
new file mode 100644
index 00000000..81b9ed89
--- /dev/null
+++ b/pkg/importer/proto/tests/errors/dulicate_enum.proto
@@ -0,0 +1,6 @@
+syntax = "proto3";
+package enums;
+enum e {
+  E0 = 0;
+  E1 = 0;
+}