archethic-foundation / archethic-foundation/archethic-node

Refactor Archethic to leverage Archethic.Mining

Open
#1,106 0 comments 0 reactions 0 assignees View on GitHub
quality
Dominant language
Elixir
Stars
82
Forks
24
PR merge metrics
No merged PRs in 30d

Description

I'm wondering if we should not move this function into the `Mining` context, so nested modules under `lib/archethic` will not call the top level `Archethic` but the siblings' context APIs.

Hence `Archethic` top level module will serve only `ArchethicWeb` which was the motivation around Phoenix contexts.

As `ArchethicWeb` doesn't need at all the `ContractContext`

_Originally posted by @samuelmanzanera in https://github.com/archethic-foundation/archethic-node/pull/1047#discussion_r1211914197_

Contributor guide

Open the contributing guide

Research direction

Start by reviewing the context boundaries under lib/archethic, especially Archethic.Mining, the top-level Archethic module, ArchethicWeb, and ContractContext. Trace the function referred to in the issue and inspect the linked pull request discussion for its current location and callers. Done means nested modules use the Mining context API and ArchethicWeb no longer depends on ContractContext.

Written by the indexing model from the issue text.

Assessment

Tech stack
elixir
Domain
blockchain
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.