Michael, I'm no expert but some of your references to DATAIN include brackets and an array element identifier after them, others don't.
In the serin command does picbasic know to put the received data byte into sequential bytes of the datain array, or should yo be putting that command in a loop and incrementing the array pointer with every pass through ?