Skip to content

fix(ci): correct Rust SDK workflow paths#1655

Open
jpr5 wants to merge 6 commits into
mainfrom
fix/rust-sdk-workflow-paths
Open

fix(ci): correct Rust SDK workflow paths#1655
jpr5 wants to merge 6 commits into
mainfrom
fix/rust-sdk-workflow-paths

Conversation

@jpr5
Copy link
Copy Markdown
Contributor

@jpr5 jpr5 commented May 12, 2026

Summary

The Rust SDK CI workflow (rust-lint-test.yml) had stale path references from before the SDK was moved to sdks/community/rust/:

  • defaults.run.working-directory pointed to ./rust instead of ./sdks/community/rust, causing every cargo command to fail with "directory not found"
  • Push trigger path filters used root-relative paths (crates/**, tests/**, Cargo.toml, .cargo/**) that no longer exist, so the workflow never triggered on pushes to main
  • The workflow filename filter referenced the old name rust.yml instead of rust-lint-test.yml
  • Swatinem/rust-cache was missing the workspaces input, so it couldn't find the Cargo workspace in the subdirectory

The pull_request path filters were already correct; only the push filters and working directory needed updating.

Changes

  • Fix defaults.run.working-directory: ./rust -> ./sdks/community/rust
  • Fix push path filters to use sdks/community/rust/ prefix
  • Fix workflow self-reference from rust.yml to rust-lint-test.yml
  • Add workspaces: sdks/community/rust to Swatinem/rust-cache@v2

Test plan

  • Verify the workflow triggers on this PR (path filter includes .github/workflows/rust-lint-test.yml)
  • Confirm cargo build/test/clippy/fmt steps pass with the corrected working directory

@jpr5 jpr5 requested a review from a team as a code owner May 12, 2026 00:17
@vercel
Copy link
Copy Markdown

vercel Bot commented May 12, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
ag-ui-dojo Ready Ready Preview, Comment May 12, 2026 4:11am

Request Review

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 12, 2026

Python Preview Packages

Version 0.0.0.dev1778558855 published to TestPyPI.

Warning: These packages are built from contributor code that may not yet have been vetted for correctness or security. Install at your own risk and do not use in production.

Install with uv

Add the TestPyPI index to your pyproject.toml:

[[tool.uv.index]]
name = "testpypi"
url = "https://test.pypi.org/simple/"
explicit = true

Then install the packages you need:

# Core SDK
uv add 'ag-ui-protocol==0.0.0.dev1778558855' --index testpypi

# Integrations (each already depends on the matching ag-ui-protocol preview)
uv add 'ag-ui-langgraph==0.0.0.dev1778558855' --index testpypi
uv add 'ag-ui-crewai==0.0.0.dev1778558855' --index testpypi
# NOTE: ag-ui-agent-spec depends on pyagentspec (git-only, not on PyPI).
# You will need to install pyagentspec separately from its git repo.
uv add 'ag-ui-agent-spec==0.0.0.dev1778558855' --index testpypi
uv add 'ag_ui_adk==0.0.0.dev1778558855' --index testpypi
uv add 'ag_ui_strands==0.0.0.dev1778558855' --index testpypi

Install with pip

pip install \
  --index-url https://test.pypi.org/simple/ \
  --extra-index-url https://pypi.org/simple/ \
  ag-ui-protocol==0.0.0.dev1778558855

Use --extra-index-url https://pypi.org/simple/ so pip can resolve
transitive dependencies (pydantic, fastapi, etc.) from real PyPI.


Commit: f3c5e53

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented May 12, 2026

Open in StackBlitz

@ag-ui/a2a-middleware

pnpm add https://pkg.pr.new/ag-ui-protocol/ag-ui/@ag-ui/a2a-middleware@1655

@ag-ui/a2ui-middleware

pnpm add https://pkg.pr.new/ag-ui-protocol/ag-ui/@ag-ui/a2ui-middleware@1655

@ag-ui/event-throttle-middleware

pnpm add https://pkg.pr.new/ag-ui-protocol/ag-ui/@ag-ui/event-throttle-middleware@1655

@ag-ui/mcp-apps-middleware

pnpm add https://pkg.pr.new/ag-ui-protocol/ag-ui/@ag-ui/mcp-apps-middleware@1655

@ag-ui/middleware-starter

pnpm add https://pkg.pr.new/ag-ui-protocol/ag-ui/@ag-ui/middleware-starter@1655

@ag-ui/a2a

pnpm add https://pkg.pr.new/ag-ui-protocol/ag-ui/@ag-ui/a2a@1655

@ag-ui/adk

pnpm add https://pkg.pr.new/ag-ui-protocol/ag-ui/@ag-ui/adk@1655

@ag-ui/ag2

pnpm add https://pkg.pr.new/ag-ui-protocol/ag-ui/@ag-ui/ag2@1655

@ag-ui/agno

pnpm add https://pkg.pr.new/ag-ui-protocol/ag-ui/@ag-ui/agno@1655

@ag-ui/aws-strands

pnpm add https://pkg.pr.new/ag-ui-protocol/ag-ui/@ag-ui/aws-strands@1655

@ag-ui/claude-agent-sdk

pnpm add https://pkg.pr.new/ag-ui-protocol/ag-ui/@ag-ui/claude-agent-sdk@1655

@ag-ui/crewai

pnpm add https://pkg.pr.new/ag-ui-protocol/ag-ui/@ag-ui/crewai@1655

@ag-ui/langchain

pnpm add https://pkg.pr.new/ag-ui-protocol/ag-ui/@ag-ui/langchain@1655

@ag-ui/langgraph

pnpm add https://pkg.pr.new/ag-ui-protocol/ag-ui/@ag-ui/langgraph@1655

@ag-ui/langroid

pnpm add https://pkg.pr.new/ag-ui-protocol/ag-ui/@ag-ui/langroid@1655

@ag-ui/llamaindex

pnpm add https://pkg.pr.new/ag-ui-protocol/ag-ui/@ag-ui/llamaindex@1655

@ag-ui/mastra

pnpm add https://pkg.pr.new/ag-ui-protocol/ag-ui/@ag-ui/mastra@1655

@ag-ui/pydantic-ai

pnpm add https://pkg.pr.new/ag-ui-protocol/ag-ui/@ag-ui/pydantic-ai@1655

@ag-ui/server-starter

pnpm add https://pkg.pr.new/ag-ui-protocol/ag-ui/@ag-ui/server-starter@1655

@ag-ui/server-starter-all-features

pnpm add https://pkg.pr.new/ag-ui-protocol/ag-ui/@ag-ui/server-starter-all-features@1655

@ag-ui/vercel-ai-sdk

pnpm add https://pkg.pr.new/ag-ui-protocol/ag-ui/@ag-ui/vercel-ai-sdk@1655

create-ag-ui-app

pnpm add https://pkg.pr.new/ag-ui-protocol/ag-ui/create-ag-ui-app@1655

@ag-ui/client

pnpm add https://pkg.pr.new/ag-ui-protocol/ag-ui/@ag-ui/client@1655

@ag-ui/core

pnpm add https://pkg.pr.new/ag-ui-protocol/ag-ui/@ag-ui/core@1655

@ag-ui/encoder

pnpm add https://pkg.pr.new/ag-ui-protocol/ag-ui/@ag-ui/encoder@1655

@ag-ui/proto

pnpm add https://pkg.pr.new/ag-ui-protocol/ag-ui/@ag-ui/proto@1655

commit: 6defdff

@jpr5
Copy link
Copy Markdown
Contributor Author

jpr5 commented May 12, 2026

CI Status

The workflow path fix is confirmed working — Cargo now runs from the correct directory (sdks/community/rust).

The Rust SDK Tests still fail due to pre-existing compilation errors in the Rust SDK itself (22 E0282 type inference errors in event_handler.rs and agent.rs, plus an E0432 import error). These have been masked since the workflow never ran successfully before (the old working-directory: ./rust meant it failed at the shell level before reaching cargo).

The compilation errors are a separate issue from this CI fix PR.

test_sse_with_httpbun connects to httpbun.org which fails on Windows CI
with a TLS error. Mark it #[ignore] so it doesn't block the workflow.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant