Currently, implementation phase has a gate for approval after the phase is complete. This gate should not be removed, but there should be another gate added that pauses between each commit.
The model is encouraged to do work in small commits already, but when this gate is on, it should stop and wait for approval before committing it and moving on to the next commit.
This means the diff viewer needs to show uncommitted changes (including new files- keep this in mind as unstaged new files don't show up in the diff by default).
Another thing to note is that currently- it looks like all changes are made then split into several commits. This model doesn't work with this gate, so either the gate being enabled should change this behavior, or the behavior as a whole should be changed
Currently, implementation phase has a gate for approval after the phase is complete. This gate should not be removed, but there should be another gate added that pauses between each commit.
The model is encouraged to do work in small commits already, but when this gate is on, it should stop and wait for approval before committing it and moving on to the next commit.
This means the diff viewer needs to show uncommitted changes (including new files- keep this in mind as unstaged new files don't show up in the diff by default).
Another thing to note is that currently- it looks like all changes are made then split into several commits. This model doesn't work with this gate, so either the gate being enabled should change this behavior, or the behavior as a whole should be changed