Add unfolding command to tree; fix RemesPath bug

ADDED
- "Make this visible" option to tree view's right click context menu
    to undo code folding in Notepad++ as necessary
    to make the node visible in the editor.
    Responds to issue #103.

FIXED
- Bug where s_join in RemesPath would treat some non-strings as empty strings,
    rather than correctly throwing an error.

CODEBASE (no change to public API)
- Removed the unnecesary IScintillaGateway interface,
    because it makes no sense to create an interface that will
    only have one implementation.
    Now all references to IScintillaGateway instead go to ScintillaGateway.
- In CI_build.yml, update actions/checkout to v6, addressing dependabot PR #109.

DOCS
- Make docs for the "Dump selected..." commands easier to read.

Author: molsonkiko

.github/workflows/CI_build.yml

@@ -22,7 +22,7 @@ jobs:
 
     steps:
     - name: Checkout repo
-      uses: actions/checkout@v5
+      uses: actions/checkout@v6
 
     - name: Add msbuild to PATH
       uses: microsoft/setup-msbuild@v2.0.0

CHANGELOG.md

@@ -52,6 +52,10 @@ and this project adheres to [Semantic Versioning](http://semver.org/).
 
 ## [9.0.0] - (UNRELEASED) YYYY-MM-DD
 
+### Added
+
+1. New [`Make this visible` option](/docs/README.md#reveal-tree-nodes-hidden-by-code-folding-added-in-v9) on the tree view's right click context menu to reveal nodes hidden by code folding. This was first requested in [issue 103](https://github.com/molsonkiko/JsonToolsNppPlugin/issues/103).
+
 ### Fixed
 
 1. __Correctly parse all ANSI files.__ Previously JsonTools assumed that Notepad++ used UTF8 to internally represent every document, and while that appears to have worked fine for most encodings, it would specifically fail on ANSI-encoded documents containing non-ASCII.
@@ -64,6 +68,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/).
 3. Minor bug where some JsonGrepper tests would always fail because they queried an external API that had previously returned JSON but now returns HTML.
 4. Bug where editing a document in [multi-selection mode](/docs/README.md#working-with-selections) would sometimes cause selections to be forgotten (fixed in [v8.4.0.3](https://github.com/molsonkiko/JsonToolsNppPlugin/commit/16f0358274c662273a2015994c5ab22d22fd23ab))
 5. Bug ([issue 104](https://github.com/molsonkiko/JsonToolsNppPlugin/issues/104)) where [error form](/docs/README.md#error-form-and-status-bar) would not always re-parse the document after the `Refresh with current errors` right-click context menu item on the error form was clicked.
+6. Bug where [`s_join` RemesPath function](/docs/RemesPath.md#non-vectorized-functions) would not throw an exception when the second argument was an array containing non-strings (previously it would silently treat nulls, objects and arrays as though they were empty strings).
 
 ## [8.4.0] - 2025-05-04
 

JsonToolsNppPlugin/Forms/TreeViewer.Designer.cs

@@ -99,6 +99,7 @@ public partial class TreeViewer : Form
         private static MouseEventHandler selectThisHandler = null;
         private static MouseEventHandler showSortFormHandler = null;
         private static MouseEventHandler selectAllChildrenHandler = null;
+        private static MouseEventHandler makeThisVisibleHandler = null;
 
         public TreeViewer(JNode json)
         {
@@ -1142,12 +1143,24 @@ private void Tree_NodeMouseClick(object sender, TreeNodeMouseClickEventArgs e)
                     sortFormOpenItem.Visible = false;
                     selectAllChildrenItem.Visible = false;
                 }
+                var makeThisVisibleItem = NodeRightClickMenu.Items[7];
+                makeThisVisibleItem.MouseUp -= makeThisVisibleHandler;
+                makeThisVisibleHandler = new MouseEventHandler(
+                    (s2, e2) =>
+                    {
+                        int posOfNode = NodePosInJsonDoc(node);
+                        int lineOfNode = Npp.editor.LineFromPosition(posOfNode);
+                        Npp.editor.EnsureVisible(lineOfNode);
+                        Npp.editor.GotoPos(posOfNode);
+                    }
+                );
+                makeThisVisibleItem.MouseUp += makeThisVisibleHandler;
                 NodeRightClickMenu.Show(MousePosition);
             }
             if (Translator.TryGetTranslationAtPath(new string[] {"forms", "TreeViewer", "controls"}, out JNode controlsTranslationsNode)
                 && controlsTranslationsNode is JObject controlTranslations)
             {
-                string[] controlNames = new string[] { "CopyValueMenuItem", "CopyKeyItem", "CopyPathItem", "ToggleSubtreesItem", "SelectThisItem", "OpenSortFormItem", "SelectAllChildrenItem" };
+                string[] controlNames = new string[] { "CopyValueMenuItem", "CopyKeyItem", "CopyPathItem", "ToggleSubtreesItem", "SelectThisItem", "OpenSortFormItem", "SelectAllChildrenItem", "MakeThisVisibleItem" };
                 for (int ii = 0; ii < controlNames.Length; ii++)
                 {
                     string name = controlNames[ii];

JsonToolsNppPlugin/Forms/TreeViewer.cs

@@ -125,7 +125,7 @@
         AAEAAAD/////AQAAAAAAAAAMAgAAAFdTeXN0ZW0uV2luZG93cy5Gb3JtcywgVmVyc2lvbj00LjAuMC4w
         LCBDdWx0dXJlPW5ldXRyYWwsIFB1YmxpY0tleVRva2VuPWI3N2E1YzU2MTkzNGUwODkFAQAAACZTeXN0
         ZW0uV2luZG93cy5Gb3Jtcy5JbWFnZUxpc3RTdHJlYW1lcgEAAAAERGF0YQcCAgAAAAkDAAAADwMAAACw
-        DgAAAk1TRnQBSQFMAgEBDwEAAdABAQHQAQEBEAEAARABAAT/AQkBAAj/AUIBTQE2AQQGAAE2AQQCAAEo
+        DgAAAk1TRnQBSQFMAgEBDwEAAdgBAQHYAQEBEAEAARABAAT/AQkBAAj/AUIBTQE2AQQGAAE2AQQCAAEo
         AwABQAMAAUADAAEBAQABCAYAARAYAAGAAgABgAMAAoABAAGAAwABgAEAAYABAAKAAgADwAEAAcAB3AHA
         AQAB8AHKAaYBAAEzBQABMwEAATMBAAEzAQACMwIAAxYBAAMcAQADIgEAAykBAANVAQADTQEAA0IBAAM5
         AQABgAF8Af8BAAJQAf8BAAGTAQAB1gEAAf8B7AHMAQABxgHWAe8BAAHWAucBAAGQAakBrQIAAf8BMwMA

JsonToolsNppPlugin/Forms/TreeViewer.resx

@@ -1979,7 +1979,9 @@ public static JNode StringJoin(List<JNode> args)
             for (int ii = 1; ii < itbl.Length; ii++)
             {
                 sb.Append(sep);
-                sb.Append((string)itbl[ii].value);
+                if (!(itbl[ii].value is string s))
+                    throw new RemesPathException($"Element {ii} of the array had type {JNode.FormatDtype(itbl[ii].type)}, but the array must contain only strings.");
+                sb.Append(s);
             }
             return new JNode(sb.ToString(), Dtype.STR, 0);
         }

JsonToolsNppPlugin/JSONTools/RemesPathFunctions.cs

@@ -121,7 +121,6 @@
     <Compile Include="PluginInfrastructure\DllExport\DllExportAttribute.cs" />
     <Compile Include="PluginInfrastructure\Docking_h.cs" />
     <Compile Include="PluginInfrastructure\GatewayDomain.cs" />
-    <Compile Include="PluginInfrastructure\IScintillaGateway.cs" />
     <Compile Include="PluginInfrastructure\MenuCmdID_h.cs" />
     <Compile Include="PluginInfrastructure\Msgs_h.cs" />
     <Compile Include="PluginInfrastructure\NotepadPPGateway.cs" />