please, consider the following mwe:
\documentclass[margin=3mm]{standalone}
\usepackage{siunitx}
\sisetup{input-symbols = {( - )},
table-space-text-post=***,
group-digits=integer,
group-four-digits,
group-separator={,}
}
\begin{document}
\begin{tabular}{S[table-format=5.5]}
\hline
123.45678 \\
1234.5678 \\
(12.345) \\
(123.45) \\
\hline
\end{tabular}
\end{document}
which gives:
as you can see, results in the first three rows are as expected, but in the last two rows the comma should not appear. did i miss something or this is bug?
edit:
recent siunitx
version v2.7n, last revised 2018-04-27 (available in miktex
from 2018.05.03) contain fix as package author Joseph Wright gives in his answer below the following result:
this update is an important step towards a complete solution to the presented problem. apparently distinguishing between input symbols (
from numbers is (very) difficult problem.