fix(web): API key settings follow-ups (error handling, CSRF origin check, empty scopes label)
メンテナーはふだん 1 日以内に返信
@mrgoonie がすでに取り組んでいます。
2026年10月14日 から。
評価
この issue はまだ評価されていません。
説明
Follow-up from the independent review of #77 (scoped publishing keys in Settings). All low severity; none block.
apps/web/app/routes/api.user-api-keys.tsDELETE with a malformed JSON body returns 500 instead of 400 (POST got the guard).apps/web/app/routes/settings.tsxthe create dialog closes before the server responds; on 400/403 the user loses the name and selected publishers.apps/web/app/components/settings/api-keys-table.tsxa key whose stored scopes parse to[]shows an empty Access cell instead of "No access".api.user-api-keys.tsscopes: nullreturns 400 while docs only describe omitting the field (fails safe; document or accept null as omitted).- The route has no Origin/CSRF check and relies on the session cookie's default
SameSite=Lax; a sibling subdomain could send cookie-bearing requests. Pre-existing; consider an Origin check on mutating auth routes. settings.tsxafter Done the plaintext key stays in the in-memory action result until the next navigation (not re-shown).- Separate: the
api.skill-favoriteclient chunk contains thekey_hashcolumn name (no data); check that no server-only module is imported client-side there.
- 主要言語
- TypeScript
- スター
- 189
- フォーク
- 40
- 平均マージ
- 7時間 38分
- マージ済み PR（30日）
- 30
環境構築
このプロジェクトには開発コンテナ、Dockerfile、コントリビューションガイドがありません。まず README を読み、一般的な手順ははじめてのコントリビューションガイドを参照してください。
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
nextlevelbuilder/skillx のほかの issue
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
nextlevelbuilder/skillx#49 ·
メンテナーはふだん 1 日以内に返信
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
nextlevelbuilder/skillx#28 ·
メンテナーはふだん 1 日以内に返信
-
refactor
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
nextlevelbuilder/skillx#27 ·
メンテナーはふだん 1 日以内に返信
-
Personal-kit API gaps for AgentKit kit publish/sync対応中かも @mrgoonie が 1 日前に担当しました。 オープン
nextlevelbuilder/skillx#88 · コメント 1 件 · 担当者 1 名 ·
メンテナーはふだん 1 日以内に返信
-
enhancement
難易度 5/5 1週間以上 初心者へのやさしさ 12/100
nextlevelbuilder/skillx#81 ·
メンテナーはふだん 1 日以内に返信
nextlevelbuilder/skillx の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
callstackincubator/simlock#639 ·
メンテナーはふだん 1 日以内に返信
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 80/100
aws/agentcore-cli#2606 ·
メンテナーはふだん 1 日以内に返信
-
ai-driven-qa bug Cosmetic
難易度 2/5 1〜3時間 初心者へのやさしさ 74/100
メンテナーはふだん 1 日以内に返信
-
ai-driven-qa bug claude Minor
難易度 2/5 1〜3時間 初心者へのやさしさ 64/100
linagora/twake-mail-admin#301 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
メンテナーはふだん 1 日以内に返信