Skip to content

Conversation

@kiliantyler
Copy link
Collaborator

@kiliantyler kiliantyler commented Jan 4, 2026

Core

  • Updated to Node 22

Testing

  • Added critical path tests
    • tests/messaging.test.js - P2P messaging layer tests
    • tests/security.test.js - PoW and Ed25519 signature tests
    • tests/state.test.js - Bloom filter, HyperLogLog, LRU cache, and peer state tests
  • scripts/test-network.js - Tests server P2P connection
  • New CI workflow (.github/workflows/test.yml) for automated test runs

Code Quality

  • Integrated Biome for linting and formatting
  • Added .editorconfig for cross-editor consistency

DevEx

  • Added VS Code configuration:
    • extensions.json with recommended extensions
    • settings.json with project-specific settings
  • Added CONTRIBUTING.md

@kiliantyler kiliantyler requested a review from lklynet January 4, 2026 04:12
@kiliantyler kiliantyler changed the title Feat/testing linting formating Testing, linting, formating. Oh my! Jan 4, 2026
@kiliantyler kiliantyler force-pushed the feat/testing-linting-formating branch from 69b507f to 6ac1364 Compare January 4, 2026 04:33
@fccview
Copy link
Collaborator

fccview commented Jan 4, 2026

Well then, we officially can't merge any of the other pull requests anymore 😆
this is amazing work tho

@DataDalton
Copy link
Contributor

Well then, we officially can't merge any of the other pull requests anymore 😆
this is amazing work tho

Most will still be fine, just a quick update from upstream should fix them

@fccview
Copy link
Collaborator

fccview commented Jan 4, 2026

Well then, we officially can't merge any of the other pull requests anymore 😆
this is amazing work tho

Most will still be fine, just a quick update from upstream should fix them

Alright, we should merge this first as high priority, can we make sure everything works and we'll just merge it?
We need to start making moves or things will keep piling up forever. (not at my laptop right now, sorry)

@DataDalton
Copy link
Contributor

Well then, we officially can't merge any of the other pull requests anymore 😆
this is amazing work tho

Most will still be fine, just a quick update from upstream should fix them

Alright, we should merge this first as high priority, can we make sure everything works and we'll just merge it?
We need to start making moves or things will keep piling up forever. (not at my laptop right now, sorry)

This should be last still and then run linting and formatting at the end of current queue

@DataDalton
Copy link
Contributor

DataDalton commented Jan 4, 2026

Although space indent 2 I'm not sure I agree with, 4 is definitely a standard already

@fccview
Copy link
Collaborator

fccview commented Jan 4, 2026

Well then, we officially can't merge any of the other pull requests anymore 😆
this is amazing work tho

Most will still be fine, just a quick update from upstream should fix them

Alright, we should merge this first as high priority, can we make sure everything works and we'll just merge it?
We need to start making moves or things will keep piling up forever. (not at my laptop right now, sorry)

This should be last still and then run linting and formatting at the end of current queue

Fair, I think regardless of the order things should start moving or we'll get in more and more of a hole lol

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