fix(ci): SMI-4785 ignore git-crypt encrypted .ts files in ESLint#1002
Merged
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
E2E Test ResultsE2E Test Results - May 7, 2026Summary
Test Results
Generated by skillsmith E2E test suite |
Dependabot PRs don't get GIT_CRYPT_KEY secret, so encrypted files under .claude/skills/, .claude/plans/, .claude/hive-mind/ reach ESLint as binary blobs and fail with "Parsing error: Invalid character" (verified PR #851 SHA 12766ef: .claude/skills/governance/templates/edge-function-test-template.ts). Mirrors the existing supabase/functions/** + supabase/migrations/** pattern in the same globalIgnores block. Closes SMI-4785. Co-Authored-By: claude-flow <ruv@ruv.net> Co-Authored-By: Claude <noreply@anthropic.com>
wrsmith108
force-pushed
the
fix/smi-4785-eslint-claude-skills-ignore
branch
from
97e8a06 to
489a32f
Compare
E2E Test ResultsE2E Test Results - May 7, 2026Summary
Test Results
Generated by skillsmith E2E test suite |
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.
Summary
secrets.GIT_CRYPT_KEYis non-empty. Dependabot PRs (and any fork PR) don't get repo secrets — encrypted.tsfiles reach ESLint as binary blobs and fail withParsing error: Invalid character..claude/skills/**/*.ts,.claude/plans/**/*.ts,.claude/hive-mind/**/*.tstoglobalIgnores.ignoresineslint.config.js(mirroring the existingsupabase/functions/**+supabase/migrations/**entries that exist for the exact same reason).Why now
Surfaced from SMI-4781 batch execution 2026-05-06: dependabot PR #851 (workflow-YAML-only — wasn't masked by the lock-regen issue SMI-4784) failed Lint at
.claude/skills/governance/templates/edge-function-test-template.ts(run 25473022995/job 74740649816). Main has been quietly green only because trusted-context CI gets the unlock secret.Test plan
npm run lintpasses (encrypted files unlocked locally — proves no false-positives introduced)[skip-impl-check]: config-only change to
eslint.config.js; no source code changes per SMI-4785 — seeeslint.config.jsignores block🤖 Generated with Ruflo
Co-Authored-By: claude-flow ruv@ruv.net
Co-Authored-By: Claude noreply@anthropic.com