Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: runtime check graph #165

Merged
merged 6 commits into from
Oct 17, 2024
Merged

feat: runtime check graph #165

merged 6 commits into from
Oct 17, 2024

Conversation

leoadonia
Copy link
Collaborator

graph check 支持只检测单个 app scope:

  • 单个 app 下, 只检测当前 app 下的 extension 是否安装.
  • 单个 app 下, 只检测 connections 中 src 和 dest 都是当前 app 下的 message, 即应该能获取到 schema.

检测范围:

  • start_graph cmd,如果有错,返回错误响应.
  • predefined graph, 如果有错,目前只打印日志,因为还不支持 message conversion,返回 false 可能会影响 app 启动. WIP: issue

@leoadonia leoadonia requested a review from halajohn as a code owner October 17, 2024 11:05
@halajohn halajohn self-requested a review October 17, 2024 13:16
@halajohn halajohn merged commit 888a918 into main Oct 17, 2024
13 checks passed
@halajohn halajohn deleted the feature/runtime-check-graph branch October 17, 2024 13:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

When the TEN runtime starts a graph in a TEN app, it also checks the graph
2 participants