-
Notifications
You must be signed in to change notification settings - Fork 88
feat(llm): expand graph extraction service APIs #361
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
Open
LRriver
wants to merge
33
commits into
apache:main
Choose a base branch
from
LRriver:extract_api
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from 31 commits
Commits
Show all changes
33 commits
Select commit
Hold shift + click to select a range
7bf6688
sync: preserve local workflow files
github-actions[bot] da27bf5
Merge remote-tracking branch 'upstream/main'
github-actions[bot] 800b150
Merge branch 'apache:main' into main
LRriver 81d89e4
Merge remote-tracking branch 'upstream/main'
github-actions[bot] af4186a
Merge remote-tracking branch 'upstream/main'
github-actions[bot] f41baa2
Merge remote-tracking branch 'upstream/main'
github-actions[bot] b8d4b40
Merge remote-tracking branch 'upstream/main'
github-actions[bot] 1a98cef
Merge remote-tracking branch 'upstream/main'
github-actions[bot] f3ba75d
sync: preserve local workflow files
github-actions[bot] e1b23ad
feat(llm): add graph extraction service APIs
LRriver acda6ce
test(llm): cover graph extraction service APIs
LRriver 5090f0c
feat(llm): support graph extract content modes
LRriver 184bc73
test(llm): cover graph extract content modes
LRriver e297b3d
fix(llm): harden graph extract runtime parsing
LRriver 402ba2e
test(llm): cover graph extract API edge cases
LRriver 6cedb5c
fix(llm): return workflow node errors safely
LRriver c435f5c
fix(llm): keep paragraph chunk boundaries
LRriver 3bb0c95
fix(llm): use extract LLM for graph extraction
LRriver 7a6aff6
fix(llm): fall back extract LLM config to chat settings
LRriver 7aa8a50
fix(llm): harden graph extract API follow-up
LRriver 93c7c84
fix(llm): address graph extract API review feedback
LRriver a2847c5
fix(llm): harden graph extract review follow-up
LRriver aac86e6
fix(llm): align graph import validation message
LRriver cd9985d
fix(llm): guard graph import service writes
LRriver 9a044f3
fix(llm): sanitize graph extract error surfaces
LRriver f1c54ab
test(llm): assert graph routes via openapi contract
LRriver 0c95388
fix(llm): sanitize import warning messages
LRriver a8b18a4
fix(llm): avoid raw import payload errors
LRriver a903a18
fix(llm): validate graph import edge endpoints
LRriver 79a3d87
fix(llm): validate graph import properties
LRriver 10524d2
fix(llm): harden graph import contracts
LRriver 8e88ab8
ci: restore upstream workflow coverage
LRriver 8790c28
fix(llm): harden graph import review follow-up
LRriver File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/graph/extract, jobs,/graph/import, and extract-and-import tests undersrc/tests/api, plus related flow/node/utils coverage, but this job only runs config/document/middleware/operators/models/indices/test_utils. Please restore an equivalent full unit/contract lane, or at least includesrc/tests/api src/tests/flows src/tests/nodes src/tests/utils, so these API regressions cannot pass with green checks.