Clean up re-exports

[zealsham]

This pull request resolves 688 , by moving reexports to the core module which gives us a cleaner lib.rs file .

This pr also implements the comment by Dan for the core modules.

[spacebear21]

This is looking good! There are just two changes that need to be made to remove core from the public API and reduce the diff by making the uri module `pub(crate).

Additionally once you've applied those changes, can you ensure you've run cargo fmt to enforce sorting/grouping of imports?

Lastly, the commit message should be updated to reflect the change as a whole and follow the seven rules (specifically rule 7 in this case).

[spacebear21]

There are formatting changes that appear unrelated to the PR. Are you using the nightly rust toolchain (per https://github.com/payjoin/rust-payjoin?tab=readme-ov-file#code-formatting)?

$ rustup default nightly
$ cargo fmt

[DanGould]

@zealsham beware this PR is super useful tech debt payback but the floor is lava because of the SesssionEventLog persister changes; the timing is gonna have to line up really well with the other PRs and it's going to require some periodic rebasing this week

[zealsham]

@spacebear21 yes , it was formated using rustup on nightly build .

[zealsham]

Thanks for the heads-up Dan, I’ll keep an eye on the related PRs and make sure to rebase as needed to keep everything aligned.

[spacebear21]

yes , it was formated using rustup on nightly build .

Hmm. I don't get those formatting changes if i run cargo fmt locally on the latest master.

I'm on:

❯ rustup default nightly
info: using existing install for 'nightly-aarch64-apple-darwin'
info: default toolchain set to 'nightly-aarch64-apple-darwin'

  nightly-aarch64-apple-darwin unchanged - rustc 1.89.0-nightly (1677d46cb 2025-06-10)

[zealsham]

rustup 1.28.2 (e4f3ad6f8 2025-04-28)
info: This is the version for the rustup toolchain manager, not the rustc compiler.
info: The currently active rustc version is `rustc 1.89.0-nightly (6ccd44760 2025-06-08)

[coveralls]

Pull Request Test Coverage Report for Build 15949260878

Details

• 0 of 0 changed or added relevant lines in 0 files are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage remained the same at 85.487%

---

Totals
Change from base Build 15935308780:	0.0%
Covered Lines:	7251
Relevant Lines:	8482

---

💛 - Coveralls

[spacebear21]

Rebased and removed the unrelated formatting changes.

Rebased and removed the unrelated formatting changes.

[spacebear21]

ACK 0a7fdca