## [mirendev/runtime#1412](https://github.com/mirendev/runtime/pull/1412)

This is ready for human review. Since my last review the branch has been rebased onto `main`: the head moved from `4f62021` to `dd217b5`, the seven commit messages are the same, and `pr_status` now reports the PR as mergeable, so the rebase point is settled. I found no new defects in the rebased code.

The server-version gap I raised still stands, and my thread at `docs/docs/deployment.md:149` stays open:
- `UploadFiles` (`appconfig/source.go:19-24`) still copies the service's `app.toml` into the upload byte for byte, so `build.source_root` reaches the server.
- `loadAppConfig` (`servers/build/build.go:1238`) parses that file with `appconfig.Parse`, which calls `DisallowUnknownFields()`. A server built before this PR will therefore reject the key.
- Line 149 and the PR description still say the server needs no changes.

Either remove the key from the projected copy, or document that this feature needs a server that knows `source_root`. Either would close the thread.

I'm still not recommending approval, for the same reasons as last time:
- The override path in `makeTarWithFilter` runs on every deploy.
- The Elixir release preference changes which release existing umbrella apps build.
- The server-compatibility question above still needs a decision.

phinze's threads and my old profile handoff thread are still for people to close.

---

**Verdict:** ready