Hey, i love your work, one thing for me, reading through it for the first time, Sometimes you write "Cost" sometimes you write "Silver" and in the table still use gp and cp, I would just put "Cost" everywhere and put cp,sp,gp behind ecery cost instead of putting just numbers and assuming a baseline of sp.
Other than that, i really really like your game and am kinda jealous of some things you came up with ;P
Edit after reading through everything: It says that int controlls casting spells, but at no point during spell crafting or use do you need int, you craft using dex and that's it. I guess learning new spells with a mind save kinda counts for it but i found it very confusing (because learning a spells doesn't mean casting it) . The spell section also doesnt actually tell you that you know no spells if you do not take the witchcraft ability.
I think the fact that you call using a crafted magic item "casting" is confusing because I was expecting there to be a small amount of actual spells somewhere (as in, that didn't need to be crafted beforehand)
Also, scrolls, how long do the curses last, cuz some don't have a duration (jewelry curses for example) and also, do you need to roll anything to use a scroll? The item sections says you can transcribe them, does that mean you can have them all in a spellbook and sell the scroll afterwards?
Sorry for being so annoying btw x)