0_RunAll.ps1 に -IgnoreErrors を渡せず、既知のエラーで必ず NG になる

[OsscJpDevInfra]

概要

0_RunAll.ps1 の結果を、そのまま合否として読めない。

1_BuildAll.ps1 には -IgnoreErrors があり、既知のエラー（ClickOnce の署名）を
合否判定から外せる。0_RunAll.ps1 にはそれを渡す口が無い。

このため 0_RunAll.ps1 を回すと、環境に由来する既知のエラーで必ず 1 本が NG になる。

現状

.\0_RunAll.ps1 -Lang Both

================ 全体のまとめ ================

スクリプト        終了コード
----------------- ----------
1_BuildAll.ps1    1          ← 既知の MSB3482 だけで NG
2_RunAllTests.ps1 0
3_SmokeTest.ps1   0

1 本が 0 以外で終了しました。上のログを確認してください。

エラーの中身はこれ 1 件だけである。

[WSClnt_sample (net48)] error MSB3482: 署名中にエラーが発生しました:
  WSClientWinCone_sample.exe の署名に失敗しました。
  SignTool Error: No certificates were found that met all the given criteria.

WSClientWinCone_sample.csproj は ClickOnce のマニフェスト署名が有効で、
証明書の拇印を直接指定している。 その証明書が無い環境では必ず失敗する
（BUILDING.md 4 節に記載済み）。

1_BuildAll.ps1 を直接呼ぶ場合は、これを除外して回せる。

.\1_BuildAll.ps1 -IgnoreErrors 'error MSB(3482|3325|3321):.*WSClientWinCone_sample\.csproj'

0_RunAll.ps1 にはこの口が無いため、同じことができない。

何が困るか

• 「NG が 1 本ある」が常態になり、本物の失敗と見分けが付かない。
  結局ログを開いて中身を読むことになる
• 現状の 0_RunAll.ps1 は、この状況をコメントで説明しているだけである

# 1_BuildAll.ps1 は既知の署名エラー（MSB3482）で 1 になることがある。
# 終了コードだけで判断せず、エラー一覧の内容を確認すること（RELEASE.md 3 節）。

説明で回避するのではなく、判定できるようにしたい。

対応

0_RunAll.ps1 に -IgnoreErrors を足し、既定でこの既知エラーを渡す。

param(
    [ValidateSet("CS", "VB", "Both")]
    [string]$Lang = "CS",
    [string[]]$IgnoreErrors = @('error MSB(3482|3325|3321):.*WSClientWinCone_sample\.csproj')
)

• 1_BuildAll.ps1 にだけ渡す（2 と 3 は受け取らない）
• 既定で渡す。 通しで回す人がいちいち指定するものではない
• 明示的に -IgnoreErrors @() を渡せば、除外なしにもできる

除外したものは 1_BuildAll.ps1 が別枠で件数と内容を出すので、
黙って消えるわけではない。

======== 既知として除外したエラー ========
  除外条件 : error MSB(3482|3325|3321):.*WSClientWinCone_sample\.csproj
  [WSClnt_sample (net48)] ... error MSB3482: ...

併せて直すもの

• ヘッダのコメント（「終了コードだけで判断せず…」）を、現状に合わせて書き直す
• -IgnoreErrors の説明を BUILDING.md /
  CHEATSHEET.md の記述と食い違わないようにする

影響

• 利用者への影響: 無し（root/programs の検証スクリプトのみ）
• net48 / net10.0: 両方（1_BuildAll.ps1 の全体に効く）
• 既定の挙動は変わる。 これまで NG だったものが OK になる。
  ただし除外した内容は表示され続けるので、見落としにはならない
• 検証: .\0_RunAll.ps1 -Lang Both（終了コードがそのまま合否として読めること）

補足

既定で除外してよいのか、という点は一考した。

「環境に依存して必ず出るエラー」を既定で除外するのは、
本来直すべきものを隠すことにもなり得る。
ただし本件は csproj が証明書の拇印を直接持っている構造上の事情で、
証明書を配布しない限り、どの環境でも必ず失敗する。
BUILDING.md 4 節でも既知として扱っている。

「必ず失敗すると分かっているものを、毎回 NG として数える」方が実害が大きいと考える。

[OsscJpDevInfra]

直しました。終了コードをそのまま合否として読めるようになりました

修正前後

同じコマンド（.\0_RunAll.ps1 -Lang CS）の結果。

	修正前	修正後
1_BuildAll.ps1	1（既知の MSB3482 だけで NG）	0
2_RunAllTests.ps1	0	0
3_SmokeTest.ps1	0	0
まとめ	「1 本が 0 以外で終了しました」	「すべて OK です」

除外は黙って消えていない。 1_BuildAll.ps1 が件数と内容を別枠で出し続ける。

======== 既知として除外したエラー ========
  除外条件 : error MSB(3482|3325|3321):.*WSClientWinCone_sample\.csproj
  [WSClnt_sample (net48)] ... error MSB3482: 署名中にエラーが発生しました:
    WSClientWinCone_sample.exe の署名に失敗しました。
    SignTool Error: No certificates were found that met all the given criteria.

  全ステップ OK

変更内容

0_RunAll.ps1

param(
    [ValidateSet("CS", "VB", "Both")]
    [string]$Lang = "CS",
    [string[]]$IgnoreErrors = @('error MSB(3482|3325|3321):.*WSClientWinCone_sample\.csproj')
)

渡す先を明示的に絞った。 Lang と同じ形で、フラグを持たせている。

$scripts = @(
    @{ Name = "1_BuildAll.ps1";    UseLang = $true;  UseIgnore = $true }
    @{ Name = "2_RunAllTests.ps1"; UseLang = $false; UseIgnore = $false }
    @{ Name = "3_SmokeTest.ps1";   UseLang = $true;  UseIgnore = $false }
)
...
if ($s.UseIgnore) { $splat.IgnoreErrors = $IgnoreErrors }

2 と 3 は IgnoreErrors という引数を持たないため、渡すと落ちる。

末尾のコメントを書き直した

これまでは「終了コードだけで判断するな」と書いてあった。
判定できるようにしたので、逆の内容になる。

# **終了コードをそのまま合否として読んでよい。**（#555）
#   既知の署名エラー（MSB3482）は -IgnoreErrors の既定値で除外しているため、
#   1_BuildAll.ps1 が 1 を返したら、それは**別の理由**である。
#   除外した内容は 1_BuildAll.ps1 のサマリに別枠で出るので、そちらも目は通すこと。

CHEATSHEET.md

古くなる記述があったので直した。

- **`1_BuildAll.ps1` を単体で回すときは `-IgnoreErrors` を付ける。**
  ClickOnce の署名（`MSB3482`）で、証明書が無い環境では必ず出る（BUILDING.md 4 節）。
  **`0_RunAll.ps1` は既定でこれを渡すので、付けなくてよい**（#555）。
  除外した内容は件数つきで別枠に出るため、そちらは目を通すこと

検証

	結果
.\0_RunAll.ps1 -Lang CS	すべて OK / ExitCode=0（3 本とも 0）
.\0_RunAll.ps1 -Lang VB	すべて OK / ExitCode=0
PowerShell 5.1 / 7 の構文	両方 OK

-Lang VB は「既定値を足したことで壊れていないこと」の確認である。
VB には ClickOnce 署名のプロジェクトが無いため、除外の効果はそもそも出ない。

引数の受け渡しは、実行前に静的にも確かめた。

1_BuildAll.ps1 の引数 : Lang, Only, SkipClean, OutputDir, IgnoreErrors
IgnoreErrors を持つ   : True

判断したこと

「既定で除外する」ことの是非は、起票時に書いたとおり一考した。

環境依存のエラーを既定で除外するのは、本来直すべきものを隠すことにもなり得る。
ただし本件は csproj が証明書の拇印を直接持っている構造上の事情で、
証明書を配布しない限り、どの環境でも必ず失敗する。

「必ず失敗すると分かっているものを、毎回 NG として数える」方が実害が大きい。
実際、この状態では本物の失敗と見分けが付かず、毎回ログを開いて中身を読むことになっていた。

除外を無効にもできる。

.\0_RunAll.ps1 -IgnoreErrors @()      # 除外なし（従来と同じ挙動）

影響

• 利用者への影響: 無し（root/programs の検証スクリプトのみ）
• 既定の挙動は変わる。 これまで NG だったものが OK になる。
  ただし除外した内容は表示され続けるので、見落としにはならない