Add icon

Most notable:
- Support for older configs, with auto-selecting defaults for missing ones
- Auto exit when all bots are done

ArchiSteamFarm/ArchiHandler.cs

@@ -35,6 +35,7 @@ namespace ArchiSteamFarm {
 				AlreadyOwned = 9,
 				InvalidKey = 14,
 				DuplicatedKey = 15,
+				BaseGameRequired = 24,
 				OnCooldown = 53
 			}
 

ArchiSteamFarm/ArchiSteamFarm.csproj

@@ -51,6 +51,9 @@
     </DocumentationFile>
     <Prefer32Bit>true</Prefer32Bit>
   </PropertyGroup>
+  <PropertyGroup>
+    <ApplicationIcon>cirno.ico</ApplicationIcon>
+  </PropertyGroup>
   <ItemGroup>
     <Reference Include="HtmlAgilityPack, Version=1.4.9.0, Culture=neutral, PublicKeyToken=bd319b19eaf3b43a, processorArchitecture=MSIL">
       <HintPath>..\packages\HtmlAgilityPack.1.4.9\lib\Net45\HtmlAgilityPack.dll</HintPath>
@@ -83,6 +86,7 @@
     <Compile Include="Bot.cs" />
     <Compile Include="CardsFarmer.cs" />
     <Compile Include="CMsgClientClanInviteAction.cs" />
+    <Compile Include="Debugging.cs" />
     <Compile Include="Logging.cs" />
     <Compile Include="Program.cs" />
     <Compile Include="Properties\AssemblyInfo.cs" />
@@ -107,7 +111,24 @@
       <Install>false</Install>
     </BootstrapperPackage>
   </ItemGroup>
+  <ItemGroup>
+    <Content Include="cirno.ico" />
+  </ItemGroup>
   <Import Project="$(MSBuildToolsPath)\Microsoft.CSharp.targets" />
+  <PropertyGroup>
+    <PostBuildEvent Condition=" '$(OS)' != 'Unix' ">(robocopy $(ProjectDir)config $(TargetDir)config /S /E) ^&amp; IF %25ERRORLEVEL%25 GEQ 2 exit 0
+
+if $(ConfigurationName) == Release (
+    mkdir "$(TargetDir)out" "$(TargetDir)out\config"
+    copy "$(TargetDir)config\example.xml" "$(TargetDir)out\config"
+    "$(SolutionDir)tools\ILMerge.exe" /out:"$(TargetDir)out\ASF.exe" "$(TargetDir)$(TargetName).exe" "$(TargetDir)*.dll" /target:exe /targetplatform:v4,C:\Windows\Microsoft.NET\Framework64\v4.0.30319 /wildcards
+    del "$(TargetDir)out\ASF.pdb"
+)</PostBuildEvent>
+  </PropertyGroup>
+  <PropertyGroup>
+    <PreBuildEvent>
+    </PreBuildEvent>
+  </PropertyGroup>
   <!-- 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">

ArchiSteamFarm/Bot.cs

@@ -35,40 +35,58 @@ namespace ArchiSteamFarm {
 	internal class Bot {
 		private const ushort CallbackSleep = 500; // In miliseconds
 
-		private readonly Dictionary<string, string> Config = new Dictionary<string, string>();
+		private static readonly HashSet<Bot> Bots = new HashSet<Bot>();
 
-		internal readonly string BotName;
 		private readonly string ConfigFile;
 		private readonly string SentryFile;
 
-		private readonly CardsFarmer CardsFarmer;
-
-		internal ulong BotID { get; private set; }
+		private bool IsRunning = false;
 		private string AuthCode, TwoFactorAuth;
 
+		internal readonly string BotName;
+
 		internal ArchiHandler ArchiHandler { get; private set; }
 		internal ArchiWebHandler ArchiWebHandler { get; private set; }
 		internal CallbackManager CallbackManager { get; private set; }
+		internal CardsFarmer CardsFarmer { get; private set; }
 		internal SteamClient SteamClient { get; private set; }
 		internal SteamFriends SteamFriends { get; private set; }
 		internal SteamUser SteamUser { get; private set; }
 		internal Trading Trading { get; private set; }
 
 		// Config variables
-		internal bool Enabled { get { return bool.Parse(Config["Enabled"]); } }
-		private string SteamLogin { get { return Config["SteamLogin"]; } }
-		private string SteamPassword { get { return Config["SteamPassword"]; } }
-		private string SteamNickname { get { return Config["SteamNickname"]; } }
-		private string SteamApiKey { get { return Config["SteamApiKey"]; } }
-		private string SteamParentalPIN { get { return Config["SteamParentalPIN"]; } }
-		internal ulong SteamMasterID { get { return ulong.Parse(Config["SteamMasterID"]); } }
-		private ulong SteamMasterClanID { get { return ulong.Parse(Config["SteamMasterClanID"]); } }
-		internal HashSet<uint> Blacklist { get; } = new HashSet<uint>();
-		internal bool Statistics { get { return bool.Parse(Config["Statistics"]); } }
+		internal bool Enabled { get; private set; } = true;
+		internal string SteamLogin { get; private set; } = "null";
+		internal string SteamPassword { get; private set; } = "null";
+		internal string SteamNickname { get; private set; } = "null";
+		internal string SteamApiKey { get; private set; } = "null";
+		internal string SteamParentalPIN { get; private set; } = "0";
+		internal ulong SteamMasterID { get; private set; } = 76561198006963719;
+		internal ulong SteamMasterClanID { get; private set; } = 0;
+		internal bool ShutdownOnFarmingFinished { get; private set; } = false;
+		internal HashSet<uint> Blacklist { get; private set; } = new HashSet<uint> { 368020 };
+		internal bool Statistics { get; private set; } = true;
+
+		internal static int GetRunningBotsCount() {
+			int result;
+			lock (Bots) {
+				result = Bots.Count;
+			}
+			return result;
+		}
+
+		internal static async Task ShutdownAllBots() {
+			List<Task> tasks = new List<Task>();
+			lock (Bots) {
+				foreach (Bot bot in Bots) {
+					tasks.Add(Task.Run(async () => await bot.Shutdown().ConfigureAwait(false)));
+				}
+			}
+			await Task.WhenAll(tasks).ConfigureAwait(false);
+		}
 
 		internal Bot(string botName) {
 			BotName = botName;
-			CardsFarmer = new CardsFarmer(this);
 
 			ConfigFile = Path.Combine(Program.ConfigDirectoryPath, BotName + ".xml");
 			SentryFile = Path.Combine(Program.ConfigDirectoryPath, BotName + ".bin");
@@ -79,44 +97,11 @@ namespace ArchiSteamFarm {
 				return;
 			}
 
-			Start();
-		}
-
-		private void ReadConfig() {
-			using (XmlReader reader = XmlReader.Create(ConfigFile)) {
-				while (reader.Read()) {
-					if (reader.NodeType != XmlNodeType.Element) {
-						continue;
-					}
-
-					string key = reader.Name;
-					if (string.IsNullOrEmpty(key)) {
-						continue;
-					}
-
-					string value = reader.GetAttribute("value");
-					if (string.IsNullOrEmpty(value)) {
-						continue;
-					}
-
-					Config.Add(key, value);
-
-					switch (key) {
-						case "Blacklist":
-							foreach (string appID in value.Split(',')) {
-								Blacklist.Add(uint.Parse(appID));
-							}
-							break;
-					}
-				}
-			}
-		}
-
-		internal void Start() {
-			if (SteamClient != null) {
-				return;
+			lock (Bots) {
+				Bots.Add(this);
 			}
 
+			// Initialize
 			SteamClient = new SteamClient();
 
 			ArchiHandler = new ArchiHandler();
@@ -141,20 +126,107 @@ namespace ArchiSteamFarm {
 			CallbackManager.Subscribe<ArchiHandler.PurchaseResponseCallback>(OnPurchaseResponse);
 
 			ArchiWebHandler = new ArchiWebHandler(this, SteamApiKey);
+			CardsFarmer = new CardsFarmer(this);
 			Trading = new Trading(this);
 
-			SteamClient.Connect();
-			Task.Run(() => HandleCallbacks());
+			// Start
+			Start();
 		}
 
-		internal void Stop() {
-			if (SteamClient == null) {
+		private void ReadConfig() {
+			using (XmlReader reader = XmlReader.Create(ConfigFile)) {
+				while (reader.Read()) {
+					if (reader.NodeType != XmlNodeType.Element) {
+						continue;
+					}
+
+					string key = reader.Name;
+					if (string.IsNullOrEmpty(key)) {
+						continue;
+					}
+
+					string value = reader.GetAttribute("value");
+					if (string.IsNullOrEmpty(value)) {
+						continue;
+					}
+
+					switch (key) {
+						case "Enabled":
+							Enabled = bool.Parse(value);
+							break;
+						case "SteamLogin":
+							SteamLogin = value;
+							break;
+						case "SteamPassword":
+							SteamPassword = value;
+							break;
+						case "SteamNickname":
+							SteamNickname = value;
+							break;
+						case "SteamApiKey":
+							SteamApiKey = value;
+							break;
+						case "SteamParentalPIN":
+							SteamParentalPIN = value;
+							break;
+						case "SteamMasterID":
+							SteamMasterID = ulong.Parse(value);
+							break;
+						case "SteamMasterClanID":
+							SteamMasterClanID = ulong.Parse(value);
+							break;
+						case "ShutdownOnFarmingFinished":
+							ShutdownOnFarmingFinished = bool.Parse(value);
+							break;
+						case "Blacklist":
+							foreach (string appID in value.Split(',')) {
+								Blacklist.Add(uint.Parse(appID));
+							}
+							break;
+						case "Statistics":
+							Statistics = bool.Parse(value);
+							break;
+						default:
+							Logging.LogGenericWarning(BotName, "Unrecognized config value: " + key + "=" + value);
+							break;
+					}
+				}
+			}
+		}
+
+		internal void Start() {
+			if (IsRunning) {
 				return;
 			}
 
+			SteamClient.Connect();
+			IsRunning = true;
+
+			Task.Run(() => HandleCallbacks());
+		}
+
+		internal async Task Stop() {
+			if (!IsRunning) {
+				return;
+			}
+
+			await CardsFarmer.StopFarming().ConfigureAwait(false);
 			SteamClient.Disconnect();
-			SteamClient = null;
-			CallbackManager = null;
+			IsRunning = false;
+		}
+
+		internal async Task Shutdown() {
+			await Stop().ConfigureAwait(false);
+			lock (Bots) {
+				Bots.Remove(this);
+			}
+			Program.OnBotShutdown(this);
+		}
+
+		internal async Task OnFarmingFinished() {
+			if (ShutdownOnFarmingFinished) {
+				await Shutdown().ConfigureAwait(false);
+			}
 		}
 
 		internal void PlayGame(params ulong[] gameIDs) {
@@ -163,7 +235,7 @@ namespace ArchiSteamFarm {
 
 		private void HandleCallbacks() {
 			TimeSpan timeSpan = TimeSpan.FromMilliseconds(CallbackSleep);
-			while (CallbackManager != null) {
+			while (IsRunning) {
 				CallbackManager.RunWaitCallbacks(timeSpan);
 			}
 		}
@@ -186,21 +258,17 @@ namespace ArchiSteamFarm {
 				sentryHash = CryptoHelper.SHAHash(sentryFileContent);
 			}
 
-			string steamLogin = SteamLogin;
-			if (string.IsNullOrEmpty(steamLogin) || steamLogin.Equals("null")) {
-				steamLogin = Program.GetUserInput(BotName, Program.EUserInputType.Login);
-				Config["SteamLogin"] = steamLogin;
+			if (SteamLogin.Equals("null")) {
+				SteamLogin = Program.GetUserInput(BotName, Program.EUserInputType.Login);
 			}
 
-			string steamPassword = SteamPassword;
-			if (string.IsNullOrEmpty(steamPassword) || steamPassword.Equals("null")) {
-				steamPassword = Program.GetUserInput(BotName, Program.EUserInputType.Password);
-				Config["SteamPassword"] = steamPassword;
+			if (SteamPassword.Equals("null")) {
+				SteamPassword = Program.GetUserInput(BotName, Program.EUserInputType.Password);
 			}
 
 			SteamUser.LogOn(new SteamUser.LogOnDetails {
-				Username = steamLogin,
-				Password = steamPassword,
+				Username = SteamLogin,
+				Password = SteamPassword,
 				AuthCode = AuthCode,
 				TwoFactorCode = TwoFactorAuth,
 				SentryFileHash = sentryHash
@@ -247,7 +315,7 @@ namespace ArchiSteamFarm {
 			}
 		}
 
-		private void OnFriendMsg(SteamFriends.FriendMsgCallback callback) {
+		private async void OnFriendMsg(SteamFriends.FriendMsgCallback callback) {
 			if (callback == null) {
 				return;
 			}
@@ -269,6 +337,15 @@ namespace ArchiSteamFarm {
 			if (message.Length == 17 && message[5] == '-' && message[11] == '-') {
 				ArchiHandler.RedeemKey(message);
 			}
+
+			switch (message) {
+				case "!farm":
+					await CardsFarmer.StartFarming().ConfigureAwait(false);
+					break;
+				case "!exit":
+					await Shutdown().ConfigureAwait(false);
+					break;
+			}
 		}
 
 		private void OnPersonaState(SteamFriends.PersonaStateCallback callback) {
@@ -304,10 +381,6 @@ namespace ArchiSteamFarm {
 				return;
 			}
 
-			if (callback.ClientSteamID != 0) {
-				BotID = callback.ClientSteamID;
-			}
-
 			EResult result = callback.Result;
 			switch (result) {
 				case EResult.AccountLogonDenied:
@@ -319,18 +392,15 @@ namespace ArchiSteamFarm {
 				case EResult.OK:
 					Logging.LogGenericInfo(BotName, "Successfully logged on!");
 
-					string steamNickname = SteamNickname;
-					if (!string.IsNullOrEmpty(steamNickname) && !steamNickname.Equals("null")) {
-						SteamFriends.SetPersonaName(steamNickname);
+					if (!SteamNickname.Equals("null")) {
+						SteamFriends.SetPersonaName(SteamNickname);
 					}
 
-					string steamParentalPIN = SteamParentalPIN;
-					if (string.IsNullOrEmpty(steamParentalPIN) || steamParentalPIN.Equals("null")) {
-						steamParentalPIN = Program.GetUserInput(BotName, Program.EUserInputType.SteamParentalPIN);
-						Config["SteamParentalPIN"] = steamParentalPIN;
+					if (SteamParentalPIN.Equals("null")) {
+						SteamParentalPIN = Program.GetUserInput(BotName, Program.EUserInputType.SteamParentalPIN);
 					}
 
-					await ArchiWebHandler.Init(SteamClient, callback.WebAPIUserNonce, callback.VanityURL, steamParentalPIN).ConfigureAwait(false);
+					await ArchiWebHandler.Init(SteamClient, callback.WebAPIUserNonce, callback.VanityURL, SteamParentalPIN).ConfigureAwait(false);
 
 					ulong clanID = SteamMasterClanID;
 					if (clanID != 0) {
@@ -347,13 +417,13 @@ namespace ArchiSteamFarm {
 				case EResult.Timeout:
 				case EResult.TryAnotherCM:
 					Logging.LogGenericWarning(BotName, "Unable to login to Steam: " + callback.Result + " / " + callback.ExtendedResult + ", retrying...");
-					Stop();
+					await Stop().ConfigureAwait(false);
 					Thread.Sleep(5000);
 					Start();
 					break;
 				default:
 					Logging.LogGenericWarning(BotName, "Unable to login to Steam: " + callback.Result + " / " + callback.ExtendedResult);
-					Stop();
+					await Shutdown().ConfigureAwait(false);
 					break;
 			}
 		}

ArchiSteamFarm/CardsFarmer.cs

@@ -31,20 +31,33 @@ namespace ArchiSteamFarm {
 	internal class CardsFarmer {
 		private const byte StatusCheckSleep = 5; // In minutes, how long to wait before checking the appID again
 
+		private readonly ManualResetEvent FarmResetEvent = new ManualResetEvent(false);
+		private readonly SemaphoreSlim Semaphore = new SemaphoreSlim(1);
 		private readonly Bot Bot;
-		private bool NowFarming;
-		private readonly AutoResetEvent AutoResetEvent = new AutoResetEvent(false);
+
+		private volatile bool NowFarming = false;
 
 		internal CardsFarmer(Bot bot) {
 			Bot = bot;
 		}
 
 		internal async Task StartFarming() {
+			await StopFarming().ConfigureAwait(false);
+
+			await Semaphore.WaitAsync().ConfigureAwait(false);
+
+			if (NowFarming) {
+				Semaphore.Release();
+				return;
+			}
+
 			Logging.LogGenericInfo(Bot.BotName, "Checking badges...");
+
 			// Find the number of badge pages
 			HtmlDocument badgesDocument = await Bot.ArchiWebHandler.GetBadgePage(1).ConfigureAwait(false);
 			if (badgesDocument == null) {
 				Logging.LogGenericWarning(Bot.BotName, "Could not get badges information, farming is stopped!");
+				Semaphore.Release();
 				return;
 			}
 
@@ -74,13 +87,13 @@ namespace ArchiSteamFarm {
 				foreach (HtmlNode badgesPageNode in badgesPageNodes) {
 					string steamLink = badgesPageNode.GetAttributeValue("href", null);
 					if (steamLink == null) {
-						Logging.LogGenericWarning(Bot.BotName, "Couldn't get steamLink for one of the games: " + badgesPageNode.OuterHtml);
+						Logging.LogGenericError(Bot.BotName, "Couldn't get steamLink for one of the games: " + badgesPageNode.OuterHtml);
 						continue;
 					}
 
 					uint appID = (uint) Utilities.OnlyNumbers(steamLink);
 					if (appID == 0) {
-						Logging.LogGenericWarning(Bot.BotName, "Couldn't get appID for one of the games: " + badgesPageNode.OuterHtml);
+						Logging.LogGenericError(Bot.BotName, "Couldn't get appID for one of the games: " + badgesPageNode.OuterHtml);
 						continue;
 					}
 
@@ -99,11 +112,31 @@ namespace ArchiSteamFarm {
 				if (await Farm(appID).ConfigureAwait(false)) {
 					appIDs.Remove(appID);
 				} else {
-					break;
+					return;
 				}
 			}
 
 			Logging.LogGenericInfo(Bot.BotName, "Farming finished!");
+			await Bot.OnFarmingFinished().ConfigureAwait(false);
+		}
+
+		internal async Task StopFarming() {
+			await Semaphore.WaitAsync().ConfigureAwait(false);
+
+			if (!NowFarming) {
+				Semaphore.Release();
+				return;
+			}
+
+			Logging.LogGenericInfo(Bot.BotName, "Sending signal to stop farming");
+			FarmResetEvent.Set();
+			while (NowFarming) {
+				Logging.LogGenericInfo(Bot.BotName, "Waiting for reaction...");
+				Thread.Sleep(1000);
+			}
+			FarmResetEvent.Reset();
+			Logging.LogGenericInfo(Bot.BotName, "Farming stopped!");
+			Semaphore.Release();
 		}
 
 		private async Task<bool?> ShouldFarm(ulong appID) {
@@ -119,12 +152,6 @@ namespace ArchiSteamFarm {
 		}
 
 		private async Task<bool> Farm(ulong appID) {
-			if (NowFarming) {
-				AutoResetEvent.Set();
-				Thread.Sleep(1000);
-				AutoResetEvent.Reset();
-			}
-
 			bool success = true;
 			bool? keepFarming = await ShouldFarm(appID).ConfigureAwait(false);
 			while (keepFarming == null || keepFarming.Value) {
@@ -132,17 +159,20 @@ namespace ArchiSteamFarm {
 					NowFarming = true;
 					Logging.LogGenericInfo(Bot.BotName, "Now farming: " + appID);
 					Bot.PlayGame(appID);
+					Semaphore.Release(); // We're farming, allow other tasks to shut us down
+				} else {
+					Logging.LogGenericInfo(Bot.BotName, "Still farming: " + appID);
 				}
-				if (AutoResetEvent.WaitOne(1000 * 60 * StatusCheckSleep)) {
+				if (FarmResetEvent.WaitOne(1000 * 60 * StatusCheckSleep)) {
 					success = false;
 					break;
 				}
 				keepFarming = await ShouldFarm(appID).ConfigureAwait(false);
 			}
 
-			Logging.LogGenericInfo(Bot.BotName, "Stopped farming: " + appID);
 			Bot.PlayGame(0);
 			NowFarming = false;
+			Logging.LogGenericInfo(Bot.BotName, "Stopped farming: " + appID);
 			return success;
 		}
 	}

ArchiSteamFarm/Debugging.cs

@@ -0,0 +1,40 @@
+/*
+    _                _      _  ____   _                           _____
+   / \    _ __  ___ | |__  (_)/ ___| | |_  ___   __ _  _ __ ___  |  ___|__ _  _ __  _ __ ___
+  / _ \  | '__|/ __|| '_ \ | |\___ \ | __|/ _ \ / _` || '_ ` _ \ | |_  / _` || '__|| '_ ` _ \
+ / ___ \ | |  | (__ | | | || | ___) || |_|  __/| (_| || | | | | ||  _|| (_| || |   | | | | | |
+/_/   \_\|_|   \___||_| |_||_||____/  \__|\___| \__,_||_| |_| |_||_|   \__,_||_|   |_| |_| |_|
+
+ Copyright 2015 Łukasz "JustArchi" Domeradzki
+ Contact: JustArchi@JustArchi.net
+
+ Licensed under the Apache License, Version 2.0 (the "License");
+ you may not use this file except in compliance with the License.
+ You may obtain a copy of the License at
+
+ http://www.apache.org/licenses/LICENSE-2.0
+					
+ Unless required by applicable law or agreed to in writing, software
+ distributed under the License is distributed on an "AS IS" BASIS,
+ WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ See the License for the specific language governing permissions and
+ limitations under the License.
+
+*/
+
+using System.Diagnostics;
+
+namespace ArchiSteamFarm {
+	internal static class Debugging {
+		internal static bool IsReleaseBuild { get; private set; } = true;
+
+		static Debugging() {
+			MarkIfDebug();
+		}
+
+		[Conditional("DEBUG")]
+		private static void MarkIfDebug() {
+			IsReleaseBuild = false;
+		}
+	}
+}

ArchiSteamFarm/Program.cs

@@ -24,7 +24,6 @@
 
 using Newtonsoft.Json.Linq;
 using System;
-using System.Collections.Generic;
 using System.IO;
 using System.Reflection;
 using System.Threading;
@@ -44,9 +43,11 @@ namespace ArchiSteamFarm {
 		internal const string ConfigDirectoryPath = "config";
 		private const string LatestGithubReleaseURL = "https://api.github.com/repos/JustArchi/ArchiSteamFarm/releases/latest";
 
-		private static readonly HashSet<Bot> Bots = new HashSet<Bot>();
 		internal static readonly object ConsoleLock = new object();
-		internal static string Version { get { return Assembly.GetExecutingAssembly().GetName().Version.ToString(); } }
+		private static readonly ManualResetEvent ShutdownResetEvent = new ManualResetEvent(false);
+		private static readonly AssemblyName AssemblyName = Assembly.GetExecutingAssembly().GetName();
+		private static readonly string ExeName = AssemblyName.Name + ".exe";
+		private static readonly string Version = AssemblyName.Version.ToString();
 
 		private static async Task CheckForUpdate() {
 			JObject response = await Utilities.UrlToJObject(LatestGithubReleaseURL).ConfigureAwait(false);
@@ -61,17 +62,22 @@ namespace ArchiSteamFarm {
 
 			string localVersion = Version;
 
+			Logging.LogGenericNotice("", "Local version: " + localVersion);
+			Logging.LogGenericNotice("", "Remote version: " + remoteVersion);
+
+			int comparisonResult = localVersion.CompareTo(remoteVersion);
 			if (localVersion.CompareTo(remoteVersion) < 0) {
 				Logging.LogGenericNotice("", "New version is available!");
-				Logging.LogGenericNotice("", "Local version: " + localVersion);
-				Logging.LogGenericNotice("", "Remote version: " + remoteVersion);
 				Logging.LogGenericNotice("", "Consider updating yourself!");
 				Thread.Sleep(5000);
+			} else if (localVersion.CompareTo(remoteVersion) > 0) {
+				Logging.LogGenericNotice("", "You're currently using pre-release version!");
+				Logging.LogGenericNotice("", "Be careful!");
 			}
 		}
 
-		internal static void Exit(int exitCode = 0) {
-			ShutdownAllBots();
+		internal static async Task Exit(int exitCode = 0) {
+			await Bot.ShutdownAllBots().ConfigureAwait(false);
 			Environment.Exit(exitCode);
 		}
 
@@ -101,12 +107,11 @@ namespace ArchiSteamFarm {
 			return result;
 		}
 
-		private static void ShutdownAllBots() {
-			lock (Bots) {
-				foreach (Bot bot in Bots) {
-					bot.Stop();
-				}
-				Bots.Clear();
+		internal static void OnBotShutdown(Bot bot) {
+			if (Bot.GetRunningBotsCount() == 0) {
+				Logging.LogGenericInfo("Main", "No bots are running, exiting");
+				Thread.Sleep(5000); // This might be the only message user gets, consider giving him some time
+				ShutdownResetEvent.Set();
 			}
 		}
 
@@ -115,29 +120,24 @@ namespace ArchiSteamFarm {
 
 			Task.Run(async () => await CheckForUpdate().ConfigureAwait(false)).Wait();
 
-			// Config directory may not be in the same directory as the .exe, check maximum of 3 levels lower
-			for (var i = 0; i < 4 && !Directory.Exists(ConfigDirectoryPath); i++) {
-				Directory.SetCurrentDirectory("..");
-			}
-
 			if (!Directory.Exists(ConfigDirectoryPath)) {
 				Logging.LogGenericError("Main", "Config directory doesn't exist!");
 				Console.ReadLine();
-				Exit(1);
+				Task.Run(async () => await Exit(1).ConfigureAwait(false)).Wait();
 			}
 
-			lock (Bots) {
-				foreach (var configFile in Directory.EnumerateFiles(ConfigDirectoryPath, "*.xml")) {
-					string botName = Path.GetFileNameWithoutExtension(configFile);
-					Bot bot = new Bot(botName);
-					Bots.Add(bot);
-					if (!bot.Enabled) {
-						Logging.LogGenericInfo(botName, "Not starting this instance because it's disabled in config file");
-					}
+			foreach (var configFile in Directory.EnumerateFiles(ConfigDirectoryPath, "*.xml")) {
+				string botName = Path.GetFileNameWithoutExtension(configFile);
+				Bot bot = new Bot(botName);
+				if (!bot.Enabled) {
+					Logging.LogGenericInfo(botName, "Not starting this instance because it's disabled in config file");
 				}
 			}
 
-			Thread.Sleep(Timeout.Infinite);
+			// Check if we got any bots running
+			OnBotShutdown(null);
+
+			ShutdownResetEvent.WaitOne();
 		}
 	}
 }

ArchiSteamFarm/Properties/AssemblyInfo.cs

@@ -32,5 +32,5 @@ using System.Runtime.InteropServices;
 // You can specify all the values or you can default the Build and Revision Numbers 
 // by using the '*' as shown below:
 // [assembly: AssemblyVersion("1.0.*")]
-[assembly: AssemblyVersion("0.4.0.0")]
-[assembly: AssemblyFileVersion("0.4.0.0")]
+[assembly: AssemblyVersion("0.5.0.0")]
+[assembly: AssemblyFileVersion("0.5.0.0")]

ArchiSteamFarm/SteamTradeOffer.cs

@@ -55,9 +55,14 @@ namespace ArchiSteamFarm {
 		internal bool from_real_time_trade { get; set; }
 
 		// Extra
+		private ulong _OtherSteamID64 = 0;
 		internal ulong OtherSteamID64 {
 			get {
-				return new SteamID((uint) accountid_other, EUniverse.Public, EAccountType.Individual).ConvertToUInt64();
+				if (_OtherSteamID64 == 0 && accountid_other != 0) {
+					_OtherSteamID64 = new SteamID((uint) accountid_other, EUniverse.Public, EAccountType.Individual).ConvertToUInt64();
+				}
+
+				return _OtherSteamID64;
 			}
 		}
 	}

ArchiSteamFarm/Trading.cs

@@ -28,9 +28,9 @@ using System.Threading.Tasks;
 
 namespace ArchiSteamFarm {
 	internal sealed class Trading {
-		private Bot Bot;
+		private readonly Bot Bot;
+		private readonly SemaphoreSlim Semaphore = new SemaphoreSlim(1);
 		private volatile byte ParsingTasks = 0;
-		private SemaphoreSlim semaphore = new SemaphoreSlim(1);
 
 		internal Trading(Bot bot) {
 			Bot = bot;
@@ -44,7 +44,7 @@ namespace ArchiSteamFarm {
 		}
 
 		private async Task ParseActiveTrades() {
-			await semaphore.WaitAsync().ConfigureAwait(false);
+			await Semaphore.WaitAsync().ConfigureAwait(false);
 
 			List<SteamTradeOffer> tradeOffers = Bot.ArchiWebHandler.GetTradeOffers();
 			if (tradeOffers != null) {
@@ -62,7 +62,7 @@ namespace ArchiSteamFarm {
 			}
 
 			ParsingTasks--;
-			semaphore.Release();
+			Semaphore.Release();
 		}
 
 		private async Task ParseTrade(SteamTradeOffer tradeOffer) {
@@ -75,11 +75,11 @@ namespace ArchiSteamFarm {
 				return;
 			}
 
-			ulong steamID = tradeOffer.OtherSteamID64;
+			ulong otherSteamID = tradeOffer.OtherSteamID64;
 			bool success = false;
 			bool tradeAccepted = false;
 
-			if (tradeOffer.items_to_give.Count == 0 || steamID == Bot.SteamMasterID) {
+			if (tradeOffer.items_to_give.Count == 0 || otherSteamID == Bot.SteamMasterID) {
 				tradeAccepted = true;
 				success = await Bot.ArchiWebHandler.AcceptTradeOffer(tradeID).ConfigureAwait(false);
 			} else {

ArchiSteamFarm/Utilities.cs

@@ -86,10 +86,6 @@ namespace ArchiSteamFarm {
 			return result;
 		}
 
-		internal static async Task<HttpResponseMessage> UrlToHttpResponse(string websiteAddress) {
-			return await UrlToHttpResponse(websiteAddress, null).ConfigureAwait(false);
-		}
-
 		internal static async Task<HtmlDocument> UrlToHtmlDocument(string websiteAddress, Dictionary<string, string> cookieVariables = null) {
 			if (string.IsNullOrEmpty(websiteAddress)) {
 				return null;

ArchiSteamFarm/config/example.xml

@@ -3,17 +3,17 @@
 
 <!-- Every bot should have it's own unique .xml configuration file, this is example on which you can base on -->
 
- <!-- Master switch to turn account on and off, set to "true" after you're done -->
- <!-- TIP: This bot instance won't run unless below switch is set to "true" -->
+<!-- Master switch to turn account on and off, set to "true" after you're done -->
+<!-- TIP: This bot instance won't run unless below switch is set to "true" -->
 <Enabled type="bool" value="false"/>
 
 <!-- This is your steam login, the one you use for logging in to steam -->
 <!-- TIP: You can use "null" if you wish to enter login on every startup -->
-<SteamLogin type="string" value="Foo"/>
+<SteamLogin type="string" value="null"/>
 
 <!-- This is your steam password, the one you use for logging in to steam -->
 <!-- TIP: You can use "null" if you wish to enter password on every startup -->
-<SteamPassword type="string" value="Bar"/>
+<SteamPassword type="string" value="null"/>
 
 <!-- This is steam nickname, the one you want to use for bot. Can be anything up to 32 characters -->
 <!-- TIP: You can use "null" if you wish to preserve your actual nickname -->
@@ -35,6 +35,12 @@
 <!-- TIP: Most likely you don't want to change it -->
 <SteamMasterClanID type="ulong" value="0"/>
 
+<!-- This switch defines if bot should disconnect once farming is finished -->
+<!-- When no bots are active, ASF will shutdown as well -->
+<!-- Some people may want to keep their bots 24/7, other disconnect them after job is done -->
+<!-- Choose yourself what you prefer -->
+<ShutdownOnFarmingFinished type="bool" value="false"/>
+
 <!-- Comma-separated list of IDs that should not be considered for farming -->
 <!-- TIP: Most likely you don't want to change it -->
 <Blacklist type="HashSet(uint)" value="368020"/>