I am not going to comment or reply about your code because I never found XML useful enough to even bother learning.
What I am adding to the discussion is an example case in support of Phanx's suggestion of using override bindings. In my AddOn SmartRes2, the key binds are overrides. You will not see them in the keybindings window (Esc, Keybindings). Because SmartRes2 deals with all things raising players from the dead after combat, here's what a player can do:
- Use the Keybindings screen (Esc, Keybindings) to set a key or combination of keys to perform action X while in combat
- Bind the same key or combination of keys in SmartRes2, which will override the bindings when the player is out of combat
This means the same keybinds can be used both in and out of combat for completely different purposes.