Skip to content

chore(standards): 正本の写しを新しくする(build-sw.mjs の疎な配列) - #75

Merged
GIGAyama merged 2 commits into
mainfrom
claude/gigayama-system-review-xm7tz1
Sep 2, 2026
Merged

GIGAyama merged 2 commits into
mainfrom
claude/gigayama-system-review-xm7tz1

Conversation

@GIGAyama

@GIGAyama GIGAyama commented Sep 2, 2026

Copy link
Copy Markdown
Owner

なにを直したか

  • tools/build-sw.mjs: 正本 standards/sw/build-sw-vite.mjs の写し。[, '不明'] が eslint の no-sparse-arrays に当たる形だった(Quoridor と Reversi では npm run lint が実際に落ちていた)のを ['', '不明'] に直したもの。動きは変わらない。

正本の側の直しは GIGAyama.github.io#146 にある。そちらを先にマージしてください(写しが正本より新しい状態を作らないため)。

確かめたこと

  • 直した正本に対する check-drift が緑
  • npm run check(このリポジトリの gate)が緑

使う人から見て

変わらない(道具の直し)。更新ログには書いていない。

🤖 Generated with Claude Code

https://claude.ai/code/session_012McVFkty7uhkRRjq2q4uB1


Generated by Claude Code

何が起きていたか
- tools/build-sw.mjs の `[, '不明']` が eslint の no-sparse-arrays に当たり、lint が落ちる形になっていた

どう直したか
- tools/build-sw.mjs: 疎な配列を直した正本(standards/sw/build-sw-vite.mjs)の写し

正本の側の直しは GIGAyama.github.io の同じ枝(claude/gigayama-system-review-xm7tz1)にある。
ここに在る検査(check-drift・gate・build-sw --check)はすべて通してある。

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_012McVFkty7uhkRRjq2q4uB1

GIGAyama commented Sep 2, 2026

Copy link
Copy Markdown
Owner Author

CI の drift が赤いのは、この PR の写し(tools/build-sw.mjs)がまだマージされていない正本GIGAyama/GIGAyama.github.io#146)から写したものだからです。

  • drift ジョブは正本を GIGAyama.github.iomain から .standards-src へ取り出して照合します。main の正本はまだ古いので、新しい写しは「ずれている」と出ます。
  • この PR 側で直せるものではありません。写しを古い正本に戻せば緑になりますが、それでは直し(eslint の no-sparse-arrays に当たる疎な配列)が届きません。
  • 手元では #146 の正本に対して check-drift が緑、このリポジトリの gate(npm run check)も緑であることを確かめてあります。

#146 を先にマージしてください。 マージされたら、この枝に main を取りこんで CI を回し直し、緑になることを確かめます。


Generated by Claude Code

@GIGAyama
GIGAyama marked this pull request as ready for review September 2, 2026 21:14
@GIGAyama
GIGAyama merged commit 1bf90c8 into main Sep 2, 2026
2 checks passed
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