fix: drop stray node_modules symlink and quote autoresearch frontmatter - #218
Merged
Conversation
… form A node_modules symlink pointing at a developer's local path was committed in #213 (2ef73b0). It slipped past .gitignore because `node_modules/` with a trailing slash only matches directories, not symlinks. Drop the tracked symlink and add the slash-less pattern so it cannot recur. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01NBqmTvJRhyysMvMvL6cp7x
…rses The unquoted description contains `: `, which js-yaml reads as a nested mapping and rejects. gray-matter therefore threw on this file, which made scripts/export-skills-catalog-v2.mjs abort on main and left autoresearch out of any catalog regeneration. With the value quoted, all 173 SKILL.md files parse and the export script runs to completion. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01NBqmTvJRhyysMvMvL6cp7x
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.
Two small repo-hygiene fixes surfaced while working on #196 / #213.
1. Remove a committed
node_modulessymlink#213 (
2ef73b0a) accidentally committed anode_modulessymlink pointing at a developer's local path (my mistake:git add -Ain a worktree that had the symlink). It slipped past.gitignorebecausenode_modules/with a trailing slash matches directories only, not symlinks. This PR removes the tracked symlink and adds the slash-lessnode_modulespattern so the symlink form is ignored too.Anyone who has pulled
mainsince #213 has a danglingnode_modulessymlink in their checkout; after this merges, agit pullremoves it.2. Quote the
autoresearchskill descriptionskills/autoresearch/SKILL.mdhad an unquoteddescription:containing:(9 subcommands: plan, ...), which js-yaml parses as a nested mapping and rejects. Consequences:scripts/export-skills-catalog-v2.mjsaborted with aYAMLExceptiononmain, so the catalog could not be regenerated.Quoting the value fixes it. Verified: all 173
SKILL.mdfiles parse with gray-matter, and the export script runs to completion.Note: this PR does not regenerate
skills-catalog-v2.json. A regen currently produces a ~9k-line diff against the committed catalog (ordering and metadata drift), which deserves its own PR.🤖 Generated with Claude Code
https://claude.ai/code/session_01NBqmTvJRhyysMvMvL6cp7x