Skip to content

fix: guard against empty query in SPARQL generator (#870)#902

Closed
SAY-5 wants to merge 1 commit into
trustgraph-ai:release/v2.4from
SAY-5:say5/fix-sparql-empty-query-870
Closed

fix: guard against empty query in SPARQL generator (#870)#902
SAY-5 wants to merge 1 commit into
trustgraph-ai:release/v2.4from
SAY-5:say5/fix-sparql-empty-query-870

Conversation

@SAY-5

@SAY-5 SAY-5 commented May 12, 2026

Copy link
Copy Markdown

Fixes #870.

Guard query.split()[0] so a future refactor that loosens or removes the startswith check cannot turn an empty/whitespace response into an unhandled IndexError. The new condition uses the already-split list, so the public behavior on valid queries is unchanged.

@github-actions

Copy link
Copy Markdown

Contributor License Agreement

Thank you for your contribution! Before we can accept it, the following contributor(s) must sign our CLA:

@SAY-5

Please read the appropriate agreement:

  • Contributing as an individual? Read the Individual CLA
  • Contributing on behalf of a company or organisation? Read the Entity CLA

Once you have read the appropriate agreement, post the following as a comment on this PR (copy and paste exactly):

I have read the CLA Document and I hereby sign the CLA

The bot will record your signature and update this PR automatically.

@cybermaggedon

Copy link
Copy Markdown
Contributor

@SAY-5 can you please take a look at the Contributor License Agreement notice above?

@cybermaggedon cybermaggedon added cla-required bug Something isn't working labels May 12, 2026
@SAY-5

SAY-5 commented May 12, 2026

Copy link
Copy Markdown
Author

Apologies for the delay. I'm not able to sign the CLA at this time, so feel free to close this if that's a blocker. The fix itself is small and the diff stands on its own if a maintainer wants to pick it up.

@cybermaggedon

Copy link
Copy Markdown
Contributor

Closed as not able to complete the CLA currently. Thanks for your interest in the project!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working cla-required

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants