Goto Declaration Unexpected Behavior for labels with hyphens

PyCharm 2022.2.3 (Professional Edition) Build #PY-222.4345.23, built on October 10, 2022

Pop!_OS 22.04 LTS

TeXiFy IDEA 0.7.23

What I did (steps to reproduce)

• Create a section with a label that has a hyphen in it
• Later, create a reference back to that section.
• Place the cursor to the left of the hyphen Goto Declaration (ctrl+b or ctrl+click)
• Observe “No usages found” message (incorrect)
• Place cursor to right of the hyphen and attempt Goto Declaration again
• Observe Expected Behavior

Minimal example to reproduce the problem

Inconsistent/Incorrect Behavior

\documentclass[12pt]{article}
\usepackage{amsmath}
\begin{document}
	\section{Foo}\label{sec:foo-bar}
		Foo Bar~\ref{sec:foo-bar}
\end{document}

Consistent/Correct Behavior

\documentclass[12pt]{article}
\usepackage{amsmath}
\begin{document}
	\section{Foo}\label{sec:foobar}
		Foo Bar~\ref{sec:foobar}
\end{document}

Expected behavior

Goto Declaration will work consistently on both \ref{sec:foo-bar} and \ref{sec:foobar}

Actual behavior

Please refer back to steps to reproduce for more info.

When attempting a ctrl+click, one may notice that when hovering to the left of a hyphen, the blue underline does not extend across the whole reference. Moving the mouse to the right of the hyphen works as expected.

Notes

Adding more hyphens does not exacerbate the issue, only the first one suffers splitting.

Removing the : does not have any effect.

[jojo2357]

Sorry, not trying to rush you or anything, just gathering information since I wasn’t sure if it would help or not

[PHPirates]

Yes yes it’s all the same issue, I know what the problem is, we made hyphens a separate lexer token to enable folding them, but reference resolving works on the leaf psi element at the caret, so that should be one element and not be split up. Should probably just not have separate hyphen lexer token in required arguments, but that’s no fun in the lexer. Unfortunately I think doing this at parser level will not work, but we’ll see.