some comments

DbgProvider/Debugger.psm1

@@ -4335,6 +4335,10 @@ function !teb
        }
 
        $stuff | Format-Table VersionInfo, TargetFriendlyName, DumpPath
+
+    These properties can also be used with, for example, -GroupBy:
+
+        dir C:\temp\dumps\ -File | ForEach-DbgDumpFile { lm ntdll } | Format-Table -GroupBy DumpPath
 #>
 function ForEachDbgDumpFile # Named thusly to avoid a warning about a bad verb name
 {

DbgProvider/public/NameIndexableList.cs

@@ -49,6 +49,10 @@ namespace MS.Dbg
             m_readOnly = true;
         }
 
+        /// <summary>
+        ///    Note that if there are duplicate names, this only returns the first item
+        ///    with the specified name. If there is no item with that name, it will throw.
+        /// </summary>
         public T this[ string name ]
         {
             get

DbgProvider/public/PsContext.cs

@@ -33,6 +33,8 @@ namespace MS.Dbg
 
         public PsContext()
         {
+            // TODO: come up with a way to do a read-only, empty context
+            // OR just get rid of context capture altogether?
             Funcs = new Dictionary< string, ScriptBlock >( 100 );
             Vars = new Dictionary< string, PSVariable >( 100 );
         }