Repository navigation
docs: README walks a mod author from the template to a signed release - #3
Merged
Merged
Conversation
Covers making the repo yours with credits, updating an older copy of the template, the just verify gate, hooking the game, and how a tagged version becomes a draft, is reviewed, signed and published. Co-Authored-By: Andrew Bates <a@sprock.io>
feat/release-draft now contains chore/standards and origin/main (b53afe8, plugin API 7). README.md merged cleanly; main's "Plugin assets (API v7)" section sits next to the walkthrough and does not contradict it. just verify exits 0. Co-Authored-By: Andrew Bates <a@sprock.io>
Brings in the exports check (an extra export is an error). README unchanged. Co-Authored-By: Andrew Bates <a@sprock.io> * origin/feat/release-draft: fix: check_mod refuses exports other than the Nvr* ABI
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
README walkthrough for a mod author, end to end (work item 3), including how a release reaches signing. Stacked on #2 (base
feat/release-draft, which is itself on #1): merge #1, then #2, then this.What changes
NOTICE+Co-Authored-By), and a new subsection "Already started from an older copy of this template?" listing exactly which files to take and which to keep.just verifyis the gate (what it runs, what it needs,just fmt), the build stamp in one paragraph, and a new "Hooking the game" part (check bytes, never hook what nEVR hooks, prefer a hook to a byte patch, work out another project's patch yourself and credit it).No code changes.
Checked against a real mod: Aaliyah's tablet
Her repo (
reflectionsa/aaliyah-tablet, public,7792a22) is a copy of this template at25bf8b2: only 7 files differ from it (CMakeLists.txt:ECHO_MOD_MINHOOKdefault flipped to ON;LICENSE,README.md,echo-mod.json, her settings sheet,src/mod.cpp). I cloned it into scratch, overlaid #1/#2's tooling on a copy there (nothing pushed, nothing sent to her repo) and ran the gate:just fmt-checkbefore: 18 violations, all insrc/mod.cpp. Afterjust fmt:just verify-> exit 0:Ran 5 tests ... OK,Ran 30 tests ... OK, smoke[host log] smoke level 1: NEVR-PLUGIN-BUILD 0.1.0 0000000000000000000000000000000000000000, release checkok. The-Werrorbuild of hermod.cppis clean.So her path is: copy the tooling files (the new README subsection),
just fmt,just verify. What she would hit, in order:just fmtrewrites hersrc/mod.cpp(indentation/line breaks only).NOTICE: the template's has placeholder credits; hers must list babs (nmdurkee), tomski, he_is_the_cat, Xenomega/EchoRelay and EchoTools/evrFileTools, which her README already names. Without editing it, her release zip carries the placeholder.Co-Authored-By, andsrc/mod.cpptakes its addresses and structure from other projects ("Doom runtime.cpp", "EchoVr-Tablet-Probing", "Probing NOTES.md"). The new AGENTS.md rules ask for the analysis to be written down and the people credited. That is hers to answer, not something I changed.CUICanvasInstance::OnLoaded0x71fc90,CUIBehavior::FireDelegate0x7269f0): a grep of nevr-runtimeorigin/main(src plugins tools) for those addresses found nothing. That is a literal-address grep only; whether nEVR hooks them by another route is unchecked, and a signer reviewer will look.Design questions about her mod (capabilities, hooks, the settings sheet) are hers; I have not contacted her or touched her repo.
Not verified
v*tags in the copies, and whether the catalogue can read releases from the copy. Correct the text if the process differs.AGENTS.md#creditand#rules-all-of-them-are-checked-or-reviewedexist in the stacked tree; not click-tested on GitHub.Quest: not applicable (PC-only template).
Co-Authored-By: Andrew Bates a@sprock.io
Update 2026-10-28: main moved, brought up to date by merge
feat/release-draft(#2) now containschore/standardsandorigin/main(b53afe8, plugin API 7). Merged it into this branch (merge commit2e58e5f, a plain fast-forward push, no force-push).README.mdmerged cleanly. Main added a "Plugin assets (API v7)" section to the README; I read it against the walkthrough and they do not contradict.CI on
2e58e5f: build (push and PR) pass, PR-title check pass.Co-Authored-By: Andrew Bates a@sprock.io
Update 2026-10-28 (2)
Merged
feat/release-draftagain (merge commit3c7ba29, fast-forward push) to carry the exports check (#2: an export outsideNvr*is an error). README unchanged.release.ymlran once on GitHub on a throwaway tag (details in #2); the README's "how a release reaches signing" text describes that workflow's real output. CI on3c7ba29: build (push and PR) pass, PR-title check pass.Co-Authored-By: Andrew Bates a@sprock.io