Armistice said:
Heap, if you're going to to adhere to the SC 2.0 paradigm, wherever possible use the tools already created. Since you no longer have to worry about how many 'legal' feats you give a character, a quality like combat training really shouldn't be neccessary. The ability to choose feats for an npc freely should take care of it.
If this was for D&D in stead of D20M, I'd keep it, but change the benefit...
As is, D&D proficiency feats go like:
Light Armor - all light armors
Medium Armor - all medium armors
Heavy Armor - all heavy armors
Shield - all sheilds except tower shields
Tower Shield - tower shields
Light Weapons - all light weapons
Martial Weapon - choose only one martial weapon
Exotic Weapon - choose only one exotic weapon
Now, tower shields and exotic weapons are kind of special cases, and are balanced by the fact that they require specific seperate feats, so ignore them for the moment.
All the other feats grant proficiencies in a broad category of weapons or armor... except for the Martial Weapon feat. And this is despite the fact that most combat classes automatically are proficient in the entire Martial Weapon category, effectively getting DOZENS of feats for free. So, to emulate that particular benefit of the combat-oriented classes, for D&D, I'd reword the attribute like so...
Combat Training (+0.5 CR)
The NPC is trained for combat (e.g. Soldier, SWAT personnel, knight). The NPC is proficient with all simple and martial weapons.
But, of course, D20 Modern proficiencies work a bit differently from the D&D standard.