Skip to content

chore(mcp): remove get_doc_content diagnostic logging #100

Description

@liplus-lin-lay

目的

#98 の真因確定後、診断ログ (PR #99 で追加) は役目を終えた。production ログのノイズとなるため撤去する。

前提

  • 真因は User-to-server token expiration 有効化による stale token 問題と確定済 (GitHub App 側で Opt-out 済)
  • 診断ログが検出すべきだった 3 仮説 (scope / props / stale) はすべて切り分け完了
  • 今後同種の調査が再発する可能性は低いが、再発時は同じ diff を復活させられるように commit 履歴として残る

制約

  • 追加した診断コードを元の状態に戻すこと (PR diag(mcp): add get_doc_content auth-path logging (patch) #99 以前の src/mcp.ts 挙動へ)
  • 401 error のログ出力は残すが、新規追加した token=${describeToken(...)} や headers={...} 等の詳細は除去。元の get_doc_content: GitHub API returned ${response.status} for ${repo}/${path}: ${body} 形式に戻す
  • describeToken helper 関数を削除
  • githubHeaders / getGitHubToken / get_doc_content entry の console.log 呼び出しを削除
  • tsc clean を維持
  • 既存成功系の挙動は一切変えない

Target files

範囲

patch (診断ログ撤去のみ、user/system observable behavior 変化なし)

関連

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Labels

enhancement新機能・改善要望ready本文が実装開始できる形まで収束している状態。ただし更新は継続可能

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions