fix typo and replace to more better word

Author: smdn

.vscode/cspell.json

@@ -11,20 +11,30 @@
   "language": "en",
   // words - list of words to be always considered correct
   "words": [
+    "cloneable",
+    "LINQ",
     "memberwise",
     "munin",
     "smdn",
+    "SYSLIB",
   ],
   "ignoreWords": [
     "dirtyconfig",
-    "netstandard",
     "multigraph",
-    "vlabel"
+    "netstandard",
+    "rrdgraph",
+    "vlabel",
+  ],
+  "ignoreRegExpList": [
+    "SYSTEM_[A-Z][A-Z0-9_]+",
   ],
   // flagWords - list of words to be always considered incorrect
   // This is useful for offensive words and common spelling errors.
   // For example "hte" should be "the"
   "flagWords": [],
+  "suggestWords": [
+    "clonable->cloneable",
+  ],
   "ignorePaths": [
     "**/bin",
     "**/obj",

src/Smdn.Net.MuninNode/Smdn.Net.MuninNode.Protocol/MuninProtocolHandler.cs

@@ -101,7 +101,12 @@ IMuninNodeProfile profile
     this.profile = profile ?? throw new ArgumentNullException(nameof(profile));
 
     banner = $"# munin node at {profile.HostName}";
+
+    // cSpell:disable
+    // the typo 'munins' is intentional, follows the actual munin-node implementation
+    // ref: https://github.com/munin-monitoring/munin/blob/master/lib/Munin/Node/Server.pm#L320-L322
     versionInformation = $"munins node on {profile.HostName} version: {profile.Version}";
+    // cSpell:enable
 
     ReinitializePluginDictionary();
   }
@@ -745,7 +750,7 @@ StringBuilder responseBuilder
       .Fields
       .Any(static f => !string.IsNullOrEmpty(f.Attributes.NegativeFieldName));
 
-    // The fields referenced by {fieldname}.negative must be defined ahread of others,
+    // The fields referenced by {fieldname}.negative must be defined ahead of others,
     // and thus lists the negative field settings first.
     // Otherwise, the following error occurs when generating the graph.
     // "[RRD ERROR] Unable to graph /var/cache/munin/www/XXX.png : undefined v name XXXXXXXXXXXXXX"
@@ -801,22 +806,26 @@ static bool IsNegativeField(IPluginField field, IReadOnlyCollection<IPluginField
       else
         return null;
     }
-
-    static string? TranslateFieldDrawAttribute(PluginFieldGraphStyle style)
-      => style switch {
-        PluginFieldGraphStyle.Default => null,
-        PluginFieldGraphStyle.Area => "AREA",
-        PluginFieldGraphStyle.Stack => "STACK",
-        PluginFieldGraphStyle.AreaStack => "AREASTACK",
-        PluginFieldGraphStyle.Line => "LINE",
-        PluginFieldGraphStyle.LineWidth1 => "LINE1",
-        PluginFieldGraphStyle.LineWidth2 => "LINE2",
-        PluginFieldGraphStyle.LineWidth3 => "LINE3",
-        PluginFieldGraphStyle.LineStack => "LINESTACK",
-        PluginFieldGraphStyle.LineStackWidth1 => "LINE1STACK",
-        PluginFieldGraphStyle.LineStackWidth2 => "LINE2STACK",
-        PluginFieldGraphStyle.LineStackWidth3 => "LINE3STACK",
-        _ => throw new InvalidOperationException($"undefined draw attribute value: ({(int)style} {style})"),
-      };
   }
+
+#pragma warning disable SA1124 // Do not use #regions
+  #region cSpell:disable
+  private static string? TranslateFieldDrawAttribute(PluginFieldGraphStyle style)
+    => style switch {
+      PluginFieldGraphStyle.Default => null,
+      PluginFieldGraphStyle.Area => "AREA",
+      PluginFieldGraphStyle.Stack => "STACK",
+      PluginFieldGraphStyle.AreaStack => "AREASTACK",
+      PluginFieldGraphStyle.Line => "LINE",
+      PluginFieldGraphStyle.LineWidth1 => "LINE1",
+      PluginFieldGraphStyle.LineWidth2 => "LINE2",
+      PluginFieldGraphStyle.LineWidth3 => "LINE3",
+      PluginFieldGraphStyle.LineStack => "LINESTACK",
+      PluginFieldGraphStyle.LineStackWidth1 => "LINE1STACK",
+      PluginFieldGraphStyle.LineStackWidth2 => "LINE2STACK",
+      PluginFieldGraphStyle.LineStackWidth3 => "LINE3STACK",
+      _ => throw new InvalidOperationException($"undefined draw attribute value: ({(int)style} {style})"),
+    };
+  #endregion cSpell:enable
+#pragma warning restore SA1124
 }

src/Smdn.Net.MuninNode/Smdn.Net.MuninPlugin/PluginGraphAttributesBuilder.Build.cs

@@ -50,8 +50,8 @@ public IPluginGraphAttributes Build()
     if (width is int w)
       attributes.Add($"graph_width {w}");
 
-    if (updateRate is TimeSpan urate)
-      attributes.Add($"update_rate {(int)urate.TotalSeconds}");
+    if (updateRate is TimeSpan rate)
+      attributes.Add($"update_rate {(int)rate.TotalSeconds}");
 
     attributes.TrimExcess();
 

src/Smdn.Net.MuninNode/Smdn.Net.MuninPlugin/PluginGraphAttributesBuilder.cs

@@ -321,6 +321,8 @@ public PluginGraphAttributesBuilder WithCategoryOther()
   public PluginGraphAttributesBuilder WithCategory(WellKnownCategory category)
     => WithCategory(TranslateWellKnownCategory(category));
 
+#pragma warning disable SA1124 // Do not use #regions
+  #region cSpell:disable
   private static string TranslateWellKnownCategory(WellKnownCategory category)
     => category switch {
       WellKnownCategory.OneSec => "1sec",
@@ -368,6 +370,8 @@ private static string TranslateWellKnownCategory(WellKnownCategory category)
       WellKnownCategory.Wireless => "wireless",
       _ => throw new ArgumentException("not a well known category", paramName: nameof(category)),
     };
+  #endregion cSpell:enable
+#pragma warning restore SA1124
 
   /// <summary>Sets a value for the <c>graph_width</c> and <c>graph_height</c>.</summary>
   /// <seealso cref="WithWidth"/>

tests/Smdn.Net.MuninNode/Smdn.Net.MuninNode/NodeBase.Accept.cs

@@ -153,7 +153,7 @@ await reader.ReadLineAsync(cancellationToken),
       );
 
       var connected = !(
-        client.Client.Poll(1 /*microsecs*/, SelectMode.SelectRead) &&
+        client.Client.Poll(1 /*microseconds*/, SelectMode.SelectRead) &&
         client.Client.Available == 0
       );
 
@@ -190,7 +190,7 @@ ex.InnerException is SocketException
       }
 
       var connected = !(
-        client.Client.Poll(1 /*microsecs*/, SelectMode.SelectRead) &&
+        client.Client.Poll(1 /*microseconds*/, SelectMode.SelectRead) &&
         client.Client.Available == 0
       );
 

tests/Smdn.Net.MuninNode/Smdn.Net.MuninPlugin/PluginFieldBase.cs

@@ -54,15 +54,15 @@ public void Name_Invalid(string name)
   [TestCase("LABEL_")]
   [TestCase("0LABEL")]
   [TestCase("LABEL0")]
-  public void Lavel_Valid(string label)
+  public void Label_Valid(string label)
     => Assert.DoesNotThrow(() => new ConcretePluginField(name: "field", label: label));
 
   [TestCase("")]
   [TestCase("\\LABEL")]
   [TestCase("LABEL\\")]
   [TestCase("#LABEL")]
   [TestCase("LABEL#")]
-  public void Lavel_Invalid(string label)
+  public void Label_Invalid(string label)
     => Assert.Throws<ArgumentException>(() => new ConcretePluginField(name: "field", label: label));
 
   [TestCase("LABEL", "LABEL")]