The wiring in MSR_24 for Q[i-1] and Q[i+1] of each MSR_block seem to run the opposite of what I’d expect - Q[i-1] is fed by the output of the block with one higher i, Q[i+1] is fed by the output of the block with one lower i. Is that on purpose or is that a mistake?
That is indeed a mistake in output naming. Q[i + 1] is meant to signify the output of the FF after the current one, i.e. towards the tail. In standard naming conventions this would be the FF numbered i + 1. However because we are numbering in reverse in the project, the next FF is numbered i - 1.
In summary, the wiring is okay and the discrepancy is due to different naming conventions.