chore: Add CLAUDE.md - #7973
Draft
Rathoz wants to merge 3 commits into
Draft
Conversation
Rathoz
changed the title
Rathoz
marked this pull request as draft
August 17, 2026 11:28
ElectricalBoy
added
the
qol
Rathoz
force-pushed
the
chore/claude-md
branch
from
August 19, 2026 13:02
7df30d5 to
aee9d04
Compare
Rathoz
force-pushed
the
feat/devcontainer
branch
2 times, most recently
from
August 20, 2026 08:28
4998a86 to
e884b19
Compare
Claude Code sessions default to whatever is on the host, which for this repository is usually a partial toolchain: lua may be a different version, and busted or luacheck may be missing entirely, so results do not say much about what CI will do. Points it at the devcontainer instead, and tells it to stop rather than fall back to the host when docker is unavailable. Also records the things that are easy to get wrong from the outside: that lint:js edits files, that snapshots are CI's to update, and that deploy.py writes to the live wiki without LUA_DEV_ENV_NAME.
Refusing to run anything without docker is stricter than it needs to be, since a complete host toolchain gives a usable answer. Allows the fallback, on the conditions that the tool is confirmed present rather than assumed, and that the result says which environment produced it, so a host result is not mistaken for a container one.
The previous wording made the devcontainer the canonical setup and the host a fallback, which is not how this repository works. Contributors install the toolchain natively too, which is straightforward on linux, where the devcontainer mostly earns its keep on macos and its deprecated lua@5.1 formula. Asks it to check what the machine has instead, run there, and keep the part that matters, that a result names the setup it came from.
Rathoz
force-pushed
the
chore/claude-md
branch
from
August 21, 2026 11:53
aee9d04 to
17f3de3
Compare
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
2 participants
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.
What
Adds a
CLAUDE.mdso Claude Code sessions check what a machine actually has before running repository commands, rather than assuming a toolchain is present. A native setup and the devcontainer are treated as equals — contributors install natively too, which is straightforward on Linux, where the devcontainer mostly earns its keep on macOS.If neither is available it is told to say so, rather than skip a check or report it as passing, and to name which setup produced a result when that is not obvious.
Also records a few things that are easy to get wrong from the outside:
lint:jsrunseslint --fixand edits files, snapshots are CI's to update, anddeploy.pywrites to the live wiki whenLUA_DEV_ENV_NAMEis unset.How it was tested
Not executable, so nothing to run. The commands it lists are the ones exercised in #7955:
npm run lua-test,luacheck,npm run lint:scss,npm run lint:js,ruff check/format --check, andnpm run build, all verified inside the container there.