Skip to content

fix(admin): reuse existing GitHub token for sparse operations - #287

Merged
lipluscodex merged 1 commit into
mainfrom
luna-286-existing-admin-auth
Oct 13, 2026
Merged

lipluscodex merged 1 commit into
mainfrom
luna-286-existing-admin-auth

Conversation

@lipluscodex

Copy link
Copy Markdown
Member

Codex Luna

Closes #286
全文索引の2管理経路を既存 Worker secret と GITHUB_TOKEN ヘッダの完全一致に変更し、未設定・空は401にします。
専用token型と設定を除去し、要求・README・運用文書を同期しました。両経路の認証拒否と取得側Bearer維持をテストします。

検証: sparse Workers tests 28/28、tsc --noEmit、git diff --check 成功。
実装モデルは親モデル継承、reasoning_effort=low。変更が認証比較と対応文書・テストに限定され、要求に対応するテストと親レビュー・CIで検証するため、この範囲に十分と判断しました。
本番確認は #284 のマージ後CD・401/plan記録で行います。

— Luna

全文索引の管理認証を既存 GITHUB_TOKEN ヘッダへ統一し、文書と認証テストを同期する。
@lipluscodex lipluscodex self-assigned this Oct 13, 2026
@lipluscodex lipluscodex linked an issue Oct 13, 2026 that may be closed by this pull request

@lipluscodex lipluscodex left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Codex Luna

自己レビュー: 068391977c24126edc2921ebbb5f9b4cd4e086b9 は #286 の指定を満たしています。

全文管理2経路を、既存 env.GITHUB_TOKEN と GITHUB_TOKEN ヘッダの完全一致に変更し、空/未設定/誤値/旧専用鍵/Bearer は拒否します。正しいヘッダで plan/stats/apply に到達し、GitHub source の取得認証は従来の Bearer GITHUB_TOKEN を保つテストを確認しました。専用secretの設定・型・文書を除去し、日英要求/README/運用文書が同期されています。

source owner/revision CAS、通常検索、既定off、予算、既存管理経路は変更していません。鍵の値や既存secretを変更する作業もありません。scope逸脱・workspace側延期作業はありません。認証テスト28件/tscのローカル報告と、最新headの test/CI/Workers Builds 全成功、mergeStateStatus CLEAN を確認し、autoモードでマージします。

マージ後のmain CDを確認し、実際の認証済みplanは #284 で別記します。手元の既存鍵の入力は未実施なので、本番認証成功・全文準備完了とは扱いません。

— Luna

@lipluscodex
lipluscodex merged commit 0e4246b into main Oct 13, 2026
3 checks passed
@lipluscodex
lipluscodex deleted the luna-286-existing-admin-auth branch October 13, 2026 14:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix(admin): reuse existing GitHub token for sparse operations

1 participant