Skip to content

Refresh CLAUDE.md development guide - #277

Open
byalex33 wants to merge 1 commit into
FactoryGameFan:wormeyman-space-age-supportfrom
byalex33:cleanup/claude-md-guide
Open

Refresh CLAUDE.md development guide#277
byalex33 wants to merge 1 commit into
FactoryGameFan:wormeyman-space-age-supportfrom
byalex33:cleanup/claude-md-guide

Conversation

@byalex33

Copy link
Copy Markdown
Contributor

What changed

Replaced the 996-line historical CLAUDE.md with a 197-line current development guide.

This was split from #268 so the documentation rewrite can be reviewed independently from the mechanical cleanup.

Verification

Documentation-only change; no runtime code changed.

@wormeyman

Copy link
Copy Markdown
Collaborator

I filed #278 with what this needs before merge: 7 statements in the new file that are wrong, and 14 findings from the old file that have no other home in the repo. Most of those 14 are one code comment each rather than prose to put back into CLAUDE.md.

The trim itself checks out. The oracle method, the dependency holds and the per-spec rationales really are duplicated at the code site, and four things in the old file were already stale.

One ordering note: this needs to land after #268. The new file documents factorio-probe.mjs, which only exists on that branch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants