Tarjan's Algorithm failure case, what am I missing?
Answer:  the low-link value is not the identifier.  I eventually found this while researching during the writing of this question.  Question posted because I wrote it and hey, maybe this will help someone else.

Everything up to the current node is popped off the stack when identifying an SCC.  My misunderstanding was trying to pop only what has the same low-link value.