Skip to content

ci: GitHub Actions CI + eslint/editorconfig + manifest 守护测试 - #17

Merged
libinyam merged 1 commit into
masterfrom
ci/workflow-lint
Aug 16, 2026
Merged

libinyam merged 1 commit into
masterfrom
ci/workflow-lint

Conversation

@libinyam

Copy link
Copy Markdown
Owner

关闭 #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)
  • 最低 Node 版本:package.json engines.node >=18(v0.3.1 已加)+ README Development 小节双语声明
  • 顺带:.gitignore 补 node_modules/

验证证据(本机)

$ npm run lint   → 0 errors
$ npm test       → tests 19, pass 19(guard 14 + manifest 5)

GitHub 端以本 PR 的 4 条 CI 腿为准——这也是测试套件第一次跑 Linux,全绿后才合并。

@libinyam
libinyam merged commit c4c81b3 into master Aug 16, 2026
4 checks passed
@libinyam
libinyam deleted the ci/workflow-lint branch August 16, 2026 03:13
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