Fix issue with C# node identification

Finally we need some contextual matchers, bummer.

Closes #51

Author: aaronik

lua/treewalker/nodes.lua

@@ -10,8 +10,13 @@ local TARGET_BLACKLIST_TYPE_MATCHERS = {
   "else",               -- else/elseif statements (lua)
   "elif",               -- else/elseif statements (py)
   "end_tag",            -- html closing tags
-  "declaration_list",   -- C# class blocks
+  "declaration_list",   -- C# class blocks - these are structural containers
   "compound_statement", -- C blocks when defined under their fn names like a psycho
+  -- Parent-child patterns: "child_type:parent_type"
+  "block:method_declaration",
+  "block:constructor_declaration",
+  "block:destructor_declaration",
+  "block:class_declaration",
 }
 
 local HIGHLIGHT_BLACKLIST_TYPE_MATCHERS = {
@@ -33,8 +38,20 @@ local M = {}
 ---@return boolean
 local function is_matched_in(node, matchers)
   for _, matcher in ipairs(matchers) do
-    if node:type():match(matcher) then
-      return true
+    -- Check for parent-child pattern: "child_type:parent_type"
+    if matcher:find(":") then
+      local child_type, parent_type = matcher:match("^([^:]+):([^:]+)$")
+      if child_type and parent_type then
+        local parent = node:parent()
+        if node:type():match(child_type) and parent and parent:type():match(parent_type) then
+          return true
+        end
+      end
+    else
+      -- Simple type matching
+      if node:type():match(matcher) then
+        return true
+      end
     end
   end
   return false

tests/treewalker/c_sharp_spec.lua

@@ -16,13 +16,25 @@ describe("In a C Sharp file", function()
     h.assert_cursor_at(27, 5)
     tw.move_down()
     h.assert_cursor_at(50, 5)
+    tw.move_up()
+    h.assert_cursor_at(27, 5)
     tw.move_in()
-    h.assert_cursor_at(52, 9)
+    h.assert_cursor_at(29, 9)
+    tw.move_down()
+    h.assert_cursor_at(31, 9)
+    tw.move_down()
+    h.assert_cursor_at(36, 9)
+    tw.move_down()
+    h.assert_cursor_at(41, 9)
+    tw.move_up()
+    h.assert_cursor_at(36, 9)
+    tw.move_up()
+    h.assert_cursor_at(31, 9)
     tw.move_out()
-    h.assert_cursor_at(50, 5)
+    h.assert_cursor_at(27, 5)
   end)
 
-  it("swaps down", function()
+  it("swaps down classes", function()
     vim.fn.cursor(7, 5)
     local first_block = lines.get_lines(6, 24)
     local second_block = lines.get_lines(26, 48)
@@ -32,7 +44,17 @@ describe("In a C Sharp file", function()
     h.assert_cursor_at(31, 5)
   end)
 
-  it("swaps up", function()
+  it("swaps down functions", function()
+    vim.fn.cursor(31, 9)
+    local first_block = lines.get_lines(31, 34)
+    local second_block = lines.get_lines(36, 39)
+    tw.swap_down()
+    assert.same(second_block, lines.get_lines(31, 34))
+    assert.same(first_block, lines.get_lines(36, 39))
+    h.assert_cursor_at(36, 9)
+  end)
+
+  it("swaps up classes", function()
     vim.fn.cursor(27, 5)
     local first_block = lines.get_lines(6, 24)
     local second_block = lines.get_lines(26, 48)
@@ -41,4 +63,15 @@ describe("In a C Sharp file", function()
     assert.same(first_block, lines.get_lines(30, 48))
     h.assert_cursor_at(7, 5)
   end)
+
+  it("swaps up functions", function()
+    vim.fn.cursor(36, 9)
+    local first_block = lines.get_lines(31, 34)
+    local second_block = lines.get_lines(36, 39)
+    tw.swap_up()
+    assert.same(second_block, lines.get_lines(31, 34))
+    assert.same(first_block, lines.get_lines(36, 39))
+    h.assert_cursor_at(31, 9)
+  end)
+
 end)