ci: GitHub Actions CI + eslint/editorconfig + manifest 守护测试 - #17
Merged
Merged
Conversation
2 tasks
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.
关闭 #3、关闭 #8。
#3 CI
.github/workflows/ci.yml:push(master)/PR 触发,矩阵{ubuntu,windows} × {node 18,22}共 4 条腿,每条跑npm ci→npm test→npm run lint。跨平台矩阵正好覆盖 guard.js 的平台分支:win32 的where预检 / ETIMEDOUT 超时,POSIX 的 127/126 分类。manifest 校验按本仓库哲学做成测试而不是 CI 脚本:
tests/manifest.test.js(5 用例)进npm test,本地与 CI 同一套闸门——plugin.json/marketplace.json/hooks.json 结构、SKILL.md 与 reviewer 的 frontmatter、以及 CLAUDE.md 硬约定 3(跨文件引用一致性)的机器守护。manifest 坏 → 本地红、CI 红。#8 lint / 格式化 / Node 版本
eslint.config.js(flat config,eslint 9)+npm run lint,CI 执行.editorconfig作为格式基线(不引入 prettier,避免对现有文件的格式化 churn)engines.node >=18(v0.3.1 已加)+ README Development 小节双语声明.gitignore补node_modules/验证证据(本机)
GitHub 端以本 PR 的 4 条 CI 腿为准——这也是测试套件第一次跑 Linux,全绿后才合并。