Repository navigation
chore(mcp): remove get_doc_content diagnostic logging - #101
Merged
Merged
Conversation
PR #99 で追加した 401 調査用の診断ログを撤去する。issue #98 の真因 (User-to-server token expiration による stale token) は確定し GitHub App 側で Opt-out 済みのため、診断ログは production ログのノイズとなり役目を終えた。 - `describeToken` helper 関数とその doc comment を削除 - `githubHeaders` / `getGitHubToken` / `get_doc_content` entry の `console.log` を削除 - 401 ブランチの `console.error` を PR #99 以前の単一行形式へ復元 (`diagHeaders` 構築と token/headers/body 多行テンプレートを除去) 既存の成功系の挙動は一切変更なし。`npx tsc --noEmit` clean。 再発時は commit 履歴から同じ diff を復活させられる。 Closes #100
Deploying with
|
| Status | Name | Latest Commit | Updated (UTC) |
|---|---|---|---|
| ✅ Deployment successful! View logs |
github-rag-mcp | e75ff41 | Apr 23 2026, 10:43 AM |
liplus-lin-lay
commented
Apr 23, 2026
liplus-lin-lay
left a comment
Member
Author
There was a problem hiding this comment.
AI セルフレビュー (auto mode / patch)
逆 diff 完全性の literal 確認
git diff aab9ad2..HEAD -- src/mcp.ts が空出力 (= PR #99 親コミットと src/mcp.ts が完全一致)。診断ログ追加前の状態に正確に戻ったことを確認。
Now (本 PR で修正すべき) — なし
Later (別 issue 候補)
- フォローアップ A:
wrangler.tomlにobservability.logs.enabled = trueを反映 (ダッシュボードで有効化済み、ソース未同期)
本 PR スコープ外、別 issue で扱う。
Accepted
- Cloudflare Workers Observability はダッシュボードで有効のまま — 診断ログ撤去後も base の access log は取得される (fetch event 等)。追加の
console.logが無いだけで運用上問題なし
検証項目
| 項目 | 結果 |
|---|---|
describeToken helper 削除 |
✅ |
githubHeaders の console.log 削除 |
✅ |
getGitHubToken の console.log 削除 |
✅ |
get_doc_content entry log 削除 |
✅ |
401 console.error が pre-PR-#99 単一行形式に復元 |
✅ |
git diff aab9ad2..HEAD -- src/mcp.ts が空 (pre-#99 と完全一致) |
✅ |
| 成功系挙動変更なし | ✅ |
| tsc --noEmit clean | ✅ (CI test pass) |
| PR title ASCII 英語 | ✅ chore(mcp): remove get_doc_content diagnostic logging |
| PR body 日本語 | ✅ |
| commit body 日本語文あり | ✅ |
Closes #100 で auto-close |
✅ |
| CI 全通過 (test / CI / Workers Builds) | ✅ |
| mergeable | ✅ CLEAN / MERGEABLE |
| patch 範囲と一致 | ✅ user/system observable behavior 変化なし |
判定
Approve — 診断ログが役目を終えた後の綺麗な撤去。逆 diff が pre-#99 と完全一致しており、意図しない副次変更なし。auto mode / patch のため AI merge で進める。
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.
概要
PR #99 で追加した
get_doc_content401 調査用の診断ログ一式を撤去する。背景
issue #98 の真因は GitHub App の User-to-server token expiration による stale token で、GitHub App 側の Opt-out で解決済み。診断ログが切り分けるべきだった 3 仮説 (scope / props propagation / stale token) は確定済みのため、production ログのノイズとなる診断出力は役目を終えた。
変更内容
src/mcp.ts:describeToken(token)helper 関数 (および doc comment) を削除githubHeaders内の token shape ログ (console.log) を削除getGitHubToken内の props / token state ログ (console.log) を削除get_doc_contenthandler 冒頭の entry ログ (console.log) を削除console.errorを PR diag(mcp): add get_doc_content auth-path logging (patch) #99 以前の単一行形式に復元 (diagHeaders構築と token/headers/body 多行テンプレートを除去)影響範囲
npx tsc --noEmitclean を維持参考
Closes #100