Skip to content

fix(lint): ignore .agents/ (skill sources upstream-managed)#72

Merged
themightychris merged 1 commit into
mainfrom
fix/ignore-skill-sources-in-lint
May 19, 2026
Merged

fix(lint): ignore .agents/ (skill sources upstream-managed)#72
themightychris merged 1 commit into
mainfrom
fix/ignore-skill-sources-in-lint

Conversation

@themightychris
Copy link
Copy Markdown
Member

PR #71 merged with a red CI — the failure was lint on the new .agents/skills/specops/scripts/lib/plans.js (CommonJS, flagged by the project's flat-config eslint).

Skill sources are upstream-managed via skills-lock.json's content hash — modifying them would break the sync. The right fix is to ignore the tree.

🤖 Generated with Claude Code

CI failed on PR #71 because `.agents/skills/specops/scripts/lib/plans.js`
uses CommonJS (`require()`, `module.exports`) — the project's eslint
config flags those because the rest of the codebase is ESM. The skill
sources are upstream-managed (pinned by `skills-lock.json` content hash);
modifying them would break the sync. Ignore the tree instead.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@themightychris themightychris merged commit 879e07a into main May 19, 2026
1 check passed
@themightychris themightychris deleted the fix/ignore-skill-sources-in-lint branch May 19, 2026 17:45
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.

1 participant