fix(gas): 正本配布で作り直されなかった giga-links.html を直し、正本の写しを新しくする - #79
Merged
Merged
Conversation
何が起きていたか
- 2026-08-30 の正本配布で web/giga-app-links.js が変わったのに、GAS へ焼きこむ
giga_links.html が作り直されないまま main へ運ばれていた。`node tools/build-app-links.mjs --check` が赤かった
- tools/build-sw.mjs の入口判定が `file://${process.argv[1]}` の文字列比較で、空白や日本語を含む
パスでは main() が呼ばれず、何も見ないまま exit 0 になっていた(--check の緑が嘘になる)
- tools/check-secrets.mjs の入口判定も同じ文字列比較で、パスによっては何も見ずに「秘密の直書きはありません」の側へ倒れていた
どう直したか
- giga_links.html: web/giga-app-links.js から作り直した(node tools/build-app-links.mjs)
- tools/build-sw.mjs: 入口判定を pathToFileURL に直した正本(standards/sw/build-sw-static.mjs)の写し
- tools/check-secrets.mjs: 入口判定を pathToFileURL に直した正本(standards/lib/check-secrets.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
Owner
Author
|
CI の
#146 を先にマージしてください。 マージされたら、この枝に main を取りこんで CI を回し直し、緑になることを確かめます。 Generated by Claude Code |
GIGAyama
marked this pull request as ready for review
September 2, 2026 21:20
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.
なにを直したか
giga-links.html: 2026-08-30 の正本配布でweb/giga-app-links.jsが変わったのに、GAS へ焼きこむこのファイルが作り直されないまま main に入っていた。node tools/build-app-links.mjs --checkが赤く、npm run checkが最初の 1 行で落ちる状態だった。node tools/build-app-links.mjsで作り直した。tools/build-sw.mjs: 正本standards/sw/build-sw-static.mjsの写し。入口判定が`file://${process.argv[1]}`の文字列比較で、空白や日本語を含むパスでは何も見ずに exit 0 になっていた(--checkの緑が嘘になる)のをpathToFileURLに直したもの。tools/check-secrets.mjs: 正本standards/lib/check-secrets.mjsの写し。同じ入口判定の直し。なぜ配布が作り直さなかったか
distribute.mjsは写してgit add .するだけで、写しから作るもの(GAS への焼きこみ、sw.js の版)を作り直していなかった。正本の側の直しは GIGAyama.github.io#146 にある。そちらを先にマージしてください(写しが正本より新しい状態を作らないため)。確かめたこと
check-driftが緑npm run check(このリポジトリの gate)が緑node tools/build-app-links.mjs --checkとnode tools/build-sw.mjs --checkが緑(docs/ の sw.js は変わらない)使う人から見て
CI と生成物の直しなので更新ログには書いていない。GAS 本体はマージだけでは配られない(
docs/copy-distribution.mdのとおり配布用テンプレートの更新が別に要る)。🤖 Generated with Claude Code
https://claude.ai/code/session_012McVFkty7uhkRRjq2q4uB1
Generated by Claude Code