Bom, a chamada da função está vindo sem os parâmetors n1 e n2, porém mesmo assim resultaria em erro, pois a questão é que toda a vez que você clica no botão ele deve trazer os novos valores de number1 e number2 para n1 e n2 e isso não está acontecendo aqui.
Entçao para contornar essa situação criei uma arrowfunction e para que toda a vez que o botão é clicado ele busca o valor de number1 e number2, atribui em n1 e n2 e faz o cálculo da média atribuindo na variável média e por fim, sendo escrito no documento com document.write.
HTML
<input type="number" name="numero1" id="number1">
<input type="number" name="numero2" id="number2">
<button>media</button>
JAVASCRIPT
var numero1 = document.querySelector("#number1");
var numero2 = document.querySelector("#number2");
var botao = document.querySelector("button");
botao.onclick = () => {
var n1 = Number(numero1.value);
var n2 = Number(numero2.value);
var media = (n1 + n2) / 2;
document.write("Média: " + media);
}
Teste o código em:
https://jsfiddle.net/veo9fj6y/