scoped_connection.h has several constructors that are decorated with [[no_discard]]. It makes no sense for constructors and should be removed. Compilers generate warnings for them. If warnings are treated as errors, it will break a build.

P.S. You overuse [[no_discard]]. Use it only for functions that allocate memory or resources that must be released.