PR Summary

Fixes a crash caused by an incorrect check with the IsWithin extension method. As the name suggests, it's supposed to check if an extent is within another extent but the check did not work properly. For example, suppose you have a PipelineAst like this: $Var1 | where {$true} and you want to check if $Var1 IsWithin that ast, the check would correctly return true. However, if you formatted it like this:

    $Var1 | where {
$true}

it would incorrectly return false because the end column of the PipelineAst is lower than the end column of the variable $Var1.

I've simplified the check to just look at the start and end offsets and it's now working as expected.
I'm not sure if a test is really needed since this is just a fix for an internal method that was simply wrong. But if it needs to be added I suppose I can just add a test like "It should not crash"

$Var1 = ls
$Var1 = $Var1 | where {
    $true
}
$Var1.<Tab>

PR Context

Fixes #25694

PR Checklist

• PR has a meaningful title
  • Use the present tense and imperative mood when describing your changes
• Summarized changes
• Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
• This PR is ready to merge. If this PR is a work in progress, please open this as a Draft Pull Request and mark it as Ready to Review when it is ready to merge.
• Breaking changes
  • None
  • OR
  • Experimental feature(s) needed
    • Experimental feature name(s):
• User-facing changes
  • Not Applicable
  • OR
  • Documentation needed
    • Issue filed:
• Testing - New and feature
  • N/A or can only be tested interactively
  • OR
  • Make sure you've added a new test if existing tests do not effectively test the code changed