Skip to content

feat: pydantic settings#1600

Merged
r4mmer merged 18 commits intomasterfrom
feat/hathorlib-pydantic-settings
Feb 24, 2026
Merged

feat: pydantic settings#1600
r4mmer merged 18 commits intomasterfrom
feat/hathorlib-pydantic-settings

Conversation

@r4mmer
Copy link
Member

@r4mmer r4mmer commented Feb 18, 2026

Motivation

The unification of the hathor and hathorlib settings was made a pre-requisite (ref) to merge #1592

Acceptance Criteria

  • Make settings on hathor and hathorlib based on pydantic
  • Load settings module on hathor and hathorlib from the same files
  • Make hathor settings a superset of hathorlib settings

Note

hathorlib now supporpts yaml settings through HATHOR_CONFIG_YAML

Warning

Beaking change: Moved hathorlib settings envvar from TXMINING_CONFIG_FILE to HATHOR_CONFIG_FILE or HATHOR_CONFIG_YAML if using yaml settings

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 18, 2026
@r4mmer r4mmer moved this from Todo to In Progress (WIP) in Hathor Network Feb 18, 2026
@github-actions
Copy link

github-actions bot commented Feb 18, 2026

🐰 Bencher Report

Branchfeat/hathorlib-pydantic-settings
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.55 m
(-9.65%)Baseline: 1.71 m
1.54 m
(99.61%)
2.05 m
(75.29%)
🐰 View full continuous benchmarking report in Bencher

…ntic-settings

* origin/master:
  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
@codecov
Copy link

codecov bot commented Feb 19, 2026

Codecov Report

❌ Patch coverage is 88.63636% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 85.68%. Comparing base (23f26ca) to head (5abf4cf).
⚠️ Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
hathor/conf/get_settings.py 72.72% 2 Missing and 1 partial ⚠️
hathor/conf/settings.py 92.85% 1 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1600      +/-   ##
==========================================
- Coverage   85.81%   85.68%   -0.13%     
==========================================
  Files         442      439       -3     
  Lines       33862    33618     -244     
  Branches     5297     5282      -15     
==========================================
- Hits        29058    28807     -251     
- Misses       3793     3797       +4     
- Partials     1011     1014       +3     
Flag Coverage Δ
test-lib 85.68% <88.63%> (-0.13%) ⬇️

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 moved this from In Progress (WIP) to In Progress (Done) in Hathor Network Feb 19, 2026
jansegre
jansegre previously approved these changes Feb 19, 2026
@r4mmer r4mmer moved this from In Progress (Done) to In Review (WIP) in Hathor Network Feb 19, 2026
msbrogli
msbrogli previously approved these changes Feb 20, 2026
@r4mmer r4mmer moved this from In Review (WIP) to In Review (Done) in Hathor Network Feb 20, 2026
…' into feat/hathorlib-pydantic-settings

* origin/feat/hathorlib-pydantic-settings:
  fix[thin_wallet]: handle address history invalid tx version (#1590)
@r4mmer r4mmer dismissed stale reviews from msbrogli and jansegre via 27012ed February 24, 2026 01:25
@github-project-automation github-project-automation bot moved this from In Review (Done) to In Review (WIP) in Hathor Network Feb 24, 2026
@r4mmer r4mmer merged commit 735f4e0 into master Feb 24, 2026
33 checks passed
@r4mmer r4mmer deleted the feat/hathorlib-pydantic-settings branch February 24, 2026 15:37
@github-project-automation github-project-automation bot moved this from In Review (WIP) to Waiting to be deployed in Hathor Network Feb 24, 2026
r4mmer added a commit that referenced this pull request Feb 24, 2026
…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
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