Skip to content

fix(ci): eslint-plugin configs 漏交 + 移除残缺 land 恢复 ootb - #48

Merged
Tanya7z merged 1 commit into
mainfrom
cursor/bc-151dec9d-a38e-4ca4-9041-841d9f8b35c6-a9c4
Jul 23, 2026
Merged

fix(ci): eslint-plugin configs 漏交 + 移除残缺 land 恢复 ootb#48
Tanya7z merged 1 commit into
mainfrom
cursor/bc-151dec9d-a38e-4ca4-9041-841d9f8b35c6-a9c4

Conversation

@cursor

@cursor cursor Bot commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

触发

ootbmain@25f6d23 / 3e6f0e8 失败(runs 3000158324030001567889)。

审查(DRY · OCP · DIP · LSP · Demeter)

级别 原则 问题
BLOCKER DRY .gitignoreconfigs/ 误伤 **/configs/**@sfmc-bds/eslint-pluginsrc/configs/recommended.ts 从未入库 → CI TS2307
BLOCKER DIP / LSP 主仓 packages/land 声明 requires: feature-economyservices.requires economy.*,但 economy 未安装 → check-modulesloadManifestV2 启动失败
MINOR LSP ESLint 10 Plugin.rules@typescript-eslint RuleModule 类型不重叠(运行时兼容,需断言)
正向 OCP / DIP eslint-plugin 规则注册表 + flat config 工厂;业务模块外置、开箱仅 afk fixture

补丁

  1. .gitignoreconfigs//configs/
  2. 补齐 modules/sdk/@sfmc-eslint-plugin/src/configs/recommended.ts(及 dist)
  3. plugin as unknown as ESLint.Plugin 对齐类型边界
  4. 移除残缺 in-repo modules/packages/land,恢复 afk-only catalog.json / module-lock.json;清理 lock 中对旁路 sfmc-modules/economy 的链接

land / economy 仍应从 Tanya7z/sfmc-modulesfetch-module install 安装,不应以半闭包形式落在主仓。

验证

npm run build --workspaces --if-present   # PASS
SFMC_ROOT=$PWD npm run check-ootb        # 12/12 PASS
Open in Web View Automation 

BLOCKER 1 — DRY/.gitignore:configs/ 误伤 **/configs/**,
@sfmc-bds/eslint-plugin 的 src/configs/recommended.ts 未入库 → TS2307。
改为 /configs/,补回规则注册表与 flat config 工厂。

BLOCKER 2 — DIP/LSP:主仓 packages/land 依赖未安装的 feature-economy,
check-modules 与 loadManifestV2 启动失败。按「业务模块外置、开箱仅 afk」
移除残缺 land,恢复 catalog/lock;land/economy 经 fetch-module 安装。

Co-authored-by: Shiroha <Tanya7z@users.noreply.github.com>
@Tanya7z
Tanya7z marked this pull request as ready for review July 23, 2026 11:13
@cursor

cursor Bot commented Jul 23, 2026

Copy link
Copy Markdown
Contributor Author

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

@Tanya7z
Tanya7z merged commit 4c3cba8 into main Jul 23, 2026
1 check passed
cursor Bot pushed a commit that referenced this pull request Jul 23, 2026
#48 已修复 eslint-plugin configs 漏交与残缺 land,但 package-lock
仍保留 modules/packages/land 与旁路 sfmc-modules/economy 包条目
(标记 extraneous)。彻底删除,避免假依赖信号(DRY)。

Co-authored-by: Shiroha <Tanya7z@users.noreply.github.com>
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.

2 participants