Skip to content

add CLAUDE.md, encourage issues rather than PRs, and update the bundled x/tools files - #1077

Merged
luantak merged 4 commits into
burrowers:masterfrom
mvdan:wk-claude
Aug 24, 2026
Merged

add CLAUDE.md, encourage issues rather than PRs, and update the bundled x/tools files#1077
luantak merged 4 commits into
burrowers:masterfrom
mvdan:wk-claude

Conversation

@mvdan

@mvdan mvdan commented Aug 24, 2026

Copy link
Copy Markdown
Member

(see commit messages - please do not squash)

@mvdan
mvdan requested a review from luantak August 24, 2026 11:43
@mvdan

mvdan commented Aug 24, 2026

Copy link
Copy Markdown
Member Author

Maintaining this project without AI is unnecessarily hard, and I've been tweaking this CLAUDE.md file over the past few weeks, so I think it makes sense to commit it now.

@luantak

luantak commented Aug 24, 2026

Copy link
Copy Markdown
Member

Please do Agents.md and then symlink it to Claude.md

@mvdan

mvdan commented Aug 24, 2026

Copy link
Copy Markdown
Member Author

done

mvdan added 4 commits August 24, 2026 13:48
And symlink CLAUDE.md to it, for the sake of Claude Code.
Because a detailed issue is more valuable than a large patch.
The bundled copies still claimed v0.42.0 even though go.mod moved on.
Comparing fresh bundles of both versions, upstream only changed API that we
prune: Callee and StaticCallee now delegate to typesinternal, dropping the
go:linkname'd usedIdent and interfaceMethod helpers, and CoreType's doc comment
was reworded. The Hasher code we keep is unchanged.

So all that is left is the version note, and dropping the blank unsafe import
which only existed for those go:linkname helpers. Mark our hashTypeParam
patches with NOTE(garble) as well, as a future re-bundle would otherwise
silently revert them.
CONTRIBUTING.md described overwriting the bundled files with bundle's output
and re-applying our edits, which stopped being practical once we pruned them
down to the typeutil_Hasher closure, as that pruning would have to be redone
every time.

Rather than keeping the same information in two places, drop the section and
fold what is still true into CLAUDE.md, next to the up to date steps.
@luantak
luantak merged commit 5c230c8 into burrowers:master Aug 24, 2026
6 checks passed
@mvdan
mvdan deleted the wk-claude branch August 24, 2026 22:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants