Add optional QID order validation and refactor label decomposition

[OmarAI2003]

• Introduced check_optional_qid_order function to validate the order of QIDs in optional statements within SPARQL queries.
• Extracted label decomposition logic into the decompose_label_features function, which is now used in both functions.
• Updated check_query_forms to include validation for the correct order of QIDs in OPTIONAL clauses.

Contributor checklist

• This pull request is on a separate branch and not the main branch
• I have tested my code with the pytest command as directed in the testing section of the contributing guide

---

Description

This PR introduces a new function check_optional_qid_order to validate the order of QIDs in optional statements within SPARQL queries. Additionally, the label decomposition logic has been refactored into the decompose_label_features function, which is now used in both check_forms_order and check_optional_qid_order for better readability and consistency. The check_query_forms function has been updated to include validation for the correct order of QIDs in OPTIONAL clauses.

Note: The queries are still in progress and will be finalized once the function is approved.

Related issue

• Last PR to close Create workflow to check query forms #450

[github-actions]

Thank you for the pull request!

The Scribe team will do our best to address your contribution as soon as we can. The following is a checklist for maintainers to make sure this process goes as well as possible. Feel free to address the points below yourself in further commits if you realize that actions are needed :)

If you're not already a member of our public Matrix community, please consider joining! We'd suggest using Element as your Matrix client, and definitely join the General and Data rooms once you're in. Also consider joining our bi-weekly Saturday dev syncs. It'd be great to have you!

Maintainer checklist

• The linting and formatting workflow within the PR checks do not indicate new errors in the files changed

• The CHANGELOG has been updated with a description of the changes for the upcoming release and the corresponding issue (if necessary)

[andrewtavis]

Wonderful work, @OmarAI2003! 🚀 Can we change the message to The QIDs in optional statement for FORM_NAME should be ordered:? That way it's easy for someone to find where they need to make the change?

[OmarAI2003]

I've done this. Thanks @andrewtavis for your instant reply. Should I include anything else?

[andrewtavis]

I'll correct my own suggestion 😇 Let's do QIDs in the optional statement for FORM_NAME should be ordered: (start as you had it and say the optional...). Aside from that we're good to fix the form orders and close this up!

[andrewtavis]

Good for a final review here, @OmarAI2003? :)

[OmarAI2003]

Good for a final review here, @OmarAI2003? :)

Yeah, I think so.

[andrewtavis]

Really amazing work here, @OmarAI2003 😊 No changes needed :) Thanks for the quick reactions to comments and for the attention to detail!