Skip to content

feat: manually mirror eino's code from bytedance#1

Merged
hi-pender merged 8 commits into
mainfrom
feat/mirror_code
Dec 11, 2024
Merged

feat: manually mirror eino's code from bytedance#1
hi-pender merged 8 commits into
mainfrom
feat/mirror_code

Conversation

@hi-pender
Copy link
Copy Markdown
Contributor

What type of PR is this?

Check the PR title.

Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

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

license-eye has totally checked 149 files.

Valid Invalid Ignored Fixed
148 1 0 0
Click to see the invalid file list
  • compose/graph_node_checker.go

Comment thread compose/graph_node_checker.go
Comment thread compose/graph_node_checker.go
@hi-pender hi-pender merged commit eff15db into main Dec 11, 2024
@hi-pender hi-pender deleted the feat/mirror_code branch December 11, 2024 06:35
@hi-pender hi-pender restored the feat/mirror_code branch December 11, 2024 06:35
@hi-pender hi-pender deleted the feat/mirror_code branch February 14, 2025 07:58
@mrh997 mrh997 mentioned this pull request Aug 22, 2025
shentongmartin added a commit that referenced this pull request Apr 15, 2026
…leware

- Rename BeforeToolCall type to Checker; pass *adk.ToolContext instead of bare toolName (Thread #1)
- Rename NewMiddleware to New to match codebase convention (Thread #4)
- Improve error messages with tool name, call ID, and args for LLM consumption (Thread #5)
- Replace schema.Pipe with StreamReaderFromArray for stream deny path (Thread #6)
- Add 4 E2E ask→resume tests: approved, denied, re-interrupt non-target, resume with updated input (Thread #7)

Change-Id: If43ac495fa3a5fc1d71a27db9614cb996d4547b5
shentongmartin added a commit that referenced this pull request Apr 15, 2026
…leware

- Rename BeforeToolCall type to Checker; pass *adk.ToolContext instead of bare toolName (Thread #1)
- Rename NewMiddleware to New to match codebase convention (Thread #4)
- Improve error messages with tool name, call ID, and args for LLM consumption (Thread #5)
- Replace schema.Pipe with StreamReaderFromArray for stream deny path (Thread #6)
- Add 4 E2E ask→resume tests: approved, denied, re-interrupt non-target, resume with updated input (Thread #7)

Change-Id: If43ac495fa3a5fc1d71a27db9614cb996d4547b5
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants