Retire planemo fork in convert-nfcore cast (last holdout, follow-up to #326)#327
Merged
Conversation
Last fork holdout. Two changes, both deterministic: - 7 research refs mode: condense -> verbatim. It was the only Mold using condense; the rest of the inventory copies research verbatim. Prior cast always passthrough-condensed (dst == src), so condense added nothing. - planemo ref now sources released 0.75.44; jmchilton fork retired everywhere. Re-cast: notes move references/patterns/ -> references/notes/, stale patterns/ dir removed, SKILL.md + _required_tools.json refreshed. cast --check clean, verifier clean (13 refs), validate 0 errors, repo-wide fork sweep now empty. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
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
1 participant
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.
Follow-up to #326 (merged). My convert-nfcore commit landed on the branch just after #326 merged, so it missed the train — this PR carries it onto main. It is the last holdout of the jmchilton/planemo fork retirement; after this the repo-wide fork sweep is empty.
What
convert-nfcore-module-to-galaxy-toolwas out of line with the rest of the inventory in three converging ways, all fixed by one move:mode: condenseuser. It was the only Mold usingcondense(7 refs); every other Mold copies research verbatim (127 refs). The prior cast always passthrough-condensed — output byte-identical to source — so condense added nothing here. Switched all 7 research refscondense → verbatim.0.75.44(carries merged planemo#1636), matching every other consumer.type: research, the notes belong underreferences/notes/, notreferences/patterns/.Result
Re-cast is now fully deterministic (no LLM phase), like
discover-shed-tooland every sibling:references/patterns/ → references/notes/, stalepatterns/dir removedplanemo.mdre-copied from0.75.44;SKILL.md+_required_tools.jsonrefreshedGates
cast --checkclean · cast verifier clean (13 refs)npm run validate0 errors ·make check-assemble-pipelinescleanjmchilton/planemosweep now empty🤖 Generated with Claude Code