Skip to content

refactor: move utilities with zero dependencies#1592

Merged
r4mmer merged 9 commits intomasterfrom
feat/hathorlib-blueprint-move-1
Feb 26, 2026
Merged

refactor: move utilities with zero dependencies#1592
r4mmer merged 9 commits intomasterfrom
feat/hathorlib-blueprint-move-1

Conversation

@r4mmer
Copy link
Member

@r4mmer r4mmer commented Feb 5, 2026

Motivation

First PR on a sequence to move the core of the blueprint implementation to the hathorlib

Acceptance Criteria

  • Move faux immutable and other utilities that have no dependencies on hathor
  • Change the hathor files to re-export the hathorlib implementation

Checklist

  • If you are requesting a merge into master, confirm this code is production-ready and can be included in future releases as soon as it gets merged

@r4mmer r4mmer self-assigned this Feb 5, 2026
@r4mmer r4mmer moved this from Todo to In Progress (Done) in Hathor Network Feb 5, 2026
@github-actions
Copy link

github-actions bot commented Feb 5, 2026

🐰 Bencher Report

Branchfeat/hathorlib-blueprint-move-1
Testbedubuntu-22.04
Click to view all benchmark results
BenchmarkLatencyBenchmark Result
minutes (m)
(Result Δ%)
Lower Boundary
minutes (m)
(Limit %)
Upper Boundary
minutes (m)
(Limit %)
sync-v2 (up to 20000 blocks)📈 view plot
🚷 view threshold
1.64 m
(-4.16%)Baseline: 1.71 m
1.54 m
(93.91%)
2.05 m
(79.86%)
🐰 View full continuous benchmarking report in Bencher

@codecov
Copy link

codecov bot commented Feb 5, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.66%. Comparing base (36fff29) to head (9557a71).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff            @@
##           master    #1592    +/-   ##
========================================
  Coverage   85.65%   85.66%            
========================================
  Files         439      439            
  Lines       33618    33515   -103     
  Branches     5282     5264    -18     
========================================
- Hits        28797    28709    -88     
+ Misses       3802     3791    -11     
+ Partials     1019     1015     -4     
Flag Coverage Δ
test-lib 85.66% <100.00%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@r4mmer r4mmer requested a review from glevco February 5, 2026 14:51
msbrogli
msbrogli previously approved these changes Feb 5, 2026
@r4mmer r4mmer moved this from In Progress (Done) to In Review (WIP) in Hathor Network Feb 5, 2026
@glevco glevco moved this from In Review (WIP) to In Review (Done) in Hathor Network Feb 6, 2026
@github-project-automation github-project-automation bot moved this from In Review (Done) to In Review (WIP) in Hathor Network Feb 6, 2026
@r4mmer r4mmer moved this from In Review (WIP) to In Progress (WIP) in Hathor Network Feb 6, 2026
@msbrogli msbrogli force-pushed the master branch 2 times, most recently from eb416fa to 21d7909 Compare February 12, 2026 23:12
@r4mmer r4mmer mentioned this pull request Feb 18, 2026
1 task
…print-move-1

* origin/master:
  feat: pydantic settings (#1600)
  fix[thin_wallet]: handle address history invalid tx version (#1590)
  refactor(nano): Make NCBlockExecutor a pure executor with no side effects
  fix[nginx]: Make sure we trust the GCP IPs to get the real client IP (#1595)
  refactor: Upgrade to Pydantic v2
  chore(github): Split GitHub main action into lint, test-cli, test-lib, test-other
  fix[nginx]: Use a larger buffer size for /v1a/status (#1594)
  chore: adjust testnet config for v0.69.0 release
  chore[storage]: Limit the total size of RocksDB WAL files (#1518)
  chore: adjust testnet config for v0.69.0 release
  chore: configure feature activations for v0.69.0 release
  refactor: wallet on_new_tx (#1561)
  refactor(nano): Remove dead reorg cleanup code from block executor
@r4mmer r4mmer moved this from In Progress (WIP) to In Progress (Done) in Hathor Network Feb 24, 2026
msbrogli
msbrogli previously approved these changes Feb 24, 2026
glevco
glevco previously approved these changes Feb 25, 2026
@github-project-automation github-project-automation bot moved this from In Progress (Done) to In Review (WIP) in Hathor Network Feb 25, 2026
@glevco glevco moved this from In Review (WIP) to In Review (Done) in Hathor Network Feb 25, 2026
@r4mmer r4mmer dismissed stale reviews from msbrogli and glevco via 9557a71 February 25, 2026 17:41
@github-project-automation github-project-automation bot moved this from In Review (Done) to In Review (WIP) in Hathor Network Feb 25, 2026
@r4mmer r4mmer merged commit 82ac020 into master Feb 26, 2026
33 of 34 checks passed
@r4mmer r4mmer deleted the feat/hathorlib-blueprint-move-1 branch February 26, 2026 03:26
@github-project-automation github-project-automation bot moved this from In Review (WIP) to Waiting to be deployed in Hathor Network Feb 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Waiting to be deployed

Development

Successfully merging this pull request may close these issues.

3 participants