Skip to content

feat(language): evaluate hgraph IR directly - #695

Merged
hhenson merged 6 commits into
codex/hgl-wiring-ir-typesfrom
codex/hgl-wiring-ir-evaluator
Sep 6, 2026
Merged

feat(language): evaluate hgraph IR directly#695
hhenson merged 6 commits into
codex/hgl-wiring-ir-typesfrom
codex/hgl-wiring-ir-evaluator

Conversation

@hhenson

@hhenson hhenson commented Sep 5, 2026

Copy link
Copy Markdown
Owner

Summary

  • move the scripted test, run, and REPL evaluator from the resolved AST to execution-facing hgraph IR
  • evaluate --set expressions through the ordinary frontend and pass owned checked values to the backend
  • add type-only fixed-output inspection for native operator families without invoking overload defaults, resolvers, or predicates
  • preserve constant struct construction, temporal folding, numeric and instant equality, and contextual collection conversion in typed HIR
  • materialize direct scalar widening as native conversion nodes while rejecting unsupported implicit widening inside time-series collections
  • preserve fixed TSL refinements at unsized list boundaries and wire outputless operators without requesting a result schema
  • update the language architecture and tool documentation to describe the remaining C++ backend adapter

Validation

  • complete HGL language suite on macOS: 30/30 passed
  • complete HGL language suite on a private Linux validation host: 30/30 passed
  • fresh native suite: 1,712/1,712 passed
  • stable-ABI wheel built with Python 3.12 and tested with Python 3.14: 3,228 passed, 10 skipped

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 5, 2026

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review Completed 2026-09-05T17:53:49.072313Z 0f20f08 Manual request
🔒 Security Review Completed 2026-09-05T17:52:31.676762Z 0f20f08 Manual request
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 23090ebdda

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread language/src/wiring/backend.cpp
Comment thread language/src/wiring/backend.cpp
Comment thread language/src/ir/canonical_types.cpp
@hhenson

hhenson commented Sep 5, 2026

Copy link
Copy Markdown
Owner Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Security review completed. No security issues were found in this pull request.

Reviewed commit: 824171462e

View security finding report

Only the user who started this review can view the report in Codex.

ℹ️ About Codex security reviews in GitHub

This is an experimental Codex feature. Security reviews are triggered when:

  • You comment "@codex security review"
  • A regular code review gets triggered (for example, "@codex review" or when a PR is opened), and you’re opted in so security review runs alongside code review

Once complete, Codex will leave suggestions, or a comment if no findings are found.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 824171462e

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread language/src/wiring/backend.cpp
Comment thread language/src/wiring/backend.cpp Outdated
@hhenson

hhenson commented Sep 5, 2026

Copy link
Copy Markdown
Owner Author

@codex review

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 6277d7f09c

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread language/src/ir/canonical_types.cpp
Comment thread language/src/ir/type_check.cpp Outdated
@chatgpt-codex-connector

Copy link
Copy Markdown

Security review completed. No security issues were found in this pull request.

Reviewed commit: 6277d7f09c

View security finding report

Only the user who started this review can view the report in Codex.

ℹ️ About Codex security reviews in GitHub

This is an experimental Codex feature. Security reviews are triggered when:

  • You comment "@codex security review"
  • A regular code review gets triggered (for example, "@codex review" or when a PR is opened), and you’re opted in so security review runs alongside code review

Once complete, Codex will leave suggestions, or a comment if no findings are found.

@hhenson

hhenson commented Sep 5, 2026

Copy link
Copy Markdown
Owner Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Security review completed. No security issues were found in this pull request.

Reviewed commit: c6d890d860

View security finding report

Only the user who started this review can view the report in Codex.

ℹ️ About Codex security reviews in GitHub

This is an experimental Codex feature. Security reviews are triggered when:

  • You comment "@codex security review"
  • A regular code review gets triggered (for example, "@codex review" or when a PR is opened), and you’re opted in so security review runs alongside code review

Once complete, Codex will leave suggestions, or a comment if no findings are found.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: c6d890d860

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread language/src/wiring/backend.cpp Outdated
Comment thread language/src/wiring/backend.cpp
@hhenson

hhenson commented Sep 5, 2026

Copy link
Copy Markdown
Owner Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Security review completed. No security issues were found in this pull request.

Reviewed commit: 0f20f08fca

View security finding report

Only the user who started this review can view the report in Codex.

ℹ️ About Codex security reviews in GitHub

This is an experimental Codex feature. Security reviews are triggered when:

  • You comment "@codex security review"
  • A regular code review gets triggered (for example, "@codex review" or when a PR is opened), and you’re opted in so security review runs alongside code review

Once complete, Codex will leave suggestions, or a comment if no findings are found.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 0f20f08fca

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread language/src/wiring/operator_types.cpp
Comment thread language/src/wiring/backend.cpp Outdated
Comment thread language/src/wiring/backend.cpp
@hhenson
hhenson force-pushed the codex/hgl-wiring-ir-evaluator branch from 0f20f08 to 2f8cf01 Compare September 5, 2026 18:20
@hhenson
hhenson force-pushed the codex/hgl-wiring-ir-evaluator branch from ce737f5 to 90edca0 Compare September 5, 2026 19:56
@hhenson
hhenson force-pushed the codex/hgl-wiring-ir-evaluator branch from 90edca0 to d399940 Compare September 6, 2026 05:34
@sonarqubecloud

sonarqubecloud Bot commented Sep 6, 2026

Copy link
Copy Markdown

@hhenson
hhenson merged commit ad13d74 into main Sep 6, 2026
12 checks passed
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.

1 participant