ag-ui-protocol / ag-ui-protocol/ag-ui

[Bug]: Python A2UI toolkit tests are not executed by the Python CI workflow

已关闭 适合新手
#2,743 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看
主要语言
Python
星标
15.9k
派生
1.4k
平均合并
2 天 41 分钟
30 天内合并 PR
165

描述

## Describe the Bug

The Python A2UI toolkit test suite is not executed by the Python unit-test workflow. A change under `sdks/python/a2ui_toolkit/tests` can therefore trigger that workflow without its tests being run.

This tracks the CI gap flagged separately in the [review of #2696](https://github.com/ag-ui-protocol/ag-ui/pull/2696#issuecomment-5636192971); it does not expand the catalog-ID fix in that PR.

## Source Reproduction

Verified on `main` at `747933694b05676203da7d5bb8d8e50432e59b75` on 2026-09-13:

1. Inspect the [Python job's test step](https://github.com/ag-ui-protocol/ag-ui/blob/747933694b05676203da7d5bb8d8e50432e59b75/.github/workflows/unit-python-sdk.yml#L120). It runs `uv run --locked python -m unittest discover tests -v` with `working-directory: sdks/python`, so discovery starts at `sdks/python/tests`.
2. The A2UI toolkit tests live in the separate `sdks/python/a2ui_toolkit/tests` directory. Its [package manifest](https://github.com/ag-ui-protocol/ag-ui/blob/747933694b05676203da7d5bb8d8e50432e59b75/sdks/python/a2ui_toolkit/pyproject.toml) declares `python -m unittest discover tests` as the test command for that package.
3. Search `unit-python-sdk.yml` for `a2ui_toolkit`: the references are in the lockfile-exception bookkeeping, with no test job invoking that suite.

The reviewer ran the toolkit's 100 tests locally for #2696, but that manual validation does not provide continuing CI coverage. This issue is based on workflow/source inspection, not a newly submitted failing CI run.

## Expected Behavior

Add explicit CI execution of the Python A2UI toolkit tests, triggered by relevant toolkit and workflow changes, using the repository's pinned Python toolchain. A failing toolkit regression must fail a visible test job, and the unmodified suite must pass.

The existing missing-lockfile exception is documented separately as PNI-279 in the workflow. Test coverage and that lockfile/release decision should be handled explicitly rather than treating lockfile inspection as test execution.

## Environment / Additional Context

Source audit of the main revision above. Existing A2UI/CI issues were searched before filing. Codex assisted with source verification and this report.

贡献指南

打开贡献指南

调研方向

Start with .github/workflows/unit-python-sdk.yml and inspect the existing Python test step, then compare it with sdks/python/a2ui_toolkit/pyproject.toml and sdks/python/a2ui_toolkit/tests. Run the toolkit's unittest discovery command with the pinned toolchain. Done means the unmodified suite passes in a visible CI job and a toolkit regression fails that job.

由索引模型根据 Issue 内容生成。

评估

技术栈
github-actions, python
领域
ci-cd, testing
Issue 类型
缺陷
难度
2/5
预计耗时
1-3 小时
活跃度
活跃
描述清晰度
描述清楚
新手友好度
76/100

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。