Skip to content

ci,test: pin Soroban toolchain, add clippy/audit jobs, and LLM transfer tests#241

Merged
codebestia merged 9 commits into
codebestia:mainfrom
iduhtheman:fix/clicked-iduhtheman-batch
Jun 26, 2026
Merged

ci,test: pin Soroban toolchain, add clippy/audit jobs, and LLM transfer tests#241
codebestia merged 9 commits into
codebestia:mainfrom
iduhtheman:fix/clicked-iduhtheman-batch

Conversation

@iduhtheman

Copy link
Copy Markdown
Contributor

Summary

CI hardening for the contracts workspace and unit tests for the AI agent's LLM-path transfer analysis.

Changes

#138 — Pin Soroban toolchain

  • contracts/rust-toolchain.toml: declares channel = "stable", targets = ["wasm32-unknown-unknown"], components = ["clippy", "rustfmt"] — all CI jobs now use a single pinned version; upgrading Rust requires an explicit PR

#137 — cargo clippy lint job (wasm32)

  • .github/workflows/contracts-ci.yml: new parallel clippy job running cargo clippy --workspace --target wasm32-unknown-unknown -- -D warnings; runs on every PR touching contracts/**

#139 — cargo audit security scan

  • .github/workflows/contracts-ci.yml: new audit job via rustsec/audit-check@v2; added schedule: cron: '0 8 * * 1' (Monday 08:00 UTC) so new CVEs surface without code changes

#146 — Unit tests for POST /transfers/analyse (LLM path)

  • apps/ai_agent/tests/test_transfers.py: 5 tests covering flagged transfer, clean transfer, missing confidence defaults to 0.0, missing flagged defaults to False, missing OPENAI_API_KEY returns 500

closes #138
closes #137
closes #139
closes #146

@drips-wave

drips-wave Bot commented Jun 26, 2026

Copy link
Copy Markdown

@iduhtheman Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@codebestia codebestia left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

LGTM!
Thank you for your contribution.

@codebestia codebestia merged commit c235136 into codebestia:main Jun 26, 2026
4 checks passed
codebestia added a commit that referenced this pull request Jun 26, 2026
ci,test: pin Soroban toolchain, add clippy/audit jobs, and LLM transfer tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants