I don't work with PICs (and not plan forever
), but If I remember PICs use about 13V on MCLR pin to enter(or all time during programming) in programming mode.
In this case:
1. reset circuit possible can limit this voltage to Vcc+Vd(due to protection diode or because it exceed absolute maximum voltage ratings conditions if protection diodes not exist)
2.R2 also can limit voltage programming -> you can expect problems during ICSP.
In this situation I will add resistor between \RESET output and MCLR pin on mcu (possible>=10k it depend what is PIC programmer current capability on output for MCLR, maybe 1k is enough?). Maybe this will not work for >10k values in normal reset situation if PIC have internal pull up for MCLR pin, but you must read docs and chose good value.