Skip to content

Add Stellar Prices API to community skills and ecosystem catalog - #141

Open
stkrolikiewicz wants to merge 3 commits into
stellar:mainfrom
stkrolikiewicz:add-stellar-prices-api
Open

stkrolikiewicz wants to merge 3 commits into
stellar:mainfrom
stkrolikiewicz:add-stellar-prices-api

Conversation

@stkrolikiewicz

Copy link
Copy Markdown

Summary

Why this skill is useful

The catalog has no skill for asset price data. The data skill points to RPC and Horizon, which return raw trades: Horizon's /trade_aggregations covers one classic pair at a time and misses Soroban AMM swaps, and RPC keeps events for about 7 days. The Soroswap SDK skill covers swap quotes on one DEX.

This skill lets an agent ask for USD and XLM prices directly: current prices with a 24h cross-venue VWAP (SDEX, Soroswap, Aquarius, Phoenix, SushiSwap), OHLCV candles, Reflector oracle readings and batch lookups. It checks for an API key first and points the user to the free Discord sign-in when there is none.

Test plan

  • copyValue returns the raw SKILL.md (HTTP 200, text/plain).
  • check:ecosystem-links (31 entries, no blob URLs) and test:ecosystem-links pass.
  • The generated llms.txt includes the new card.
  • npx skills add rumblefishdev/stellar-prices-api --skill stellar-prices-api installs the skill.

Copilot AI lite review requested due to automatic review settings September 30, 2026 11:20

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

🔵 Needs a closer look

Add or update a standards evaluation scenario covering the new API.

Review effort: Lite
Findings: None

What changed in this PR

Adds the Stellar Prices API to the community skills catalog and ecosystem documentation.

Changes:

  • Adds the API under Data Indexing.
  • Adds its community skill card and installation metadata.
File Description
skills/​standards/​ecosystem.md Documents the Stellar Prices API; a matching eval scenario is still required.
site/​src/​data/​skills.ts Adds the community ecosystem card.

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

@stkrolikiewicz

Copy link
Copy Markdown
Author

Example run: a fresh Claude Code session with only this skill installed, asked
"What is the difference of value of 1000 XLM between last Monday and today?"

The agent loads the skill and checks for an API key. It then fetches the current
XLM price and daily and hourly OHLCV candles, looks up field meanings in the
API spec, and compares the Monday average, the same time of day and the close:

Screenshot 2026-09-30 at 14 15 43 Screenshot 2026-09-30 at 14 15 29

@oceans404

Copy link
Copy Markdown
Contributor

@copilot resolve the merge conflicts in this pull request

Copilot AI lite review requested due to automatic review settings October 6, 2026 09:51

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

🟡 Changes recommended

Correct the stale API and documentation URLs before approval.

Review effort: Lite
Findings: 1 Medium severity

Open (1)

Comment thread skills/standards/ecosystem.md Outdated
Updated links for Stellar Prices API documentation and website.

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Copilot AI lite review requested due to automatic review settings October 6, 2026 09:57

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

Only a minor API documentation URL discrepancy remains, with no approval-blocking issues.

Review effort: Lite
Findings: None

Resolved since last review (1)

@stellar-triage-bot

Copy link
Copy Markdown
Contributor

🤖 Automated message from the Stellar Triage Bot.

Thanks for this. The skill is unusually honest about its edges. The "0" sentinel notes and the Regenerate warning will save agents real mistakes.

I verified the technical content at f711370:

  • The raw SKILL.md URL returns 200 as text/plain. The skill directory carries an MIT license.
  • The API is live. /health and the public OpenAPI spec return 200, and /v1 returns 403 without a key, as documented.
  • The venues, Reflector readings and 100-asset batch match the spec. The service ingests mainnet ledgers.
  • The RPC claim is right: the default retention window is 120,960 ledgers, about 7 days.
  • Copilot's URL finding is fixed. Both prices-api/ links in ecosystem.md return 200.

One process note: you are a main contributor to rumblefishdev/stellar-prices-api. This PR adds that project to the official skills/standards/ecosystem.md. That listing decision belongs to a maintainer, not to me, so I am not merging it. @kaankacar will decide.

One content item is worth doing while it waits. Copilot raised it, and I agree:

  • README.md asks for a matching evals/ scenario when a change touches what a skill teaches. Please add one under evals/scenarios/standards/ for asset price lookups. The existing files there show the shape.

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