Skip to content

feat: SNP integration #2291

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 6 commits into from
Jun 3, 2025
Merged

feat: SNP integration #2291

merged 6 commits into from
Jun 3, 2025

Conversation

zone117x
Copy link
Member

@zone117x zone117x commented May 26, 2025

Closes #2270

Implements SNP event streaming support in the API. An integration test harness is also implemented which spins up a local SNP server, populates it with events from a tsv (simulating a stacks-node), then ensures the API is able to sync all events from SNP.

This PR uses a new approach to orchestrating service dependencies. Rather than using docker-compose, the docker containers are configured programmatically in test initialization code.

Copy link

github-actions bot commented May 26, 2025

Vercel deployment URL: https://stacks-blockchain-em58c98xo-hirosystems.vercel.app 🚀

Copy link

codecov bot commented May 26, 2025

Codecov Report

Attention: Patch coverage is 68.75000% with 15 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/index.ts 0.00% 6 Missing and 1 partial ⚠️
src/datastore/pg-write-store.ts 0.00% 4 Missing ⚠️
src/event-stream/snp-event-stream.ts 87.87% 4 Missing ⚠️

📢 Thoughts on this report? Let us know!

console.log(`Using REDIS_STREAM_KEY_PREFIX: ${process.env.SNP_REDIS_STREAM_KEY_PREFIX}`);
const snpContainer = await startContainer({
docker,
image: 'hirosystems/salt-n-pepper:1.1.1',
Copy link
Member Author

Choose a reason for hiding this comment

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

This image is private on docker hub so CI isn't working. Need to make it public

Copy link
Collaborator

@rafaelcr rafaelcr left a comment

Choose a reason for hiding this comment

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

This looks great! Thanks @zone117x !

@zone117x zone117x merged commit 9a159e1 into develop Jun 3, 2025
27 checks passed
@zone117x zone117x deleted the feat/snp-ingestion branch June 3, 2025 02:22
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.

stacks-blockchain-api : Add support for S&P event streams
2 participants