Fix #122

Mirror Z axis of the attach node orientation to stop "back' and "front"
nodes looking inside the part.

Plugins/Source/CHANGES.txt

@@ -1 +1,2 @@
 Pre-release changes:
+- [Fix] #122: For some parts attach point is wrongly detected.

Plugins/Source/KISAddonPointer.cs

@@ -681,8 +681,12 @@ public enum PointerState {
 
   /// <summary>Sets pointer origin to the current attachment node</summary>
   private static void UpdatePointerAttachNode() {
-    pointerNodeTransform.localPosition = GetCurrentAttachNode().position;
-    pointerNodeTransform.localRotation = KIS_Shared.GetNodeRotation(GetCurrentAttachNode());
+    var node = GetCurrentAttachNode();
+    pointerNodeTransform.localPosition = node.position;
+    // HACK(ihsoft): For some reason Z orientation axis is get mirrored in the parts. It results in
+    //   a weird behavior when aligning parts in "back" or "front" node attach modes. It may be a
+    //   KIS code bug but I gave up finding it.
+    pointerNodeTransform.localRotation = KIS_Shared.GetNodeRotation(node, mirrorZ: true);
   }
 
   /// <summary>Destroyes object(s) allocated to represent a pointer.</summary>

Plugins/Source/KIS_Shared.cs

@@ -495,8 +495,7 @@ public enum MessageAction {
                                Quaternion adjust) {
     Vector3 refDir = hit.transform.TransformDirection(Vector3.up);
     Quaternion rotation = Quaternion.LookRotation(hit.normal, refDir);
-    source.rotation = (rotation * adjust) * childNode.localRotation;
-    source.position = source.position - (childNode.position - hit.point);
+    MoveAlign(source, childNode, hit.point, rotation * adjust);
   }
 
   public static void MoveAlign(Transform source, Transform childNode, Transform target,
@@ -549,8 +548,17 @@ public enum MessageAction {
     return null;
   }
 
-  public static Quaternion GetNodeRotation(AttachNode attachNode) {
-    return Quaternion.LookRotation(attachNode.orientation);
+  /// <summary>Returns a rotation for the attach node.</summary>
+  /// <param name="attachNode">A node to get orientation from.</param>
+  /// <param name="mirrorZ">If <c>true</c> then Z axis in the node's orientation will be mirrored.
+  /// E.g. <c>(1, 1, 1)</c> will be translated into <c>(1, 1, -1)</c>.</param>
+  /// <returns>Rotation quaternion.</returns>
+  public static Quaternion GetNodeRotation(AttachNode attachNode, bool mirrorZ = false) {
+    var orientation = attachNode.orientation;
+    if (mirrorZ) {
+      orientation.z = -orientation.z;
+    }
+    return Quaternion.LookRotation(orientation);
   }
 
   public static void AssignAttachIcon(Part part, AttachNode node, Color iconColor,