Looks fine. Mayyyyyyybe the 1k resistor is a little high, depends on the gain of the transistor. 5 volts would feed it with 5mA on its base, which might not be enough to fully switch it on. However, as it's only driving the relatively high resistance coil in the relay it's nothing to worry about if it works.

If you were driving it with 3.3V, I'd probably swap that transistor out for a Darlington pair to make sure the gain is high enough to fully switch on at that kind of drive voltage. You can get Darlington pairs in the same pinout as normal transistors so it would be a straight swap on the board.

Having said all that, if it's working in your application, leave it. 🤷

The manufacturer has rated this relay for nominal motor (ie inductive) loads of 0.5 A, with peak surge of 2.5 A, at 120 VAC. The flyback diode should be sufficient, since the switching rate would not be very high to control a relay. The terminal block supports a max wire diameter of 1.5mm2 (~16 AWG) which would be more than plenty for a 0.5 A load.

If you're actually running >60 VAC through the relay, then it's good practice to add a slot that prevents creepage currents between the low-voltage DC section and the higher-voltage power section.

For smaller applications, this seems reasonable enough, at least to me.