Your diagram is weird. Isn’t the opnsense box supposed to replace the router. Or at least it should be between the existing router and you clients. Pc 1 go to opnsense Lan. And opnsense wan to the router and internet.
You’re creating all kinds of loops which is generally a bad idea. Your data should flow in 1 direction like a tree.
Unless there are a lot of details you’re not sharing.
Also remember generally a router is not a switch. Plug all your PCs into a switch. Plug a wifi access point into the switch. And then have the switch go into the lan of your opnsense.
And then have the wan go out to the internet.
You can still do this but as others have said you need to have 2 separate lans. Your old Lan can go to PC 1 from old router. Then opnsense wan goes to your existing Lan and importantly you are now creating a new Lan on the lan side of opnsense. Here you can connect the PC 2 to test with. Each PC should only be on 1 Lan and each Lan should have a separate subnet.
See this post and the last comment even references a diagram to exactly what you want: https://forum.opnsense.org/index.php?topic=32774.0
There are all kinds of routing protocols and algorithms at play which don’t like loops and multiple routers competing to control the same subnet.