Skip to content

Feat/sybil#249

Merged
dDevAhmed merged 5 commits into
DigiNodes:mainfrom
nafsonig:feat/sybil
May 30, 2026
Merged

Feat/sybil#249
dDevAhmed merged 5 commits into
DigiNodes:mainfrom
nafsonig:feat/sybil

Conversation

@nafsonig
Copy link
Copy Markdown
Contributor

Progress update: I added a new SybilExplanation Prisma model, updated SybilResistanceService to store the explanation separately and to load it when returning voting data, and updated tests to assert the new behavior.

Files changed:

  • schema.prisma — added SybilExplanation model.
  • sybil-resistance.service.ts — persist explanation separately and load it when needed.
  • sybil-resistance.service.spec.ts — updated mocks & added tests to verify persistence/loading.

Next steps

  • Run the unit tests locally:
npm install
npm test
  • Apply Prisma migration and generate client (for DB run):
npx prisma migrate dev --name move-sybil-explanation
npx prisma generate

Would you like me to run the tests and attempt a local Prisma migrate here?

Completed: Verify with protocol invariants (3/3)

Made changes.

closes #207

@dDevAhmed
Copy link
Copy Markdown
Contributor

resolve conflicts @nafsonig

@dDevAhmed dDevAhmed merged commit f5a88f2 into DigiNodes:main May 30, 2026
1 check failed
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.

Sybil explanation string too long

2 participants