Merge branch 'master' of git.moenext.com:mycard/windbot

.github/workflows/test-build.yml

@@ -8,7 +8,7 @@ on:
 
 jobs:
   build:
-    runs-on: windows-2019
+    runs-on: windows-2022
 
     env:
       Solution_Name: WindBot.sln

App.config

 <?xml version="1.0" encoding="utf-8"?>
 <configuration>
     <startup> 
-        <supportedRuntime version="v4.0" sku=".NETFramework,Version=v4.0"/>
+        <supportedRuntime version="v4.0" sku=".NETFramework,Version=v4.8"/>
     </startup>
 </configuration>

BotWrapper/BotWrapper.csproj

-<?xml version="1.0" encoding="utf-8"?>
-<Project ToolsVersion="14.0" DefaultTargets="Build" xmlns="http://schemas.microsoft.com/developer/msbuild/2003">
-  <Import Project="$(MSBuildExtensionsPath)\$(MSBuildToolsVersion)\Microsoft.Common.props" Condition="Exists('$(MSBuildExtensionsPath)\$(MSBuildToolsVersion)\Microsoft.Common.props')" />
-  <PropertyGroup>
-    <Configuration Condition=" '$(Configuration)' == '' ">Debug</Configuration>
-    <Platform Condition=" '$(Platform)' == '' ">AnyCPU</Platform>
-    <ProjectGuid>{0665CA3B-C14F-40EC-ABFB-AD46A695F5A3}</ProjectGuid>
-    <OutputType>WinExe</OutputType>
-    <AppDesignerFolder>Properties</AppDesignerFolder>
-    <RootNamespace>BotWrapper</RootNamespace>
-    <AssemblyName>Bot</AssemblyName>
-    <TargetFrameworkVersion>v4.0</TargetFrameworkVersion>
-    <FileAlignment>512</FileAlignment>
-    <TargetFrameworkProfile />
-  </PropertyGroup>
-  <PropertyGroup Condition=" '$(Configuration)|$(Platform)' == 'Debug|x86'">
-    <PlatformTarget>x86</PlatformTarget>
-    <DebugSymbols>true</DebugSymbols>
-    <DebugType>full</DebugType>
-    <Optimize>false</Optimize>
-    <OutputPath>bin\Debug\</OutputPath>
-    <DefineConstants>DEBUG;TRACE</DefineConstants>
-    <ErrorReport>prompt</ErrorReport>
-    <WarningLevel>4</WarningLevel>
-  </PropertyGroup>
-  <PropertyGroup Condition=" '$(Configuration)|$(Platform)' == 'Release|x86'">
-    <PlatformTarget>x86</PlatformTarget>
-    <DebugType>pdbonly</DebugType>
-    <Optimize>true</Optimize>
-    <OutputPath>bin\Release\</OutputPath>
-    <DefineConstants>TRACE</DefineConstants>
-    <ErrorReport>prompt</ErrorReport>
-    <WarningLevel>4</WarningLevel>
-  </PropertyGroup>
-  <PropertyGroup>
-    <StartupObject>BotWrapper.BotWrapper</StartupObject>
-  </PropertyGroup>
-  <PropertyGroup>
-    <ApplicationIcon>WindBot.ico</ApplicationIcon>
-  </PropertyGroup>
-  <ItemGroup>
-    <Reference Include="System" />
-    <Reference Include="System.Data" />
-    <Reference Include="System.Xml" />
-  </ItemGroup>
-  <ItemGroup>
-    <Compile Include="BotWrapper.cs" />
-    <Compile Include="Properties\AssemblyInfo.cs" />
-  </ItemGroup>
-  <ItemGroup>
-    <None Include="app.config" />
-    <None Include="bot.conf">
-      <CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
-    </None>
-  </ItemGroup>
-  <ItemGroup>
-    <Content Include="WindBot.ico" />
-  </ItemGroup>
-  <Import Project="$(MSBuildToolsPath)\Microsoft.CSharp.targets" />
+<?xml version="1.0" encoding="utf-8"?>
+<Project ToolsVersion="14.0" DefaultTargets="Build" xmlns="http://schemas.microsoft.com/developer/msbuild/2003">
+  <Import Project="$(MSBuildExtensionsPath)\$(MSBuildToolsVersion)\Microsoft.Common.props" Condition="Exists('$(MSBuildExtensionsPath)\$(MSBuildToolsVersion)\Microsoft.Common.props')" />
+  <PropertyGroup>
+    <Configuration Condition=" '$(Configuration)' == '' ">Debug</Configuration>
+    <Platform Condition=" '$(Platform)' == '' ">AnyCPU</Platform>
+    <ProjectGuid>{0665CA3B-C14F-40EC-ABFB-AD46A695F5A3}</ProjectGuid>
+    <OutputType>WinExe</OutputType>
+    <AppDesignerFolder>Properties</AppDesignerFolder>
+    <RootNamespace>BotWrapper</RootNamespace>
+    <AssemblyName>Bot</AssemblyName>
+    <TargetFrameworkVersion>v4.8</TargetFrameworkVersion>
+    <FileAlignment>512</FileAlignment>
+    <TargetFrameworkProfile />
+  </PropertyGroup>
+  <PropertyGroup Condition=" '$(Configuration)|$(Platform)' == 'Debug|x86'">
+    <PlatformTarget>x86</PlatformTarget>
+    <DebugSymbols>true</DebugSymbols>
+    <DebugType>full</DebugType>
+    <Optimize>false</Optimize>
+    <OutputPath>bin\Debug\</OutputPath>
+    <DefineConstants>DEBUG;TRACE</DefineConstants>
+    <ErrorReport>prompt</ErrorReport>
+    <WarningLevel>4</WarningLevel>
+    <Prefer32Bit>false</Prefer32Bit>
+  </PropertyGroup>
+  <PropertyGroup Condition=" '$(Configuration)|$(Platform)' == 'Release|x86'">
+    <PlatformTarget>x86</PlatformTarget>
+    <DebugType>pdbonly</DebugType>
+    <Optimize>true</Optimize>
+    <OutputPath>bin\Release\</OutputPath>
+    <DefineConstants>TRACE</DefineConstants>
+    <ErrorReport>prompt</ErrorReport>
+    <WarningLevel>4</WarningLevel>
+    <Prefer32Bit>false</Prefer32Bit>
+  </PropertyGroup>
+  <PropertyGroup>
+    <StartupObject>BotWrapper.BotWrapper</StartupObject>
+  </PropertyGroup>
+  <PropertyGroup>
+    <ApplicationIcon>WindBot.ico</ApplicationIcon>
+  </PropertyGroup>
+  <ItemGroup>
+    <Reference Include="System" />
+    <Reference Include="System.Data" />
+    <Reference Include="System.Xml" />
+  </ItemGroup>
+  <ItemGroup>
+    <Compile Include="BotWrapper.cs" />
+    <Compile Include="Properties\AssemblyInfo.cs" />
+  </ItemGroup>
+  <ItemGroup>
+    <None Include="app.config" />
+    <None Include="bot.conf">
+      <CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
+    </None>
+  </ItemGroup>
+  <ItemGroup>
+    <Content Include="WindBot.ico" />
+  </ItemGroup>
+  <Import Project="$(MSBuildToolsPath)\Microsoft.CSharp.targets" />
   <!-- To modify your build process, add your task inside one of the targets below and uncomment it. 
        Other similar extension points exist, see Microsoft.Common.targets.
   <Target Name="BeforeBuild">
   </Target>
   <Target Name="AfterBuild">
   </Target>
-  -->
+  -->
 </Project>
\ No newline at end of file

BotWrapper/Properties/AssemblyInfo.cs

@@ -10,7 +10,7 @@ using System.Runtime.InteropServices;
 [assembly: AssemblyConfiguration("")]
 [assembly: AssemblyCompany("IceYGO")]
 [assembly: AssemblyProduct("WindBot")]
-[assembly: AssemblyCopyright("Copyright © IceYGO 2017")]
+[assembly: AssemblyCopyright("Copyright © IceYGO 2015-2025")]
 [assembly: AssemblyTrademark("")]
 [assembly: AssemblyCulture("")]
 

BotWrapper/app.config

-<?xml version="1.0" encoding="utf-8"?>
-<configuration>
-<startup><supportedRuntime version="v4.0" sku=".NETFramework,Version=v4.0"/></startup></configuration>
+<?xml version="1.0" encoding="utf-8"?>
+<configuration>
+<startup><supportedRuntime version="v4.0" sku=".NETFramework,Version=v4.8"/></startup></configuration>

Game/AI/Decks/AlbazExecutor.cs

@@ -2186,7 +2186,7 @@ namespace WindBot.Game.AI.Decks
 
         public override int OnSelectOption(IList<int> options)
         {
-            ClientCard currentSolvingChain = Duel.GetCurrentSolvingChainCard();
+            ChainInfo currentSolvingChain = Duel.GetCurrentSolvingChainInfo();
             if (currentSolvingChain != null)
             {
                 // 1190=Add to Hand, 1152=Special Summon
@@ -2246,8 +2246,8 @@ namespace WindBot.Game.AI.Decks
 
         public override int OnSelectPlace(int cardId, int player, CardLocation location, int available)
         {
-            ClientCard currentSovingChain = Duel.GetCurrentSolvingChainCard();
-            if (currentSovingChain != null && currentSovingChain.Controller == 0 && currentSovingChain.IsCode(CardId.SprindTheIrondashDragon))
+            ChainInfo currentSovingChain = Duel.GetCurrentSolvingChainInfo();
+            if (currentSovingChain != null && currentSovingChain.ActivatePlayer == 0 && currentSovingChain.IsCode(CardId.SprindTheIrondashDragon))
             {
                 return SprindTheIrondashDragonMoveZone(available, null);
             }
@@ -2435,21 +2435,21 @@ namespace WindBot.Game.AI.Decks
 
         public override void OnChainSolved(int chainIndex)
         {
-            ClientCard currentCard = Duel.GetCurrentSolvingChainCard();
+            ChainInfo currentCard = Duel.GetCurrentSolvingChainInfo();
             if (currentCard != null)
             {
                 // if activation is negated, it can activate again.
-                if (currentCard.Controller == 0)
+                if (currentCard.ActivatePlayer == 0)
                 {
                     List<int> activateCheck = new List<int> { CardId.NadirServant, CardId.FusionDeployment, CardId.BrandedFusion, CardId.BrandedInRed };
                     if (currentCard.IsCode(activateCheck))
                     {
-                        activatedCardIdList.Add(currentCard.Id);
+                        activatedCardIdList.Add(currentCard.ActivateId);
                     }
                 }
                 if (!Duel.IsCurrentSolvingChainNegated())
                 {
-                    if (currentCard.Controller == 1)
+                    if (currentCard.ActivatePlayer == 1)
                     {
                         if (currentCard.IsCode(_CardId.MaxxC))
                             enemyActivateMaxxC = true;
@@ -2458,7 +2458,7 @@ namespace WindBot.Game.AI.Decks
                         if (currentCard.IsCode(CardId.DimensionShifter))
                             dimensionShifterCount = 2;
                     }
-                    if (currentCard.Controller == 0 && currentCard.IsCode(CardId.NadirServant))
+                    if (currentCard.ActivatePlayer == 0 && currentCard.IsCode(CardId.NadirServant))
                     {
                         nadirActivated = true;
                     }

Game/AI/Decks/DogmatikaExecutor.cs

@@ -1153,8 +1153,8 @@ namespace WindBot.Game.AI.Decks
 
         public override void OnChainSolved(int chainIndex)
         {
-            ClientCard currentCard = Duel.GetCurrentSolvingChainCard();
-            if (currentCard != null && !Duel.IsCurrentSolvingChainNegated() && currentCard.Controller == 1)
+            ChainInfo currentCard = Duel.GetCurrentSolvingChainInfo();
+            if (currentCard != null && !Duel.IsCurrentSolvingChainNegated() && currentCard.ActivatePlayer == 1)
             {
                 if (currentCard.IsCode(_CardId.MaxxC))
                     enemyActivateMaxxC = true;
@@ -1166,7 +1166,7 @@ namespace WindBot.Game.AI.Decks
                 {
                     for (int i = 0; i < 5; ++i)
                     {
-                        if (Enemy.SpellZone[i] == currentCard)
+                        if (Enemy.SpellZone[i] == currentCard.RelatedCard)
                         {
                             infiniteImpermanenceList.Add(4 - i);
                             break;

Game/AI/Decks/ExosisterExecutor.cs

@@ -771,8 +771,8 @@ namespace WindBot.Game.AI.Decks
 
         public override void OnChainSolved(int chainIndex)
         {
-            ClientCard currentCard = Duel.GetCurrentSolvingChainCard();
-            if (currentCard != null && !Duel.IsCurrentSolvingChainNegated() && currentCard.Controller == 1)
+            ChainInfo currentCard = Duel.GetCurrentSolvingChainInfo();
+            if (currentCard != null && !Duel.IsCurrentSolvingChainNegated() && currentCard.ActivatePlayer == 1)
             {
                 if (currentCard.IsCode(_CardId.MaxxC))
                     enemyActivateMaxxC = true;
@@ -782,7 +782,7 @@ namespace WindBot.Game.AI.Decks
                 {
                     for (int i = 0; i < 5; ++i)
                     {
-                        if (Enemy.SpellZone[i] == currentCard)
+                        if (Enemy.SpellZone[i] == currentCard.RelatedCard)
                         {
                             infiniteImpermanenceList.Add(4 - i);
                             break;

Game/AI/Decks/HorusExecutor.cs

@@ -153,7 +153,7 @@ namespace WindBot.Game.AI.Decks
         {
             // We should summon Horus the Black Flame Dragon LV6 if he can lvlup.
             if (Enemy.GetMonsterCount() != 0 && !Util.IsAllEnemyBetterThanValue(2300 - 1, false))
-                foreach (ClientCard card in Main.SummonableCards)
+                foreach (ClientCard card in Duel.MainPhase.SummonableCards)
                     if (card.IsCode(11224103))
                         return false;
 

Game/AI/Decks/LabrynthExecutor.cs

@@ -1646,17 +1646,17 @@ namespace WindBot.Game.AI.Decks
 
         public override void OnChainSolved(int chainIndex)
         {
-            ClientCard currentCard = Duel.GetCurrentSolvingChainCard();
+            ChainInfo currentCard = Duel.GetCurrentSolvingChainInfo();
             if (currentCard != null && !Duel.IsCurrentSolvingChainNegated())
             {
-                if (currentCard.Controller == 1)
+                if (currentCard.ActivatePlayer == 1)
                 {
                     if (currentCard.IsCode(_CardId.MaxxC))
                         enemyActivateMaxxC = true;
                     if (currentCard.IsCode(CardId.DimensionShifter))
                         dimensionShifterCount = 2;
                 }
-                if (currentCard.Controller == 0)
+                if (currentCard.ActivatePlayer == 0)
                 {
                     if (currentCard.IsCode(CardId.LabrynthCooclock))
                         cooclockAffected = true;

Game/AI/Decks/OldSchoolExecutor.cs

@@ -69,10 +69,10 @@ namespace WindBot.Game.AI.Decks
             if (_lastDoubleSummon == Duel.Turn)
                 return false;
 
-            if (Main.SummonableCards.Count == 0)
+            if (Duel.MainPhase.SummonableCards.Count == 0)
                 return false;
 
-            if (Main.SummonableCards.Count == 1 && Main.SummonableCards[0].Level < 5)
+            if (Duel.MainPhase.SummonableCards.Count == 1 && Duel.MainPhase.SummonableCards[0].Level < 5)
             {
                 bool canTribute = false;
                 foreach (ClientCard handCard in Bot.Hand)

Game/AI/Decks/RyzealExecutor.cs

@@ -64,6 +64,7 @@ namespace WindBot.Game.AI.Decks
             AddExecutor(ExecutorType.Activate, _CardId.GhostOgreAndSnowRabbit, GhostOgreAndSnowRabbitActivate);
             AddExecutor(ExecutorType.Activate, _CardId.AshBlossom, AshBlossomActivate);
 
+            AddExecutor(ExecutorType.Activate, CardId.RyzealCross, RyzealCrossActivateCard);
             AddExecutor(ExecutorType.Activate, _CardId.EvilswarmExcitonKnight, EvilswarmExcitonKnightActivate);
             AddExecutor(ExecutorType.Activate, CardId.RyzealDeadnader, RyzealDeadnaderActivate);
             AddExecutor(ExecutorType.Activate, CardId.RyzealDuodrive, RyzealDuodriveActivate);
@@ -82,7 +83,6 @@ namespace WindBot.Game.AI.Decks
             AddExecutor(ExecutorType.Activate, CardId.Bonfire, BonfireActivate);
             AddExecutor(ExecutorType.Activate, CardId.DonnerDaggerFurHire, DonnerDaggerFurHireActivate);
             AddExecutor(ExecutorType.Activate, CardId.Number60DugaresTheTimeless, Number60DugaresTheTimelessActivate);
-            AddExecutor(ExecutorType.Activate, CardId.RyzealCross, RyzealCrossActivateCard);
             AddExecutor(ExecutorType.Activate, CardId.TripleTacticsTalent, TripleTacticsTalentActivate);
             AddExecutor(ExecutorType.Activate, CardId.Bonfire, BonfireActivateToSearchNecessary);
             AddExecutor(ExecutorType.Activate, CardId.SeventhTachyon, SeventhTachyonActivate);
@@ -372,6 +372,26 @@ namespace WindBot.Game.AI.Decks
             return true;
         }
 
+        public bool CheckCardShouldNegate(ChainInfo chainInfo)
+        {
+            if (chainInfo == null) return false;
+            ClientCard card = chainInfo.RelatedCard;
+
+            if (card == null) return false;
+            if (card.IsMonster() && card.HasSetcode(SetcodeTimeLord) && Duel.Phase == DuelPhase.Standby) return false;
+            if (NotToNegateIdList.Contains(card.Id)) return false;
+            if (card.HasSetcode(_Setcode.Danger) && card.Location == CardLocation.Hand) return false;
+            if (card.IsMonster() && chainInfo.HasLocation(CardLocation.MonsterZone) && chainInfo.HasPosition(CardPosition.Defence))
+            {
+                if (Enemy.MonsterZone.Any(c => CheckNumber41(c)) || Bot.MonsterZone.Any(c => CheckNumber41(c))) return false;
+            }
+            if (DefaultCheckWhetherCardIsNegated(card)) return false;
+            if (Duel.Player == 1 && card.IsCode(_CardId.MulcharmyPurulia, _CardId.MulcharmyFuwalos, _CardId.MulcharmyNyalus)) return false;
+            if (card.IsDisabled()) return false;
+
+            return true;
+        }
+
         /// <summary>
         /// Check whether bot is at advantage.
         /// </summary>
@@ -901,7 +921,7 @@ namespace WindBot.Game.AI.Decks
 
         public override IList<ClientCard> OnSelectCard(IList<ClientCard> cards, int min, int max, int hint, bool cancelable)
         {
-            ClientCard currentSolvingChain = Duel.GetCurrentSolvingChainCard();
+            ChainInfo currentSolvingChain = Duel.GetCurrentSolvingChainInfo();
             if (currentSolvingChain != null)
             {
                 if (botSolvingCross)
@@ -933,7 +953,7 @@ namespace WindBot.Game.AI.Decks
                     }
                 }
 
-                if (currentSolvingChain.Controller == 1 && currentSolvingChain.IsCode(_CardId.EvenlyMatched))
+                if (currentSolvingChain.ActivatePlayer == 1 && currentSolvingChain.IsCode(_CardId.EvenlyMatched))
                 {
                     Logger.DebugWriteLine("=== Evenly Matched activated.");
                     List<ClientCard> banishList = new List<ClientCard>();
@@ -975,7 +995,7 @@ namespace WindBot.Game.AI.Decks
                     return Util.CheckSelectCount(banishList, cards, min, max);
                 }
 
-                if (currentSolvingChain.Controller == 0)
+                if (currentSolvingChain.ActivatePlayer == 0)
                 {
                     if (hint == HintMsg.AddToHand)
                     {
@@ -1681,31 +1701,31 @@ namespace WindBot.Game.AI.Decks
         public override void OnChainSolved(int chainIndex)
         {
             botSolvingCross = false;
-            ClientCard currentCard = Duel.GetCurrentSolvingChainCard();
-            if (currentCard != null)
+            ChainInfo currentChain = Duel.GetCurrentSolvingChainInfo();
+            if (currentChain != null && !Duel.IsCurrentSolvingChainNegated())
             {
                 if (!Duel.IsCurrentSolvingChainNegated())
                 {
-                    if (currentCard.IsCode(_CardId.LockBird))
+                    if (currentChain.IsCode(_CardId.LockBird))
                         lockBirdSolved = true;
-                    if (currentCard.IsCode(_CardId.DimensionShifter))
+                    if (currentChain.IsCode(_CardId.DimensionShifter))
                         dimensionShifterCount = 2;
-                    if (currentCard.Controller == 1)
+                    if (currentChain.ActivatePlayer == 1)
                     {
-                        if (currentCard.IsCode(_CardId.MaxxC))
+                        if (currentChain.IsCode(_CardId.MaxxC))
                             enemyActivateMaxxC = true;
-                        if (currentCard.IsCode(_CardId.MulcharmyPurulia))
+                        if (currentChain.IsCode(_CardId.MulcharmyPurulia))
                             enemyActivatePurulia = true;
-                        if (currentCard.IsCode(_CardId.MulcharmyFuwalos))
+                        if (currentChain.IsCode(_CardId.MulcharmyFuwalos))
                             enemyActivateFuwalos = true;
-                        if (currentCard.IsCode(_CardId.MulcharmyNyalus))
+                        if (currentChain.IsCode(_CardId.MulcharmyNyalus))
                             enemyActivateNyalus = true;
                     }
-                    if (currentCard.Controller == 0)
+                    if (currentChain.ActivatePlayer == 0)
                     {
                         foreach (int checkId in CheckBotSolvedList)
                         {
-                            if (currentCard.IsCode(checkId))
+                            if (currentChain.IsCode(checkId))
                             {
                                 botSolvedCardIdList.Add(checkId);
                             }
@@ -2758,10 +2778,10 @@ namespace WindBot.Game.AI.Decks
             // whether to negate by cross
             if (ActivateDescription == Util.GetStringId(CardId.RyzealCross, 3))
             {
-                ClientCard currentSolvingChain = Duel.GetCurrentSolvingChainCard();
-                if (currentSolvingChain != null && !Duel.IsCurrentSolvingChainNegated())
+                ChainInfo currentChainInfo = Duel.GetCurrentSolvingChainInfo();
+                if (currentChainInfo != null && !Duel.IsCurrentSolvingChainNegated())
                 {
-                    if (CheckCardShouldNegate(currentSolvingChain))
+                    if (CheckCardShouldNegate(currentChainInfo))
                     {
                         Logger.DebugWriteLine("** cross negate");
                         activatedCardIdList.Add(CardId.RyzealCross + 2);
@@ -2780,6 +2800,11 @@ namespace WindBot.Game.AI.Decks
                     Bot.Banished.Any(c => c != null && c.IsFaceup() && c.HasSetcode(SetcodeRyzeal) && (c.IsCanRevive() || !c.HasType(CardType.Xyz))));
             flag |= Bot.MonsterZone.Count(c => c != null && c.IsFaceup() && c.HasType(CardType.Xyz) && c.HasSetcode(SetcodeRyzeal) && (c.Overlays.Count() > 0 || canSetMaterial)) > 0;
 
+            if (Duel.MainPhase.SpecialSummonableCards.Any(c => c.IsCode(CardId.RyzealDuodrive)))
+            {
+                flag |= RyzealDuodriveSpSummonCheck();
+            }
+
             return flag;
         }
 

Game/AI/Decks/SwordsoulExecutor.cs

@@ -695,8 +695,8 @@ namespace WindBot.Game.AI.Decks
 
         public override void OnChainSolved(int chainIndex)
         {
-            ClientCard currentCard = Duel.GetCurrentSolvingChainCard();
-            if (currentCard != null && !Duel.IsCurrentSolvingChainNegated() && currentCard.Controller == 1)
+            ChainInfo currentCard = Duel.GetCurrentSolvingChainInfo();
+            if (currentCard != null && !Duel.IsCurrentSolvingChainNegated() && currentCard.ActivatePlayer == 1)
             {
                 if (currentCard.IsCode(_CardId.MaxxC))
                     enemyActivateMaxxC = true;
@@ -706,7 +706,7 @@ namespace WindBot.Game.AI.Decks
                 {
                     for (int i = 0; i < 5; ++i)
                     {
-                        if (Enemy.SpellZone[i] == currentCard)
+                        if (Enemy.SpellZone[i] == currentCard.RelatedCard)
                         {
                             infiniteImpermanenceList.Add(4 - i);
                             break;

Game/AI/Decks/WitchcraftExecutor.cs

@@ -201,8 +201,8 @@ namespace WindBot.Game.AI.Decks
 
         public override void OnChainSolved(int chainIndex)
         {
-            ClientCard currentCard = Duel.GetCurrentSolvingChainCard();
-            if (currentCard != null && currentCard.Controller == 1)
+            ChainInfo currentCard = Duel.GetCurrentSolvingChainInfo();
+            if (currentCard != null && currentCard.ActivatePlayer == 1)
             {
                 if (Duel.IsCurrentSolvingChainNegated())
                 {
@@ -212,16 +212,16 @@ namespace WindBot.Game.AI.Decks
                         if (Bot.MonsterZone.GetFirstMatchingCard(c => c.HasRace(CardRace.SpellCaster) && c.IsFaceup()) != null
                             && Bot.HasInSpellZone(CardId.MagicianRightHand, true))
                         {
-                            Logger.DebugWriteLine("MagicianRightHand negate: " + currentCard.Name ?? "???");
+                            Logger.DebugWriteLine("MagicianRightHand negate: " + currentCard.RelatedCard.Name ?? "???");
                             MagicianRightHand_used = true;
                         }
                     }
-                    if (!MagiciansLeftHand_used && currentCard.IsTrap() && currentCard.Controller == 1)
+                    if (!MagiciansLeftHand_used && currentCard.IsTrap() && currentCard.ActivatePlayer == 1)
                     {
                         if (Bot.MonsterZone.GetFirstMatchingCard(c => c.HasRace(CardRace.SpellCaster) && c.IsFaceup()) != null
                             && Bot.HasInSpellZone(CardId.MagiciansLeftHand, true))
                         {
-                            Logger.DebugWriteLine("MagiciansLeftHand negate: " + currentCard.Name ?? "???");
+                            Logger.DebugWriteLine("MagiciansLeftHand negate: " + currentCard.RelatedCard.Name ?? "???");
                             MagiciansLeftHand_used = true;
                         }
                     }
@@ -236,7 +236,7 @@ namespace WindBot.Game.AI.Decks
                     {
                         for (int i = 0; i < 5; ++i)
                         {
-                            if (Enemy.SpellZone[i] == currentCard)
+                            if (Enemy.SpellZone[i] == currentCard.RelatedCard)
                             {
                                 Impermanence_list.Add(4 - i);
                                 break;

Game/AI/Executor.cs

@@ -16,9 +16,6 @@ namespace WindBot.Game.AI
         public GameAI AI { get; private set; }
         public AIUtil Util { get; private set; }
 
-        protected MainPhase Main { get; private set; }
-        protected BattlePhase Battle { get; private set; }
-
         protected ExecutorType Type { get; private set; }
         protected ClientCard Card { get; private set; }
         protected int ActivateDescription { get; private set; }
@@ -247,16 +244,6 @@ namespace WindBot.Game.AI
             return;
         }
 
-        public void SetMain(MainPhase main)
-        {
-            Main = main;
-        }
-
-        public void SetBattle(BattlePhase battle)
-        {
-            Battle = battle;
-        }
-
         /// <summary>
         /// Set global variables Type, Card, ActivateDescription for Executor
         /// </summary>

Game/ChainInfo.cs

+using System;
+using System.Collections.Generic;
+using System.IO;
+using System.Linq;
+using YGOSharp.OCGWrapper;
+using YGOSharp.OCGWrapper.Enums;
+
+namespace WindBot.Game
+{
+    public class ChainInfo
+    {
+        public ClientCard RelatedCard { get; private set; }
+        public int ActivatePlayer { get; private set; }
+        public int ActivateId { get; private set; }
+        public int ActivateController { get; private set; }
+        public int ActivatePosition { get; private set; }
+        public int ActivateSequence { get; private set; }
+        public CardLocation ActivateLocation { get; private set; }
+        public int ActivateLevel { get; private set; }
+        public int ActivateRank { get; private set; }
+        public int ActivateType { get; private set; }
+        public int ActivateRace { get; private set; }
+        public int ActivateAttack { get; private set; }
+        public int ActivateDefense { get; private set; }
+        public bool IsSpecialSummoned { get; private set; }
+        public int ActivateDescription { get; private set; }
+
+        public ChainInfo(ClientCard card)
+            : this(card, card.Controller, 0)
+        {
+        }
+
+        public ChainInfo(ClientCard card, int player, int desc)
+        {
+            RelatedCard = card;
+            ActivatePlayer = player;
+            ActivateId = card.Id;
+            ActivateController = card.Controller;
+            ActivatePosition = card.Position;
+            ActivateSequence = card.Sequence;
+            ActivateLocation = card.Location;
+            ActivateLevel = card.Level;
+            ActivateRank = card.Rank;
+            ActivateType = card.Type;
+            ActivateRace = card.Race;
+            ActivateAttack = card.Attack;
+            ActivateDefense = card.Defense;
+            ActivateAttack = card.Attack;
+            ActivateDefense = card.Defense;
+            IsSpecialSummoned = card.IsSpecialSummoned;
+            ActivateDescription = desc;
+        }
+
+        public bool HasPosition(CardPosition position)
+        {
+            return (ActivatePosition & (int)position) != 0;
+        }
+
+        public bool HasLocation(CardLocation location)
+        {
+            return ((int)ActivateLocation & (int)location) != 0;
+        }
+
+        public bool IsCode(int id)
+        {
+            return RelatedCard != null && RelatedCard.IsCode(id);
+        }
+
+        public bool IsCode(IList<int> ids)
+        {
+            return RelatedCard != null && RelatedCard.IsCode(ids);
+        }
+
+        public bool IsCode(params int[] ids)
+        {
+            return RelatedCard != null && RelatedCard.IsCode(ids);
+        }
+
+        public bool HasType(CardType type)
+        {
+            return RelatedCard != null && (RelatedCard.Type & (int)type) != 0;
+        }
+        public bool IsSpell()
+        {
+            return HasType(CardType.Spell);
+        }
+
+        public bool IsTrap()
+        {
+            return HasType(CardType.Trap);
+        }
+    }
+}

Game/Duel.cs

@@ -20,6 +20,7 @@ namespace WindBot.Game
         public int LastChainPlayer { get; set; }
         public CardLocation LastChainLocation { get; set; }
         public IList<ClientCard> CurrentChain { get; set; }
+        public IList<ChainInfo> CurrentChainInfo { get; set; }
         public IList<ClientCard> ChainTargets { get; set; }
         public IList<ClientCard> LastChainTargets { get; set; }
         public IList<ClientCard> ChainTargetOnly { get; set; }
@@ -37,6 +38,7 @@ namespace WindBot.Game
             LastChainPlayer = -1;
             LastChainLocation = 0;
             CurrentChain = new List<ClientCard>();
+            CurrentChainInfo = new List<ChainInfo>();
             ChainTargets = new List<ClientCard>();
             LastChainTargets = new List<ClientCard>();
             ChainTargetOnly = new List<ClientCard>();
@@ -45,6 +47,8 @@ namespace WindBot.Game
             LastSummonedCards = new List<ClientCard>();
             SolvingChainIndex = 0;
             NegatedChainIndexList = new List<int>();
+            MainPhase = new MainPhase();
+            BattlePhase = new BattlePhase();
         }
 
         public ClientCard GetCard(int player, CardLocation loc, int seq)
@@ -180,6 +184,12 @@ namespace WindBot.Game
             return CurrentChain[SolvingChainIndex - 1];
         }
 
+        public ChainInfo GetCurrentSolvingChainInfo()
+        {
+            if (SolvingChainIndex == 0 || SolvingChainIndex > CurrentChainInfo.Count) return null;
+            return CurrentChainInfo[SolvingChainIndex - 1];
+        }
+
         public bool IsCurrentSolvingChainNegated()
         {
             return SolvingChainIndex > 0 && NegatedChainIndexList.Contains(SolvingChainIndex);

Game/GameAI.cs

@@ -193,7 +193,6 @@ namespace WindBot.Game
         /// <returns>A new BattlePhaseAction containing the action to do.</returns>
         public BattlePhaseAction OnSelectBattleCmd(BattlePhase battle)
         {
-            Executor.SetBattle(battle);
             foreach (CardExecutor exec in Executor.Executors)
             {
                 if (exec.Type == ExecutorType.GoToMainPhase2 && battle.CanMainPhaseTwo && exec.Func()) // check if should enter main phase 2 directly
@@ -440,7 +439,6 @@ namespace WindBot.Game
         /// <returns>A new MainPhaseAction containing the action to do.</returns>
         public MainPhaseAction OnSelectIdleCmd(MainPhase main)
         {
-            Executor.SetMain(main);
             CheckSurrender();
             foreach (CardExecutor exec in Executor.Executors)
             {

Game/GameBehavior.cs

@@ -433,6 +433,7 @@ namespace WindBot.Game
 
             // in case of ending duel in chain's solving
             _duel.CurrentChain.Clear();
+            _duel.CurrentChainInfo.Clear();
             _duel.ChainTargets.Clear();
             _duel.ChainTargetOnly.Clear();
             _duel.SummoningCards.Clear();
@@ -813,6 +814,8 @@ namespace WindBot.Game
             if (card.Id == 0)
                 card.SetId(cardId);
             int cc = GetLocalPlayer(packet.ReadByte());
+            packet.ReadInt16(); // trigger location + trigger sequence
+            int desc = packet.ReadInt32();
             if (_debug)
                 if (card != null) Logger.WriteLine("(" + cc.ToString() + " 's " + (card.Name ?? "UnKnowCard") + " activate effect from " + (CardLocation)pcl + ")");
             _duel.LastChainLocation = (CardLocation)pcl;
@@ -821,6 +824,7 @@ namespace WindBot.Game
             _duel.ChainTargetOnly.Clear();
             _duel.LastSummonPlayer = -1;
             _duel.CurrentChain.Add(card);
+            _duel.CurrentChainInfo.Add(new ChainInfo(card, cc, desc));
             _duel.LastChainPlayer = cc;
 
         }
@@ -855,6 +859,7 @@ namespace WindBot.Game
             _duel.LastChainPlayer = -1;
             _duel.LastChainLocation = 0;
             _duel.CurrentChain.Clear();
+            _duel.CurrentChainInfo.Clear();
             _duel.ChainTargets.Clear();
             _duel.LastChainTargets.Clear();
             _duel.ChainTargetOnly.Clear();

Properties/AssemblyInfo.cs

@@ -10,7 +10,7 @@ using System.Runtime.InteropServices;
 [assembly: AssemblyConfiguration("")]
 [assembly: AssemblyCompany("IceYGO")]
 [assembly: AssemblyProduct("WindBot")]
-[assembly: AssemblyCopyright("Copyright © IceYGO 2015-2017")]
+[assembly: AssemblyCopyright("Copyright © IceYGO 2015-2025")]
 [assembly: AssemblyTrademark("")]
 [assembly: AssemblyCulture("")]
 

README.md

 # WindBot
 
-A C# bot for YGOPro, compatible with the [YGOSharp](https://github.com/IceYGO/ygosharp) and [SRVPro](https://github.com/moecube/srvpro) server.
+A C# bot for [YGOPro](https://github.com/Fluorohydride/ygopro), compatible with the [YGOSharp](https://github.com/IceYGO/ygosharp) and [SRVPro](https://github.com/mycard/srvpro) server.
 
 ### How to use:
 
@@ -154,6 +154,11 @@ In this situation, it will be multi-threaded. This can be useful for servers, si
 
 The parameters are same as commandlines, but low cased.
 
+Note: Currently the server bind to all interfaces, so it requires elevated privileges to run. You can otherwise use the following command to add a URL ACL for your port (2399 for example), which allows all users to access it:
+```
+netsh http add urlacl url=http://+:2399/ user=Everyone
+```
+
 ### Known issues
 
 * If one chain includes two activation that use `AI.SelectCard`, the second one won't select correctly.

WindBot.csproj

@@ -9,7 +9,7 @@
     <AppDesignerFolder>Properties</AppDesignerFolder>
     <RootNamespace>WindBot</RootNamespace>
     <AssemblyName>WindBot</AssemblyName>
-    <TargetFrameworkVersion>v4.0</TargetFrameworkVersion>
+    <TargetFrameworkVersion>v4.8</TargetFrameworkVersion>
     <FileAlignment>512</FileAlignment>
     <TargetFrameworkProfile />
   </PropertyGroup>
@@ -21,6 +21,7 @@
     <PlatformTarget>x86</PlatformTarget>
     <ErrorReport>prompt</ErrorReport>
     <CodeAnalysisRuleSet>MinimumRecommendedRules.ruleset</CodeAnalysisRuleSet>
+    <Prefer32Bit>false</Prefer32Bit>
   </PropertyGroup>
   <PropertyGroup Condition="'$(Configuration)|$(Platform)' == 'Release|x86'">
     <OutputPath>bin\Release\</OutputPath>
@@ -30,9 +31,10 @@
     <PlatformTarget>x86</PlatformTarget>
     <ErrorReport>prompt</ErrorReport>
     <CodeAnalysisRuleSet>MinimumRecommendedRules.ruleset</CodeAnalysisRuleSet>
+    <Prefer32Bit>false</Prefer32Bit>
   </PropertyGroup>
   <PropertyGroup>
-    <StartupObject />
+    <StartupObject>WindBot.Program</StartupObject>
   </PropertyGroup>
   <PropertyGroup>
     <ApplicationIcon>WindBot.ico</ApplicationIcon>
@@ -40,6 +42,12 @@
   <PropertyGroup>
     <AutoGenerateBindingRedirects>true</AutoGenerateBindingRedirects>
   </PropertyGroup>
+  <PropertyGroup Condition="'$(Configuration)|$(Platform)' == 'Debug|AnyCPU'">
+    <Prefer32Bit>false</Prefer32Bit>
+  </PropertyGroup>
+  <PropertyGroup Condition="'$(Configuration)|$(Platform)' == 'Release|AnyCPU'">
+    <Prefer32Bit>false</Prefer32Bit>
+  </PropertyGroup>
   <ItemGroup>
     <Reference Include="Mono.Data.Sqlite">
       <HintPath>.\Mono.Data.Sqlite.dll</HintPath>
@@ -137,6 +145,7 @@
     <Compile Include="Game\AI\ExecutorType.cs" />
     <Compile Include="Game\BattlePhase.cs" />
     <Compile Include="Game\BattlePhaseAction.cs" />
+    <Compile Include="Game\ChainInfo.cs" />
     <Compile Include="Game\ClientCard.cs" />
     <Compile Include="Game\ClientField.cs" />
     <Compile Include="Game\Deck.cs" />