サンプルに、使っていない参照と陳腐化した参照が残っている

[OsscJpDevInfra]

概要

サンプルの設定に、実態と合っていない参照が残っている。

#553（CS / VB の設定同期）で突き合わせをした際に見つかった。
同期とは別軸の話なので、切り出す。

2 種類ある。

	内容
① 使っていない参照	パッケージも <Reference> も在るが、コードが使っていない
② 陳腐化した参照	bindingRedirect が、実際に配布されるアセンブリと違う版を指している

現状

① 使っていない参照 — Microsoft.AspNet.FriendlyUrls

WebForms_Sample の FriendlyUrls は、CS / VB とも使用箇所が全部コメントアウトされている。

// CS : App_Start/RouteConfig.cs
//using Microsoft.AspNet.FriendlyUrls;
//routes.EnableFriendlyUrls(settings);

' VB : App_Start/RouteConfig.vb
'Imports Microsoft.AspNet.FriendlyUrls
'routes.EnableFriendlyUrls(settings)

にもかかわらず、パッケージと参照は残っている。

<!-- packages.config -->
<package id="Microsoft.AspNet.FriendlyUrls.Core" version="1.0.2" targetFramework="net48" />
<package id="Microsoft.AspNet.FriendlyUrls.Core.ja" version="1.0.2" targetFramework="net48" />

<!-- WebForms_Sample.csproj / .vbproj -->
<Reference Include="Microsoft.AspNet.FriendlyUrls">
  <HintPath>..\packages\Microsoft.AspNet.FriendlyUrls.Core.1.0.2\lib\net45\Microsoft.AspNet.FriendlyUrls.dll</HintPath>
</Reference>

CS / VB は完全に一致している。 ずれてはいないので #553 の対象ではない。

#553 では、これと似て非なるものを 2 件消してある。
Microsoft.AspNet.WebApi（MVC_Sample）と Microsoft.AspNet.FriendlyUrls（WebForms_Sample）の
メタ パッケージで、こちらは <Reference> がどこからも指していなかった
（実体は .Core / .Client / .WebHost 側）。純粋に余計な行だったため、その場で消した。

本件の .Core / .Core.ja は、<Reference> が実際に指している点が違う。
消すなら csproj / vbproj にも手を入れる（CS / VB の 4 ファイル）。

② 陳腐化した参照 — Newtonsoft.Json の bindingRedirect

CS/Frameworks/Tests/TestBatch/SimpleBatch/app.config

<bindingRedirect oldVersion="0.0.0.0-12.0.0.0" newVersion="12.0.0.0"/>

実際に配布されるアセンブリは 13.0.0.0（実測）。
12.0.0.0 は存在しないので、この範囲に入る要求は解決に失敗する。

VB 側の同種のずれ（11.0.0.0 → 実体 13.0.0.0）は #553 で直した
（WSClientWin_sample / WSClientWin2_sample / WSClientWPF_sample）。
CS 側のこれは同期の話ではないため、手を付けていない。

検討したいこと

① をどうするか

コメントアウトの意図が判断できない。

• 例として残しているなら、参照も残すのが筋。ただし「なぜ残すのか」を書いておかないと、
  次に見た人が同じ判断に迷う
• 単なる残骸なら、パッケージと <Reference> ごと消す

案
A: 消す	packages.config ＋ csproj / vbproj の 4 ファイル。ビルドで確認
B: 残して、理由を書く	RouteConfig に「機能の例として残している」旨を 1 行
C: 有効にする	拡張子なし URL を実際に使う。サンプルの挙動が変わるので別の判断が要る

② をどうするか

実体に合わせて 13.0.0.0 にするのが素直。
ただし**bindingRedirect は NuGet の生成物**なので、手で直すとパッケージ操作で戻る。
Frameworks/ANALYSIS.md の
「依存パッケージのバージョン ドリフト」と同じ枠で扱うのが妥当と考える。

他にも無いか

今回は目に付いた 2 件だけを挙げている。全体は洗っていない。
洗うなら、次のような観点になる。

• packages.config に在って、csproj の <Reference> から参照されていないパッケージ
• <Reference> に在って、コードが使っていないアセンブリ
• bindingRedirect の版と、実際に配布されるアセンブリの版の食い違い

3 つ目は機械的に確かめられる（AssemblyName.GetAssemblyName() で実体の版を取れる）。

影響

• 利用者への影響: 無し（サンプルの参照整理のみ）。
  ただし ① で C（有効にする） を選ぶ場合は、WebForms_Sample の URL が変わるため影響が出る
• net48 / net10.0: net48 のみ（packages.config は net48 の仕組み）
• 検証: 1_BuildAll.ps1 -Lang Both ＋ 3_SmokeTest.ps1 -Lang Both

関連

#553（CS / VB の設定同期。ここで見つけた）

[OsscJpDevInfra]

① を実施しました。② は #556 へ切り出しました

起票時に挙げた 2 種類のうち、

	対応
① 使っていない参照（FriendlyUrls）	消した（本コメント）
② 陳腐化した参照（bindingRedirect）	#556 へ切り出した（全体を洗ったら 8 件あったため）

① FriendlyUrls を参照ごと外した

CS / VB の 6 ファイル。

ファイル	差分
CS/…/WebForms_Sample/packages.config	−2（.Core / .Core.ja）
CS/…/WebForms_Sample.csproj	−3（<Reference> ブロック）
VB/…/WebForms_Sample/packages.config	−2
VB/…/WebForms_Sample.vbproj	−3
CS/…/App_Start/RouteConfig.cs	+3 / −4
VB/…/App_Start/RouteConfig.vb	+3 / −5

RouteConfig も直した（判断が要った箇所）

パッケージだけ消して、コメントアウトされた使用例を残すと罠になる。
外した人が // を消すとビルド エラーになり、理由がどこにも書かれていない。

コメントアウトを消し、なぜ空なのかを残した。

public static void RegisterRoutes(RouteCollection routes)
{
    // ルート定義はここに書く。
    // FriendlyUrls（拡張子なし URL）の例は、参照ごと外した（#554）。
    // 使っていないパッケージを残すと、参照だけが増えていくため。
}

RegisterRoutes は Global.asax から呼ばれ続けるので、空メソッドとして残すのが正である
（ASP.NET の定型の形）。

踏んだこと — RouteConfig.cs だけ Shift-JIS だった

CS/…/App_Start/RouteConfig.cs   cp932 / BOM 無し   ← これだけ
VB/…/App_Start/RouteConfig.vb   utf-8 / BOM あり
packages.config ×2              utf-8 / BOM あり
csproj / vbproj                 utf-8 / BOM あり

UTF-8 で書き戻すと文字化けする。 ファイルごとに文字コードを判定して保つ形にした。
書き戻し後も HEAD と同じ cp932 / BOM 無しであることを確認済み。

本件とは無関係な既存の不統一である。
CODING.md は .bat / .ps1 の文字コードを定めているが、.cs については明示が無い。
気になるようなら別途。

検証

.\0_RunAll.ps1 -Lang Both : すべて OK / ExitCode=0

1_BuildAll.ps1	0
2_RunAllTests.ps1	0
3_SmokeTest.ps1	0

#555 で -IgnoreErrors を既定にしたので、この 1 本で合否が読める。

参照を外してもビルドが通ることが、「使っていない」の裏付けになっている。

② を切り出した理由 → #556

起票時は TestBatch/SimpleBatch の 1 件だけを挙げていたが、
全体を洗ったら 8 件あった（bindingRedirect 847 件中）。

	件数
一致	832
不一致	8
実体なし	7

Newtonsoft.Json は 10 / 11 / 12 とバラバラで、単体テストの 3 プロジェクトすべてが該当。
Google.Protobuf の乖離が最大（3.6.1.0 → 3.30.x）。

別 Issue にした理由は 2 つ。

1. bindingRedirect は NuGet の生成物で、パッケージ操作でまた書き換わる。
   「直す」より「ずれを検知する仕組み」の方が本質であり、
   本 Issue（未使用の参照）とは対処の性質が違う
2. 本 Issue の当初の主題は未使用の参照で、そちらは片付いた。
   8 件に広がった別種の問題を同じ Issue に抱えると、何が終わったのか分からなくなる

本 Issue で残っていること

無し。 ① は実施し、② は #556 へ移した。

起票時に挙げた「他にも無いか」の 3 観点のうち、
3 つ目（bindingRedirect と実体の食い違い）は #556 で機械的に洗った。
残る 2 つ（packages.config に在って <Reference> から参照されないパッケージ／
<Reference> に在ってコードが使っていないアセンブリ）は洗っていない。
必要なら別途。