Skip to content

Kotlin sdk recursive glob matching#1399

Merged
jgindin merged 1 commit into
google:mainfrom
jgindin:kotlin-sdk-recursive-glob-matching
May 19, 2026
Merged

Kotlin sdk recursive glob matching#1399
jgindin merged 1 commit into
google:mainfrom
jgindin:kotlin-sdk-recursive-glob-matching

Conversation

@jgindin
Copy link
Copy Markdown
Collaborator

@jgindin jgindin commented May 11, 2026

Description

Update loadExamples to support recursive glob pattern matching for loading examples, maintaining backward compatibility for directory paths and ensuring deterministic output order. Added robust automated unit tests in CatalogPruningTest.kt.

Port of Python SDK commit 94c1c9b

Pre-launch Checklist

If you need help, consider asking for advice on the discussion board.

@jgindin jgindin requested a review from nan-yu May 11, 2026 16:27
@jgindin jgindin self-assigned this May 11, 2026
@github-project-automation github-project-automation Bot moved this to Todo in A2UI May 11, 2026
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request introduces a streaming A2UI parser for v0.8 and v0.9 specifications, featuring a new StreamingParser hierarchy and a TopologyAnalyzer for component reachability. It also enhances catalog management with message-level pruning and glob pattern support for loading examples. Review feedback identifies a critical argument swap in the AgentExtension constructor and several performance bottlenecks in the parser, including redundant regex compilations, inefficient list allocations in hot loops, and O(N^2) insertion operations.

Comment thread agent_sdks/kotlin/src/main/kotlin/com/google/a2ui/a2a/A2uiA2a.kt Outdated
Comment thread agent_sdks/kotlin/src/main/kotlin/com/google/a2ui/core/parser/StreamingParser.kt Outdated
Comment thread agent_sdks/kotlin/src/main/kotlin/com/google/a2ui/core/parser/StreamingParser.kt Outdated
Comment thread agent_sdks/kotlin/src/main/kotlin/com/google/a2ui/core/parser/StreamingParser.kt Outdated
Comment thread agent_sdks/kotlin/src/main/kotlin/com/google/a2ui/core/parser/StreamingParser.kt Outdated
@jgindin jgindin force-pushed the kotlin-sdk-recursive-glob-matching branch 3 times, most recently from acfd580 to e5e2a69 Compare May 12, 2026 15:52
@jgindin jgindin force-pushed the kotlin-sdk-recursive-glob-matching branch 10 times, most recently from 11b1ae9 to b44e2d9 Compare May 19, 2026 16:44
@jgindin jgindin requested a review from nan-yu May 19, 2026 16:44
@jgindin jgindin force-pushed the kotlin-sdk-recursive-glob-matching branch 3 times, most recently from 4b4e191 to 6e022ef Compare May 19, 2026 20:14
Update loadExamples to support recursive glob pattern matching for
loading examples, maintaining backward compatibility for directory paths
and ensuring deterministic output order. Added robust automated unit
tests in CatalogPruningTest.kt.

Port of Python SDK commit 94c1c9b
@jgindin jgindin force-pushed the kotlin-sdk-recursive-glob-matching branch from 6e022ef to 12284b9 Compare May 19, 2026 20:51
@jgindin jgindin merged commit 40c6b94 into google:main May 19, 2026
18 checks passed
@jgindin jgindin deleted the kotlin-sdk-recursive-glob-matching branch May 19, 2026 21:11
@github-project-automation github-project-automation Bot moved this from Todo to Done in A2UI May 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants