That seems like a logical structure to me. Shopify doesn’t set any application structure itself so you can organise your app, how you’d like ![]()
One thing to flag is the direct api access is coming soon to POS (currently in unstable). Which means you could look up the order info directly from POS and save yourself an API call potentially in future.