fix(ci): repair MegaLinter full-repo scan failures on main#123
Open
ChipWolf wants to merge 1 commit into
Open
Conversation
The push-event scan sets VALIDATE_ALL_CODEBASE, surfacing four blocking findings that per-file PR scans never linted: - LUA_STYLUA: reformat headless-install.lua to double-quote string style. - TYPESCRIPT_STANDARD: exclude skills/agent-audit/lib/opencode-plugin.ts, a standalone skill resource outside tsconfig.json's include (same class as the already-excluded vendored rtk.ts). - JAVASCRIPT_STANDARD: exclude skills/*.workflow.js Workflow-tool script bodies, whose top-level return/await parses as 'return' outside function. - YAML_V8R: exclude home/dot_glzr/glazewm/config.yaml; SchemaStore ships the GlazeWM v2 schema while this config is v3 (no upstream v3 schema). Co-Authored-By: Claude Opus 4.8 <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
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.
What
Fixes the four blocking MegaLinter findings that fail the workflow on
main. The push-event scan setsVALIDATE_ALL_CODEBASE, so it lints the whole repo, surfacing files that per-file PR scans never checked.LUA_STYLUAhome/dot_config/nvim/scripts/headless-install.luaTYPESCRIPT_STANDARDskills/agent-audit/lib/opencode-plugin.tstsconfig.json'sinclude, soparserOptions.projectcan't parse it; excluded (same class as the already-excluded vendoredrtk.ts)JAVASCRIPT_STANDARDskills/consensus/resources/consensus.workflow.jsreturn/awaitis valid there but parses as'return' outside of functionunder standardjs; excludedskills/*.workflow.jsYAML_V8Rhome/dot_glzr/glazewm/config.yaml**/glazewm/config.yamlto its GlazeWM v2 schema, but this config is v3 (restructured; no upstream v3 schema exists), so every v3 key reads as an unknown property; excludedWhy these exclusions are genuine false positives (not silencing)
Per repo policy, the one real finding (lua formatting) is fixed, not muted. The three exclusions each carry an inline comment naming the linter and reason, and were verified against evidence:
glazewm.jsonis the v2 schema (top-levelbar/workspaces,general.window_animations) and thatglzr-io/glazewmships noconfig-schema.json..workflow.jsparse error is structural — top-levelreturn/awaitcannot be made standardjs-valid without breaking the Workflow runtime..tsfile is outside the maintained opencode-plugins TS project, mirroring the existingrtk.tsexclusion.Reviewer notes
executable_skills-review"not executable" bash-exec hit) are unaffected — they don't fail CI and are out of scope here.🤖 Generated with Claude Code