Fix inside-outside for implicit targets (#398)

* Fix inside outside inference

* Attempt to fix inside-outside for implicit target

* Fixes

* Revert "Fixes"

This reverts commit 80a9ca1.

* Another attempt at implicit fix

* Add log fix back

* Revert stuff

Author: pokey

src/core/commandRunner/CommandRunner.ts

@@ -66,6 +66,11 @@ export default class CommandRunner {
         action.getTargetPreferences(...extraArgs)
       );
 
+      if (this.graph.debug.active) {
+        this.graph.debug.log("Full targets:");
+        this.graph.debug.log(JSON.stringify(targets, null, 3));
+      }
+
       const processedTargetsContext: ProcessedTargetsContext = {
         currentSelections:
           vscode.window.activeTextEditor?.selections.map((selection) => ({

src/processTargets/index.ts

@@ -270,17 +270,17 @@ function processPrimitiveTarget(
   const modifiedSelections = markSelections.flatMap((markSelection) =>
     processModifier(context, target, markSelection)
   );
+  if (target.isImplicit) {
+    modifiedSelections.forEach((typedSelection) => {
+      typedSelection.context.isRawSelection = true;
+    });
+  }
+
   const typedSelections = modifiedSelections.map(
     ({ selection, context: selectionContext }) =>
       processSelectionType(context, target, selection, selectionContext)
   );
 
-  if (target.isImplicit) {
-    typedSelections.forEach((typedSelection) => {
-      typedSelection.selectionContext.isRawSelection = true;
-    });
-  }
-
   return typedSelections.map((selection) =>
     processPosition(context, target, selection)
   );