Invoke-ScriptAnalyzer throw a NullReferenceException

[LaurentDardenne]

This code return a string type instead a DiagnosticRecord instance:

$ModuleFullPath='C:\temp\TestRule.psm1'

@'
Function RuleOne{         
 [CmdletBinding()]
 [OutputType([Microsoft.Windows.PowerShell.ScriptAnalyzer.Generic.DiagnosticRecord[]])]

 Param(
       [Parameter(Mandatory = $true)]
       [ValidateNotNullOrEmpty()]
       [System.Management.Automation.Language.FunctionDefinitionAst]
      $FunctionDefinitionAst
 )

  process {
   try {         
    $Result=New-object System.Collections.Arraylist
    $FunctionName=$FunctionDefinitionAst.Name
    $Result.Add('[String]') > $null
#Other error case    
#     $DR = [Microsoft.Windows.PowerShell.ScriptAnalyzer.Generic.DiagnosticRecord]@{"Message"  = $Messages.MeasureRequiresRunAsAdministrator; 
#                                                 "Extent"   = $assignmentAst.Extent;
#                                                 "RuleName" = $PSCmdlet.MyInvocation.InvocationName;
#                                                 "Severity" = "Information"}
#     $Result.Add($DR)
    return $result
   }
   catch
   {
      $ER= New-Object -Typename System.Management.Automation.ErrorRecord -Argumentlist $_.Exception, 
                                                                             "TestRule-$FunctionName", 
                                                                             "NotSpecified",
                                                                             $FunctionDefinitionAst
      $PSCmdlet.ThrowTerminatingError($ER) 
   }          
  }#process
}#

Export-ModuleMember -Function RuleOne
'@ > $ModuleFullPath 

$E=$T=$null
$Ast=[System.Management.Automation.Language.Parser]::ParseFile(
        $ModuleFullPath,
        [ref]$T,
        [ref]$E
      )

RuleOne $ast.Endblock.Statements[0]  
#RuleOne 
Invoke-ScriptAnalyzer -Path $ModuleFullPath -CustomRulePath  $ModuleFullPath   
#Invoke-ScriptAnalyzer : La référence d'objet n'est pas définie à une instance d'un objet.

[kapilmb]

The code returns string type because -

• A string is being added to the array that is returned.
• Declaring outputtype does not convert the output to the desired output type (https://technet.microsoft.com/en-us/library/hh847785.aspx.). The author needs to ensure that the return type and output type are identical. We also have a rule to checks this - PSuseOutputTypeCorrectly

For the other error, where you uncomment $DR assignment and comment out $Result.Add('[String]') > $null, you need to check if $FunctionDefinitionAst is of type FunctionDefinitionAst. ScriptAnalyzer engine gives input of type Ast to the RuleOne function. The function needs to cast to appropriate *Ast type.

Function RuleOne{         
 [CmdletBinding()]
 [OutputType([Microsoft.Windows.PowerShell.ScriptAnalyzer.Generic.DiagnosticRecord[]])]
 Param(
       [Parameter(Mandatory = $true)]
       [ValidateNotNullOrEmpty()]
       [System.Management.Automation.Language.Ast]
      $ast
 )

  process {
   try {
    $Result=New-object System.Collections.Arraylist
    $FunctionDefinitionAst = $ast -as [System.Management.Automation.Language.FunctionDefinitionAst]
    if ($FunctionDefinitionAst -eq $null)
    {        
        $Result
    }           
    $FunctionName=$FunctionDefinitionAst.Name
    $DR = [Microsoft.Windows.PowerShell.ScriptAnalyzer.Generic.DiagnosticRecord]@{"Message"  = $Messages.MeasureRequiresRunAsAdministrator; 
                                                 "Extent"   = $assignmentAst.Extent;
                                                 "RuleName" = $PSCmdlet.MyInvocation.InvocationName;
                                                 "Severity" = "Information"}
     $Result.Add($DR)
     $Result
   }
   catch
   {
      $ER= New-Object -Typename System.Management.Automation.ErrorRecord -Argumentlist $_.Exception, 
                                                                             "TestRule-$FunctionName", 
                                                                             "NotSpecified",
                                                                             $FunctionDefinitionAst
      $PSCmdlet.ThrowTerminatingError($ER) 
   }          
  }#process
}#

[LaurentDardenne]

The author needs to ensure that the return type and output type are identical.

I agree.

I reformulate :
When a custom rule emit a type other than a DiagnosticRecord instance, PSScripanalyzer throw a exception.
In this case why not clearly provide the reason ?
By example : "The rule 'xxx' must emit a DiagnosticRecord instance."

ScriptAnalyzer engine gives input of type Ast to the RuleOne function. The function needs to cast to appropriate *Ast type.

Ah :-/
It is not indicate inside this file.
And in CommunityAnalyzerRules module no rules applies this cast.
I thought the custom rule received a single type of ast that indicated in the parameter.

[kapilmb]

In this case why not clearly provide the reason ?

Yes, you are right. There is clearly a room for improvement here. Will create an issue to track this work item.

I thought the custom rule received a single type of ast that indicated in the parameter.

Correct. The custom rule is given an input of type Ast. Instead of casting it to any *Ast type, which was a bad example that I gave, rules typically use the Ast.Find or Ast.FindAll methods to access the desired node in the AST. If you set your input parameter to FunctionDefinitionAst type or any other *Ast type, it causes downcasting from Ast to FunctionDefinitionAst and it is not guaranteed to succeed unless the given Ast is of type FunctionDefinitionAst.

[LaurentDardenne]

There is clearly a room for improvement here. Will create an issue to track this work item.

In this case, here are anothers examples.
I thinks that the exception management of PSScriptAnalyzer should be most user friendly.

$ModuleFullPath='C:\temp\TestRule.psm1'
@'
Function RuleOne{         
 [CmdletBinding()]
 [OutputType([Microsoft.Windows.PowerShell.ScriptAnalyzer.Generic.DiagnosticRecord[]])]

 Param(
       [Parameter(Mandatory = $true)]
       [ValidateNotNullOrEmpty()]
       [System.Management.Automation.Language.FunctionDefinitionAst]
      $FunctionDefinition #there is no respect for the naming convention (ending with 'ast' or 'token')
 )

  process {             
   try {         
    $Result=New-object System.Collections.Arraylist
    $DR = [Microsoft.Windows.PowerShell.ScriptAnalyzer.Generic.DiagnosticRecord]@{"Message"  = 'RuleOne'; 
                                                 "Extent"   = $FunctionDefinitionast.Extent;
                                                 "RuleName" = $PSCmdlet.MyInvocation.InvocationName;
                                                 "Severity" = "Information"}
    $Result.Add($DR) >$null
    $result        

   }
   catch
   {
      $ER= New-Object -Typename System.Management.Automation.ErrorRecord -Argumentlist $_.Exception, 
                                                                             "Test use case-$FunctionDefinitionAst.Name", 
                                                                             "NotSpecified",
                                                                             $FunctionDefinitionAst
      $PSCmdlet.ThrowTerminatingError($ER) 
   }          
  }#process
}#RuleOne

Export-ModuleMember -Function RuleOne
'@ > $ModuleFullPath 
Invoke-ScriptAnalyzer -Path $ModuleFullPath -CustomRulePath  $ModuleFullPath
# Invoke-ScriptAnalyzer : Une exception de type 'System.Exception' a été levée.
# At line:1 char:1
# + Invoke-ScriptAnalyzer -Path $ModuleFullPath -CustomRulePath  $ModuleF ...
# + ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
#     + CategoryInfo          : ResourceExists: (Microsoft.Windo....ScriptAnalyzer:ScriptAnalyzer) [Invoke-ScriptAnalyzer], Exception
#     + FullyQualifiedErrorId : Cannot find ScriptAnalyzer rules in the specified path,Microsoft.Windows.PowerShell.ScriptAnalyzer.Commands.InvokeScriptAnalyzerCommand

The naming convention (ending with 'ast' or 'token') must be explicit in the documentation.

IHMO, the text of the exception is to fuzzy.
CategoryInfo is not correct, because ResourceExists is for "An error that occurs when a resource already exists."
I thinks that 'ObjectNotFound' or 'InvalidData' is preferable.

FullyQualifiedErrorId contains the reason of the error, when i use the 'ErrorView' variable the result is for me useless :

$ErrorView='CategoryView'
Invoke-ScriptAnalyzer -Path $ModuleFullPath -CustomRulePath  $ModuleFullPath
ResourceExists: (Microsoft.Windo....ScriptAnalyzer:ScriptAnalyzer) [Invoke-ScriptAnalyzer], Exception

A example with Get-Childitem :

dir z:\
ObjectNotFound: (z:String) [Get-ChildItem], DriveNotFoundException
$ErrorView='normalView'
dir z:\
dir : Cannot find drive. A drive with the name 'z' does not exist.
At line:1 char:1
+ dir z:\
+ ~~~~~~~
    + CategoryInfo          : ObjectNotFound: (z:String) [Get-ChildItem], DriveNotFoundException
    + FullyQualifiedErrorId : DriveNotFound,Microsoft.PowerShell.Commands.GetChildItemCommand

An another example :

$ModuleFullPath='C:\temp\TestRule.psm1'
@'
Function RuleOne{         
 [CmdletBinding()]
 [OutputType([Microsoft.Windows.PowerShell.ScriptAnalyzer.Generic.DiagnosticRecord[]])]

 Param(
       [Parameter(Mandatory = $true)]
       [ValidateNotNullOrEmpty()]
       [System.Management.Automation.Language.FunctionDefinitionAst]
      $FunctionDefinitionast
 )

  process  # !!!  no '{'             
   try {         
    $Result=New-object System.Collections.Arraylist
    $DR = [Microsoft.Windows.PowerShell.ScriptAnalyzer.Generic.DiagnosticRecord]@{"Message"  = 'RuleOne'; 
                                                 "Extent"   = $FunctionDefinitionast.Extent;
                                                 "RuleName" = $PSCmdlet.MyInvocation.InvocationName;
                                                 "Severity" = "Information"}
    $Result.Add($DR) >$null
    $result        
   }
   catch
   {
      $ER= New-Object -Typename System.Management.Automation.ErrorRecord -Argumentlist $_.Exception, 
                                                                             "Test use case-$FunctionDefinitionAst.Name", 
                                                                             "NotSpecified",
                                                                             $FunctionDefinitionAst
      $PSCmdlet.ThrowTerminatingError($ER) 
   }          
  }#process
}#

Export-ModuleMember -Function RuleOne
'@ > $ModuleFullPath 
Invoke-ScriptAnalyzer -Path $ModuleFullPath -CustomRulePath  $ModuleFullPath
WARNING: Cannot find rule extension 'C:\temp\TestRule.psm1'.
Invoke-ScriptAnalyzer : Une exception de type 'System.Exception' a été levée.

In this case i get a warning but the principal is that i dont know what happened here.
The exception on syntax error in the module is not propagated.

ipmo $ModuleFullPath -EV E -EA 'SilentlyContinue'
$E

[kapilmb]

@LaurentDardenne Thanks for pointing out the limitations. I will leave this issue open till we address the issues reported here. Please feel free to post any other exceptions that you find needs improvement.

[LaurentDardenne]

In this following case

Function TestParameterSet{
 [CmdletBinding(DefaultParameterSetName='F2')]       
  Param (
    [Parameter(Position=2")]
   [string] $A
   )
   Write-Host"ParameterSetName =$($PsCmdlet.ParameterSetName)"
}

The error message is correct :

$ErrorView='NormalView'
Invoke-ScriptAnalyzer -Path 'C:\temp\Rule 4.ps1'

# Invoke-ScriptAnalyzer : Parse error in file C:\temp\Rule 4.ps1:  Missing statement after '=' in named argument at line
# 4 column 25.
# At line:1 char:1
# + Invoke-ScriptAnalyzer -Path 'C:\temp\Rule 4.ps1'
# + ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
#     + CategoryInfo          : ParserError: (MissingExpressionInNamedArgument:String) [Invoke-ScriptAnalyzer], ParseException
#     + FullyQualifiedErrorId : Parse error in file C:\temp\Rule 4.ps1:  Missing statement after '=' in named argument at line 
#                               4 column 25.,Microsoft.Windows.PowerShell.ScriptAnalyzer.Commands.InvokeScriptAnalyzerCommand

But we have 7 errors :

$ErrorView='CategoryView'
Invoke-ScriptAnalyzer -Path 'C:\temp\Rule 4.ps1'

ParserError: (MissingExpressionInNamedArgument:String) [Invoke-ScriptAnalyzer], ParseException
ParserError: (TerminatorExpectedAtEndOfString:String) [Invoke-ScriptAnalyzer], ParseException
ParserError: (MissingEndParenthesisInExpression:String) [Invoke-ScriptAnalyzer], ParseException
ParserError: (InvalidFunctionParameter:String) [Invoke-ScriptAnalyzer], ParseException
ParserError: (MissingEndParen...onParameterList:String) [Invoke-ScriptAnalyzer], ParseException
ParserError: (MissingEndCurlyBrace:String) [Invoke-ScriptAnalyzer], ParseException
ParserError: (ParameterAttrib...ntOrScriptBlock:String) [Invoke-ScriptAnalyzer], ParseException

The first should be sufficient.

[LaurentDardenne]

Just a question about the exceptions mentioned above.
According to this:

It generates DiagnosticResults (errors and warnings) to inform users about potential code defects and suggests possible solutions for improvements.

A file with an incorrect syntax it should not send into the pipeline a DiagnosticRecord with severity error instead of an exception ?

[kapilmb]

PSScriptAnalyzer only relays the parse errors generated by the PowerShell engine. DiagnosticRecords do not wrap ParseErrors because they serve different functions. A DiagnosticRecords is intended to convey issues that are encoutered on top of ParseError-free PowerShell code.

[LaurentDardenne]

This code throw an exception (it is correct) :

$code=@'
function SuppressTwoVariables{
    [System.Diagnostics.CodeAnalysis.SuppressMessageAttribute("PSAvoidDefaultValueForMandatoryParameter", "b")]
    [System.Diagnostics.CodeAnalysis.SuppressMessageAttribute("PSAvoidDefaultValueForMandatoryParameter", "noexist")]
    [CmdletBinding()]
    Param(
        [Parameter(Mandatory=$true)]
        [string]
        $a="unused",

        [Parameter(Mandatory=$true)]
        [int]
        $b=3
        )
}
'@
Invoke-ScriptAnalyzer -ScriptDefinition $code|Format-List

But the error message and the FullyQualifiedErrorId are a bit strange :

Invoke-ScriptAnalyzer : Suppression Message Attribute error at line 3 in script definition : 
Cannot find any DiagnosticRecord with the Rule Suppression ID noexist.
At line:1 char:1
+ Invoke-ScriptAnalyzer -ScriptDefinition $code|Format-List
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidArgument: (Microsoft.Windo...RuleSuppression:RuleSuppression) [Invoke-ScriptAnalyzer], ArgumentException
    + FullyQualifiedErrorId : Suppression Message Attribute error at line 3 in script definition : 
Cannot find any DiagnosticRecord with the Rule Suppression ID noexist.,Microsoft.Windows.PowerShell.ScriptAnalyzer.Commands.InvokeScriptAnalyzerCommand

[LaurentDardenne]

This error do not contains the original exception :

Invoke-ScriptAnalyzer -Path "$pwd\OptimizationRules.psm1" -CustomRulePath "$pwd\OptimizationRules.psd1" -Verbose
VERBOSE: Settings not provided. Will look for settings file in the given path
G:\PS\PSScriptAnalyzerRules\Modules\OptimizationRules\Logs\OptimizationRules.psm1.
VERBOSE: Cannot find a settings file in the given path
G:\PS\PSScriptAnalyzerRules\Modules\OptimizationRules\Logs\OptimizationRules.psm1.
VERBOSE: Checking module 'G:\PS\PSScriptAnalyzerRules\Modules\OptimizationRules\Logs\OptimizationRules.psd1' ...
WARNING: Cannot find rule extension
'G:\PS\PSScriptAnalyzerRules\Modules\OptimizationRules\Logs\OptimizationRules.psd1'.
Invoke-ScriptAnalyzer : Une exception de type 'System.Exception' a été levée.
At line:1 char:1
+ Invoke-ScriptAnalyzer -Path "$pwd\OptimizationRules.psm1" -CustomRule ...
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : ResourceExists: (Microsoft.Windo....ScriptAnalyzer:ScriptAnalyzer) [Invoke-ScriptAnalyzer], Exception
    + FullyQualifiedErrorId : Cannot find ScriptAnalyzer rules in the specified path,Microsoft.Windows.PowerShell.ScriptAnalyzer.Commands.InvokeScriptAnalyzerCommand

With this single message, I do not know the real reason.

In my case the import-module fail because a file was missing :

ipmo .\OptimizationRules.psd1
Import-LocalizedData : Cannot find the Windows PowerShell data file 'OptimizationRules.Resources.psd1' in directory
'G:\PS\PSScriptAnalyzerRules\Modules\OptimizationRules\Logs\fr-FR\', or in any parent culture directories.