Skip to content

Add stellarpath-cli to ECOSYSTEM_CARDS - #137

Open
DANTE-1903 wants to merge 1 commit into
stellar:mainfrom
DANTE-1903:add-stellarpath-cli
Open

DANTE-1903 wants to merge 1 commit into
stellar:mainfrom
DANTE-1903:add-stellarpath-cli

Conversation

@DANTE-1903

Copy link
Copy Markdown

Adds stellarpath-cli to the ecosystem skills page. stellarpath-cli is a fast, Rust-based static analyzer for Stellar and Soroban repositories.

Copilot AI balanced review requested due to automatic review settings September 21, 2026 15:08
@kaankacar

Copy link
Copy Markdown
Contributor

🤖 Automated message from Kaan's Automated Triage Bot.

👀 Picked this up — a review will follow shortly.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🟢 Approval recommended

The new entry follows the established ecosystem card schema and raw GitHub URL convention.

Review effort: Balanced
Findings: None

What changed in this PR

Adds stellarpath-cli, a Rust static analyzer for Stellar and Soroban projects, to the community skills catalog.

Changes:

  • Adds the skill’s description, repository label, and raw SKILL.md URL.
File Description
site/​src/​data/​skills.ts Registers the stellarpath-cli ecosystem card.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@stellar-triage-bot

Copy link
Copy Markdown
Contributor

🤖 Automated message from the Stellar Triage Bot.

Thanks for stellarpath-cli. The lint rule numbers match the Soroban Common Mistakes guide that is already in the catalog (#17 bare panic!, #18 unwrap/expect, #19 missing events). That is a nice link between the two entries. I read the full source, and the scan, start and lint commands match the SKILL.md.

A few items before the listing:

  1. CI has not built the code since lint mode landed. Every CI run since 2026-09-21 fails at cargo fmt --check, so clippy, check and tests are skipped. The lint mode and the SEP detector came in that range. The skill tells agents to cargo install from main, so please get CI green. I do not build submitted code, so CI is the build evidence.
  2. SEP-53 has the wrong name. src/detector/sep.rs prints "Implements SEP-53 (Sign-In with Stellar)". SEP-53 is "Sign and Verify Messages" (ecosystem/sep-0053.md in stellar-protocol, status Final). An agent will repeat the wrong name.
  3. "What to expect" in SKILL.md differs from the output. start has no --format flag and prints plain text. The Mistake Bump next from 15.5.9 to 15.5.18 in /site in the npm_and_yarn group across 1 directory #19 finding is file-level, so it has no line number.
  4. Merge conflict. Please merge main into your branch. site/src/data/skills.ts now conflicts after the Stellar Registry card. Keep both entries.

Optional: the SEP detector also matches a plain "SEP-24" in a comment. "References" is a more exact word than "Implements" in the output and on the card.

When these are in, I will re-check and merge.

This branch has not been deployed

No deployments
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.

3 participants