Skip to content

Conversation

ishaksebsib
Copy link
Contributor

@ishaksebsib ishaksebsib commented Sep 8, 2025

Description

This PR adds validation for field types in schema definitions to prevent the use of unknown or custom types (identifiers) in node, vector, and edge fields. It enforces the use of built-in types only.

Previously, invalid type definitions could still pass the Helix check

Related Issues

None

Checklist when merging to main

  • No compiler warnings (if applicable)
  • Code is formatted with rustfmt
  • No useless or dead code (if applicable)
  • Code is easy to understand
  • Doc comments are used for all functions, enums, structs, and fields (where appropriate)
  • All tests pass
  • Performance has not regressed (assuming change was not to fix a bug)
  • Version number has been updated in helix-cli/Cargo.toml and helixdb/Cargo.toml

Additional Notes

None

@xav-db xav-db merged commit aea14f7 into HelixDB:dev Sep 15, 2025
13 checks passed
@ishaksebsib ishaksebsib deleted the feat/hql-add-field-type-validation branch September 15, 2025 09:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants